@timber-js/app 0.2.0-alpha.7 → 0.2.0-alpha.9

package/src/server/rsc-entry/ssr-renderer.ts

@@ -156,16 +156,19 @@ export async function renderSsrResponse(opts: SsrRenderOptions): Promise<Respons
   try {
     const ssrResponse = await callSsr(ssrStream, navContext);
 
-    // Signal promotion: yield one microtask so async component rejections
-    // propagate to the RSC onError callback, then check if any signals
-    // were captured during rendering inside Suspense boundaries.
-    // The Response hasn't been sent yet — it's an unconsumed stream.
+    // Signal promotion: check if any signals were captured during rendering
+    // inside Suspense boundaries. If no signals are present yet, yield one
+    // microtask so async component rejections propagate to the RSC onError
+    // callback before we commit the response.
     //
-    // Uses queueMicrotask instead of setTimeout(0) to avoid yielding to
-    // the full event loop (timers phase). Microtask resolution happens
-    // within the same tick, eliminating per-request idle time under load.
+    // When signals are already captured (onSignal already fired), skip the
+    // yield entirely — react immediately. Uses queueMicrotask instead of
+    // setTimeout(0) for the fallback to avoid yielding to the full event
+    // loop (timers phase).
     // See design/05-streaming.md §"deferSuspenseFor and the Hold Window"
-    await new Promise<void>((r) => queueMicrotask(r));
+    if (!signals.redirectSignal && !signals.denySignal && !signals.renderError) {
+      await new Promise<void>((r) => queueMicrotask(r));
+    }
 
     const promoted = checkCapturedSignals(/* skipHandledDeny */ true);
     if (promoted) {

package/src/server/tracing.ts

@@ -252,6 +252,29 @@ export async function addSpanEvent(
   }
 }
 
+/**
+ * Fire-and-forget span event — no await, no microtask overhead.
+ *
+ * Used on the cache hot path where awaiting addSpanEvent creates an
+ * unnecessary microtask per cache operation. If OTEL is not loaded yet,
+ * the event is silently dropped (acceptable for diagnostics).
+ *
+ * See TIM-370 for perf motivation.
+ */
+export function addSpanEventSync(
+  name: string,
+  attributes?: Record<string, string | number | boolean>
+): void {
+  // Fast path: if OTEL API hasn't been loaded yet, skip entirely.
+  // _otelApi is undefined (not yet loaded), null (failed to load), or the module.
+  if (!_otelApi) return;
+
+  const activeSpan = _otelApi.trace.getActiveSpan();
+  if (activeSpan) {
+    activeSpan.addEvent(name, attributes);
+  }
+}
+
 /**
  * Try to extract the OTEL trace ID from the current active span context.
  * Returns undefined if OTEL is not active or no span exists.

package/src/server/waituntil-bridge.ts

@@ -18,7 +18,10 @@ import { waitUntilAls } from './als-registry.js';
  * Called by generated entry points (Nitro node-server/bun, Cloudflare)
  * to bind the platform's lifecycle extension for the request duration.
  */
-export function runWithWaitUntil<T>(waitUntilFn: (promise: Promise<unknown>) => void, fn: () => T): T {
+export function runWithWaitUntil<T>(
+  waitUntilFn: (promise: Promise<unknown>) => void,
+  fn: () => T
+): T {
   return waitUntilAls.run(waitUntilFn, fn);
 }