sourcecode 1.31.28__tar.gz → 1.31.30__tar.gz

sourcecode-1.31.30/.continue-here.md

@@ -0,0 +1,89 @@
+# Handoff — Sesión 34 (2026-05-26)
+
+## Position
+
+**Project:** sourcecode (atlas-cli)  
+**Activity:** Post-audit bug fixing + new feature (MCP nudge)  
+**Branch:** master  
+**Last commit:** 11d23b0 — feat(mcp): add one-time MCP setup nudge after successful commands  
+**Tests:** 1601 passed, 3 skipped ✅
+
+---
+
+## Work Done This Session
+
+### Session 34 (this session)
+**New feature: one-time MCP setup nudge**
+
+**FEAT — `sourcecode mcp_nudge.py` (new module)**
+- `nudge_mcp_if_needed()`: fires after first successful analysis command when Claude Desktop/Cursor installed but sourcecode not yet in MCP config. Writes to stderr only. Creates `~/.sourcecode/nudge_shown` flag to fire at most once per session.
+- `clear_nudge_flag()`: called by `mcp init` on success — next run finds `is_installed=True` → no nudge.
+- Module-level imports for testability; `_IMPORTS_OK=False` stubs if onboarding package missing.
+
+**CLI hooks (4 call sites in `cli.py`):**
+| Command | Location |
+|---|---|
+| `sourcecode . --compact` | `main()` — after clipboard copy |
+| `sourcecode prepare-context` / `onboard` | `prepare_context_cmd()` — end |
+| `sourcecode endpoints` | `endpoints_cmd()` — end |
+| `sourcecode impact` | `impact_cmd()` — success path only (after not_found check) |
+
+**`mcp init` success** → `clear_nudge_flag()` called.
+
+**Tests:** `tests/test_mcp_nudge.py` — 11 tests, all pass.
+
+---
+
+## Remaining from Bug Audit Backlog
+
+### P1 backlog
+| # | Issue | File | Effort |
+|---|---|---|---|
+| P1-5 | `project_summary` quality — picks license/badges/marketing blurbs | `summarizer.py` | Medium |
+| P1-6 | `relevant_files.score` ranking quality audit (score exposed, ranking weak) | `ranking_engine.py` | Medium |
+
+### P2 unfixed
+| # | Issue | File | Effort |
+|---|---|---|---|
+| P2-13 | `--no-cache` flag inconsistent across subcommands | `cli.py` | Medium |
+| P2-14 | URLs in code_notes truncated (needs repro) | `code_notes_analyzer.py` | Low |
+| P2-16 | `entry_points.controllers.methods` mismatches `endpoints` count | `cli.py` | Medium |
+| P2-17 | `--compact --help` token claim stale (now runs arch analyzer too) | `cli.py` | Trivial |
+| P2-18 | `impact <file_path>` fails FQN resolution | `repository_ir.py` | Low |
+
+---
+
+## Key Files Modified (this session)
+
+```
+src/sourcecode/mcp_nudge.py         — NEW: nudge_mcp_if_needed, clear_nudge_flag
+src/sourcecode/cli.py               — 4 nudge call sites + clear_nudge_flag in mcp_init
+tests/test_mcp_nudge.py             — NEW: 11 tests
+```
+
+---
+
+## Decisions This Session
+
+| Decision | Rationale |
+|---|---|
+| Module-level imports in mcp_nudge.py (not lazy) | Enables `patch("sourcecode.mcp_nudge.detect_clients", ...)` in tests |
+| Flag at `~/.sourcecode/nudge_shown` | Reuses existing state root; simple touch file |
+| Nudge on impact only when resolution != not_found | not_found = exit 1 = failure; no nudge on failed commands |
+| `clear_nudge_flag()` in mcp_init success, not in detection | Separation: flag reset is explicit intent (user ran init), not a side effect of checking |
+
+---
+
+## Next Session
+
+**Recommended:** P2-17 (trivial) → P2-18 (low) → P2-14 (low, needs repro) → P1-5 (medium) → P1-6 (medium) → P2-13 (medium) → P2-16 (medium)
+
+**P2-17:** `--compact --help` says token count — verify claim still accurate after P2-10 fix (compact now runs ArchitectureAnalyzer). Update if stale.
+
+**P2-18:** `sourcecode impact <file_path>` — resolve file path to FQN before IR lookup. Currently passes raw path to `_resolve_target()`, fails to match.
+
+**P1-5:** Audit `summarizer.py`. Prefer: first real descriptive paragraph, architecture description, domain/stack. Exclude: 'Important:', license, badges, install snippets, TOC, link collections, sponsorship, release notes.
+
+**Resume command:** `/gsd:resume-work`
+
+**Test baseline:** `python3 -m pytest tests/ → 1601 passed, 3 skipped`

{sourcecode-1.31.28 → sourcecode-1.31.30}/PKG-INFO

@@ -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 → sourcecode-1.31.30}/README.md

@@ -2,7 +2,7 @@
 
 **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)
 
 ---
@@ -40,7 +40,7 @@ pipx install sourcecode
 
 ```bash
 sourcecode version
-# sourcecode 1.31.28
+# sourcecode 1.31.30
 ```
 
 ---

{sourcecode-1.31.28 → sourcecode-1.31.30}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "sourcecode"
-version = "1.31.28"
+version = "1.31.30"
 description = "Deterministic codebase context for AI coding agents"
 readme = "README.md"
 requires-python = ">=3.9"

{sourcecode-1.31.28 → sourcecode-1.31.30}/src/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-1.31.28 → sourcecode-1.31.30}/src/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-1.31.30/src/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-1.31.28 → sourcecode-1.31.30}/src/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.30/tests/test_mcp_nudge.py

@@ -0,0 +1,153 @@
+"""Tests for sourcecode.mcp_nudge — one-time MCP setup nudge."""
+from __future__ import annotations
+
+import sys
+from pathlib import Path
+from unittest.mock import MagicMock, patch
+
+import pytest
+
+import sourcecode.mcp_nudge as nudge_mod
+from sourcecode.mcp_nudge import nudge_mcp_if_needed, clear_nudge_flag
+
+
+# ── helpers ───────────────────────────────────────────────────────────────────
+
+def _make_client(app_installed: bool, config_path: Path) -> MagicMock:
+    client = MagicMock()
+    client.app_installed = app_installed
+    client.config_path = config_path
+    return client
+
+
+# ── nudge_mcp_if_needed ───────────────────────────────────────────────────────
+
+class TestNudgeMcpIfNeeded:
+    """Core nudge logic — isolation via tmp_path flag dir."""
+
+    def _run(self, flag_path: Path, clients, is_installed_val: bool, capsys) -> str:
+        """Invoke nudge_mcp_if_needed with patched flag, clients and is_installed."""
+        with (
+            patch.object(nudge_mod, "_FLAG", flag_path),
+            patch("sourcecode.mcp_nudge.detect_clients", return_value=clients),
+            patch("sourcecode.mcp_nudge.is_installed", return_value=is_installed_val),
+            patch("sourcecode.mcp_nudge.read_config", return_value={}),
+        ):
+            nudge_mcp_if_needed()
+        return capsys.readouterr().err
+
+    def test_nudge_shown_when_desktop_installed_not_configured(self, tmp_path, capsys):
+        """Client installed, not in config → nudge printed."""
+        flag = tmp_path / "nudge_shown"
+        client = _make_client(app_installed=True, config_path=tmp_path / "config.json")
+        stderr = self._run(flag, [client], is_installed_val=False, capsys=capsys)
+        assert "sourcecode mcp init" in stderr
+        assert flag.exists(), "Flag must be created after nudge"
+
+    def test_no_nudge_when_already_configured(self, tmp_path, capsys):
+        """Client installed AND already in config → no nudge."""
+        flag = tmp_path / "nudge_shown"
+        client = _make_client(app_installed=True, config_path=tmp_path / "config.json")
+        stderr = self._run(flag, [client], is_installed_val=True, capsys=capsys)
+        assert stderr == ""
+        assert not flag.exists()
+
+    def test_no_nudge_when_client_not_installed(self, tmp_path, capsys):
+        """Client NOT installed → no nudge even if config absent."""
+        flag = tmp_path / "nudge_shown"
+        client = _make_client(app_installed=False, config_path=tmp_path / "config.json")
+        stderr = self._run(flag, [client], is_installed_val=False, capsys=capsys)
+        assert stderr == ""
+        assert not flag.exists()
+
+    def test_no_nudge_when_flag_exists(self, tmp_path, capsys):
+        """Flag already present → no nudge (second run in same session)."""
+        flag = tmp_path / "nudge_shown"
+        flag.touch()
+        client = _make_client(app_installed=True, config_path=tmp_path / "config.json")
+        stderr = self._run(flag, [client], is_installed_val=False, capsys=capsys)
+        assert stderr == ""
+
+    def test_no_nudge_when_no_clients(self, tmp_path, capsys):
+        """No clients detected → no nudge."""
+        flag = tmp_path / "nudge_shown"
+        stderr = self._run(flag, [], is_installed_val=False, capsys=capsys)
+        assert stderr == ""
+        assert not flag.exists()
+
+    def test_second_call_no_repeat(self, tmp_path, capsys):
+        """Nudge fires once; second call (flag now exists) is silent."""
+        flag = tmp_path / "nudge_shown"
+        client = _make_client(app_installed=True, config_path=tmp_path / "config.json")
+
+        with (
+            patch.object(nudge_mod, "_FLAG", flag),
+            patch("sourcecode.mcp_nudge.detect_clients", return_value=[client]),
+            patch("sourcecode.mcp_nudge.is_installed", return_value=False),
+            patch("sourcecode.mcp_nudge.read_config", return_value={}),
+        ):
+            nudge_mcp_if_needed()
+            capsys.readouterr()  # consume first write
+            nudge_mcp_if_needed()
+
+        second_stderr = capsys.readouterr().err
+        assert second_stderr == ""
+
+    def test_nudge_message_exact_text(self, tmp_path, capsys):
+        """Message matches spec exactly."""
+        flag = tmp_path / "nudge_shown"
+        client = _make_client(app_installed=True, config_path=tmp_path / "config.json")
+        stderr = self._run(flag, [client], is_installed_val=False, capsys=capsys)
+        assert stderr == (
+            "→ Claude Desktop detected. "
+            "Run `sourcecode mcp init` to enable agent integration.\n"
+        )
+
+    def test_import_error_is_silent(self, tmp_path, capsys):
+        """If onboarding modules can't be imported, nudge fails silently."""
+        flag = tmp_path / "nudge_shown"
+        with (
+            patch.object(nudge_mod, "_FLAG", flag),
+            patch.dict("sys.modules", {
+                "sourcecode.mcp.onboarding.detector": None,  # type: ignore[dict-item]
+                "sourcecode.mcp.onboarding.applier": None,   # type: ignore[dict-item]
+            }),
+        ):
+            nudge_mcp_if_needed()  # must not raise
+        assert capsys.readouterr().err == ""
+
+
+# ── clear_nudge_flag ──────────────────────────────────────────────────────────
+
+class TestClearNudgeFlag:
+    def test_clears_existing_flag(self, tmp_path):
+        flag = tmp_path / "nudge_shown"
+        flag.touch()
+        with patch.object(nudge_mod, "_FLAG", flag):
+            clear_nudge_flag()
+        assert not flag.exists()
+
+    def test_noop_when_flag_absent(self, tmp_path):
+        flag = tmp_path / "nudge_shown"
+        with patch.object(nudge_mod, "_FLAG", flag):
+            clear_nudge_flag()  # must not raise
+        assert not flag.exists()
+
+    def test_after_clear_nudge_fires_again_if_not_configured(self, tmp_path, capsys):
+        """After mcp init clears flag, if somehow not configured, nudge re-fires."""
+        flag = tmp_path / "nudge_shown"
+        flag.touch()
+        client = _make_client(app_installed=True, config_path=tmp_path / "config.json")
+
+        with patch.object(nudge_mod, "_FLAG", flag):
+            clear_nudge_flag()
+
+        with (
+            patch.object(nudge_mod, "_FLAG", flag),
+            patch("sourcecode.mcp_nudge.detect_clients", return_value=[client]),
+            patch("sourcecode.mcp_nudge.is_installed", return_value=False),
+            patch("sourcecode.mcp_nudge.read_config", return_value={}),
+        ):
+            nudge_mcp_if_needed()
+
+        assert "mcp init" in capsys.readouterr().err