fix: update mock-connector for new types

Author: danielroe

cli/src/mock-app.ts

@@ -230,13 +230,18 @@ function createMockConnectorApp(stateManager: MockConnectorStateManager) {
     requireAuth(event)
 
     const body = await event.req.json().catch(() => ({}))
-    const otp = (body as { otp?: string })?.otp
+    const { otp } = body as { otp?: string; interactive?: boolean; openUrls?: boolean }
 
-    const { results, otpRequired } = stateManager.executeOperations({ otp })
+    const { results, otpRequired, authFailure, urls } = stateManager.executeOperations({ otp })
 
     return {
       success: true,
-      data: { results, otpRequired },
+      data: {
+        results,
+        otpRequired,
+        authFailure,
+        urls,
+      },
     } satisfies ApiResponse<ConnectorEndpoints['POST /execute']['data']>
   })
 

cli/src/mock-state.ts

@@ -58,6 +58,8 @@ export interface ExecuteOptions {
 export interface ExecuteResult {
   results: Array<{ id: string; result: OperationResult }>
   otpRequired?: boolean
+  authFailure?: boolean
+  urls?: string[]
 }
 
 export function createMockConnectorState(config: MockConnectorConfig): MockConnectorStateData {
@@ -305,7 +307,15 @@ export class MockConnectorStateManager {
       }
     }
 
-    return { results }
+    const authFailure = results.some(r => r.result.authFailure)
+    const allUrls = results.flatMap(r => r.result.urls ?? [])
+    const urls = [...new Set(allUrls)]
+
+    return {
+      results,
+      authFailure: authFailure || undefined,
+      urls: urls.length > 0 ? urls : undefined,
+    }
   }
 
   /** Apply side effects of a completed operation. Param keys match schemas.ts. */

cli/src/types.ts

@@ -98,6 +98,7 @@ export interface ExecuteResponseData {
   results: Array<{ id: string; result: OperationResult }>
   otpRequired?: boolean
   authFailure?: boolean
+  urls?: string[]
 }
 
 /** POST /approve-all response data */
@@ -133,7 +134,10 @@ export interface ConnectorEndpoints {
   'POST /approve': { body: never; data: PendingOperation }
   'POST /approve-all': { body: never; data: ApproveAllResponseData }
   'POST /retry': { body: never; data: PendingOperation }
-  'POST /execute': { body: { otp?: string }; data: ExecuteResponseData }
+  'POST /execute': {
+    body: { otp?: string; interactive?: boolean; openUrls?: boolean }
+    data: ExecuteResponseData
+  }
   'GET /org/:org/users': { body: never; data: Record<string, OrgRole> }
   'GET /org/:org/teams': { body: never; data: string[] }
   'GET /team/:scopeTeam/users': { body: never; data: string[] }

test/nuxt/components/HeaderConnectorModal.spec.ts

@@ -99,6 +99,10 @@ function resetMockState() {
     error: null,
     lastExecutionTime: null,
   }
+  mockSettings.value.connector = {
+    webAuth: false,
+    autoOpenURL: false,
+  }
 }
 
 function simulateConnect() {
@@ -112,6 +116,27 @@ vi.mock('~/composables/useConnector', () => ({
   useConnector: createMockUseConnector,
 }))
 
+const mockSettings = ref({
+  relativeDates: false,
+  includeTypesInInstall: true,
+  accentColorId: null,
+  hidePlatformPackages: true,
+  selectedLocale: null,
+  preferredBackgroundTheme: null,
+  searchProvider: 'npm',
+  connector: {
+    webAuth: false,
+    autoOpenURL: false,
+  },
+  sidebar: {
+    collapsed: [],
+  },
+})
+
+vi.mock('~/composables/useSettings', () => ({
+  useSettings: () => ({ settings: mockSettings }),
+}))
+
 vi.mock('~/composables/useSelectedPackageManager', () => ({
   useSelectedPackageManager: () => ref('npm'),
 }))