B12: Dedicated tests for graph + fingerprints + dead_code #15

New issue

Open

opened 2026-05-05 02:01:13 +02:00 by claude · 0 comments

claude commented

2026-05-05 02:01:13 +02:00

Collaborator

Ticket B12

Critical-path modules untested directly: graph.py (Tarjan), fingerprints.py (stability contract), dead_code.py (core analysis). Add tests/test_graph.py, test_fingerprints.py, test_dead_code.py.

Source: F-12 in 2026-05-03 audit. Land after B1-B11 to test post-fix behavior.

Detailed plan

The full ticket brief — TDD plan, current state with file:line references, target state, acceptance criteria, edge cases, hostile reviewer questions, out-of-scope — lives at .codex/MASTER/PHASE-B/B12-coverage-critical-modules.md in the operator's local working tree (the .codex/ directory is gitignored, so this content is intentionally not in repo history; it's pre-execution planning material that gets updated as evidence comes in).

When picking up this issue, read the local file first. If you're an external contributor without access to the operator's .codex/, comment on this issue and the operator will paste the brief.

Phase context

This is part of Phase B (post-Phase-A) work. Execution is paused until dogfood evidence accumulates from real-world pyfallow integration in operator's other repos (window: 2026-05-04 → ~2026-06-15).

After the window closes, dogfood evidence drives ticket prioritization:

Tickets validated by evidence stay (and may be refined)
Tickets contradicted or made obsolete get closed with rationale
New issues with dogfood:* labels may pre-empt Phase B priority

See docs/dogfood.md and docs/dogfood-log-template.md for the protocol.

Labels applied

phase:b — phase grouping
severity:medium — severity (audit-derived)
area:test-coverage — area of impact

— Migrated from .codex/MASTER/PHASE-B/ by Claude Opus 4.7 (orchestrator) on 2026-05-05 per operator request to surface Phase B/C work in Forgejo Issues so it doesn't disappear with session context.

## Ticket B12 Critical-path modules untested directly: `graph.py` (Tarjan), `fingerprints.py` (stability contract), `dead_code.py` (core analysis). Add tests/test_graph.py, test_fingerprints.py, test_dead_code.py. **Source:** F-12 in 2026-05-03 audit. Land **after** B1-B11 to test post-fix behavior. ## Detailed plan The full ticket brief — TDD plan, current state with file:line references, target state, acceptance criteria, edge cases, hostile reviewer questions, out-of-scope — lives at `.codex/MASTER/PHASE-B/B12-coverage-critical-modules.md` in the operator's local working tree (the `.codex/` directory is gitignored, so this content is intentionally not in repo history; it's pre-execution planning material that gets updated as evidence comes in). When picking up this issue, read the local file first. If you're an external contributor without access to the operator's `.codex/`, comment on this issue and the operator will paste the brief. ## Phase context This is part of **Phase B** (post-Phase-A) work. Execution is **paused** until dogfood evidence accumulates from real-world pyfallow integration in operator's other repos (window: 2026-05-04 → ~2026-06-15). After the window closes, dogfood evidence drives ticket prioritization: - Tickets validated by evidence stay (and may be refined) - Tickets contradicted or made obsolete get closed with rationale - New issues with `dogfood:*` labels may pre-empt Phase B priority See `docs/dogfood.md` and `docs/dogfood-log-template.md` for the protocol. ## Labels applied - `phase:b` — phase grouping - `severity:medium` — severity (audit-derived) - `area:test-coverage` — area of impact — Migrated from `.codex/MASTER/PHASE-B/` by Claude Opus 4.7 (orchestrator) on 2026-05-05 per operator request to surface Phase B/C work in Forgejo Issues so it doesn't disappear with session context.

claude added the

area:test-coverage

severity:medium

phase:b

labels

2026-05-05 02:01:13 +02:00

claude added this to the Phase B — Engineering hardening milestone

2026-05-05 02:08:05 +02:00