feat: add limit/pagination to scout_opportunities endpoint #148

Closed
opened 2026-03-24 03:40:08 +00:00 by cal · 1 comment
Owner

The GET /api/v2/scout_opportunities endpoint returns all records with no limit parameter. Add an optional limit: int query param (default 100, max 500).

Found during smoke test development — endpoints without limits cause excessive data transfer.

The `GET /api/v2/scout_opportunities` endpoint returns all records with no `limit` parameter. Add an optional `limit: int` query param (default 100, max 500). Found during smoke test development — endpoints without limits cause excessive data transfer.
Claude added the
ai-working
label 2026-03-24 05:31:08 +00:00
Claude removed the
ai-working
label 2026-03-24 05:32:47 +00:00
Collaborator

PR #154 opened: #154

Added limit: Optional[int] = 100 to GET /api/v2/scout_opportunities, capped at 500 via max(0, min(limit, 500)). Also removed a pre-existing unused fn import (required to pass ruff pre-commit hook).

PR #154 opened: https://git.manticorum.com/cal/paper-dynasty-database/pulls/154 Added `limit: Optional[int] = 100` to `GET /api/v2/scout_opportunities`, capped at 500 via `max(0, min(limit, 500))`. Also removed a pre-existing unused `fn` import (required to pass ruff pre-commit hook).
Claude added the
ai-pr-opened
label 2026-03-24 05:32:52 +00:00
cal closed this issue 2026-03-24 12:07:36 +00:00
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cal/paper-dynasty-database#148
No description provided.