@async/framework 0.11.6 → 0.11.8

package/browser.ts

@@ -2379,9 +2379,10 @@ const __componentModule = (() => {
 })();
 
 const __serverModule = (() => {
-  const serverEnvelopeKeys = new Set(["value", "signals", "boundary", "html", "redirect", "error"]);
-  const appliedServerResult = Symbol.for("@async/framework.appliedServerResult");
-  const appliedServerValues = new WeakSet();
+  const serverEnvelopeKind = Symbol.for("@async/framework.serverResult");
+  const serverEnvelopeWireKey = "__async_server_result__";
+  const serverEnvelopeWireVersion = 1;
+  const serverResultInvocation = Symbol("@async/framework.serverResultInvocation");
 
   function createServerProxy({
     endpoint = "/__async/server",
@@ -2401,7 +2402,7 @@ const __serverModule = (() => {
 
     async function run(id, args = [], context = {}) {
       assertServerId(id);
-      const runContext = { ...defaults, ...context };
+      const runContext = createServerResultContext({ ...defaults, ...context });
       const body = {
         args,
         input: context.input ?? defaultInput(runContext),
@@ -2424,9 +2425,7 @@ const __serverModule = (() => {
         throw new Error(`Server function "${id}" failed with ${response.status}.`);
       }
 
-      const result = await readServerResponse(id, response);
-      await applyServerResult(result, runContext);
-      return markAppliedServerValue(unwrapServerResult(result));
+      return consumeServerResult(await readServerResponse(id, response), runContext);
     }
 
     return createServerNamespace(run, {
@@ -2461,12 +2460,13 @@ const __serverModule = (() => {
     if (!isServerEnvelope(result)) {
       return result;
     }
-    if (result[appliedServerResult] || appliedServerValues.has(result)) {
+    const invocation = getServerResultInvocation(context);
+    if (invocation.applied.has(result)) {
       return result;
     }
+    invocation.applied.add(result);
 
     if (result.error) {
-      markAppliedServerResult(result);
       throw toError(result.error);
     }
 
@@ -2488,32 +2488,35 @@ const __serverModule = (() => {
       await context.router?.navigate?.(result.redirect);
     }
 
-    markAppliedServerResult(result);
-
     return result;
   }
 
+  async function consumeServerResult(result, context = {}) {
+    await applyServerResult(result, context);
+    return unwrapServerResult(result);
+  }
+
   function unwrapServerResult(result) {
-    if (isServerEnvelope(result) && Object.hasOwn(result, "value")) {
-      return result.value;
+    if (isServerEnvelope(result)) {
+      return Object.hasOwn(result, "value") ? result.value : undefined;
     }
     return result;
   }
 
-  function markAppliedServerValue(value) {
-    if (value && typeof value === "object") {
-      appliedServerValues.add(value);
-    }
-    return value;
+  function createServerResultContext(context = {}) {
+    const invocation = context[serverResultInvocation] ?? {
+      applied: new WeakSet()
+    };
+    return {
+      ...context,
+      [serverResultInvocation]: invocation
+    };
   }
 
-  function markAppliedServerResult(result) {
-    Object.defineProperty(result, appliedServerResult, {
-      configurable: true,
-      enumerable: false,
-      value: true
-    });
-    return result;
+  function getServerResultInvocation(context = {}) {
+    return context[serverResultInvocation] ?? {
+      applied: new WeakSet()
+    };
   }
 
   function defaultInput(context = {}) {
@@ -2548,9 +2551,7 @@ const __serverModule = (() => {
           throw new Error("Server namespace is not directly callable.");
         }
         const context = contextProvider() ?? {};
-        const result = await run(parts.join("."), args, context);
-        await applyServerResult(result, context);
-        return unwrapServerResult(result);
+        return run(parts.join("."), args, context);
       };
 
       const proxy = new Proxy(callable, {
@@ -2607,7 +2608,7 @@ const __serverModule = (() => {
 
   async function readServerResponse(id, response) {
     if (response.status === 204) {
-      return { value: undefined };
+      return undefined;
     }
     const type = response.headers.get("content-type") ?? "";
     if (type.includes("application/json")) {
@@ -2625,7 +2626,7 @@ const __serverModule = (() => {
     if (typeof response.text !== "function") {
       throw new Error(`Server function "${id}" transport returned an invalid response: missing text().`);
     }
-    return { value: await response.text() };
+    return response.text();
   }
 
   function snapshotSignalPaths(paths = [], signals) {
@@ -2770,7 +2771,8 @@ const __serverModule = (() => {
     if (!value || typeof value !== "object" || Array.isArray(value)) {
       return false;
     }
-    return Object.keys(value).some((key) => serverEnvelopeKeys.has(key));
+    return value[serverEnvelopeKind] === true
+      || value[serverEnvelopeWireKey] === serverEnvelopeWireVersion;
   }
 
   function toError(value) {
@@ -2792,11 +2794,11 @@ const __serverModule = (() => {
       throw new TypeError("Server function id must be a non-empty string.");
     }
   }
-  return { createServerProxy, resolveServerCommandArguments, applyServerResult, unwrapServerResult, defaultInput, createServerNamespace, createSignalReader, assertServerId };
+  return { createServerProxy, resolveServerCommandArguments, applyServerResult, consumeServerResult, unwrapServerResult, createServerResultContext, defaultInput, createServerNamespace, createSignalReader, assertServerId };
 })();
 
 const __handlersModule = (() => {
-  const { applyServerResult, defaultInput, resolveServerCommandArguments, unwrapServerResult } = __serverModule;
+  const { defaultInput, resolveServerCommandArguments } = __serverModule;
   const { attachRegistryInspection, createRegistryStore } = __registryStoreModule;
   const { createLazyRegistry, isLazyDescriptor } = __lazyRegistryModule;
   const builtInTokens = new Set(["prevent", "preventDefault", "stopPropagation", "stopImmediatePropagation"]);
@@ -2894,8 +2896,7 @@ const __handlersModule = (() => {
               signalPaths: resolved.signalPaths,
               signalValues: resolved.signalValues
             });
-            await applyServerResult(result, runContext);
-            results.push(unwrapServerResult(result));
+            results.push(result);
             continue;
           }
 

package/browser.umd.js

@@ -2389,9 +2389,10 @@
   })();
 
   const __serverModule = (() => {
-    const serverEnvelopeKeys = new Set(["value", "signals", "boundary", "html", "redirect", "error"]);
-    const appliedServerResult = Symbol.for("@async/framework.appliedServerResult");
-    const appliedServerValues = new WeakSet();
+    const serverEnvelopeKind = Symbol.for("@async/framework.serverResult");
+    const serverEnvelopeWireKey = "__async_server_result__";
+    const serverEnvelopeWireVersion = 1;
+    const serverResultInvocation = Symbol("@async/framework.serverResultInvocation");
 
     function createServerProxy({
       endpoint = "/__async/server",
@@ -2411,7 +2412,7 @@
 
       async function run(id, args = [], context = {}) {
         assertServerId(id);
-        const runContext = { ...defaults, ...context };
+        const runContext = createServerResultContext({ ...defaults, ...context });
         const body = {
           args,
           input: context.input ?? defaultInput(runContext),
@@ -2434,9 +2435,7 @@
           throw new Error(`Server function "${id}" failed with ${response.status}.`);
         }
 
-        const result = await readServerResponse(id, response);
-        await applyServerResult(result, runContext);
-        return markAppliedServerValue(unwrapServerResult(result));
+        return consumeServerResult(await readServerResponse(id, response), runContext);
       }
 
       return createServerNamespace(run, {
@@ -2471,12 +2470,13 @@
       if (!isServerEnvelope(result)) {
         return result;
       }
-      if (result[appliedServerResult] || appliedServerValues.has(result)) {
+      const invocation = getServerResultInvocation(context);
+      if (invocation.applied.has(result)) {
         return result;
       }
+      invocation.applied.add(result);
 
       if (result.error) {
-        markAppliedServerResult(result);
         throw toError(result.error);
       }
 
@@ -2498,32 +2498,35 @@
         await context.router?.navigate?.(result.redirect);
       }
 
-      markAppliedServerResult(result);
-
       return result;
     }
 
+    async function consumeServerResult(result, context = {}) {
+      await applyServerResult(result, context);
+      return unwrapServerResult(result);
+    }
+
     function unwrapServerResult(result) {
-      if (isServerEnvelope(result) && Object.hasOwn(result, "value")) {
-        return result.value;
+      if (isServerEnvelope(result)) {
+        return Object.hasOwn(result, "value") ? result.value : undefined;
       }
       return result;
     }
 
-    function markAppliedServerValue(value) {
-      if (value && typeof value === "object") {
-        appliedServerValues.add(value);
-      }
-      return value;
+    function createServerResultContext(context = {}) {
+      const invocation = context[serverResultInvocation] ?? {
+        applied: new WeakSet()
+      };
+      return {
+        ...context,
+        [serverResultInvocation]: invocation
+      };
     }
 
-    function markAppliedServerResult(result) {
-      Object.defineProperty(result, appliedServerResult, {
-        configurable: true,
-        enumerable: false,
-        value: true
-      });
-      return result;
+    function getServerResultInvocation(context = {}) {
+      return context[serverResultInvocation] ?? {
+        applied: new WeakSet()
+      };
     }
 
     function defaultInput(context = {}) {
@@ -2558,9 +2561,7 @@
             throw new Error("Server namespace is not directly callable.");
           }
           const context = contextProvider() ?? {};
-          const result = await run(parts.join("."), args, context);
-          await applyServerResult(result, context);
-          return unwrapServerResult(result);
+          return run(parts.join("."), args, context);
         };
 
         const proxy = new Proxy(callable, {
@@ -2617,7 +2618,7 @@
 
     async function readServerResponse(id, response) {
       if (response.status === 204) {
-        return { value: undefined };
+        return undefined;
       }
       const type = response.headers.get("content-type") ?? "";
       if (type.includes("application/json")) {
@@ -2635,7 +2636,7 @@
       if (typeof response.text !== "function") {
         throw new Error(`Server function "${id}" transport returned an invalid response: missing text().`);
       }
-      return { value: await response.text() };
+      return response.text();
     }
 
     function snapshotSignalPaths(paths = [], signals) {
@@ -2780,7 +2781,8 @@
       if (!value || typeof value !== "object" || Array.isArray(value)) {
         return false;
       }
-      return Object.keys(value).some((key) => serverEnvelopeKeys.has(key));
+      return value[serverEnvelopeKind] === true
+        || value[serverEnvelopeWireKey] === serverEnvelopeWireVersion;
     }
 
     function toError(value) {
@@ -2802,11 +2804,11 @@
         throw new TypeError("Server function id must be a non-empty string.");
       }
     }
-    return { createServerProxy, resolveServerCommandArguments, applyServerResult, unwrapServerResult, defaultInput, createServerNamespace, createSignalReader, assertServerId };
+    return { createServerProxy, resolveServerCommandArguments, applyServerResult, consumeServerResult, unwrapServerResult, createServerResultContext, defaultInput, createServerNamespace, createSignalReader, assertServerId };
   })();
 
   const __handlersModule = (() => {
-    const { applyServerResult, defaultInput, resolveServerCommandArguments, unwrapServerResult } = __serverModule;
+    const { defaultInput, resolveServerCommandArguments } = __serverModule;
     const { attachRegistryInspection, createRegistryStore } = __registryStoreModule;
     const { createLazyRegistry, isLazyDescriptor } = __lazyRegistryModule;
     const builtInTokens = new Set(["prevent", "preventDefault", "stopPropagation", "stopImmediatePropagation"]);
@@ -2904,8 +2906,7 @@
                 signalPaths: resolved.signalPaths,
                 signalValues: resolved.signalValues
               });
-              await applyServerResult(result, runContext);
-              results.push(unwrapServerResult(result));
+              results.push(result);
               continue;
             }