Skip to content

Navigation Menu

Sign in
Sign up

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

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

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

Conversation

@xianjianlf2

@xianjianlf2 xianjianlf2 commented Aug 3, 2026

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

AltStyle によって変換されたページ (->オリジナル) /