Skip to content

Raise orderBookOrders limit max to 500 - #198

Closed
hasanbal wants to merge 1 commit into
mainfrom
devin/1793984610-orderbookorders-limit-500
Closed

hasanbal wants to merge 1 commit into
mainfrom
devin/1793984610-orderbookorders-limit-500

Conversation

@hasanbal

@hasanbal hasanbal commented Nov 6, 2026

Copy link
Copy Markdown
Collaborator

Summary

Follows elliottech/zklighter-perps#5221, which raises the server-side range of /api/v1/orderBookOrders limit from [1:250] to [1:500].

The SDK checks this limit on the client side, so calls with limit > 250 fail pydantic validation before any request is sent:

- limit: Annotated[int, Field(le=250, strict=True, ge=1)]
+ limit: Annotated[int, Field(le=500, strict=True, ge=1)]

This PR updates order_book_orders, order_book_orders_with_http_info and order_book_orders_without_preload_content in lighter/api/order_api.py, and sets maximum to 500 in openapi.json. The edits are made by hand to match what the generator would output for the updated spec.

Merge only after #5221 is deployed: before that, the server still rejects limit > 250.

Link to Devin session: https://app.devin.ai/sessions/5aa2c74235e7430d8230f583c769fead
Open in Devin Desktop: https://app.devin.ai/desktop/session/5aa2c74235e7430d8230f583c769fead?variant=devin
Requested by: @hasanbal

Co-Authored-By: Hasan Bal <mailhasanbal@gmail.com>
@devin-ai-integration

Copy link
Copy Markdown
Contributor

I'll fix CI failures and address comments from users with write access that start with 'Devin'.

  • Disable automatic comment, CI, and merge conflict monitoring

@hasanbal hasanbal closed this Nov 6, 2026
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