Skip to content

Commit 1ff06d0

Browse files
ncrmroclaude
andauthored
fix: Update CI test paths to enable merge queue validation (#173)
* test: Add test-merge-queue branch to workflow triggers This enables testing merge queue functionality without affecting main branch. Workflows modified: - validate.yml - claude-code-test.yml - cla.yml Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * test: Second test PR for merge queue validation * test: First test PR for merge queue validation (#172) * fix: Update CI test paths and enable merge queue validation Changes: - Fix claude-code-test.yml paths from .claude/commands/ to .claude/skills/ (generator now outputs to skills/ directory) - Remove test-merge-queue branch references from workflow triggers - Remove temporary test files This fix enables the merge queue to properly validate PRs before merging. The CI tests were previously passing on PRs because they skip actual tests, but would fail in the merge queue due to incorrect paths. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * fix: Update CI test paths for skill directory structure The test workflow expected flat skill files but the generator creates skill directories. Updated all path checks from `.claude/skills/X.md` to `.claude/skills/X/SKILL.md` format. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> * fix: Use --dangerously-skip-permissions instead of --yes for Claude CLI The --yes flag doesn't exist in the Claude CLI. The correct flag for bypassing permission prompts in CI is --dangerously-skip-permissions. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
1 parent eda2388 commit 1ff06d0

1 file changed

Lines changed: 18 additions & 18 deletions

File tree

.github/workflows/claude-code-test.yml

Lines changed: 18 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -82,20 +82,20 @@ jobs:
8282
8383
# Validate generated commands exist
8484
echo "Checking generated commands..."
85-
ls -la test_project/.claude/commands/
85+
ls -la test_project/.claude/skills/
8686
87-
# Verify command files exist
88-
test -f test_project/.claude/commands/fruits.identify.md || (echo "Missing fruits.identify.md" && exit 1)
89-
test -f test_project/.claude/commands/fruits.classify.md || (echo "Missing fruits.classify.md" && exit 1)
87+
# Verify command files exist (skills are directories with SKILL.md)
88+
test -f test_project/.claude/skills/fruits.identify/SKILL.md || (echo "Missing fruits.identify/SKILL.md" && exit 1)
89+
test -f test_project/.claude/skills/fruits.classify/SKILL.md || (echo "Missing fruits.classify/SKILL.md" && exit 1)
9090
9191
# Verify command content
92-
grep -q "# fruits.identify" test_project/.claude/commands/fruits.identify.md
93-
grep -q "raw_items" test_project/.claude/commands/fruits.identify.md
94-
grep -q "identified_fruits.md" test_project/.claude/commands/fruits.identify.md
92+
grep -q "# fruits.identify" test_project/.claude/skills/fruits.identify/SKILL.md
93+
grep -q "raw_items" test_project/.claude/skills/fruits.identify/SKILL.md
94+
grep -q "identified_fruits.md" test_project/.claude/skills/fruits.identify/SKILL.md
9595
96-
grep -q "# fruits.classify" test_project/.claude/commands/fruits.classify.md
97-
grep -q "identified_fruits.md" test_project/.claude/commands/fruits.classify.md
98-
grep -q "classified_fruits.md" test_project/.claude/commands/fruits.classify.md
96+
grep -q "# fruits.classify" test_project/.claude/skills/fruits.classify/SKILL.md
97+
grep -q "identified_fruits.md" test_project/.claude/skills/fruits.classify/SKILL.md
98+
grep -q "classified_fruits.md" test_project/.claude/skills/fruits.classify/SKILL.md
9999
100100
echo "Command generation validated successfully!"
101101
@@ -173,7 +173,7 @@ jobs:
173173
174174
echo "Fresh test project setup complete"
175175
echo "Available commands:"
176-
ls -la test_project/.claude/commands/
176+
ls -la test_project/.claude/skills/
177177
178178
# STEP 1: Use /deepwork_jobs.define to CREATE the fruits job
179179
- name: Create job with /deepwork_jobs.define
@@ -184,7 +184,7 @@ jobs:
184184
echo "=== Running /deepwork_jobs.define to create fruits job ==="
185185
186186
# Provide detailed, deterministic instructions for creating the job
187-
claude --yes --print "/deepwork_jobs.define" <<'PROMPT_EOF'
187+
claude --dangerously-skip-permissions --print "/deepwork_jobs.define" <<'PROMPT_EOF'
188188
I want to create a simple job called "fruits" for identifying and classifying fruits.
189189
190190
Here are the EXACT specifications - please create the job.yml with these exact details:
@@ -233,7 +233,7 @@ jobs:
233233
run: |
234234
echo "=== Running /deepwork_jobs.implement to generate step instructions ==="
235235
236-
claude --yes --print "/deepwork_jobs.implement" <<'PROMPT_EOF'
236+
claude --dangerously-skip-permissions --print "/deepwork_jobs.implement" <<'PROMPT_EOF'
237237
Please implement the "fruits" job that was just defined.
238238
239239
For the identify step, create instructions that:
@@ -270,9 +270,9 @@ jobs:
270270
uv run deepwork sync --path test_project
271271
272272
echo "=== Checking generated commands ==="
273-
ls -la test_project/.claude/commands/
273+
ls -la test_project/.claude/skills/
274274
275-
if [ -f "test_project/.claude/commands/fruits.identify.md" ] && [ -f "test_project/.claude/commands/fruits.classify.md" ]; then
275+
if [ -f "test_project/.claude/skills/fruits.identify/SKILL.md" ] && [ -f "test_project/.claude/skills/fruits.classify/SKILL.md" ]; then
276276
echo "SUCCESS: Slash commands generated"
277277
else
278278
echo "ERROR: Slash commands were not generated"
@@ -287,7 +287,7 @@ jobs:
287287
run: |
288288
echo "=== Running /fruits.identify with test input ==="
289289
290-
claude --yes --print "/fruits.identify" <<'PROMPT_EOF'
290+
claude --dangerously-skip-permissions --print "/fruits.identify" <<'PROMPT_EOF'
291291
raw_items: apple, car, banana, chair, orange, table, mango, laptop, grape, bicycle
292292
PROMPT_EOF
293293
@@ -309,7 +309,7 @@ jobs:
309309
run: |
310310
echo "=== Running /fruits.classify ==="
311311
312-
claude --yes --print "/fruits.classify"
312+
claude --dangerously-skip-permissions --print "/fruits.classify"
313313
314314
# Verify output was created
315315
if [ -f "classified_fruits.md" ]; then
@@ -368,7 +368,7 @@ jobs:
368368
name: claude-code-e2e-outputs
369369
path: |
370370
test_project/.deepwork/jobs/fruits/
371-
test_project/.claude/commands/fruits.*.md
371+
test_project/.claude/skills/fruits.*/SKILL.md
372372
test_project/identified_fruits.md
373373
test_project/classified_fruits.md
374374
retention-days: 7

0 commit comments

Comments
 (0)