cluxion-agentplugin-preprocessing 0.3.19__tar.gz → 0.3.21__tar.gz

{cluxion_agentplugin_preprocessing-0.3.19 → cluxion_agentplugin_preprocessing-0.3.21}/PKG-INFO

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

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "cluxion-agentplugin-preprocessing"
-version = "0.3.19"
+version = "0.3.21"
 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.19 → cluxion_agentplugin_preprocessing-0.3.21}/rust/cluxion_queue/src/context.rs

@@ -46,10 +46,12 @@ struct Msg {
 }
 
 pub fn compress(payload: &Value) -> Result<Value, QueueError> {
-    let raw_messages = payload
-        .get("messages")
-        .and_then(Value::as_array)
-        .ok_or_else(|| QueueError::Usage("missing required field: messages".into()))?;
+    let raw_messages = match payload.get("messages") {
+        None => return Err(QueueError::Usage("missing required field: messages".into())),
+        Some(value) => value.as_array().ok_or_else(|| {
+            QueueError::Usage("messages must be a list".into())
+        })?,
+    };
     let mut messages: Vec<Msg> = Vec::with_capacity(raw_messages.len());
     for raw in raw_messages {
         messages.push(Msg {

{cluxion_agentplugin_preprocessing-0.3.19 → cluxion_agentplugin_preprocessing-0.3.21}/src/cluxion_agentplugin_preprocessing/plugin.yaml

@@ -1,5 +1,5 @@
 name: cluxion-agentplugin-preprocessing
-version: 0.3.19
+version: 0.3.20
 description: "Universal agent preprocessing plugin: honesty contracts, clarification, Rust work queue, resource-aware harness handoff. Connected AI calls cluxion tools; plugin does not own models."
 author: cluxion
 kind: standalone

{cluxion_agentplugin_preprocessing-0.3.19 → cluxion_agentplugin_preprocessing-0.3.21}/src/cluxion_runtime/core/context_compress.py

@@ -58,8 +58,10 @@ class _Msg:
 
 def compress(payload: Mapping[str, object]) -> dict[str, object]:
     raw_messages = payload.get("messages")
-    if not isinstance(raw_messages, list):
+    if raw_messages is None:
         raise RuntimeError("missing required field: messages")
+    if not isinstance(raw_messages, list):
+        raise RuntimeError("messages must be a list")
     messages = [
         _Msg(
             role=str(raw.get("role", "user")) if isinstance(raw, dict) else "user",

{cluxion_agentplugin_preprocessing-0.3.19 → cluxion_agentplugin_preprocessing-0.3.21}/src/cluxion_runtime/core/hybrid_forget.py

@@ -6,9 +6,11 @@ is removed permanently. Stage 3 is Python-only; the Rust mirror does not run thi
 
 from __future__ import annotations
 
+import os
 import re
 import shutil
 import subprocess
+import sys
 import uuid
 from dataclasses import dataclass
 from typing import TYPE_CHECKING, Protocol
@@ -18,7 +20,14 @@ from cluxion_runtime.core.preprocess import estimate_tokens
 if TYPE_CHECKING:
     from collections.abc import Sequence
 
-_FORGETFORGE_BIN = "forgetforge"
+def _resolve_bin(name: str) -> str:
+    candidate = os.path.join(os.path.dirname(sys.executable), name)
+    if os.path.isfile(candidate) and os.access(candidate, os.X_OK):
+        return candidate
+    return shutil.which(name) or name
+
+
+_FORGETFORGE_BIN = _resolve_bin("forgetforge")
 _JUNK_MARKERS = ("[cluxion: duplicate", "[cluxion digest]")
 _IMPORTANCE_KEYWORDS = (
     "decide",
@@ -75,7 +84,9 @@ class ForgetResult:
 
 
 def forgetforge_available() -> bool:
-    return shutil.which(_FORGETFORGE_BIN) is not None
+    return (os.path.isfile(_FORGETFORGE_BIN) and os.access(_FORGETFORGE_BIN, os.X_OK)) or (
+        shutil.which(_FORGETFORGE_BIN) is not None
+    )
 
 
 def apply_hybrid_forget(

{cluxion_agentplugin_preprocessing-0.3.19 → cluxion_agentplugin_preprocessing-0.3.21}/src/cluxion_runtime/core/llm_compress.py

@@ -13,13 +13,23 @@ import os
 import re
 import shutil
 import subprocess
+import sys
 from typing import TYPE_CHECKING, Protocol
 
 if TYPE_CHECKING:
     from collections.abc import Mapping, Sequence
 
 DEFAULT_TIMEOUT_S = 120.0
-_HERMES_BIN = "hermes"
+
+
+def _resolve_bin(name: str) -> str:
+    candidate = os.path.join(os.path.dirname(sys.executable), name)
+    if os.path.isfile(candidate) and os.access(candidate, os.X_OK):
+        return candidate
+    return shutil.which(name) or name
+
+
+_HERMES_BIN = _resolve_bin("hermes")
 logger = logging.getLogger(__name__)
 
 _SUMMARY_INSTRUCTIONS = (
@@ -54,7 +64,9 @@ class _MessageLike(Protocol):
 
 
 def hermes_available() -> bool:
-    return shutil.which(_HERMES_BIN) is not None
+    return (os.path.isfile(_HERMES_BIN) and os.access(_HERMES_BIN, os.X_OK)) or (
+        shutil.which(_HERMES_BIN) is not None
+    )
 
 
 def summarize_messages(

cluxion_agentplugin_preprocessing-0.3.21/tests/runtime/test_bin_resolution.py

@@ -0,0 +1,103 @@
+"""PATH-independent resolution for forgetforge and hermes console scripts."""
+
+from __future__ import annotations
+
+import os
+import stat
+import sys
+from pathlib import Path
+
+import pytest
+
+from cluxion_runtime.core import hybrid_forget, llm_compress
+
+
+def _write_executable(path: Path) -> None:
+    path.write_text("#!/bin/sh\necho ok\n")
+    path.chmod(path.stat().st_mode | stat.S_IXUSR)
+
+
+@pytest.fixture
+def isolated_bin_dir(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> Path:
+    """Temp venv bin dir; PATH excludes it so only sys.executable dir can resolve bins."""
+    bin_dir = tmp_path / "bin"
+    bin_dir.mkdir()
+    monkeypatch.setenv("PATH", str(tmp_path / "empty"))
+    monkeypatch.setattr(sys, "executable", str(bin_dir / "python"))
+    return bin_dir
+
+
+def test_forgetforge_available_via_venv_bin_without_path(
+    isolated_bin_dir: Path,
+    monkeypatch: pytest.MonkeyPatch,
+) -> None:
+    forgetforge = isolated_bin_dir / "forgetforge"
+    _write_executable(forgetforge)
+    monkeypatch.setattr(hybrid_forget, "_FORGETFORGE_BIN", hybrid_forget._resolve_bin("forgetforge"))
+
+    assert hybrid_forget.forgetforge_available() is True
+    assert os.path.isabs(hybrid_forget._FORGETFORGE_BIN)
+    assert str(forgetforge) == hybrid_forget._FORGETFORGE_BIN
+
+
+def test_forgetforge_unavailable_when_missing_everywhere(
+    isolated_bin_dir: Path,
+    monkeypatch: pytest.MonkeyPatch,
+) -> None:
+    monkeypatch.setattr(hybrid_forget, "_FORGETFORGE_BIN", hybrid_forget._resolve_bin("forgetforge"))
+    assert hybrid_forget.forgetforge_available() is False
+
+
+def test_forgetforge_available_via_path_fallback(
+    tmp_path: Path,
+    monkeypatch: pytest.MonkeyPatch,
+) -> None:
+    bin_dir = tmp_path / "on-path"
+    bin_dir.mkdir()
+    forgetforge = bin_dir / "forgetforge"
+    _write_executable(forgetforge)
+
+    monkeypatch.setenv("PATH", str(bin_dir))
+    monkeypatch.setattr(sys, "executable", str(tmp_path / "venv" / "bin" / "python"))
+    monkeypatch.setattr(hybrid_forget, "_FORGETFORGE_BIN", hybrid_forget._resolve_bin("forgetforge"))
+
+    assert hybrid_forget.forgetforge_available() is True
+    assert str(forgetforge) == hybrid_forget._FORGETFORGE_BIN
+
+
+def test_hermes_available_via_venv_bin_without_path(
+    isolated_bin_dir: Path,
+    monkeypatch: pytest.MonkeyPatch,
+) -> None:
+    hermes = isolated_bin_dir / "hermes"
+    _write_executable(hermes)
+    monkeypatch.setattr(llm_compress, "_HERMES_BIN", llm_compress._resolve_bin("hermes"))
+
+    assert llm_compress.hermes_available() is True
+    assert os.path.isabs(llm_compress._HERMES_BIN)
+    assert str(hermes) == llm_compress._HERMES_BIN
+
+
+def test_hermes_unavailable_when_missing_everywhere(
+    isolated_bin_dir: Path,
+    monkeypatch: pytest.MonkeyPatch,
+) -> None:
+    monkeypatch.setattr(llm_compress, "_HERMES_BIN", llm_compress._resolve_bin("hermes"))
+    assert llm_compress.hermes_available() is False
+
+
+def test_hermes_available_via_path_fallback(
+    tmp_path: Path,
+    monkeypatch: pytest.MonkeyPatch,
+) -> None:
+    bin_dir = tmp_path / "on-path"
+    bin_dir.mkdir()
+    hermes = bin_dir / "hermes"
+    _write_executable(hermes)
+
+    monkeypatch.setenv("PATH", str(bin_dir))
+    monkeypatch.setattr(sys, "executable", str(tmp_path / "venv" / "bin" / "python"))
+    monkeypatch.setattr(llm_compress, "_HERMES_BIN", llm_compress._resolve_bin("hermes"))
+
+    assert llm_compress.hermes_available() is True
+    assert str(hermes) == llm_compress._HERMES_BIN

{cluxion_agentplugin_preprocessing-0.3.19 → cluxion_agentplugin_preprocessing-0.3.21}/tests/runtime/test_context_compress.py

@@ -177,5 +177,10 @@ def test_tool_path_continues_when_stage1_above_trigger(monkeypatch) -> None:
 
 
 def test_missing_messages_raises(backend) -> None:
-    with pytest.raises(RuntimeError, match="messages"):
+    with pytest.raises(RuntimeError, match="missing required field: messages"):
         queue_bridge.compress_context({"context_limit_tokens": 1000})
+
+
+def test_non_list_messages_raises(backend) -> None:
+    with pytest.raises(RuntimeError, match="messages must be a list"):
+        queue_bridge.compress_context({"messages": "not-a-list", "context_limit_tokens": 1000})

{cluxion_agentplugin_preprocessing-0.3.19 → cluxion_agentplugin_preprocessing-0.3.21}/tests/runtime/test_context_compress_llm_forget.py

@@ -9,7 +9,11 @@ import pytest
 
 from cluxion_runtime.core import context_compress
 from cluxion_runtime.core.context_compress import _Msg
-from cluxion_runtime.core.hybrid_forget import _cold_demote, apply_hybrid_forget
+from cluxion_runtime.core.hybrid_forget import (
+    _FORGETFORGE_BIN,
+    _cold_demote,
+    apply_hybrid_forget,
+)
 from cluxion_runtime.core.preprocess import estimate_tokens
 
 
@@ -95,7 +99,13 @@ def test_cold_demote_returns_true_on_successful_store_only(monkeypatch) -> None:
 
     assert _cold_demote("recoverable content", "sess-abc123") is True
     assert len(calls) == 1
-    assert calls[0] == ["forgetforge", "store", "sess-abc123", "--content", "recoverable content"]
+    assert calls[0] == [
+        _FORGETFORGE_BIN,
+        "store",
+        "sess-abc123",
+        "--content",
+        "recoverable content",
+    ]
 
 
 def test_cold_demote_returns_false_when_store_fails(monkeypatch) -> None: