amend

Author: dmerand

packages/store/src/cli/commands/store/list.ts

@@ -43,7 +43,8 @@ Use \`--organization-id\` to scope the organization listing to a single organiza
       required: false,
     }),
     'organization-id': Flags.string({
-      description: 'List stores for a single organization, by ID. Only valid with `--from auto` or `--from organization`.',
+      description:
+        'List stores for a single organization, by ID. Only valid with `--from auto` or `--from organization`.',
       env: 'SHOPIFY_FLAG_ORGANIZATION_ID',
       required: false,
     }),

packages/store/src/cli/services/store/list/bp-source.test.ts

@@ -143,65 +143,90 @@ describe('listBusinessPlatformStores', () => {
     })
   })
 
-  test('sorts accessible shops by creation date, most recent first', async () => {
+  test('fetches a single bounded page per organization and orders newest first', async () => {
     vi.mocked(ensureAuthenticatedBusinessPlatform).mockResolvedValue('bp-token')
     vi.mocked(fetchOrganizationsWithAccessInfo).mockResolvedValue({
       organizations: [{id: '1234', businessName: 'Acme'}],
       currentUserResolved: true,
     })
-    vi.mocked(businessPlatformOrganizationsRequestDoc)
-      .mockResolvedValueOnce({
-        organization: {
-          id: '1234',
-          name: 'Acme',
-          accessibleShops: {
-            edges: [
-              {
-                node: {
-                  id: 'gid://shopify/Shop/1',
-                  externalId: '1',
-                  name: 'Older Shop',
-                  storeType: 'PRODUCTION',
-                  primaryDomain: 'older.myshopify.com',
-                  url: null,
-                  createdAt: '2025-01-01T00:00:00Z',
-                },
+    vi.mocked(businessPlatformOrganizationsRequestDoc).mockResolvedValue({
+      organization: {
+        id: '1234',
+        name: 'Acme',
+        accessibleShops: {
+          edges: [
+            {
+              node: {
+                id: 'gid://shopify/Shop/2',
+                externalId: '2',
+                name: 'Newer Shop',
+                storeType: 'DEVELOPMENT',
+                primaryDomain: 'newer.myshopify.com',
+                url: null,
+                createdAt: '2026-05-01T00:00:00Z',
               },
-            ],
-            pageInfo: {hasNextPage: true, endCursor: 'cursor-1'},
-          },
-        },
-      } as any)
-      .mockResolvedValueOnce({
-        organization: {
-          id: '1234',
-          name: 'Acme',
-          accessibleShops: {
-            edges: [
-              {
-                node: {
-                  id: 'gid://shopify/Shop/2',
-                  externalId: '2',
-                  name: 'Newer Shop',
-                  storeType: 'DEVELOPMENT',
-                  primaryDomain: 'newer.myshopify.com',
-                  url: null,
-                  createdAt: '2026-05-01T00:00:00Z',
-                },
+            },
+            {
+              node: {
+                id: 'gid://shopify/Shop/1',
+                externalId: '1',
+                name: 'Older Shop',
+                storeType: 'PRODUCTION',
+                primaryDomain: 'older.myshopify.com',
+                url: null,
+                createdAt: '2025-01-01T00:00:00Z',
               },
-            ],
-            pageInfo: {hasNextPage: false, endCursor: null},
-          },
+            },
+          ],
+          pageInfo: {hasNextPage: false},
         },
-      } as any)
+      },
+    } as any)
 
     const result = await listBusinessPlatformStores()
 
     expect(result.status === 'success' ? result.entries.map((entry) => entry.store) : []).toEqual([
       'newer.myshopify.com',
       'older.myshopify.com',
     ])
-    expect(businessPlatformOrganizationsRequestDoc).toHaveBeenCalledTimes(2)
+    expect(businessPlatformOrganizationsRequestDoc).toHaveBeenCalledTimes(1)
+    expect(businessPlatformOrganizationsRequestDoc).toHaveBeenCalledWith(
+      expect.objectContaining({variables: {first: 250}}),
+    )
+  })
+
+  test('reports hasMore when an organization has more stores than the fetched page', async () => {
+    vi.mocked(ensureAuthenticatedBusinessPlatform).mockResolvedValue('bp-token')
+    vi.mocked(fetchOrganizationsWithAccessInfo).mockResolvedValue({
+      organizations: [{id: '1234', businessName: 'Acme'}],
+      currentUserResolved: true,
+    })
+    vi.mocked(businessPlatformOrganizationsRequestDoc).mockResolvedValue({
+      organization: {
+        id: '1234',
+        name: 'Acme',
+        accessibleShops: {
+          edges: [
+            {
+              node: {
+                id: 'gid://shopify/Shop/1',
+                externalId: '1',
+                name: 'Acme Dev',
+                storeType: 'DEVELOPMENT',
+                primaryDomain: 'acme.myshopify.com',
+                url: null,
+                createdAt: '2026-05-01T00:00:00Z',
+              },
+            },
+          ],
+          pageInfo: {hasNextPage: true},
+        },
+      },
+    } as any)
+
+    const result = await listBusinessPlatformStores()
+
+    expect(result.status === 'success' ? result.hasMore : undefined).toBe(true)
   })
 
   test('returns successful organizations and records failed ones', async () => {
@@ -256,8 +281,7 @@ describe('listBusinessPlatformStores', () => {
         },
       ],
       currentUserEmail: 'merchant@example.com',
-      warning:
-        'Skipped 1 organization whose store listing failed. Showing results from 1 successful organization.',
+      warning: 'Skipped 1 organization whose store listing failed. Showing results from 1 successful organization.',
       failedOrganizations: [
         {
           organizationId: '5678',