Show list of added UIDs on the first deploy

Author: isaacroldan

packages/app/src/cli/services/context/id-matching.test.ts

@@ -4,9 +4,11 @@ import {RemoteSource} from './identifiers.js'
 import {ExtensionInstance} from '../../models/extensions/extension-instance.js'
 import {testDeveloperPlatformClient, testFunctionExtension, testUIExtension} from '../../models/app/app.test-data.js'
 import {describe, expect, vi, test, beforeAll} from 'vitest'
+import {outputInfo} from '@shopify/cli-kit/node/output'
 
 vi.mock('../dev/fetch')
 vi.mock('../dev/create-extension')
+vi.mock('@shopify/cli-kit/node/output')
 
 const REGISTRATION_A: RemoteSource = {
   uuid: 'UUID_A',
@@ -796,3 +798,35 @@ describe('automaticMatchmaking: with Atomic Deployments enabled', () => {
     expect(got).toEqual(expected)
   })
 })
+
+describe('outputAddedIDs', () => {
+  test('prints extension IDs when extensions are matched without UID', async () => {
+    // Clear any previous mock calls
+    vi.mocked(outputInfo).mockClear()
+
+    // Extension B has a valid UID
+    REGISTRATION_B.id = EXTENSION_B.uid
+
+    // When: Extensions are matched by UUID (not by UID)
+    await automaticMatchmaking(
+      [EXTENSION_A, EXTENSION_B, EXTENSION_C],
+      [REGISTRATION_A, REGISTRATION_B, REGISTRATION_C],
+      {
+        'extension-a': 'UUID_A',
+        'extension-b': 'UUID_B',
+        'extension-c': 'UUID_C',
+      },
+      testDeveloperPlatformClient({supportsAtomicDeployments: true}),
+    )
+
+    // Then: outputInfo should be called with the expected messages
+    expect(outputInfo).toHaveBeenCalledWith('Generating extension IDs\n')
+    expect(outputInfo).toHaveBeenCalledWith(expect.stringContaining('\x1B[36mextension-a\x1B[39m | Added ID: UUID_A'))
+    expect(outputInfo).not.toHaveBeenCalledWith(expect.stringContaining('Added ID: UUID_B'))
+    expect(outputInfo).toHaveBeenCalledWith(expect.stringContaining('\x1B[35mextension-c\x1B[39m | Added ID: UUID_C'))
+    expect(outputInfo).toHaveBeenCalledWith('\n')
+
+    // Verify it was called 4 times total (header + 2 extensions + footer)
+    expect(outputInfo).toHaveBeenCalledTimes(4)
+  })
+})

packages/app/src/cli/services/context/id-matching.ts

@@ -5,6 +5,8 @@ import {groupBy, partition} from '@shopify/cli-kit/common/collection'
 import {uniqBy, difference} from '@shopify/cli-kit/common/array'
 import {pickBy} from '@shopify/cli-kit/common/object'
 import {slugify} from '@shopify/cli-kit/common/string'
+import {outputInfo} from '@shopify/cli-kit/node/output'
+import colors from '@shopify/cli-kit/node/colors'
 
 export interface LocalRemoteSource {
   local: LocalSource
@@ -79,32 +81,53 @@ function matchByUIDandUUID(
   toConfirm: {local: LocalSource; remote: RemoteSource}[]
   toManualMatch: {local: LocalSource[]; remote: RemoteSource[]}
 } {
-  const matched: IdentifiersExtensions = {}
+  const matchedByUID: IdentifiersExtensions = {}
   const pendingLocal: LocalSource[] = []
+  const matchedByUUID: IdentifiersExtensions = {}
 
   // First, try to match by UID, then by UUID.
   local.forEach((localSource) => {
     const matchByUID = remote.find((remoteSource) => remoteSource.id === localSource.uid)
     const matchByUUID = remote.find((remoteSource) => remoteSource.uuid === ids[localSource.localIdentifier])
 
     if (matchByUID) {
-      matched[localSource.localIdentifier] = matchByUID.id
+      matchedByUID[localSource.localIdentifier] = matchByUID.id
     } else if (matchByUUID) {
-      matched[localSource.localIdentifier] = matchByUUID.uuid
+      matchedByUUID[localSource.localIdentifier] = matchByUUID.uuid
     } else {
       pendingLocal.push(localSource)
     }
   })
 
   const pendingRemote = remote.filter(
     (remoteSource) =>
-      !Object.values(matched).includes(remoteSource.uuid) && !Object.values(matched).includes(remoteSource.id),
+      !Object.values(matchedByUUID).includes(remoteSource.uuid) &&
+      !Object.values(matchedByUID).includes(remoteSource.id),
   )
 
   // Then, try to match by name and type as a last resort.
   const {matched: matchedByName, toCreate, toConfirm, toManualMatch} = matchByNameAndType(pendingLocal, pendingRemote)
 
-  return {matched: {...matched, ...matchedByName}, toCreate, toConfirm, toManualMatch}
+  // List of modules that were matched using anything other than the UID, meaning that they are being migrated to dev dash
+  const totalMatchedWithoutUID = {...matchedByUUID, ...matchedByName}
+  const localMatchedWithoutUID = local.filter((localSource) => totalMatchedWithoutUID[localSource.localIdentifier])
+
+  outputAddedIDs(localMatchedWithoutUID)
+
+  return {matched: {...matchedByUID, ...totalMatchedWithoutUID}, toCreate, toConfirm, toManualMatch}
+}
+
+function outputAddedIDs(localMatchedWithoutUID: LocalSource[]) {
+  if (localMatchedWithoutUID.length === 0) return
+  const colorList = [colors.cyan, colors.magenta, colors.blue, colors.green, colors.yellow, colors.red]
+
+  const maxHandleLength = localMatchedWithoutUID.reduce((max, local) => Math.max(max, local.handle.length), 0)
+  outputInfo('Generating extension IDs\n')
+  localMatchedWithoutUID.forEach((local, index) => {
+    const color = colorList[index % colorList.length] ?? colors.white
+    outputInfo(`${color(local.handle.padStart(maxHandleLength))} | Added ID: ${local.uid}`)
+  })
+  outputInfo('\n')
 }
 
 /**