chore: pin rig skill to local HEAD instead of stale SHA - #299
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
@copilot use step to gh skill install --local-skill --force |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Done in eb8a2b7. Replaced the |
|
🧠 Matt Pocock Skills Reviewer has completed the skills-based review. ✅ |
There was a problem hiding this comment.
Skills-Based Review 🧠
Applied /codebase-design — clean infrastructure maintenance, no blocking issues.
📋 Review Summary
What was reviewed
daily-rig-file-summarizer.mdanddaily-rig-sampler.md:skills:SHA pinning replaced withsteps:+gh skill install . rig --from-local --force- Both
.lock.ymlfiles regenerated viagh aw compile --strict
Assessment
- ✅
gh skill install . rig --from-local --forcecorrectly points to the repo root; skill discovery findsskills/rig/SKILL.mdvia the standard convention - ✅ Removing the
Collect skill install failuresinfrastructure is correct — the newsteps:approach surfaces failures as native workflow step failures - ✅ Lock files consistently regenerated; no stale manifest entries remain
- ✅ PR description correctly notes
daily-rig-task-generator.mdis unaffected (reads skill viacatin the agent body)
Minor observation (non-blocking)
daily-rig-claude-compat.md (not changed here) uses gh skill install skills/rig rig --from-local while this PR uses . as the repo root. Both resolve correctly, but aligning to . in a follow-up would keep the pattern consistent.
🧠 Reviewed using Matt Pocock's skills by Matt Pocock Skills Reviewer · sonnet46 42 AIC · ⌖ 5.77 AIC · ⊞ 6.3K
Comment /matt to run again
Workflows that inject
skills/rig/SKILL.mdas agent context were pinned to a 6-month-old commit SHA, meaning they ran against a frozen snapshot of the API reference rather than the version under active development in this repo.Changes
daily-rig-file-summarizer.md,daily-rig-sampler.md— updatedskills:entry from31d2dbdf...to current HEAD (eb773e967...)*.lock.yml— regenerated viagh aw compile --strictdaily-rig-task-generator.mdis unaffected — it reads the skill viacat skills/rig/SKILL.mdin the agent body, which always resolves from the workspace checkout.