refactor(mcp): trim resource template listing and redundant tests

Undefined fields drop out of the JSON result, so the listings pass them as is. The devframe and hub initiate tests that only repeated what the HTTP and in-memory tests already catch are removed.

Author: antfubot

packages/agentic/src/mcp/build-server.ts

@@ -361,7 +361,10 @@ function registerResourceHandlers(
       }
     }
 
-    resources.push(...await listTemplateResources(ctx, templates))
+    for (const { template } of templates) {
+      for (const entry of await template.list?.(ctx) ?? [])
+        resources.push({ ...entry, mimeType: entry.mimeType ?? template.mimeType })
+    }
 
     return { resources }
   })
@@ -435,34 +438,10 @@ interface ResourceListing {
   mimeType?: string
 }
 
-/** The entries the `list` of each template returns, in template order, for `resources/list`. */
-async function listTemplateResources(ctx: DevframeNodeContext, templates: CompiledResourceTemplate[]): Promise<ResourceListing[]> {
-  const resources: ResourceListing[] = []
-  for (const { template } of templates) {
-    for (const entry of await template.list?.(ctx) ?? []) {
-      const mimeType = entry.mimeType ?? template.mimeType
-      resources.push({
-        uri: entry.uri,
-        name: entry.name,
-        ...(entry.title !== undefined ? { title: entry.title } : {}),
-        ...(entry.description !== undefined ? { description: entry.description } : {}),
-        ...(mimeType !== undefined ? { mimeType } : {}),
-      })
-    }
-  }
-  return resources
-}
-
 /** Serve `resources/templates/list`: one entry per template, in the given order. */
 function registerResourceTemplateHandlers(server: Server, templates: CompiledResourceTemplate[]): void {
   server.setRequestHandler('resources/templates/list', async () => ({
-    resourceTemplates: templates.map(({ template }) => ({
-      uriTemplate: template.uriTemplate,
-      name: template.name,
-      ...(template.title !== undefined ? { title: template.title } : {}),
-      ...(template.description !== undefined ? { description: template.description } : {}),
-      ...(template.mimeType !== undefined ? { mimeType: template.mimeType } : {}),
-    })),
+    resourceTemplates: templates.map(({ template: { uriTemplate, name, title, description, mimeType } }) => ({ uriTemplate, name, title, description, mimeType })),
   }))
 }
 

packages/devframe/src/adapters/__tests__/initiate.test.ts

@@ -393,46 +393,6 @@ describe('adapters/handler', () => {
     }
   })
 
-  it('mcp: resource templates reach the route through resources/templates/list and resources/read', async () => {
-    const wsPort = await getPort({ port: 18148, host: '127.0.0.1' })
-    const unsubscribe = vi.fn()
-    const devtools = initDevframe(defineTestDef('handler-mcp-templates'), {
-      base: '/__handler-mcp-templates/',
-      auth: false,
-      mcp: {
-        resourceTemplates: [{
-          uriTemplate: 'handler://page/{slug}',
-          name: 'page',
-          mimeType: 'text/plain',
-          read: params => ({ text: `Page ${params.slug}.` }),
-        }],
-        onResourcesChanged: () => unsubscribe,
-      },
-      ws: { port: wsPort },
-    })
-
-    const call = (method: string, params?: Record<string, unknown>) => devtools.handler(new Request('http://localhost:3000/__handler-mcp-templates/__mcp', {
-      method: 'POST',
-      headers: {
-        'content-type': 'application/json',
-        'accept': 'application/json, text/event-stream',
-        'origin': 'http://localhost:3000',
-      },
-      body: JSON.stringify({ jsonrpc: '2.0', id: 1, method, params }),
-    })).then(res => res.text())
-
-    try {
-      await devtools.ready
-      expect(await call('resources/templates/list')).toContain('"uriTemplate":"handler://page/{slug}"')
-      expect(await call('resources/read', { uri: 'handler://page/home' })).toContain('Page home.')
-      expect(await call('resources/read', { uri: 'handler://page/a/b' })).toContain('unknown resource URI')
-    }
-    finally {
-      await devtools.close()
-    }
-    expect(unsubscribe).toHaveBeenCalledTimes(1)
-  })
-
   // The `'auto'` default: an omitted `mcp` mounts the route exactly when
   // `setup()` left a non-empty agent surface.
   function defineAgentTestDef(id: string) {

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

@@ -482,19 +482,6 @@ describe('initHub', () => {
   })
 
   describe('aggregate MCP resource templates', () => {
-    it('answers resources/templates/list with method not found by default', async () => {
-      const wsPort = await getPort({ port: 18241, host: '127.0.0.1' })
-      const hub = initHub({ base: DEVFRAMES_HUB_BASE, auth: false, host: '127.0.0.1', ws: { port: wsPort }, mcp: true, devframes: [makeFrame('alpha')] })
-
-      try {
-        await hub.ready
-        expect(await callMcp(hub, 'resources/templates/list')).toContain('-32601')
-      }
-      finally {
-        await hub.close()
-      }
-    })
-
     it('serves mcp.resourceTemplates and wires mcp.onResourcesChanged to the hub context', async () => {
       const wsPort = await getPort({ port: 18242, host: '127.0.0.1' })
       const unsubscribe = vi.fn()