You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
What broke: bun run check:bundled-skill reports all 20 nested skill files as missing on Windows, even though bundled-skill.ts correctly embeds them. As a result, bun run validate fails on Windows.
Run bun run check:bundled-skill (or bun run validate).
Expected vs Actual
Expected: bundled-skill.ts is up to date (21 files). exit 0
Actual:
bundled-skill.ts is missing these files:
- examples\command-template.md
- examples\dag-workflow.yaml
- guides\cli.md
... (20 entries total, all with backslashes)
Exit 2 in --check mode, breaking bun run validate.
Root Cause
scripts/check-bundled-skill.ts line 35 uses path.relative(), which on Windows returns paths with backslashes (examples\command-template.md). The substring check on line 41 (bundledSrc.includes(f)) then fails because bundled-skill.ts declares its imports with forward slashes (examples/command-template.md).
The single top-level skill file (SKILL.md, no separator) is the only one that passes — which is why the error lists exactly 20 files, not 21.
The bundle itself is fine; only the verifier is broken.
Suggested Fix
Normalize separators before the includes check in scripts/check-bundled-skill.ts:
constskillFiles=listSkillFiles(SKILL_ROOT).map(f=>f.split('\').join('/')) // normalize for Windows.sort();
Apply the same normalization in the missing-files print loop so the suggested entries match what users would paste into bundled-skill.ts.
User Flow
Windows contributor check-bundled-skill.ts bundled-skill.ts
───────────────────── ──────────────────────── ────────────────
runs bun run validate ──▶ listSkillFiles() returns
'examples\command-template.md'
(backslash from path.relative)
[X] bundledSrc.includes(f) → false
(file actually present with '/')
sees 20 false missing ◀── exit 2
Environment
Platform: CLI (script invoked by bun run validate)
Summary
bun run check:bundled-skillreports all 20 nested skill files as missing on Windows, even thoughbundled-skill.tscorrectly embeds them. As a result,bun run validatefails on Windows.dev) — addedscripts/check-bundled-skill.tsand wired it into thevalidatepipeline.major(blocksbun run validatefor any Windows contributor ondev)Steps to Reproduce
devon a Windows machine (PR feat(cli): setup overhaul + archon doctor + complete bundled skill #1566 merged).bun run check:bundled-skill(orbun run validate).Expected vs Actual
bundled-skill.ts is up to date (21 files).exit 0--checkmode, breakingbun run validate.Root Cause
scripts/check-bundled-skill.tsline 35 usespath.relative(), which on Windows returns paths with backslashes (examples\command-template.md). The substring check on line 41 (bundledSrc.includes(f)) then fails becausebundled-skill.tsdeclares its imports with forward slashes (examples/command-template.md).The single top-level skill file (
SKILL.md, no separator) is the only one that passes — which is why the error lists exactly 20 files, not 21.The bundle itself is fine; only the verifier is broken.
Suggested Fix
Normalize separators before the includes check in
scripts/check-bundled-skill.ts:Apply the same normalization in the missing-files print loop so the suggested entries match what users would paste into
bundled-skill.ts.User Flow
Environment
bun run validate)Logs
Impact
bun run validate,bun run check:bundled-skill(CI guard added in feat(cli): setup overhaul + archon doctor + complete bundled skill #1566)check:bundled-skilland run the other validate steps individually.Scope
cli(the script lives at repo root underscripts/, but its purpose is verifyingpackages/cli/src/bundled-skill.ts)scripts/check-bundled-skill.tsRelated
dev)