Repository navigation
Report how many files each run read, and how many it used - #37
Merged
Merged
Conversation
Sol has no file access: files reach it only as the attachments the client pasted into the packet. So what a run could read is exactly what the Attached Paths section produced, and that is now counted once when the packet passes its integrity checks and stored on the job. History rows, the run fact table, and the answered review all report it without decrypting a packet again, which matters because the packet leaves retention before the record does. The second number is the one worth having. The result view separates the paths the review actually names from the ones that were attached and never mentioned, because an attachment nothing refers to did no work. A run reading "3 of 7" says the evidence sent was larger than the evidence the decision turned on. A path counts as referenced when the released text names it, or names its file name where the full path would be unwieldy. A base name shorter than four characters is not matched, so an incidental word cannot fake a citation. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01FxxEQsJEXq9hZmjpNH5PxY
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Sol has no file access — files reach it only as the attachments the client pasted into the packet. So "files inspected" is exactly what
## Attached Pathsproduced, and that is now counted, stored, and shown per run.Two numbers, because one of them isn't the interesting one
Files read: 3 of 7Files read: 7 attachedThe gap is the useful part: an attachment nothing refers to did no work. A run reading
3 of 7tells you the evidence you sent was larger than the evidence the decision turned on.The result view lists the referenced paths separately from the attached-but-never-mentioned ones.
Where it appears
… · 2 files, visible before opening a reviewFiles read, beside Protocol and ResearchCounted once, at commit
commitJobalready reads the packet for its integrity checks, so the count is taken there and stored on the job. History and the answered review then report it without decrypting a packet again — which matters because the packet leaves retention before the record does.Matching rule
A path counts as referenced when the released text names it, or names its file name where the full path would be unwieldy (
overlay.cppforDmaKit/overlay.cpp). A base name shorter than four characters is not matched, so an incidental word can't fake a citation.Verification
analyzePacketQualitynames the paths in order and returns an empty list when there are noneattachedFiles: 2at commit, the paths reach the phone, and the count survives onto the answered job; a packet with none records zeroFILES | 2 attached, 48 Bwith both paths listed,FILES READ | 2 attachedin the fact table, scroll stable119 unit tests (118 pass / 1 Windows-only skip), mocked e2e, lint, build clean.
Generated by Claude Code