Repository navigation

Fix CI PTY test failures with increased timeouts and MWE - #12

Closed

MBanucu wants to merge 267 commits into

shekohex:mainfrom

MBanucu:fix-ci-pty-tests

This pull request is big! We're only showing the most recent 250 commits

Commits on Jan 22, 2026
Commits on Jan 23, 2026
Commits on Jan 24, 2026
Commits on Jan 25, 2026
Commits on Jan 26, 2026
- refactor(web): improve static file handling security and path resolution
  MBanucu
  committed
  57e6657
- chore: remove temporary documentation and config files
  MBanucu
  committed
  056ca8b
- chore: prepare for npm test release
  MBanucu
  committed
  4d0f55c
- feat: add automated testing scripts for npm packages and OpenCode plugins
  MBanucu
  committed
  d64bd87
- chore: prepare test release 0.1.4-test.1
  MBanucu
  committed
  2202836
Commits on Jan 27, 2026
- refactor(web): simplify server to always serve built assets
  MBanucu
  committed
  6344783
- chore: clean up development scripts and enable e2e test session
  MBanucu
  committed
  afdabcf
- refactor(test-web-server): simplify test-web-server.ts structure
  MBanucu
  committed
  a458a3a
- refactor: clean up PTY session configuration and test artifacts
  MBanucu
  committed
  519b98b
- refactor: move test-web-server.ts to e2e directory
  MBanucu
  committed
  5173b16
- chore: remove redundant manual e2e test file
  MBanucu
  committed
  d74d8f7
- refactor: move start-server.ts to test directory
  MBanucu
  committed
  31f96a6
- feat(web/router): add router infrastructure with parameter support and middleware
  MBanucu
  committed
  48be6e8
- feat(web/router): migrate API routes to unified router system
  MBanucu
  committed
  a5a1ade
- docs: update routing improvements plan for Phase 2 completion
  MBanucu
  committed
  11fbf60
- refactor(web): consolidate error handling and add input validation
  MBanucu
  committed
  ded723b
- docs(plan): document Phase 3 completion in routing improvements plan
  MBanucu
  committed
  cf1cb6c
- refactor(server): replace custom router with Bun's built-in routing
  MBanucu
  committed
  3a444a2
- refactor(web-server): serve static files via dynamic routes
  MBanucu
  committed
  ee6388f
- fix(test): await startWebServer calls in remaining tests
  MBanucu
  committed
  4d16001
- refactor(websocket): move websocket upgrade to dedicated /ws route
  MBanucu
  committed
  b5b2172
- refactor(server): redirect root path to index.html
  MBanucu
  committed
  14fbb25
- feat(web/server): implement WebSocket pub/sub and fix session updates
  MBanucu
  committed
  4e01b86
- refactor(web): restructure into client, server, and shared subdirectories
  MBanucu
  committed
  560f1ed
- refactor: remove Pino logger and console.log statements
  MBanucu
  committed
  ed3d0e1
- fix(websocket): do not return response on successful upgrade
  MBanucu
  committed
  bc6be8a
- fix: update version and author in package.json; clean up error handling in useWebSocket
  MBanucu
  committed
  d6de54e
- docs: comprehensively update project documentation
  MBanucu
  committed
  e506a6e
- refactor(pty): streamline server URL handling and add port conflict resolution
  MBanucu
  committed
  55f4ff9
- Merge upstream/main to make branch mergeable
  MBanucu
  committed
  d7926d4
- docs: remove completed routing simplification plan
  MBanucu
  committed
  6b01be5
- chore: bump version to 0.1.4-test.4
  MBanucu
  committed
  255f6de
- refactor: rename slash command to pty-server-url
  MBanucu
  committed
  7795c56
- chore: update package metadata to shekohex
  MBanucu
  committed
  512800a
- ci: remove security job from CI workflow
  MBanucu
  committed
  66e705d
- ci: clean up CI workflow triggers
  MBanucu
  committed
  3aa35f5
- fix: update import paths to resolve TypeScript module resolution errors
  MBanucu
  committed
  914582d
- ci: disable lint step and simplify test command
  MBanucu
  committed
  f525abd
- ci: update build command to build:all:prod
  MBanucu
  committed
  1965607
- ci: update GitHub Actions to latest versions
  MBanucu
  committed
  17e9b45
- ci: fix test execution in non-interactive CI environment
  MBanucu
  committed
  f40087e
- ci: use unbuffer to allocate TTY and propagate exit codes
  MBanucu
  committed
  4e1093f
- test: add debug logging to failing tests
  MBanucu
  committed
  03be5e7
- Increase test timeouts and add bun-pty MWE to fix CI failures
  MBanucu
  committed
  e19ae63
- Increase timeouts to 1000ms for PTY tests to fix CI timing issues
  MBanucu
  committed
  949552c
- Revert timeouts to original values (100ms web-server, 200ms pty-echo) as 1000ms did not fix CI failures
  MBanucu
  committed
  35fa7cf
- Add console.log for debugging CI failures in PTY tests
  MBanucu
  committed
  f99ac25
- Skip PTY-dependent tests in CI due to manager initialization issues, while keeping MWE tests
  MBanucu
  committed
  13cbbd4
- Add extensive console.log in PTY manager and lifecycle to debug CI failures
  MBanucu
  committed
  149a4a7
- Temporarily enable PTY tests in CI to capture debug logs from plugin
  MBanucu
  committed
  7917d00
- Fix web-server test expectations for PTY session command and args
  MBanucu
  committed
  99e9461
- Add extensive console.log for debugging CI PTY failures
  MBanucu
  committed
  7a0d49c
- Increase timeouts in pty-echo test and add raw data logging
  MBanucu
  committed
  9d3622d
- Add newline to input in pty-echo test
  MBanucu
  committed
  1c411e8
- Remove debug console.log statements from production code
  MBanucu
  committed
  0d9188f
- ci(workflows): remove unnecessary expect installation and unbuffer from CI
  MBanucu
  committed
  021b49a
- feat(debug): re-add console.log for CI PTY debugging
  MBanucu
  committed
  3eb7e2f
- ci: filter to run only pty-echo test for debugging
  MBanucu
  committed
  315f160
- ci: run only pty-echo.test.ts for debugging
  MBanucu
  committed
  fa3dbfe
- test: add second pty-echo test to verify no interference
  MBanucu
  committed
  1582da7
- ci: add web-server tests to CI filter
  MBanucu
  committed
  ff15aad
- ci: run tests sequentially to avoid Bun crash
  MBanucu
  committed
  8cce3f5
- ci: use --concurrency 1 to run tests sequentially
  MBanucu
  committed
  d8c2f11
- ci: revert to && for sequential test execution
  MBanucu
  committed
  2a402ef
- debug: add timestamps to console.log for CI analysis
  MBanucu
  committed
  1814ca1
- fix: remove sequential test execution in CI
  MBanucu
  committed
  cee167c
- ci: add websocket test to CI
  MBanucu
  committed
  b17bb16
- ci: run all tests without filter
  MBanucu
  committed
  feb8549
- ci: run all tests sequentially to avoid state interference
  MBanucu
  committed
  d38384d
- ci: use --max-concurrency=1 for sequential test execution
  MBanucu
  committed
  5132983
- fix: add clearRawOutputCallbacks and use in tests to prevent interference
  MBanucu
  committed
  1f1b6de
- fix: clear callbacks in beforeEach to ensure clean state
  MBanucu
  committed
  ccca6d4
- ci: revert to sequential tests due to persistent parallel interference
  MBanucu
  committed
  bf10537
- ci: exclude pty-echo and web-server tests to test parallel with others
  MBanucu
  committed
  a048fc1
- ci: add back web-server.test.ts to test parallel compatibility
  MBanucu
  committed
  d2720eb
- ci: use matrix strategy to run test groups in parallel jobs
  MBanucu
  committed
  6dd35be
- ci: isolate PTY tests in separate matrix jobs to prevent interference
  MBanucu
  committed
  56d4e0e
- ci: revert to single job to test if interference is fixed
  MBanucu
  committed
  4935b8e
- refactor: add onData to SpawnOptions for isolated output handling in tests
  MBanucu
  committed
  66eefe3
- ci: use --concurrency 1 to run tests serially and avoid interference
  MBanucu
  committed
  8086ee0
- ci: run tests sequentially with && to avoid interference
  MBanucu
  committed
  62a87ff
- Fix bun tests by using global PTYManager in web server tests
  MBanucu
  committed
  45f3f88
- ci: run tests with --concurrency 1 to prevent interference between tests using global manager
  MBanucu
  committed
  4f48161
- fix(ci): correct --concurrency flag syntax to =1
  MBanucu
  committed
  88442f8
- fix: handle missing dist/web directory in buildStaticRoutes for tests
  MBanucu
  committed
  bc3adf8
- fix: run HTTP Endpoints tests serially to prevent interference
  MBanucu
  committed
  0ecf2d5
- fix: make all web server tests run serially to prevent interference between describe blocks
  MBanucu
  committed
  ffcedc5
- fix: use separate PTYManager instance in web server tests to avoid interference from global manager mocks
  MBanucu
  committed
  2579e16
- Fix CI tests by making web server tests serial and passing manager to server
  MBanucu
  committed
  76349aa
Commits on Jan 28, 2026

Commits

Commits on Jan 22, 2026

Commits on Jan 23, 2026

Commits on Jan 24, 2026