From 042160c3905535339376bbc61af47150e5c68c32 Mon Sep 17 00:00:00 2001 From: Ismo Vuorinen Date: Sun, 8 Mar 2026 03:55:04 +0200 Subject: [PATCH] fix(claude): correct skill docs per PR review feedback Fix validate skill description to say "precommit" instead of "test", and fix check-pins SHA guidance to use origin/main instead of HEAD. --- .claude/skills/check-pins/SKILL.md | 7 ++++--- .claude/skills/validate/SKILL.md | 2 +- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/.claude/skills/check-pins/SKILL.md b/.claude/skills/check-pins/SKILL.md index ecd3f5f..8edad8a 100644 --- a/.claude/skills/check-pins/SKILL.md +++ b/.claude/skills/check-pins/SKILL.md @@ -30,10 +30,11 @@ This verifies that test workflows use `./action-name` format (local references a - `./action-name` in `action.yml` (non-test) files must use `ivuorinen/actions/action-name@` - External actions must be pinned to SHA commits, not version tags -**How to get the current SHA:** +**How to get the SHA for pinning:** ```bash -git rev-parse HEAD +# After pushing, get the SHA of the latest commit on the remote +git rev-parse origin/main ``` -Use this SHA when pinning internal action references after committing changes. +Use a SHA that exists on the remote. Local-only commits won't resolve when the action is used externally. diff --git a/.claude/skills/validate/SKILL.md b/.claude/skills/validate/SKILL.md index 1daeb02..86a47e9 100644 --- a/.claude/skills/validate/SKILL.md +++ b/.claude/skills/validate/SKILL.md @@ -1,6 +1,6 @@ --- name: validate -description: Run full validation pipeline (docs, format, lint, test) +description: Run full validation pipeline (docs, format, lint, precommit) disable-model-invocation: true ---