Skip to content

fix(sdks): extract shared pagination config from Python-only overlay - #750

Merged
leggetter merged 1 commit into
mainfrom
fix/split-pagination-overlay
Mar 13, 2026
Merged

fix(sdks): extract shared pagination config from Python-only overlay#750
leggetter merged 1 commit into
mainfrom
fix/split-pagination-overlay

Conversation

@alexluong

Copy link
Copy Markdown
Collaborator

Summary

The pagination cursor config (x-speakeasy-pagination) was bundled inside the Python-specific overlay, which only applies to the Python SDK source. After #744 split the Speakeasy sources by language, Go and TS SDKs lost auto-pagination support.

  • Extract pagination config into a new shared pagination-config-overlay.yaml applied to all three sources
  • Keep only Python builtin name overrides (dirdirection, nextnext_cursor, prevprev_cursor) in python-pagination-fixes-overlay.yaml
  • Both overlays pass speakeasy overlay validate

Test plan

  • speakeasy overlay validate passes for both overlays
  • speakeasy overlay apply confirms 4 pagination configs applied
  • Re-run SDK generation after merge to verify pagination restored in Go/TS

🤖 Generated with Claude Code

The pagination cursor config (x-speakeasy-pagination) was bundled in the
Python pagination fixes overlay, which is only applied to the Python SDK
source. This meant Go and TS SDKs lost auto-pagination support after
#744 split the sources.

Extract the pagination config into a shared overlay applied to all three
sources, keeping only the Python builtin name overrides (dir, next, prev)
in the Python-specific overlay.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@vercel

vercel Bot commented Mar 13, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
outpost-docs Ready Ready Preview, Comment Mar 13, 2026 11:25am
outpost-website Ready Ready Preview, Comment Mar 13, 2026 11:25am

Request Review

@vercel
vercel Bot temporarily deployed to Preview – outpost-website March 13, 2026 11:24 Inactive
@vercel
vercel Bot temporarily deployed to Preview – outpost-docs March 13, 2026 11:25 Inactive
@leggetter

Copy link
Copy Markdown
Collaborator

Good spot, @alexluong 👍

@leggetter
leggetter merged commit 7d1de86 into main Mar 13, 2026
5 checks passed
@leggetter
leggetter deleted the fix/split-pagination-overlay branch March 13, 2026 11:49
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.

2 participants