cluxion-agentplugin-supercoder 0.2.3__tar.gz → 0.2.5__tar.gz

{cluxion_agentplugin_supercoder-0.2.3 → cluxion_agentplugin_supercoder-0.2.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cluxion-agentplugin-supercoder
-Version: 0.2.3
+Version: 0.2.5
 Summary: Universal agent coding harness plugin: cursor logic, safe patch, line budget, Rust index, test gates.
 Project-URL: Homepage, https://github.com/cluxion/cluxion-Agentplugin-supercoder
 Project-URL: Repository, https://github.com/cluxion/cluxion-Agentplugin-supercoder

{cluxion_agentplugin_supercoder-0.2.3 → cluxion_agentplugin_supercoder-0.2.5}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "cluxion-agentplugin-supercoder"
-version = "0.2.3"
+version = "0.2.5"
 description = "Universal agent coding harness plugin: cursor logic, safe patch, line budget, Rust index, test gates."
 readme = "README.md"
 requires-python = ">=3.11"
@@ -30,7 +30,6 @@ Issues = "https://github.com/cluxion/cluxion-Agentplugin-supercoder/issues"
 [tool.hatch.build.targets.wheel]
 packages = ["src/cluxion_agentplugin_supercoder"]
 
-
 [tool.pytest.ini_options]
 testpaths = ["tests"]
 addopts = "-q"

{cluxion_agentplugin_supercoder-0.2.3 → cluxion_agentplugin_supercoder-0.2.5}/src/cluxion_agentplugin_supercoder/__init__.py

@@ -1,5 +1,5 @@
 from __future__ import annotations
 
-__version__ = "0.2.3"
+__version__ = "0.2.5"
 
 __all__ = ["__version__"]

{cluxion_agentplugin_supercoder-0.2.3 → cluxion_agentplugin_supercoder-0.2.5}/src/cluxion_agentplugin_supercoder/core/lint_gate.py

@@ -46,7 +46,9 @@ def _discover_ruff() -> str | None:
 
 def check_file(path: str | Path, *, cwd: str | Path | None = None) -> dict[str, Any]:
     """Lint one file and return structured advisory findings."""
-    target = Path(path)
+    target = Path(path).resolve()
+    if target.is_dir():
+        return {"ok": False, "error": "path is a directory"}
     language = language_for_path(target)
     if language not in LINTABLE_LANGUAGES:
         return _unchecked(language or "", "no_linter")

{cluxion_agentplugin_supercoder-0.2.3 → cluxion_agentplugin_supercoder-0.2.5}/src/cluxion_agentplugin_supercoder/doctor/probes.py

@@ -3,8 +3,12 @@
 from __future__ import annotations
 
 import importlib.metadata
+import importlib.util
+import json
+import os
 import shutil
 from collections.abc import Callable
+from pathlib import Path
 
 from .framework import DoctorContext
 
@@ -115,4 +119,69 @@ def handler_exception_coverage(ctx: DoctorContext) -> tuple[str, str]:
         return "skip", f"cannot invoke guard: {e}"
 
 
+# NEW deterministic probes for previously-skipped catalog checks (import-avail, json-det, abi3/sqlite patterns adapted)
+# hermes_requirements_installed (import availability using find_spec to satisfy linter)
+@_register("hermes_requirements_installed")
+def hermes_requirements_installed(ctx: DoctorContext) -> tuple[str, str]:
+    try:
+        if importlib.util.find_spec("psutil") and importlib.util.find_spec("yaml"):
+            return "pass", "psutil+PyYAML importable"
+        return "warn", "missing dep"
+    except Exception as e:
+        return "skip", f"import check error: {e}"
+
+
+# repo_map_deterministic (json determinism + real call)
+@_register("repo_map_deterministic")
+def repo_map_deterministic(ctx: DoctorContext) -> tuple[str, str]:
+    try:
+        from cluxion_agentplugin_supercoder.core.repo_map import build_repo_map
+        m1 = build_repo_map(ctx.cwd, budget_chars=2000)
+        m2 = build_repo_map(ctx.cwd, budget_chars=2000)
+        def strip(d):
+            if isinstance(d, dict):
+                return {k: strip(v) for k, v in d.items() if k != "_stats"}
+            if isinstance(d, list):
+                return [strip(x) for x in d]
+            return d
+        if strip(m1) == strip(m2):
+            j1 = json.dumps(m1, sort_keys=True)
+            j2 = json.dumps(m2, sort_keys=True)
+            if j1 == j2:
+                return "pass", "deterministic + json roundtrip ok"
+            return "warn", "map match but json not"
+        return "fail", "non-deterministic"
+    except Exception as e:
+        return "skip", f"cannot run: {e}"
+
+
+# ruff_binary_discoverable (real env/path probe)
+@_register("ruff_binary_discoverable")
+def ruff_binary_discoverable(ctx: DoctorContext) -> tuple[str, str]:
+    try:
+        envb = os.environ.get("CLUXION_SUPERCODER_RUFF_BIN")
+        if envb and Path(envb).is_file():
+            return "pass", envb
+        cands = [Path(ctx.cwd)/".venv/bin/ruff", shutil.which("ruff")]
+        for c in cands:
+            if c and Path(c).is_file():
+                return "pass", str(c)
+        return "warn", "no ruff binary (advisory)"
+    except Exception as e:
+        return "skip", f"probe error: {e}"
+
+
+# file_hash_consistency (real check)
+@_register("file_hash_consistency")
+def file_hash_consistency(ctx: DoctorContext) -> tuple[str, str]:
+    try:
+        from cluxion_agentplugin_supercoder.core.hash_patch import _normalize_newlines, file_hash
+        c = 'a=1\r\nb=2'
+        if file_hash(c) == file_hash(_normalize_newlines(c)):
+            return "pass", "CRLF safe"
+        return "fail", "hash mismatch"
+    except Exception as e:
+        return "skip", f"hash error: {e}"
+
+
 # note: other checks in catalog will be reported as skip (no probe)

{cluxion_agentplugin_supercoder-0.2.3 → cluxion_agentplugin_supercoder-0.2.5}/src/cluxion_agentplugin_supercoder/plugin.py

@@ -1,7 +1,7 @@
 from __future__ import annotations
 
 import json
-from collections.abc import Callable
+from collections.abc import Callable, Mapping
 
 from cluxion_agentplugin_supercoder import runner
 from cluxion_agentplugin_supercoder.core.test_gate import suggest_test_commands
@@ -116,6 +116,7 @@ def register(ctx: object) -> None:
 
 def _wrap(callback: Callable[[dict[str, object]], runner.ToolResult]) -> Callable[[dict[str, object]], str]:
     def handler(args: dict[str, object], **_: object) -> str:
+        args = args if isinstance(args, Mapping) else {}
         try:
             return callback(args).to_json()
         except (ValueError, TypeError, OSError) as exc:

{cluxion_agentplugin_supercoder-0.2.3 → cluxion_agentplugin_supercoder-0.2.5}/tests/test_doctor.py

@@ -24,7 +24,7 @@ def test_run_doctor_returns_result_and_deterministic():
         catalog_path=cat,
         probes=PROBES,
         plugin="supercoder",
-        version="0.2.3",
+        version="0.2.4",
     )
     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="supercoder",
-        version="0.2.3",
+        version="0.2.4",
     )
     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="supercoder",
-        version="0.2.3",
+        version="0.2.4",
     )
     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,22 @@ 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="supercoder",
+        version="0.2.4",
+    )
+    statuses = {c.check_id: c.status for c in result.checks}
+    # assert at least two newly implemented return non-skip
+    new_checks = ["hermes_requirements_installed", "repo_map_deterministic", "ruff_binary_discoverable", "file_hash_consistency"]
+    non_skip_count = sum(1 for k in new_checks if k in statuses and statuses[k] != "skip")
+    assert non_skip_count >= 2, f"only {non_skip_count} new probes non-skip"
+
+
 def test_probe_exception_becomes_fail():
     def bad_probe(ctx):
         raise RuntimeError("boom")
@@ -66,7 +82,7 @@ def test_probe_exception_becomes_fail():
         catalog_path=_catalog_path(),
         probes={"hermes_on_path": bad_probe},
         plugin="supercoder",
-        version="0.2.3",
+        version="0.2.4",
     )
     statuses = {c.check_id: c.status for c in result.checks}
     assert statuses["hermes_on_path"] == "fail"
@@ -78,6 +94,6 @@ 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.2.3", checks=checks)
+    r = DoctorResult(plugin="p", version="0.2.4", checks=checks)
     assert r.ok is True
     # exit would be 0

{cluxion_agentplugin_supercoder-0.2.3 → cluxion_agentplugin_supercoder-0.2.5}/tests/test_lint_gate.py

@@ -95,3 +95,16 @@ def test_lint_gate_tool(tmp_path: Path) -> None:
     assert result.ok is True
     assert result.payload["clean"] is False
     assert result.payload["finding_count"] >= 1
+
+
+def test_lint_gate_rejects_directory(tmp_path: Path) -> None:
+    d = tmp_path / "emptydir"
+    d.mkdir()
+    result = lint_gate.check_file(d)
+    assert result.get("ok") is False
+    assert result.get("error") == "path is a directory"
+
+    # also via runner tool
+    res_tool = runner.lint_gate_tool({"cwd": str(tmp_path), "path": "emptydir"})
+    assert res_tool.ok is False
+    assert res_tool.payload.get("error") == "path is a directory"

{cluxion_agentplugin_supercoder-0.2.3 → cluxion_agentplugin_supercoder-0.2.5}/tests/test_plugin.py

@@ -1,3 +1,5 @@
+"""Tests for plugin registration and _wrap error handling."""
+
 from __future__ import annotations
 
 import json
@@ -5,18 +7,15 @@ import json
 from cluxion_agentplugin_supercoder import plugin, runner
 
 
-class FakeContext:
-    def __init__(self) -> None:
-        self.tools: dict[str, dict[str, object]] = {}
-
-    def register_tool(
-        self, *, name: str, toolset: str, schema: dict[str, object], handler: object, emoji: str = ""
-    ) -> None:
-        self.tools[name] = {"toolset": toolset, "schema": schema, "handler": handler, "emoji": emoji}
+def test_register_exposes_all_tools() -> None:
+    class FakeCtx:
+        def __init__(self):
+            self.tools = {}
 
+        def register_tool(self, name, toolset=None, schema=None, handler=None, emoji=None):
+            self.tools[name] = {"toolset": toolset, "schema": schema}
 
-def test_register_tools() -> None:
-    ctx = FakeContext()
+    ctx = FakeCtx()
     plugin.register(ctx)
     assert sorted(ctx.tools) == [
         "supercoder_brief",
@@ -46,6 +45,19 @@ def test_wrap_catches_isadirectory_and_typeerror() -> None:
     assert res_type == {"ok": False, "error": "bad int coercion"}
 
 
+def test_wrap_coerces_none_args_to_clean_error() -> None:
+    def any_handler(args):
+        # if args was None, before fix: AttributeError on .get inside tools
+        # now coerced to {}, so callback runs and may raise ValueError for missing keys
+        if not args:
+            raise ValueError("args required after coercion")
+        return runner.ToolResult(True, {"echo": args})
+
+    wrapped = plugin._wrap(any_handler)
+    res = json.loads(wrapped(None))
+    assert res == {"ok": False, "error": "args required after coercion"}
+
+
 def test_repo_map_deterministic(tmp_path) -> None:
     # ensure deterministic output (no varying cache stats in payload)
     from cluxion_agentplugin_supercoder.core import repo_map as rm

{cluxion_agentplugin_supercoder-0.2.3 → cluxion_agentplugin_supercoder-0.2.5}/uv.lock

@@ -160,7 +160,7 @@ wheels = [
 
 [[package]]
 name = "cluxion-agentplugin-supercoder"
-version = "0.2.3"
+version = "0.2.5"
 source = { editable = "." }
 dependencies = [
     { name = "psutil" },