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

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

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

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

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

{cluxion_agentplugin_preprocessing-0.3.7 → cluxion_agentplugin_preprocessing-0.3.9}/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.9}/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.9}/src/cluxion_runtime/core/clarification.py

@@ -9,6 +9,7 @@ mixed-language on purpose — they classify user input, they are not output.
 from __future__ import annotations
 
 import os
+import re
 from dataclasses import dataclass
 
 from cluxion_runtime.core.types import WorkIntent, WorkItem
@@ -125,11 +126,10 @@ def _question(question_id: str, locale: str) -> ClarificationQuestion:
     return ClarificationQuestion(question_id, prompt, why)
 
 
-_AMBIGUOUS_KEYWORDS = (
+# PHRASES: multi-word or longer markers — use plain substring match
+_AMBIGUOUS_PHRASES = (
     "maybe",
     "perhaps",
-    "either",
-    "or",
     "not sure",
     "unsure",
     "아마",
@@ -139,6 +139,18 @@ _AMBIGUOUS_KEYWORDS = (
     "헷갈",
     "애매",
 )
+# WORDS: short standalone tokens — match on whole-word boundaries only
+_AMBIGUOUS_WORDS = ("or", "either")
+
+
+def _has_ambiguous(text: str) -> bool:
+    """Return True if text contains ambiguous phrasing or choice words."""
+    t = text.lower()
+    if any(p in t for p in _AMBIGUOUS_PHRASES):
+        return True
+    return any(re.search(r"\b" + re.escape(w) + r"\b", t) for w in _AMBIGUOUS_WORDS)
+
+
 _SCOPE_KEYWORDS = ("all", "everything", "전부", "전체", "모든")
 _TARGET_MISSING_KEYWORDS = ("fix", "implement", "refactor", "patch", "수정", "구현", "리팩터", "패치")
 _LOW_CONFIDENCE_THRESHOLD = 0.62
@@ -155,7 +167,7 @@ def assess_clarification(item: WorkItem, intent: WorkIntent) -> ClarificationRes
         reasons.append("low_intent_confidence")
         questions.append(_question("intent_direction", locale))
 
-    if _has_any(text, _AMBIGUOUS_KEYWORDS):
+    if _has_ambiguous(text):
         reasons.append("ambiguous_language")
         questions.append(_question("disambiguate_choice", locale))
 
@@ -204,7 +216,7 @@ def assess_clarification(item: WorkItem, intent: WorkIntent) -> ClarificationRes
 def _needs_direction_confirmation(text: str, intent: WorkIntent) -> bool:
     if intent.category == "general" and len(text) < 240:
         return False
-    if _has_any(text, _AMBIGUOUS_KEYWORDS):
+    if _has_ambiguous(text):
         return True
     if intent.category in {"engineering", "security", "documentation", "local_model"}:
         return True

{cluxion_agentplugin_preprocessing-0.3.7 → cluxion_agentplugin_preprocessing-0.3.9}/tests/runtime/test_clarification.py

@@ -68,3 +68,28 @@ def test_resolve_locale_fallbacks() -> None:
     assert clarification.resolve_locale("한국어 텍스트") == "ko"
     assert clarification.resolve_locale("plain", explicit="ko-KR") == "ko"
     assert clarification.resolve_locale("plain", explicit="fr") == "en"
+
+
+# Regression tests for word-boundary fix on 'or'/'either'
+def test_clear_prompts_no_false_positive_clarification() -> None:
+    clear_prompts = [
+        "Generate a sales report",
+        "Optimize the code base",
+        "sort a list of numbers",
+        "What is the hex code for the color blue?",
+    ]
+    for prompt in clear_prompts:
+        item = WorkItem(f"w-clear-{prompt[:10]}", prompt)
+        plan = build_harness_plan(item)
+        assert plan.clarification_required is False, f"False positive on: {prompt}"
+
+
+def test_real_or_choice_still_requires_clarification() -> None:
+    or_prompts = [
+        "Should I use X or Y?",
+        "do A or B?",
+    ]
+    for prompt in or_prompts:
+        item = WorkItem(f"w-or-{prompt[:10]}", prompt)
+        plan = build_harness_plan(item)
+        assert plan.clarification_required is True, f"Missed ambiguity on: {prompt}"

{cluxion_agentplugin_preprocessing-0.3.7 → cluxion_agentplugin_preprocessing-0.3.9}/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