Skip to content

fix(cdp): bound browser tool connection waits - #27

Open
xianjianlf2 wants to merge 1 commit into
different-ai:mainfrom
xianjianlf2:fix/cdp-timeouts-and-cancellation
Open

fix(cdp): bound browser tool connection waits#27
xianjianlf2 wants to merge 1 commit into
different-ai:mainfrom
xianjianlf2:fix/cdp-timeouts-and-cancellation

Conversation

@xianjianlf2

Copy link
Copy Markdown

Summary

  • Add explicit timeouts to /json/list discovery and WebSocket handshakes so an unhealthy CDP listener fails instead of hanging.
  • Propagate OpenCode's context.abort through discovery, connection, CDP commands, navigation waits, and snapshot operations.
  • Reject pending CDP commands when a connection closes and add a discoverable bun test script with HTTP, WebSocket, cancellation, and cleanup coverage.
  • Include the rebuilt dist/plugin.js used by the published plugin.

Validation

  • bun test
  • bun run build
  • node --check bin/cli.js
  • git diff --check

Addresses different-ai/openwork#3443

Signed-off-by: MarkXian <mark-xian@foxmail.com>
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