docs: Point pr-review fixer at migrated github-pr-review skill - #49
Conversation
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) <noreply@anthropic.com> Signed-off-by: Gloire Rubambiza <gloire@ibm.com>
clawgenti
left a comment
There was a problem hiding this comment.
Docs-only update that replaces two references to the old github:pr-review skill name with github-pr-review and adds a concise Skill Source subsection documenting the on-host agent-skills checkout path, cron payload wiring, and openclaw cron edit usage — all accurate and consistent with the migrated skill.
All checks pass. Ready for human review.
Reviewed by clawgenti using github:pr-review
cwiklik
left a comment
There was a problem hiding this comment.
Clean docs pointer for the skill migration (epic #2149, unit 2). I verified the cross-repo references:
skills/github-pr-review/SKILL.mdexists inrossoctl/agent-skills— the documented pathagent-skills/skills/github-pr-review/SKILL.mdis correct.- agent-skills#28 ("Migrate github-pr-review skill from rossoctl/rossoctl") is merged.
- The former
github:pr-reviewskill is still present inrossoctl/rossoctl/.claude/skills/, matching the doc's "intentionally left in place until post-rewire monitoring" note.
The "Skill Source" subsection is a good addition — documenting that wiring lives in the fixer cron payload (openclaw cron edit, never hand-editing jobs.json) and the checkout-currency requirement. Correctly scoped: the live cron repoint is called out as a separate operational step, so the doc describes the target state without implying the runner is already switched.
Docs-only, DCO green, signed. LGTM.
Assisted-By: Claude Code
Summary
Points the pr-review fixer at the migrated
github-pr-reviewskill inrossoctl/agent-skills, replacing the formergithub:pr-reviewskill that lived inrossoctl/rossoctl/.claude/skills/. This is unit 2 of the skill-migration epic rossoctl/rossoctl#2149; the skill landed in agent-skills via rossoctl/agent-skills#28.Updates
standing-orders/pr-review.md:github-pr-reviewsourced from the agent-skills checkout.openclaw cron edit, never by hand-editing jobs.json), and the checkout-currency requirement.The live fixer cron payload is repointed separately on the runner (operational step). The former skill is intentionally left in place in rossoctl/rossoctl until post-rewire monitoring confirms no review degradation (see #2149), keeping a one-step rollback.
Fixes #26
Assisted-By: Claude Code