@async/framework 0.11.1 → 0.11.3

package/framework.ts

@@ -188,6 +188,28 @@ const __asyncSignalModule = (() => {
         };
       },
 
+      _cloneSignalDeclaration() {
+        return asyncSignal(id, fn);
+      },
+
+      _restore(snapshot = {}) {
+        if (!isAsyncSignalSnapshot(snapshot)) {
+          return state.set(snapshot);
+        }
+        if (activeAbort && !activeAbort.aborted) {
+          activeAbort.cancel(new Error(`Async signal "${registeredId}" restored from snapshot.`));
+        }
+        value = snapshot.value;
+        loading = Boolean(snapshot.loading);
+        error = snapshot.error ?? null;
+        status = typeof snapshot.status === "string" ? snapshot.status : inferStatus({ value, loading, error });
+        if (Number.isFinite(snapshot.version)) {
+          version = snapshot.version;
+        }
+        notify();
+        return state;
+      },
+
       _bindRegistry(nextRegistry, nextId) {
         registry = nextRegistry;
         registeredId = nextId;
@@ -273,6 +295,27 @@ const __asyncSignalModule = (() => {
     return Boolean(value?.[asyncSignalKind]);
   }
 
+  function isAsyncSignalSnapshot(value) {
+    if (!value || typeof value !== "object" || Array.isArray(value)) {
+      return false;
+    }
+    return Object.hasOwn(value, "value")
+      && (Object.hasOwn(value, "loading")
+        || Object.hasOwn(value, "error")
+        || Object.hasOwn(value, "status")
+        || Object.hasOwn(value, "version"));
+  }
+
+  function inferStatus({ value, loading, error }) {
+    if (loading) {
+      return "loading";
+    }
+    if (error) {
+      return "error";
+    }
+    return value === undefined ? "idle" : "ready";
+  }
+
   function attachCancel(signal, controller) {
     Object.defineProperty(signal, "cancel", {
       configurable: true,
@@ -920,7 +963,12 @@ const __cacheModule = (() => {
         return registryStore.entries(`${type}.entries`);
       },
 
-      _adoptMany() {
+      _adoptMany(map = {}) {
+        for (const [id, definition] of Object.entries(map ?? {})) {
+          if (!definitions.has(id)) {
+            registryApi.register(id, definition);
+          }
+        }
         return registryApi;
       }
     }, registryStore, type);
@@ -1078,6 +1126,10 @@ const __signalsModule = (() => {
 
       snapshot() {
         return value;
+      },
+
+      _cloneSignalDeclaration() {
+        return createSignal(value);
       }
     };
 
@@ -1120,6 +1172,10 @@ const __signalsModule = (() => {
         return backing.snapshot();
       },
 
+      _cloneSignalDeclaration() {
+        return computed(fn);
+      },
+
       _bindRegistry(registry, id) {
         return registry.effect(() => {
           backing.set(fn.call({
@@ -1144,6 +1200,9 @@ const __signalsModule = (() => {
       [effectKind]: true,
       kind: "effect",
       fn,
+      _cloneSignalDeclaration() {
+        return effect(fn);
+      },
       _bindRegistry(registry) {
         return registry.effect(fn);
       }
@@ -1362,10 +1421,14 @@ const __signalsModule = (() => {
       },
 
       _adoptMany(map = {}) {
-        for (const id of Object.keys(map ?? {})) {
-          if (entries.has(id)) {
-            bindEntry(id, entries.get(id));
+        for (const [id, signalLike] of Object.entries(map ?? {})) {
+          if (!entries.has(id)) {
+            const entry = cloneSignalDeclaration(signalLike);
+            entries.set(id, entry);
+            bindEntry(id, entry);
+            continue;
           }
+          bindEntry(id, entries.get(id));
         }
         return registry;
       }
@@ -1443,6 +1506,16 @@ const __signalsModule = (() => {
     return createSignal(signalLike);
   }
 
+  function cloneSignalDeclaration(signalLike) {
+    if (typeof signalLike?._cloneSignalDeclaration === "function") {
+      return signalLike._cloneSignalDeclaration();
+    }
+    if (isSignalLike(signalLike)) {
+      return createSignal(typeof signalLike.snapshot === "function" ? signalLike.snapshot() : signalLike.value);
+    }
+    return createSignal(signalLike);
+  }
+
   function isSignalLike(value) {
     return Boolean(value && typeof value === "object" && typeof value.subscribe === "function");
   }
@@ -1621,7 +1694,7 @@ const __signalsModule = (() => {
       frame.add(path);
     }
   }
-  return { createSignal, computed, effect, createSignalRegistry, isSignalRef, signal };
+  return { createSignal, computed, effect, createSignalRegistry, cloneSignalDeclaration, isSignalRef, signal };
 })();
 
 const __htmlModule = (() => {
@@ -1863,7 +1936,12 @@ const __componentModule = (() => {
         return lazyComponents.get(id);
       },
 
-      _adoptMany() {
+      _adoptMany(map = {}) {
+        for (const [id, Component] of Object.entries(map ?? {})) {
+          if (!entries.has(id)) {
+            registry.register(id, Component);
+          }
+        }
         return registry;
       }
     }, registryStore, type);
@@ -1911,12 +1989,16 @@ const __componentModule = (() => {
     });
 
     const output = Component.call(context, props);
+    if (output && typeof output.then === "function") {
+      throw new TypeError(`Component "${componentName(Component)}" returned a Promise. Async components are not supported by synchronous renderComponent(). Use an async partial or handler instead.`);
+    }
     const html = renderScopedTemplate(output);
 
     return {
       html,
       attach(target) {
-        for (const hook of attachHooks) {
+        for (let index = 0; index < attachHooks.length; index += 1) {
+          const hook = attachHooks[index];
           runtime.scheduler?.enqueue("lifecycle", () => {
             const cleanup = hook(target);
             if (typeof cleanup === "function") {
@@ -1924,7 +2006,7 @@ const __componentModule = (() => {
             }
           }, {
             scope,
-            key: `attach:${attachHooks.indexOf(hook)}`
+            key: `attach:${index}`
           }) ?? runAttachHook(hook, target);
         }
       },
@@ -1932,8 +2014,12 @@ const __componentModule = (() => {
         this.attach(target);
       },
       visible(target, observeVisible) {
-        for (const hook of visibleHooks) {
-          const cleanup = observeVisible(target, () => {
+        if (visibleHooks.length === 0) {
+          return;
+        }
+        const cleanup = observeVisible(target, () => {
+          for (let index = 0; index < visibleHooks.length; index += 1) {
+            const hook = visibleHooks[index];
             runtime.scheduler?.enqueue("lifecycle", () => {
               const hookCleanup = hook(target);
               if (typeof hookCleanup === "function") {
@@ -1941,12 +2027,12 @@ const __componentModule = (() => {
               }
             }, {
               scope,
-              key: `visible:${visibleHooks.indexOf(hook)}`
+              key: `visible:${index}`
             }) ?? runVisibleHook(hook, target);
-          });
-          if (typeof cleanup === "function") {
-            cleanups.push(cleanup);
           }
+        });
+        if (typeof cleanup === "function") {
+          cleanups.push(cleanup);
         }
       },
       cleanup() {
@@ -2202,11 +2288,12 @@ const __serverModule = (() => {
         signal: context.abort
       });
 
+      assertTransportResponse(id, response);
       if (!response.ok) {
         throw new Error(`Server function "${id}" failed with ${response.status}.`);
       }
 
-      const result = await readServerResponse(response);
+      const result = await readServerResponse(id, response);
       await applyServerResult(result, runContext);
       return markAppliedServerValue(unwrapServerResult(result));
     }
@@ -2247,6 +2334,11 @@ const __serverModule = (() => {
       return result;
     }
 
+    if (result.error) {
+      markAppliedServerResult(result);
+      throw toError(result.error);
+    }
+
     if (result.signals && context.signals) {
       for (const [path, value] of Object.entries(result.signals)) {
         context.signals.set?.(path, value);
@@ -2265,15 +2357,7 @@ const __serverModule = (() => {
       await context.router?.navigate?.(result.redirect);
     }
 
-    if (result.error) {
-      throw toError(result.error);
-    }
-
-    Object.defineProperty(result, appliedServerResult, {
-      configurable: true,
-      enumerable: false,
-      value: true
-    });
+    markAppliedServerResult(result);
 
     return result;
   }
@@ -2292,6 +2376,15 @@ const __serverModule = (() => {
     return value;
   }
 
+  function markAppliedServerResult(result) {
+    Object.defineProperty(result, appliedServerResult, {
+      configurable: true,
+      enumerable: false,
+      value: true
+    });
+    return result;
+  }
+
   function defaultInput(context = {}) {
     const form = findForm(context);
     if (form) {
@@ -2369,10 +2462,37 @@ const __serverModule = (() => {
     return namespace([]);
   }
 
-  async function readServerResponse(response) {
+  function assertTransportResponse(id, response) {
+    if (!response || typeof response !== "object") {
+      throw new Error(`Server function "${id}" transport returned an invalid response: expected a fetch Response-like object.`);
+    }
+    if (typeof response.ok !== "boolean") {
+      throw new Error(`Server function "${id}" transport returned an invalid response: missing boolean ok.`);
+    }
+    if (!response.headers || typeof response.headers.get !== "function") {
+      throw new Error(`Server function "${id}" transport returned an invalid response: missing headers.get(name).`);
+    }
+  }
+
+  async function readServerResponse(id, response) {
+    if (response.status === 204) {
+      return { value: undefined };
+    }
     const type = response.headers.get("content-type") ?? "";
     if (type.includes("application/json")) {
-      return response.json();
+      if (typeof response.json !== "function") {
+        throw new Error(`Server function "${id}" transport returned an invalid response: missing json().`);
+      }
+      try {
+        return await response.json();
+      } catch (cause) {
+        throw new Error(`Server function "${id}" returned invalid JSON: ${errorMessage(cause)}`, {
+          cause
+        });
+      }
+    }
+    if (typeof response.text !== "function") {
+      throw new Error(`Server function "${id}" transport returned an invalid response: missing text().`);
     }
     return { value: await response.text() };
   }
@@ -2485,6 +2605,9 @@ const __serverModule = (() => {
     if (tag === "[object File]" || tag === "[object Blob]" || tag === "[object FormData]") {
       throw new Error("Server proxy JSON transport does not support File, Blob, or FormData values yet.");
     }
+    if (isUnsupportedJsonTransportObject(value, tag)) {
+      throw new Error("Server proxy JSON transport does not support URLSearchParams, Headers, Request, Response, ReadableStream, ArrayBuffer, or typed array values yet.");
+    }
     if (Array.isArray(value)) {
       for (const item of value) {
         assertJsonTransportable(item, stack);
@@ -2498,6 +2621,16 @@ const __serverModule = (() => {
     stack.delete(value);
   }
 
+  function isUnsupportedJsonTransportObject(value, tag = Object.prototype.toString.call(value)) {
+    return tag === "[object URLSearchParams]"
+      || tag === "[object Headers]"
+      || tag === "[object Request]"
+      || tag === "[object Response]"
+      || tag === "[object ReadableStream]"
+      || tag === "[object ArrayBuffer]"
+      || ArrayBuffer.isView(value);
+  }
+
   function joinEndpoint(endpoint, id) {
     return `${String(endpoint).replace(/\/$/, "")}/${encodeURIComponent(id)}`;
   }
@@ -2519,6 +2652,10 @@ const __serverModule = (() => {
     return new Error(String(value));
   }
 
+  function errorMessage(error) {
+    return error instanceof Error ? error.message : String(error);
+  }
+
   function assertServerId(id) {
     if (typeof id !== "string" || id.length === 0) {
       throw new TypeError("Server function id must be a non-empty string.");
@@ -2644,7 +2781,12 @@ const __handlersModule = (() => {
         return results;
       },
 
-      _adoptMany() {
+      _adoptMany(map = {}) {
+        for (const [id, fn] of Object.entries(map ?? {})) {
+          if (!handlers.has(id)) {
+            registry.register(id, fn);
+          }
+        }
         return registry;
       }
     }, registryStore, type);
@@ -2751,7 +2893,8 @@ const __schedulerModule = (() => {
     const phases = [...(options.phases ?? defaultPhases)];
     const queues = new Map(phases.map((phase) => [phase, []]));
     const keyedJobs = new Map();
-    const destroyedScopes = new Set();
+    const destroyedObjectScopes = new WeakSet();
+    const destroyedPrimitiveScopes = new Set();
     const objectScopeIds = new WeakMap();
     const onError = typeof options.onError === "function" ? options.onError : undefined;
     const maxDepth = options.maxDepth ?? 100;
@@ -2799,7 +2942,7 @@ const __schedulerModule = (() => {
           throw new TypeError("scheduler.enqueue(phase, fn) requires a function.");
         }
         const scope = options.scope;
-        if (scope !== undefined && destroyedScopes.has(scope)) {
+        if (isScopeDestroyed(scope)) {
           return noop;
         }
 
@@ -2903,14 +3046,29 @@ const __schedulerModule = (() => {
 
       markScopeDestroyed(scope) {
         if (scope !== undefined) {
-          destroyedScopes.add(scope);
+          if (isObjectScope(scope)) {
+            destroyedObjectScopes.add(scope);
+          } else {
+            destroyedPrimitiveScopes.add(scope);
+          }
           api.cancelScope(scope);
         }
         return api;
       },
 
+      reviveScope(scope) {
+        if (scope !== undefined) {
+          if (isObjectScope(scope)) {
+            destroyedObjectScopes.delete(scope);
+          } else {
+            destroyedPrimitiveScopes.delete(scope);
+          }
+        }
+        return api;
+      },
+
       isScopeDestroyed(scope) {
-        return scope !== undefined && destroyedScopes.has(scope);
+        return isScopeDestroyed(scope);
       },
 
       inspect() {
@@ -2922,7 +3080,7 @@ const __schedulerModule = (() => {
           strategy,
           phases: [...phases],
           pending: counts,
-          scopesDestroyed: destroyedScopes.size,
+          scopesDestroyed: destroyedPrimitiveScopes.size,
           flushing,
           scheduled
         };
@@ -2937,7 +3095,7 @@ const __schedulerModule = (() => {
           queue.length = 0;
         }
         keyedJobs.clear();
-        destroyedScopes.clear();
+        destroyedPrimitiveScopes.clear();
       }
     };
 
@@ -2977,7 +3135,7 @@ const __schedulerModule = (() => {
         if (job.key) {
           keyedJobs.delete(job.key);
         }
-        if (job.canceled || (job.scope !== undefined && destroyedScopes.has(job.scope))) {
+        if (job.canceled || isScopeDestroyed(job.scope)) {
           continue;
         }
         try {
@@ -3034,6 +3192,20 @@ const __schedulerModule = (() => {
       }
       return String(scope);
     }
+
+    function isScopeDestroyed(scope) {
+      if (scope === undefined) {
+        return false;
+      }
+      if (isObjectScope(scope)) {
+        return destroyedObjectScopes.has(scope);
+      }
+      return destroyedPrimitiveScopes.has(scope);
+    }
+  }
+
+  function isObjectScope(scope) {
+    return (typeof scope === "object" && scope !== null) || typeof scope === "function";
   }
 
   function scheduleMicrotask(fn) {
@@ -3094,6 +3266,7 @@ const __loaderModule = (() => {
 
       scan(rootOrFragment = rootNode) {
         assertActive();
+        reviveScopes(rootOrFragment);
         bindSignalAttributes(rootOrFragment);
         bindClassAttributes(rootOrFragment);
         bindEventAttributes(rootOrFragment);
@@ -3597,6 +3770,12 @@ const __loaderModule = (() => {
       }
     }
 
+    function reviveScopes(scope) {
+      for (const element of elementsIn(scope)) {
+        schedulerInstance.reviveScope?.(element);
+      }
+    }
+
     return api;
   }
 
@@ -3896,7 +4075,12 @@ const __partialsModule = (() => {
         return normalizePartialResult(result, partialContext);
       },
 
-      _adoptMany() {
+      _adoptMany(map = {}) {
+        for (const [id, fn] of Object.entries(map ?? {})) {
+          if (!entries.has(id)) {
+            registry.register(id, fn);
+          }
+        }
         return registry;
       }
     }, registryStore, type);
@@ -4048,7 +4232,11 @@ const __routerModule = (() => {
       },
 
       _adoptMany(map = {}) {
-        for (const pattern of Object.keys(map ?? {})) {
+        for (const [pattern, definition] of Object.entries(map ?? {})) {
+          if (!entries.has(pattern)) {
+            registry.register(pattern, definition);
+            continue;
+          }
           adoptRoute(pattern, entries.get(pattern));
         }
         return registry;
@@ -4160,7 +4348,10 @@ const __routerModule = (() => {
         }
         const matched = api.match(url);
         if (matched?.route?.partial && partials?.resolve?.(matched.route.partial)) {
-          return partials.render(matched.route.partial, matched.params, contextFor(matched));
+          return partials.render(matched.route.partial, matched.params, {
+            ...contextFor(matched),
+            prefetch: true
+          });
         }
         return Promise.resolve(null);
       },
@@ -4541,7 +4732,7 @@ const __appModule = (() => {
   const { createRouteRegistry, createRouter } = __routerModule;
   const { createScheduler } = __schedulerModule;
   const { createServerNamespace } = __serverModule;
-  const { createSignal, createSignalRegistry } = __signalsModule;
+  const { cloneSignalDeclaration, createSignal, createSignalRegistry } = __signalsModule;
   const { createRegistryStore } = __registryStoreModule;
   const { attributeName, normalizeAttributeConfig } = __attributesModule;
   const { createLazyRegistry, defineRegistrySnapshot, sameRegistryValue } = __lazyRegistryModule;
@@ -4624,7 +4815,7 @@ const __appModule = (() => {
       registryAssets: options.registryAssets,
       importModule: options.importModule
     });
-    const registry = options.registry ?? app.registry.view({ target });
+    const registry = options.registry ?? createRuntimeRegistry(app.registry, { target });
     const signals = options.signals ?? createSignalRegistry(undefined, { registry, type: "signal", lazyRegistry });
     const handlers = options.handlers ?? createHandlerRegistry(undefined, { registry, type: "handler", lazyRegistry });
     const serverCache = createCacheRegistry(undefined, { registry, type: "cache.server" });
@@ -5005,6 +5196,22 @@ const __appModule = (() => {
     registry?.registerMany?.(entries);
   }
 
+  function createRuntimeRegistry(appRegistry, { target } = {}) {
+    const declarations = appRegistry.rawSnapshot();
+    return createRegistryStore({
+      ...declarations,
+      signal: cloneSignalDeclarations(declarations.signal)
+    }, { target });
+  }
+
+  function cloneSignalDeclarations(signals = {}) {
+    const cloned = {};
+    for (const [id, signalLike] of Object.entries(signals ?? {})) {
+      cloned[id] = cloneSignalDeclaration(signalLike);
+    }
+    return cloned;
+  }
+
   function emptyDeclarations() {
     return {
       signal: {},
@@ -5196,6 +5403,13 @@ const __appModule = (() => {
   function setOrRegisterSignal(signals, path, value) {
     const id = String(path).split(".")[0];
     if (signals.has?.(id)) {
+      if (path === id) {
+        const entry = signals._entry?.(id);
+        if (typeof entry?._restore === "function" && isAsyncSignalSnapshot(value)) {
+          entry._restore(value);
+          return;
+        }
+      }
       signals.set(path, value);
       return;
     }
@@ -5205,6 +5419,17 @@ const __appModule = (() => {
     }
   }
 
+  function isAsyncSignalSnapshot(value) {
+    if (!value || typeof value !== "object" || Array.isArray(value)) {
+      return false;
+    }
+    return Object.hasOwn(value, "value")
+      && (Object.hasOwn(value, "loading")
+        || Object.hasOwn(value, "error")
+        || Object.hasOwn(value, "status")
+        || Object.hasOwn(value, "version"));
+  }
+
   function attachServerCache(server, cache) {
     try {
       server.cache = cache;
@@ -5455,7 +5680,12 @@ const __serverRegistryModule = (() => {
         return registry;
       },
 
-      _adoptMany() {
+      _adoptMany(map = {}) {
+        for (const [id, fn] of Object.entries(map ?? {})) {
+          if (!entries.has(id)) {
+            registry.register(id, fn);
+          }
+        }
         return registry;
       }
     }, registryStore, type);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@async/framework",
-  "version": "0.11.1",
+  "version": "0.11.3",
   "description": "No-build Loader app runtime with browser and server entrypoints, signals, command events, route partials, cache split, SSR activation, and streaming boundaries.",
   "type": "module",
   "main": "./server.js",
@@ -115,8 +115,9 @@
     "pack:check": "pnpm run bundle:check && npm pack --dry-run --ignore-scripts",
     "pipeline:github:check": "async-pipeline github check",
     "pipeline:github:generate": "async-pipeline github generate",
-    "pipeline:pages": "async-pipeline run pages",
+    "pipeline:pages": "async-pipeline run-task docs.site",
     "pipeline:publish": "async-pipeline run publish",
+    "pipeline:publish:github:release": "async-pipeline publish github release --package . --registry https://npm.pkg.github.com",
     "pipeline:publish:npm": "async-pipeline publish npm --package .",
     "pipeline:release-doctor": "async-pipeline run release-doctor",
     "pipeline:release:doctor": "async-pipeline release doctor --package .",
@@ -130,7 +131,7 @@
     "test": "node --test tests/*.test.js"
   },
   "devDependencies": {
-    "@async/pipeline": "0.8.5",
+    "@async/pipeline": "0.9.1",
     "happy-dom": "20.10.5",
     "terser": "5.48.0"
   }