instruktai-python-logger 0.4.2__tar.gz → 0.4.4__tar.gz

{instruktai_python_logger-0.4.2 → instruktai_python_logger-0.4.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: instruktai-python-logger
-Version: 0.4.2
+Version: 0.4.4
 Summary: Centralized logging utilities for Python services.
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown

{instruktai_python_logger-0.4.2 → instruktai_python_logger-0.4.4}/instrukt_ai_logging/logging.py

@@ -564,6 +564,12 @@ def iter_recent_log_lines_merged(files: Iterable[Path], since: timedelta) -> Ite
     entries stay adjacent to their parent). The resulting per-file streams are
     merged via a k-way heap merge, producing a single chronologically-ordered
     output stream.
+
+    Files whose mtime falls before the cutoff are skipped entirely — no line
+    inside them can be newer than the file itself, so reading them only burns
+    I/O. This matters for log directories with large rotation history or
+    sibling streams (git-wrapper.log, etc.) the caller didn't explicitly
+    request via `--logs`.
     """
     cutoff = _now_utc() - since
 
@@ -588,6 +594,16 @@ def iter_recent_log_lines_merged(files: Iterable[Path], since: timedelta) -> Ite
         finally:
             f.close()
 
-    streams = [_file_stream(p) for p in files]
+    eligible: list[Path] = []
+    for path in files:
+        try:
+            mtime = datetime.fromtimestamp(path.stat().st_mtime, tz=timezone.utc)
+        except OSError:
+            continue
+        if mtime < cutoff:
+            continue
+        eligible.append(path)
+
+    streams = [_file_stream(p) for p in eligible]
     for _, line in heapq.merge(*streams, key=lambda item: item[0]):
         yield line

{instruktai_python_logger-0.4.2 → instruktai_python_logger-0.4.4}/instruktai_python_logger.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: instruktai-python-logger
-Version: 0.4.2
+Version: 0.4.4
 Summary: Centralized logging utilities for Python services.
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown

{instruktai_python_logger-0.4.2 → instruktai_python_logger-0.4.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "instruktai-python-logger"
-version = "0.4.2"
+version = "0.4.4"
 description = "Centralized logging utilities for Python services."
 readme = "README.md"
 requires-python = ">=3.11"

{instruktai_python_logger-0.4.2 → instruktai_python_logger-0.4.4}/tests/test_log_aggregation.py

@@ -111,6 +111,32 @@ def test_iter_recent_log_lines_merged_respects_since_cutoff(app_log_dir: Path) -
     assert "fresh" in lines[0]
 
 
+def test_iter_recent_log_lines_merged_skips_files_with_mtime_before_cutoff(
+    app_log_dir: Path,
+) -> None:
+    import os
+    from datetime import datetime, timezone
+
+    fresh = app_log_dir / "demo-app.log"
+    fresh.write_text(
+        f"{_now_iso(-1)} level=INFO logger=demo.daemon msg=fresh\n",
+        encoding="utf-8",
+    )
+    stale = app_log_dir / "stale.log"
+    stale.write_text(
+        f"{_now_iso(-1)} level=INFO logger=stale msg=stale-but-line-looks-recent\n",
+        encoding="utf-8",
+    )
+    # Force stale.log's mtime to 1 day ago — older than --since=10m cutoff.
+    one_day_ago = (datetime.now(tz=timezone.utc) - timedelta(days=1)).timestamp()
+    os.utime(stale, (one_day_ago, one_day_ago))
+
+    files = resolve_log_files("demo-app")
+    lines = list(iter_recent_log_lines_merged(files, since=timedelta(minutes=10)))
+    msgs = [line.strip().rsplit("=", 1)[-1] for line in lines]
+    assert msgs == ["fresh"]
+
+
 def test_iter_recent_log_lines_merged_keeps_continuation_lines_with_parent(
     app_log_dir: Path,
 ) -> None: