oikos/.claude/skills/pr-review/SKILL.md

name, description, disable-model-invocation, user-invocable, argument-hint, allowed-tools

name	description	disable-model-invocation	user-invocable	argument-hint	allowed-tools
pr-review	Review a PR against the Oikos Hard Constraints and decide close/request-changes/merge	true	true	<pr-number>	Bash(gh pr *) Bash(gh api *) Bash(git *) Bash(npm test *) Bash(npm run test:*) Read Grep Glob

Run from inside oikos/. $1 is the PR number.

1. Fetch — gh pr view $1 --repo ulsklyc/oikos, gh pr diff $1 --repo ulsklyc/oikos, gh pr checks $1 --repo ulsklyc/oikos. Note failing checks. Delegate the deep read to the pr-reviewer subagent (Agent({ subagent_type: "pr-reviewer", ... })) so main-thread context stays free.
2. Constraint check — walk the diff against CLAUDE.md Hard Constraints:
   • No frontend frameworks / bundlers / CSS libraries added
   • No require, only import/export
   • No eval, no innerHTML
   • All UI text via t('key'); de locale updated
   • Migrations append-only (server/db.js)
   • Design values from public/styles/tokens.css
   • Route handlers wrapped in try/catch
   • API shape { data } / { error, code } Also check: PR has tests touching the relevant test-<module>.js, CHANGELOG entry under ## [Unreleased], commit subjects are Conventional Commits.
3. Decide
   • Blocking issue found → gh pr review $1 --request-changes --body "<english, grouped by file:line>" and stop.
   • Not a fit → explain politely and gh pr close $1 --comment "...".
   • Clean → gh pr review $1 --approve --body "LGTM" then gh pr merge $1 --squash --delete-branch. After merge, fetch main locally and consider running /release-prep.

Guardrails

• Comments and review bodies: always English.
• Never merge with failing required checks. Never force-merge.
• Never close a PR without a reason comment — silent closes burn contributor trust.
• Never push directly to the PR branch. If a small fix is warranted, ask the contributor.