Skip to content

v0.2.126 - #130

Open
ggelango wants to merge 17 commits into
gautham/v0.2.125from
gautham/v0.2.126
Open

v0.2.126#130
ggelango wants to merge 17 commits into
gautham/v0.2.125from
gautham/v0.2.126

Conversation

@ggelango

@ggelango ggelango commented May 9, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread fleet/_async/client.py
Comment thread fleet_harness.py Outdated

@cursor cursor 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.

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

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit b8e9118. Configure here.

Comment thread fleet/client.py
instance.instance.load()
# Resources load lazily on first `db()`/`browser()`/`resources()` access via
# `_load_resources()`. Skipping the eager preload avoids fail-fast 502s while
# the container is still warming up.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Removing load() breaks db() and browser() calls

High Severity

The instance.instance.load() call was removed with a comment claiming resources load lazily on first db()/browser()/resources() access via _load_resources(). However, InstanceClient.db() and InstanceClient.browser() (and their async counterparts) do not call _load_resources() — only resources() does. Since _resources_state is initialized with empty dicts, calling db() or browser() after make() or get() will now raise a KeyError, breaking the SDK's primary workflow.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit b8e9118. Configure here.

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