Merge pull request #5171 from Shopify/skip-uid-update-on-report-mode

[Fix] Skip UID update when in unsafe report mode

Author: amcaplan

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

@@ -1,5 +1,6 @@
 import {linkedAppContext} from './app-context.js'
 import {fetchSpecifications} from './generate/fetch-extension-specifications.js'
+import {addUidToTomlsIfNecessary} from './app/add-uid-to-extension-toml.js'
 import link from './app/config/link.js'
 import {appFromIdentifiers} from './context.js'
 
@@ -259,6 +260,7 @@ describe('linkedAppContext', () => {
       })
 
       // Then
+      expect(vi.mocked(addUidToTomlsIfNecessary)).not.toHaveBeenCalled()
       expect(loadSpy).toHaveBeenCalledWith(expect.any(Object), expect.objectContaining({mode: 'report'}))
       loadSpy.mockRestore()
     })
@@ -280,6 +282,7 @@ describe('linkedAppContext', () => {
       })
 
       // Then
+      expect(vi.mocked(addUidToTomlsIfNecessary)).toHaveBeenCalled()
       expect(loadSpy).toHaveBeenCalledWith(expect.any(Object), expect.objectContaining({mode: 'strict'}))
       loadSpy.mockRestore()
     })

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

@@ -101,7 +101,12 @@ export async function linkedAppContext({
   await logMetadata(remoteApp, organization, forceRelink)
 
   // Add UIDs to extension TOML files if using app-management.
-  await addUidToTomlsIfNecessary(localApp.allExtensions, developerPlatformClient)
+  // If in unsafe report mode, it is possible the UIDs are not loaded in memory
+  // even if they are present in the file, so we can't be sure whether or not
+  // it's necessary.
+  if (!unsafeReportMode) {
+    await addUidToTomlsIfNecessary(localApp.allExtensions, developerPlatformClient)
+  }
 
   return {app: localApp, remoteApp, developerPlatformClient, specifications, organization}
 }