Skip to content

Sync next into main#100

Merged
b-Tomas merged 5 commits into
mainfrom
next
May 27, 2026
Merged

Sync next into main#100
b-Tomas merged 5 commits into
mainfrom
next

Conversation

@b-Tomas

@b-Tomas b-Tomas commented May 27, 2026

Copy link
Copy Markdown
Member

No description provided.

b-Tomas and others added 5 commits May 26, 2026 09:17
Mirrors inorbit-ai/inorbit-connector-python#74 so the next branch gets
the same pre-merge gate as main.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
* Replace account_id param with lazy fetch from GET /user

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* Address review: validate accountIds shape, downgrade log, assert cache

Co-Authored-By: Claude Code <noreply@anthropic.com>

* Remove stale references

* Fix formatting issues

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
* Consolidate InOrbit URL env vars

INORBIT_API_URL meant the configuration endpoint in models but the REST
API in the demo example and downstream executor. Shared env vars across
the Connect SDK stack were unusable. Establish new names:

- INORBIT_CONFIG_URL: configuration endpoint (was INORBIT_API_URL in
  models, INORBIT_URL in demo)
- INORBIT_API_URL: REST API endpoint (was INORBIT_REST_API_URL in models)

Old names removed with no fallback (breaking). Module constant
INORBIT_REST_API_URL renamed to INORBIT_DEFAULT_REST_API_URL to
disambiguate from the env var. Exported INORBIT_CLOUD_SDK_ROBOT_CONFIG_URL
preserved (imported by connector-python).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* Rename INORBIT_CONFIG_URL -> INORBIT_CONNECTION_CONFIG_URL

* Fix linter

---------

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Remove `with_counter_metric_async()` (deprecated in v2.0.2) and
`RobotSessionPool.register_command_callback()` (deprecated in v1.7.2).
Both had zero production callers. Also removes the `deprecated` package
dependency since no other code imports it.

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
@b-Tomas b-Tomas self-assigned this May 27, 2026
@b-Tomas b-Tomas requested a review from miguelgarcia May 27, 2026 13:23
@b-Tomas b-Tomas merged commit 66d1df9 into main May 27, 2026
24 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.

2 participants