review
Run plainworks' standing engineering-baseline review over a change set (a branch, commit range, or HEAD~1) or over a whole package/tree. Sequences nine focused passes — structure & placement, canonica
SKILL.md.github/skills/review/SKILL.md
GitHubReview
Apply the baseline. Review code and evidence, not the author's rationale; plans define scope, not exceptions to requirements.
Scope and execution
Review directly by default. Launch an independent reviewer only when the user requests it; use one bounded reviewer unless distinct additional scopes are explicitly requested. Pass the scope, constraints, and relevant files, not the authoring transcript. Reevaluate returned findings against source before acting.
For changes, include staged/unstaged/untracked work when requested, plus close callers, callees, and affected contracts. Report pre-existing defects in that impact area; do not silently widen into an unrelated audit. For project review, state the package/domain boundary.
Reviews are read-only unless fixes are authorized. For each real defect choose Redesign / Align / Enhance / Drop with a one-line reason. Preserve user staging and unrelated work; do not commit, amend, push, resolve remote threads, or post replies implicitly.
Load checks by trigger
Check placement and canonical reuse first for code changes. A blocker there stops acceptance; report any unperformed checks rather than claiming a complete review. Apply every relevant lens below, in order; load only its file, not the entire reference directory. For prose-only work, use documentation/claim checks rather than runtime test suites.
| Trigger | Checklist |
|---|---|
| Code placement, imports, exports, or package wiring | Pass 00 — Structure and placement |
| Shared behavior or a proposed helper/adapter | Pass 01 — Canonical-owner reuse |
| Runtime APIs, errors, concurrency, or composition | Pass 02 — Principle conformance |
| Trust boundaries, auth, crypto, privacy, or external input | Pass 03 — Security & privacy |
| Behavior, readability, maintainability, or dead code | Pass 04 — Quality: simplicity, maintainability, current patterns |
| Behavior changes, tests, fixtures, or coverage | Pass 05 — Tests & TDD |
| Documentation, dependencies, CI, or release | Pass 06 — Docs & supply chain |
| Comments and public API documentation | Pass 07 — Comments & TSDoc |
| Interactive UI/client changes | Pass 08 — UI: accessibility, responsive, performance |
Modes
- Changes: diff and affected contracts.
- Project: bounded tree audit.
- Review and fix: select only when fixes are requested; confirm the proposed fix scope before editing.
Evidence and output
Use validate for scoped gates; reuse fresh evidence and retain required full acceptance gates. Green tooling is necessary, not proof of lifecycle/security correctness.
Report severity | file:line | defect and impact | evidence | proposed action. Blocker means broken behavior/contract or a hard-rule violation; should-fix means a substantiated maintainability defect; nit is optional style, never a gate. List incomplete/skipped checks and uncertainty. Do not invent findings or treat grep hits as proof.