Skip to content

Add make run-all command for one-click service startup - #11

Merged
kaysono0 merged 1 commit into
mainfrom
add-make-run-all-command
May 23, 2026
Merged

kaysono0 merged 1 commit into
mainfrom
add-make-run-all-command

Conversation

@kaysono0

Copy link
Copy Markdown
Owner

Summary

Add make run-all command for one-click service startup to improve developer experience.

Changes

  • Makefile: Added run-all and stop targets that delegate to shell scripts
  • scripts/run-all.sh: New script that:
    • Auto-detects Python environment (.venv or python3)
    • Generates certificates if needed
    • Starts WebRTC service (https://localhost:8080)
    • Starts Dashboard service (http://127.0.0.1:8081) with proper origin allowlist
    • Waits for services to be ready with health check (up to 10 seconds)
    • Cleans up child processes on failure
  • scripts/stop-all.sh: New script to stop all running services by PID or command pattern
  • CHANGELOG.md: Updated with new feature

Usage

# Start all services
make run-all

# Stop all services
make stop

Verification

  • make run-all starts both services successfully
  • Services respond with HTTP 200
  • API endpoints return expected JSON envelopes
  • make test-unit
  • make harness-smoke
  • make test-e2e

Checklist

  • Preserves WebRTC Service and Dashboard Service boundaries
  • Preserves JSON response envelopes
  • Updates CHANGELOG.md for user-visible changes
  • Updates docs for startup/harness behavior changes (scripts are internal)

@kaysono0 can click here to continue refining the PR

- Add 'make run-all' to start both WebRTC and Dashboard services with health check
- Add 'make stop' to stop all running services
- Add scripts/run-all.sh with auto certificate generation and Python detection
- Add scripts/stop-all.sh for service cleanup
- Update CHANGELOG.md
@kaysono0
kaysono0 marked this pull request as ready for review May 23, 2026 07:47
@kaysono0
kaysono0 merged commit 6e61a49 into main May 23, 2026
1 of 2 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.

2 participants