Skip to content

fix: recursively deploy built-in skill directories - #92

Merged
jeff-r2026 merged 1 commit into
Tencent:mainfrom
Yong-yuan-X:fix/88-recursive-builtin-skills
Jul 1, 2026
Merged

fix: recursively deploy built-in skill directories#92
jeff-r2026 merged 1 commit into
Tencent:mainfrom
Yong-yuan-X:fix/88-recursive-builtin-skills

Conversation

@Yong-yuan-X

Copy link
Copy Markdown
Contributor

Summary

Fixes #88.

deployBuiltinSkills previously copied only top-level files from each built-in skill directory. This caused team-wiki-codebase to be deployed without its nested references/ and scripts/ directories, leaving the skill incomplete after teamai pull.

This PR changes built-in skill deployment to recursively copy the full skill directory while skipping .-prefixed files/directories as suggested in #88.

It also updates ESM path resolution to use fileURLToPath(import.meta.url), which keeps the built-in skills path valid on Windows and remains correct on Linux/macOS.

Changes

  • Recursively copy built-in skill directories during deployment.
  • Skip .-prefixed files/directories during recursive copy.
  • Add a regression test that verifies team-wiki-codebase deploys nested references/ and scripts/ files.
  • Use fileURLToPath(import.meta.url) for cross-platform built-in skills path resolution.

Testing

  • npx vitest run src/__tests__/skip-uninstalled-tools.test.ts
  • npx tsc --noEmit
  • npm run build
  • git diff --check

@jeff-r2026
jeff-r2026 merged commit 3e9f553 into Tencent:main Jul 1, 2026
7 checks passed
@hsuchifeng hsuchifeng mentioned this pull request Jul 3, 2026
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[bug] deployBuiltinSkills 不递归子目录,导致 team-wiki-codebase skill 部署后 references/scripts 丢失

2 participants