Skip to content

Conversation

@DamianB-BitFlipper
Copy link

@DamianB-BitFlipper DamianB-BitFlipper commented Jan 27, 2026

The way that the prime cli handles "unresponsive" is not accurate. It is simply a timeout. Remove the concept of "unresponsive", and treat it as a timeout. If unresponsive sandboxes are an issue, the way to detect them would be with a liveliness checks and heartbeats.


Note

Simplifies sandbox timeout handling and removes the "unresponsive" concept.

  • Removes SandboxUnresponsiveError (and its exports) and maps gateway timeouts to CommandTimeoutError in both sync/async execute_command
  • Adds a small client timeout buffer (effective_timeout + 2s) to prevent premature client-side timeouts
  • Bumps prime-sandboxes to 0.2.12
  • Minor CLI/utilities cleanup (logging/messages/formatting) in RL commands and env utils; no functional changes

Written by Cursor Bugbot for commit 64a2dff. This will update automatically on new commits. Configure here.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 3027b44635

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

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.

2 participants