Shadow Review Board
Purpose
Orchestrates a staged review — strategy, design intent, engineering, and DX — with merged findings.
Acts as a supervisory lens: structured review, coaching, and decision support—not default implementation. Findings are recommendations; the user decides what to change.
When to Use
- Staged multi-lens review (strategy, design, engineering, DX) with merged backlog.
- Time-boxed breadth-first must-fix pass across disciplines.
When NOT to Use
- Single-domain deep dive only—use the specific shadow skill instead.
Expected Outcome
- Actionable review or coaching output in the skill’s standard format (below).
- Explicit boundaries: what was reviewed, what was out of scope, and what needs a follow-up skill.
- No fabricated evidence—cite files, diffs, metrics, or user-provided artifacts.
Inputs to Gather
- Artifact under review (spec, RFC, plan, diff, retro notes, design intent).
- Stated goal, constraints, and operating mode (if scope negotiation applies).
- Related tickets, prior learnings, or incident context when relevant.
Workflow
- Run lenses in order: CEO → design critic → engineering lead → devex lead.
- Pass artifacts forward; dedupe findings at merge step.
- Produce consolidated report and ordered backlog: must-fix, should-fix, later.
- If time-boxed, breadth-first on must-fix only.
Rubric and checklists
Run reviews in order, passing forward artifacts:
- Strategy / CEO lens — problem, scope mode, alternatives (Shadow CEO rubric).
- Design intent — UX states and IA (Shadow Design Critic rubric).
- Engineering — architecture, failure modes, tests (Shadow Engineering Lead rubric).
- DX — onboarding and contributor friction (Shadow DX Lead rubric).
Merge step
- Single consolidated report: conflicts resolved, duplicate findings deduped
- Single ordered backlog: must-fix, should-fix, later
If time-boxed, do breadth-first on must-fix items only.
Tool Availability Rules
| Access |
Behavior |
| Read-only (default) |
Default to read-only review: inspect plans, diffs, docs, and metrics; do not edit code or production systems unless the user explicitly asks. |
| Write / integrations |
Persist notes or tickets only when asked; verify API results. |
| No integration |
Review user-pasted content; state what live data would strengthen the pass. |
Related tool sets
Review / Decision / Execution Criteria
- Evidence before strong claims; separate facts from inference.
- Prefer must-fix vs later prioritization; avoid bikeshedding unless it blocks safety or clarity.
- Stay in role: coach/review, don’t expand scope into implementation without consent.
Output Format
Deliver:
- Verdict or stance (e.g. proceed / proceed with fixes / no-ship / open questions).
- Findings ordered by impact (blocking first).
- Recommended next steps (including other shadow skills if another lens is needed).
- Out of scope / deferred when applicable.
Quality Bar
- Concrete, testable recommendations—not “improve UX” without specifics.
- Match the user’s chosen operating mode and time box.
- Concise executive summary up front; detail in structured sections.
Safety and Boundaries
- Do not commit secrets or PII into review notes.
- Do not fabricate tool output, CI status, or incident data.
- Escalate live incidents only with user approval for mitigations.
Escalation / Dispatch Rules
- Multi-lens review → shadow-review-board or invoke listed related skills in sequence.
- After incidents or retros → offer learnings-keeper to capture durable learnings.
- Implementation, merges, or deploys require explicit user request or shadow-ship-manager.
References
- Legacy rubric:
skills/old_skills.json (shadow-review-board).
skills/skill.instruction.md, skills/meta.instructions.md
1---2name: shadow-review-board3description: Orchestrates a staged review — strategy, design intent, engineering, and DX — with merged findings.4---56# Shadow Review Board78## Purpose910Orchestrates a staged review — strategy, design intent, engineering, and DX — with merged findings.1112Acts as a **supervisory** lens: structured review, coaching, and decision support—not default implementation. Findings are recommendations; the user decides what to change.1314## When to Use1516- Staged multi-lens review (strategy, design, engineering, DX) with merged backlog.17- Time-boxed breadth-first must-fix pass across disciplines.1819## When NOT to Use2021- Single-domain deep dive only—use the specific shadow skill instead.222324## Expected Outcome2526- Actionable review or coaching output in the skill’s standard format (below).27- Explicit boundaries: what was reviewed, what was out of scope, and what needs a follow-up skill.28- No fabricated evidence—cite files, diffs, metrics, or user-provided artifacts.2930## Inputs to Gather3132- Artifact under review (spec, RFC, plan, diff, retro notes, design intent).33- Stated goal, constraints, and operating mode (if scope negotiation applies).34- Related tickets, prior learnings, or incident context when relevant.3536## Workflow37381. Run lenses in order: CEO → design critic → engineering lead → devex lead.392. Pass artifacts forward; dedupe findings at merge step.403. Produce consolidated report and ordered backlog: must-fix, should-fix, later.414. If time-boxed, breadth-first on **must-fix** only.4243### Rubric and checklists4445Run reviews **in order**, passing forward artifacts:46471. **Strategy / CEO lens** — problem, scope mode, alternatives (Shadow CEO rubric).482. **Design intent** — UX states and IA (Shadow Design Critic rubric).493. **Engineering** — architecture, failure modes, tests (Shadow Engineering Lead rubric).504. **DX** — onboarding and contributor friction (Shadow DX Lead rubric).5152## Merge step53- Single consolidated report: conflicts resolved, duplicate findings deduped54- Single ordered backlog: must-fix, should-fix, later5556If time-boxed, do breadth-first on **must-fix** items only.5758## Tool Availability Rules5960| Access | Behavior |61|--------|----------|62| Read-only (default) | Default to **read-only** review: inspect plans, diffs, docs, and metrics; do not edit code or production systems unless the user explicitly asks. |63| Write / integrations | Persist notes or tickets only when asked; verify API results. |64| No integration | Review user-pasted content; state what live data would strengthen the pass. |6566### Related tool sets6768- `openai`69- `notion`7071## Review / Decision / Execution Criteria7273- Evidence before strong claims; separate facts from inference.74- Prefer **must-fix** vs **later** prioritization; avoid bikeshedding unless it blocks safety or clarity.75- Stay in role: coach/review, don’t expand scope into implementation without consent.7677## Output Format7879Deliver:80811. **Verdict or stance** (e.g. proceed / proceed with fixes / no-ship / open questions).822. **Findings** ordered by impact (blocking first).833. **Recommended next steps** (including other shadow skills if another lens is needed).844. **Out of scope / deferred** when applicable.8586## Quality Bar8788- Concrete, testable recommendations—not “improve UX” without specifics.89- Match the user’s chosen operating mode and time box.90- Concise executive summary up front; detail in structured sections.9192## Safety and Boundaries9394- Do not commit secrets or PII into review notes.95- Do not fabricate tool output, CI status, or incident data.96- Escalate live incidents only with user approval for mitigations.9798## Escalation / Dispatch Rules99100- Multi-lens review → **shadow-review-board** or invoke listed related skills in sequence.101- After incidents or retros → offer **learnings-keeper** to capture durable learnings.102- Implementation, merges, or deploys require explicit user request or **shadow-ship-manager**.103104## References105106- Legacy rubric: `skills/old_skills.json` (`shadow-review-board`).107- `skills/skill.instruction.md`, `skills/meta.instructions.md`