Repository navigation
wireframe-reviewer: append review notes, sole gate, generalised arch-test checks - #40
Merged
Merged
Conversation
…test checks Lifted from kendo .claude/agents/wireframe-reviewer.md (2026-07-27): - Step 9 appends a Review Notes section (Top Issues, AC Traceability) to WIREFRAMES.md; re-runs replace it instead of stacking - Reframed as the only gate the wireframe passes; no acceptance-reviewer downstream, the implementer is the audience - Step 5 frontend architecture checks (app-boundary and domain-structure rules) with a skip when the repo has none - Edit added to the tools list so Step 9 can run; tool budget 22 Generalised: frontend/tests/js/architecture/app-boundaries.spec.ts and domain-structure.spec.ts became "the repo's frontend architecture tests, if any"; @central/@tenant became "app areas"; UnoCSS token wording keeps the catalog's design-token phrasing. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01UX7YHkAamvBe8Hh66f95b7
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.
Why
Re-sync
agents/wireframe-reviewer.mdwith kendo's.claude/agents/wireframe-reviewer.mdas of 2026-07-27 (catalog-resync plan, rowagents/wireframe-reviewer.md).What
## Review Notessection (Reviewed / Score / Result, Top Issues, AC Traceability) to WIREFRAMES.md. A re-run replaces the section instead of appending a second one.toolsgainsEditso Step 9 can run; rules and constraints updated (read-only except that one Edit; budget 22 tool calls).Generalised
frontend/tests/js/architecture/app-boundaries.spec.ts/domain-structure.spec.ts→ "the repo's frontend architecture tests, if any (e.g. app-boundary and domain-structure rules)" with generic glob hints.@central//@tenant//@shared/→ "app areas" and "the shared directory";CUSTOM_STORE_EXCEPTIONS→ "the exceptions list".bg-kendo-surface, UnoCSS attributify greps) keeps the catalog's design-token phrasing, per the item instructions.acceptance-reviewerandtask-alignment-reviewerreferences removed; plans haveplan-reviewer+surface-reviewer, tasks have a self-administered coverage gate.Out of scope
skills/wireframe(PR wireframe: lift references, zero-UI guard, resume mode, wireframe-reviewer self-gate #39) andagents/plan-reviewer.md(next PR).Test plan
grep -rn -i 'kendo\|KD-[0-9]\|CRIT-\|script\.nl\|warroom\|war room\|phpstan\|knip' agents/wireframe-reviewer.md— emptygrep -rn 'acceptance-reviewer\|simplicity-reviewer\|silent-failure-hunter\|efficiency-hunter\|task-alignment-reviewer' agents/wireframe-reviewer.md— empty🤖 Generated with Claude Code
https://claude.ai/code/session_01UX7YHkAamvBe8Hh66f95b7