sourcecode 1.33.9__tar.gz → 1.33.11__tar.gz

{sourcecode-1.33.9 → sourcecode-1.33.11}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sourcecode
-Version: 1.33.9
+Version: 1.33.11
 Summary: Persistent structural context and ultra-fast repeated analysis for AI coding agents
 License-File: LICENSE
 Keywords: agents,ai,codebase,context,developer-tools,llm
@@ -39,7 +39,7 @@ Description-Content-Type: text/markdown
 
 **Persistent structural context and ultra-fast repeated analysis for AI coding agents.**
 
-![Version](https://img.shields.io/badge/version-1.33.9-blue)
+![Version](https://img.shields.io/badge/version-1.33.11-blue)
 ![Python](https://img.shields.io/badge/python-3.10%2B-green)
 
 ---

{sourcecode-1.33.9 → sourcecode-1.33.11}/README.md

@@ -2,7 +2,7 @@
 
 **Persistent structural context and ultra-fast repeated analysis for AI coding agents.**
 
-![Version](https://img.shields.io/badge/version-1.33.9-blue)
+![Version](https://img.shields.io/badge/version-1.33.11-blue)
 ![Python](https://img.shields.io/badge/python-3.10%2B-green)
 
 ---

{sourcecode-1.33.9 → sourcecode-1.33.11}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "sourcecode"
-version = "1.33.9"
+version = "1.33.11"
 description = "Persistent structural context and ultra-fast repeated analysis for AI coding agents"
 readme = "README.md"
 requires-python = ">=3.9"

{sourcecode-1.33.9 → sourcecode-1.33.11}/src/sourcecode/__init__.py

@@ -1,3 +1,3 @@
 """sourcecode — Deterministic codebase context maps for AI coding agents."""
 
-__version__ = "1.33.9"
+__version__ = "1.33.11"

{sourcecode-1.33.9 → sourcecode-1.33.11}/src/sourcecode/cli.py

@@ -2677,10 +2677,11 @@ def prepare_context_cmd(
             # relevant_files goal: untested SOURCE files. Test files belong in test_gaps.
             # Without this filter, high-churn test files rank above untested source files.
             _rfs = [f for f in _rfs if getattr(f, "role", None) != "test"]
-        out["relevant_files"] = [
-            _serialize_relevant_file(f)
-            for f in _rfs
-        ]
+        _serialized_rfs = [_serialize_relevant_file(f) for f in _rfs]
+        out["relevant_files"] = _serialized_rfs
+        if task == "fix-bug":
+            # ranked_files was the v1 name for this field — emit as backward-compat alias.
+            out["ranked_files"] = _serialized_rfs
     if _task_include("key_dependencies") and output.key_dependencies:
         out["key_dependencies"] = output.key_dependencies
     if _task_include("gaps") and output.gaps:
@@ -3291,13 +3292,33 @@ def impact_cmd(
     )
     from sourcecode.output_budget import trim_to_budget as _trim, BUDGET_IMPACT
 
+    import sys as _sys_ic
+    # Legacy-compat: old syntax was `impact <path> <target>`.
+    # Detect: target resolves to an existing directory (not a class name), and
+    # the path arg is not a valid directory (looks like a class name).
+    _target_as_path = Path(target)
+    if _target_as_path.is_dir() and not path.resolve().is_dir():
+        # Gate on isatty() — non-TTY (MCP, pipes) must not receive text mixed into JSON stdout.
+        if getattr(_sys_ic.stderr, "isatty", lambda: False)():
+            _sys_ic.stderr.write(
+                f"[impact] Legacy argument order detected: '{target}' is a directory, not a class name.\n"
+                f"[impact] Swapping: target='{path}', path='{target}'. "
+                f"New syntax: sourcecode impact <target> [path]\n"
+            )
+            _sys_ic.stderr.flush()
+        target, path = str(path), _target_as_path
+
     root = path.resolve()
     if not root.is_dir():
         _emit_error_json(
             INVALID_INPUT_CODE,
             f"'{root}' is not a valid directory.",
             path=str(root),
-            hint="Pass an existing repository directory.",
+            hint=(
+                "Pass an existing repository directory as the second argument. "
+                "New syntax: sourcecode impact <target> [path] — "
+                "target is the class name, path is the repo root."
+            ),
             expected="A directory path.",
         )
         raise typer.Exit(1)
@@ -3621,11 +3642,15 @@ def fix_bug_cmd(
       sourcecode onboard .         — Full architecture context first
     """
     if not symptom:
-        typer.echo(
-            "[fix-bug] Results are significantly better with --symptom. "
-            "Example: --symptom 'NullPointerException in PaymentService'",
-            err=True,
-        )
+        import sys as _sys_fb
+        # Only emit advisory to interactive terminals — non-TTY (MCP, pipes, scripts)
+        # must never receive informational text mixed into JSON stdout.
+        if getattr(_sys_fb.stderr, "isatty", lambda: False)():
+            typer.echo(
+                "[fix-bug] Results are significantly better with --symptom. "
+                "Example: --symptom 'NullPointerException in PaymentService'",
+                err=True,
+            )
     ctx.invoke(
         prepare_context_cmd,
         task="fix-bug",
@@ -3904,8 +3929,23 @@ def activate_cmd(
 
 @app.command("version")
 def version_cmd() -> None:
-    """Show version and exit."""
-    typer.echo(f"sourcecode {__version__}")
+    """Show version and exit.
+
+    Outputs human-readable text on interactive terminals.
+    Outputs structured JSON on non-TTY (MCP, pipes, scripts):
+      {"cli_version": "1.33.11", "mcp_schema_version": "1.33.11",
+       "compatibility_schema_version": "1.0"}
+    """
+    import sys as _sys_ver
+    if getattr(_sys_ver.stdout, "isatty", lambda: False)():
+        typer.echo(f"sourcecode {__version__}")
+    else:
+        import json as _json_ver
+        typer.echo(_json_ver.dumps({
+            "cli_version": __version__,
+            "mcp_schema_version": __version__,
+            "compatibility_schema_version": "1.0",
+        }, ensure_ascii=False))
 
 
 # ── config ────────────────────────────────────────────────────────────────────
@@ -4351,6 +4391,46 @@ def mcp_remove(
     typer.echo("  Re-add:  sourcecode mcp init")
 
 
+@mcp_app.command("list-tools")
+def mcp_list_tools(
+    json_output: bool = typer.Option(False, "--json", help="Output as JSON."),
+) -> None:
+    """List all MCP tools exposed by the sourcecode server.
+
+    \b
+    Shows each tool name, its description, and the CLI command it maps to.
+    Useful for discovering capabilities when using sourcecode as an MCP server.
+
+    \b
+    Examples:
+      sourcecode mcp list-tools
+      sourcecode mcp list-tools --json
+    """
+    import asyncio
+    import json as _json
+
+    from sourcecode.mcp.server import mcp as _mcp
+
+    tools = asyncio.run(_mcp.list_tools())
+    tools_sorted = sorted(tools, key=lambda t: t.name)
+
+    if json_output:
+        payload = [
+            {"name": t.name, "description": (t.description or "").strip()}
+            for t in tools_sorted
+        ]
+        typer.echo(_json.dumps(payload, indent=2, ensure_ascii=False))
+        return
+
+    typer.echo(f"sourcecode MCP tools ({len(tools_sorted)} available)\n")
+    for t in tools_sorted:
+        desc_first_line = (t.description or "").strip().splitlines()[0] if t.description else ""
+        typer.echo(f"  {t.name:<35} {desc_first_line}")
+    typer.echo("")
+    typer.echo("Use:  sourcecode mcp serve   — start MCP server on stdio")
+    typer.echo("Use:  sourcecode mcp init    — configure MCP client")
+
+
 # ── Cache subcommands ─────────────────────────────────────────────────────────
 
 

{sourcecode-1.33.9 → sourcecode-1.33.11}/src/sourcecode/mcp/server.py

@@ -959,9 +959,11 @@ _TELEMETRY_ACTIONS = frozenset({"status", "enable", "disable"})
 
 @mcp.tool()
 def version() -> dict:
-    """Print sourcecode CLI version.
+    """Return sourcecode version and MCP compatibility metadata.
 
     Maps to: sourcecode version
+    Returns structured JSON: cli_version, mcp_schema_version, compatibility_schema_version.
+    cli_version and mcp_schema_version are always identical (released together).
     """
     return _execute(["version"])
 

{sourcecode-1.33.9 → sourcecode-1.33.11}/src/sourcecode/prepare_context.py

@@ -1225,8 +1225,8 @@ class TaskContextBuilder:
                 # Distinguish: no_staged_changes (CI, no --since) vs no_diff (empty range)
                 if _pr_scope_source == "no_staged_changes":
                     _no_diff_msg = (
-                        "No --since ref provided and no staged changes found. "
-                        "Use --since <ref>"
+                        "No --since ref provided and no staged/uncommitted changes found. "
+                        "Provide --since <ref> to specify the base commit for the diff."
                     )
                     return TaskOutput(
                         task="review-pr", goal=spec.goal,
@@ -1239,7 +1239,8 @@ class TaskContextBuilder:
                         error_message=_no_diff_msg,
                         error_hints=[
                             "Add --since <ref> to specify a base commit.",
-                            "Examples: --since origin/main  |  --since HEAD~3  |  --since main",
+                            "Common values: --since HEAD~1 (last commit)  |  --since origin/main  |  --since main",
+                            "If reviewing uncommitted changes: stage them first (git add), then run without --since.",
                         ],
                         gaps=[_no_diff_msg],
                         ci_decision="no_diff_source",

{sourcecode-1.33.9 → sourcecode-1.33.11}/src/sourcecode/repository_ir.py

@@ -232,6 +232,12 @@ _INJECT_ANNOTATIONS: frozenset[str] = frozenset({
     "@Autowired", "@Inject", "@Value", "@Qualifier", "@Resource",
 })
 
+# Lombok annotations that generate constructors injecting fields
+_LOMBOK_CTOR_ANNOTATIONS: frozenset[str] = frozenset({
+    "@RequiredArgsConstructor",  # injects private final fields
+    "@AllArgsConstructor",       # injects all non-static fields
+})
+
 _JAVA_ROLE_MAP: dict[str, str] = {
     # Spring MVC / Spring Boot
     "@RestController": "controller",
@@ -934,6 +940,58 @@ def _build_relations(
                     )},
                 ))
 
+    # ── Constructor injection ─────────────────────────────────────────────────
+    # Spring 4.3+ omits @Autowired when there is a single constructor.
+    # Both annotated and bare constructors get injects edges from ClassName#<init>
+    # to each resolvable parameter type so the reverse graph can propagate impact.
+    for sym in symbols:
+        if sym.symbol_kind != "constructor" or not sym.param_types:
+            continue
+        for simple_type in sym.param_types:
+            base = re.sub(r'<.*', '', simple_type).strip()
+            fqn = import_map.get(base)
+            if fqn:
+                edges.append(RelationEdge(
+                    from_symbol=sym.symbol,
+                    to_symbol=fqn,
+                    type="injects",
+                    confidence="high",
+                    evidence={"type": "constructor_param", "value": simple_type},
+                ))
+
+    # ── Lombok constructor injection ──────────────────────────────────────────
+    # @RequiredArgsConstructor: injects all private final fields.
+    # @AllArgsConstructor: injects all non-static fields.
+    # No explicit constructor symbol exists; edges are emitted from the class FQN.
+    for sym in symbols:
+        if sym.type not in ("class", "interface"):
+            continue
+        _has_req = "@RequiredArgsConstructor" in sym.annotations
+        _has_all = "@AllArgsConstructor" in sym.annotations
+        if not (_has_req or _has_all):
+            continue
+        _lombok_ann = "@RequiredArgsConstructor" if _has_req else "@AllArgsConstructor"
+        for _line in source.splitlines():
+            fld = _FIELD_DECL_RE.match(_line.strip())
+            if not fld:
+                continue
+            _mods = _parse_modifier_str(fld.group("modifiers") or "")
+            if "static" in _mods:
+                continue
+            if _has_req and "final" not in _mods:
+                continue
+            _ftype = fld.group("type").strip()
+            _base = re.sub(r'<.*', '', _ftype).strip()
+            _fqn = import_map.get(_base)
+            if _fqn:
+                edges.append(RelationEdge(
+                    from_symbol=sym.symbol,
+                    to_symbol=_fqn,
+                    type="injects",
+                    confidence="medium",
+                    evidence={"type": "lombok_constructor", "value": _lombok_ann},
+                ))
+
     for m in re.finditer(
         r'(?:class|interface)\s+(\w+)(?:\s+extends\s+([\w.<>?,\s]+?))?'
         r'(?:\s+implements\s+([\w.<>?,\s]+?))?\s*\{',