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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cluxion-agentplugin-supercoder
-Version: 0.2.4
+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.4 → cluxion_agentplugin_supercoder-0.2.5}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "cluxion-agentplugin-supercoder"
-version = "0.2.4"
+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.4 → cluxion_agentplugin_supercoder-0.2.5}/src/cluxion_agentplugin_supercoder/__init__.py

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

{cluxion_agentplugin_supercoder-0.2.4 → 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.4 → 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.4 → 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.4 → 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.4 → cluxion_agentplugin_supercoder-0.2.5}/uv.lock

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