cluxion-agentplugin-preprocessing 0.3.6__tar.gz → 0.3.8__tar.gz

{cluxion_agentplugin_preprocessing-0.3.6 → cluxion_agentplugin_preprocessing-0.3.8}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cluxion-agentplugin-preprocessing
-Version: 0.3.6
+Version: 0.3.8
 Summary: Universal agent plugin for Cluxion preprocessing, honesty contracts, clarification, Rust work queue, and resource-aware harness handoff.
 Project-URL: Homepage, https://github.com/cluxion/cluxion-Agentplugin-preprocessing
 Project-URL: Repository, https://github.com/cluxion/cluxion-Agentplugin-preprocessing

{cluxion_agentplugin_preprocessing-0.3.6 → cluxion_agentplugin_preprocessing-0.3.8}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "cluxion-agentplugin-preprocessing"
-version = "0.3.6"
+version = "0.3.8"
 description = "Universal agent plugin for Cluxion preprocessing, honesty contracts, clarification, Rust work queue, and resource-aware harness handoff."
 readme = "README.md"
 requires-python = ">=3.11"

{cluxion_agentplugin_preprocessing-0.3.6 → cluxion_agentplugin_preprocessing-0.3.8}/src/cluxion_agentplugin_preprocessing/__init__.py

@@ -2,6 +2,6 @@
 
 from __future__ import annotations
 
-__version__ = "0.3.6"
+__version__ = "0.3.8"
 
 __all__ = ["__version__"]

{cluxion_agentplugin_preprocessing-0.3.6 → cluxion_agentplugin_preprocessing-0.3.8}/src/cluxion_agentplugin_preprocessing/doctor/probes.py

@@ -3,7 +3,12 @@
 from __future__ import annotations
 
 import importlib.metadata
+import importlib.util
+import json as _json
+import os
 import shutil
+import sys
+import tempfile
 from collections.abc import Callable
 
 from .framework import DoctorContext
@@ -96,7 +101,7 @@ def native_module_importable(ctx: DoctorContext) -> tuple[str, str]:
             return "pass", "imported (native backend available)"
         return "warn", "imported but expected symbols missing"
     except Exception:
-        return "warn", "native missing \u2192 using fallback (slower)"
+        return "warn", "native missing → using fallback (slower)"
 
 
 # plugin-specific probes (deterministic ones only)
@@ -174,4 +179,148 @@ def handler_exception_coverage(ctx: DoctorContext) -> tuple[str, str]:
         return "skip", f"cannot invoke guard: {e}"
 
 
+def _safe_read_hermes_config():
+    try:
+        import yaml
+        cfg_path = os.path.expanduser("~/.hermes/config.yaml")
+        if not os.path.exists(cfg_path):
+            return None, "absent"
+        with open(cfg_path, encoding="utf-8") as f:
+            data = yaml.safe_load(f) or {}
+        return data, "ok"
+    except Exception as e:
+        return None, f"read_error:{type(e).__name__}"
+
+
+@_register("psutil_importable")
+def psutil_importable(ctx: DoctorContext) -> tuple[str, str]:
+    try:
+        import psutil
+        _ = psutil.virtual_memory()
+        return "pass", "importable"
+    except ImportError:
+        return "fail", "psutil not installed"
+    except Exception as e:
+        return "skip", f"uncertainty: {type(e).__name__}"
+
+
+@_register("pyyaml_importable")
+def pyyaml_importable(ctx: DoctorContext) -> tuple[str, str]:
+    try:
+        import yaml
+        yaml.safe_load("test: 1")
+        return "pass", "importable"
+    except ImportError:
+        return "fail", "PyYAML not installed"
+    except Exception as e:
+        return "skip", f"uncertainty: {type(e).__name__}"
+
+
+@_register("fcntl_available_on_posix")
+def fcntl_available_on_posix(ctx: DoctorContext) -> tuple[str, str]:
+    if os.name != "posix":
+        return "skip", "non-POSIX (Windows)"
+    try:
+        import fcntl
+        # real check: lock a temp file
+        with tempfile.NamedTemporaryFile(delete=False) as tf:
+            tf_path = tf.name
+        try:
+            with open(tf_path, "a+b") as f:
+                fcntl.flock(f.fileno(), fcntl.LOCK_EX)
+                fcntl.flock(f.fileno(), fcntl.LOCK_UN)
+            return "pass", "fcntl works on POSIX"
+        finally:
+            os.unlink(tf_path)
+    except Exception as e:
+        return "skip", f"fcntl issue: {type(e).__name__}"
+
+
+@_register("playwright_optional_available")
+def playwright_optional_available(ctx: DoctorContext) -> tuple[str, str]:
+    if importlib.util.find_spec("playwright") is not None:
+        return "pass", "importable"
+    return "warn", "optional, not installed"
+
+
+@_register("abi3_wheel_compatible")
+def abi3_wheel_compatible(ctx: DoctorContext) -> tuple[str, str]:
+    # since requires-python >=3.11 the check is always pass
+    return "pass", f"Python {sys.version_info.major}.{sys.version_info.minor} >= 3.11 abi3 floor"
+
+
+@_register("sqlite_wal_mode_compatible")
+def sqlite_wal_mode_compatible(ctx: DoctorContext) -> tuple[str, str]:
+    try:
+        import sqlite3
+        conn = sqlite3.connect(":memory:")
+        try:
+            mode = conn.execute("PRAGMA journal_mode=WAL").fetchone()[0]
+            ver = sqlite3.sqlite_version
+            if str(mode).lower() == "wal":
+                return "pass", f"wal supported (sqlite {ver})"
+            return "warn", f"got {mode} (sqlite {ver})"
+        finally:
+            conn.close()
+    except Exception as e:
+        return "skip", f"uncertainty: {type(e).__name__}"
+
+
+@_register("json_serialization_deterministic")
+def json_serialization_deterministic(ctx: DoctorContext) -> tuple[str, str]:
+    try:
+        d = {"z": 1, "a": 2, "nested": {"b": 3}}
+        j1 = _json.dumps(d, sort_keys=True, separators=(",", ":"))
+        j2 = _json.dumps(d, sort_keys=True, separators=(",", ":"))
+        if j1 == j2:
+            return "pass", "roundtrip bytes equal"
+        return "fail", "non-deterministic"
+    except Exception as e:
+        return "skip", f"uncertainty: {type(e).__name__}"
+
+
+@_register("hermes_plugin_enabled")
+def hermes_plugin_enabled(ctx: DoctorContext) -> tuple[str, str]:
+    data, status = _safe_read_hermes_config()
+    if status != "ok":
+        return "skip", f"config not present: {status}"
+    try:
+        plugins = (data or {}).get("plugins", {}) or {}
+        enabled = plugins.get("enabled", []) or []
+        disabled = plugins.get("disabled", []) or []
+        names = ["cluxion-agentplugin-preprocessing", "hermes-cluxion"]
+        for n in names:
+            if n in enabled and n not in disabled:
+                return "pass", f"{n} in enabled"
+        if any(n in enabled for n in names):
+            return "warn", "present but also disabled?"
+        return "warn", "not in plugins.enabled"
+    except Exception as e:
+        return "skip", f"uncertainty: {type(e).__name__}"
+
+
+@_register("env_var_consistency")
+def env_var_consistency(ctx: DoctorContext) -> tuple[str, str]:
+    try:
+        known = [
+            "CLUXION_QUEUE_STORE_DIR",
+            "CLUXION_PREPROCESS_DISPATCH_DIR",
+            "CLUXION_QUEUE_BACKEND",
+        ]
+        issues = []
+        for var in known:
+            val = os.environ.get(var)
+            if val and "DIR" in var:
+                try:
+                    p = os.path.expanduser(val)
+                    os.makedirs(p, exist_ok=True)
+                except Exception:
+                    issues.append(f"{var}=invalid_path")
+        if issues:
+            return "warn", ";".join(issues)
+        return "pass", "defaults or valid"
+    except Exception as e:
+        return "skip", f"uncertainty: {type(e).__name__}"
+
+
 # note: other checks in catalog will be reported as skip (no probe)

{cluxion_agentplugin_preprocessing-0.3.6 → cluxion_agentplugin_preprocessing-0.3.8}/src/cluxion_agentplugin_preprocessing/plugin.py

@@ -247,6 +247,10 @@ def _handle_browser_type(args: dict[str, object], **_: object) -> str:
 
 
 def _handle_doctor(args: dict[str, object], **_: object) -> str:
+    return _json_result(lambda: _run_doctor(args))
+
+
+def _run_doctor(args: dict[str, object]) -> str:
     pkg = "cluxion_agentplugin_preprocessing.doctor"
     catalog_path = Path(str(importlib.resources.files(pkg).joinpath("catalog.json")))
     result = run_doctor(

{cluxion_agentplugin_preprocessing-0.3.6 → cluxion_agentplugin_preprocessing-0.3.8}/src/cluxion_agentplugin_preprocessing/runner.py

@@ -65,6 +65,7 @@ def bootstrap(payload: Mapping[str, object], *, command_runner: CommandRunner |
 def serve_local(payload: Mapping[str, object], *, command_runner: CommandRunner | None = None) -> RuntimeResult:
     """Run cluxion-runtime serve-local with dry-run by default."""
     model = _required_str(payload, "model")
+    port = _port(payload, "port", 23003)
     command = [
         _runtime_binary(None),
         "serve-local",
@@ -73,7 +74,7 @@ def serve_local(payload: Mapping[str, object], *, command_runner: CommandRunner
         "--host",
         _str(payload, "host", "127.0.0.1"),
         "--port",
-        str(_int(payload, "port", 23003)),
+        str(port),
         "--max-tokens",
         str(_int(payload, "max_tokens", 128_000)),
     ]
@@ -89,6 +90,7 @@ def serve_local(payload: Mapping[str, object], *, command_runner: CommandRunner
 def hermes_config(payload: Mapping[str, object], *, command_runner: CommandRunner | None = None) -> RuntimeResult:
     """Render Hermes config patch for a local endpoint."""
     model = _required_str(payload, "model")
+    port = _port(payload, "port", 23003)
     command = (
         _runtime_binary(None),
         "hermes-local-config",
@@ -97,7 +99,7 @@ def hermes_config(payload: Mapping[str, object], *, command_runner: CommandRunne
         "--host",
         _str(payload, "host", "127.0.0.1"),
         "--port",
-        str(_int(payload, "port", 23003)),
+        str(port),
         "--context-length",
         str(_int(payload, "context_length", 131_072)),
         "--provider-key",
@@ -144,6 +146,12 @@ def queue_brief(payload: Mapping[str, object], *, command_runner: CommandRunner
 
 def context_compress(payload: Mapping[str, object], *, command_runner: CommandRunner | None = None) -> RuntimeResult:
     """Compress conversation context deterministically once it exceeds the trigger ratio."""
+    messages = payload.get("messages")
+    if not isinstance(messages, list):
+        raise ValueError("messages must be a list of objects")
+    for i, m in enumerate(messages):
+        if not isinstance(m, dict) or not isinstance(m.get("content"), str):
+            raise ValueError(f"message[{i}] must be an object with string content")
     command = (_runtime_binary(None), "context-compress", "--json-stdin")
     stdin = json.dumps(dict(payload), ensure_ascii=False)
     return _execute_json(command, stdin, command_runner)
@@ -318,6 +326,16 @@ def _int(payload: Mapping[str, object], key: str, default: int) -> int:
     return value if value > 0 else default
 
 
+def _port(payload: Mapping[str, object], key: str, default: int) -> int:
+    try:
+        value = int(payload.get(key, default))
+    except (TypeError, ValueError):
+        value = default
+    if not (1 <= value <= 65535):
+        raise ValueError(f"{key} must be between 1 and 65535")
+    return value
+
+
 def _non_negative_int(payload: Mapping[str, object], key: str, default: int) -> int:
     try:
         value = int(payload.get(key, default))

{cluxion_agentplugin_preprocessing-0.3.6 → cluxion_agentplugin_preprocessing-0.3.8}/tests/test_doctor.py

@@ -24,7 +24,7 @@ def test_run_doctor_returns_result_and_deterministic():
         catalog_path=cat,
         probes=PROBES,
         plugin="preprocessing",
-        version="0.3.6",
+        version="0.3.7",
     )
     assert isinstance(r1, DoctorResult)
     j1 = render_json(r1)
@@ -33,7 +33,7 @@ def test_run_doctor_returns_result_and_deterministic():
         catalog_path=cat,
         probes=PROBES,
         plugin="preprocessing",
-        version="0.3.6",
+        version="0.3.7",
     )
     j2 = render_json(r2)
     assert j1 == j2  # byte identical
@@ -49,7 +49,7 @@ def test_cross_cutting_checks_present():
         catalog_path=cat,
         probes=PROBES,
         plugin="preprocessing",
-        version="0.3.6",
+        version="0.3.7",
     )
     statuses = {c.check_id: c.status for c in result.checks}
     for key in ("hermes_on_path", "entry_point_registered", "toolset_valid"):
@@ -57,6 +57,21 @@ def test_cross_cutting_checks_present():
         assert statuses[key] in ("pass", "warn", "fail", "skip")
 
 
+def test_new_probes_non_skip():
+    cat = _catalog_path()
+    result = run_doctor(
+        cwd=Path.cwd(),
+        catalog_path=cat,
+        probes=PROBES,
+        plugin="preprocessing",
+        version="0.3.7",
+    )
+    statuses = {c.check_id: c.status for c in result.checks}
+    for key in ("psutil_importable", "json_serialization_deterministic"):
+        assert key in statuses
+        assert statuses[key] in ("pass", "warn", "fail")  # non-skip
+
+
 def test_probe_exception_becomes_fail():
     def bad_probe(ctx):
         raise RuntimeError("boom")
@@ -66,7 +81,7 @@ def test_probe_exception_becomes_fail():
         catalog_path=_catalog_path(),
         probes={"hermes_on_path": bad_probe},
         plugin="preprocessing",
-        version="0.3.6",
+        version="0.3.7",
     )
     statuses = {c.check_id: c.status for c in result.checks}
     assert statuses["hermes_on_path"] == "fail"
@@ -78,5 +93,5 @@ def test_warn_only_is_ok():
     checks = (
         CheckResult(check_id="x", category="c", severity="medium", status="warn", detail="w"),
     )
-    r = DoctorResult(plugin="p", version="0.3.6", checks=checks)
+    r = DoctorResult(plugin="p", version="0.3.7", checks=checks)
     assert r.ok is True

{cluxion_agentplugin_preprocessing-0.3.6 → cluxion_agentplugin_preprocessing-0.3.8}/tests/test_runner.py

@@ -176,3 +176,38 @@ def _isolated_env(key: str, value: str) -> Iterator[None]:
             os.environ.pop(key, None)
         else:
             os.environ[key] = old_value
+
+
+def test_serve_local_rejects_out_of_range_port() -> None:
+    try:
+        runner.serve_local({"model": "test", "port": 999999999})
+    except ValueError as exc:
+        assert "port must be between 1 and 65535" in str(exc)
+    else:
+        raise AssertionError("expected ValueError")
+
+
+def test_hermes_config_rejects_invalid_port() -> None:
+    try:
+        runner.hermes_config({"model": "test", "port": 0})
+    except ValueError as exc:
+        assert "port must be between 1 and 65535" in str(exc)
+    else:
+        raise AssertionError("expected ValueError")
+
+
+def test_context_compress_rejects_malformed_messages() -> None:
+    try:
+        runner.context_compress({"messages": [42, "x", None, {"role": "user"}]})
+    except ValueError as exc:
+        assert "message[0] must be an object with string content" in str(exc)
+    else:
+        raise AssertionError("expected ValueError")
+
+
+def test_context_compress_accepts_valid_messages_structure() -> None:
+    try:
+        runner.context_compress({"messages": [{"content": "hello world"}]})
+    except ValueError as exc:
+        assert "must be an object with string content" not in str(exc)
+    # ok if other error like runtime missing