PR #61090: fix(skills): exclude .git and node_modules when copying skills to workspace

• Repository: openclaw/openclaw
• Author: joelnishanth
• State: closed | merged: True
• Link: https://github.com/openclaw/openclaw/pull/61090

Description (problem / solution / changelog)

Closes #60879

Summary

• Added a filter to the fsp.cp call in syncSkillsToWorkspace that excludes .git and node_modules directories when copying skill trees to the sandbox workspace.

Root Cause

syncSkillsToWorkspace in src/agents/skills/workspace.ts uses fsp.cp with recursive: true but no filter option, so the entire skill directory tree is copied verbatim, including .git (which can contain large binary pack files) and node_modules.

Other file-walking code in the repo (e.g. computeSkillFingerprint, walkDirWithLimit in the skill scanner, listCandidateSkillDirs) already skips dot-directories and node_modules, so this was the one path that was missing the exclusion.

Changes

src/agents/skills/workspace.ts - added a filter callback to fsp.cp that rejects entries named .git or node_modules.

Test Plan

• npx vitest run src/agents/skills/ - 34/34 pass
• No lint errors on touched files

Risks and Mitigations

• The filter is conservative: it only skips .git and node_modules by basename, matching the patterns used by the rest of the skill file-walking code.
• No user-facing skill data lives in .git or node_modules, so excluding them has no functional impact.

Joel Nishanth - offlyn.AI

Changed files

• CHANGELOG.md (modified, +1/-0)
• src/agents/skills.build-workspace-skills-prompt.syncs-merged-skills-into-target-workspace.test.ts (modified, +15/-0)
• src/agents/skills/workspace.ts (modified, +4/-0)