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

sourcecode/__init__.py

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

sourcecode/cli.py

@@ -739,12 +739,15 @@ def main(
     # compact is designed to be a bounded summary; --full removes truncation limits,
     # which contradicts compact's purpose. Use --agent --full for expanded output.
     if compact and full:
-        typer.echo(
-            "Error: --compact and --full are mutually exclusive. "
-            "--compact produces a bounded summary; --full removes truncation limits and "
-            "is meant for --agent mode. Use --agent --full for expanded output.",
-            err=True,
-        )
+        import json as _json_flags, sys as _sys_flags
+        _sys_flags.stdout.write(_json_flags.dumps({
+            "error": "incompatible_flags",
+            "message": "--compact and --full are mutually exclusive. "
+                       "--compact produces a bounded summary; --full removes truncation limits "
+                       "and is meant for --agent mode. Use --agent --full for expanded output.",
+            "exit_code": 1,
+        }, ensure_ascii=False) + "\n")
+        _sys_flags.stdout.flush()
         raise typer.Exit(code=1)
 
     # P0-2 FIX: --full without --compact or --agent has no effect in contract/raw mode.

sourcecode/mcp/server.py

@@ -453,6 +453,36 @@ def get_impact_context(repo_path: str = ".", target: str = "", depth: int = 4) -
         )
 
 
+@mcp.tool()
+def modernize_context(repo_path: str = ".", format: str = "json") -> dict:
+    """Analyzes codebase for modernization opportunities: dead zones, hotspot scores, upgrade candidates.
+
+    Maps to: sourcecode modernize <repo_path>
+    Returns: hotspot_candidates (high fan-in + git churn), dead_zone_candidates (isolated classes),
+             high_coupling_nodes, subsystem_summary, cross_module_tangles, recommendation.
+
+    Best for: refactor planning, identifying where to start, finding safe removal candidates.
+    Use get_compact_context or get_agent_context first for project orientation.
+
+    repo_path: absolute path to the Java repository (default: current working directory).
+    format: output format — "json" (default). Only json is supported; yaml is not available
+            for modernize output.
+    """
+    _raw = repo_path
+    try:
+        if not isinstance(repo_path, str):
+            return _err("repo_path must be a string", "INVALID_ARGUMENT")
+        if not isinstance(format, str) or format not in ("json", "yaml"):
+            return _err("format must be 'json' or 'yaml'", "INVALID_ARGUMENT")
+        repo_path = _normalize_repo_path(repo_path)
+        return _execute(["modernize", repo_path])
+    except Exception as exc:
+        return _err(
+            f"Internal error: {type(exc).__name__}: {exc} — repo_path recibido: {_raw}",
+            "INTERNAL_ERROR",
+        )
+
+
 _TELEMETRY_ACTIONS = frozenset({"status", "enable", "disable"})
 
 

sourcecode/prepare_context.py

@@ -1134,6 +1134,27 @@ class TaskContextBuilder:
         # No-git and invalid-ref cases were already handled in step 0 (early returns).
         if task_name == "review-pr":
             if not _pr_scope_files:
+                # 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>"
+                    )
+                    return TaskOutput(
+                        task="review-pr", goal=spec.goal,
+                        project_summary=None, architecture_summary=None,
+                        relevant_files=[], suspected_areas=[],
+                        improvement_opportunities=[], test_gaps=[],
+                        key_dependencies=[], code_notes_summary=None,
+                        limitations=[], confidence="low",
+                        error_code="no_diff_source",
+                        error_message=_no_diff_msg,
+                        gaps=[_no_diff_msg],
+                        ci_decision="no_diff_source",
+                        scope_source=_pr_scope_source,
+                        scope_files=[],
+                        repo_root=str(_pr_git_root),
+                    )
                 _no_diff_hint = "review-pr requires changed files or --since <ref>."
                 return TaskOutput(
                     task="review-pr", goal=spec.goal,
@@ -1221,6 +1242,76 @@ class TaskContextBuilder:
                 symptom=symptom if task_name == "fix-bug" else None,
             )
 
+        # ── Fast-mode fallback: never return empty relevant_files when source files exist ──
+        # When --fast is active on a large repo, all_paths may be restricted to a handful of
+        # changed/noise files that all get filtered out by _rank_files. Inject fallback signals:
+        # 1. detected entry points (already computed, zero I/O cost)
+        # 2. recently committed files (git log -10 --name-only)
+        # 3. files matching symptom keywords in path (when fix-bug + --symptom)
+        if fast and not relevant_files and task_name not in ("delta", "review-pr"):
+            import subprocess as _sp_fb
+            _fb_seen: set[str] = set()
+            _fb_candidates: list[RelevantFile] = []
+
+            # 1. Entry points from detection
+            for _ep in entry_points:
+                _ep_path = _ep.path.replace("\\", "/")
+                if _ep_path not in _fb_seen and (self.root / _ep_path).exists():
+                    _fb_candidates.append(RelevantFile(
+                        path=_ep_path,
+                        role="entrypoint",
+                        score=0.5,
+                        reason="fast-mode fallback: detected entry point",
+                        why="entry_point signal from manifest/annotation detection",
+                    ))
+                    _fb_seen.add(_ep_path)
+
+            # 2. Recently committed files (git log -10 --name-only)
+            try:
+                _gl_r = _sp_fb.run(
+                    ["git", "log", "--name-only", "--pretty=format:", "-10"],
+                    capture_output=True, text=True, cwd=str(self.root), timeout=5,
+                )
+                for _gl_f in _gl_r.stdout.splitlines():
+                    _gl_f = _gl_f.strip().replace("\\", "/")
+                    if (not _gl_f or _gl_f in _fb_seen):
+                        continue
+                    if Path(_gl_f).suffix.lower() not in _ALL_EXTENSIONS:
+                        continue
+                    if not (self.root / _gl_f).exists():
+                        continue
+                    _fb_candidates.append(RelevantFile(
+                        path=_gl_f,
+                        role="source",
+                        score=0.3,
+                        reason="fast-mode fallback: recently committed file (git log -10)",
+                        why="recent commit history signal",
+                    ))
+                    _fb_seen.add(_gl_f)
+            except Exception:
+                pass
+
+            # 3. Symptom keyword path matches (fix-bug only)
+            if task_name == "fix-bug" and symptom:
+                import re as _re_fb
+                _fb_kws = [w.lower() for w in _re_fb.split(r"[\s\W]+", symptom) if len(w) > 2]
+                for _fb_p in all_paths:
+                    if _fb_p in _fb_seen:
+                        continue
+                    if Path(_fb_p).suffix.lower() not in _ALL_EXTENSIONS:
+                        continue
+                    if any(kw in _fb_p.lower() for kw in _fb_kws):
+                        _fb_candidates.append(RelevantFile(
+                            path=_fb_p,
+                            role="source",
+                            score=0.2,
+                            reason=f"fast-mode fallback: path matches symptom ({symptom!r})",
+                            why="symptom keyword in file path",
+                        ))
+                        _fb_seen.add(_fb_p)
+
+            relevant_files = _fb_candidates[:20]
+
         # ── IC-006: fix-bug suspected_areas — recompute from ranked files + bug notes ──
         # relevant_files is now ranked by RankingEngine (git churn, fan_in, centrality, notes).
         # suspected_areas should reflect that ranking, not raw comment count.
@@ -2569,24 +2660,15 @@ class TaskContextBuilder:
             if DiffSourceType.WORKTREE_STAGED.value not in sources:
                 sources.append(DiffSourceType.WORKTREE_STAGED.value)
 
-        # ── HEAD~1 fallback — working tree clean, surface last commit ────────
-        # Without --since: if no unstaged/staged changes exist, fall back to the
-        # last committed diff so a clean tree never silently returns no_changes.
+        # ── FIX-P1: no --since + clean working tree → no_staged_changes signal ──
+        # Previously: silently fell back to HEAD~1 diff, masking a missing --since.
+        # Now: emit "no_staged_changes" scope so the caller can return no_diff_source
+        # (exit 1) and prompt the user to provide --since.
+        # Rationale: in CI, the tree is always clean; without --since there is no
+        # meaningful diff to review. Local dev should stage changes before review-pr.
         if since is None and not committed_files and not uncommitted_files:
-            head_ok = _run("git", "rev-parse", "--verify", "HEAD~1")
-            if head_ok is not None:  # HEAD~1 exists
-                head_committed = _run("git", "diff", "--name-only", "--relative", "HEAD~1..HEAD")
-                if head_committed:
-                    committed_files = head_committed
-                    sources.append(DiffSourceType.HEAD_MINUS_1.value)
-            else:
-                # First commit — no HEAD~1; diff against git empty tree
-                _GIT_EMPTY_TREE = "4b825dc642cb6eb9a060e54bf8d69288fbee4904"
-                first_committed = _run("git", "diff", "--name-only", "--relative",
-                                       _GIT_EMPTY_TREE, "HEAD")
-                if first_committed:
-                    committed_files = first_committed
-                    sources.append("initial_commit")
+            # Return sentinel — step 5d converts this to no_diff_source (exit 1).
+            return [], "no_staged_changes", [], []
 
         # ── Drop paths outside self.root ──────────────────────────────────────
         def _drop_outside(lst: list[str]) -> list[str]:

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sourcecode
-Version: 1.31.27
+Version: 1.31.28
 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.27-blue)
+![Version](https://img.shields.io/badge/version-1.31.28-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.27
+# sourcecode 1.31.28
 ```
 
 ---

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

@@ -1,4 +1,4 @@
-sourcecode/__init__.py,sha256=VXhPMdJM6x_WGp7qMeGlW63Lz-PmJxDFOdXqGhLNfm4,104
+sourcecode/__init__.py,sha256=u-6FWeYcsHBD4kgFtNzOZPQeoIf-YlQrC_JzWDvNiaA,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=8qcr6G5RF17EqtLsM7Weu3G6Pb3GbfgudK-Knx01-0w,151980
+sourcecode/cli.py,sha256=YRHo7R58Nf-NYASQNCW39AjsJnOi5mrl5iziaPdMUxk,152204
 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
@@ -26,7 +26,7 @@ sourcecode/metrics_analyzer.py,sha256=m0ENgtqKeBL17kUIK3fmGkgo7UfXBNHxCMj0H_Y5K7
 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=_WZqHLqdRD5YftN7ebm2jXd1c-iJpNE8TzVKjK6LUbE,196037
+sourcecode/prepare_context.py,sha256=RxzyMoxgGmMdwqxMwd9CoEsOJARgKJ6YrZte_FeH68A,200168
 sourcecode/progress.py,sha256=qn30sWaHOkjTgXsSBmiPkz7Rsbwc5oSlIe6JNEMYp_k,3149
 sourcecode/ranking_engine.py,sha256=ZAucq_YX2KkWUuAZf4P0lhtQ_38vEFnUhuGtSZd1S0E,12970
 sourcecode/redactor.py,sha256=xuGcadGEHaPw4qZXlMDvzMCsr4VOkdp3oBQptHyJk8c,2884
@@ -64,7 +64,7 @@ sourcecode/detectors/terraform.py,sha256=cxORPR_zVLOJpHlh4e9JnFpkQsn_UnqMMom5yG6
 sourcecode/detectors/tooling.py,sha256=8CKbtxwQoABP-WyBRNmdAmHDOvAH57AR1cF4UKuWEdQ,2074
 sourcecode/mcp/__init__.py,sha256=XU4HfRGbdid8wdUA0x_4f7uKZD1z3mv_XUY_WU_T9Mw,179
 sourcecode/mcp/runner.py,sha256=7PnFjKYbgxFeDnqVeSntXHxZX7ZtK3-krDkEuVjI24M,1386
-sourcecode/mcp/server.py,sha256=BqoXi2Ncu7huGVie9AbsUNAxq9cbASLJ_Zgf68aZWsI,20122
+sourcecode/mcp/server.py,sha256=DVXWcmGiAjNQe3fxJS-AUH03rsA_VFjS6048f8RMkt0,21540
 sourcecode/mcp/onboarding/__init__.py,sha256=sj2PWqEBmMc4zBNkomg89WtL0M6S7A9yb7_wAuSWNP4,66
 sourcecode/mcp/onboarding/applier.py,sha256=yfSMT0NKdZsjavtLkC8yQ7OtkfepOl5IXGByqg6bdEY,1894
 sourcecode/mcp/onboarding/backup.py,sha256=ihqGOR8QTX8HASRSEDyfFyXr5bkXrygPHamv4p9KTmk,1452
@@ -76,8 +76,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.27.dist-info/METADATA,sha256=yC3wqkw7X5sccpT8RShxa7uR2_fQsik7FCAJJZ33yY0,31103
-sourcecode-1.31.27.dist-info/WHEEL,sha256=QccIxa26bgl1E6uMy58deGWi-0aeIkkangHcxk2kWfw,87
-sourcecode-1.31.27.dist-info/entry_points.txt,sha256=ex3F9rmbXeyDIoFQHtkEqTsKSaJow8F0LrVu8XfIktQ,57
-sourcecode-1.31.27.dist-info/licenses/LICENSE,sha256=7DdHrU9Z_3e7dSvq4ISijZNjnuHo5NIHNiHDouMQ9JU,10491
-sourcecode-1.31.27.dist-info/RECORD,,
+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,,