veryfront 0.1.1095 → 0.1.1098

package/esm/deno.js

@@ -1,6 +1,6 @@
 export default {
     "name": "veryfront",
-    "version": "0.1.1095",
+    "version": "0.1.1098",
     "license": "Apache-2.0",
     "nodeModulesDir": "auto",
     "minimumDependencyAge": {

package/esm/src/rendering/orchestrator/module-loader/esm-rewriter.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"esm-rewriter.d.ts","sourceRoot":"","sources":["../../../../../src/src/rendering/orchestrator/module-loader/esm-rewriter.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,oCAAoC,CAAC;AAWzE,wBAAgB,eAAe,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,MAAM,CAmCrE;AAED,wBAAsB,cAAc,CAClC,GAAG,EAAE,MAAM,EACX,MAAM,EAAE,MAAM,EACd,YAAY,EAAE,cAAc,EAC5B,QAAQ,EAAE,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,GAC5B,OAAO,CAAC,MAAM,CAAC,CA8CjB"}
+{"version":3,"file":"esm-rewriter.d.ts","sourceRoot":"","sources":["../../../../../src/src/rendering/orchestrator/module-loader/esm-rewriter.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,oCAAoC,CAAC;AAkCzE,wBAAgB,eAAe,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,MAAM,CAmCrE;AAED,wBAAsB,cAAc,CAClC,GAAG,EAAE,MAAM,EACX,MAAM,EAAE,MAAM,EACd,YAAY,EAAE,cAAc,EAC5B,QAAQ,EAAE,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,GAC5B,OAAO,CAAC,MAAM,CAAC,CAwEjB"}

package/esm/src/rendering/orchestrator/module-loader/esm-rewriter.js

@@ -1,7 +1,28 @@
 import { rendererLogger } from "../../../utils/index.js";
 import { MODULE_NOT_FOUND } from "../../../errors/index.js";
 import { generateHash } from "./cache.js";
+import { parseImports } from "../../../transforms/esm/lexer.js";
 const logger = rendererLogger.component("module-loader");
+/**
+ * Specifiers `code` imports statically, as opposed to through `import(...)` or
+ * merely mentioning in a string.
+ *
+ * A lex failure returns every discovered specifier as static, so an unparseable
+ * bundle keeps the pre-graceful-degradation behaviour of failing loudly rather
+ * than quietly shipping a remote dependency.
+ */
+async function staticImportSpecifiers(code) {
+    try {
+        const imports = await parseImports(code);
+        return new Set(imports.filter((imp) => imp.d === -1 && imp.n).map((imp) => imp.n));
+    }
+    catch (error) {
+        logger.debug("Could not lex a fetched module; treating its imports as static", {
+            error: error instanceof Error ? error.message : String(error),
+        });
+        return new Set(code.match(/https:\/\/esm\.sh\/[^"']+/g) ?? []);
+    }
+}
 function escapeRegExp(value) {
     return value.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
 }
@@ -53,17 +74,39 @@ export async function fetchEsmModule(url, tmpDir, localAdapter, esmCache) {
             allEsmUrls.add(match[1]);
     }
     const urlArray = Array.from(allEsmUrls);
-    const cachedPaths = await Promise.all(urlArray.map((esmUrl) => fetchEsmModule(esmUrl, tmpDir, localAdapter, esmCache)));
+    const staticUrls = await staticImportSpecifiers(code);
+    // Nested pre-fetches of a URL this module only reaches lazily are
+    // best-effort: a broken esm.sh build for one package logs a warning and the
+    // URL stays in the emitted code for the runtime to resolve at call time. A
+    // URL the module imports statically is part of its own import graph and must
+    // still resolve here, so the emitted artifact's static dependencies stay
+    // local. See `transforms/esm/specifier-resolver.ts` for the same rule on the
+    // SSR transform path.
+    const settledPaths = await Promise.allSettled(urlArray.map((esmUrl) => fetchEsmModule(esmUrl, tmpDir, localAdapter, esmCache)));
     if (urlArray.length) {
         const replacementMap = new Map();
         for (let i = 0; i < urlArray.length; i++) {
             const url = urlArray[i];
-            const cached = cachedPaths[i];
-            if (url && cached)
-                replacementMap.set(url, `file://${cached}`);
+            const result = settledPaths[i];
+            if (!url || !result)
+                continue;
+            if (result.status === "fulfilled") {
+                replacementMap.set(url, `file://${result.value}`);
+                continue;
+            }
+            // A statically imported dependency must be local before this module is
+            // handed to the runtime loader, so its failure stays fatal.
+            if (staticUrls.has(url))
+                throw result.reason;
+            logger.warn("Leaving an unfetchable lazy esm.sh module for runtime resolution", {
+                url,
+                error: result.reason instanceof Error ? result.reason.message : String(result.reason),
+            });
+        }
+        if (replacementMap.size) {
+            const combinedPattern = new RegExp(Array.from(replacementMap.keys()).map(escapeRegExp).join("|"), "g");
+            code = code.replace(combinedPattern, (m) => replacementMap.get(m) ?? m);
         }
-        const combinedPattern = new RegExp(urlArray.map(escapeRegExp).join("|"), "g");
-        code = code.replace(combinedPattern, (m) => replacementMap.get(m) ?? m);
     }
     const hash = await generateHash(url);
     const tempFilePath = `${tmpDir}/esm-${hash}.js`;

package/esm/src/rendering/orchestrator/module-loader/index.js

@@ -147,7 +147,7 @@ export async function loadModule(filePath, config) {
     const tmpDir = await getModuleCacheDir(config);
     const localAdapter = await getLocalAdapter();
     const tempFilePath = await transformModuleWithDeps(filePath, tmpDir, localAdapter, config);
-    const moduleUrl = `file://${tempFilePath}?t=${Date.now()}`;
+    const moduleUrl = `file://${tempFilePath}`;
     try {
         return await import(moduleUrl);
     }

package/esm/src/transforms/esm/specifier-resolver.d.ts

@@ -11,6 +11,9 @@ import { type CacheOptions } from "./http-cache-helpers.js";
 export type CacheHttpModuleFn = (url: string, options: CacheOptions) => Promise<string | null>;
 /**
  * Build a map of specifier replacements by resolving all imports in the code.
+ *
+ * Resolution failure is fatal for a static import and best-effort for a
+ * specifier only ever used in `import(...)`. See {@link isDynamicOnly}.
  */
 export declare function buildReplacements(code: string, baseUrl: string | undefined, options: CacheOptions, cacheHttpModule: CacheHttpModuleFn): Promise<Map<string, string>>;
 /**

package/esm/src/transforms/esm/specifier-resolver.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"specifier-resolver.d.ts","sourceRoot":"","sources":["../../../../src/src/transforms/esm/specifier-resolver.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAKH,OAAO,EACL,KAAK,YAAY,EAQlB,MAAM,yBAAyB,CAAC;AAEjC,kFAAkF;AAClF,MAAM,MAAM,iBAAiB,GAAG,CAAC,GAAG,EAAE,MAAM,EAAE,OAAO,EAAE,YAAY,KAAK,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAAC;AAgF/F;;GAEG;AACH,wBAAsB,iBAAiB,CACrC,IAAI,EAAE,MAAM,EACZ,OAAO,EAAE,MAAM,GAAG,SAAS,EAC3B,OAAO,EAAE,YAAY,EACrB,eAAe,EAAE,iBAAiB,GACjC,OAAO,CAAC,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC,CAiB9B;AAED;;GAEG;AACH,wBAAsB,oBAAoB,CACxC,IAAI,EAAE,MAAM,EACZ,SAAS,EAAE,MAAM,EACjB,OAAO,EAAE,YAAY,EACrB,eAAe,EAAE,iBAAiB,GACjC,OAAO,CAAC,MAAM,CAAC,CAKjB"}
+{"version":3,"file":"specifier-resolver.d.ts","sourceRoot":"","sources":["../../../../src/src/transforms/esm/specifier-resolver.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAQH,OAAO,EACL,KAAK,YAAY,EAQlB,MAAM,yBAAyB,CAAC;AAEjC,kFAAkF;AAClF,MAAM,MAAM,iBAAiB,GAAG,CAAC,GAAG,EAAE,MAAM,EAAE,OAAO,EAAE,YAAY,KAAK,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAAC;AA4G/F;;;;;GAKG;AACH,wBAAsB,iBAAiB,CACrC,IAAI,EAAE,MAAM,EACZ,OAAO,EAAE,MAAM,GAAG,SAAS,EAC3B,OAAO,EAAE,YAAY,EACrB,eAAe,EAAE,iBAAiB,GACjC,OAAO,CAAC,GAAG,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC,CAoC9B;AAED;;GAEG;AACH,wBAAsB,oBAAoB,CACxC,IAAI,EAAE,MAAM,EACZ,SAAS,EAAE,MAAM,EACjB,OAAO,EAAE,YAAY,EACrB,eAAe,EAAE,iBAAiB,GACjC,OAAO,CAAC,MAAM,CAAC,CAKjB"}

package/esm/src/transforms/esm/specifier-resolver.js

@@ -8,7 +8,9 @@
  */
 import { basename } from "../../platform/compat/path/index.js";
 import { resolveImport } from "../../modules/import-map/resolver.js";
+import { rendererLogger } from "../../utils/index.js";
 import { parseImports, replaceSpecifiers } from "./lexer.js";
+const logger = rendererLogger.component("specifier-resolver");
 import { isCanonicalReactEsmUrl, isExternalScheme, isHttpUrl, isInternalBare, isParentHttpModule, isRelative, resolveBareSpecifier, } from "./http-cache-helpers.js";
 function isLocalMappedSpecifier(specifier) {
     return specifier.startsWith("/_vf_modules/") ||
@@ -80,20 +82,68 @@ async function resolveSpecifier(specifier, baseUrl, options, cacheHttpModule) {
         return mapped;
     return resolveSpecifier(mapped, baseUrl, options, cacheHttpModule);
 }
+/**
+ * Specifiers this module only ever reaches through `import(...)`.
+ *
+ * The distinction decides what a resolution failure means. A static import is
+ * part of the emitted module's own import graph, so the artifact contract holds
+ * for it: every static dependency resolves to a local path before the module is
+ * handed to the runtime loader, and a failure to do that is fatal, exactly as
+ * it was before graceful degradation existed.
+ *
+ * A dynamic specifier is resolved by the runtime at call time and is routinely
+ * guarded by the caller (`platform/adapters/redis/modules.js` only calls
+ * `await import("redis")` when the redis adapter is actually used). Pre-fetching
+ * it is an optimisation, so failing to pre-fetch it leaves the specifier in
+ * place rather than taking down a render that would never have imported it.
+ */
+function isDynamicOnly(imports) {
+    const dynamic = new Set();
+    const staticSpecifiers = new Set();
+    for (const imp of imports) {
+        if (!imp.n)
+            continue;
+        (imp.d > -1 ? dynamic : staticSpecifiers).add(imp.n);
+    }
+    for (const specifier of staticSpecifiers)
+        dynamic.delete(specifier);
+    return dynamic;
+}
 /**
  * Build a map of specifier replacements by resolving all imports in the code.
+ *
+ * Resolution failure is fatal for a static import and best-effort for a
+ * specifier only ever used in `import(...)`. See {@link isDynamicOnly}.
  */
 export async function buildReplacements(code, baseUrl, options, cacheHttpModule) {
     const imports = await parseImports(code);
     const uniqueSpecifiers = [...new Set(imports.map((imp) => imp.n).filter(Boolean))];
-    const results = await Promise.all(uniqueSpecifiers.map(async (specifier) => ({
+    const dynamicOnly = isDynamicOnly(imports);
+    const settled = await Promise.allSettled(uniqueSpecifiers.map(async (specifier) => ({
         specifier,
         resolved: await resolveSpecifier(specifier, baseUrl, options, cacheHttpModule),
     })));
     const replacements = new Map();
-    for (const { specifier, resolved } of results) {
-        if (resolved && resolved !== specifier)
-            replacements.set(specifier, resolved);
+    for (let i = 0; i < settled.length; i++) {
+        const outcome = settled[i];
+        const specifier = uniqueSpecifiers[i];
+        if (!outcome || specifier === undefined)
+            continue;
+        if (outcome.status === "fulfilled") {
+            const { specifier: resolvedFor, resolved } = outcome.value;
+            if (resolved && resolved !== resolvedFor)
+                replacements.set(resolvedFor, resolved);
+            continue;
+        }
+        // A static import must resolve. Leaving one unresolved would emit a module
+        // whose own import graph reaches outside the local cache, which is not what
+        // the runtime loader is handed anywhere else.
+        if (!dynamicOnly.has(specifier))
+            throw outcome.reason;
+        logger.warn("Leaving an unresolvable dynamic specifier for runtime resolution", {
+            specifier,
+            error: outcome.reason instanceof Error ? outcome.reason.message : String(outcome.reason),
+        });
     }
     return replacements;
 }

package/esm/src/utils/version-constant.d.ts

@@ -1,3 +1,3 @@
 /** Shared version value. */
-export declare const VERSION = "0.1.1095";
+export declare const VERSION = "0.1.1098";
 //# sourceMappingURL=version-constant.d.ts.map

package/esm/src/utils/version-constant.js

@@ -1,4 +1,4 @@
 // Keep in sync with deno.json version.
 // scripts/release.ts updates this constant during releases.
 /** Shared version value. */
-export const VERSION = "0.1.1095";
+export const VERSION = "0.1.1098";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "veryfront",
-  "version": "0.1.1095",
+  "version": "0.1.1098",
   "description": "The simplest way to build AI-powered apps",
   "keywords": [
     "react",
@@ -328,10 +328,10 @@
     "@types/react": "19.2.14",
     "@types/react-dom": "19.2.3",
     "ws": "8.21.0",
-    "@veryfront/ext-bundler-esbuild": "0.1.1095",
-    "@veryfront/ext-content-mdx": "0.1.1095",
-    "@veryfront/ext-css-tailwind": "0.1.1095",
-    "@veryfront/ext-parser-babel": "0.1.1095"
+    "@veryfront/ext-bundler-esbuild": "0.1.1098",
+    "@veryfront/ext-content-mdx": "0.1.1098",
+    "@veryfront/ext-css-tailwind": "0.1.1098",
+    "@veryfront/ext-parser-babel": "0.1.1098"
   },
   "devDependencies": {
     "@types/node": "20.9.0"