@timber-js/app 0.2.0-alpha.42 → 0.2.0-alpha.43

package/dist/server/slot-resolver.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"slot-resolver.d.ts","sourceRoot":"","sources":["../../src/server/slot-resolver.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AAMH,OAAO,KAAK,EAAE,mBAAmB,EAAE,UAAU,EAAE,MAAM,eAAe,CAAC;AAErE,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,oBAAoB,CAAC;AAE9D,KAAK,eAAe,GAAG,CAAC,GAAG,IAAI,EAAE,OAAO,EAAE,KAAK,KAAK,CAAC,YAAY,CAAC;AAkHlE;;;;;;;;;;GAUG;AACH,wBAAsB,kBAAkB,CACtC,QAAQ,EAAE,mBAAmB,EAC7B,KAAK,EAAE,UAAU,EACjB,CAAC,EAAE,eAAe,EAClB,YAAY,CAAC,EAAE,mBAAmB,GACjC,OAAO,CAAC,KAAK,CAAC,YAAY,GAAG,IAAI,CAAC,CA0EpC"}
+{"version":3,"file":"slot-resolver.d.ts","sourceRoot":"","sources":["../../src/server/slot-resolver.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;GAgBG;AAMH,OAAO,KAAK,EAAE,mBAAmB,EAAE,UAAU,EAAE,MAAM,eAAe,CAAC;AAGrE,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,oBAAoB,CAAC;AAE9D,KAAK,eAAe,GAAG,CAAC,GAAG,IAAI,EAAE,OAAO,EAAE,KAAK,KAAK,CAAC,YAAY,CAAC;AAkHlE;;;;;;;;;;GAUG;AACH,wBAAsB,kBAAkB,CACtC,QAAQ,EAAE,mBAAmB,EAC7B,KAAK,EAAE,UAAU,EACjB,CAAC,EAAE,eAAe,EAClB,YAAY,CAAC,EAAE,mBAAmB,GACjC,OAAO,CAAC,KAAK,CAAC,YAAY,GAAG,IAAI,CAAC,CAyFpC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@timber-js/app",
-  "version": "0.2.0-alpha.42",
+  "version": "0.2.0-alpha.43",
   "description": "Vite-native React framework built for Servers and Serverless Platforms — correct HTTP semantics, real status codes, pages that work without JavaScript",
   "keywords": [
     "cloudflare-workers",

package/src/server/slot-resolver.ts

@@ -22,6 +22,7 @@ import { SlotAccessGate } from './access-gate.js';
 import { wrapSegmentWithErrorBoundaries } from './error-boundary-wrapper.js';
 import type { InterceptionContext, RouteMatch } from './pipeline.js';
 import { DenySignal } from './primitives.js';
+import { logRenderError } from './logger.js';
 import type { ManifestSegmentNode } from './route-matcher.js';
 
 type CreateElementFn = (...args: unknown[]) => React.ReactElement;
@@ -174,11 +175,19 @@ export async function resolveSlotElement(
       // §"Slot Access Failure = Graceful Degradation"
       const denyFallback = await renderDefaultFallback(slotNode, h);
 
-      // Wrap the slot page to catch DenySignal (from notFound() or deny())
-      // at the component level. This prevents the signal from reaching the
-      // RSC onError callback and being tracked as a page-level denial, which
-      // would cause the pipeline to replace the entire successful SSR response
-      // with a deny page. Instead, the slot gracefully degrades.
+      // Wrap the slot page to catch ALL errors at the component level.
+      // This prevents errors from leaving unresolved Flight rows in the
+      // RSC stream — when a slot component throws and the error propagates
+      // to React's Flight renderer, it may not emit a resolution row for
+      // the slot's lazy reference. The client's createFromReadableStream
+      // then throws "Connection closed" when the stream ends with pending
+      // references. By catching all errors here and returning a fallback,
+      // React sees a resolved component and emits a proper Flight row.
+      //
+      // DenySignal (from notFound() or deny()) returns the deny fallback.
+      // All other errors return the deny fallback or null — the slot
+      // gracefully degrades rather than breaking the entire page.
+      // See TIM-524.
       const SafeSlotPage = async (props: Record<string, unknown>) => {
         try {
           return await (SlotPage as (props: Record<string, unknown>) => unknown)(props);
@@ -186,7 +195,14 @@ export async function resolveSlotElement(
           if (error instanceof DenySignal) {
             return denyFallback;
           }
-          throw error;
+          // Log the error but don't re-throw — returning fallback ensures
+          // the Flight row is resolved and the page hydrates correctly.
+          logRenderError({
+            method: '',
+            path: '',
+            error,
+          });
+          return denyFallback;
         }
       };