Repository navigation
Haiku cards: read every file in full; a brief caps the search space, never the reading depth - #367
Merged
Merged
Conversation
…e, never the reading depth Haiku tokens are cheap enough that reading is the default, not a cost to manage. Each card gains a "Read in full" section: every reported file is read to its last line, a search hit is only where to read, a brief that caps reading depth is overridden and noted, and a file only scanned is listed as unread (status PARTIAL). harvest-scout now marks each hit match or substring (clamp vs CLAM); census-counter cross-checks counts by reading; spec-editor re-reads after editing; guarded-executor reads complete outputs. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011BMJnDBAfzcuQh5HeT39Rh
Contributor
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configuration
Warning Billing warning: we have not been able to collect payment for this subscription for more than 72 hours. Please update the payment method or pay any pending invoices in Billing to avoid service interruption. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Each card gains an "Output is capped; reading is not" section. Reading stays unbounded; a response has an output cap the worker cannot see, so a long map or reply written in one call can be cut off silently. substrate-mapper writes long maps as numbered parts and re-reads the end of each part; the reply-only cards stop at a whole item, report PARTIAL and list what remains, so the orchestrator splits the brief instead of asking for shallower reading. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011BMJnDBAfzcuQh5HeT39Rh
128k tokens per response at the API against a 1M context; the harness may set a lower per-response cap. A 12,255-word map came through one Write intact, so the split threshold rises from ~4,000 to ~10,000 words. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011BMJnDBAfzcuQh5HeT39Rh
Thinking tokens count toward the 128k output limit, so a long Write after heavy thinking is the case the split rule protects. The protocol note now cites "What's new in Claude Haiku 5.5" and records the three facts that matter for subagent runs: earlier thinking blocks stay in context, the same text counts as about 30% more tokens, and long prompts use long-context pricing. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_011BMJnDBAfzcuQh5HeT39Rh
AdaWorldAPI
marked this pull request as ready for review
October 17, 2026 11:14
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
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
2 participants
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.
Haiku tokens are cheap enough that reading every file in full costs effectively nothing. So reading is the default for the Haiku cards, not something a brief rations. This is the same change as in lance-graph, applied to ndarray's five cards. ndarray has no
guarded-executor, because only the orchestrator compiles here.What changed in each card:
read: <path> (<n> lines, full);unread, and the status becomes PARTIAL.harvest-scoutmarks each hitmatchorsubstring;census-countercross-checks each command count by reading the files;quote-harvesterreads the whole file;spec-editorre-reads the file after editing.Generated by Claude Code