instruktai-python-logger 0.6.3__tar.gz → 0.6.4__tar.gz

{instruktai_python_logger-0.6.3 → instruktai_python_logger-0.6.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: instruktai-python-logger
-Version: 0.6.3
+Version: 0.6.4
 Summary: Centralized logging utilities for Python services.
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown
@@ -28,6 +28,10 @@ From GitHub:
 pip install git+ssh://git@github.com/InstruktAI/python-logger.git
 ```
 
+For a local checkout, `make install` prepares dependencies and development
+tooling only. Run `make install-runtime` separately when this user's rotation
+scheduler should be wired.
+
 ## API
 
 - Python entrypoint: `instrukt_ai_logging.configure_logging(...)`
@@ -88,14 +92,16 @@ write its logs must not start blind.
 
 ## Rotation
 
-Rotation is transparent and runs entirely in user space:
-`configure_logging(...)` idempotently ensures a per-user rotation conf plus a
-per-user scheduler (a launchd LaunchAgent on macOS running `newsyslog`, a
-systemd user timer on Linux running `logrotate`) on every process start. No
-`/etc` configuration, no sudo, and no system rotation daemon ever touches
-these files, so rotated/recreated logs always stay owned by the producing
-user. Run `instrukt-ai-log-setup` to check rotation-asset status manually;
-normal operation never requires it.
+Rotation runs entirely in user space. At deploy, run
+`instrukt-ai-log-setup` once (or `make install-runtime` in this repository) to
+wire the per-user rotation conf plus scheduler: a launchd LaunchAgent on macOS
+running `newsyslog`, or a systemd user timer on Linux running `logrotate`.
+Re-running setup on an already-wired machine reports healthy.
+
+`configure_logging(...)` only creates and opens the log file; it never writes
+rotation assets or invokes launchd/systemd. No `/etc` configuration, no sudo,
+and no system rotation daemon ever touches these files, so rotated/recreated
+logs always stay owned by the producing user.
 
 ## CLI
 

{instruktai_python_logger-0.6.3 → instruktai_python_logger-0.6.4}/README.md

@@ -21,6 +21,10 @@ From GitHub:
 pip install git+ssh://git@github.com/InstruktAI/python-logger.git
 ```
 
+For a local checkout, `make install` prepares dependencies and development
+tooling only. Run `make install-runtime` separately when this user's rotation
+scheduler should be wired.
+
 ## API
 
 - Python entrypoint: `instrukt_ai_logging.configure_logging(...)`
@@ -81,14 +85,16 @@ write its logs must not start blind.
 
 ## Rotation
 
-Rotation is transparent and runs entirely in user space:
-`configure_logging(...)` idempotently ensures a per-user rotation conf plus a
-per-user scheduler (a launchd LaunchAgent on macOS running `newsyslog`, a
-systemd user timer on Linux running `logrotate`) on every process start. No
-`/etc` configuration, no sudo, and no system rotation daemon ever touches
-these files, so rotated/recreated logs always stay owned by the producing
-user. Run `instrukt-ai-log-setup` to check rotation-asset status manually;
-normal operation never requires it.
+Rotation runs entirely in user space. At deploy, run
+`instrukt-ai-log-setup` once (or `make install-runtime` in this repository) to
+wire the per-user rotation conf plus scheduler: a launchd LaunchAgent on macOS
+running `newsyslog`, or a systemd user timer on Linux running `logrotate`.
+Re-running setup on an already-wired machine reports healthy.
+
+`configure_logging(...)` only creates and opens the log file; it never writes
+rotation assets or invokes launchd/systemd. No `/etc` configuration, no sudo,
+and no system rotation daemon ever touches these files, so rotated/recreated
+logs always stay owned by the producing user.
 
 ## CLI
 

{instruktai_python_logger-0.6.3 → instruktai_python_logger-0.6.4}/instrukt_ai_logging/install.py

@@ -155,9 +155,19 @@ def _ensure_launchd_loaded(plist_path: Path) -> list[str]:
     except (OSError, subprocess.TimeoutExpired) as exc:
         return [f"launchctl bootstrap failed to run: {exc}"]
 
-    if result.returncode != 0 and "already bootstrapped" not in result.stderr.lower():
+    if result.returncode == 0 or "already bootstrapped" in result.stderr.lower():
+        return []
+
+    # macOS reports EIO(5), not "already bootstrapped", when re-bootstrapping an already-loaded agent.
+    # Confirm with `print` exit status; a nonzero result preserves the original bootstrap failure.
+    try:
+        loaded_result = _run_scheduler_command(["launchctl", "print", f"gui/{uid}/{_LAUNCHD_LABEL}"])
+    except (OSError, subprocess.TimeoutExpired):
         return [f"launchctl bootstrap failed: {result.stderr.strip()}"]
-    return []
+
+    if loaded_result.returncode == 0:
+        return []
+    return [f"launchctl bootstrap failed: {result.stderr.strip()}"]
 
 
 def _ensure_systemd_loaded() -> list[str]:

{instruktai_python_logger-0.6.3 → instruktai_python_logger-0.6.4}/instrukt_ai_logging/logging.py

@@ -12,8 +12,6 @@ from logging.handlers import WatchedFileHandler
 from pathlib import Path
 from typing import Any, Protocol, cast, runtime_checkable
 
-from instrukt_ai_logging.install import ensure_rotation
-
 # Standard logging levels are: NOTSET=0, DEBUG=10, INFO=20, WARNING=30, ERROR=40, CRITICAL=50
 TRACE: int = 5
 logging.addLevelName(TRACE, "TRACE")
@@ -485,11 +483,6 @@ def configure_logging(
     for prefix in muted:
         logging.getLogger(prefix).setLevel(logging.WARNING)
 
-    # Rotation bootstrap is not logging: a failure here is surfaced as a
-    # warning, never fatal to an already-working log file.
-    for problem in ensure_rotation():
-        logging.getLogger(app_logger_prefix).warning("log rotation ensure problem: %s", problem)
-
     return log_file
 
 

{instruktai_python_logger-0.6.3 → instruktai_python_logger-0.6.4}/instruktai_python_logger.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: instruktai-python-logger
-Version: 0.6.3
+Version: 0.6.4
 Summary: Centralized logging utilities for Python services.
 Requires-Python: >=3.11
 Description-Content-Type: text/markdown
@@ -28,6 +28,10 @@ From GitHub:
 pip install git+ssh://git@github.com/InstruktAI/python-logger.git
 ```
 
+For a local checkout, `make install` prepares dependencies and development
+tooling only. Run `make install-runtime` separately when this user's rotation
+scheduler should be wired.
+
 ## API
 
 - Python entrypoint: `instrukt_ai_logging.configure_logging(...)`
@@ -88,14 +92,16 @@ write its logs must not start blind.
 
 ## Rotation
 
-Rotation is transparent and runs entirely in user space:
-`configure_logging(...)` idempotently ensures a per-user rotation conf plus a
-per-user scheduler (a launchd LaunchAgent on macOS running `newsyslog`, a
-systemd user timer on Linux running `logrotate`) on every process start. No
-`/etc` configuration, no sudo, and no system rotation daemon ever touches
-these files, so rotated/recreated logs always stay owned by the producing
-user. Run `instrukt-ai-log-setup` to check rotation-asset status manually;
-normal operation never requires it.
+Rotation runs entirely in user space. At deploy, run
+`instrukt-ai-log-setup` once (or `make install-runtime` in this repository) to
+wire the per-user rotation conf plus scheduler: a launchd LaunchAgent on macOS
+running `newsyslog`, or a systemd user timer on Linux running `logrotate`.
+Re-running setup on an already-wired machine reports healthy.
+
+`configure_logging(...)` only creates and opens the log file; it never writes
+rotation assets or invokes launchd/systemd. No `/etc` configuration, no sudo,
+and no system rotation daemon ever touches these files, so rotated/recreated
+logs always stay owned by the producing user.
 
 ## CLI
 

{instruktai_python_logger-0.6.3 → instruktai_python_logger-0.6.4}/pyproject.toml

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

{instruktai_python_logger-0.6.3 → instruktai_python_logger-0.6.4}/tests/test_configure_logging.py

@@ -5,7 +5,7 @@ from pathlib import Path
 from tempfile import TemporaryDirectory
 
 import pytest
-from instrukt_ai_logging import InstruktAILogger, configure_logging, get_logger, install
+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
@@ -41,8 +41,6 @@ _FIELD_SESSION = f"session={_KV_SESSION}"
 _FIELD_N = f"n={_KV_N}"
 
 _APP_NAME = "teleclaude"
-_ROTATION_PROBLEM_MARKER = "log rotation ensure problem"
-_ROTATION_FAILURE_REASON = "no session bus"
 
 
 @pytest.fixture()
@@ -121,33 +119,6 @@ def test_configure_logging_fails_fast_when_file_unwritable(isolated_logging, mon
             os.chmod(log_path, 0o644)
 
 
-class _FakeFailingCompletedProcess:
-    def __init__(self) -> None:
-        self.returncode = 1
-        self.stderr = _ROTATION_FAILURE_REASON
-        self.stdout = ""
-
-
-def test_configure_logging_warns_without_raising_when_rotation_ensure_fails(isolated_logging, monkeypatch):
-    with TemporaryDirectory() as tmp:
-        monkeypatch.setenv("XDG_STATE_HOME", tmp)
-        monkeypatch.setenv("TELECLAUDE_LOG_LEVEL", "INFO")
-        monkeypatch.setenv("TELECLAUDE_THIRD_PARTY_LOG_LEVEL", "WARNING")
-
-        def _fake_run(args):
-            return _FakeFailingCompletedProcess()
-
-        monkeypatch.setattr(install, "_run_scheduler_command", _fake_run)
-
-        # A rotation-ensure failure must never crash logging — it's surfaced
-        # as a WARNING through the configured logger, not raised.
-        log_path = configure_logging(_APP_NAME)
-
-        content = _read_text(log_path)
-        assert _ROTATION_PROBLEM_MARKER in content
-        assert _ROTATION_FAILURE_REASON in content
-
-
 def test_configure_logging_resolves_xdg_state_home_location(isolated_logging, monkeypatch):
     with TemporaryDirectory() as tmp:
         monkeypatch.setenv("XDG_STATE_HOME", tmp)

{instruktai_python_logger-0.6.3 → instruktai_python_logger-0.6.4}/tests/test_install.py

@@ -223,6 +223,58 @@ def test_ensure_rotation_tolerates_already_bootstrapped_launchd(
     assert problems == []
 
 
+@pytest.mark.functional
+def test_main_treats_print_confirmed_launchd_eio_as_healthy(
+    rotation_env: Path, monkeypatch: pytest.MonkeyPatch
+) -> None:
+    monkeypatch.setattr(sys, "platform", "darwin")
+
+    def _fake_run(args: list[str]) -> _FakeCompletedProcess:
+        if args[1] == "bootstrap":
+            return _FakeCompletedProcess(returncode=5, stderr="Bootstrap failed: 5: Input/output error")
+        return _FakeCompletedProcess(returncode=0)
+
+    monkeypatch.setattr(install, "_run_scheduler_command", _fake_run)
+
+    install.main()
+
+
+@pytest.mark.functional
+def test_main_reports_unconfirmed_launchd_eio(rotation_env: Path, monkeypatch: pytest.MonkeyPatch) -> None:
+    monkeypatch.setattr(sys, "platform", "darwin")
+
+    def _fake_run(args: list[str]) -> _FakeCompletedProcess:
+        if args[1] == "bootstrap":
+            return _FakeCompletedProcess(returncode=5, stderr="Bootstrap failed: 5: Input/output error")
+        return _FakeCompletedProcess(returncode=1)
+
+    monkeypatch.setattr(install, "_run_scheduler_command", _fake_run)
+
+    with pytest.raises(SystemExit) as exc_info:
+        install.main()
+
+    assert exc_info.value.code == 1
+
+
+@pytest.mark.functional
+def test_main_reports_launchd_eio_when_discriminator_times_out(
+    rotation_env: Path, monkeypatch: pytest.MonkeyPatch
+) -> None:
+    monkeypatch.setattr(sys, "platform", "darwin")
+
+    def _fake_run(args: list[str]) -> _FakeCompletedProcess:
+        if args[1] == "bootstrap":
+            return _FakeCompletedProcess(returncode=5, stderr="Bootstrap failed: 5: Input/output error")
+        raise subprocess.TimeoutExpired(cmd=args, timeout=install._SCHEDULER_COMMAND_TIMEOUT_S)
+
+    monkeypatch.setattr(install, "_run_scheduler_command", _fake_run)
+
+    with pytest.raises(SystemExit) as exc_info:
+        install.main()
+
+    assert exc_info.value.code == 1
+
+
 def test_main_prints_status_and_exits_zero_when_healthy(
     rotation_env: Path,
     successful_subprocess: list[list[str]],