chore(agents): rework agentic setup for lean single-session delivery - #2040
Merged
Conversation
Modernize the 11-agent system around three goals: kill follow-up churn, cut per-cycle latency/tokens, and rebalance model spend to judgment density. - Agents: slim all 11 definitions 2,864 -> 591 lines by pointing at CLAUDE.md instead of restating it; static output formats move to .claude/templates/. Model matrix: dev-team-lead -> opus, product-owner -> sonnet, translator + docs-writer -> haiku. - Churn killers: fix-or-block Reviewer Verdict Policy (low-effort findings fixed in the same PR; deferrals need a filed, justified issue), external review loop capped at 2 rounds, left-shifted user verification before merge, dependabot adoption follow-ups batched into one issue per run. - Latency killers: reviews fan out immediately after PR creation (parallel with CI; single ci-wait at merge), dev-team-lead launched once per story and continued via SendMessage, wiki synced once per skill run (per-agent pulls removed), e2e prior-failure archaeology moved from every launch to /epic-close + /fix-e2e, reviewers get a pre-fetched diff and file scopes, tiered dependabot changelog analysis. - New scripts/i18n-audit.mjs replaces the translator's 71-line hand-run audit (already surfaced 18 real missing-key bugs); bash-guard push regex hardened (HEAD:refs/heads/beta bypass) with a behavioral test file; ci-wait now judges the latest check-run per name and beta timeout raised to 600s; board.sh guards against silent 1000-item truncation. - Correctness: implementation-checklist stale facts fixed (formatters.ts, --spacing-* tokens); agent-memory entries contradicting the new policies corrected. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Contributor
|
🎉 This PR is included in version 2.14.0-beta.26 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Contributor
|
🎉 This PR is included in version 2.14.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
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
In-depth rework of the agent system targeting the 2.2-fixes-per-feature churn ratio (92
fix:vs 42feat:in the last 300 commits) — keep the intended process (11 agents, 3-mode dev-team-lead, skill lifecycle, trailers, beta/main), make it lean, and make work complete in the session that started it.Churn killers
pr-<n>image before merging in interactive sessions.Latency killers
ci-wait.shcall happens at merge (was 3–5 waits per cycle).diffPath+ per-reviewerfilesscopes — one diff fetch instead of 4–9.Token/model rebalance (aliases resolve to latest tier)
.claude/templates/(read on demand).Correctness fixes
scripts/i18n-audit.mjs(new): deterministic parity/coverage/duplicate/glossary audit replacing the translator's 71-line hand-executed protocol. Running it already found 18 genuine missing-key bugs (e.g.t('common:validation.amountRequired')—common.jsonhas novalidationobject) — follow-up material for a small fix PR.bash-guard.mjs: push regex hardened (git push origin HEAD:refs/heads/betawas not blocked; baregit pushon a protected branch now blocked too); no-trailer escape hatch removed (the hook only fires on agent commits;check-trailers.shkeeps the human skip for CI). Behavioral test file added — 8/8 pass.ci-wait.sh: judges the latest check-run per name (re-runs were shadowed by stale failures); beta timeout 300s → 600s.board.sh: errors loudly instead of silently truncating at the 1000-item list limit.format.ts→formatters.ts,--space-*→--spacing-*per tokens.css); contradicted agent-memory entries corrected (versioned trailer instruction, wiki-pull ritual, legacy 33-item deferred-findings backlog annotated).Test plan
node --check/bash -non all touched scriptsnode scripts/hooks/bash-guard.test.mjs)node scripts/i18n-audit.mjsruns against the repo (18 genuine findings, 1 warning)server/,client/,shared/,e2e/) modified🤖 Generated with Claude Code