Skip to content

feat: implement safe dev process orchestration - #77

Merged
dDevAhmed merged 1 commit into
mainfrom
feat/cli-031-dev-orchestration
Sep 8, 2026
Merged

feat: implement safe dev process orchestration#77
dDevAhmed merged 1 commit into
mainfrom
feat/cli-031-dev-orchestration

Conversation

@dDevAhmed

Copy link
Copy Markdown
Contributor

Summary

Implements CLI-031 with a security-first stellarforge dev MVP for supported generated Node.js projects.

This PR is temporarily stacked on CLI-030 so the generated-project conventions remain available while the earlier PRs complete repository-level gates.

Included

  • registers stellarforge dev
  • switches root CLI execution to Commander parseAsync
  • resolves supported project scripts from validated package.json structure
  • invokes npm using executable + argument arrays rather than executing package script text directly
  • supervises child processes with shell: false
  • prefixes stdout/stderr by process label
  • filters inherited environment variables to an explicit allowlist
  • excludes arbitrary secrets and NODE_OPTIONS
  • handles SIGINT/SIGTERM and removes handlers deterministically
  • terminates sibling processes after startup/runtime failure
  • actionable validation/subprocess errors
  • deterministic fake-child tests for clean exit, startup failure, unexpected exit, SIGINT and SIGTERM

Security

  • no shell command interpolation
  • no command strings from project metadata are executed directly
  • no broad process.env inheritance
  • no environment dump in logs/errors
  • child cleanup is centralized and tested

Closes #34

@dDevAhmed
dDevAhmed force-pushed the feat/cli-030-api-service-template branch from 6b101d4 to a01e862 Compare September 8, 2026 22:28
@dDevAhmed
dDevAhmed force-pushed the feat/cli-031-dev-orchestration branch from e7afedc to f5f61a3 Compare September 8, 2026 22:32
@dDevAhmed
dDevAhmed changed the base branch from feat/cli-030-api-service-template to main September 8, 2026 22:32
@dDevAhmed
dDevAhmed merged commit ba26b2a into main Sep 8, 2026
3 of 4 checks passed
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.

CLI-031 — Implement stellarforge dev process orchestration MVP

1 participant