review-change

Solid

Platform-adaptive review orchestrator. Reviews the current change by running review-implementation (find → classify) AND the workflow's own internal review pack — only the passes that apply to this project and this change (review-code, review-security, review-verify, review-debt, review-design, review-a11y, review-brand, review-perf, review-seo) — never the inapplicable ones (no a11y/SEO/brand for a CLI, library, or infra change). Self-contained: no external review skills required; installed platform skills run as optional extras. Synthesizes one classified report plus an explicit manual-verification checklist. Findings only — never refactors. On Claude Code and want hand-tuned per-skill model/effort tiers? Install the `#claude` branch instead (`npx skills add gtrabanco/agentic-workflow#claude`) — see the README. This branch is model-agnostic: the skill inherits whatever model and effort your agent session is already using. Triggers: "review this change", "full review before merge", "review-change", "run the

Code & Development 19 stars 3 forks Updated 1 weeks ago MIT

Install

View on GitHub

Quality Score: 82/100

Stars 20%
43
Recency 20%
90
Frontmatter 20%
70
Documentation 15%
100
Issue Health 10%
50
License 10%
100
Description 5%
100

Skill Content

# Review Change The quality gate for a change: get every review that *applies* — and skip the ones that don't — in one synthesized, classified report. **Findings only; never edits or refactors.** ## Turn contract — verify before ending the turn ``` ✓ This review runs in a conversation that did NOT implement the change; if this conversation wrote the diff, STOP and hand off to a fresh one (the reviewer works from the diff + the SPEC, not the author's mental state). ✓ The synthesized decision table + manual-verification checklist + `Decision: PASS | FAIL` were returned in the fixed output format ✓ Every non-fix-now finding got a destination (triaged — issue / decision / drop) ✓ The closing `→ Next:` block is printed as the ABSOLUTE last output ``` About to end the turn with any box unchecked? The turn is NOT done — complete the missing box first (weak models drop end-of-document duties; this list is first on purpose). ## When to use - **Mandatory before every merge** — every unit (feature, single-pass, or fix) gets a `review-change` pass before its merge gate; that end review is never skipped, and it must run in a conversation that did **not** implement the change — the conversation that wrote a diff shares the author's mental model and tends to catch only mechanical issues, missing design defects a context-clean, adversarial reviewer would find (see the turn-contract box above). If the reviewing conversation authored the diff, stop and hand off to a fresh one...

Details

Author
gtrabanco
Repository
gtrabanco/agentic-workflow
Created
1 months ago
Last Updated
1 weeks ago
Language
TypeScript
License
MIT

Bundled in these plugins

Similar Skills

Semantically similar based on skill content — not just same category