Skip to content

#6 Add BulkPairings.export_games#4

Open
nstillman-te wants to merge 2 commits intomasterfrom
feature/chess-118-bulk-pairings-export-games
Open

#6 Add BulkPairings.export_games#4
nstillman-te wants to merge 2 commits intomasterfrom
feature/chess-118-bulk-pairings-export-games

Conversation

@nstillman-te
Copy link
Copy Markdown

Part of #6.

Add client.bulk_pairings.export_games for GET /api/bulk-pairing/{id}/games: export games of a bulk pairing as PGN or NDJSON stream. Supports moves, tags, clocks, evals, accuracy, opening, division, literate, pgnInJson. Uses existing Game model for NDJSON; requests-mock test (endpoint requires auth).

Checklist when adding a new endpoint
  • Added new endpoint to the README.rst
  • Ensured that my endpoint name does not repeat the name of the client (export_games, not bulk_pairing_export_games)
  • Typed the returned JSON using TypedDicts in berserk/types/ (NDJSON response uses existing Game model; no new types)
  • Written tests for GET endpoints not requiring authentification (endpoint requires auth; added requests-mock test for path and params)
  • Added the endpoint to CHANGELOG.rst in the To be released section

nstillman-te and others added 2 commits February 24, 2026 15:10
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
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