Appearance
08 — Review, improve, and carry context forward
Goal and starting workspace
- Consumer, after a 07 attempt; preserve real implementation/ref/results, even if incomplete.
- Review a concrete defect and improve the reusable toolkit based on evidence.
Build it yourself
- Author
workshop/artifacts/review/total-after-limit.mjs: exportfaultyWindow(matches, limit)that slices then incorrectly reportsitems.lengthas total. Authorprobe.mjsimporting it and asserting that three input matches at limit 1 incorrectly produce total 1, not 3. Keep these inert teaching files outside API code. - Run
node workshop/artifacts/review/probe.mjs. The probe succeeds only when it detects the intentional defect (1 returned instead of total 3). - Ask independent reviewer to connect the exact defective line, authoritative requirement, failing anchor and impact.
- Distinguish supported finding from speculation; reviewer must not edit the fixture/API.
- Author extension: improve the canonical checklist from this finding, bump to
1.2.0, build, then explicitly update consumer registration and re-invoke reviewer. - Author
workshop/artifacts/review/review-rubric.mdlinking requirement/line/reproduction. Fillcontinuation-brief.mdin that directory: goal, actual ref, decisions, files, exact commands/exits, installed toolkit version, MCP/fallback, pending work. - Start a new session/client and re-read contract/source/ref rather than paste full chat history.
Inspect the example
- Browse examples branch.
- Defect/probe, rubric and unfilled handoff template are complete instructional assets. No sample claims actual review execution.
Bring in this step
sh
# CONSUMER
npm run lab:example -- --step 08-review-and-handoff --preview
npm run lab:example -- --step 08-review-and-handoff --stage
npm run lab:example -- --step 08-review-and-handoff --applyIf you already changed these files
- Never replace your actual handoff/results with sample text. Stage a fresh reference and merge structure only.
- Changed fixtures or role definitions remain protected. Recovery.
Client steps
VS Code
- Select the independent reviewer with observed tool restrictions; inspect local diff and defect fixture.
- Record line-level finding, requirement and reproduction. This is local review, not hosted Copilot Code Review.
- After author package update, replace only lab registration and reload before verifying checklist improvement.
Copilot CLI
- Use a separate reviewer context; retain
git diffand probe output, no implementation permissions by assumption. - Inspect package version/source after 1.2.0 update using list JSON and skill inspection.
- Start a fresh session in the same consumer and provide only continuation brief plus authoritative paths.
Copilot app
- Pick reviewer only after inspecting real permissions; advisory-only roles are not technically read-only.
- Preserve ref and evidence when starting the next session/client. Recheck bridge version or use supported CLI/VS Code.
- Hosted/cloud review is optional and approval-gated; no Mobile/local MCP or plugin transfer assumptions.
Verify the result
npm run verify:exercise -- --step 08-review-and-handoff;node workshop/artifacts/review/probe.mjs.- If feature complete:
npm run verify:solution; otherwise retain exact failing outcome and remaining work. - Evidence includes a supported finding, improved canonical asset and observed updated invocation, plus continuation from a fresh context.
- Recovery: remove only lab package/MCP registrations after use; preserve authored artifacts. Stop local processes.
- Optional: 09 — Spec-Kit status.