| name | install-reviewer |
| description | Scaffold the `jbaruch/coding-policy` gh-aw PR review workflows into a consumer repository: copies the packaged paired workflow templates (OpenAI + Anthropic reviewers), compiles them with `gh aw`, and opens a PR. After merge, every pull request in the repo is reviewed against the latest published `jbaruch/coding-policy` rules by the reviewer whose family differs from the PR's declared author model — avoiding self-review bias per `rules/author-model-declaration.md`. Use when the user wants to add, install, enable, scaffold, set up, or wire up an automated policy review / PR reviewer / coding-policy CI reviewer in a consumer repo. Also use to upgrade, update, refresh, or pull the latest reviewer templates into a consumer repo that already has the scaffold installed — the skill switches to override mode in that case.
|
Install Reviewer Skill
Scaffold the gh-aw PR policy reviewer pair (OpenAI + Anthropic) into a consumer repository. Process steps in order. Do not skip ahead.
The skill runs in one of two modes determined by the user's request:
- install (default) — the consumer hasn't run the skill before, no scaffolded reviewer files exist. The current behavior of every step.
- upgrade (
--override) — refresh a previously-installed reviewer to the current plugin version
- Trigger phrases: "upgrade", "update", "refresh", "pull latest reviewer templates", "override"
- Pass
--override to all five scripts: preflight, branch, scaffold, commit, push
- Branch:
feat/upgrade-coding-policy-review
- Commit message:
ci(review): upgrade ...
- Preflight skips branch-clear checks; instead refuses if any rewritable target is in a state the upgrade could clobber — uncommitted edits, untracked content, symlinks, or tracked deletions (HEAD-present, working-tree-removed)
- Scaffold snapshots and restores all four reviewer files on compile failure
Step 1 — Run Preflight Checks
.tessl/plugins/jbaruch/coding-policy/skills/install-reviewer/preflight.sh
.tessl/plugins/jbaruch/coding-policy/skills/install-reviewer/preflight.sh --override
Runs every precondition (git worktree, GitHub CLI install + auth, gh-aw extension at minimum version, plugin template, origin remote, plus mode-dependent branch state) and returns one JSON object: {"ok": bool, "override": bool, "failures": [...], "warnings": [...]}.
- Exit 0, empty
failures — every precondition passed; proceed to Step 2.
- Exit 1, populated
failures — report each failure's reason verbatim and stop. Every failure carries a concrete recovery command. The gh-aw extension is github/gh-aw (under the github org, not the plugin owner); its version floor and install pin are the script's policy — see skills/install-reviewer/preflight.sh (GH_AW_MIN / GH_AW_PIN constants at the top of the file).
- Non-empty
warnings — informational only; never affects ok or the exit code. Report each reason verbatim alongside the Step 1 outcome and remember them for Step 7's PR body. Do not stop; proceed to Step 2.
Step 2 — Refuse Overwrite (install mode only)
In install mode: if any of .github/workflows/review-openai.md, .github/workflows/review-openai.lock.yml, .github/workflows/review-anthropic.md, or .github/workflows/review-anthropic.lock.yml exists, stop and report that prior review setup is present. Do not overwrite — a lock alone or a source alone indicates deliberate in-progress configuration. If none exist, proceed to Step 3.
In upgrade mode: skip this step. Preflight has verified rewritable targets carry no state the upgrade could clobber (no uncommitted edits, untracked content, symlinks, or tracked deletions); scaffold.sh snapshots and restores them on compile failure.
Step 3 — Establish Feature Branch
.tessl/plugins/jbaruch/coding-policy/skills/install-reviewer/branch.sh
.tessl/plugins/jbaruch/coding-policy/skills/install-reviewer/branch.sh --override
Establishes the feature branch the rest of the steps commit on. Install mode creates feat/add-coding-policy-review from origin's default branch. Upgrade mode targets feat/upgrade-coding-policy-review and probes both remote (git ls-remote --exit-code --heads) and local state to handle the fresh-clone-while-upgrade-PR-open case: if the local branch exists it's checked out (state checked-out); else if the remote branch exists it's checked out with upstream tracking so the upcoming push fast-forwards (state checked-out-tracking); else it's created from the default branch (state created). Idempotent: emits {"state": "already-on-branch", ...} on re-run when HEAD already matches the target. Real ls-remote/fetch errors (network, auth) propagate verbatim with non-zero exit. Proceed immediately to Step 4.
Step 4 — Scaffold Workflow Files
.tessl/plugins/jbaruch/coding-policy/skills/install-reviewer/scaffold.sh
.tessl/plugins/jbaruch/coding-policy/skills/install-reviewer/scaffold.sh --override
Creates .github/workflows/ if missing, copies both packaged templates into review-openai.md and review-anthropic.md, compiles them via gh aw compile review-openai review-anthropic to produce the matching .lock.yml files, ensures .gitattributes marks the lock files as generated (linguist-generated=true, merge=ours) per rules/file-hygiene.md, and creates-or-merges .env.example documenting the reviewer CI secrets — CODEX_API_KEY/OPENAI_API_KEY, ANTHROPIC_API_KEY, TESSL_TOKEN — with a GitHub Actions secrets-settings deep link per rules/no-secrets.md. When the file lacks the deep link, the reviewer block is prepended into the file header; when the link is present, only the missing keys are appended. Existing consumer entries are preserved. Emits a JSON summary on success; exits non-zero with a stderr diagnostic and rolls back every artifact it touched on compile failure (in upgrade mode the rollback restores the prior contents of all four target files from snapshots in addition to restoring actions-lock.json). The two templates scaffold atomically: either both land or neither does. Proceed immediately to Step 5.
Step 5 — Commit
.tessl/plugins/jbaruch/coding-policy/skills/install-reviewer/commit.sh
.tessl/plugins/jbaruch/coding-policy/skills/install-reviewer/commit.sh --override
Stages the seven scaffolded files (review-openai.md, review-openai.lock.yml, review-anthropic.md, review-anthropic.lock.yml, actions-lock.json, .gitattributes, .env.example) and commits with the canonical message — ci(review): add jbaruch/coding-policy PR review workflows in install mode, ci(review): upgrade jbaruch/coding-policy PR review workflows in upgrade mode. Idempotent: emits {"state": "no-op", …} on re-run when the working tree already matches a prior successful run. If a pre-commit hook rejects the commit, the script exits non-zero — fix the hook's finding and re-run; do not --no-verify. Proceed immediately to Step 6.
Step 6 — Push
.tessl/plugins/jbaruch/coding-policy/skills/install-reviewer/push.sh
.tessl/plugins/jbaruch/coding-policy/skills/install-reviewer/push.sh --override
Pushes the appropriate branch (feat/add-coding-policy-review in install mode, feat/upgrade-coding-policy-review in upgrade mode) to origin with upstream tracking. Idempotent: emits {"state": "up-to-date", …} if origin already matches local HEAD. Proceed immediately to Step 7.
Step 7 — Open PR
gh pr create with title ci(review): add jbaruch/coding-policy PR review workflows (install mode) or ci(review): upgrade jbaruch/coding-policy PR review workflows (upgrade mode), and a body that follows the four required content blocks (cross-family rule explainer, required secrets, load-indicator note, conditional warnings section) defined at:
skills/install-reviewer/PR_BODY_TEMPLATE.md
In upgrade mode, also include a brief diff line in the PR body showing what's being upgraded — the consumer's outgoing plugin version (read from their committed lock-file header banner if discoverable, or stated as "previous" if not) and the new plugin version (the version the agent is currently running under). The human reviewer should be able to see what's being upgraded without diffing every line of YAML.
Return the PR URL. If Step 1 emitted any warnings, surface them inline in your user-facing summary too (not only in the PR body) so the user sees them immediately without opening the PR. Finish here — the user validates the secrets, acts on any warnings, and merges.
