Repository navigation
review-branch: lift kendo reviewer-pair redesign - #20
Merged
Merged
Conversation
Replace the five-agent pipeline (acceptance + simplicity, conditional
silent-failure / efficiency) with kendo's always-on pair:
runtime-integrity-reviewer + precedent-reviewer, spawned in parallel on
every branch. Lifted from kendo .claude/skills/review-branch (2026-07-27).
Lifted:
- "The two questions" section: one question per reviewer, no trigger
patterns, no conditional specialists.
- Step 2 resolves docs/plans/<slug>/ first, then docs/bugs/<slug>/; the
review is optional on bug branches (bug-fix-verifier is their gate).
- No-directory fallback still spawns both agents and reports in chat.
- Step 1 no longer pulls the full diff into the orchestrator.
- The REVIEW_CLAUDE.md format is inlined in Step 4 with Runtime Integrity
/ Precedent sections. Synthesis rules add the "Required Fixes means
don't open a PR" calibration.
Generalised:
- origin/development -> origin/{{DEFAULT_BRANCH}}.
- "ADR projections in backend/CLAUDE.md and frontend/CLAUDE.md" -> "the
repo's standing rules (every CLAUDE.md, plus an ADR set where one
exists)".
- Dropped the .claude/REVIEWERS.md citation (DECISIONS.md D2); each agent
carries its own calibration.
references/review-file-format.md is no longer linked from this skill. It
is deleted in the follow-up pr PR, which removes the last link to it.
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UX7YHkAamvBe8Hh66f95b7
This was referenced Sep 2, 2026
jasperboerhof
added a commit
that referenced
this pull request
Sep 2, 2026
…allbacks Lifted from kendo (2026-07-27): steps 4-6 resolve against a <root>; new "Which root" section (docs/plans/ then docs/bugs/ for /review-branch, plans win on collision); /review-branch and /pr fallback lines match the docs/bugs/ support already merged in review-branch (#20) and pr (#26). Generalised: the issue-key example in step 5 uses {{ISSUE_KEY_PREFIX}}. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UX7YHkAamvBe8Hh66f95b7
4 tasks done
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
1 participant
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.
Why
Kendo redesigned
review-branchon 2026-07-27 (KD-1061,79e77d0b) to spawn the always-on pairruntime-integrity-reviewer+precedent-reviewer. The catalog still spawned the retired five-agent pipeline.What
runtime-integrity-reviewerandprecedent-revieweralways, in parallel, on every branch. No trigger patterns, no conditional specialists.docs/plans/<slug>/first, thendocs/bugs/<slug>/. On a bug branch the review is optional;bug-fix-verifieris the gate.REVIEW_CLAUDE.mdformat is inlined in Step 4 with Runtime Integrity / Precedent sections, matching kendo.Generalised
origin/developmentbecameorigin/{{DEFAULT_BRANCH}}..claude/REVIEWERS.mdcitation is dropped per DECISIONS.md D2. Each agent carries its own calibration text.Out of scope
references/review-file-format.mdstill exists but is no longer linked from this skill. It is deleted in the follow-upprPR, which removes the last link to it. Deleting it here would leaveskills/pr/SKILL.mdwith a dangling link.skills/plan-feature/references/plan-directory.mdstill says/review-branchskips the pipeline when no plan exists. Kendo updated that file with a "Which root" section. That file belongs to theplan-featurelift.Test plan
grep -rn -i 'kendo\|KD-[0-9]\|CRIT-\|script\.nl\|warroom\|war room\|phpstan\|knip' skills/review-branch/SKILL.mdis emptygrep -rn 'acceptance-reviewer\|simplicity-reviewer\|silent-failure-hunter\|efficiency-hunter\|task-alignment-reviewer' skills/review-branch/SKILL.mdis emptygrep -n REVIEWERS skills/review-branch/SKILL.mdis empty🤖 Generated with Claude Code
https://claude.ai/code/session_01UX7YHkAamvBe8Hh66f95b7