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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cluxion-agentplugin-preprocessing
-Version: 0.3.7
+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.7 → cluxion_agentplugin_preprocessing-0.3.8}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "cluxion-agentplugin-preprocessing"
-version = "0.3.7"
+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.7 → cluxion_agentplugin_preprocessing-0.3.8}/src/cluxion_agentplugin_preprocessing/__init__.py

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

{cluxion_agentplugin_preprocessing-0.3.7 → 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.7 → 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.7 → 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