FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix(cdp): bound browser tool connection waits by xianjianlf2 · Pull Request #27 · different-ai/opencode-browser · GitHub

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

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>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL