@timber-js/app 0.1.44 → 0.1.46

package/dist/server/compress.d.ts

@@ -6,14 +6,19 @@
 export declare const COMPRESSIBLE_TYPES: Set<string>;
 /**
  * Parse Accept-Encoding and return the best supported encoding.
- * Prefers brotli (br) over gzip. Returns null if no supported encoding.
+ * Returns 'gzip' if the client accepts it, null otherwise.
  *
- * We always prefer brotli regardless of quality values because:
- * 1. Brotli achieves better compression ratios than gzip
- * 2. All modern browsers that send br in Accept-Encoding support it well
- * 3. Respecting q-values for br vs gzip adds complexity with no real benefit
+ * Brotli (br) is intentionally not handled at the application level.
+ * At the streaming-friendly quality levels (0–4), brotli's compression
+ * ratio advantage over gzip is marginal, and node:zlib's brotli transform
+ * buffers output internally — turning smooth streaming responses into
+ * large infrequent bursts. Brotli's real wins come from offline/static
+ * compression at higher quality levels (5–11), which CDNs and reverse
+ * proxies (Cloudflare, nginx, Caddy) apply on cached responses.
+ *
+ * See design/25-production-deployments.md.
  */
-export declare function negotiateEncoding(acceptEncoding: string): 'br' | 'gzip' | null;
+export declare function negotiateEncoding(acceptEncoding: string): 'gzip' | null;
 /**
  * Determine if a response should be compressed.
  *

package/dist/server/compress.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"compress.d.ts","sourceRoot":"","sources":["../../src/server/compress.ts"],"names":[],"mappings":"AAaA;;;;GAIG;AACH,eAAO,MAAM,kBAAkB,aAc7B,CAAC;AASH;;;;;;;;GAQG;AACH,wBAAgB,iBAAiB,CAAC,cAAc,EAAE,MAAM,GAAG,IAAI,GAAG,MAAM,GAAG,IAAI,CAU9E;AAID;;;;;;;;GAQG;AACH,wBAAgB,cAAc,CAAC,QAAQ,EAAE,QAAQ,GAAG,OAAO,CAmB1D;AAID;;;;;;;;GAQG;AACH,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,OAAO,EAAE,QAAQ,EAAE,QAAQ,GAAG,QAAQ,CAmC/E"}
+{"version":3,"file":"compress.d.ts","sourceRoot":"","sources":["../../src/server/compress.ts"],"names":[],"mappings":"AAYA;;;;GAIG;AACH,eAAO,MAAM,kBAAkB,aAc7B,CAAC;AASH;;;;;;;;;;;;;GAaG;AACH,wBAAgB,iBAAiB,CAAC,cAAc,EAAE,MAAM,GAAG,MAAM,GAAG,IAAI,CASvE;AAID;;;;;;;;GAQG;AACH,wBAAgB,cAAc,CAAC,QAAQ,EAAE,QAAQ,GAAG,OAAO,CAmB1D;AAID;;;;;;;;GAQG;AACH,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,OAAO,EAAE,QAAQ,EAAE,QAAQ,GAAG,QAAQ,CAiC/E"}

package/dist/server/index.js

@@ -787,11 +787,13 @@ function pathnameMatchesPattern(pathname, pattern) {
 */
 function createPipeline(config) {
 	const { proxy, matchRoute, render, earlyHints, stripTrailingSlash = true, slowRequestMs = 3e3, enableServerTiming = false, onPipelineError } = config;
+	let activeRequests = 0;
 	return async (req) => {
 		const url = new URL(req.url);
 		const method = req.method;
 		const path = url.pathname;
 		const startTime = performance.now();
+		activeRequests++;
 		return runWithTraceId(generateTraceId(), async () => {
 			return runWithRequestContext(req, async () => {
 				const runRequest = async () => {
@@ -815,22 +817,30 @@ function createPipeline(config) {
 								result = ensureMutableResponse(result);
 								result.headers.set("Server-Timing", serverTiming);
 							}
+						} else {
+							const totalMs = Math.round(performance.now() - startTime);
+							result = ensureMutableResponse(result);
+							result.headers.set("Server-Timing", `total;dur=${totalMs}`);
 						}
 						return result;
 					});
 					const durationMs = Math.round(performance.now() - startTime);
 					const status = response.status;
+					const concurrency = activeRequests;
+					activeRequests--;
 					logRequestCompleted({
 						method,
 						path,
 						status,
-						durationMs
+						durationMs,
+						concurrency
 					});
 					if (slowRequestMs > 0 && durationMs > slowRequestMs) logSlowRequest({
 						method,
 						path,
 						durationMs,
-						threshold: slowRequestMs
+						threshold: slowRequestMs,
+						concurrency
 					});
 					return response;
 				};
@@ -977,6 +987,14 @@ function createPipeline(config) {
 			markResponseFlushed();
 			return response;
 		} catch (error) {
+			if (error instanceof DenySignal) return new Response(null, { status: error.status });
+			if (error instanceof RedirectSignal) {
+				responseHeaders.set("Location", error.location);
+				return new Response(null, {
+					status: error.status,
+					headers: responseHeaders
+				});
+			}
 			logRenderError({
 				method,
 				path,