You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
index_status reports a whole-file parse-error range (src/cli/cli.c as 1-13047), which is not actionable #1984
Built from source at ae5de7fe (main), release line v0.10.8
Platform
macOS (Apple Silicon)
Install channel
Built from source
Binary variant
standard
What happened, and what did you expect?
index_status reports a parse-error range covering an entire file when only a small part
of it failed to parse. On this repository it reported src/cli/cli.c as one error range
of 1-13047 — the whole file. The file indexed fine; the report was wrong.
A range that names the whole file is not actionable. "Grep 13,000 lines for the problem"
is the same as no signal at all, and it makes the coverage report look far worse than the
indexing actually is.
I expected the reported range to name the region that actually failed, so it points at
something a person can open and read.
Reproduction
Index this repository itself: codebase-memory-mcp cli index_repository '{"repo_path":"<path to this repo>"}'
Call check_index_coverage (or index_status) and read the reported error ranges.
Result: src/cli/cli.c is reported as a single error range 1-13047, the full file.
Expected: a range naming the region that failed to parse.
This is the parent defect of the coverage-report work tracked in #963, which is closed.
The two narrower faults — a duplicate range, and a range ending one line past the end of
the file — are already tracked separately as #1965 and #1966. This issue is the remaining
one: the range itself is far too wide.
Thank you for separating the non-actionable whole-file range from the duplicate and past-EOF faults. PR #1941 is the focused implementation: it uses the existing preprocessed parse to narrow C-family ranges and gives genuinely whole-file failures the explicit parse_unusable result instead of pretending that a 13,000-line range is useful. Leaving this open until that PR lands.
Fixed on main by aa44c28 (#1941, thanks to @CaptainMittens): whole-file parse-error ranges are now named for what they are and the coverage report no longer hides what it dropped.
Activity
DeusData commentedon Sep 1, 2026
Thank you for separating the non-actionable whole-file range from the duplicate and past-EOF faults. PR #1941 is the focused implementation: it uses the existing preprocessed parse to narrow C-family ranges and gives genuinely whole-file failures the explicit
parse_unusableresult instead of pretending that a 13,000-line range is useful. Leaving this open until that PR lands.DeusData commentedon Sep 6, 2026
Fixed on main by aa44c28 (#1941, thanks to @CaptainMittens): whole-file parse-error ranges are now named for what they are and the coverage report no longer hides what it dropped.