fix: include UI apps build in CI release pipeline (#575)

The release workflow only ran `npm run build` (TypeScript), skipping the
UI apps build. This meant ui-apps/dist/ was missing from npm packages.

- Change `npm run build` to `npm run build:all` in build-and-verify and
  publish-npm jobs
- Copy ui-apps/dist into the npm publish directory
- Add ui-apps/dist/**/* to the published package files list
- Bump version to 2.34.2

Conceived by Romuald Czlonkowski - https://www.aiadvisors.pl/en

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

.github/workflows/release.yml

@@ -283,8 +283,8 @@ jobs:
       - name: Install dependencies
         run: npm ci
       
-      - name: Build project
-        run: npm run build
+      - name: Build project (server + UI apps)
+        run: npm run build:all
 
       # Database is already built and committed during development
       # Rebuilding here causes segfault due to memory pressure (exit code 139)
@@ -322,8 +322,8 @@ jobs:
       - name: Install dependencies
         run: npm ci
       
-      - name: Build project
-        run: npm run build
+      - name: Build project (server + UI apps)
+        run: npm run build:all
 
       # Database is already built and committed during development
       - name: Verify database exists
@@ -347,6 +347,8 @@ jobs:
           # Copy necessary files
           cp -r dist $PUBLISH_DIR/
           cp -r data $PUBLISH_DIR/
+          mkdir -p $PUBLISH_DIR/ui-apps
+          cp -r ui-apps/dist $PUBLISH_DIR/ui-apps/
           cp README.md $PUBLISH_DIR/
           cp LICENSE $PUBLISH_DIR/
           cp .env.example $PUBLISH_DIR/
@@ -377,7 +379,7 @@ jobs:
           pkg.license = 'MIT';
           pkg.bugs = { url: 'https://github.com/czlonkowski/n8n-mcp/issues' };
           pkg.homepage = 'https://github.com/czlonkowski/n8n-mcp#readme';
-          pkg.files = ['dist/**/*', 'data/nodes.db', '.env.example', 'README.md', 'LICENSE'];
+          pkg.files = ['dist/**/*', 'ui-apps/dist/**/*', 'data/nodes.db', '.env.example', 'README.md', 'LICENSE'];
           delete pkg.private;
           require('fs').writeFileSync('./package.json', JSON.stringify(pkg, null, 2));
           "

CHANGELOG.md

@@ -7,6 +7,32 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [2.34.2] - 2026-02-07
+
+### Fixed
+
+- **CI: UI apps missing from npm package**: Release pipeline only ran `npm run build` (TypeScript), so `ui-apps/dist/` was never built and excluded from published packages
+  - Changed build step to `npm run build:all` in `build-and-verify` and `publish-npm` jobs
+  - Added `ui-apps/dist/` to npm publish staging directory
+  - Added `ui-apps/dist/**/*` to published package files list
+
+Conceived by Romuald Czlonkowski - https://www.aiadvisors.pl/en
+
+## [2.34.1] - 2026-02-07
+
+### Changed
+
+- **MCP Apps: Align with official ext-apps spec** for Claude Desktop/web compatibility
+  - URI scheme changed from `n8n-mcp://ui/{id}` to `ui://n8n-mcp/{id}` per MCP ext-apps spec
+  - `_meta.ui.resourceUri` now set on tool definitions (`tools/list`) instead of tool call responses
+  - `UIMetadata.ui.app` renamed to `UIMetadata.ui.resourceUri`
+  - Added `_meta` field to `ToolDefinition` type
+  - Added `UIAppRegistry.injectToolMeta()` method for enriching tool definitions
+  - UI apps now use `@modelcontextprotocol/ext-apps` `App` class instead of `window.__MCP_DATA__`
+  - Updated `ReadResource` URI parser to match new `ui://` scheme
+
+Conceived by Romuald Czlonkowski - https://www.aiadvisors.pl/en
+
 ## [2.34.0] - 2026-02-07
 
 ### Added

package.json

@@ -1,6 +1,6 @@
 {
   "name": "n8n-mcp",
-  "version": "2.34.0",
+  "version": "2.34.2",
   "description": "Integration between n8n workflow automation and Model Context Protocol (MCP)",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",

src/mcp/server.ts

@@ -651,6 +651,7 @@ export class N8NDocumentationMCPServer {
         });
       });
       
+      UIAppRegistry.injectToolMeta(tools);
       return { tools };
     });
 
@@ -781,12 +782,6 @@ export class N8NDocumentationMCPServer {
           mcpResponse.structuredContent = structuredContent;
         }
 
-        // Inject UI app metadata if available
-        const uiApp = UIAppRegistry.getAppForTool(name);
-        if (uiApp && uiApp.html) {
-          mcpResponse._meta = { ui: { app: uiApp.config.uri } };
-        }
-
         return mcpResponse;
       } catch (error) {
         logger.error(`Error executing tool ${name}`, error);
@@ -857,8 +852,8 @@ export class N8NDocumentationMCPServer {
     // Handle ReadResource for UI apps
     this.server.setRequestHandler(ReadResourceRequestSchema, async (request) => {
       const uri = request.params.uri;
-      // Parse n8n-mcp://ui/{id} pattern
-      const match = uri.match(/^n8n-mcp:\/\/ui\/(.+)$/);
+      // Parse ui://n8n-mcp/{id} pattern
+      const match = uri.match(/^ui:\/\/n8n-mcp\/(.+)$/);
       if (!match) {
         throw new Error(`Unknown resource URI: ${uri}`);
       }

src/mcp/ui/app-configs.ts

@@ -5,7 +5,7 @@ export const UI_APP_CONFIGS: UIAppConfig[] = [
     id: 'operation-result',
     displayName: 'Operation Result',
     description: 'Visual summary of workflow operations (create, update, delete, test)',
-    uri: 'n8n-mcp://ui/operation-result',
+    uri: 'ui://n8n-mcp/operation-result',
     mimeType: 'text/html',
     toolPatterns: [
       'n8n_create_workflow',
@@ -21,7 +21,7 @@ export const UI_APP_CONFIGS: UIAppConfig[] = [
     id: 'validation-summary',
     displayName: 'Validation Summary',
     description: 'Visual summary of node and workflow validation results',
-    uri: 'n8n-mcp://ui/validation-summary',
+    uri: 'ui://n8n-mcp/validation-summary',
     mimeType: 'text/html',
     toolPatterns: [
       'validate_node',

src/mcp/ui/registry.ts

@@ -60,6 +60,21 @@ export class UIAppRegistry {
     return Array.from(this.entries.values());
   }
 
+  /**
+   * Enrich tool definitions with _meta.ui.resourceUri for tools that have
+   * a matching UI app. Per MCP ext-apps spec, this goes on the tool
+   * definition (tools/list), not the tool call response.
+   */
+  static injectToolMeta(tools: Array<{ name: string; [key: string]: any }>): void {
+    if (!this.loaded) return;
+    for (const tool of tools) {
+      const entry = this.toolIndex.get(tool.name);
+      if (entry && entry.html) {
+        tool._meta = { ui: { resourceUri: entry.config.uri } };
+      }
+    }
+  }
+
   /** Reset registry state. Intended for testing only. */
   static reset(): void {
     this.entries.clear();

src/mcp/ui/types.ts

@@ -13,7 +13,7 @@ export interface UIAppConfig {
 
 export interface UIMetadata {
   ui: {
-    app: string;
+    resourceUri: string;
   };
 }
 

src/types/index.ts

@@ -44,6 +44,11 @@ export interface ToolDefinition {
   };
   /** Tool behavior hints for AI assistants */
   annotations?: ToolAnnotations;
+  _meta?: {
+    ui?: {
+      resourceUri?: string;
+    };
+  };
 }
 
 export interface ResourceDefinition {

tests/unit/mcp/ui/app-configs.test.ts

@@ -27,9 +27,9 @@ describe('UI_APP_CONFIGS', () => {
     }
   });
 
-  it('should have URIs following n8n-mcp://ui/{id} pattern', () => {
+  it('should have URIs following ui://n8n-mcp/{id} pattern', () => {
     for (const config of UI_APP_CONFIGS) {
-      expect(config.uri).toBe(`n8n-mcp://ui/${config.id}`);
+      expect(config.uri).toBe(`ui://n8n-mcp/${config.id}`);
     }
   });
 
@@ -71,9 +71,9 @@ describe('UI_APP_CONFIGS', () => {
     }
   });
 
-  it('should have URIs that start with the n8n-mcp://ui/ scheme', () => {
+  it('should have URIs that start with the ui://n8n-mcp/ scheme', () => {
     for (const config of UI_APP_CONFIGS) {
-      expect(config.uri).toMatch(/^n8n-mcp:\/\/ui\//);
+      expect(config.uri).toMatch(/^ui:\/\/n8n-mcp\//);
     }
   });
 

tests/unit/mcp/ui/meta-injection.test.ts

@@ -11,7 +11,7 @@ import { existsSync, readFileSync } from 'fs';
 const mockExistsSync = vi.mocked(existsSync);
 const mockReadFileSync = vi.mocked(readFileSync);
 
-describe('UI Meta Injection Logic', () => {
+describe('UI Meta Injection on Tool Definitions', () => {
   beforeEach(() => {
     vi.clearAllMocks();
     UIAppRegistry.reset();
@@ -24,74 +24,69 @@ describe('UI Meta Injection Logic', () => {
       UIAppRegistry.load();
     });
 
-    it('should add _meta.ui for matching tools', () => {
-      const uiApp = UIAppRegistry.getAppForTool('n8n_create_workflow');
-      expect(uiApp).not.toBeNull();
-      expect(uiApp!.html).not.toBeNull();
+    it('should add _meta.ui.resourceUri to matching tool definitions', () => {
+      const tools: any[] = [
+        { name: 'n8n_create_workflow', description: 'Create workflow', inputSchema: { type: 'object', properties: {} } },
+      ];
 
-      // Simulate the injection logic from server.ts
-      const mcpResponse: any = {
-        content: [{ type: 'text', text: 'result' }],
-      };
+      UIAppRegistry.injectToolMeta(tools);
 
-      if (uiApp && uiApp.html) {
-        mcpResponse._meta = { ui: { app: uiApp.config.uri } };
-      }
-
-      expect(mcpResponse._meta).toBeDefined();
-      expect(mcpResponse._meta.ui.app).toBe('n8n-mcp://ui/operation-result');
+      expect(tools[0]._meta).toBeDefined();
+      expect(tools[0]._meta.ui.resourceUri).toBe('ui://n8n-mcp/operation-result');
     });
 
-    it('should add _meta.ui for validation tools', () => {
-      const uiApp = UIAppRegistry.getAppForTool('validate_workflow');
-      expect(uiApp).not.toBeNull();
+    it('should add _meta.ui.resourceUri to validation tool definitions', () => {
+      const tools: any[] = [
+        { name: 'validate_workflow', description: 'Validate', inputSchema: { type: 'object', properties: {} } },
+      ];
 
-      const mcpResponse: any = {
-        content: [{ type: 'text', text: 'validation result' }],
-      };
+      UIAppRegistry.injectToolMeta(tools);
 
-      if (uiApp && uiApp.html) {
-        mcpResponse._meta = { ui: { app: uiApp.config.uri } };
-      }
-
-      expect(mcpResponse._meta).toBeDefined();
-      expect(mcpResponse._meta.ui.app).toBe('n8n-mcp://ui/validation-summary');
+      expect(tools[0]._meta).toBeDefined();
+      expect(tools[0]._meta.ui.resourceUri).toBe('ui://n8n-mcp/validation-summary');
     });
 
-    it('should NOT add _meta.ui for non-matching tools', () => {
-      const uiApp = UIAppRegistry.getAppForTool('get_node_info');
-      expect(uiApp).toBeNull();
+    it('should NOT add _meta to non-matching tool definitions', () => {
+      const tools: any[] = [
+        { name: 'get_node_info', description: 'Get info', inputSchema: { type: 'object', properties: {} } },
+      ];
 
-      const mcpResponse: any = {
-        content: [{ type: 'text', text: 'node info' }],
-      };
+      UIAppRegistry.injectToolMeta(tools);
 
-      if (uiApp && uiApp.html) {
-        mcpResponse._meta = { ui: { app: uiApp.config.uri } };
-      }
-
-      expect(mcpResponse._meta).toBeUndefined();
+      expect(tools[0]._meta).toBeUndefined();
     });
 
-    it('should produce _meta with exact shape { ui: { app: string } }', () => {
-      const uiApp = UIAppRegistry.getAppForTool('n8n_create_workflow')!;
-      const meta = { ui: { app: uiApp.config.uri } };
+    it('should inject _meta on matching tools and skip non-matching in a mixed list', () => {
+      const tools: any[] = [
+        { name: 'n8n_create_workflow', description: 'Create', inputSchema: { type: 'object', properties: {} } },
+        { name: 'get_node_info', description: 'Info', inputSchema: { type: 'object', properties: {} } },
+        { name: 'validate_node', description: 'Validate', inputSchema: { type: 'object', properties: {} } },
+      ];
 
-      expect(meta).toEqual({
+      UIAppRegistry.injectToolMeta(tools);
+
+      expect(tools[0]._meta).toBeDefined();
+      expect(tools[0]._meta.ui.resourceUri).toBe('ui://n8n-mcp/operation-result');
+      expect(tools[1]._meta).toBeUndefined();
+      expect(tools[2]._meta).toBeDefined();
+      expect(tools[2]._meta.ui.resourceUri).toBe('ui://n8n-mcp/validation-summary');
+    });
+
+    it('should produce _meta with exact shape { ui: { resourceUri: string } }', () => {
+      const tools: any[] = [
+        { name: 'n8n_create_workflow', description: 'Create', inputSchema: { type: 'object', properties: {} } },
+      ];
+
+      UIAppRegistry.injectToolMeta(tools);
+
+      expect(tools[0]._meta).toEqual({
         ui: {
-          app: 'n8n-mcp://ui/operation-result',
+          resourceUri: 'ui://n8n-mcp/operation-result',
         },
       });
-      expect(Object.keys(meta)).toEqual(['ui']);
-      expect(Object.keys(meta.ui)).toEqual(['app']);
-      expect(typeof meta.ui.app).toBe('string');
-    });
-
-    it('should produce _meta.ui.app that matches the config uri', () => {
-      const uiApp = UIAppRegistry.getAppForTool('validate_node')!;
-      const meta = { ui: { app: uiApp.config.uri } };
-      expect(meta.ui.app).toBe(uiApp.config.uri);
-      expect(meta.ui.app).toBe('n8n-mcp://ui/validation-summary');
+      expect(Object.keys(tools[0]._meta)).toEqual(['ui']);
+      expect(Object.keys(tools[0]._meta.ui)).toEqual(['resourceUri']);
+      expect(typeof tools[0]._meta.ui.resourceUri).toBe('string');
     });
   });
 
@@ -101,114 +96,50 @@ describe('UI Meta Injection Logic', () => {
       UIAppRegistry.load();
     });
 
-    it('should NOT add _meta.ui even for matching tools', () => {
-      const uiApp = UIAppRegistry.getAppForTool('n8n_create_workflow');
-      expect(uiApp).not.toBeNull();
-      expect(uiApp!.html).toBeNull();
+    it('should NOT add _meta even for matching tools', () => {
+      const tools: any[] = [
+        { name: 'n8n_create_workflow', description: 'Create', inputSchema: { type: 'object', properties: {} } },
+      ];
 
-      const mcpResponse: any = {
-        content: [{ type: 'text', text: 'result' }],
-      };
+      UIAppRegistry.injectToolMeta(tools);
 
-      if (uiApp && uiApp.html) {
-        mcpResponse._meta = { ui: { app: uiApp.config.uri } };
-      }
-
-      expect(mcpResponse._meta).toBeUndefined();
+      expect(tools[0]._meta).toBeUndefined();
     });
 
-    it('should NOT add _meta.ui for validation tools without HTML', () => {
-      const uiApp = UIAppRegistry.getAppForTool('validate_node');
-      expect(uiApp).not.toBeNull();
-      expect(uiApp!.html).toBeNull();
+    it('should NOT add _meta for validation tools without HTML', () => {
+      const tools: any[] = [
+        { name: 'validate_node', description: 'Validate', inputSchema: { type: 'object', properties: {} } },
+      ];
 
-      const mcpResponse: any = {
-        content: [{ type: 'text', text: 'result' }],
-      };
+      UIAppRegistry.injectToolMeta(tools);
 
-      if (uiApp && uiApp.html) {
-        mcpResponse._meta = { ui: { app: uiApp.config.uri } };
-      }
-
-      expect(mcpResponse._meta).toBeUndefined();
+      expect(tools[0]._meta).toBeUndefined();
     });
   });
 
   describe('when registry has not been loaded at all', () => {
-    it('should NOT add _meta because getAppForTool returns null', () => {
-      // Registry never loaded - reset() was called in beforeEach
-      const uiApp = UIAppRegistry.getAppForTool('n8n_create_workflow');
-      expect(uiApp).toBeNull();
+    it('should NOT add _meta because registry is not loaded', () => {
+      const tools: any[] = [
+        { name: 'n8n_create_workflow', description: 'Create', inputSchema: { type: 'object', properties: {} } },
+      ];
 
-      const mcpResponse: any = {
-        content: [{ type: 'text', text: 'result' }],
-      };
+      UIAppRegistry.injectToolMeta(tools);
 
-      if (uiApp && uiApp.html) {
-        mcpResponse._meta = { ui: { app: uiApp.config.uri } };
-      }
-
-      expect(mcpResponse._meta).toBeUndefined();
+      expect(tools[0]._meta).toBeUndefined();
     });
   });
 
-  describe('coexistence with structuredContent', () => {
+  describe('empty tool list', () => {
     beforeEach(() => {
       mockExistsSync.mockReturnValue(true);
       mockReadFileSync.mockReturnValue('<html>ui</html>');
       UIAppRegistry.load();
     });
 
-    it('should coexist with structuredContent on the response', () => {
-      const uiApp = UIAppRegistry.getAppForTool('n8n_create_workflow');
-
-      const mcpResponse: any = {
-        content: [{ type: 'text', text: 'result' }],
-        structuredContent: { workflowId: '123', status: 'created' },
-      };
-
-      if (uiApp && uiApp.html) {
-        mcpResponse._meta = { ui: { app: uiApp.config.uri } };
-      }
-
-      expect(mcpResponse.structuredContent).toBeDefined();
-      expect(mcpResponse.structuredContent.workflowId).toBe('123');
-      expect(mcpResponse._meta).toBeDefined();
-      expect(mcpResponse._meta.ui.app).toBe('n8n-mcp://ui/operation-result');
-    });
-
-    it('should not overwrite existing _meta properties when merging', () => {
-      const uiApp = UIAppRegistry.getAppForTool('n8n_create_workflow');
-
-      const mcpResponse: any = {
-        content: [{ type: 'text', text: 'result' }],
-        _meta: { existingProp: 'value' },
-      };
-
-      if (uiApp && uiApp.html) {
-        mcpResponse._meta = { ...mcpResponse._meta, ui: { app: uiApp.config.uri } };
-      }
-
-      expect(mcpResponse._meta.existingProp).toBe('value');
-      expect(mcpResponse._meta.ui.app).toBe('n8n-mcp://ui/operation-result');
-    });
-
-    it('should work with responses that have both structuredContent and existing _meta', () => {
-      const uiApp = UIAppRegistry.getAppForTool('validate_workflow');
-
-      const mcpResponse: any = {
-        content: [{ type: 'text', text: 'validation ok' }],
-        structuredContent: { valid: true, errors: [] },
-        _meta: { timing: 42 },
-      };
-
-      if (uiApp && uiApp.html) {
-        mcpResponse._meta = { ...mcpResponse._meta, ui: { app: uiApp.config.uri } };
-      }
-
-      expect(mcpResponse.structuredContent.valid).toBe(true);
-      expect(mcpResponse._meta.timing).toBe(42);
-      expect(mcpResponse._meta.ui.app).toBe('n8n-mcp://ui/validation-summary');
+    it('should handle an empty tools array without error', () => {
+      const tools: any[] = [];
+      UIAppRegistry.injectToolMeta(tools);
+      expect(tools.length).toBe(0);
     });
   });
 });

tests/unit/mcp/ui/registry.test.ts

@@ -287,6 +287,75 @@ describe('UIAppRegistry', () => {
     });
   });
 
+  describe('injectToolMeta()', () => {
+    it('should not modify tools before load() is called', () => {
+      const tools: any[] = [
+        { name: 'n8n_create_workflow', description: 'Create', inputSchema: { type: 'object', properties: {} } },
+      ];
+      UIAppRegistry.injectToolMeta(tools);
+      expect(tools[0]._meta).toBeUndefined();
+    });
+
+    describe('after loading with HTML', () => {
+      beforeEach(() => {
+        mockExistsSync.mockReturnValue(true);
+        mockReadFileSync.mockReturnValue('<html>loaded</html>');
+        UIAppRegistry.load();
+      });
+
+      it('should set _meta.ui.resourceUri on matching operation tools', () => {
+        const tools: any[] = [
+          { name: 'n8n_create_workflow', description: 'Create', inputSchema: { type: 'object', properties: {} } },
+        ];
+        UIAppRegistry.injectToolMeta(tools);
+        expect(tools[0]._meta).toEqual({ ui: { resourceUri: 'ui://n8n-mcp/operation-result' } });
+      });
+
+      it('should set _meta.ui.resourceUri on matching validation tools', () => {
+        const tools: any[] = [
+          { name: 'validate_node', description: 'Validate', inputSchema: { type: 'object', properties: {} } },
+        ];
+        UIAppRegistry.injectToolMeta(tools);
+        expect(tools[0]._meta).toEqual({ ui: { resourceUri: 'ui://n8n-mcp/validation-summary' } });
+      });
+
+      it('should not set _meta on tools without a matching UI app', () => {
+        const tools: any[] = [
+          { name: 'search_nodes', description: 'Search', inputSchema: { type: 'object', properties: {} } },
+        ];
+        UIAppRegistry.injectToolMeta(tools);
+        expect(tools[0]._meta).toBeUndefined();
+      });
+
+      it('should handle a mix of matching and non-matching tools', () => {
+        const tools: any[] = [
+          { name: 'n8n_delete_workflow', description: 'Delete', inputSchema: { type: 'object', properties: {} } },
+          { name: 'get_node_essentials', description: 'Essentials', inputSchema: { type: 'object', properties: {} } },
+          { name: 'validate_workflow', description: 'Validate', inputSchema: { type: 'object', properties: {} } },
+        ];
+        UIAppRegistry.injectToolMeta(tools);
+        expect(tools[0]._meta?.ui?.resourceUri).toBe('ui://n8n-mcp/operation-result');
+        expect(tools[1]._meta).toBeUndefined();
+        expect(tools[2]._meta?.ui?.resourceUri).toBe('ui://n8n-mcp/validation-summary');
+      });
+    });
+
+    describe('after loading without HTML', () => {
+      beforeEach(() => {
+        mockExistsSync.mockReturnValue(false);
+        UIAppRegistry.load();
+      });
+
+      it('should not set _meta when HTML is not available', () => {
+        const tools: any[] = [
+          { name: 'n8n_create_workflow', description: 'Create', inputSchema: { type: 'object', properties: {} } },
+        ];
+        UIAppRegistry.injectToolMeta(tools);
+        expect(tools[0]._meta).toBeUndefined();
+      });
+    });
+  });
+
   describe('reset()', () => {
     it('should clear loaded state so getters return defaults', () => {
       mockExistsSync.mockReturnValue(true);

ui-apps/package.json

@@ -11,6 +11,7 @@
     "preview": "vite preview"
   },
   "dependencies": {
+    "@modelcontextprotocol/ext-apps": "^1.0.1",
     "react": "^19.0.0",
     "react-dom": "^19.0.0"
   },

ui-apps/src/shared/hooks/useToolData.ts

@@ -1,30 +1,34 @@
 import { useState, useEffect } from 'react';
-
-declare global {
-  interface Window {
-    __MCP_DATA__?: unknown;
-  }
-}
+import { App } from '@modelcontextprotocol/ext-apps';
 
 export function useToolData<T>(): T | null {
   const [data, setData] = useState<T | null>(null);
 
   useEffect(() => {
-    // Try window.__MCP_DATA__ first (injected by host)
-    if (window.__MCP_DATA__) {
-      setData(window.__MCP_DATA__ as T);
-      return;
-    }
+    const app = new App();
 
-    // Try embedded script tag
-    const scriptEl = document.getElementById('mcp-data');
-    if (scriptEl?.textContent) {
-      try {
-        setData(JSON.parse(scriptEl.textContent) as T);
-      } catch {
-        // Ignore parse errors
+    app.ontoolresult = (result: any) => {
+      // The host pushes tool result content; parse the first text item as JSON
+      if (result?.content) {
+        const textItem = Array.isArray(result.content)
+          ? result.content.find((c: any) => c.type === 'text')
+          : null;
+        if (textItem?.text) {
+          try {
+            setData(JSON.parse(textItem.text) as T);
+          } catch {
+            // Not JSON — use raw text as-is
+            setData(textItem.text as unknown as T);
+          }
+        }
       }
-    }
+    };
+
+    app.connect();
+
+    return () => {
+      app.close();
+    };
   }, []);
 
   return data;