@decocms/runtime 1.2.0 → 1.2.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@decocms/runtime",
-  "version": "1.2.0",
+  "version": "1.2.2",
   "type": "module",
   "scripts": {
     "check": "tsc --noEmit",
@@ -9,7 +9,7 @@
   "dependencies": {
     "@cloudflare/workers-types": "^4.20250617.0",
     "@decocms/bindings": "^1.0.7",
-    "@modelcontextprotocol/sdk": "1.25.1",
+    "@modelcontextprotocol/sdk": "1.25.2",
     "@ai-sdk/provider": "^3.0.0",
     "hono": "^4.10.7",
     "jose": "^6.0.11",

package/src/asset-server/index.test.ts

@@ -214,13 +214,21 @@ describe("createAssetHandler", () => {
   });
 
   describe("SPA fallback for routes with dots", () => {
+    const acceptsHtml = {
+      headers: {
+        accept: "text/html",
+      },
+    };
     test("serves index.html for /user/john.doe (non-existent path with dot)", async () => {
       const handler = createAssetHandler({
         env: "production",
         clientDir: tempDir,
       });
 
-      const request = new Request("http://localhost:3000/user/john.doe");
+      const request = new Request(
+        "http://localhost:3000/user/john.doe",
+        acceptsHtml,
+      );
       const result = await handler(request);
 
       expect(result).not.toBeNull();
@@ -235,7 +243,10 @@ describe("createAssetHandler", () => {
         clientDir: tempDir,
       });
 
-      const request = new Request("http://localhost:3000/page/v2.0");
+      const request = new Request(
+        "http://localhost:3000/page/v2.0",
+        acceptsHtml,
+      );
       const result = await handler(request);
 
       expect(result).not.toBeNull();
@@ -249,7 +260,10 @@ describe("createAssetHandler", () => {
         clientDir: tempDir,
       });
 
-      const request = new Request("http://localhost:3000/files/report.2024");
+      const request = new Request(
+        "http://localhost:3000/files/report.2024",
+        acceptsHtml,
+      );
       const result = await handler(request);
 
       expect(result).not.toBeNull();
@@ -263,7 +277,10 @@ describe("createAssetHandler", () => {
         clientDir: tempDir,
       });
 
-      const request = new Request("http://localhost:3000/assets/style.css");
+      const request = new Request(
+        "http://localhost:3000/assets/style.css",
+        acceptsHtml,
+      );
       const result = await handler(request);
 
       expect(result).not.toBeNull();
@@ -281,6 +298,7 @@ describe("createAssetHandler", () => {
       // This CSS file doesn't exist, so it should fall back to index.html
       const request = new Request(
         "http://localhost:3000/assets/nonexistent.css",
+        acceptsHtml,
       );
       const result = await handler(request);
 
@@ -295,7 +313,10 @@ describe("createAssetHandler", () => {
         clientDir: tempDir,
       });
 
-      const request = new Request("http://localhost:3000/dashboard");
+      const request = new Request(
+        "http://localhost:3000/dashboard",
+        acceptsHtml,
+      );
       const result = await handler(request);
 
       expect(result).not.toBeNull();

package/src/asset-server/index.ts

@@ -1,5 +1,5 @@
 import { devServerProxy } from "./dev-server-proxy";
-import { resolve, dirname } from "path";
+import { resolve, dirname, join, extname } from "path";
 
 export interface AssetServerConfig {
   /**
@@ -209,9 +209,21 @@ export function createAssetHandler(config: AssetServerConfig = {}) {
       return null; // Path traversal attempt blocked
     }
 
+    // Try to serve the index.html file relative to the requested file
+    const indexRelativeToFilePath = join(filePath, "index.html");
     // Try to serve the requested file, fall back to index.html for SPA routing
     const indexPath = resolve(clientDir, "index.html");
-    for (const pathToTry of [filePath, indexPath]) {
+
+    const acceptHeader = request.headers.get("accept");
+    const acceptsHtml =
+      acceptHeader?.includes("text/html") ||
+      (acceptHeader?.includes("*/*") &&
+        ["", ".html"].includes(extname(filePath)));
+
+    const fallbackPaths = acceptsHtml
+      ? [indexRelativeToFilePath, indexPath]
+      : [];
+    for (const pathToTry of [filePath, ...fallbackPaths]) {
       try {
         const file = Bun.file(pathToTry);
         if (await file.exists()) {

package/src/tools.ts

@@ -44,6 +44,7 @@ export interface Tool<
   TSchemaIn extends ZodTypeAny = ZodTypeAny,
   TSchemaOut extends ZodTypeAny | undefined = undefined,
 > {
+  _meta?: Record<string, unknown>;
   id: string;
   description?: string;
   inputSchema: TSchemaIn;
@@ -59,6 +60,7 @@ export interface Tool<
  * Streamable tool interface for tools that return Response streams.
  */
 export interface StreamableTool<TSchemaIn extends ZodSchema = ZodSchema> {
+  _meta?: Record<string, unknown>;
   id: string;
   inputSchema: TSchemaIn;
   streamable?: true;
@@ -71,6 +73,7 @@ export interface StreamableTool<TSchemaIn extends ZodSchema = ZodSchema> {
  * Uses a structural type with relaxed execute signature to allow tools with any schema.
  */
 export type CreatedTool = {
+  _meta?: Record<string, unknown>;
   id: string;
   description?: string;
   inputSchema: ZodTypeAny;
@@ -666,6 +669,7 @@ export const createMCPServer = <
         {
           _meta: {
             streamable: isStreamableTool(tool),
+            ...(tool._meta ?? {}),
           },
           description: tool.description,
           inputSchema:
@@ -681,13 +685,28 @@ export const createMCPServer = <
               : z.object({}).shape,
         },
         async (args) => {
-          let result = await tool.execute({
+          const result = await tool.execute({
             context: args,
             runtimeContext: createRuntimeContext(),
           });
 
+          // For streamable tools, the Response is handled at the transport layer
+          // Do NOT call result.bytes() - it buffers the entire response in memory
+          // causing massive memory leaks (2GB+ Uint8Array accumulation)
           if (isStreamableTool(tool) && result instanceof Response) {
-            result = { bytes: await result.bytes() };
+            return {
+              structuredContent: {
+                streamable: true,
+                status: result.status,
+                statusText: result.statusText,
+              },
+              content: [
+                {
+                  type: "text",
+                  text: `Streaming response: ${result.status} ${result.statusText}`,
+                },
+              ],
+            };
           }
           return {
             structuredContent: result as Record<string, unknown>,
@@ -830,7 +849,18 @@ export const createMCPServer = <
 
     await server.connect(transport);
 
-    return await transport.handleRequest(req);
+    try {
+      return await transport.handleRequest(req);
+    } finally {
+      // CRITICAL: Close transport to prevent memory leaks
+      // Without this, ReadableStream/WritableStream controllers accumulate
+      // causing thousands of stream objects to be retained in memory
+      try {
+        await transport.close?.();
+      } catch {
+        // Ignore close errors - transport may already be closed
+      }
+    }
   };
 
   const callTool: CallTool = async ({ toolCallId, toolCallInput }) => {