Skip to content

feat: add daemon cleanup stress test and tighten stream foreground handling#59

Open
DjDeveloperr wants to merge 1 commit into
mainfrom
add-cli-cleanup-stress-test
Open

feat: add daemon cleanup stress test and tighten stream foreground handling#59
DjDeveloperr wants to merge 1 commit into
mainfrom
add-cli-cleanup-stress-test

Conversation

@DjDeveloperr
Copy link
Copy Markdown
Collaborator

Summary

  • Added a dedicated daemon cleanup stress harness that starts isolated temporary project daemons, exercises health/metrics, and verifies process-group, listener-port, and metadata cleanup.
  • Wired the harness into npm run test:stress:daemon and documented it under testing.
  • Tightened viewer foreground detection so hidden or offscreen canvases stop reporting as active, reducing unnecessary encoder work.
  • Paired that client signal with native session cleanup so the video path pauses frame input when the last subscriber leaves and resumes when streaming restarts.
  • Documented the current encoding behavior and the low-footprint interpretation of viewer activity.

Testing

  • cargo test --manifest-path server/Cargo.toml
  • cargo fmt --manifest-path server/Cargo.toml --check
  • npm run --prefix client typecheck
  • npx prettier --check ...
  • node --check scripts/stress/daemon-cleanup.mjs
  • Ran scripts/stress/daemon-cleanup.mjs once against the debug binary with a passing cleanup result

@DjDeveloperr DjDeveloperr marked this pull request as ready for review May 21, 2026 18:35
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