Skip to content

Clarify browser timeout continuation semantics - #139

Merged
MagMueller merged 2 commits into
mainfrom
clarify-timed-out-invocation
Aug 3, 2026
Merged

Clarify browser timeout continuation semantics#139
MagMueller merged 2 commits into
mainfrom
clarify-timed-out-invocation

Conversation

@MagMueller

@MagMueller MagMueller commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Summary

  • state explicitly that a timed-out browser_execute invocation cannot issue further CDP commands
  • clarify that this does not close the shared CDP connection
  • mirror the same distinction in the browser skill

This changes guidance and error wording only. Timeout isolation and recovery behavior are unchanged.

Testing

  • targeted browser timeout/session regressions: 25 passed, 6 environment-dependent skipped
  • bun run typecheck in packages/bcode-browser
  • repository pre-push typecheck (17 packages)

Summary by cubic

Clarify timeout semantics for browser_execute: a timed-out call can’t issue more CDP commands, but the CDP session stays open and the next browser_execute continues on the same connection/target; the last command from the timed-out call may still complete. Updated the timeout error text in packages/bcode-browser/src/browser-execute.ts and the guidance in packages/bcode-browser/skills/browser-execute/SKILL.md; no behavior changes.

Written for commit c015c79. Summary will update on new commits.

Review in cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

Re-trigger cubic

@MagMueller
MagMueller merged commit a08812e into main Aug 3, 2026
3 checks passed
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