Skip to content

fix: page advisor lint results#324

Open
ahfoysal wants to merge 1 commit into
supabase:mainfrom
ahfoysal:fix/advisors-pagination
Open

fix: page advisor lint results#324
ahfoysal wants to merge 1 commit into
supabase:mainfrom
ahfoysal:fix/advisors-pagination

Conversation

@ahfoysal

@ahfoysal ahfoysal commented Jul 7, 2026

Copy link
Copy Markdown

What

Adds optional limit and offset inputs to get_advisors so large advisor responses can be fetched in smaller chunks.

This keeps the existing behavior unchanged when pagination options are omitted, but when either option is provided the returned lints array is sliced and a small pagination object is included with total, offset, limit, and next_offset when another page exists.

Fixes #124.

Validation

  • CI=1 ./node_modules/.bin/vitest run --project unit src/server.test.ts from packages/mcp-server-supabase
  • ./node_modules/.bin/tsc --noEmit from packages/mcp-server-supabase
  • ./node_modules/.bin/biome check packages/mcp-server-supabase/src/tools/debugging-tools.ts packages/mcp-server-supabase/src/server.test.ts packages/mcp-server-supabase/test/mocks.ts
  • git diff --check

@ahfoysal ahfoysal requested a review from a team as a code owner July 7, 2026 11:17
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.

get_advisors tool returns responses exceeding token limits without pagination support

1 participant