fix: paginate platform branch list

Author: luanvdw

packages/cli/src/controllers/branch.ts

@@ -77,15 +77,33 @@ async function listBranches(
   projectId: string,
   signal: AbortSignal,
 ): Promise<RawBranchRecord[]> {
-  const { data, error, response } = await client.GET("/v1/projects/{projectId}/branches", {
-    params: { path: { projectId } },
-    signal,
-  });
-  if (error || !data) {
-    throw branchApiError("Failed to list branches", response, error);
+  const collected: RawBranchRecord[] = [];
+  let cursor: string | undefined;
+
+  // eslint-disable-next-line no-constant-condition
+  while (true) {
+    const query: Record<string, string | undefined> = {};
+    if (cursor !== undefined) {
+      query.cursor = cursor;
+    }
+
+    const { data, error, response } = await client.GET("/v1/projects/{projectId}/branches", {
+      params: { path: { projectId }, query },
+      signal,
+    });
+    if (error || !data) {
+      throw branchApiError("Failed to list branches", response, error);
+    }
+
+    collected.push(...data.data as RawBranchRecord[]);
+
+    if (!data.pagination.hasMore || !data.pagination.nextCursor) {
+      break;
+    }
+    cursor = data.pagination.nextCursor;
   }
 
-  return data.data as RawBranchRecord[];
+  return collected;
 }
 
 function toBranchSummary(branch: RawBranchRecord): BranchSummary {

packages/cli/tests/branch-controller.test.ts

@@ -14,7 +14,7 @@ afterEach(() => {
 
 function createMockClient() {
   return {
-    GET: vi.fn().mockImplementation((pathName: string) => {
+    GET: vi.fn().mockImplementation((pathName: string, request?: { params?: { query?: { cursor?: string } } }) => {
       if (pathName === "/v1/projects") {
         return {
           data: {
@@ -32,13 +32,25 @@ function createMockClient() {
       }
 
       if (pathName === "/v1/projects/{projectId}/branches") {
+        const cursor = request?.params?.query?.cursor;
+        if (cursor === "cursor_2") {
+          return {
+            data: {
+              data: [
+                { id: "br_feature", gitName: "feature/auth", role: "preview" },
+              ],
+              pagination: { hasMore: false, nextCursor: null },
+            },
+            response: { status: 200 },
+          };
+        }
+
         return {
           data: {
             data: [
               { id: "br_main", gitName: "main", role: "production" },
-              { id: "br_feature", gitName: "feature/auth", role: "preview" },
             ],
-            pagination: { hasMore: false, nextCursor: null },
+            pagination: { hasMore: true, nextCursor: "cursor_2" },
           },
           response: { status: 200 },
         };
@@ -93,7 +105,13 @@ describe("branch controller", () => {
     expect(client.GET).toHaveBeenCalledWith(
       "/v1/projects/{projectId}/branches",
       expect.objectContaining({
-        params: { path: { projectId: "proj_123" } },
+        params: { path: { projectId: "proj_123" }, query: {} },
+      }),
+    );
+    expect(client.GET).toHaveBeenCalledWith(
+      "/v1/projects/{projectId}/branches",
+      expect.objectContaining({
+        params: { path: { projectId: "proj_123" }, query: { cursor: "cursor_2" } },
       }),
     );
     expect(result).toEqual({