@checkstack/backend 0.6.3 → 0.6.5

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # @checkstack/backend
 
+## 0.6.5
+
+### Patch Changes
+
+- 35a91e5: Fix truncated static file responses in production container
+
+  Hono's `c.body()` wasn't fully consuming Bun's `ReadableStream` from `file.stream()`, causing truncated responses (e.g. 129B instead of 1098B for the favicon). Switched to reading the file as `ArrayBuffer` before passing to `c.body()`, ensuring the full content is delivered.
+
+## 0.6.4
+
+### Patch Changes
+
+- a713e0f: Fix static file Content-Length header stripped by Hono middleware
+
+  Hono's CORS middleware wraps raw `Response` objects and strips Bun's auto-generated headers. Switched to using `c.body()` + `c.header()` so Content-Type and Content-Length survive the middleware pipeline. Extracted a shared `serveFile` helper for all static file routes.
+
 ## 0.6.3
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@checkstack/backend",
-  "version": "0.6.3",
+  "version": "0.6.5",
   "checkstack": {
     "type": "backend"
   },

package/src/index.ts

@@ -1,5 +1,5 @@
 import type { Server } from "bun";
-import { Hono } from "hono";
+import { type Context, Hono } from "hono";
 import { PluginManager } from "./plugin-manager";
 import { logger } from "hono/logger";
 import { migrate } from "drizzle-orm/node-postgres/migrator";
@@ -174,6 +174,13 @@ app.get("/.well-known/jwks.json", async (c) => {
 const frontendDistPath = process.env.CHECKSTACK_FRONTEND_DIST;
 if (frontendDistPath && fs.existsSync(frontendDistPath)) {
   rootLogger.info(`📦 Serving frontend from: ${frontendDistPath}`);
+  /** Serve a static file via Hono's context to preserve headers through middleware. */
+  const serveFile = async (c: Context, filePath: string) => {
+    const file = Bun.file(filePath);
+    const content = await file.arrayBuffer();
+    c.header("Content-Type", file.type);
+    return c.body(content);
+  };
 
   // Serve static assets (JS, CSS, images, etc.)
   app.get("/assets/*", async (c) => {
@@ -181,10 +188,7 @@ if (frontendDistPath && fs.existsSync(frontendDistPath)) {
     const filePath = path.join(frontendDistPath, "assets", assetPath);
 
     if (fs.existsSync(filePath)) {
-      const file = Bun.file(filePath);
-      return new Response(file, {
-        headers: { "Content-Type": file.type },
-      });
+      return serveFile(c, filePath);
     }
     return c.notFound();
   });
@@ -195,10 +199,7 @@ if (frontendDistPath && fs.existsSync(frontendDistPath)) {
     const filePath = path.join(frontendDistPath, "vendor", vendorPath);
 
     if (fs.existsSync(filePath)) {
-      const file = Bun.file(filePath);
-      return new Response(file, {
-        headers: { "Content-Type": file.type },
-      });
+      return serveFile(c, filePath);
     }
     return c.notFound();
   });
@@ -218,20 +219,14 @@ if (frontendDistPath && fs.existsSync(frontendDistPath)) {
       const staticFilePath = path.join(frontendDistPath, reqPath);
       // Only serve if it's a file (not a directory) and exists
       if (fs.existsSync(staticFilePath) && fs.statSync(staticFilePath).isFile()) {
-        const file = Bun.file(staticFilePath);
-        return new Response(file, {
-          headers: { "Content-Type": file.type },
-        });
+        return serveFile(c, staticFilePath);
       }
     }
 
     // SPA fallback: serve index.html for all remaining non-API routes
     const indexPath = path.join(frontendDistPath, "index.html");
     if (fs.existsSync(indexPath)) {
-      const file = Bun.file(indexPath);
-      return new Response(file, {
-        headers: { "Content-Type": "text/html" },
-      });
+      return serveFile(c, indexPath);
     }
     return c.notFound();
   });