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

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

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

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

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

{cluxion_agentplugin_supercoder-0.2.4 → cluxion_agentplugin_supercoder-0.2.6}/src/cluxion_agentplugin_supercoder/core/hash_patch.py

@@ -114,13 +114,29 @@ def _candidate_spans(text: str, reference: str, line_drift: int) -> list[tuple[i
 
 def _best_fuzzy_span(text: str, reference: str) -> tuple[int, int, str, float, bool] | None:
     best: tuple[int, int, str, float] | None = None
+    best_lines: tuple[int, int] | None = None
     ambiguous = False
+    lines = text.splitlines(keepends=True)
+    offsets = [0]
+    for line in lines:
+        offsets.append(offsets[-1] + len(line))
     for start, end, block in _candidate_spans(text, reference, MAX_LINE_DRIFT):
+        # compute line range [start_line, end_line) for overlap test
+        start_line = 0
+        while start_line < len(offsets) - 1 and offsets[start_line + 1] <= start:
+            start_line += 1
+        end_line = start_line
+        while end_line < len(offsets) - 1 and offsets[end_line] < end:
+            end_line += 1
         score = SequenceMatcher(None, block, reference, autojunk=False).ratio()
         if best is None or score > best[3]:
             best = (start, end, block, score)
+            best_lines = (start_line, end_line)
             ambiguous = False
         elif score >= DEFAULT_FUZZY_THRESHOLD and best and abs(score - best[3]) < 0.015:
+            # only treat as ambiguous if a genuinely different (non-overlapping) location matches closely
+            if best_lines is not None and not (end_line <= best_lines[0] or start_line >= best_lines[1]):
+                continue  # overlapping window on same location -> not real ambiguity
             ambiguous = True
     if best is None:
         return None

{cluxion_agentplugin_supercoder-0.2.4 → cluxion_agentplugin_supercoder-0.2.6}/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.6}/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.6}/tests/test_hash_patch.py

@@ -80,3 +80,39 @@ def test_sha256_prefix_and_bad_hash_rejected(tmp_path: Path) -> None:
     path.write_text("one\n", encoding="utf-8")
     with pytest.raises(ValueError):
         apply_patch(path, old_text="one\n", new_text="two\n", expected_file_hash="abc")
+
+
+def test_fuzzy_short_2line_block_with_drift_applies(tmp_path: Path) -> None:
+    # regression: 2-line unique target with minor drift (trailing space + typo) must apply
+    # even though different window widths produce overlapping candidates
+    path = tmp_path / "multi.py"
+    body = """def foo():
+    x = 1
+    return x
+
+def bar():
+    y = 2
+    return y
+"""
+    path.write_text(body, encoding="utf-8")
+    # drifted old_text: trailing space on first line of block, one-char typo on second
+    drifted = "    y = 2 \n    return z\n"
+    new = "    y = 42\n    return y\n"
+    result = apply_patch(path, old_text=drifted, new_text=new)
+    assert result.success is True
+    assert result.strategy == "fuzzy"
+    content = path.read_text(encoding="utf-8")
+    assert "y = 42" in content
+    assert "return y" in content
+    assert "def bar():" in content
+
+
+def test_duplicate_blocks_still_ambiguous(tmp_path: Path) -> None:
+    # genuine duplicate (non-overlapping) must still refuse
+    path = tmp_path / "dups.py"
+    block = "def f():\n    return 1\n"
+    path.write_text(block + "\n" + block, encoding="utf-8")
+    near = "def f():\n    return 2\n"
+    result = apply_patch(path, old_text=near, new_text="def f():\n    return 3\n")
+    assert result.success is False
+    assert result.strategy == "no_match"

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

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