test(map): track symbol-aware fixture index#101
Conversation
Greptile SummaryThis PR fixes a CI failure by committing the previously gitignored Confidence Score: 5/5
|
| Filename | Overview |
|---|---|
| tests/fixtures/map-fixture/.codebase-context/index.json | Adds a four-chunk symbol-aware index fixture backing the keyInterfaces tests. Minor inconsistency: chunk c2 declares endLine: 20 but its content is only 6 lines (ending at line ~15). The buildId: "test-build-13" warrants a clarification that this was pre-designed rather than retroactively tuned to pass tests per AGENTS.md Evaluation Integrity rules. |
| .gitignore | Adds a targeted negation exception !tests/fixtures/map-fixture/.codebase-context/index.json after the broad **/.codebase-context/* exclusion. Pattern ordering is correct and the directory itself is not excluded, so the negation will take effect as expected. |
Sequence Diagram
sequenceDiagram
participant CI as CI Runner
participant Git as Git / .gitignore
participant Test as codebase-map.test.ts
participant Fixture as map-fixture/.codebase-context/index.json
participant Builder as buildCodebaseMap()
CI->>Git: checkout repo
Git-->>CI: index.json now tracked (negation exception added)
CI->>Test: pnpm test
Test->>Builder: createProjectState(FIXTURE_ROOT)
Builder->>Fixture: read index.json
Fixture-->>Builder: 4 chunks (3 symbolAware)
Builder-->>Test: "CodebaseMapSummary { keyInterfaces: [...] }"
Test->>Test: "assert keyInterfaces[0].name === 'SearchOptions'"
Test->>Test: assert signatureHint, kind, file, importerCount
Test-->>CI: ✅ pass
Reviews (1): Last reviewed commit: "test(map): track symbol-aware fixture in..." | Re-trigger Greptile
|
Follow-up on the Greptile review: commit 6aab54c fixes the c2 metadata mismatch by aligning �ndLine with the embedded synthetic fixture content, and replaces the misleading est-build-13 label with the stable fixture identifier map-fixture-v1. This fixture is hand-maintained regression data for codebase-map, not a frozen eval artifact. |
Summary
This restores the tracked fixture data that
tests/codebase-map.test.tsexpects for symbol-awarekeyInterfacescoverage.What changed
tests/fixtures/map-fixture/.codebase-context/index.json.gitignoreexception so this fixture artifact stays tracked in GitWhy
origin/masterwas red because the fixture index was only present locally and ignored by Git, so CI never saw the symbol-aware chunks that back thekeyInterfacesassertions and markdown snapshot.Verification
pnpm test -- tests/codebase-map.test.tspnpm run type-checkThe full local suite on this worktree still shows unrelated baseline timeouts/build prerequisites outside this fix; this PR is intentionally limited to the concrete GitHub failure it repairs.