Skip to content

chore(sdk): sync generated SDK (v1.0.17) - #234

Closed
abonneth wants to merge 1 commit into
mainfrom
sync/agp
Closed

abonneth wants to merge 1 commit into
mainfrom
sync/agp

Conversation

@abonneth

@abonneth abonneth commented Oct 5, 2026 •

Copy link
Copy Markdown
Collaborator

Auto-generated from the upstream codegen home.

Version: 1.0.17 (patch; every sync bumps +0.0.1)
Upstream: agent_platform@e525a14


Note

Low Risk
Additive credential resolution for client construction; reads a local config file but does not change how keys are sent on requests.

Overview
Bumps the Python SDK to v1.0.17 (package version, lockfile, and HTTP client metadata).

Client / AsyncClient now fill in a missing api_key automatically: explicit argument, then HAI_API_KEY, then the value hai login wrote to the config .env ($XDG_CONFIG_HOME/hai/.env or ~/.config/hai/.env). If nothing is found, construction fails with an ApiError that mentions passing api_key, setting the env var, or running hai login.

Reviewed by Cursor Bugbot for commit 8722418. Bugbot is set up for automated code reviews on this repo. Configure here.

@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 using default effort and found 2 potential issues.

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 79e51ab. Configure here.

Comment thread src/hai_agents/client.py
Comment thread src/hai_agents/client.py
"""Stop sessions this client bridged; a local client also stops its runtime once no other client uses it."""
try:
if self._sessions is not None:
self._sessions.close()

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Close calls missing session methods

Medium Severity

close and aclose invoke self._sessions.close() and self._sessions.aclose(), but neither LocalSessionsClient nor the generated sessions clients define those methods. After sessions are initialized, the new context-manager cleanup raises AttributeError instead of stopping bridged sessions.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 79e51ab. Configure here.

@abonneth

abonneth commented Oct 5, 2026

Copy link
Copy Markdown
Collaborator Author

Folded into #233.

@abonneth abonneth closed this Oct 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant