feat: implement daily log rotation and /var/log/pgcli location (v4.3.7)

Improvements to logging system:
- Implement automatic daily log rotation at midnight
- Keep 30 days of log history
- Change default log location to /var/log/pgcli/pgcli.log
- Automatic fallback to ~/.config/pgcli/log if no system permissions
- Use TimedRotatingFileHandler for better log management
- Log files formatted as: pgcli.log.YYYY-MM-DD

Changes:
- pgcli/main.py: Added logging.handlers import and updated initialize_logging()
- pgcli/pgclirc: Updated documentation for new log behavior
- pgcli/__init__.py: Bumped version to 4.3.7
- changelog.rst: Added Internal section entry

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: DiegoDAF

changelog.rst

@@ -45,6 +45,8 @@ Internal:
   * Use github trusted publisher for pypi release
   * Update dev requirements and replace requirements-dev.txt with pyproject.toml
   * Use ruff instead of black
+* Implement daily log rotation at midnight with 30-day retention using TimedRotatingFileHandler
+* Change default log location to /var/log/pgcli/pgcli.log (with automatic fallback to ~/.config/pgcli/log if no permissions)
 
 Bug fixes:
 ----------

pgcli/__init__.py

@@ -1 +1 @@
-__version__ = "4.3.6"
+__version__ = "4.3.7"

pgcli/main.py

@@ -9,6 +9,7 @@
 import sys
 import traceback
 import logging
+import logging.handlers
 import threading
 import shutil
 import functools
@@ -550,7 +551,18 @@ def write_to_file(self, pattern, **_):
     def initialize_logging(self):
         log_file = self.config["main"]["log_file"]
         if log_file == "default":
-            log_file = config_location() + "log"
+            # Try /var/log/pgcli first, fallback to user config dir if no permissions
+            default_system_log = "/var/log/pgcli/pgcli.log"
+            try:
+                os.makedirs(os.path.dirname(default_system_log), exist_ok=True)
+                # Test if we can write to /var/log/pgcli
+                with open(default_system_log, "a"):
+                    pass
+                log_file = default_system_log
+            except (OSError, PermissionError):
+                # Fallback to user's config directory if no system permissions
+                log_file = config_location() + "log"
+
         ensure_dir_exists(log_file)
         log_level = self.config["main"]["log_level"]
 
@@ -559,7 +571,18 @@ def initialize_logging(self):
         if log_level.upper() == "NONE":
             handler = logging.NullHandler()
         else:
-            handler = logging.FileHandler(os.path.expanduser(log_file))
+            # Use TimedRotatingFileHandler for daily log rotation
+            # Rotates at midnight, keeps 30 days of logs
+            expanded_log_file = os.path.expanduser(log_file)
+            handler = logging.handlers.TimedRotatingFileHandler(
+                expanded_log_file,
+                when='midnight',
+                interval=1,
+                backupCount=30,
+                encoding='utf-8'
+            )
+            # Format: pgcli.log.2025-12-02
+            handler.suffix = "%Y-%m-%d"
 
         level_map = {
             "CRITICAL": logging.CRITICAL,

pgcli/pgclirc

@@ -66,9 +66,12 @@ generate_aliases = False
 alias_map_file =
 
 # log_file location.
-# In Unix/Linux: ~/.config/pgcli/log
-# In Windows: %USERPROFILE%\AppData\Local\dbcli\pgcli\log
-# %USERPROFILE% is typically C:\Users\{username}
+# Default behavior:
+#   - Tries /var/log/pgcli/pgcli.log first (if running with permissions)
+#   - Falls back to ~/.config/pgcli/log (Unix/Linux) or %USERPROFILE%\AppData\Local\dbcli\pgcli\log (Windows)
+# Log rotation: Daily at midnight, keeps 30 days of logs
+# Format: pgcli.log.YYYY-MM-DD
+# You can specify a custom path here if needed
 log_file = default
 
 # keyword casing preference. Possible values: "lower", "upper", "auto"