fix(hub): always publish renderer manifest

Author: dvcolomban

packages/hub/src/node/__tests__/initiate.test.ts

@@ -7,6 +7,7 @@ import { createRpcClient } from 'devframe/rpc/client'
 import { createWsRpcChannel } from 'devframe/rpc/transports/ws-client'
 import { getPort } from 'get-port-please'
 import { describe, expect, it } from 'vitest'
+import { DOCK_RENDERERS_STATE_KEY } from '../../constants'
 import { DEVFRAMES_HUB_BASE, initHub } from '../initiate'
 
 function makeDist(html: string): string {
@@ -44,6 +45,23 @@ function connectWsClient(url: string) {
 }
 
 describe('initHub', () => {
+  it('publishes an empty renderer manifest when no renderers are registered', async () => {
+    expect.assertions(2)
+    const hub = initHub({ base: DEVFRAMES_HUB_BASE, auth: false })
+
+    try {
+      await hub.ready
+      const context = await hub.context
+      expect(context.rpc.sharedState.keys()).toContain(DOCK_RENDERERS_STATE_KEY)
+
+      const rendererManifest = await context.rpc.sharedState.get(DOCK_RENDERERS_STATE_KEY)
+      expect(rendererManifest.value()).toEqual({})
+    }
+    finally {
+      await hub.close()
+    }
+  })
+
   it('connectionMeta() before ready throws DF8003', () => {
     const hub = initHub({ base: DEVFRAMES_HUB_BASE, auth: false })
     expect(() => hub.connectionMeta()).toThrow(/DF8003|finished initializing/)

packages/hub/src/node/initiate.ts

@@ -546,24 +546,22 @@ export function initHub(options: InitHubOptions): HubInstance {
       // into the connection meta right after this `init` returns.
       await options.ui?.setup?.(ctx)
 
-      // Publish the renderer manifest — one `ClientScriptEntry` per dock
-      // `type`, `importFrom` base-absolute so it resolves to the served module
-      // from any page depth. Clients read it from shared state and import a
-      // module lazily the first time a dock of that type mounts.
-      if (rendererRegistrations.length > 0) {
-        const manifest: Record<string, ClientScriptEntry> = {}
-        for (const registration of rendererRegistrations) {
-          manifest[registration.type] = {
-            importFrom: joinURL(base, '__renderers', `${registration.type}.mjs`),
-            ...(registration.importName ? { importName: registration.importName } : {}),
-          }
+      // Publish the authoritative renderer manifest, including an empty one.
+      // Each `importFrom` is base-absolute so it resolves to the served module
+      // from any page depth. Clients import a module lazily the first time a
+      // dock of that type mounts.
+      const manifest: Record<string, ClientScriptEntry> = {}
+      for (const registration of rendererRegistrations) {
+        manifest[registration.type] = {
+          importFrom: joinURL(base, '__renderers', `${registration.type}.mjs`),
+          ...(registration.importName ? { importName: registration.importName } : {}),
         }
-        const manifestState = await ctx.rpc.sharedState.get<Record<string, ClientScriptEntry>>(
-          DOCK_RENDERERS_STATE_KEY,
-          { initialValue: {} },
-        )
-        manifestState.mutate(() => manifest)
       }
+      const manifestState = await ctx.rpc.sharedState.get<Record<string, ClientScriptEntry>>(
+        DOCK_RENDERERS_STATE_KEY,
+        { initialValue: {} },
+      )
+      manifestState.mutate(() => manifest)
 
       // Aggregate MCP — one Streamable-HTTP endpoint over the shared
       // context's whole registry (tool ids are namespaced per plugin, and the