@askrjs/node 0.2.0 → 0.2.2

package/README.md

@@ -94,7 +94,10 @@ import { serve } from "@askrjs/node";
 
 const running = await serve(app, {
   port: 3000,
-  assets: { root: "./dist/client" },
+  assets: {
+    root: "./dist/client",
+    exclude: (pathname) => pathname.startsWith("/files/"),
+  },
 });
 
 await running.close();
@@ -103,7 +106,9 @@ await running.close();
 `serve` handles static assets and closes both the HTTP server and the application during shutdown.
 When an asset root is configured, extension-bearing `GET` and `HEAD` paths are reserved for static
 files: missing files return `404` without falling through to application routing, source maps are not
-served, and resolved files must remain inside the configured root. Fingerprinted files under
+served, and resolved files must remain inside the configured root. Use `assets.exclude(pathname)` to
+bypass static interception for dotted application routes; the predicate receives the decoded URL
+pathname. Fingerprinted files under
 `/assets/` receive immutable caching; other files receive `no-cache`.
 Both `listen` and `serve` bind to `127.0.0.1` by default. A non-loopback
 `host` also requires `allowPublicBind: true` so public exposure is explicit. Public listeners should

package/dist/index.d.ts

@@ -58,6 +58,8 @@ interface ServeOptions extends ListenOptions {
   /** Serves static files from this directory before falling back to the application. */
   readonly assets?: {
     readonly root: string;
+    /** Returns true when an extension-bearing path must bypass static serving. */
+    readonly exclude?: (pathname: string) => boolean;
   };
   /** OS signals that trigger a graceful shutdown; pass `false` to disable automatic shutdown handling. */
   readonly signals?: false | readonly NodeJS.Signals[];

package/dist/index.js

@@ -6,6 +6,7 @@ import { createReadStream } from "node:fs";
 import { realpath, stat } from "node:fs/promises";
 import { extname, resolve, sep } from "node:path";
 import { pipeline } from "node:stream";
+import * as serverHttp from "@askrjs/server/http";
 //#region src/client-address.ts
 /**
 * Reserved request header containing the TCP peer address authenticated by the Node adapter.
@@ -623,6 +624,12 @@ const mimeTypes = {
 	".woff": "font/woff",
 	".woff2": "font/woff2"
 };
+const parseContentType = serverHttp.contentType ?? ((value) => {
+	if (!value) return void 0;
+	const separator = value.indexOf(";");
+	const type = (separator === -1 ? value : value.slice(0, separator)).trim();
+	return type ? type.toLowerCase() : void 0;
+});
 function isAssetPath(pathname) {
 	return extname(pathname) !== "";
 }
@@ -658,8 +665,8 @@ async function serve(app, options = {}) {
 	const preparedHandlerOptions = prepareNodeHandlerOptions(handlerOptions);
 	const applicationHandler = createNodeHandler({ async fetch(request, dispatchOptions) {
 		const result = await app.fetch(request, dispatchOptions);
-		const contentType = result.headers.get("content-type");
-		if (!result.headers.has("cache-control") && contentType !== null && /^text\/html(?:;|$)/iu.test(contentType)) {
+		const responseContentType = parseContentType(result.headers.get("content-type"));
+		if (!result.headers.has("cache-control") && responseContentType === "text/html") {
 			const headers = new Headers(result.headers);
 			headers.set("cache-control", "no-cache");
 			return new Response(result.body, {
@@ -683,7 +690,7 @@ async function serve(app, options = {}) {
 			return;
 		}
 		const method = request.method ?? "GET";
-		if ((method === "GET" || method === "HEAD") && isAssetPath(pathname)) {
+		if ((method === "GET" || method === "HEAD") && isAssetPath(pathname) && !options.assets?.exclude?.(pathname)) {
 			const extension = extname(pathname).toLowerCase();
 			const unresolvedCandidate = resolve(root, `.${pathname}`);
 			const inside = isWithinRoot(root, unresolvedCandidate);
@@ -700,12 +707,14 @@ async function serve(app, options = {}) {
 				file = void 0;
 			}
 			if (!candidate || !file?.isFile()) {
+				if (response.destroyed || response.writableEnded) return;
 				response.writeHead(404, {
 					"content-type": "text/plain; charset=utf-8",
 					"x-content-type-options": "nosniff"
 				}).end("Not Found");
 				return;
 			}
+			if (response.destroyed || response.writableEnded) return;
 			response.writeHead(200, {
 				"content-type": mimeTypes[extension] ?? "application/octet-stream",
 				"content-length": Number(file.size),
@@ -713,7 +722,15 @@ async function serve(app, options = {}) {
 				"x-content-type-options": "nosniff"
 			});
 			if (method === "HEAD") response.end();
-			else pipeline(createReadStream(candidate), response, () => void 0);
+			else {
+				const source = createReadStream(candidate);
+				const stopSource = () => source.destroy();
+				response.once("close", stopSource);
+				pipeline(source, response, (error) => {
+					response.off("close", stopSource);
+					if (error) source.destroy();
+				});
+			}
 			return;
 		}
 		applicationHandler(request, response);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@askrjs/node",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "description": "Node http adapter for @askrjs/server",
   "keywords": [
     "askr",
@@ -57,18 +57,18 @@
     "prepublishOnly": "npm run check"
   },
   "dependencies": {
-    "@askrjs/auth": ">=0.2.0 <0.3.0",
-    "@askrjs/server": ">=0.2.0 <0.3.0",
+    "@askrjs/auth": ">=0.2.1 <0.3.0",
+    "@askrjs/server": ">=0.2.1 <0.3.0",
     "@types/ws": "^8.18.1",
     "ws": "^8.21.3"
   },
   "devDependencies": {
-    "@askrjs/schema": ">=0.2.0 <0.3.0",
-    "@types/node": "^26.2.0",
-    "publint": "^0.3.23",
+    "@askrjs/schema": ">=0.2.1 <0.3.0",
+    "@types/node": "^26.3.0",
+    "publint": "^0.3.24",
     "typescript": "^7.0.2",
-    "vite-plus": "^0.2.8",
-    "vitest": "^4.1.10"
+    "vite-plus": "^0.3.0",
+    "vitest": "^4.1.11"
   },
   "engines": {
     "node": ">=24.0.0"