Fix CI: update test expectations, lint formatting, unexport AppAssetsConfig

Co-authored-by: Claude Code <claude-code@anthropic.com>

Author: isaacroldan

packages/app/src/cli/models/extensions/specification.ts

@@ -151,7 +151,7 @@ export interface ExtensionSpecification<TConfiguration extends BaseConfigType =
   appAssetsConfig?: (config: TConfiguration) => AppAssetsConfig | undefined
 }
 
-export interface AppAssetsConfig {
+interface AppAssetsConfig {
   /** The config key that points to the assets directory (e.g. 'admin.static_root') */
   assetsKey: string
   /** The assets directory relative to the extension directory */

packages/app/src/cli/services/build/steps/include-assets-step.test.ts

@@ -1082,10 +1082,7 @@ describe('executeIncludeAssetsStep', () => {
 
       // When / Then — overwrites existing manifest.json
       await expect(executeIncludeAssetsStep(step, contextWithConfig)).resolves.not.toThrow()
-      expect(fs.writeFile).toHaveBeenCalledWith(
-        '/test/output/manifest.json',
-        expect.any(String),
-      )
+      expect(fs.writeFile).toHaveBeenCalledWith('/test/output/manifest.json', expect.any(String))
     })
 
     test('writes an empty manifest when anchor resolves to a non-array value', async () => {

packages/app/src/cli/services/dev/extension.test.ts

@@ -69,6 +69,7 @@ describe('devUIExtensions()', () => {
       devOptions: {...options, websocketURL: 'wss://mock.url/extensions'},
       payloadStore: {mock: 'payload-store'},
       getExtensions: expect.any(Function),
+      getAppAssets: expect.any(Function),
     })
   })
 

packages/app/src/cli/services/dev/processes/dev-session/dev-session.ts

@@ -351,9 +351,11 @@ export class DevSession {
       .filter((event) => event.type !== 'deleted')
       .map((event) => event.extension.uid)
 
-    // PENDING: Clean up. This is a temporary workaround because `admin` is not compatible with inheritedUids in Core.
-    // It needs to be included in the manifest always.
-    updatedUids.push('admin')
+    // WORKAROUND. This is a temporary fix because `admin` is not compatible with inheritedUids in Core.
+    // It needs to be included in the manifest always if present in the app.
+    if (appEvent.app.allExtensions.some((ext) => ext.type === 'admin')) {
+      updatedUids.push('admin')
+    }
 
     const nonUpdatedUids = appEvent.app.allExtensions
       .filter((ext) => !updatedUids.includes(ext.uid))