Skip to content

Fix skill script path traversal#8

Merged
8421bit merged 1 commit intomainfrom
fix/issue-5-skill-path-traversal
Apr 29, 2026
Merged

Fix skill script path traversal#8
8421bit merged 1 commit intomainfrom
fix/issue-5-skill-path-traversal

Conversation

@8421bit
Copy link
Copy Markdown
Owner

@8421bit 8421bit commented Apr 29, 2026

Summary

  • Add skills-directory path guards for skill execution paths.
  • Block skillName and scriptFile traversal before filesystem access.
  • Reuse the guard for skill create/delete, skill content reads, and sandbox validation.
  • Execute skill scripts with execFile instead of shell-built command strings.
  • Add regression tests for the reported traversal pattern and related boundary cases.

Fixes #5.

Verification

  • npm test
  • npm run build

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.

CWE-22 Path Traversal Vulnerability in executeSkillScript Allows Unauthorized File Access

1 participant