Skip to content

Add process-local JavaScript session monitoring - #201

Open
cpendery (cpendery) wants to merge 3 commits into
cpendery/interactive-monitorfrom
cpendery/process-local-test-monitoring
Open

Add process-local JavaScript session monitoring#201
cpendery (cpendery) wants to merge 3 commits into
cpendery/interactive-monitorfrom
cpendery/process-local-test-monitoring

Conversation

@cpendery

@cpendery cpendery (cpendery) commented Sep 3, 2026

Copy link
Copy Markdown
Member

Summary

Adds opt-in monitoring for process-local JavaScript terminal sessions, building on #200's interactive monitor client.

  • starts one lazy IPC bridge per Node process and exposes monitored sessions through tui-test sessions
  • routes frames and raw interactive input to an exact process-owned PTY generation
  • keeps one attachment lease across frame reconnects so terminal resize cannot release a failed test early
  • keeps failed test terminals available for attachment, waits for connected monitors to detach, then rethrows the original error
  • preserves replacement sessions when same-name open, restart, close, and monitor-wait operations overlap
  • adds labels and test metadata for discovery plus environment-based monitoring configuration

Dependency

This PR targets cpendery/interactive-monitor and depends on #200.

Validation

  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets --all-features -- -D warnings
  • cargo clippy -p tui-test-rs --all-targets --no-default-features -- -D warnings
  • cargo test --workspace -- --test-threads=1
  • Node test suite: 68 tests passed
  • manual Windows end-to-end check: discovered and attached to a failed process-local session, forwarded raw input, resized the viewer while the attachment lease remained active, detached, and preserved the original error

@cpendery
cpendery (cpendery) force-pushed the cpendery/process-local-test-monitoring branch from 2dfa961 to 4f71d4e Compare September 4, 2026 07:00
@cpendery
cpendery (cpendery) marked this pull request as ready for review September 4, 2026 07:36
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