Add failsafe and improve loging on unexepcete select app error

Author: alfonso-noriega

packages/app/src/cli/prompts/dev.ts

@@ -64,7 +64,16 @@ export async function selectAppPrompt(
       }
     },
   })
-  return currentAppChoices.find((app) => app.apiKey === apiKey)!
+
+  const appChoice = currentAppChoices.find((app) => app.apiKey === apiKey)!
+
+  if (!appChoice) {
+    throw new Error(
+      `Unable to select an app: the selection prompt was interrupted multiple times./n
+      Api key ${apiKey} was selected but not found in ${currentAppChoices.map((app) => app.apiKey).join(', ')}`,
+    )
+  }
+  return appChoice
 }
 
 interface SelectStorePromptOptions {

packages/app/src/cli/services/dev/select-app.ts

@@ -4,7 +4,10 @@ import {Organization, MinimalOrganizationApp, OrganizationApp} from '../../model
 import {getCachedCommandInfo, setCachedCommandTomlPreference} from '../local-storage.js'
 import {CreateAppOptions, DeveloperPlatformClient} from '../../utilities/developer-platform-client.js'
 import {AppConfigurationFileName} from '../../models/app/loader.js'
-import {BugError} from '@shopify/cli-kit/node/error'
+import {AbortError, BugError} from '@shopify/cli-kit/node/error'
+import {outputInfo, outputDebug} from '@shopify/cli-kit/node/output'
+
+const MAX_PROMPT_RETRIES = 2
 
 /**
  * Select an app from env, list or create a new one:
@@ -30,25 +33,89 @@ export async function selectOrCreateApp(
     const name = await appNamePrompt(options.name)
     return developerPlatformClient.createApp(org, {...options, name})
   } else {
-    const app = await selectAppPrompt(searchForAppsByNameFactory(developerPlatformClient, org.id), apps, hasMorePages, {
-      directory: options.directory,
-    })
+    // Capture app selection context
+    const cachedData = getCachedCommandInfo()
+    const tomls = (cachedData?.tomls as {[key: string]: AppConfigurationFileName}) ?? {}
+
+    for (let attempt = 0; attempt < MAX_PROMPT_RETRIES; attempt++) {
+      try {
+        // eslint-disable-next-line no-await-in-loop
+        const app = await selectAppPrompt(
+          searchForAppsByNameFactory(developerPlatformClient, org.id),
+          apps,
+          hasMorePages,
+          {directory: options.directory},
+        )
+
+        const selectedToml = tomls[app.apiKey]
+        if (selectedToml) setCachedCommandTomlPreference(selectedToml)
+
+        // eslint-disable-next-line no-await-in-loop
+        const fullSelectedApp = await developerPlatformClient.appFromIdentifiers(app.apiKey)
+
+        if (!fullSelectedApp) {
+          throw new BugError(
+            `Unable to fetch app ${app.apiKey} from Shopify`,
+            'Try running `shopify app config link` to connect to an app you have access to.',
+          )
+        }
 
-    const data = getCachedCommandInfo()
-    const tomls = (data?.tomls as {[key: string]: AppConfigurationFileName}) ?? {}
-    const selectedToml = tomls[app.apiKey]
+        return fullSelectedApp
+      } catch (error) {
+        // Don't retry BugError - those indicate actual bugs, not transient issues
+        if (error instanceof BugError) {
+          throw error
+        }
 
-    if (selectedToml) setCachedCommandTomlPreference(selectedToml)
+        const errorObj = error as Error
 
-    const fullSelectedApp = await developerPlatformClient.appFromIdentifiers(app.apiKey)
+        // Log each attempt for observability
+        outputDebug(`App selection attempt ${attempt + 1}/${MAX_PROMPT_RETRIES} failed: ${errorObj.message}`)
 
-    if (!fullSelectedApp) {
-      // This is unlikely, and a bug. But we still want a nice user facing message plus appropriate context logged.
-      throw new BugError(
-        `Unable to fetch app ${app.apiKey} from Shopify`,
-        'Try running `shopify app config link` to connect to an app you have access to.',
-      )
+        // If we have retries left, inform user and retry
+        if (attempt < MAX_PROMPT_RETRIES - 1) {
+          outputInfo('App selection was interrupted. Retrying...')
+        } else {
+          const tryMessage = [
+            'This may happen if:',
+            '  • The process received a signal (SIGINT, SIGTERM) or was terminated',
+            '  • Running in an unstable environment (container restart, resource limits)',
+            '  • Network interruption during app fetching',
+            '',
+            'Try running the command again. If the issue persists:',
+            '  • Check system resources and stability',
+            '  • Try running outside of containers/WSL if applicable',
+            '  • Report this issue with the error detailsmad a verbose log',
+          ]
+            .filter(Boolean)
+            .join('\n')
+
+          throw new BugError(errorObj.message, tryMessage)
+        }
+      }
     }
-    return fullSelectedApp
+
+    // User-facing error message with key diagnostic info
+    const errorMessage = [
+      'Unable to select an app: the selection prompt was interrupted multiple times.',
+      '',
+      `Available apps: ${apps.length}`,
+    ].join('\n')
+
+    const tryMessage = [
+      'This may happen if:',
+      '  • The process received a signal (SIGINT, SIGTERM) or was terminated',
+      '  • Running in an unstable environment (container restart, resource limits)',
+      '  • Network interruption during app fetching',
+      '',
+      'Try running the command again. If the issue persists:',
+      '  • Check system resources and stability',
+      '  • Try running outside of containers/WSL if applicable',
+      '  • Report this issue with the error details',
+    ]
+      .filter(Boolean)
+      .join('\n')
+
+    throw new BugError(errorMessage, tryMessage)
   }
 }

packages/cli-kit/src/public/node/ui.tsx

@@ -426,7 +426,7 @@ export async function renderAutocompletePrompt<T>(
   }
 
   return runWithTimer('cmd_all_timing_prompts_ms')(async () => {
-    let selectedValue: T
+    let selectedValue: T | undefined
     await render(
       <AutocompletePrompt
         {...newProps}
@@ -439,7 +439,14 @@ export async function renderAutocompletePrompt<T>(
         exitOnCtrlC: false,
       },
     )
-    return selectedValue!
+
+    if (selectedValue === undefined) {
+      throw new Error(
+        'Prompt was interrupted before a selection was made. This can happen if the process received a signal, was terminated, or the prompt was aborted.',
+      )
+    }
+
+    return selectedValue
   })
 }