Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Comments

Close side panel

Pagination#21

Merged
jspahrsummers merged 5 commits intomainmodelcontextprotocol/modelcontextprotocol:mainfrom
justin/paginationmodelcontextprotocol/modelcontextprotocol:justin/paginationCopy head branch name to clipboard
Oct 7, 2024
Merged

Pagination#21
jspahrsummers merged 5 commits intomainmodelcontextprotocol/modelcontextprotocol:mainfrom
justin/paginationmodelcontextprotocol/modelcontextprotocol:justin/paginationCopy head branch name to clipboard

Conversation

@jspahrsummers
Copy link
Member

@jspahrsummers jspahrsummers commented Oct 7, 2024

Adds optional cursor-based pagination to all list-type requests.

I've also split out the listing of resources and resource templates, which makes the interfaces simpler. Although this is a breaking change, I don't think anything actually exists that depends on this right now.

In keeping with our MCP design philosophy, pagination logic is entirely up to the server (including whether it exists at all): the server returns however many results it wants to, and an optional cursor if the client can request more.

Resolves #6.

@jspahrsummers jspahrsummers requested review from a team, ashwin-ant, dsp-ant and k6l3 October 7, 2024 14:51
@udaygajavalli
Copy link

Say I made an API call which is paginated at my server end upon receiving the tool/call. Does it support pagination?

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.

Support for Pagination

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.