Skip to content

chore(deps): lock robosystems-client 1.13.1 - #10

Merged
jfrench9 merged 1 commit into
mainfrom
chore/client-1-13-1
Aug 31, 2026
Merged

jfrench9 merged 1 commit into
mainfrom
chore/client-1-13-1

Conversation

@jfrench9

Copy link
Copy Markdown
Member

Summary

Pulls through robosystems-client 1.13.1, which replays a rate-limited request instead of surfacing the 429 (RoboFinSystems/robosystems-python-client#201). The rejection is raised by a request dependency before the endpoint handler runs, so the call had no effect and the replay is safe even for a POST with no idempotency key.

This covers demo_loop's own client use — graph provisioning, teardown, API keys.

This alone does not fix the failing demo run. The episode load that hit the rate limit runs from the robosystems checkout and takes its SDK from that repo's lockfile, not this one — RoboFinSystems/robosystems#1318 is the half that does, and it also stops the runner from swallowing dropped events into a counter.

Changes

  • uv.lockrobosystems-client 1.12.0 → 1.13.1. Lock only; the >=1,<2 floor stays where it is.

Testing

  • ruff check, ruff format --check, basedpyright clean; pytest 6 passed.

1.13.1 replays a rate-limited request instead of surfacing the 429. The
rejection is raised before the endpoint handler runs, so the call had no
effect and the replay is safe.

This covers `demo_loop`'s own client use (provisioning, teardown, API
keys). The episode load that actually hit the rate limit runs from the
robosystems checkout and takes its SDK from *that* repo's lockfile, so
this alone does not fix the failing run — RoboFinSystems/robosystems#1318
is the half that does.

Lock only: the `>=1,<2` floor stays where it is.
@jfrench9
jfrench9 merged commit c798bee into main Aug 31, 2026
4 checks passed
@jfrench9
jfrench9 deleted the chore/client-1-13-1 branch September 1, 2026 22:33
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