fix(skills): ignore hidden directories when walking skill files

[adboio]

Problem

A user hit Local skill bundle contains more than 1000 files when invoking a custom skill on a cloud run. Two issues behind it:

• The bundler walks the entire skill directory tree and its ignore list was only .DS_Store / .git / node_modules / __pycache__, so hidden directories like .venv (easily 5–20k files) counted toward the limit and got uploaded.
• The error named neither the offending skill nor a remedy — and the offender can be a skill pulled in via dependencies: that the user never tagged.

Changes

• Skip hidden directories when walking skill files, via a shared isIgnoredSkillEntry predicate. This extends the existing findSkillDirs rule that hidden dirs are never skills. Hidden files (.gitignore) still count.
• Apply the same predicate in listSkillFiles, so the skill-contents UI and team-skill exports match what a cloud bundle uploads (previously that walker skipped nothing — publishing a team skill would ship a .venv to the skills store).
• The bundle limit errors now name the skill, its resolved path, and what to do: Skill "alpha" (/home/u/.claude/skills/alpha) contains more than 1000 files. Cloud runs upload every file in the skill folder, so move data and build artifacts out of it.

Left alone: the 1000-file cap itself (only the PostHog server's 30MB artifact size limit is a real platform constraint; raising/dropping the count cap can be a follow-up if legit >1000-file skills show up), and the marketplace install direction (unpacking downloaded archives stays a faithful copy).

Testing

• New skill-bundler.test.ts unzips real bundles: hidden dirs and junk excluded while hidden files and nested content survive; 1100 files inside .venv bundle fine (the reported scenario); the limit error names skill and path.
• Matching listSkillFiles case in skill-discovery.test.ts.
• All 90 workspace-server skills tests pass; tsc --noEmit and Biome clean.

🤖 Generated with Claude Code

[trunk-io]

Merging to main in this repository is managed by Trunk.

• To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

[github-actions]

React Doctor found no issues in the changed files. 🎉

_{Reviewed by React Doctor for commit 0f3162c.}

[greptile-apps]

_{Reviews (1): Last reviewed commit: "fix(skills): ignore hidden directories w..." | Re-trigger Greptile}