sourcecode 1.27.0__tar.gz → 1.28.0__tar.gz

sourcecode-1.28.0/.continue-here.md

@@ -0,0 +1,108 @@
+# Continue Here — atlas-cli --agent token reduction
+
+**Paused:** 2026-05-16 (sesión 4)
+**Repo:** `/Users/user/Downloads/atlas-cli`
+**Branch:** master
+**Commit:** `f1cb001 corrigiendo bug --agent que duplicaba tokens sin justificación` (committed)
+
+---
+
+## Estado actual
+
+Sesión 4: refactorización de `agent_view()` en `serializer.py` completada. **Todo committed.** Working tree limpio.
+
+---
+
+## Qué se hizo esta sesión
+
+### Problema resuelto
+
+`--agent` producía ~11.337 tokens en SAS vs ~5.885 de `--compact`, con contenido casi idéntico. Objetivo: reducir tokens significativamente manteniendo solo lo que diferencia `--agent`.
+
+### Cambios en `agent_view()` (`src/sourcecode/serializer.py`)
+
+| Cambio | Impacto |
+|--------|---------|
+| Elimina `production_dependencies` / `dev_tools` / `test_utilities` / `build_tooling` (dep_groups asdict completo) | Mayor fuente de bloat: -3.000-5.000 tokens en repos grandes |
+| `key_dependencies`: de asdict+50 items → `name+version+role+risk_flags` solo, cap 20 | -500-1.500 tokens |
+| Elimina `confidence_summary.hard_signals` / `soft_signals` / `ignored_signals` | -500-1.000 tokens |
+| Elimina bloque `agent_mode` (contenido estático sin señal) | -80 tokens siempre |
+| Elimina `signals.env_vars.keys[]` (duplica compact `env_map`) | -300-500 tokens |
+| `signals.code_notes.top` cap 10→5, strip a `kind+path+line+text` | -200 tokens |
+| Entry points (no-bootstrap): strip a `path+kind+confidence` (como compact) | reducción moderada |
+| Elimina `development_entry_points` (noise para agentes de producción) | pequeña |
+| `architecture.layers` capado a 5 | pequeña |
+
+**Reducción estimada para SAS:** −40–60% tokens vs `--agent` anterior.
+
+### Campos que permanecen únicos de `--agent`
+
+| Campo | Por qué justifica tokens extra vs `--compact` |
+|-------|----------------------------------------------|
+| `file_relevance` (top-20 scored) | Compact no tiene equivalente. Agente sabe qué leer sin explorar. |
+| `architecture.layers` | Compact solo tiene `architecture_summary` string. |
+| `suspicious_dependencies` | Deps declarados sin import observado. Compact no lo tiene. |
+| `confidence_reasons` | Explicación accionable de secciones low-confidence. Compact no lo tiene. |
+
+### Tests actualizados
+
+- `tests/test_block5_quality.py` — 5 tests de `agent_mode` reemplazados por tests de las nuevas garantías (no `agent_mode`, no `hard_signals`, no `env_vars.keys`, no dep_groups)
+- `tests/test_pipeline_integrity.py` — `test_agent_splits_development_and_auxiliary_eps` → verifica que dev/aux EPs NO aparecen
+
+---
+
+## Tests al pausar
+
+Suite completa (sin fallos pre-existentes): **todo verde**
+
+Fallos pre-existentes (no relacionados, ya presentes antes):
+- `test_block2_coverage.py::test_java_marked_unsupported` — DocRecord bug en doc_analyzer.py
+- `test_dependency_analyzer_node_python.py::test_python_requirements_without_lockfile` — versión typer mismatch
+
+---
+
+## Pendiente / próximos pasos opcionales
+
+1. **Smoke test en saint-server** para medir tokens reales antes/después:
+   ```bash
+   sourcecode saint-server --agent --output /tmp/agent_new.json
+   python3 -c "import json,sys; d=json.load(open('/tmp/agent_new.json')); print('agent tokens (est):', len(json.dumps(d))//4)"
+   ```
+
+2. **Verificar que `suspicious_dependencies` es útil en repos reales** — en repos Python/JS (no JVM, donde el index de imports no funciona). En JVM siempre vacío por diseño.
+
+3. **Ajustar caps según medición real**: si `file_relevance` (20 files) sigue siendo el mayor contribuyente de tokens, considerar reducir a 15. Medir con `--full` vs default.
+
+4. **Smoke tests delta pendientes** de sesión anterior — ver sección "Smoke tests sesión 3" en historial de commits.
+
+---
+
+## Para retomar
+
+```bash
+cd /Users/user/Downloads/atlas-cli
+git log --oneline -3
+# debe mostrar: f1cb001 corrigiendo bug --agent que duplicaba tokens sin justificación
+
+python3 -m pytest tests/ \
+  --ignore=tests/test_block2_coverage.py \
+  --deselect=tests/test_dependency_analyzer_node_python.py::test_python_requirements_without_lockfile_keeps_declared_versions \
+  -q
+# Expected: todo verde
+
+# Verificación rápida agent_view
+python3 -c "
+from sourcecode.serializer import agent_view
+from sourcecode.schema import SourceMap, StackInfo, SourceMapMetadata
+sm = SourceMap(stacks=[StackInfo(stack='python', primary=True)], project_type='python', project_summary='test')
+r = agent_view(sm)
+assert 'agent_mode' not in r
+assert 'production_dependencies' not in r
+assert 'hard_signals' not in r.get('confidence_summary', {})
+print('OK — agent_mode/dep_groups/hard_signals all absent')
+"
+```
+
+---
+
+*Pausado 2026-05-16 sesión 4 — gsd:pause-work*

{sourcecode-1.27.0 → sourcecode-1.28.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: sourcecode
-Version: 1.27.0
+Version: 1.28.0
 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
 
 **Compressed AI-ready context for Java/Spring enterprise codebases.**
 
-![Version](https://img.shields.io/badge/version-1.27.0-blue)
+![Version](https://img.shields.io/badge/version-1.28.0-blue)
 ![Python](https://img.shields.io/badge/python-3.10%2B-green)
 
 ---
@@ -255,7 +255,7 @@ pipx install sourcecode
 
 ```bash
 sourcecode version
-# sourcecode 1.27.0
+# sourcecode 1.28.0
 ```
 
 ---

{sourcecode-1.27.0 → sourcecode-1.28.0}/README.md

@@ -2,7 +2,7 @@
 
 **Compressed AI-ready context for Java/Spring enterprise codebases.**
 
-![Version](https://img.shields.io/badge/version-1.27.0-blue)
+![Version](https://img.shields.io/badge/version-1.28.0-blue)
 ![Python](https://img.shields.io/badge/python-3.10%2B-green)
 
 ---
@@ -36,7 +36,7 @@ pipx install sourcecode
 
 ```bash
 sourcecode version
-# sourcecode 1.27.0
+# sourcecode 1.28.0
 ```
 
 ---

{sourcecode-1.27.0 → sourcecode-1.28.0}/pyproject.toml

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

{sourcecode-1.27.0 → sourcecode-1.28.0}/src/sourcecode/__init__.py

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

{sourcecode-1.27.0 → sourcecode-1.28.0}/src/sourcecode/prepare_context.py

@@ -2309,10 +2309,11 @@ class TaskContextBuilder:
             r = subprocess.run(
                 ["git", "branch", "-a", "--format=%(refname:short)"],
                 cwd=str(self.root),
-                capture_output=True, text=True, timeout=5,
+                capture_output=True, text=True,
+                encoding="utf-8", errors="replace", timeout=5,
             )
             if r.returncode == 0:
-                all_refs = [b.strip() for b in r.stdout.splitlines() if b.strip()]
+                all_refs = [b.strip() for b in (r.stdout or "").splitlines() if b.strip()]
                 branches = [b for b in all_refs if "HEAD" not in b][:10]
                 ref_lower = invalid_ref.lower()
                 if ref_lower == "master" and any(b.rstrip("/").endswith("main") for b in all_refs):

sourcecode-1.28.0/tests/test_encoding_regression.py

@@ -0,0 +1,138 @@
+"""Regression tests for C3 — UTF-8 double-encoding of accented characters.
+
+Fixture files:
+  - tests/fixtures/latin1_sample_iso.java  — Latin-1 encoded (bytes 0xe1, 0xf3, etc.)
+  - tests/fixtures/latin1_sample.java      — UTF-8 encoded (multibyte sequences)
+
+Both should decode cleanly to the correct Unicode characters.
+The old code read Latin-1 files with errors='replace', producing double-encoded garbage.
+"""
+from __future__ import annotations
+
+from pathlib import Path
+
+import pytest
+
+from sourcecode.tree_utils import safe_read_text
+
+FIXTURES = Path(__file__).parent / "fixtures"
+LATIN1_FILE = FIXTURES / "latin1_sample_iso.java"
+UTF8_FILE = FIXTURES / "latin1_sample.java"
+
+
+class TestSafeReadText:
+    """safe_read_text must decode both UTF-8 and Latin-1 files correctly."""
+
+    def test_utf8_file_reads_correctly(self):
+        content = safe_read_text(UTF8_FILE)
+        # These characters must appear verbatim — not as double-encoded sequences
+        assert "días" in content, f"Expected 'días', got: {content[:200]!r}"
+        assert "ñ" in content or "función" in content or "ñoño" in content
+        assert "Ã" not in content, "Double-encoded: U+00C3 found (Ã). Encoding was mangled."
+
+    def test_latin1_file_reads_correctly(self):
+        content = safe_read_text(LATIN1_FILE)
+        # Latin-1 0xe1 = 'á' — must decode to the correct character
+        assert "á" in content, f"Expected 'á', got: {content!r}"
+        assert "ó" in content, f"Expected 'ó', got: {content!r}"
+        # No replacement chars
+        assert "�" not in content, "Replacement character found — file was not decoded correctly."
+        # No double-encoding: 'á' in double-encoded UTF-8-as-Latin-1 would appear as 'á'
+        assert "Ã" not in content, "Double-encoded: U+00C3 found (Ã). Latin-1 file was not decoded correctly."
+
+    def test_nonexistent_file_raises_oserror(self):
+        with pytest.raises(OSError):
+            safe_read_text(FIXTURES / "does_not_exist.java")
+
+
+class TestCodeNotesEncoding:
+    """CodeNotesAnalyzer must extract notes from Latin-1 files without encoding corruption."""
+
+    def test_latin1_todo_extracted_cleanly(self):
+        from sourcecode.code_notes_analyzer import CodeNotesAnalyzer
+
+        # analyze() scans FIXTURES dir — Latin-1 file is there
+        analyzer = CodeNotesAnalyzer()
+        notes, _, _ = analyzer.analyze(FIXTURES)
+        # Should have extracted TODO without replacement chars
+        todo_texts = [n.text for n in notes if "latin1_sample_iso" in n.path]
+        for text in todo_texts:
+            assert "â" not in text, f"Double-encoded byte found in TODO text: {text!r}"
+            assert "Ã" not in text, f"Double-encoded char in TODO text: {text!r}"
+            assert "�" not in text, f"Replacement char in TODO text: {text!r}"
+
+
+class TestGetAvailableRefsWindowsEncoding:
+    """Regression: _get_available_refs crashed on Windows due to cp1252 encoding and None stdout.
+
+    Bug: subprocess.run(text=True) without encoding= uses the OS default (cp1252 on Windows).
+    Git output containing byte 0x8d raises UnicodeDecodeError (not caught). If stdout is None,
+    r.stdout.splitlines() raises AttributeError. Both cause unhandled tracebacks.
+    Fix: encoding="utf-8", errors="replace", and (r.stdout or "").splitlines().
+    """
+
+    def _make_builder(self) -> "TaskContextBuilder":
+        from sourcecode.prepare_context import TaskContextBuilder
+        return TaskContextBuilder(FIXTURES)
+
+    def test_case_a_stdout_none_does_not_crash(self, monkeypatch):
+        """stdout=None must not raise AttributeError on splitlines()."""
+        import subprocess
+        import types
+
+        fake = types.SimpleNamespace(returncode=0, stdout=None, stderr="")
+        monkeypatch.setattr(subprocess, "run", lambda *a, **kw: fake)
+
+        builder = self._make_builder()
+        refs, suggested = builder._get_available_refs("main")
+        assert refs == []
+        assert suggested is None
+
+    def test_case_b_non_utf8_bytes_no_unicode_error(self, monkeypatch):
+        """Bytes invalid in cp1252 (e.g. 0x8d) must not raise UnicodeDecodeError.
+
+        With encoding='utf-8' + errors='replace', replacement chars appear instead.
+        """
+        import subprocess
+        import types
+
+        # Simulate git returning a branch name that survived errors="replace" decoding
+        # (the replacement char � stands in for the bad byte)
+        replaced = "develop\nfeature/caf�\n"
+        fake = types.SimpleNamespace(returncode=0, stdout=replaced, stderr="")
+        monkeypatch.setattr(subprocess, "run", lambda *a, **kw: fake)
+
+        builder = self._make_builder()
+        refs, _ = builder._get_available_refs("main")
+        assert "develop" in refs
+
+    def test_case_c_nonexistent_ref_returns_available_refs(self, monkeypatch):
+        """Non-existent --since ref must yield a list of available branches, not a crash."""
+        import subprocess
+        import types
+
+        fake = types.SimpleNamespace(
+            returncode=0,
+            stdout="develop\norigin/develop\nrelease/1.2\n",
+            stderr="",
+        )
+        monkeypatch.setattr(subprocess, "run", lambda *a, **kw: fake)
+
+        builder = self._make_builder()
+        refs, suggested = builder._get_available_refs("main")
+        assert "develop" in refs
+        # invalid_ref="main", all_refs contains no "master" → no suggestion
+        assert suggested is None
+
+    def test_case_d_nonzero_returncode_returns_empty(self, monkeypatch):
+        """returncode != 0 must not parse stdout — returns empty list silently."""
+        import subprocess
+        import types
+
+        fake = types.SimpleNamespace(returncode=128, stdout="fatal: not a git repo\n", stderr="")
+        monkeypatch.setattr(subprocess, "run", lambda *a, **kw: fake)
+
+        builder = self._make_builder()
+        refs, suggested = builder._get_available_refs("main")
+        assert refs == []
+        assert suggested is None

sourcecode-1.27.0/.continue-here.md

@@ -1,173 +0,0 @@
-# Continue Here — atlas-cli delta engine overhaul
-
-**Paused:** 2026-05-13 (sesión 3)
-**Repo:** `/Users/user/Downloads/atlas-cli`
-**Branch:** master
-**Commit:** `8258c4c implementando delta` (committed, pushed to origin/master)
-
----
-
-## Estado actual
-
-Sesión 3: `prepare-context delta` completamente reimplementado como motor de contexto incremental real. **Todo committed y pusheado.** Working tree limpio (solo este fichero untracked).
-
----
-
-## Qué se implementó esta sesión
-
-### Motor delta real (`_build_delta_impact`)
-
-Reemplaza el uso de `_rank_files` para la tarea `delta`. Nuevo flujo:
-
-1. **`_classify_changed_file(path)`** — clasificador estático determinista por ruta/nombre:
-   - Tipos: `security`, `api_endpoint`, `business_logic`, `data_access`, `mybatis_mapper`, `spring_config`, `spring_profile`, `configuration`, `build_manifest`, `db_migration`, `domain_model`, `dto`, `test`, `ide_noise`, `unknown_source`, `unknown_config`, `documentation`
-   - Impacto: `critical / high / medium / low / noise`
-   - Áreas de riesgo: `security`, `api`, `transactions`, `business_logic`, `persistence`, `config`, `dependencies`, `tests`
-   - `.idea/`, `.vscode/`, etc. → `ide_noise`, `is_noise=True`
-
-2. **`_build_delta_impact(changed_files, all_paths, entry_points, since)`** — análisis de impacto:
-   - Changed files siempre incluidos en `relevant_files` (nunca filtrados por score)
-   - Expansión a ficheros relacionados: mismo módulo DDD o mismo directorio padre
-   - Solo expande a stems de alto impacto (controller, service, repository, mapper, security, config)
-   - Límite: 10 ficheros relacionados máximo
-   - Returns: `(relevant_files, impact_summary, affected_modules, risk_areas, why, analysis_gaps)`
-
-3. **Nuevos campos en `TaskOutput`** (delta only):
-   - `impact_summary: str` — resumen textual del impacto global
-   - `affected_modules: list[str]` — módulos DDD afectados
-   - `risk_areas: list[dict]` — por área con severity y ficheros
-   - `since: str` — ref usada
-
-4. **Nuevo output delta** (cli.py):
-   - `project_summary` suprimido (change-focused, no snapshot)
-   - `gaps` habilitado (delta-specific gaps, no ConfidenceAnalyzer gaps)
-   - Serializa: `since`, `impact_summary`, `affected_modules`, `risk_areas`, `reasoning`
-
-### Bugs corregidos
-
-| Bug | Fix |
-|-----|-----|
-| `relevant_files: []` con cambios reales | Changed files usan `_build_delta_impact`, no `_rank_files`. Nunca filtrados por score. |
-| `.idea/vcs.xml` → `relevant_files: []` | Clasificado como `ide_noise`, score=0.1, role="noise". Aparece con mensaje explicativo. |
-| Output idéntico a `--compact` | Delta ya no llama `_rank_files` — lógica completamente separada. |
-| `relevant_files` genérico sin relación al diff | Ahora construye desde changed_files hacia afuera, no desde entrypoints hacia adentro. |
-
----
-
-## Ficheros modificados esta sesión
-
-```
-src/sourcecode/prepare_context.py   +_classify_changed_file, +_build_delta_impact, +TaskOutput fields, build() delta branch
-src/sourcecode/cli.py               delta content filter: project_summary=False, gaps=True, new fields serialized
-tests/test_signal_hierarchy.py      test_delta_task_returns_valid_json: project_summary not in data, impact_summary in data
-```
-
----
-
-## Tests
-
-- **743 passed, 3 skipped** — sin regresiones nuevas
-- Fallos pre-existentes (no relacionados):
-  - `test_block2_coverage.py::test_java_marked_unsupported`
-  - `test_dependency_analyzer_node_python.py::test_python_requirements_without_lockfile`
-  - `test_pipeline_integrity.py::TestBenchmarkContamination`
-
----
-
-## Smoke tests pendientes (necesitan saint-server)
-
-### Smoke delta (nuevo)
-
-```bash
-# Test básico: relevant_files no vacío con cambios reales
-cd saint-server
-git diff --name-only main | head -5  # confirmar hay cambios
-sourcecode prepare-context delta . --since main --output /tmp/delta_test.json
-python3 -c "
-import json; d=json.load(open('/tmp/delta_test.json'))
-assert d['task'] == 'delta'
-assert len(d['relevant_files']) > 0, 'relevant_files is EMPTY — bug not fixed'
-assert 'impact_summary' in d, 'missing impact_summary'
-assert 'affected_modules' in d or 'risk_areas' in d, 'no impact fields'
-assert 'project_summary' not in d, 'project_summary should be suppressed'
-print('delta PASS:', d['impact_summary'])
-print('changed_files:', d.get('changed_files', [])[:3])
-print('affected_modules:', d.get('affected_modules', []))
-"
-
-# Test IDE noise: .idea change → baja prioridad
-python3 -c "
-from sourcecode.prepare_context import TaskContextBuilder
-from pathlib import Path
-b = TaskContextBuilder.__new__(TaskContextBuilder)
-b.root = Path('.')
-relevant, impact, modules, risks, why, gaps = b._build_delta_impact(
-    changed_files=['.idea/vcs.xml'],
-    all_paths=[],
-    entry_points=[],
-    since='main',
-)
-assert len(relevant) == 1
-assert relevant[0].role == 'noise'
-assert relevant[0].score == 0.1
-assert 'no semantic impact' in impact
-print('IDE noise PASS:', impact)
-"
-
-# Test seguridad: cambio en SecurityConfig → impacto critical
-python3 -c "
-from sourcecode.prepare_context import TaskContextBuilder
-b = TaskContextBuilder.__new__(TaskContextBuilder)
-cls = b._classify_changed_file('src/.../security/SecurityConfig.java')
-assert cls['artifact_type'] == 'security'
-assert cls['impact_level'] == 'critical'
-assert 'security' in cls['risk_areas']
-print('security PASS:', cls)
-"
-```
-
-### Smoke tests sesión 2 (pendientes desde antes)
-
-```bash
-# FIX-1: síntoma inyecta ficheros en fix-bug
-sourcecode prepare-context fix-bug saint-server --symptom "ausencias contador" \
-  --output /tmp/fix_symptom.json
-
-# FIX-2: resource_names resueltos en security_surface  
-sourcecode saint-server --compact --output /tmp/sec_test.json
-
-# FIX-4: score presente en file_relevance
-sourcecode saint-server --agent --output /tmp/agent_test.json
-
-# (ver .continue-here.md sesión 2 para comandos completos de verificación)
-```
-
----
-
-## Mejoras opcionales para sesión siguiente
-
-1. **`_classify_changed_file` con más patrones**: añadir `cli` (como `cli.py`) → `cli_entrypoint`, `interceptor` → `security`, `aspect` → `cross_cutting`
-2. **`--since` sin argumento**: en vez de default `HEAD~1`, inferir mejor (e.g., base del branch actual vs main)
-3. **Soporte para staged/untracked**: cuando `since` es None, incluir `git status --porcelain` para capturar staged + untracked (no solo `git diff --name-only --relative`)
-4. **Test unitario dedicado** para `_build_delta_impact` y `_classify_changed_file` en `tests/test_task_differentiation.py`
-
----
-
-## Para retomar
-
-```bash
-cd /Users/user/Downloads/atlas-cli
-git log --oneline -3   # debe mostrar 8258c4c implementando delta
-python3 -m pytest tests/ \
-  --ignore=tests/test_block2_coverage.py \
-  --ignore=tests/test_dependency_analyzer_node_python.py \
-  --ignore=tests/test_pipeline_integrity.py -q
-# Expected: 743 passed, 3 skipped
-
-# Smoke test rápido del delta
-python3 run_cli.py prepare-context delta . --since HEAD~3 | python3 -m json.tool | head -30
-```
-
----
-
-*Pausado 2026-05-13 sesión 3 — gsd:pause-work*

sourcecode-1.27.0/tests/test_encoding_regression.py

@@ -1,62 +0,0 @@
-"""Regression tests for C3 — UTF-8 double-encoding of accented characters.
-
-Fixture files:
-  - tests/fixtures/latin1_sample_iso.java  — Latin-1 encoded (bytes 0xe1, 0xf3, etc.)
-  - tests/fixtures/latin1_sample.java      — UTF-8 encoded (multibyte sequences)
-
-Both should decode cleanly to the correct Unicode characters.
-The old code read Latin-1 files with errors='replace', producing double-encoded garbage.
-"""
-from __future__ import annotations
-
-from pathlib import Path
-
-import pytest
-
-from sourcecode.tree_utils import safe_read_text
-
-FIXTURES = Path(__file__).parent / "fixtures"
-LATIN1_FILE = FIXTURES / "latin1_sample_iso.java"
-UTF8_FILE = FIXTURES / "latin1_sample.java"
-
-
-class TestSafeReadText:
-    """safe_read_text must decode both UTF-8 and Latin-1 files correctly."""
-
-    def test_utf8_file_reads_correctly(self):
-        content = safe_read_text(UTF8_FILE)
-        # These characters must appear verbatim — not as double-encoded sequences
-        assert "días" in content, f"Expected 'días', got: {content[:200]!r}"
-        assert "ñ" in content or "función" in content or "ñoño" in content
-        assert "Ã" not in content, "Double-encoded: U+00C3 found (Ã). Encoding was mangled."
-
-    def test_latin1_file_reads_correctly(self):
-        content = safe_read_text(LATIN1_FILE)
-        # Latin-1 0xe1 = 'á' — must decode to the correct character
-        assert "á" in content, f"Expected 'á', got: {content!r}"
-        assert "ó" in content, f"Expected 'ó', got: {content!r}"
-        # No replacement chars
-        assert "�" not in content, "Replacement character found — file was not decoded correctly."
-        # No double-encoding: 'á' in double-encoded UTF-8-as-Latin-1 would appear as 'á'
-        assert "Ã" not in content, "Double-encoded: U+00C3 found (Ã). Latin-1 file was not decoded correctly."
-
-    def test_nonexistent_file_raises_oserror(self):
-        with pytest.raises(OSError):
-            safe_read_text(FIXTURES / "does_not_exist.java")
-
-
-class TestCodeNotesEncoding:
-    """CodeNotesAnalyzer must extract notes from Latin-1 files without encoding corruption."""
-
-    def test_latin1_todo_extracted_cleanly(self):
-        from sourcecode.code_notes_analyzer import CodeNotesAnalyzer
-
-        # analyze() scans FIXTURES dir — Latin-1 file is there
-        analyzer = CodeNotesAnalyzer()
-        notes, _, _ = analyzer.analyze(FIXTURES)
-        # Should have extracted TODO without replacement chars
-        todo_texts = [n.text for n in notes if "latin1_sample_iso" in n.path]
-        for text in todo_texts:
-            assert "â" not in text, f"Double-encoded byte found in TODO text: {text!r}"
-            assert "Ã" not in text, f"Double-encoded char in TODO text: {text!r}"
-            assert "�" not in text, f"Replacement char in TODO text: {text!r}"