fix(mcp): align org team pagination with openapi (#107)

Author: dhairyashiil

apps/mcp-server/README.md

@@ -274,8 +274,8 @@ Each tool exposes MCP [tool annotations](https://modelcontextprotocol.io/specifi
 ### Organizations: Teams (2)
 | Tool | Title | Hint | Description |
 |---|---|---|---|
-| `get_org_teams` | List All Org Teams | Read | List all teams in an organization; requires org admin access |
-| `get_my_teams` | List My Teams | Read | List teams the authenticated user belongs to |
+| `get_org_teams` | List All Org Teams | Read | List all teams in an organization; requires org admin access; supports `take`/`skip` pagination (`take` max 250) |
+| `get_my_teams` | List My Teams | Read | List teams the authenticated user belongs to; supports `take`/`skip` pagination (`take` max 250) |
 
 ### Organizations: Routing Forms (2)
 | Tool | Title | Hint | Description |

apps/mcp-server/src/register-tools.ts

@@ -647,7 +647,7 @@ export function registerTools(server: McpServer): void {
     {
       title: "List All Org Teams",
       description:
-        "List all teams in an organization. Requires ORG_ADMIN role. Use get_me to obtain your organizationId. Returns team IDs, names, and slugs needed by other team-scoped tools. If this fails with 403, use get_my_teams instead.",
+        "List all teams in an organization. Supports take/skip pagination (take max 250). Requires ORG_ADMIN role. Use get_me to obtain your organizationId. Returns team IDs, names, and slugs needed by other team-scoped tools. If this fails with 403, use get_my_teams instead.",
       inputSchema: getOrgTeamsSchema,
       annotations: READ_ONLY,
     },
@@ -658,7 +658,7 @@ export function registerTools(server: McpServer): void {
     {
       title: "List My Teams",
       description:
-        "List teams the authenticated user belongs to. Works for any org member (org admins see all teams). Use get_me to obtain your organizationId. Returns team IDs, names, and slugs needed by other team-scoped tools.",
+        "List teams the authenticated user belongs to. Supports take/skip pagination (take max 250). Works for any org member (org admins see all teams). Use get_me to obtain your organizationId. Returns team IDs, names, and slugs needed by other team-scoped tools.",
       inputSchema: getMyTeamsSchema,
       annotations: READ_ONLY,
     },

apps/mcp-server/src/tools/organizations/teams.test.ts

@@ -17,6 +17,15 @@ describe("getOrgTeams", () => {
     expect(getOrgTeamsSchema).toBeDefined();
   });
 
+  it("enforces OpenAPI pagination bounds", () => {
+    expect(getOrgTeamsSchema.take.safeParse(1.5).success).toBe(false);
+    expect(getOrgTeamsSchema.take.safeParse(0).success).toBe(false);
+    expect(getOrgTeamsSchema.take.safeParse(250).success).toBe(true);
+    expect(getOrgTeamsSchema.take.safeParse(251).success).toBe(false);
+    expect(getOrgTeamsSchema.skip.safeParse(1.5).success).toBe(false);
+    expect(getOrgTeamsSchema.skip.safeParse(-1).success).toBe(false);
+  });
+
   it("calls the admin teams endpoint", async () => {
     mockCalApi.mockResolvedValueOnce({ status: "success", data: [{ id: 1, name: "Engineering" }] });
     const result = await getOrgTeams({ orgId: 1 });
@@ -49,6 +58,15 @@ describe("getMyTeams", () => {
     expect(getMyTeamsSchema).toBeDefined();
   });
 
+  it("enforces OpenAPI pagination bounds", () => {
+    expect(getMyTeamsSchema.take.safeParse(1.5).success).toBe(false);
+    expect(getMyTeamsSchema.take.safeParse(0).success).toBe(false);
+    expect(getMyTeamsSchema.take.safeParse(250).success).toBe(true);
+    expect(getMyTeamsSchema.take.safeParse(251).success).toBe(false);
+    expect(getMyTeamsSchema.skip.safeParse(1.5).success).toBe(false);
+    expect(getMyTeamsSchema.skip.safeParse(-1).success).toBe(false);
+  });
+
   it("calls the /me teams endpoint", async () => {
     mockCalApi.mockResolvedValueOnce({ status: "success", data: [{ id: 2, name: "Sales" }] });
     const result = await getMyTeams({ orgId: 1 });

apps/mcp-server/src/tools/organizations/teams.ts

@@ -7,8 +7,8 @@ export const getOrgTeamsSchema = {
     .number()
     .int()
     .describe("Organization ID. Use get_me to obtain your organizationId — never guess."),
-  take: z.number().optional().describe("Max results to return (default 250)"),
-  skip: z.number().optional().describe("Results to skip (offset)"),
+  take: z.number().int().min(1).max(250).optional().describe("Max results to return (1-250)"),
+  skip: z.number().int().min(0).optional().describe("Results to skip (offset, min 0)"),
 };
 
 export async function getOrgTeams(params: { orgId: number; take?: number; skip?: number }) {
@@ -28,8 +28,8 @@ export const getMyTeamsSchema = {
     .number()
     .int()
     .describe("Organization ID. Use get_me to obtain your organizationId — never guess."),
-  take: z.number().optional().describe("Max results to return (default 250)"),
-  skip: z.number().optional().describe("Results to skip (offset)"),
+  take: z.number().int().min(1).max(250).optional().describe("Max results to return (1-250)"),
+  skip: z.number().int().min(0).optional().describe("Results to skip (offset, min 0)"),
 };
 
 export async function getMyTeams(params: { orgId: number; take?: number; skip?: number }) {