Skip to content

Commit ae59ac8

Browse files
os-zhuangclaude
andcommitted
fix(ci): strip ANSI codes before asserting the skills --list output
The skills CLI colors the count even when piped (Found \e[32m9\e[39m skills), so the repo-root listing greps in the new skills-boundary step never matched in CI. The subpath-install set-equality assertion had already passed. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent d454775 commit ae59ac8

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

.github/workflows/scaffold-e2e.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -143,7 +143,9 @@ jobs:
143143
exit 1
144144
fi
145145
146-
listing=$(npx -y skills add "$GITHUB_WORKSPACE" --list 2>&1)
146+
# Strip ANSI color codes — the skills CLI colors the count even
147+
# when piped ("Found \e[32m9\e[39m skills"), which broke the greps.
148+
listing=$(npx -y skills add "$GITHUB_WORKSPACE" --list 2>&1 | sed 's/\x1b\[[0-9;]*m//g')
147149
echo "$listing"
148150
if echo "$listing" | grep -q 'dogfood-verification'; then
149151
echo "::error::repo-root discovery surfaced an internal skill (metadata.internal marker missing?)"

0 commit comments

Comments
 (0)