Skip to content

fix(sdk): eliminate shell injection vectors#140

Draft
diberry wants to merge 1 commit intodevfrom
squad/895-shell-injection-fixes
Draft

fix(sdk): eliminate shell injection vectors#140
diberry wants to merge 1 commit intodevfrom
squad/895-shell-injection-fixes

Conversation

@diberry
Copy link
Copy Markdown
Owner

@diberry diberry commented Apr 10, 2026

CI validation for bradygaster#916

…ckend

Replace all execSync calls with execFileSync using explicit argv arrays.
Refactor git helper functions to accept string[] instead of string.split.
Add validateTaskRef() and validateStateKey() for input validation.
Add parse-time validation for script task refs.
Add security-focused tests proving injection attempts are blocked.

Addresses bradygaster#895 (P0-1, P0-2)

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

1 participant