| name | self-review |
| description | Review the current branch against this repo's architecture, security, performance, cross-platform and test-coverage rules before opening a PR. Use when the user asks to self-review, review my changes, check my branch before a PR, or says they are about to open a PR. |
Self-review
Run the repo's review standard against the working branch, before the PR exists.
Nothing runs this in CI. It is the only review a change gets before a human reads it, which is why
it happens now rather than after: a finding fixed here costs one message, the same finding on the PR
costs a review cycle.
Procedure
Read .github/instructions/code-review.instructions.md
and follow it. It is the specification for this task, not background reading — it carries the
procedure (establish the diff, run npm run lint and npm test, then the five dimensions) as well
as the invariants and the reporting bar. Do not restate it here; run it.
That file is the single copy of the standard. It lives under .github/instructions/ so Copilot code
review picks it up natively; every other agent, this one included, reaches it from here. Changes to
how reviews work go there, not into this wrapper.
Two things this skill adds on top, both agent-neutral — apply whichever your agent can:
Run the judgement pass in a fresh context. The diff and the deterministic layer
(npm run lint, npm test) run inline. The five dimensions should be reviewed by a context that
did not write the change — a subagent, or a separate pass given only the diff and the instructions
file. If the session that wrote the code also grades it, it reviews its own reasoning and finds it
sound.
Report, don't apply. Surface findings in the chat. No GitHub comments, no files written — the PR
does not exist yet. Then offer; the author decides what is a real finding.
Notes
- This is the same skill as
.claude/skills/self-review/,
placed here because agents built on the Agent Skills open standard (Command Code and others)
discover skills under .agents/skills/ rather than .claude/skills/. Both are thin wrappers over
the one instructions file — neither is a second copy of the standard.
- The repo-specific knowledge this adds over a generic review: Electron's bundled Node,
isomorphic-git, electron-store, loopback binding, the Windows spawn shims.
- If a finding reveals a rule the instructions file does not yet cover, say so. It is meant to
accumulate what the project learns.
