Skip to content

Commit 539bd82

Browse files
committed
fix(test-classifier): point reusable workflow at fetch-skills.sh
The agent-skills migration (#41) renamed sync-skills.sh -> fetch-skills.sh and changed its behavior to vendor SKILL.md from navapbc/agent-skills into .skills-vendor/, but the reusable workflow's setup step was never updated. It chmod'd and ran the nonexistent sync-skills.sh, aborting every consumer run under `set -euo pipefail`: chmod: cannot access 'testing/classifier/scripts/sync-skills.sh': No such file or directory Update both references to fetch-skills.sh, rename the step to describe what it actually does, and drop the no-op AI_REVIEW_TOOL env (fetch-skills.sh doesn't read it; the var is still used by the classifier run step).
1 parent 981402c commit 539bd82

1 file changed

Lines changed: 3 additions & 5 deletions

File tree

.github/workflows/test-classifier.yml

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -196,13 +196,11 @@ jobs:
196196
cp -R .classifier-bundle/testing "$GITHUB_WORKSPACE/testing"
197197
chmod +x testing/classifier/.skills/_lib/ai-classifier-dispatch.sh
198198
chmod +x testing/classifier/.skills/test-classifier/scripts/test-classifier-dispatcher.sh
199-
chmod +x testing/classifier/scripts/sync-skills.sh
199+
chmod +x testing/classifier/scripts/fetch-skills.sh
200200
201-
- name: Sync canonical SKILL.md into the chosen tool's directory
201+
- name: Vendor SKILL.md from agent-skills
202202
working-directory: testing/classifier
203-
env:
204-
AI_REVIEW_TOOL: ${{ inputs.tool }}
205-
run: scripts/sync-skills.sh
203+
run: scripts/fetch-skills.sh
206204

207205
# ── Bedrock auth (provider=bedrock only) ────────────────────────────────
208206
# Exchange this run's GitHub OIDC token for short-lived STS credentials by

0 commit comments

Comments
 (0)