sourcecode 1.30.5__tar.gz → 1.30.7__tar.gz

sourcecode-1.30.7/.continue-here.md

@@ -0,0 +1,173 @@
+# Continue Here — atlas-cli sesión 10
+
+**Paused:** 2026-05-16 (sesión 10)
+**Repo:** `/Users/user/Downloads/atlas-cli`
+**Branch:** master
+**Último commit:** `f269c64` — (fix)solucionando bug de review-pr en repos masivos
+**Working tree:** CLEAN
+
+---
+
+## Lo que se hizo esta sesión
+
+### Fix 1: Contrato de verdad en `behavioral_impact` (Fase 1 del rediseño)
+
+Motivación: output parecía "falsa autoridad" — emitía paths sin indicar nivel de evidencia.
+
+**Cambios en `flow_analyzer.py`:**
+
+Nuevas helpers:
+- `_classify_evidence_type(clean, class_name)` → `"direct_injection" | "direct_call" | "heuristic_only" | "none"`
+- `_worst_evidence(levels)` → evidencia más débil de la cadena
+- `_compute_confidence(evidence_level, trace_len)` → `"high" | "medium" | "low"`
+- `_build_trace_step(source, target, evidence_type)` → e.g. `"ArticleController injects ArticleFavoriteService"`
+- `_impact_item(statement, support, certainty)` → dict con 3 campos
+
+**Nuevo schema de cada entry en `behavioral_impact`:**
+
+```json
+{
+  "entry_point": "ArticleController.favoriteArticle",
+  "affected_path": ["ArticleFavoriteService.favorite", "ArticleFavoriteRepository.save"],
+  "impact": [
+    {
+      "statement": "article favorite persistence affected",
+      "support": "ArticleFavoriteRepository is a repository in path",
+      "certainty": "medium"
+    }
+  ],
+  "end_state": "DB write",
+  "confidence": "high",
+  "evidence_level": "direct_injection",
+  "trace": [
+    "ArticleController injects ArticleFavoriteService",
+    "ArticleFavoriteService injects ArticleFavoriteRepository"
+  ]
+}
+```
+
+Regla: no trace → no entry. `heuristic_only` → confidence máximo `low`.
+Scope de `impact`: solo persistence + security annotations + `@Transactional`.
+
+---
+
+### Fix 2: Git-first scope resolution para `review-pr`
+
+Motivación: `review-pr` colgaba en repos grandes (Broadleaf Commerce) por traversal completo del filesystem antes de conocer el scope.
+
+**Nuevo flujo:**
+```
+ANTES: filesystem scan → heuristics → analysis
+AHORA: git diff → scope resolution → targeted loading → analysis
+```
+
+**Cambios en `prepare_context.py`:**
+
+Nuevos métodos en `TaskContextBuilder`:
+- `_resolve_git_root()` → `git rev-parse --show-toplevel`
+- `_get_pr_scope_files(since)` → union de `git diff HEAD~1` + `git diff HEAD` + staged + untracked; filtra paths `../`
+- `_expand_scope_for_analysis(scope_files)` → scope files + siblings en mismos directorios (depth=1)
+
+Nuevo Step 0 en `build()` (antes del scanner):
+- Resuelve git root + scope completo
+- Early returns para no-git y ref inválida (sin hacer ningún scan)
+
+Step 1 (scanner) — rama git-first:
+- Skip `AdaptiveScanner` completamente → `file_tree = {}`
+- `all_paths = _expand_scope_for_analysis(scope_files)` (bounded context para behavioral_impact)
+
+Step 2 (detection) — rama git-first:
+- Skip workspace sub-scans (cada workspace corría su propio `AdaptiveScanner`)
+
+Step 5d — simplificado:
+- Ya no re-corre git (scope pre-resuelto en step 0)
+- Solo: `_delta_files = set(_pr_scope_files)`
+
+Nuevos campos en `TaskOutput`: `scope_source`, `scope_files`, `repo_root`.
+
+Output JSON nuevo bloque:
+```json
+"scope": {
+  "source": "git_diff,untracked",
+  "files": [...],
+  "repo_root": "/path/to/repo"
+}
+```
+
+**Tiempo real:** 1.2s (antes: ∞ en Broadleaf).
+
+---
+
+## Estado de tests
+
+```
+777 passed, 3 skipped, 1 deselected
+```
+
+---
+
+## Commits de esta sesión
+
+| Hash | Descripción |
+|------|-------------|
+| `fc861e1` | (fix)incrementando representatividad del análisis de flujo |
+| `f269c64` | (fix)solucionando bug de review-pr en repos masivos |
+
+---
+
+## Pendiente
+
+1. **Smoke test `behavioral_impact`** en repo real spring-boot:
+   ```bash
+   cd ~/Documents/workspace/spring-boot-realworld-example-app
+   sourcecode prepare-context review-pr . 2>/dev/null | python3 -c "
+   import json, sys
+   d = json.load(sys.stdin)
+   for bi in d.get('behavioral_impact', []):
+       print('entry:', bi['entry_point'])
+       print('confidence:', bi['confidence'])
+       print('trace:', bi['trace'])
+       print()
+   "
+   ```
+
+2. **Smoke test en Broadleaf Commerce** — verificar que no cuelga:
+   ```bash
+   cd ~/Documents/workspace/broadleaf-commerce
+   time sourcecode prepare-context review-pr . 2>/dev/null | python3 -c "
+   import json, sys
+   d = json.load(sys.stdin)
+   print('scope_source:', d.get('scope', {}).get('source'))
+   print('scope_files:', len(d.get('scope', {}).get('files', [])))
+   "
+   ```
+
+3. **Evaluar si `execution_paths` sigue siendo útil** o si `behavioral_impact` lo reemplaza.
+
+4. **Fase 2 del rediseño behavioral_impact** — smoke en Broadleaf, medir false positives.
+
+5. **Tests unitarios para `_classify_diff_severity`** — sin tests.
+
+6. **`User.java` clasificado como `"source"` no `"domain_model"`** — fix opcional.
+
+---
+
+## Para retomar
+
+```bash
+cd /Users/user/Downloads/atlas-cli
+git log --oneline -3
+
+python3 -m pytest tests/ \
+  --ignore=tests/test_block2_coverage.py \
+  --ignore=tests/test_packaging.py \
+  --deselect=tests/test_dependency_analyzer_node_python.py::test_python_requirements_without_lockfile_keeps_declared_versions \
+  -q
+# Expected: 777 passed
+
+python3 -m pytest tests/test_v1_10_regressions.py::TestBehavioralImpact tests/test_v1_10_regressions.py::TestReviewPrSuspectedAreas -v
+```
+
+---
+
+*Pausado 2026-05-16 sesión 10 — gsd:pause-work*

{sourcecode-1.30.5 → sourcecode-1.30.7}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sourcecode
-Version: 1.30.5
+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.5-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.5
+# sourcecode 1.30.7
 ```
 
 ---

{sourcecode-1.30.5 → sourcecode-1.30.7}/README.md

@@ -2,7 +2,7 @@
 
 **Deterministic, behavior-aware codebase context for AI agents and PR review.**
 
-![Version](https://img.shields.io/badge/version-1.30.5-blue)
+![Version](https://img.shields.io/badge/version-1.30.7-blue)
 ![Python](https://img.shields.io/badge/python-3.10%2B-green)
 
 ---
@@ -38,7 +38,7 @@ pipx install sourcecode
 
 ```bash
 sourcecode version
-# sourcecode 1.30.5
+# sourcecode 1.30.7
 ```
 
 ---

{sourcecode-1.30.5 → sourcecode-1.30.7}/pyproject.toml

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

{sourcecode-1.30.5 → sourcecode-1.30.7}/src/sourcecode/__init__.py

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

{sourcecode-1.30.5 → sourcecode-1.30.7}/src/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:
@@ -1874,6 +1879,12 @@ def prepare_context_cmd(
             out["impact_summary"] = output.impact_summary
         if output.why_these_files:
             out["reasoning"] = output.why_these_files
+        # git-first scope metadata
+        out["scope"] = {
+            "source": output.scope_source or "git_diff",
+            "files": output.scope_files,
+            "repo_root": output.repo_root or "",
+        }
     if output.limitations:
         out["limitations"] = output.limitations
     if output.symptom: