Repository navigation
fix(review): set Wave 1 tool-round budget to 80 (port of agent-afk#2815) - #11
Merged
Merged
Conversation
Port the budget paragraph from agent-afk PR #2815. The research-agent default of 50 is too tight for Wave 1: measured workers exhausted their budget 25-36% of the time over 5 weeks of traces, ending in a tools-stripped wind-down on partial evidence. Because AFK resolves plugin skills project → user → bundled (first name wins), the user-scope agent-workflow-amplifiers plugin shadows the bundled /review. agent-afk#2815 alone has no effect for operators using this plugin; both copies need the change. Text is byte-identical to the agent-afk bundled copy at a6ca3fc (verified with diff, exit 0).
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.
Summary
Ports the Wave 1 tool-round budget paragraph from agent-afk#2815 into this repo's copy of the review skill.
Why
Wave 1 of
/reviewdispatches tworesearch-agentworkers in parallel. Theresearch-agentdefault ismax_tool_use_iterations: 50. Over 5 weeks of session traces, those workers exhausted their budget 25–36% of the time, causing a tools-stripped wind-down that produces findings built on partial evidence.Why this repo specifically
AFK resolves plugin skills project → user → bundled (first name wins —
src/agent/tools/skill-bridge.ts:332). The operator's user-scopeagent-workflow-amplifiersplugin (cloned from this repo) shadows the bundled/review.agent-afk#2815alone has no effect for operators using this plugin; both copies need the change.What changed
One paragraph inserted between the Wave 1 bullet list and the "Each agent receives:" paragraph in
skills/review/SKILL.md:Text is byte-identical to the agent-afk bundled copy at
a6ca3fc(verified:diffexit 0).Verified
diff skills/review/SKILL.md <agent-afk-bundled-copy>→ exit 0 (byte-identical)skills/review/SKILL.md— none foundrelease-please-config.jsonpins.claude-plugin/plugin.jsonversion; version bump is release-please's job, not included herescripts/orpackage.json