From 4f559bf67aac41e63342a7a781e548ad90def8fd Mon Sep 17 00:00:00 2001 From: Gloire Rubambiza Date: Mon, 10 Aug 2026 11:12:14 -0400 Subject: [PATCH] docs: Point pr-review fixer at migrated github-pr-review skill Update the standing order to reflect that the fixer now sources the github-pr-review skill from the agent-skills checkout (the migrated, version-controlled skill) instead of the former github:pr-review skill in rossoctl/rossoctl/.claude/skills/. Documents the cron payload wiring and the checkout-currency requirement. Assisted-By: Claude Code (Anthropic AI) Signed-off-by: Gloire Rubambiza --- standing-orders/pr-review.md | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/standing-orders/pr-review.md b/standing-orders/pr-review.md index 327af32..c1fbafa 100644 --- a/standing-orders/pr-review.md +++ b/standing-orders/pr-review.md @@ -11,7 +11,7 @@ ### Scope - Repositories: the core allowlist (see `config/core-repos.txt`) - Only open PRs with the `ready-for-ai-review` label -- Uses the `github:pr-review` skill +- Uses the `github-pr-review` skill, sourced from the `agent-skills` checkout (see Skill Source below) ### Architecture: Scanner/Fixer Pattern @@ -85,7 +85,7 @@ Entries are removed from state when: **Cron jobs:** - `pr-review-scanner`: bash-only, no LLM cost -- `pr-review-fixer`: agent turn using `github:pr-review` skill +- `pr-review-fixer`: agent turn using the `github-pr-review` skill (see Skill Source below) ### Reports @@ -115,3 +115,12 @@ Entries are removed from state when: - Scanner: every 15 minutes (lightweight, no LLM) - Fixer: every 15 minutes, offset ~5 min from scanner - Manual run: `openclaw cron run pr-review-scanner` / `openclaw cron run pr-review-fixer` + +### Skill Source +- The fixer sources the `github-pr-review` skill from the on-host `agent-skills` checkout + (`agent-skills/skills/github-pr-review/SKILL.md`), NOT from a per-repo `.claude/skills/` copy. +- This is the migrated, version-controlled skill (rossoctl/agent-skills); it replaced the + former `github:pr-review` skill that lived in `rossoctl/rossoctl/.claude/skills/`. +- Wiring lives in the fixer cron's agentTurn payload (the skill path the review step follows); + update it with `openclaw cron edit --message "…"`, never by hand-editing jobs.json. +- Keep the checkout current (`git pull`) so the fixer reads the latest reviewed skill.