fix: address review feedback — 3 bugs

1. Warning body now says 'version' instead of 'release' on deploy-without-release path
2. Skip appInstallCount API call when prompt will be skipped (--force / --allow-updates+--allow-deletes)
3. Pass remoteApp.id (Management API app ID) instead of options.appId (API key) to appInstallCount
4. Add nullish coalesce for installCount in AppManagementClient (type safety)

Author: Alfonso Noriega

packages/app/src/cli/prompts/deploy-release.ts

@@ -162,7 +162,7 @@ async function deployConfirmationPrompt({
       confirmation: appTitle,
       ...(showInstallCountWarning
         ? {
-            body: `This release removes extensions and related data from ${installCount} app installations.\nUse caution as this may include production data on live stores.`,
+            body: `This ${release ? 'release' : 'version'} removes extensions and related data from ${installCount} app installations.\nUse caution as this may include production data on live stores.`,
           }
         : {}),
     })

packages/app/src/cli/services/context/identifiers.ts

@@ -58,11 +58,14 @@ export async function ensureDeploymentIdsPresence(options: EnsureDeploymentIdsPr
     activeAppVersion: options.activeAppVersion,
   })
 
+  const shouldFetchInstallCount =
+    extensionIdentifiersBreakdown.onlyRemote.length > 0 && !options.force && !(options.allowUpdates && options.allowDeletes)
+
   let installCount: number | undefined
-  if (extensionIdentifiersBreakdown.onlyRemote.length > 0) {
+  if (shouldFetchInstallCount) {
     try {
       installCount = await options.developerPlatformClient.appInstallCount({
-        id: options.appId,
+        id: options.remoteApp.id,
         apiKey: options.remoteApp.apiKey,
         organizationId: options.remoteApp.organizationId,
       })