sourcecode 1.31.28__py3-none-any.whl → 1.31.30__py3-none-any.whl

sourcecode/__init__.py

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

sourcecode/cli.py

@@ -760,8 +760,11 @@ def main(
             err=True,
         )
 
-    # P0-2 FIX: --changed-only silently implies --compact; inform the user.
-    if changed_only and not compact and not agent:
+    # P1-2 FIX: --changed-only silently implies --compact; inform only on TTY.
+    # PowerShell 5.1 interprets any stderr write (even with exit 0) as NativeCommandError.
+    # Gate on isatty() so pipeline consumers never see informational noise on stderr.
+    import sys as _sys_tty
+    if changed_only and not compact and not agent and _sys_tty.stderr.isatty():
         typer.echo(
             "[info] --changed-only implies --compact (bounding output to changed files).",
             err=True,
@@ -1952,6 +1955,10 @@ def main(
             if _copy_to_clipboard(content):
                 typer.echo("✓ copied to clipboard", err=True)
 
+    # 8. One-time MCP setup nudge (stderr only — does not affect exit code or stdout)
+    from sourcecode.mcp_nudge import nudge_mcp_if_needed as _nudge
+    _nudge()
+
 
 # ── prepare-context output helpers ────────────────────────────────────────────
 
@@ -2180,7 +2187,11 @@ def prepare_context_cmd(
 
     target = path.resolve()
     if not target.exists() or not target.is_dir():
-        typer.echo(f"Error: '{target}' is not a valid directory.", err=True)
+        _emit_error_json(
+            "invalid_path",
+            f"'{target}' is not a valid directory.",
+            path=str(target),
+        )
         raise typer.Exit(code=1)
 
     if dry_run:
@@ -2312,6 +2323,13 @@ def prepare_context_cmd(
         out["improvement_opportunities"] = output.improvement_opportunities
     if _task_include("test_gaps") and output.test_gaps:
         out["test_gaps"] = output.test_gaps
+    # P0-2: fast-mode truncation transparency — always emit when truncated, even if test_gaps is []
+    # Use `is True` (strict) so MagicMock objects in tests don't trigger this branch.
+    if getattr(output, "truncated", False) is True:
+        out["truncated"] = True
+        _tr = getattr(output, "truncated_reason", None)
+        if isinstance(_tr, str) and _tr:
+            out["truncated_reason"] = _tr
     if _task_include("code_notes_summary") and output.code_notes_summary:
         out["code_notes_summary"] = output.code_notes_summary
     if _task_include("changed_files") and output.changed_files:
@@ -2554,6 +2572,9 @@ def prepare_context_cmd(
             if _copy_to_clipboard(_pc_content):
                 typer.echo("✓ copied to clipboard", err=True)
 
+    from sourcecode.mcp_nudge import nudge_mcp_if_needed as _nudge
+    _nudge()
+
 
 # ── Telemetry commands ────────────────────────────────────────────────────────
 
@@ -2697,7 +2718,11 @@ def repo_ir_cmd(
 
     root = path.resolve()
     if not root.is_dir():
-        typer.echo(f"Error: {root} is not a directory", err=True)
+        _emit_error_json(
+            "invalid_path",
+            f"'{root}' is not a valid directory.",
+            path=str(root),
+        )
         raise typer.Exit(1)
 
     if files:
@@ -2849,7 +2874,11 @@ def impact_cmd(
 
     root = path.resolve()
     if not root.is_dir():
-        typer.echo(f"Error: {root} is not a directory", err=True)
+        _emit_error_json(
+            "invalid_path",
+            f"'{root}' is not a valid directory.",
+            path=str(root),
+        )
         raise typer.Exit(1)
 
     file_list = find_java_files(root)
@@ -2899,6 +2928,9 @@ def impact_cmd(
     if result.get("resolution") == "not_found":
         raise typer.Exit(code=1)
 
+    from sourcecode.mcp_nudge import nudge_mcp_if_needed as _nudge
+    _nudge()
+
 
 # ── endpoints ─────────────────────────────────────────────────────────────────
 
@@ -2949,7 +2981,11 @@ def endpoints_cmd(
 
     target = path.resolve()
     if not target.exists() or not target.is_dir():
-        typer.echo(f"Error: '{target}' is not a valid directory.", err=True)
+        _emit_error_json(
+            "invalid_path",
+            f"'{target}' is not a valid directory.",
+            path=str(target),
+        )
         raise typer.Exit(code=1)
 
     data = _extract_java_endpoints(target)
@@ -2969,6 +3005,9 @@ def endpoints_cmd(
             if _copy_to_clipboard(output):
                 typer.echo("✓ copied to clipboard", err=True)
 
+    from sourcecode.mcp_nudge import nudge_mcp_if_needed as _nudge
+    _nudge()
+
 
 # ── Enterprise Workflow Commands ──────────────────────────────────────────────
 #
@@ -3062,7 +3101,10 @@ def review_pr_cmd(
         help="Copy output to clipboard after a successful run.",
     ),
 ) -> None:
-    """[Pro] PR review: blast radius, risk ranking, execution paths, security/txn impact.
+    """[Pro*] PR review: blast radius, risk ranking, execution paths, security/txn impact.
+
+    Note: [Pro*] label is reserved for a future licensing gate. This command currently
+    runs without authentication. Behavior may change in a future version.
 
     \b
     Answers: "What does this PR break and how risky is it?"
@@ -3122,7 +3164,10 @@ def fix_bug_cmd(
         help="Copy output to clipboard after a successful run.",
     ),
 ) -> None:
-    """[Pro] Bug triage: risk-ranked files, suspected areas, related annotations.
+    """[Pro*] Bug triage: risk-ranked files, suspected areas, related annotations.
+
+    Note: [Pro*] label is reserved for a future licensing gate. This command currently
+    runs without authentication. Behavior may change in a future version.
 
     \b
     Answers: "Where in this codebase should I look to fix this symptom?"
@@ -3174,7 +3219,10 @@ def modernize_cmd(
         help="Copy output to clipboard after a successful run.",
     ),
 ) -> None:
-    """[Pro] Modernization planning: coupling, dead zones, risky modules, refactor candidates.
+    """[Pro*] Modernization planning: coupling, dead zones, risky modules, refactor candidates.
+
+    Note: [Pro*] label is reserved for a future licensing gate. This command currently
+    runs without authentication. Behavior may change in a future version.
 
     \b
     Answers: "Where should I refactor first, and what's safest to touch?"
@@ -3205,7 +3253,11 @@ def modernize_cmd(
 
     root = path.resolve()
     if not root.is_dir():
-        typer.echo(f"Error: {root} is not a directory", err=True)
+        _emit_error_json(
+            "invalid_path",
+            f"'{root}' is not a valid directory.",
+            path=str(root),
+        )
         raise typer.Exit(1)
 
     file_list = find_java_files(root)
@@ -3430,6 +3482,24 @@ def mcp_serve() -> None:
     from sourcecode.mcp.server import mcp as _mcp
 
     log = logging.getLogger(__name__)
+
+    # P0-1: Strip UTF-8 BOM from stdin.buffer before the MCP server reads it.
+    # PowerShell 5.1 on Windows writes \xEF\xBB\xBF at the start of stdin,
+    # which breaks JSON parsing at line 1 column 1.
+    # peek(3) loads bytes into BufferedReader's internal buffer without consuming;
+    # read(3) discards only if the prefix is the UTF-8 BOM sequence.
+    # No-op on Linux/macOS/Git Bash where stdin never starts with a BOM.
+    # Guard: CliRunner / test stubs replace sys.stdin with StringIO (no .buffer).
+    try:
+        _stdin_buf = getattr(_sys.stdin, "buffer", None)
+        if _stdin_buf is not None and hasattr(_stdin_buf, "peek"):
+            _bom_prefix = _stdin_buf.peek(3)[:3]
+            if _bom_prefix == b"\xef\xbb\xbf":
+                _stdin_buf.read(3)
+                log.info("sourcecode-mcp stripped UTF-8 BOM from stdin (PowerShell 5.1 workaround)")
+    except Exception:
+        pass  # Never abort server startup over BOM detection
+
     log.info("sourcecode-mcp starting (stdio transport)")
     try:
         _mcp.run()
@@ -3574,6 +3644,10 @@ def mcp_init(
     typer.echo("")
     typer.echo("  Remove:  sourcecode mcp remove")
 
+    # Clear nudge flag: next run finds is_installed=True → no nudge.
+    from sourcecode.mcp_nudge import clear_nudge_flag as _clear_nudge
+    _clear_nudge()
+
 
 @mcp_app.command("status")
 def mcp_status() -> None:

sourcecode/mcp_nudge.py

@@ -0,0 +1,84 @@
+"""MCP setup nudge — one-time stderr hint after successful analysis commands.
+
+Fires when:
+  1. At least one known MCP client (Claude Desktop, Cursor) is installed
+  2. sourcecode is NOT yet registered in that client's config
+  3. The nudge hasn't been shown this session (~/.sourcecode/nudge_shown flag)
+
+Cleared by: a successful `sourcecode mcp init` (deletes the flag so the
+post-init detection finds is_installed=True and never nudges again).
+
+Side effects: writes only to stderr — stdout (JSON/YAML output) is untouched.
+Exit code of the calling command is unaffected.
+"""
+from __future__ import annotations
+
+import sys
+from pathlib import Path
+
+# Stable path used as a session-level "already shown" sentinel.
+_FLAG: Path = Path.home() / ".sourcecode" / "nudge_shown"
+
+_MSG = (
+    "→ Claude Desktop detected. "
+    "Run `sourcecode mcp init` to enable agent integration.\n"
+)
+
+# Module-level imports so names are patchable in tests.
+# Falls back to no-op stubs if onboarding package is unavailable.
+try:
+    from sourcecode.mcp.onboarding.detector import detect_clients  # noqa: PLC0415
+    from sourcecode.mcp.onboarding.applier import is_installed, read_config  # noqa: PLC0415
+    _IMPORTS_OK = True
+except Exception:  # pragma: no cover
+    _IMPORTS_OK = False
+
+    def detect_clients() -> list:  # type: ignore[misc]
+        return []
+
+    def is_installed(config: dict) -> bool:  # type: ignore[misc]
+        return False
+
+    def read_config(path: Path) -> dict:  # type: ignore[misc]
+        return {}
+
+
+def nudge_mcp_if_needed() -> None:
+    """Print MCP setup nudge to stderr at most once (until mcp init succeeds)."""
+    # Fast path: already shown this session.
+    if _FLAG.exists():
+        return
+
+    try:
+        clients = detect_clients()
+    except Exception:  # pragma: no cover
+        return
+
+    needs_nudge = any(
+        c.app_installed and not is_installed(read_config(c.config_path))
+        for c in clients
+    )
+
+    if not needs_nudge:
+        return
+
+    # Write nudge and persist flag.
+    sys.stderr.write(_MSG)
+    sys.stderr.flush()
+    try:
+        _FLAG.parent.mkdir(parents=True, exist_ok=True)
+        _FLAG.touch()
+    except OSError:
+        pass  # Non-fatal: nudge will fire again next run, which is acceptable.
+
+
+def clear_nudge_flag() -> None:
+    """Delete the session flag so post-mcp-init runs don't re-show the nudge.
+
+    Called by `mcp init` after a successful installation.  On the next run,
+    detection finds is_installed=True → needs_nudge=False → no nudge shown.
+    """
+    try:
+        _FLAG.unlink(missing_ok=True)
+    except OSError:
+        pass

sourcecode/prepare_context.py

@@ -388,6 +388,9 @@ class TaskOutput:
     deployment_risks: list[str] = field(default_factory=list)
     deployment: Optional[dict] = None
     entry_points_structured: Optional[dict] = None
+    # P0-2: fast-mode truncation transparency
+    truncated: bool = False
+    truncated_reason: Optional[str] = None
 
 
 @dataclass
@@ -1991,6 +1994,21 @@ class TaskContextBuilder:
                 untested.sort(key=lambda p: (len(p.split("/")), p))
                 test_gaps = untested[:15]
 
+        # P0-2: fast mode truncation transparency for generate-tests.
+        # When --fast is active the test-gap discovery block is skipped entirely,
+        # so test_gaps stays []. Without a signal the receiver interprets [] as
+        # "no gaps", which is incorrect. Emit explicit truncation metadata and
+        # downgrade confidence so the agent knows the analysis is incomplete.
+        _fast_truncated = fast and task_name == "generate-tests"
+        _fast_truncated_reason = "fast mode skips test gap discovery" if _fast_truncated else None
+        if _fast_truncated:
+            import sys as _sys_warn
+            _sys_warn.stderr.write(
+                "[warn] prepare-context generate-tests --fast: test gap discovery skipped. "
+                "Output will contain truncated=true and confidence=low.\n"
+            )
+            _sys_warn.stderr.flush()
+
         # ── 8. Confidence + gaps ──────────────────────────────────────────────
         from sourcecode.confidence_analyzer import ConfidenceAnalyzer
         from dataclasses import asdict as _asdict
@@ -2012,6 +2030,9 @@ class TaskContextBuilder:
 
         conf_summary, analysis_gaps = ConfidenceAnalyzer().analyze(sm_for_conf)
         confidence = conf_summary.overall
+        # P0-2: fast-mode truncation overrides confidence to signal incomplete analysis
+        if _fast_truncated:
+            confidence = "low"
         _has_mybatis = any(
             f.name == "MyBatis"
             for s in stacks
@@ -2130,6 +2151,9 @@ class TaskContextBuilder:
             deployment_risks=_cb_deploy_risks,
             deployment=_cb_deployment,
             entry_points_structured=_cb_bootstrap,
+            # P0-2: fast-mode truncation transparency
+            truncated=_fast_truncated,
+            truncated_reason=_fast_truncated_reason,
         )
 
     def render_prompt(self, output: TaskOutput) -> str:

{sourcecode-1.31.28.dist-info → sourcecode-1.31.30.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sourcecode
-Version: 1.31.28
+Version: 1.31.30
 Summary: Deterministic codebase context for AI coding agents
 License:                                  Apache License
                                    Version 2.0, January 2004
@@ -225,7 +225,7 @@ Description-Content-Type: text/markdown
 
 **AI-ready change intelligence for Java/Spring enterprise monoliths.**
 
-![Version](https://img.shields.io/badge/version-1.31.28-blue)
+![Version](https://img.shields.io/badge/version-1.31.30-blue)
 ![Python](https://img.shields.io/badge/python-3.10%2B-green)
 
 ---
@@ -263,7 +263,7 @@ pipx install sourcecode
 
 ```bash
 sourcecode version
-# sourcecode 1.31.28
+# sourcecode 1.31.30
 ```
 
 ---

{sourcecode-1.31.28.dist-info → sourcecode-1.31.30.dist-info}/RECORD

@@ -1,4 +1,4 @@
-sourcecode/__init__.py,sha256=u-6FWeYcsHBD4kgFtNzOZPQeoIf-YlQrC_JzWDvNiaA,104
+sourcecode/__init__.py,sha256=thS4KBhwEMTmgKe-XGLoD9avfVdy_-cdN7sf6C2_kA0,104
 sourcecode/adaptive_scanner.py,sha256=XffluXKzJUXrMtjEiAOnSNPZnztdIcts17T9ouHeID0,10521
 sourcecode/architecture_analyzer.py,sha256=Ry3aYT9dc7XuLmWLT5IZ93RkCf_P14Qtew0nGPvUl_8,42184
 sourcecode/architecture_summary.py,sha256=z34_6v7cSwy98cof2UVciGho7SCrZ93tiqMmq5WNzRQ,20405
@@ -6,7 +6,7 @@ sourcecode/ast_extractor.py,sha256=_btmeOJIe3t-NicF94D5ZAesa2YIJ0_QNExGnbHxGFE,5
 sourcecode/cache.py,sha256=TiYa3ECjBKtvlfCk7GvQ9v6gZkAITpH3ow9PubA7sUo,22946
 sourcecode/canonical_ir.py,sha256=_HM3AUmKSdna9u4dCoU6rpgSA6HdF8gzOKZykIUCNGY,23277
 sourcecode/classifier.py,sha256=yWeq6agTjkFa3zuNa-gdVIHtjoBoPoVlJnX-b7tdVJs,7851
-sourcecode/cli.py,sha256=YRHo7R58Nf-NYASQNCW39AjsJnOi5mrl5iziaPdMUxk,152204
+sourcecode/cli.py,sha256=7vwOxUdbWcKz42DnxKKp0TyxVhBmbl_eCOvcqct_gkw,155247
 sourcecode/code_notes_analyzer.py,sha256=EJemNCNc9Dn-1RZYu-aNbK0ELzmsyC4s6FdHi3XyNEI,9392
 sourcecode/confidence_analyzer.py,sha256=_jckZSxksV-OU38vbkxfVNBnWCtlCq8Vwfg23x1uspA,19054
 sourcecode/context_scorer.py,sha256=QpChSpsmaAYz91rXA4Ue5xzQmNz_ZboZN09YOHScq1U,14679
@@ -22,11 +22,12 @@ sourcecode/file_classifier.py,sha256=QrYm7MlG29HQdAR1WOfpnIIBysAz62c5coz9eQ76meo
 sourcecode/flow_analyzer.py,sha256=dSiuY4w49k29jW_EPXUOND9B5uVbuCA7kjnuHi-pIWA,28781
 sourcecode/git_analyzer.py,sha256=0Gyj-vMpIIN4nfriKXVRouNYBeJ59s6pQDX2Xu9Pq-U,13177
 sourcecode/graph_analyzer.py,sha256=iUK-7pSV-cvGqqD2hENdYmhnm0wcXFEyK-xnu5ul8OU,62515
+sourcecode/mcp_nudge.py,sha256=rJzr_dirC6L3VMsUBmzsSSJb-j-nphZy-qq7i2JPjEQ,2625
 sourcecode/metrics_analyzer.py,sha256=m0ENgtqKeBL17kUIK3fmGkgo7UfXBNHxCMj0H_Y5K7c,22750
 sourcecode/output_budget.py,sha256=43307mJEyUPU3MI-QEQoVxrcAvNyUzdzF_SAPgisBQE,6603
 sourcecode/path_filters.py,sha256=ROFRQ8eSLBEMiixK9f45-RO7um4VEEcjoD5AA4I427I,3739
 sourcecode/pr_comment_renderer.py,sha256=smHslxiG14lrytCkq5nFrFu-qTHgA-t-LFYfdrfjz2o,14423
-sourcecode/prepare_context.py,sha256=RxzyMoxgGmMdwqxMwd9CoEsOJARgKJ6YrZte_FeH68A,200168
+sourcecode/prepare_context.py,sha256=RM7ka0rduJy8kwGHzLU9if6q7D9ST7tGjOf5LnsdTuw,201451
 sourcecode/progress.py,sha256=qn30sWaHOkjTgXsSBmiPkz7Rsbwc5oSlIe6JNEMYp_k,3149
 sourcecode/ranking_engine.py,sha256=ZAucq_YX2KkWUuAZf4P0lhtQ_38vEFnUhuGtSZd1S0E,12970
 sourcecode/redactor.py,sha256=xuGcadGEHaPw4qZXlMDvzMCsr4VOkdp3oBQptHyJk8c,2884
@@ -76,8 +77,8 @@ sourcecode/telemetry/consent.py,sha256=wLMvGNJeSSyZoNkQXpoUioY6mMv4Qdvuw7S9jAEWn
 sourcecode/telemetry/events.py,sha256=oEvvulfsv5GIDWG2174gSS6tNB95w38AIYiYeifGKlE,2294
 sourcecode/telemetry/filters.py,sha256=Asa71oRl7q3Wt_FMwuufIZJFzSYdgRNKS8LHCIyFeYE,4805
 sourcecode/telemetry/transport.py,sha256=KJeIPCPWMdmbCP3ySGs2iUlia34U6vWne2dZsUezesw,1560
-sourcecode-1.31.28.dist-info/METADATA,sha256=IopWHYcmGBKmum7GTpRUesF4DHPc1bSC4kVW3-BDYuU,31103
-sourcecode-1.31.28.dist-info/WHEEL,sha256=QccIxa26bgl1E6uMy58deGWi-0aeIkkangHcxk2kWfw,87
-sourcecode-1.31.28.dist-info/entry_points.txt,sha256=ex3F9rmbXeyDIoFQHtkEqTsKSaJow8F0LrVu8XfIktQ,57
-sourcecode-1.31.28.dist-info/licenses/LICENSE,sha256=7DdHrU9Z_3e7dSvq4ISijZNjnuHo5NIHNiHDouMQ9JU,10491
-sourcecode-1.31.28.dist-info/RECORD,,
+sourcecode-1.31.30.dist-info/METADATA,sha256=R8F2pFXTmotZdlErM06krEELPX74sev65gQk0WUvtrA,31103
+sourcecode-1.31.30.dist-info/WHEEL,sha256=QccIxa26bgl1E6uMy58deGWi-0aeIkkangHcxk2kWfw,87
+sourcecode-1.31.30.dist-info/entry_points.txt,sha256=ex3F9rmbXeyDIoFQHtkEqTsKSaJow8F0LrVu8XfIktQ,57
+sourcecode-1.31.30.dist-info/licenses/LICENSE,sha256=7DdHrU9Z_3e7dSvq4ISijZNjnuHo5NIHNiHDouMQ9JU,10491
+sourcecode-1.31.30.dist-info/RECORD,,