sourcecode 1.30.6__py3-none-any.whl → 1.30.7__py3-none-any.whl

sourcecode/__init__.py

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

sourcecode/cli.py

@@ -1760,7 +1760,7 @@ def prepare_context_cmd(
         out["architecture_summary"] = output.architecture_summary
     if _task_include("confidence"):
         out["confidence"] = output.confidence
-    if _task_include("relevant_files"):
+    if task != "review-pr" and _task_include("relevant_files"):
         out["relevant_files"] = [
             {k: v for k, v in asdict(f).items() if v != ""}
             for f in output.relevant_files
@@ -1862,6 +1862,11 @@ def prepare_context_cmd(
             out["configuration_impact"] = output.configuration_impact
         if output.test_coverage_risk:
             out["test_coverage_risk"] = output.test_coverage_risk
+        # honest split: runtime files vs build artifacts — no mixed ranking
+        if output.runtime_changes:
+            out["runtime_changes"] = output.runtime_changes
+        if output.build_changes:
+            out["build_changes"] = output.build_changes
         if output.review_hotspots:
             out["review_hotspots"] = output.review_hotspots
         if output.suggested_review_order:

sourcecode/prepare_context.py

@@ -357,6 +357,9 @@ class TaskOutput:
     scope_source: Optional[str] = None   # "git_diff" | "staged" | "untracked" | "full_scan_fallback"
     scope_files: list[str] = field(default_factory=list)
     repo_root: Optional[str] = None
+    # honest output schema (review-pr only): runtime vs build split
+    runtime_changes: list[dict] = field(default_factory=list)
+    build_changes: dict = field(default_factory=dict)
 
 
 # ─────────────────────────────────────────────────────────────────────────────
@@ -416,6 +419,26 @@ _ALL_EXTENSIONS: frozenset[str] = _SOURCE_EXTENSIONS | frozenset({
     ".md", ".toml", ".yaml", ".yml", ".json", ".xml",
 })
 
+_ARTIFACT_CHANGE_EFFECT: dict[str, str] = {
+    "entrypoint":     "modifies application startup, CLI entry, or framework bootstrap — all request flows may be affected",
+    "controller":     "alters HTTP routing, API contract, or response shape — API consumers are affected",
+    "service":        "changes business rules, transaction scope, or orchestration logic — callers and dependents affected",
+    "repository":     "modifies persistence queries or data access patterns — data consistency and service layer affected",
+    "mapper":         "alters SQL-to-object binding or query templates — data shape and repositories affected",
+    "security":       "changes authentication flow, access control rules, or session handling — all secured endpoints affected",
+    "spring_config":  "modifies bean wiring, datasource, or framework-wide settings — all wired beans potentially affected",
+    "spring_profile": "changes environment-specific overrides — behavior differs per active profile",
+    "config":         "adjusts configuration values — all modules reading this config are affected",
+    "build_manifest": "changes dependencies, plugins, or project structure — compile-time and runtime classpath affected",
+    "db_migration":   "modifies database schema — existing queries, mappings, and constraints may break",
+    "domain_model":   "alters entity structure — cascades to repositories, DTOs, serializers, and mappers",
+    "dto":            "changes data transfer contract — serialization and API consumers may break",
+    "test":           "modifies test coverage or test behavior — no production code affected",
+    "documentation":  "updates documentation only — no runtime impact",
+    "ide_noise":      "IDE/tooling artifact — no application impact",
+    "source":         "modifies application source — artifact role derived from file path structure",
+}
+
 # Maps frontend symptom keywords → backend terms likely to contain the root cause.
 # Used to boost service/interceptor files when the symptom is UI-only.
 _FRONTEND_SYMPTOM_MAP: dict[str, list[str]] = {
@@ -830,6 +853,8 @@ class TaskContextBuilder:
         _pr_review_hotspots: list[str] = []
         _pr_suggested_review_order: list[str] = []
         _pr_base_ref: Optional[str] = None
+        _pr_runtime_changes: list[dict] = []
+        _pr_build_changes: dict = {}
 
         if task_name == "review-pr":
             _pr_base_ref = since or "HEAD"
@@ -879,14 +904,21 @@ class TaskContextBuilder:
                 "risk_level": _test_risk_level,
             }
 
-            # Review hotspots: top changed files ranked by impact score
+            # Pre-classify changed files once — reused for hotspots, order, and runtime/build split
+            _pr_changed_cls: dict[str, dict] = {
+                f: self._classify_changed_file(f) for f in (_delta_files or set())
+            }
+
+            # Review hotspots: top changed RUNTIME files ranked by impact score — no build artifacts
             _pr_review_hotspots = sorted(
-                _delta_files or set(),
+                [f for f, cls in _pr_changed_cls.items()
+                 if not cls["is_noise"] and cls["artifact_type"] != "build_manifest"],
                 key=lambda f: _delta_impact_score_per_file.get(f, 0.0),
                 reverse=True,
             )[:8]
 
             # Suggested review order: security first, then api → service → persistence → config
+            # build_manifest is intentionally absent from _ORDER_TYPES
             _ORDER_TYPES = ["security", "controller", "service", "repository", "mapper",
                             "spring_config", "config", "domain_model", "dto"]
             _seen_order: set[str] = set()
@@ -894,7 +926,7 @@ class TaskContextBuilder:
                 for _ra in _delta_risk_areas:
                     for _f in _ra.get("affected_files", []):
                         if _f not in _seen_order:
-                            _cls = self._classify_changed_file(_f)
+                            _cls = _pr_changed_cls.get(_f) or self._classify_changed_file(_f)
                             if _cls["artifact_type"] == _otype:
                                 _pr_suggested_review_order.append(_f)
                                 _seen_order.add(_f)
@@ -903,6 +935,56 @@ class TaskContextBuilder:
                     _pr_suggested_review_order.append(_f)
                     _seen_order.add(_f)
 
+            # Build runtime_changes and build_changes — honest split, no score numbers
+            _pr_runtime_changes: list[dict] = []
+            _pr_build_changes: dict = {}
+            _build_artifact_files: list[str] = []
+
+            _SCORE_TO_CONFIDENCE = {
+                "high":   lambda s: s >= 0.60,
+                "medium": lambda s: 0.40 <= s < 0.60,
+            }
+
+            for _f in sorted(_delta_files or set()):
+                _f_cls = _pr_changed_cls.get(_f) or self._classify_changed_file(_f)
+                _f_atype = _f_cls["artifact_type"]
+                if _f_cls["is_noise"]:
+                    continue
+                if _f_atype == "build_manifest":
+                    _build_artifact_files.append(_f)
+                    continue
+                # role: always "inferred" — no runtime evidence for role classification
+                _cls_conf = _f_cls["confidence"]  # "high"|"medium"|"low" from _classify_changed_file
+                _role_basis = "naming" if _cls_conf == "high" else ("path" if _cls_conf == "medium" else "extension")
+                _role_obj = {
+                    "type": "inferred",
+                    "confidence": "medium" if _cls_conf == "high" else "low",
+                    "basis": _role_basis,
+                }
+                # evidence: list what we actually know
+                _evidence: list[str] = ["changed in git diff"]
+                _f_module = _f_cls.get("module", "")
+                if _f_module:
+                    _evidence.append(f"matched module path: {_f_module}")
+                _evidence.append("NO call graph evidence")
+                # file confidence: replaces numeric score
+                _impact_s = _delta_impact_score_per_file.get(_f, 0.45)
+                _f_conf = "high" if _impact_s >= 0.60 else ("medium" if _impact_s >= 0.40 else "low")
+                _pr_runtime_changes.append({
+                    "path": _f,
+                    "role": _role_obj,
+                    "confidence": _f_conf,
+                    "artifact_type": _f_atype,
+                    "evidence": _evidence,
+                    "change_effect": _ARTIFACT_CHANGE_EFFECT.get(_f_atype, "modifies application logic"),
+                })
+
+            if _build_artifact_files:
+                _pr_build_changes = {
+                    "files": _build_artifact_files,
+                    "impact": "dependency/configuration only",
+                }
+
         # ── 6d. review-pr: execution paths + behavioral impact ──────────────
         _execution_paths: list[dict] = []
         _behavioral_impact: list[dict] = []
@@ -1158,6 +1240,9 @@ class TaskContextBuilder:
             scope_source=_pr_scope_source if task_name == "review-pr" else None,
             scope_files=list(_pr_scope_files) if task_name == "review-pr" and _pr_scope_files else [],
             repo_root=str(_pr_git_root) if task_name == "review-pr" and _pr_git_root else None,
+            # honest output schema: runtime vs build split (review-pr only)
+            runtime_changes=_pr_runtime_changes,
+            build_changes=_pr_build_changes,
         )
 
     def render_prompt(self, output: TaskOutput) -> str:
@@ -1990,26 +2075,8 @@ class TaskContextBuilder:
             "build_manifest": frozenset(),
         }
 
-        # deterministic change_effect descriptions per artifact type
-        _CHANGE_EFFECT: dict[str, str] = {
-            "entrypoint":     "modifies application startup, CLI entry, or framework bootstrap — all request flows may be affected",
-            "controller":     "alters HTTP routing, API contract, or response shape — API consumers are affected",
-            "service":        "changes business rules, transaction scope, or orchestration logic — callers and dependents affected",
-            "repository":     "modifies persistence queries or data access patterns — data consistency and service layer affected",
-            "mapper":         "alters SQL-to-object binding or query templates — data shape and repositories affected",
-            "security":       "changes authentication flow, access control rules, or session handling — all secured endpoints affected",
-            "spring_config":  "modifies bean wiring, datasource, or framework-wide settings — all wired beans potentially affected",
-            "spring_profile": "changes environment-specific overrides — behavior differs per active profile",
-            "config":         "adjusts configuration values — all modules reading this config are affected",
-            "build_manifest": "changes dependencies, plugins, or project structure — compile-time and runtime classpath affected",
-            "db_migration":   "modifies database schema — existing queries, mappings, and constraints may break",
-            "domain_model":   "alters entity structure — cascades to repositories, DTOs, serializers, and mappers",
-            "dto":            "changes data transfer contract — serialization and API consumers may break",
-            "test":           "modifies test coverage or test behavior — no production code affected",
-            "documentation":  "updates documentation only — no runtime impact",
-            "ide_noise":      "IDE/tooling artifact — no application impact",
-            "source":         "modifies application source — artifact role derived from file path structure",
-        }
+        # use module-level constant (single source of truth)
+        _CHANGE_EFFECT = _ARTIFACT_CHANGE_EFFECT
 
         # change_type taxonomy — closed set, derived from artifact type
         _ARTIFACT_CHANGE_TYPES: dict[str, list[str]] = {

{sourcecode-1.30.6.dist-info → sourcecode-1.30.7.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sourcecode
-Version: 1.30.6
+Version: 1.30.7
 Summary: Deterministic codebase context for AI coding agents
 License:                                  Apache License
                                    Version 2.0, January 2004
@@ -221,7 +221,7 @@ Description-Content-Type: text/markdown
 
 **Deterministic, behavior-aware codebase context for AI agents and PR review.**
 
-![Version](https://img.shields.io/badge/version-1.30.6-blue)
+![Version](https://img.shields.io/badge/version-1.30.7-blue)
 ![Python](https://img.shields.io/badge/python-3.10%2B-green)
 
 ---
@@ -257,7 +257,7 @@ pipx install sourcecode
 
 ```bash
 sourcecode version
-# sourcecode 1.30.6
+# sourcecode 1.30.7
 ```
 
 ---

{sourcecode-1.30.6.dist-info → sourcecode-1.30.7.dist-info}/RECORD

@@ -1,10 +1,10 @@
-sourcecode/__init__.py,sha256=MEBCm2OG1EipFt5XTCDIkAml5j9KHaqNtfp_BSFcGa0,103
+sourcecode/__init__.py,sha256=_36S-_MqGDmoL3m2QfY-6fG2RgOsO0Bw-a0ejwIaNW0,103
 sourcecode/adaptive_scanner.py,sha256=RTNExwWPXzjgLaRueT7UuxkPj5ZEToWjGbx1j0LSZ9E,10250
 sourcecode/architecture_analyzer.py,sha256=MyBa0Hf5HmkudZQDLKrjcWDKETXETXl0mQX1swtTwAA,39091
 sourcecode/architecture_summary.py,sha256=z34_6v7cSwy98cof2UVciGho7SCrZ93tiqMmq5WNzRQ,20405
 sourcecode/ast_extractor.py,sha256=XgrZg2DcWcUm9r87cRG3KGO7IK2TIL_N-CvhSbUmmh4,49901
 sourcecode/classifier.py,sha256=pYve2J1LqtYssU3lYLMDz18PT-CjN5c18QYE7R_IG1Q,7507
-sourcecode/cli.py,sha256=zP55Nf483vXefZa_3KG-wTZbaoxZ8ek3xKwZJHiZtng,81101
+sourcecode/cli.py,sha256=youcol8GuxebasBeta6ti50WZLZXIPH2W7JdxlZieLE,81392
 sourcecode/code_notes_analyzer.py,sha256=y1MJBnPZHYp4i6cQCXUb9ATIyifS_qMQWjw_8lPkpsU,9215
 sourcecode/confidence_analyzer.py,sha256=xw_Jv8pAd0wd8t2vvQlorw8Ih0rSF3YCoFS8K-_4aXg,15762
 sourcecode/context_scorer.py,sha256=QpChSpsmaAYz91rXA4Ue5xzQmNz_ZboZN09YOHScq1U,14679
@@ -21,7 +21,7 @@ sourcecode/flow_analyzer.py,sha256=m29PJPdAwH4n3ZNqMidgi97csSUUtav5SM9lkDy_sr8,2
 sourcecode/git_analyzer.py,sha256=_pCg2V4d2aa17k9hayTzpexAj8syvyk4y9NYNvvgOAI,12802
 sourcecode/graph_analyzer.py,sha256=iUK-7pSV-cvGqqD2hENdYmhnm0wcXFEyK-xnu5ul8OU,62515
 sourcecode/metrics_analyzer.py,sha256=m0ENgtqKeBL17kUIK3fmGkgo7UfXBNHxCMj0H_Y5K7c,22750
-sourcecode/prepare_context.py,sha256=WDsG7XA5yjtvsVvlSFn0E4rb3wVu_vUe4ztDOZ6rYfo,136632
+sourcecode/prepare_context.py,sha256=d7Ba95KFB8qqj4-EZCZYA7YTWFkQhnYQCnWPq249-NY,139888
 sourcecode/progress.py,sha256=qn30sWaHOkjTgXsSBmiPkz7Rsbwc5oSlIe6JNEMYp_k,3149
 sourcecode/ranking_engine.py,sha256=virVglafZufioHpZpwktjMvUiL0TZELWQCQnQNV8dFo,9360
 sourcecode/redactor.py,sha256=xuGcadGEHaPw4qZXlMDvzMCsr4VOkdp3oBQptHyJk8c,2884
@@ -62,8 +62,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.30.6.dist-info/METADATA,sha256=ydDuZ4ucf78fNK7tdu0BQlkCaO27zHFYT-eoOxW0CC0,26770
-sourcecode-1.30.6.dist-info/WHEEL,sha256=QccIxa26bgl1E6uMy58deGWi-0aeIkkangHcxk2kWfw,87
-sourcecode-1.30.6.dist-info/entry_points.txt,sha256=ex3F9rmbXeyDIoFQHtkEqTsKSaJow8F0LrVu8XfIktQ,57
-sourcecode-1.30.6.dist-info/licenses/LICENSE,sha256=7DdHrU9Z_3e7dSvq4ISijZNjnuHo5NIHNiHDouMQ9JU,10491
-sourcecode-1.30.6.dist-info/RECORD,,
+sourcecode-1.30.7.dist-info/METADATA,sha256=NYty06PpgrIYpT82Glyvhc9vkOxlI_wOYlV7_D6Nhy4,26770
+sourcecode-1.30.7.dist-info/WHEEL,sha256=QccIxa26bgl1E6uMy58deGWi-0aeIkkangHcxk2kWfw,87
+sourcecode-1.30.7.dist-info/entry_points.txt,sha256=ex3F9rmbXeyDIoFQHtkEqTsKSaJow8F0LrVu8XfIktQ,57
+sourcecode-1.30.7.dist-info/licenses/LICENSE,sha256=7DdHrU9Z_3e7dSvq4ISijZNjnuHo5NIHNiHDouMQ9JU,10491
+sourcecode-1.30.7.dist-info/RECORD,,