fix(security): resolve Dependabot alerts and CodeQL findings by rsaz · Pull Request #105 · expressots/expressots-cli · GitHub
Skip to content

fix(security): resolve Dependabot alerts and CodeQL findings#105

Open
rsaz wants to merge 1 commit into
mainfrom
fix/post-merge-security-cleanup
Open

fix(security): resolve Dependabot alerts and CodeQL findings#105
rsaz wants to merge 1 commit into
mainfrom
fix/post-merge-security-cleanup

Conversation

@rsaz

@rsaz rsaz commented Jun 19, 2026

Copy link
Copy Markdown
Member

Summary

  • Add npm overrides for js-yaml@4.2.0 and undici@7.28.0 to clear the three open Dependabot alerts on main
  • Bump semver to 7.8.4 (patch-level security/maintenance)
  • Refactor src/dev/form.ts docker orchestration to use safeSpawn / safeSpawnSync instead of shell-string execSync, resolving the two open CodeQL shell-injection warnings without changing CLI behavior

Test plan

  • npm ci
  • npm test (522 passing)
  • npm audit reports 0 vulnerabilities locally after lockfile refresh

Add npm overrides for js-yaml and undici, bump semver to 7.8.4, and route
container-dev docker calls through safeSpawn/safeSpawnSync to avoid shell
injection warnings without changing runtime behavior.
@codecov

codecov Bot commented Jun 19, 2026

Copy link
Copy Markdown

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