feat: add limit/pagination to mlbplayers endpoint #141

New Issue

cal · 2026-03-24T03:40:00Z

cal commented

2026-03-24 03:40:00 +00:00

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

Found during smoke test development — this endpoint returns the entire table (~5k rows) causing slow responses.

The `GET /api/v2/mlbplayers` endpoint returns all 4,900+ records with no `limit` parameter. Add an optional `limit: int` query param (default 100, max 500). Found during smoke test development — this endpoint returns the entire table (~5k rows) causing slow responses.

Claude added the

ai-working

label 2026-03-24 09:31:08 +00:00

cal referenced this issue from a commit

2026-03-24 09:32:20 +00:00

feat: add limit/pagination to mlbplayers endpoint (#141)

Claude referenced a pull request that will close this issue

2026-03-24 09:32:26 +00:00

feat: add limit/pagination to mlbplayers endpoint (#141) #162

Claude removed the

ai-working

label 2026-03-24 09:32:31 +00:00

Claude commented

2026-03-24 09:32:33 +00:00

PR #162 opened: #162

Added limit: int = 100 to GET /api/v2/mlbplayers, capped via max(0, min(limit, 500)). Applied after all filters so both CSV and JSON paths respect the limit.

PR #162 opened: https://git.manticorum.com/cal/paper-dynasty-database/pulls/162 Added `limit: int = 100` to `GET /api/v2/mlbplayers`, capped via `max(0, min(limit, 500))`. Applied after all filters so both CSV and JSON paths respect the limit.

Claude added the

ai-pr-opened

label 2026-03-24 09:32:36 +00:00

cal closed this issue

2026-03-24 12:06:50 +00:00

cal referenced this issue from a commit

2026-03-24 12:06:51 +00:00

Merge pull request 'feat: add limit/pagination to mlbplayers endpoint (#141)' (#162) from issue/141-feat-add-limit-pagination-to-mlbplayers-endpoint into main

Sign in to join this conversation.