test: agent-task benchmark on the corpus — grade the product, with a token ratchet #51
Labels
No labels
code-review
correctness
dos
performance
security
severity/high
severity/low
severity/medium
tech-debt
Kind/Breaking
Kind/Bug
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/Security
Kind/Testing
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Reviewed
Confirmed
Reviewed
Duplicate
Reviewed
Invalid
Reviewed
Won't Fix
Status
Abandoned
Status
Blocked
Status
Need More Info
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
h-dv/code-index#51
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
P3 — the only metric here that measures the actual value proposition. Depends on #40, #45.
From
_prdoc/records/brainstorm-2026-07-28-oss-corpus-test-system.md§5.3.Why
Every other corpus metric measures internals (refs resolved, spans stable, phantoms absent). None would notice a change that improves resolution while making the tools worse to use — wrong shape, missing disclosure, more round-trips, bloated payloads.
The product claim is "the right answer at roughly 10× less context than Read/grep." That claim is currently only prose.
What
~20 realistic questions per corpus repo, each with a hand-verified answer:
X?Y?Zsafe to delete?W?Scored on correctness (did it find every true answer, no fabrications) and token cost (bytes/tokens to reach it, round-trips needed).
Compare against a ripgrep+Read baseline on the same question, counting the baseline's false positives — yielding precision-per-token, which is the moat claim expressed as a number.
Replaces a known-vacuous test
crates/mcp-server/tests/agent_workflow_bench.rsis#[ignore]d, has 1 assert against 4println!, runs one Rust project, and measures self-reported bytes. It cannot catch a regression. This supersedes it — and per #38's framing, that vacuity is precisely why it must be rebuilt with a success oracle rather than extended.Ratchet
Tokens-per-correct-answer lands in
baseline.json(#45). A change that answers the same questions using materially more context fails, even if every internal metric improved.Acceptance
#[ignore]d bench deleted, not left alongside