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

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

@@ -1,12 +1,9 @@
 Metadata-Version: 2.4
 Name: instruktai-python-logger
-Version: 0.4.4
+Version: 0.5.0
 Summary: Centralized logging utilities for Python services.
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown
-Provides-Extra: dev
-Requires-Dist: pytest>=8; extra == "dev"
-Requires-Dist: ruff>=0.8; extra == "dev"
 
 # InstruktAI Python Logger
 
@@ -22,7 +19,7 @@ Publishing notes live in `docs/publishing.md`.
 From PyPI (recommended):
 
 ```bash
-pip install instrukt-ai-logger
+pip install instruktai-python-logger
 ```
 
 From GitHub:

{instruktai_python_logger-0.4.4 → instruktai_python_logger-0.5.0}/README.md

@@ -12,7 +12,7 @@ Publishing notes live in `docs/publishing.md`.
 From PyPI (recommended):
 
 ```bash
-pip install instrukt-ai-logger
+pip install instruktai-python-logger
 ```
 
 From GitHub:

{instruktai_python_logger-0.4.4 → instruktai_python_logger-0.5.0}/instrukt_ai_logging/__init__.py

@@ -4,30 +4,31 @@ See `README.md` for usage and `docs/design.md` for design intent.
 """
 
 __all__ = [
+    "TRACE",
+    "InstruktAILogger",
+    "InstruktAILoggerProtocol",
     "__version__",
     "configure_logging",
     "get_logger",
-    "InstruktAILogger",
-    "InstruktAILoggerProtocol",
     "resolve_log_file",
     "resolve_log_files",
-    "TRACE",
 ]
 
 try:
-    from importlib.metadata import packages_distributions, version as _pkg_version
+    from importlib.metadata import packages_distributions
+    from importlib.metadata import version as _pkg_version
 
     _dists = packages_distributions().get(__name__, [])
     __version__ = _pkg_version(_dists[0]) if _dists else "0.0.0"
 except Exception:  # pragma: no cover
     __version__ = "0.0.0"
 
-from instrukt_ai_logging.logging import (  # noqa: E402  (intentional re-export)
-    configure_logging,
-    get_logger,
+from instrukt_ai_logging.logging import (
+    TRACE,
     InstruktAILogger,
     InstruktAILoggerProtocol,
+    configure_logging,
+    get_logger,
     resolve_log_file,
     resolve_log_files,
-    TRACE,
 )

{instruktai_python_logger-0.4.4 → instruktai_python_logger-0.5.0}/instrukt_ai_logging/cli.py

@@ -6,8 +6,8 @@ import re
 import sys
 import threading
 import time
+from collections.abc import Iterator
 from pathlib import Path
-from typing import Iterator
 
 from instrukt_ai_logging.logging import (
     iter_recent_log_lines_merged,
@@ -102,6 +102,15 @@ def _parse_stems(value: str) -> list[str]:
     return [item.strip() for item in value.split(",") if item.strip()]
 
 
+def _line_passes(line: str, *, keep: re.Pattern[str] | None, drop: re.Pattern[str] | None) -> bool:
+    """Apply the inclusive `--grep` keep filter then the `--exclude` drop filter."""
+    if keep is not None and not keep.search(line):
+        return False
+    if drop is not None and drop.search(line):
+        return False
+    return True
+
+
 def _stem_of(path: Path) -> str:
     """Return the stem before the first `.log` suffix (e.g. `cron.log.1` → `cron`)."""
     name = path.name
@@ -116,6 +125,64 @@ def _is_rotation_suffix(path: Path) -> bool:
     return idx > 0 and name[idx + len(".log") :] != ""
 
 
+def _follow_files(
+    files: list[Path],
+    stems: list[str] | None,
+    *,
+    keep: re.Pattern[str] | None,
+    drop: re.Pattern[str] | None,
+) -> None:
+    """Follow each selected file concurrently, applying the keep/drop filters.
+
+    New lines from any file are written to stdout as they arrive — no cross-file
+    timestamp merge in follow mode, since real-time arrival ordering is its own
+    merge. For the explicit `--logs`/`--include` case, follow each named stem's
+    canonical `<stem>.log` so we don't follow rotated history.
+    """
+    follow_files: list[Path]
+    if stems is None:
+        follow_files = [p for p in files if not _is_rotation_suffix(p)]
+    else:
+        follow_files = []
+        for stem in stems:
+            for p in files:
+                if p.name == f"{stem}.log":
+                    follow_files.append(p)
+                    break
+
+    if not follow_files:
+        return
+
+    stop_event = threading.Event()
+
+    def _follow_one(path: Path) -> None:
+        try:
+            for line in iter_follow_lines(path, start_at_end=True):
+                if stop_event.is_set():
+                    return
+                if not _line_passes(line, keep=keep, drop=drop):
+                    continue
+                sys.stdout.write(line)
+                sys.stdout.flush()
+        except OSError:
+            return
+
+    threads = [
+        threading.Thread(target=_follow_one, args=(path,), daemon=True, name=f"follow-{path.name}")
+        for path in follow_files
+    ]
+    for t in threads:
+        t.start()
+
+    try:
+        while any(t.is_alive() for t in threads):
+            for t in threads:
+                t.join(timeout=0.25)
+    except KeyboardInterrupt:
+        stop_event.set()
+        return
+
+
 def main() -> None:
     from instrukt_ai_logging import __version__
 
@@ -126,22 +193,26 @@ def main() -> None:
         version=f"%(prog)s {__version__}",
     )
     parser.add_argument("app", help="App/service name (folder under /var/log/instrukt-ai/)")
-    parser.add_argument(
-        "--since", default="10m", help="Time window (e.g. 10m, 2h, 1d). Default: 10m"
-    )
+    parser.add_argument("--since", default="10m", help="Time window (e.g. 10m, 2h, 1d). Default: 10m")
     parser.add_argument(
         "-f",
         "--follow",
         action="store_true",
         help="Follow the log (tail -f style) after printing the --since window",
     )
-    parser.add_argument("--grep", default="", help="Regex to filter lines (optional)")
+    parser.add_argument("--grep", default="", help="Regex to keep only matching lines (optional)")
+    parser.add_argument(
+        "--exclude",
+        default="",
+        help="Regex to drop matching lines (applied after --grep; optional)",
+    )
     parser.add_argument(
         "--logs",
+        "--include",
         default="",
         help=(
             "Comma-separated list of stems to include (e.g. 'cron,docs-watch'). "
-            "Default: all log files in the app's directory."
+            "Default: all log files in the app's directory. Alias: --include."
         ),
     )
     args = parser.parse_args()
@@ -152,6 +223,7 @@ def main() -> None:
         raise SystemExit(str(e)) from e
 
     pattern = re.compile(args.grep) if args.grep else None
+    exclude_pattern = re.compile(args.exclude) if args.exclude else None
     stems = _parse_stems(args.logs) or None
 
     files = resolve_log_files(args.app, stems=stems)
@@ -159,66 +231,15 @@ def main() -> None:
         if stems:
             available = sorted({_stem_of(p) for p in resolve_log_files(args.app)})
             available_str = ", ".join(available) if available else "(none)"
-            raise SystemExit(
-                f"No log files matched stems {stems} in app '{args.app}'. "
-                f"Available: {available_str}"
-            )
+            raise SystemExit(f"No log files matched stems {stems} in app '{args.app}'. Available: {available_str}")
         raise SystemExit(f"No log files found for app '{args.app}'")
 
     for line in iter_recent_log_lines_merged(files, since):
-        if pattern and not pattern.search(line):
-            continue
-        sys.stdout.write(line)
+        if _line_passes(line, keep=pattern, drop=exclude_pattern):
+            sys.stdout.write(line)
 
     if not args.follow:
         return
 
     sys.stdout.flush()
-
-    # Follow each selected file concurrently. New lines from any file are
-    # written to stdout as they arrive — no cross-file timestamp merge in
-    # follow mode, since real-time arrival ordering is its own merge.
-    # Filter to files that actually exist for the canonical (default) glob —
-    # for the explicit `--logs=` case, follow each named stem's canonical
-    # file (the unsuffixed `<stem>.log`) so we don't follow rotated history.
-    if stems is None:
-        follow_files = [p for p in files if not _is_rotation_suffix(p)]
-    else:
-        follow_files = []
-        for stem in stems:
-            for p in files:
-                if p.name == f"{stem}.log":
-                    follow_files.append(p)
-                    break
-
-    if not follow_files:
-        return
-
-    stop_event = threading.Event()
-
-    def _follow_one(path: Path) -> None:
-        try:
-            for line in iter_follow_lines(path, start_at_end=True):
-                if stop_event.is_set():
-                    return
-                if pattern and not pattern.search(line):
-                    continue
-                sys.stdout.write(line)
-                sys.stdout.flush()
-        except OSError:
-            return
-
-    threads = [
-        threading.Thread(target=_follow_one, args=(path,), daemon=True, name=f"follow-{path.name}")
-        for path in follow_files
-    ]
-    for t in threads:
-        t.start()
-
-    try:
-        while any(t.is_alive() for t in threads):
-            for t in threads:
-                t.join(timeout=0.25)
-    except KeyboardInterrupt:
-        stop_event.set()
-        return
+    _follow_files(files, stems, keep=pattern, drop=exclude_pattern)

instruktai_python_logger-0.5.0/instrukt_ai_logging/install.py

@@ -0,0 +1,96 @@
+"""Install log rotation for InstruktAI logs."""
+
+from __future__ import annotations
+
+import argparse
+import os
+import sys
+from pathlib import Path
+
+DEFAULT_NEWSYSLOG_CONF = Path("/etc/newsyslog.d/instrukt-ai.conf")
+DEFAULT_LOGROTATE_CONF = Path("/etc/logrotate.d/instrukt-ai")
+
+
+def _write_file(path: Path, content: str, *, force: bool) -> None:
+    if path.exists() and not force:
+        raise SystemExit(f"Refusing to overwrite existing file: {path}. Use --force.")
+    path.parent.mkdir(parents=True, exist_ok=True)
+    path.write_text(content, encoding="utf-8")
+
+
+def _install_newsyslog(
+    log_root: Path,
+    *,
+    force: bool,
+    conf_path: Path = DEFAULT_NEWSYSLOG_CONF,
+) -> Path:
+    """Install newsyslog rules covering every app's *.log file.
+
+    Format: path  owner:group  mode  count  size(KB)  when  flags
+    Defaults: 50 MB rotation, 5 archives, gzip-compress archives (Z flag).
+
+    macOS newsyslog requires literal filenames — its config does NOT
+    expand globs at runtime. We therefore enumerate every existing
+    *.log file under log_root and emit one explicit line per file.
+    Re-run `instrukt-ai-log-setup --force` after a new producer creates
+    its first log file (which can be wired into the app's installer or
+    a daemon's periodic cleanup task).
+    """
+    if log_root.is_dir():
+        log_files = sorted(p for p in log_root.glob("*/*.log") if p.is_file())
+    else:
+        log_files = []
+    lines = "".join(f"{path}  640  5  50000  *  Z\n" for path in log_files)
+    if not lines:
+        # No log files yet — write a placeholder comment so the conf is
+        # discoverable on disk for future re-runs.
+        lines = f"# instrukt-ai-log-setup: no *.log files found under {log_root}\n"
+    _write_file(conf_path, lines, force=force)
+    return conf_path
+
+
+def _install_logrotate(
+    log_root: Path,
+    *,
+    force: bool,
+    conf_path: Path = DEFAULT_LOGROTATE_CONF,
+) -> Path:
+    """Install a glob-based logrotate rule covering every app's *.log file.
+
+    Defaults: 50 MB rotation, 5 archives, gzip-compress archives.
+    """
+    content = f"""{log_root}/*/*.log {{
+    size 50M
+    rotate 5
+    compress
+    delaycompress
+    missingok
+    notifempty
+    copytruncate
+}}
+"""
+    _write_file(conf_path, content, force=force)
+    return conf_path
+
+
+def main() -> None:
+    parser = argparse.ArgumentParser(prog="instrukt-ai-log-setup", add_help=True)
+    parser.add_argument(
+        "--log-root",
+        default=os.environ.get("INSTRUKT_AI_LOG_ROOT", "/var/log/instrukt-ai"),
+        help="Base log root (default: /var/log/instrukt-ai or INSTRUKT_AI_LOG_ROOT)",
+    )
+    parser.add_argument("--force", action="store_true", help="Overwrite existing config")
+    args = parser.parse_args()
+
+    log_root = Path(args.log_root)
+    if sys.platform == "darwin":
+        conf = _install_newsyslog(log_root, force=args.force)
+    else:
+        conf = _install_logrotate(log_root, force=args.force)
+
+    sys.stdout.write(f"installed: {conf}\n")
+
+
+if __name__ == "__main__":
+    main()

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

@@ -4,17 +4,18 @@ import heapq
 import logging
 import os
 import re
+from collections.abc import Iterable, Iterator
 from dataclasses import dataclass
-from datetime import datetime, timedelta, timezone
+from datetime import UTC, datetime, timedelta
 from logging.handlers import WatchedFileHandler
 from pathlib import Path
-from typing import Any, Iterable, Iterator, Protocol, cast, runtime_checkable
+from typing import Any, Protocol, cast, runtime_checkable
 
 # Standard logging levels are: NOTSET=0, DEBUG=10, INFO=20, WARNING=30, ERROR=40, CRITICAL=50
 TRACE: int = 5
 logging.addLevelName(TRACE, "TRACE")
 # Monkey-patch logging so our _level_name_to_int helper finds it automatically.
-setattr(logging, "TRACE", TRACE)
+logging.TRACE = TRACE  # type: ignore[attr-defined]
 
 
 def _normalize_env_prefix(name: str) -> str:
@@ -68,12 +69,12 @@ def _parse_csv(value: str) -> list[str]:
 
 
 def _now_utc() -> datetime:
-    return datetime.now(tz=timezone.utc)
+    return datetime.now(tz=UTC)
 
 
 class UtcMillisFormatter(logging.Formatter):
     def formatTime(self, record: logging.LogRecord, datefmt: str | None = None) -> str:
-        dt = datetime.fromtimestamp(record.created, tz=timezone.utc)
+        dt = datetime.fromtimestamp(record.created, tz=UTC)
         if datefmt:
             return dt.strftime(datefmt)
         return dt.strftime("%Y-%m-%dT%H:%M:%S") + f".{int(record.msecs):03d}Z"
@@ -161,18 +162,17 @@ class LogfmtFormatter(UtcMillisFormatter):
             f"msg={_format_logfmt_string(str(message), max_chars=self.max_message_chars, force_quote=True)}",
         ]
 
-        kv: object = getattr(record, "kv", None)
-        if isinstance(kv, dict):
-            for key in sorted(kv.keys(), key=lambda x: str(x)):
+        raw_kv: object = getattr(record, "kv", None)
+        if isinstance(raw_kv, dict):
+            kv = cast("dict[object, object]", raw_kv)
+            for key in sorted(kv.keys(), key=lambda k: str(k)):
                 if key == "msg":
                     continue
                 if not isinstance(key, str):
                     continue
                 if not _SAFE_KEY.fullmatch(key):
                     continue
-                parts.append(
-                    f"{key}={_format_logfmt_value(kv[key], max_chars=self.max_message_chars)}"
-                )
+                parts.append(f"{key}={_format_logfmt_value(kv[key], max_chars=self.max_message_chars)}")
 
         if record.exc_info:
             try:
@@ -216,9 +216,7 @@ class InstruktAILogger(logging.Logger):
     All `**kv` values are serialized to text by the formatter.
     """
 
-    def _log_with_kv(
-        self, level: int, msg: object, args: tuple[object, ...], **kwargs: Any
-    ) -> None:
+    def _log_with_kv(self, level: int, msg: object, args: tuple[object, ...], **kwargs: Any) -> None:
         exc_info = kwargs.pop("exc_info", None)
         stack_info = kwargs.pop("stack_info", False)
         stacklevel = kwargs.pop("stacklevel", 1)
@@ -229,7 +227,7 @@ class InstruktAILogger(logging.Logger):
 
         merged_extra: dict[str, object] = {}
         if isinstance(extra, dict):
-            merged_extra.update(extra)
+            merged_extra.update(cast("dict[str, object]", extra))
 
         existing_kv = merged_extra.get("kv")
         if isinstance(existing_kv, dict):
@@ -251,33 +249,33 @@ class InstruktAILogger(logging.Logger):
         if self.isEnabledFor(TRACE):
             self._log_with_kv(TRACE, msg, args, **kwargs)
 
-    def debug(self, msg: object, *args: object, **kwargs: Any) -> None:  # type: ignore[override]
+    def debug(self, msg: object, *args: object, **kwargs: Any) -> None:
         if self.isEnabledFor(logging.DEBUG):
             self._log_with_kv(logging.DEBUG, msg, args, **kwargs)
 
-    def info(self, msg: object, *args: object, **kwargs: Any) -> None:  # type: ignore[override]
+    def info(self, msg: object, *args: object, **kwargs: Any) -> None:
         if self.isEnabledFor(logging.INFO):
             self._log_with_kv(logging.INFO, msg, args, **kwargs)
 
-    def warning(self, msg: object, *args: object, **kwargs: Any) -> None:  # type: ignore[override]
+    def warning(self, msg: object, *args: object, **kwargs: Any) -> None:
         if self.isEnabledFor(logging.WARNING):
             self._log_with_kv(logging.WARNING, msg, args, **kwargs)
 
-    def error(self, msg: object, *args: object, **kwargs: Any) -> None:  # type: ignore[override]
+    def error(self, msg: object, *args: object, **kwargs: Any) -> None:
         if self.isEnabledFor(logging.ERROR):
             self._log_with_kv(logging.ERROR, msg, args, **kwargs)
 
-    def critical(self, msg: object, *args: object, **kwargs: Any) -> None:  # type: ignore[override]
+    def critical(self, msg: object, *args: object, **kwargs: Any) -> None:
         if self.isEnabledFor(logging.CRITICAL):
             self._log_with_kv(logging.CRITICAL, msg, args, **kwargs)
 
-    def exception(self, msg: object, *args: object, **kwargs: Any) -> None:  # type: ignore[override]
+    def exception(self, msg: object, *args: object, **kwargs: Any) -> None:
         kwargs.setdefault("exc_info", True)
         if self.isEnabledFor(logging.ERROR):
             self._log_with_kv(logging.ERROR, msg, args, **kwargs)
 
-    def log(self, level: int, msg: object, *args: object, **kwargs: Any) -> None:  # type: ignore[override]
-        if not isinstance(level, int):
+    def log(self, level: int, msg: object, *args: object, **kwargs: Any) -> None:
+        if not isinstance(level, int):  # pyright: ignore[reportUnnecessaryIsInstance]
             raise TypeError("level must be an int")
         if self.isEnabledFor(level):
             self._log_with_kv(level, msg, args, **kwargs)
@@ -313,7 +311,7 @@ class _ThirdPartySelectorFilter(logging.Filter):
         self.app_logger_prefix = app_logger_prefix
         self.spotlight_prefixes = spotlight_prefixes
 
-    def filter(self, record: logging.LogRecord) -> bool:  # noqa: A003
+    def filter(self, record: logging.LogRecord) -> bool:
         name = record.name
         if name == self.app_logger_prefix or name.startswith(self.app_logger_prefix + "."):
             return True
@@ -466,9 +464,7 @@ def configure_logging(
     handler.setLevel(logging.NOTSET)
     handler.setFormatter(formatter)
     handler.addFilter(
-        _ThirdPartySelectorFilter(
-            app_logger_prefix=app_logger_prefix, spotlight_prefixes=tuple(spotlight)
-        )
+        _ThirdPartySelectorFilter(app_logger_prefix=app_logger_prefix, spotlight_prefixes=tuple(spotlight))
     )
 
     # Configure root.
@@ -532,11 +528,7 @@ def iter_recent_log_lines(log_file: Path, since: timedelta) -> list[str]:
     cutoff = _now_utc() - since
 
     candidates = sorted(
-        [
-            p
-            for p in log_file.parent.glob(log_file.name + "*")
-            if p.is_file() and not p.name.endswith(".gz")
-        ],
+        [p for p in log_file.parent.glob(log_file.name + "*") if p.is_file() and not p.name.endswith(".gz")],
         key=lambda p: p.stat().st_mtime,
     )
 
@@ -597,7 +589,7 @@ def iter_recent_log_lines_merged(files: Iterable[Path], since: timedelta) -> Ite
     eligible: list[Path] = []
     for path in files:
         try:
-            mtime = datetime.fromtimestamp(path.stat().st_mtime, tz=timezone.utc)
+            mtime = datetime.fromtimestamp(path.stat().st_mtime, tz=UTC)
         except OSError:
             continue
         if mtime < cutoff:

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

@@ -1,12 +1,9 @@
 Metadata-Version: 2.4
 Name: instruktai-python-logger
-Version: 0.4.4
+Version: 0.5.0
 Summary: Centralized logging utilities for Python services.
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown
-Provides-Extra: dev
-Requires-Dist: pytest>=8; extra == "dev"
-Requires-Dist: ruff>=0.8; extra == "dev"
 
 # InstruktAI Python Logger
 
@@ -22,7 +19,7 @@ Publishing notes live in `docs/publishing.md`.
 From PyPI (recommended):
 
 ```bash
-pip install instrukt-ai-logger
+pip install instruktai-python-logger
 ```
 
 From GitHub:

{instruktai_python_logger-0.4.4 → instruktai_python_logger-0.5.0}/instruktai_python_logger.egg-info/SOURCES.txt

@@ -9,9 +9,9 @@ instruktai_python_logger.egg-info/PKG-INFO
 instruktai_python_logger.egg-info/SOURCES.txt
 instruktai_python_logger.egg-info/dependency_links.txt
 instruktai_python_logger.egg-info/entry_points.txt
-instruktai_python_logger.egg-info/requires.txt
 instruktai_python_logger.egg-info/top_level.txt
 tests/test_cli_follow.py
 tests/test_configure_logging.py
+tests/test_install.py
 tests/test_log_aggregation.py
 tests/test_trace_logging.py

instruktai_python_logger-0.5.0/pyproject.toml

@@ -0,0 +1,92 @@
+[build-system]
+requires = ["setuptools>=69", "wheel"]
+build-backend = "setuptools.build_meta"
+
+[project]
+name = "instruktai-python-logger"
+version = "0.5.0"
+description = "Centralized logging utilities for Python services."
+readme = "README.md"
+requires-python = ">=3.11"
+dependencies = []
+
+[dependency-groups]
+dev = ["pytest>=8", "ruff>=0.8", "pyright>=1.1", "mypy>=1.10"]
+
+[project.scripts]
+instrukt-ai-logs = "instrukt_ai_logging.cli:main"
+instrukt-ai-log-setup = "instrukt_ai_logging.install:main"
+
+[tool.setuptools.packages.find]
+where = ["."]
+include = ["instrukt_ai_logging*"]
+
+[tool.setuptools.package-data]
+instrukt_ai_logging = ["py.typed"]
+
+[tool.pytest.ini_options]
+addopts = "-q"
+testpaths = ["tests"]
+
+[tool.ruff]
+line-length = 120
+target-version = "py311"
+src = ["instrukt_ai_logging", "tests"]
+
+[tool.ruff.lint]
+select = ["E", "F", "I", "C90", "B", "UP", "RUF"]
+ignore = ["E501"]
+
+[tool.ruff.lint.mccabe]
+max-complexity = 20
+
+[tool.pyright]
+typeCheckingMode = "strict"
+pythonVersion = "3.11"
+include = ["instrukt_ai_logging", "tests"]
+# Flat layout: the package lives at the repo root, so add it as an import source
+# for files outside the package (e.g. tests importing `instrukt_ai_logging`).
+extraPaths = ["."]
+
+# Strict typing guards the library; the test suite is exercised by pytest, not by
+# strict static typing. Relax the strict-only "unknown type" family for tests so
+# unannotated fixtures and pytest's own surfaces do not block lint.
+[[tool.pyright.executionEnvironments]]
+root = "tests"
+reportUnknownMemberType = false
+reportUnknownParameterType = false
+reportMissingParameterType = false
+reportUnknownArgumentType = false
+reportUnknownVariableType = false
+reportUnknownLambdaType = false
+reportPrivateUsage = false
+reportCallIssue = false
+
+[tool.mypy]
+python_version = "3.11"
+strict = true
+files = ["instrukt_ai_logging"]
+# mypy is unpinned (mypy>=1.10); keep defensive `type: ignore`s that a given
+# version may not need rather than flagging them and breaking on version drift.
+warn_unused_ignores = false
+
+# Tests are exercised by pytest, not strict typing. `tests/__init__.py` makes mypy
+# name test modules `tests.*` under both full and scoped (per-file commit-gate)
+# runs, so this relaxation matches in both. `call-arg` is disabled because tests
+# call the kv-logger through stdlib `logging.getLogger`, whose base type omits **kv.
+[[tool.mypy.overrides]]
+module = "tests.*"
+disallow_untyped_defs = false
+disallow_incomplete_defs = false
+disallow_untyped_decorators = false
+check_untyped_defs = false
+ignore_missing_imports = true
+disable_error_code = ["call-arg"]
+
+[tool.guardrails]
+max_ignore_files = 8
+max_global_ignores = 8
+max_per_file_ignores = 50
+max_override_sections = 13
+max_module_lines = 1000
+required_rule_groups = ["E", "F", "I", "C90", "B", "UP", "RUF"]

{instruktai_python_logger-0.4.4 → instruktai_python_logger-0.5.0}/tests/test_configure_logging.py

@@ -4,9 +4,41 @@ from pathlib import Path
 from tempfile import TemporaryDirectory
 
 import pytest
-
 from instrukt_ai_logging import InstruktAILogger, configure_logging, get_logger
 
+# Log markers shared by each test and its assertions — named constants per
+# software-development/procedure/snapshot-testing (no bare literals in content
+# assertions). The logfmt field fragments are derived from the markers so the
+# expected output stays in sync with what is logged.
+_LOGGER_OURS = "teleclaude.core"
+_LOGGER_HTTPCORE = "httpcore.http11"
+_LOGGER_TELEGRAM = "telegram.ext.ExtBot"
+_LOGGER_MUTED = "teleclaude.cli.tui"
+
+_MSG_OURS = "hello from ours"
+_MSG_THIRD_PARTY = "hello from third-party"
+_MSG_HTTPCORE = "httpcore info"
+_MSG_TELEGRAM = "telegram info"
+_MSG_HELLO = "hello"
+_MSG_CORE_DEBUG = "core debug"
+_MSG_TUI_DEBUG = "tui debug"
+_MSG_TUI_WARNING = "tui warning"
+
+_KV_SESSION = "abc123"
+_KV_N = 1
+
+_FIELD_LOGGER_OURS = f"logger={_LOGGER_OURS}"
+_FIELD_LOGGER_HTTPCORE = f"logger={_LOGGER_HTTPCORE}"
+_FIELD_LOGGER_TELEGRAM = f"logger={_LOGGER_TELEGRAM}"
+_FIELD_LOGGER_MUTED = f"logger={_LOGGER_MUTED}"
+_FIELD_MSG_OURS = f'msg="{_MSG_OURS}"'
+_FIELD_MSG_HTTPCORE = f'msg="{_MSG_HTTPCORE}"'
+_FIELD_MSG_HELLO = f'msg="{_MSG_HELLO}"'
+_FIELD_MSG_CORE_DEBUG = f'msg="{_MSG_CORE_DEBUG}"'
+_FIELD_MSG_TUI_WARNING = f'msg="{_MSG_TUI_WARNING}"'
+_FIELD_SESSION = f"session={_KV_SESSION}"
+_FIELD_N = f"n={_KV_N}"
+
 
 @pytest.fixture()
 def isolated_logging():
@@ -38,13 +70,13 @@ def test_our_logs_respect_app_level_and_third_party_baseline(isolated_logging, m
 
         log_path = configure_logging("teleclaude")
 
-        logging.getLogger("teleclaude.core").debug("hello from ours")
-        logging.getLogger("httpcore.http11").info("hello from third-party")
+        logging.getLogger(_LOGGER_OURS).debug(_MSG_OURS)
+        logging.getLogger(_LOGGER_HTTPCORE).info(_MSG_THIRD_PARTY)
 
         content = _read_text(log_path)
-        assert "logger=teleclaude.core" in content
-        assert 'msg="hello from ours"' in content
-        assert "logger=httpcore.http11" not in content
+        assert _FIELD_LOGGER_OURS in content
+        assert _FIELD_MSG_OURS in content
+        assert _FIELD_LOGGER_HTTPCORE not in content
 
 
 def test_configure_logging_uses_single_file_handler(isolated_logging, monkeypatch):
@@ -60,9 +92,9 @@ def test_configure_logging_uses_single_file_handler(isolated_logging, monkeypatc
 
 
 def test_get_logger_returns_instrukt_logger(isolated_logging):
-    logger = get_logger("teleclaude.core")
+    logger = get_logger(_LOGGER_OURS)
     assert isinstance(logger, InstruktAILogger)
-    logger.info("hello", session="abc123", n=1)
+    logger.info(_MSG_HELLO, session=_KV_SESSION, n=_KV_N)
 
 
 def test_spotlight_allows_selected_third_party_only(isolated_logging, monkeypatch):
@@ -83,17 +115,17 @@ def test_spotlight_allows_selected_third_party_only(isolated_logging, monkeypatc
         telegram_logger.setLevel(logging.INFO)
 
         try:
-            logging.getLogger("httpcore.http11").info("httpcore info")
-            logging.getLogger("telegram.ext.ExtBot").info("telegram info")
+            logging.getLogger(_LOGGER_HTTPCORE).info(_MSG_HTTPCORE)
+            logging.getLogger(_LOGGER_TELEGRAM).info(_MSG_TELEGRAM)
         finally:
             httpcore_logger.setLevel(previous_httpcore_level)
             telegram_logger.setLevel(previous_telegram_level)
 
         content = _read_text(log_path)
-        assert "logger=httpcore.http11" in content
-        assert 'msg="httpcore info"' in content
-        assert "logger=telegram.ext.ExtBot" not in content
-        assert "telegram info" not in content
+        assert _FIELD_LOGGER_HTTPCORE in content
+        assert _FIELD_MSG_HTTPCORE in content
+        assert _FIELD_LOGGER_TELEGRAM not in content
+        assert _MSG_TELEGRAM not in content
 
 
 def test_named_kv_logger_emits_pairs(isolated_logging, monkeypatch):
@@ -104,12 +136,12 @@ def test_named_kv_logger_emits_pairs(isolated_logging, monkeypatch):
 
         log_path = configure_logging("teleclaude")
 
-        logging.getLogger("teleclaude.core").info("hello", session="abc123", n=1)
+        logging.getLogger(_LOGGER_OURS).info(_MSG_HELLO, session=_KV_SESSION, n=_KV_N)
 
         content = _read_text(log_path)
-        assert 'msg="hello"' in content
-        assert "session=abc123" in content
-        assert "n=1" in content
+        assert _FIELD_MSG_HELLO in content
+        assert _FIELD_SESSION in content
+        assert _FIELD_N in content
 
 
 def test_muted_loggers_forced_to_warning(isolated_logging, monkeypatch):
@@ -117,20 +149,20 @@ def test_muted_loggers_forced_to_warning(isolated_logging, monkeypatch):
         monkeypatch.setenv("INSTRUKT_AI_LOG_ROOT", tmp)
         monkeypatch.setenv("TELECLAUDE_LOG_LEVEL", "DEBUG")
         monkeypatch.setenv("TELECLAUDE_THIRD_PARTY_LOG_LEVEL", "WARNING")
-        monkeypatch.setenv("TELECLAUDE_MUTED_LOGGERS", "teleclaude.cli.tui")
+        monkeypatch.setenv("TELECLAUDE_MUTED_LOGGERS", _LOGGER_MUTED)
 
         log_path = configure_logging("teleclaude")
 
-        logging.getLogger("teleclaude.core").debug("core debug")
-        logging.getLogger("teleclaude.cli.tui").debug("tui debug")
-        logging.getLogger("teleclaude.cli.tui").warning("tui warning")
+        logging.getLogger(_LOGGER_OURS).debug(_MSG_CORE_DEBUG)
+        logging.getLogger(_LOGGER_MUTED).debug(_MSG_TUI_DEBUG)
+        logging.getLogger(_LOGGER_MUTED).warning(_MSG_TUI_WARNING)
 
         content = _read_text(log_path)
         # Core debug should appear (app level is DEBUG)
-        assert "logger=teleclaude.core" in content
-        assert 'msg="core debug"' in content
+        assert _FIELD_LOGGER_OURS in content
+        assert _FIELD_MSG_CORE_DEBUG in content
         # Muted logger's debug should NOT appear (forced to WARNING)
-        assert "tui debug" not in content
+        assert _MSG_TUI_DEBUG not in content
         # Muted logger's warning SHOULD appear
-        assert "logger=teleclaude.cli.tui" in content
-        assert 'msg="tui warning"' in content
+        assert _FIELD_LOGGER_MUTED in content
+        assert _FIELD_MSG_TUI_WARNING in content

instruktai_python_logger-0.5.0/tests/test_install.py

@@ -0,0 +1,131 @@
+from __future__ import annotations
+
+from pathlib import Path
+
+import pytest
+from instrukt_ai_logging.install import _install_logrotate, _install_newsyslog
+
+# Protocol-significant tokens for the rotation configs — named constants per
+# software-development/procedure/snapshot-testing (no bare literals in content
+# assertions). These are newsyslog field identities, not prose.
+_NEWSYSLOG_MODE = "640"
+_NEWSYSLOG_COUNT = "5"
+_NEWSYSLOG_SIZE_KB = "50000"
+_NEWSYSLOG_WHEN = "*"
+_NEWSYSLOG_COMPRESS_FLAG = "Z"  # gzip-compress archives
+_COMMENT_PREFIX = "#"
+_PREEXISTING_CONF = "preexisting"
+
+
+def test_install_newsyslog_emits_one_line_per_existing_log_file(tmp_path: Path) -> None:
+    log_root = tmp_path / "logs"
+    (log_root / "teleclaude").mkdir(parents=True)
+    (log_root / "teleclaude" / "teleclaude.log").touch()
+    (log_root / "teleclaude" / "cron.log").touch()
+    (log_root / "teleclaude" / "docs-watch.log").touch()
+    (log_root / "other-app").mkdir()
+    (log_root / "other-app" / "other-app.log").touch()
+    (log_root / "teleclaude" / "notes.txt").touch()  # ignored: not *.log
+    conf_path = tmp_path / "etc" / "newsyslog.d" / "instrukt-ai.conf"
+
+    _install_newsyslog(log_root, force=False, conf_path=conf_path)
+
+    rotated = [
+        line.split()[0]
+        for line in conf_path.read_text(encoding="utf-8").splitlines()
+        if line and not line.startswith(_COMMENT_PREFIX)
+    ]
+    assert rotated == sorted(
+        [
+            str(log_root / "other-app" / "other-app.log"),
+            str(log_root / "teleclaude" / "cron.log"),
+            str(log_root / "teleclaude" / "docs-watch.log"),
+            str(log_root / "teleclaude" / "teleclaude.log"),
+        ]
+    )
+
+
+def test_install_newsyslog_uses_compressing_rotation_defaults(tmp_path: Path) -> None:
+    log_root = tmp_path / "logs"
+    (log_root / "app").mkdir(parents=True)
+    (log_root / "app" / "app.log").touch()
+    conf_path = tmp_path / "newsyslog.conf"
+
+    _install_newsyslog(log_root, force=False, conf_path=conf_path)
+
+    fields = conf_path.read_text(encoding="utf-8").split()
+    # path  mode  count  size(KB)  when  flags
+    _, mode, count, size_kb, when, flags = fields
+    assert mode == _NEWSYSLOG_MODE
+    assert count == _NEWSYSLOG_COUNT
+    assert size_kb == _NEWSYSLOG_SIZE_KB
+    assert when == _NEWSYSLOG_WHEN
+    assert _NEWSYSLOG_COMPRESS_FLAG in flags  # gzip-compress archives
+
+
+def test_install_newsyslog_writes_placeholder_when_no_log_files_exist(tmp_path: Path) -> None:
+    log_root = tmp_path / "logs"
+    log_root.mkdir()  # exists but contains no *.log files
+    conf_path = tmp_path / "newsyslog.conf"
+
+    _install_newsyslog(log_root, force=False, conf_path=conf_path)
+
+    content = conf_path.read_text(encoding="utf-8")
+    assert content.startswith(_COMMENT_PREFIX)
+    assert all(line.startswith(_COMMENT_PREFIX) or not line for line in content.splitlines())
+
+
+def test_install_newsyslog_refuses_overwrite_without_force(tmp_path: Path) -> None:
+    log_root = tmp_path / "logs"
+    (log_root / "app").mkdir(parents=True)
+    (log_root / "app" / "app.log").touch()
+    conf_path = tmp_path / "newsyslog.conf"
+    conf_path.write_text(_PREEXISTING_CONF, encoding="utf-8")
+
+    with pytest.raises(SystemExit):
+        _install_newsyslog(log_root, force=False, conf_path=conf_path)
+    assert conf_path.read_text(encoding="utf-8") == _PREEXISTING_CONF
+
+
+def test_install_newsyslog_overwrites_with_force(tmp_path: Path) -> None:
+    log_root = tmp_path / "logs"
+    (log_root / "app").mkdir(parents=True)
+    (log_root / "app" / "app.log").touch()
+    conf_path = tmp_path / "newsyslog.conf"
+    conf_path.write_text(_PREEXISTING_CONF, encoding="utf-8")
+
+    _install_newsyslog(log_root, force=True, conf_path=conf_path)
+    assert conf_path.read_text(encoding="utf-8") != _PREEXISTING_CONF
+
+
+def test_install_logrotate_writes_glob_block_with_compressing_defaults(tmp_path: Path) -> None:
+    log_root = tmp_path / "logs"
+    log_root.mkdir()
+    conf_path = tmp_path / "logrotate.conf"
+
+    _install_logrotate(log_root, force=False, conf_path=conf_path)
+
+    content = conf_path.read_text(encoding="utf-8")
+    # logrotate handles multi-level globs natively, unlike macOS newsyslog.
+    assert f"{log_root}/*/*.log" in content
+    # Protocol-significant directives — these are logrotate keywords, not prose.
+    for directive in (
+        "size 50M",
+        "rotate 5",
+        "compress",
+        "delaycompress",
+        "missingok",
+        "copytruncate",
+    ):
+        assert directive in content
+
+
+def test_install_logrotate_refuses_overwrite_without_force(tmp_path: Path) -> None:
+    log_root = tmp_path / "logs"
+    log_root.mkdir()
+    conf_path = tmp_path / "logrotate.conf"
+    conf_path.write_text(_PREEXISTING_CONF, encoding="utf-8")
+
+    with pytest.raises(SystemExit):
+        _install_logrotate(log_root, force=False, conf_path=conf_path)
+    assert conf_path.read_text(encoding="utf-8") == _PREEXISTING_CONF

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

@@ -1,12 +1,13 @@
 from __future__ import annotations
 
+import sys
 from collections.abc import Iterator
-from datetime import timedelta
+from datetime import UTC, timedelta
 from pathlib import Path
 from tempfile import TemporaryDirectory
 
 import pytest
-
+from instrukt_ai_logging.cli import main
 from instrukt_ai_logging.logging import (
     iter_recent_log_lines_merged,
     resolve_log_files,
@@ -18,6 +19,23 @@ def _write(path: Path, content: str) -> None:
     path.write_text(content, encoding="utf-8")
 
 
+# Log markers shared by each fixture and its assertions — named constants per
+# software-development/procedure/snapshot-testing (no bare literals in content
+# assertions).
+_MSG_FRESH = "fresh"
+_MSG_BOOM = "boom"
+_MSG_AFTER = "after"
+_TRACEBACK_LINE = "  Traceback (most recent call last):"
+_FILE_LINE = "  File 'foo.py', line 1"
+_MSG_KEEP = "keep-me"
+_MSG_GIT_NOISE = "git-noise"
+_MSG_REAL_ERROR = "real-error"
+_MSG_GIT_ERROR = "git-error"
+_MSG_INFO = "info-line"
+_MSG_CRON = "cron-line"
+_MSG_DAEMON = "daemon-line"
+
+
 @pytest.fixture()
 def app_log_dir(monkeypatch: pytest.MonkeyPatch) -> Iterator[Path]:
     tmp = TemporaryDirectory()
@@ -68,9 +86,9 @@ def test_resolve_log_files_returns_empty_when_dir_missing(monkeypatch: pytest.Mo
 
 
 def _now_iso(offset_minutes: int = 0) -> str:
-    from datetime import datetime, timezone
+    from datetime import datetime
 
-    dt = datetime.now(tz=timezone.utc) + timedelta(minutes=offset_minutes)
+    dt = datetime.now(tz=UTC) + timedelta(minutes=offset_minutes)
     return dt.strftime("%Y-%m-%dT%H:%M:%S") + f".{int(dt.microsecond / 1000):03d}Z"
 
 
@@ -101,21 +119,21 @@ def test_iter_recent_log_lines_merged_respects_since_cutoff(app_log_dir: Path) -
     daemon = app_log_dir / "demo-app.log"
     daemon.write_text(
         f"{_now_iso(-30)} level=INFO logger=demo.daemon msg=old\n"
-        f"{_now_iso(-1)} level=INFO logger=demo.daemon msg=fresh\n",
+        f"{_now_iso(-1)} level=INFO logger=demo.daemon msg={_MSG_FRESH}\n",
         encoding="utf-8",
     )
 
     files = resolve_log_files("demo-app")
     lines = list(iter_recent_log_lines_merged(files, since=timedelta(minutes=5)))
     assert len(lines) == 1
-    assert "fresh" in lines[0]
+    assert _MSG_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
+    from datetime import datetime
 
     fresh = app_log_dir / "demo-app.log"
     fresh.write_text(
@@ -128,7 +146,7 @@ def test_iter_recent_log_lines_merged_skips_files_with_mtime_before_cutoff(
         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()
+    one_day_ago = (datetime.now(tz=UTC) - timedelta(days=1)).timestamp()
     os.utime(stale, (one_day_ago, one_day_ago))
 
     files = resolve_log_files("demo-app")
@@ -142,17 +160,78 @@ def test_iter_recent_log_lines_merged_keeps_continuation_lines_with_parent(
 ) -> None:
     daemon = app_log_dir / "demo-app.log"
     daemon.write_text(
-        f"{_now_iso(-1)} level=ERROR logger=demo.daemon msg=boom\n"
-        f"  Traceback (most recent call last):\n"
-        f"  File 'foo.py', line 1\n"
-        f"{_now_iso(0)} level=INFO logger=demo.daemon msg=after\n",
+        f"{_now_iso(-1)} level=ERROR logger=demo.daemon msg={_MSG_BOOM}\n"
+        f"{_TRACEBACK_LINE}\n"
+        f"{_FILE_LINE}\n"
+        f"{_now_iso(0)} level=INFO logger=demo.daemon msg={_MSG_AFTER}\n",
         encoding="utf-8",
     )
 
     files = resolve_log_files("demo-app")
     lines = list(iter_recent_log_lines_merged(files, since=timedelta(minutes=10)))
     assert len(lines) == 4
-    assert "boom" in lines[0]
-    assert lines[1].startswith("  Traceback")
-    assert lines[2].startswith("  File")
-    assert "after" in lines[3]
+    assert _MSG_BOOM in lines[0]
+    assert lines[1].startswith(_TRACEBACK_LINE)
+    assert lines[2].startswith(_FILE_LINE)
+    assert _MSG_AFTER in lines[3]
+
+
+def test_cli_exclude_drops_matching_lines(
+    app_log_dir: Path, monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str]
+) -> None:
+    daemon = app_log_dir / "demo-app.log"
+    daemon.write_text(
+        f"{_now_iso(-1)} level=ERROR logger=demo.git.exec msg={_MSG_GIT_NOISE}\n"
+        f"{_now_iso(0)} level=INFO logger=demo.daemon msg={_MSG_KEEP}\n",
+        encoding="utf-8",
+    )
+
+    monkeypatch.setattr(sys, "argv", ["instrukt-ai-logs", "demo-app", "--exclude", r"logger=demo\.git\.exec"])
+    main()
+
+    out = capsys.readouterr().out
+    assert _MSG_KEEP in out
+    assert _MSG_GIT_NOISE not in out
+
+
+def test_cli_exclude_applies_after_grep(
+    app_log_dir: Path, monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str]
+) -> None:
+    daemon = app_log_dir / "demo-app.log"
+    daemon.write_text(
+        f"{_now_iso(-2)} level=ERROR logger=demo.git.exec msg={_MSG_GIT_ERROR}\n"
+        f"{_now_iso(-1)} level=ERROR logger=demo.daemon msg={_MSG_REAL_ERROR}\n"
+        f"{_now_iso(0)} level=INFO logger=demo.daemon msg={_MSG_INFO}\n",
+        encoding="utf-8",
+    )
+
+    # Keep only ERROR lines, then drop the git seam ones.
+    monkeypatch.setattr(
+        sys,
+        "argv",
+        ["instrukt-ai-logs", "demo-app", "--grep", "level=ERROR", "--exclude", r"logger=demo\.git\.exec"],
+    )
+    main()
+
+    out = capsys.readouterr().out
+    assert _MSG_REAL_ERROR in out
+    assert _MSG_GIT_ERROR not in out
+    assert _MSG_INFO not in out
+
+
+def test_cli_include_alias_matches_logs_flag(
+    app_log_dir: Path, monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str]
+) -> None:
+    (app_log_dir / "demo-app.log").write_text(
+        f"{_now_iso(0)} level=INFO logger=demo.daemon msg={_MSG_DAEMON}\n", encoding="utf-8"
+    )
+    (app_log_dir / "cron.log").write_text(
+        f"{_now_iso(0)} level=INFO logger=demo.cron msg={_MSG_CRON}\n", encoding="utf-8"
+    )
+
+    monkeypatch.setattr(sys, "argv", ["instrukt-ai-logs", "demo-app", "--include", "cron"])
+    main()
+
+    out = capsys.readouterr().out
+    assert _MSG_CRON in out
+    assert _MSG_DAEMON not in out

{instruktai_python_logger-0.4.4 → instruktai_python_logger-0.5.0}/tests/test_trace_logging.py

@@ -7,6 +7,23 @@ from instrukt_ai_logging.logging import (
     get_logger,
 )
 
+# Log markers shared by each test and its assertions — named constants per
+# software-development/procedure/snapshot-testing (no bare literals in content
+# assertions). The logfmt field fragments are derived from the markers so the
+# expected output stays in sync with what is logged.
+_TRACE_MSG = "This is a trace message"
+_DEBUG_MSG = "This is a debug message"
+_KV_KEY = "kv_key"
+_KV_VALUE = "kv_value"
+_FILTERED_TRACE_MSG = "This should not appear"
+_FILTERED_DEBUG_MSG = "This should appear"
+
+_LEVEL_TRACE_FIELD = "level=TRACE"
+_LEVEL_DEBUG_FIELD = "level=DEBUG"
+_TRACE_MSG_FIELD = f'msg="{_TRACE_MSG}"'
+_DEBUG_MSG_FIELD = f'msg="{_DEBUG_MSG}"'
+_KV_FIELD = f"{_KV_KEY}={_KV_VALUE}"
+
 
 @pytest.fixture
 def clean_env():
@@ -29,23 +46,23 @@ def test_trace_level_logging(clean_env):
         logger = get_logger("test_trace_app")
 
         # Log at TRACE level
-        logger.trace("This is a trace message", kv_key="kv_value")
+        logger.trace(_TRACE_MSG, **{_KV_KEY: _KV_VALUE})
 
         # Log at DEBUG level
-        logger.debug("This is a debug message")
+        logger.debug(_DEBUG_MSG)
 
         # Read the log file
-        with open(log_file, "r") as f:
+        with open(log_file) as f:
             lines = f.readlines()
 
         assert len(lines) == 2
 
-        assert "level=TRACE" in lines[0]
-        assert 'msg="This is a trace message"' in lines[0]
-        assert "kv_key=kv_value" in lines[0]
+        assert _LEVEL_TRACE_FIELD in lines[0]
+        assert _TRACE_MSG_FIELD in lines[0]
+        assert _KV_FIELD in lines[0]
 
-        assert "level=DEBUG" in lines[1]
-        assert 'msg="This is a debug message"' in lines[1]
+        assert _LEVEL_DEBUG_FIELD in lines[1]
+        assert _DEBUG_MSG_FIELD in lines[1]
 
 
 def test_trace_level_filtering(clean_env):
@@ -59,11 +76,11 @@ def test_trace_level_filtering(clean_env):
         log_file = configure_logging(app_name)
         logger = get_logger("test_trace_filtering")
 
-        logger.trace("This should not appear")
-        logger.debug("This should appear")
+        logger.trace(_FILTERED_TRACE_MSG)
+        logger.debug(_FILTERED_DEBUG_MSG)
 
-        with open(log_file, "r") as f:
+        with open(log_file) as f:
             lines = f.readlines()
 
         assert len(lines) == 1
-        assert "level=DEBUG" in lines[0]
+        assert _LEVEL_DEBUG_FIELD in lines[0]

instruktai_python_logger-0.4.4/instrukt_ai_logging/install.py

@@ -1,59 +0,0 @@
-"""Install log rotation for InstruktAI logs."""
-
-from __future__ import annotations
-
-import argparse
-import os
-import sys
-from pathlib import Path
-
-
-def _write_file(path: Path, content: str, *, force: bool) -> None:
-    if path.exists() and not force:
-        raise SystemExit(f"Refusing to overwrite existing file: {path}. Use --force.")
-    path.parent.mkdir(parents=True, exist_ok=True)
-    path.write_text(content, encoding="utf-8")
-
-
-def _install_newsyslog(log_root: Path, *, force: bool) -> Path:
-    conf_path = Path("/etc/newsyslog.d/instrukt-ai.conf")
-    line = f"{log_root}/" + "*/*.log  640  10  100000  *  N\n"
-    _write_file(conf_path, line, force=force)
-    return conf_path
-
-
-def _install_logrotate(log_root: Path, *, force: bool) -> Path:
-    conf_path = Path("/etc/logrotate.d/instrukt-ai")
-    content = f"""{log_root}/*/*.log {{
-    size 100M
-    rotate 10
-    missingok
-    notifempty
-    copytruncate
-}}
-"""
-    _write_file(conf_path, content, force=force)
-    return conf_path
-
-
-def main() -> None:
-    parser = argparse.ArgumentParser(prog="instrukt-ai-log-setup", add_help=True)
-    parser.add_argument(
-        "--log-root",
-        default=os.environ.get("INSTRUKT_AI_LOG_ROOT", "/var/log/instrukt-ai"),
-        help="Base log root (default: /var/log/instrukt-ai or INSTRUKT_AI_LOG_ROOT)",
-    )
-    parser.add_argument("--force", action="store_true", help="Overwrite existing config")
-    args = parser.parse_args()
-
-    log_root = Path(args.log_root)
-    if sys.platform == "darwin":
-        conf = _install_newsyslog(log_root, force=args.force)
-    else:
-        conf = _install_logrotate(log_root, force=args.force)
-
-    sys.stdout.write(f"installed: {conf}\n")
-
-
-if __name__ == "__main__":
-    main()

instruktai_python_logger-0.4.4/instruktai_python_logger.egg-info/requires.txt

@@ -1,4 +0,0 @@
-
-[dev]
-pytest>=8
-ruff>=0.8

instruktai_python_logger-0.4.4/pyproject.toml

@@ -1,33 +0,0 @@
-[build-system]
-requires = ["setuptools>=69", "wheel"]
-build-backend = "setuptools.build_meta"
-
-[project]
-name = "instruktai-python-logger"
-version = "0.4.4"
-description = "Centralized logging utilities for Python services."
-readme = "README.md"
-requires-python = ">=3.11"
-dependencies = []
-
-[project.optional-dependencies]
-dev = ["pytest>=8", "ruff>=0.8"]
-
-[project.scripts]
-instrukt-ai-logs = "instrukt_ai_logging.cli:main"
-instrukt-ai-log-setup = "instrukt_ai_logging.install:main"
-
-[tool.setuptools.packages.find]
-where = ["."]
-include = ["instrukt_ai_logging*"]
-
-[tool.setuptools.package-data]
-instrukt_ai_logging = ["py.typed"]
-
-[tool.pytest.ini_options]
-addopts = "-q"
-testpaths = ["tests"]
-
-[tool.ruff]
-line-length = 100
-target-version = "py311"