@ecopages/core 0.2.0-beta.31 → 0.2.0-beta.32

package/src/build/runtime/build-request-identity.js

@@ -1,54 +1,79 @@
-import path from "node:path";
-import { createJsxCacheKey, createPluginCacheKey, createSourceTransformCacheKey } from "../cache/cache-keys.js";
+import path from 'node:path';
+import { createJsxCacheKey, createPluginCacheKey, createSourceTransformCacheKey } from '../cache/cache-keys.js';
 function normalizeEntrypoints(entrypoints) {
-  if (Array.isArray(entrypoints)) {
-    return entrypoints.map((entrypoint) => path.resolve(entrypoint)).sort().join("|");
-  }
-  return Object.entries(entrypoints).sort(([left], [right]) => left.localeCompare(right)).map(([key, value]) => `${key}:${path.resolve(value)}`).join("|");
+    if (Array.isArray(entrypoints)) {
+        return entrypoints
+            .map((entrypoint) => path.resolve(entrypoint))
+            .sort()
+            .join('|');
+    }
+    return Object.entries(entrypoints)
+        .sort(([left], [right]) => left.localeCompare(right))
+        .map(([key, value]) => `${key}:${path.resolve(value)}`)
+        .join('|');
 }
+/**
+ * Preserves list order for collections where Rolldown precedence depends on position.
+ */
 function normalizeOrderedStringList(values, label) {
-  if (!values || values.length === 0) {
-    return `${label}:default`;
-  }
-  return `${label}:${values.join(",")}`;
+    if (!values || values.length === 0) {
+        return `${label}:default`;
+    }
+    return `${label}:${values.join(',')}`;
 }
+/**
+ * Sorts unordered sets so equivalent membership produces one identity.
+ */
 function normalizeUnorderedStringList(values, label) {
-  if (!values || values.length === 0) {
-    return `${label}:default`;
-  }
-  return `${label}:${[...values].sort().join(",")}`;
+    if (!values || values.length === 0) {
+        return `${label}:default`;
+    }
+    return `${label}:${[...values].sort().join(',')}`;
 }
 function normalizeDefine(define) {
-  if (!define || Object.keys(define).length === 0) {
-    return "define:default";
-  }
-  return `define:${Object.entries(define).sort(([left], [right]) => left.localeCompare(right)).map(([key, value]) => `${key}=${value}`).join(",")}`;
+    if (!define || Object.keys(define).length === 0) {
+        return 'define:default';
+    }
+    return `define:${Object.entries(define)
+        .sort(([left], [right]) => left.localeCompare(right))
+        .map(([key, value]) => `${key}=${value}`)
+        .join(',')}`;
 }
-function createBuildRequestIdentity(options) {
-  return [
-    normalizeEntrypoints(options.entrypoints),
-    options.root ? path.resolve(options.root) : "root:default",
-    options.outdir ? path.resolve(options.outdir) : "outdir:default",
-    options.outbase ? path.resolve(options.outbase) : "outbase:default",
-    options.splitting ?? "splitting:default",
-    options.bundle ?? "bundle:default",
-    options.externalPackages ?? "externalPackages:default",
-    options.target ?? "target:default",
-    options.format ?? "format:default",
-    options.sourcemap ?? "sourcemap:default",
-    options.minify ?? "minify:default",
-    options.treeshaking ?? "treeshaking:default",
-    options.naming ?? "naming:default",
-    normalizeOrderedStringList(options.conditions, "conditions"),
-    normalizeDefine(options.define),
-    normalizeUnorderedStringList(options.external, "external"),
-    createJsxCacheKey(options.jsx),
-    createPluginCacheKey(options.plugins),
-    createSourceTransformCacheKey(options.sourceTransforms)
-  ].join("::");
+/**
+ * Canonical identity for one complete {@link BuildOptions} request.
+ *
+ * @remarks
+ * Semantically unordered collections (`external`, `define` keys, array entrypoints)
+ * are sorted. Plugin, `conditions`, and source-transform order is preserved because
+ * Rolldown hook and `conditionNames` precedence depend on it.
+ *
+ * Deprecated / ignored adapter fields (`splitting`, `bundle`, `outbase`) still
+ * participate in the key so requests that differ only on those fields do not
+ * coalesce. Empty `conditions` / `define` / `external` normalize to the same
+ * sentinel as `undefined`.
+ */
+export function createBuildRequestIdentity(options) {
+    return [
+        normalizeEntrypoints(options.entrypoints),
+        options.root ? path.resolve(options.root) : 'root:default',
+        options.outdir ? path.resolve(options.outdir) : 'outdir:default',
+        options.outbase ? path.resolve(options.outbase) : 'outbase:default',
+        options.splitting ?? 'splitting:default',
+        options.bundle ?? 'bundle:default',
+        options.externalPackages ?? 'externalPackages:default',
+        options.target ?? 'target:default',
+        options.format ?? 'format:default',
+        options.sourcemap ?? 'sourcemap:default',
+        options.minify ?? 'minify:default',
+        options.treeshaking ?? 'treeshaking:default',
+        options.naming ?? 'naming:default',
+        normalizeOrderedStringList(options.conditions, 'conditions'),
+        normalizeDefine(options.define),
+        normalizeUnorderedStringList(options.external, 'external'),
+        createJsxCacheKey(options.jsx),
+        createPluginCacheKey(options.plugins),
+        createSourceTransformCacheKey(options.sourceTransforms),
+    ].join('::');
 }
-const createBuildOptionsDedupeKey = createBuildRequestIdentity;
-export {
-  createBuildOptionsDedupeKey,
-  createBuildRequestIdentity
-};
+/** @deprecated Use {@link createBuildRequestIdentity}. */
+export const createBuildOptionsDedupeKey = createBuildRequestIdentity;

package/src/build/runtime/build-request-policy.js

@@ -1,46 +1,84 @@
-import { getAppBrowserBuildPlugins, getAppServerBuildPlugins, getAppTranspileOptions } from "../build-adapter.js";
-import { resolveBuildProfileOptions } from "./build-profile-options.js";
-import { getJsxOwnershipPlugins } from "../browser/jsx-ownership-plugins.js";
-import { getAppSourceTransforms } from "../../plugins/source-transform.js";
-function resolveServerAppBuildPlugins(appConfig) {
-  return [...getAppServerBuildPlugins(appConfig), ...getJsxOwnershipPlugins(appConfig)];
+import { getAppBrowserBuildPlugins, getAppServerBuildPlugins, getAppTranspileOptions } from '../build-adapter.js';
+import { resolveBuildProfileOptions } from './build-profile-options.js';
+import { getJsxOwnershipPlugins } from '../browser/jsx-ownership-plugins.js';
+import { getAppSourceTransforms } from '../../plugins/source-transform.js';
+/**
+ * Resolves app-owned server plugins plus JSX ownership for one config.
+ *
+ * @remarks
+ * Always goes through {@link getAppServerBuildPlugins} so sealed-manifest and
+ * loader-fallback apps share one plugin list with the alias resolver. Used by
+ * both request assembly and unified-graph cache keys.
+ */
+export function resolveServerAppBuildPlugins(appConfig) {
+    return [...getAppServerBuildPlugins(appConfig), ...getJsxOwnershipPlugins(appConfig)];
 }
-function mergeCallerBuildPlugins(appPlugins, callerPlugins) {
-  if (!callerPlugins || callerPlugins.length === 0) {
-    return appPlugins;
-  }
-  const appByName = new Map(appPlugins.map((plugin) => [plugin.name, plugin]));
-  const uniqueCallerPlugins = callerPlugins.filter((plugin) => !appByName.has(plugin.name));
-  return [...uniqueCallerPlugins, ...appPlugins];
+/**
+ * Merges caller plugins with app-owned manifest plugins.
+ *
+ * @remarks
+ * App-manifest plugins cannot be silently replaced by same-name caller plugins.
+ * Unique caller plugins are prepended so first-wins Rolldown hooks (for example
+ * React vendor/alias rewrites) still run before general app loaders. Use
+ * `excludeAppBuildPlugins` on browser requests to omit app-owned plugins.
+ */
+export function mergeCallerBuildPlugins(appPlugins, callerPlugins) {
+    if (!callerPlugins || callerPlugins.length === 0) {
+        return appPlugins;
+    }
+    const appByName = new Map(appPlugins.map((plugin) => [plugin.name, plugin]));
+    const uniqueCallerPlugins = callerPlugins.filter((plugin) => !appByName.has(plugin.name));
+    return [...uniqueCallerPlugins, ...appPlugins];
 }
-function createServerBuildRequest(appConfig, input) {
-  const profile = input.profile ?? "route-module";
-  const plugins = mergeCallerBuildPlugins(resolveServerAppBuildPlugins(appConfig), input.plugins);
-  const { plugins: _callerPlugins, profile: _profile, ...overrides } = input;
-  return {
-    ...resolveBuildProfileOptions(profile, appConfig, overrides),
-    ...overrides,
-    entrypoints: input.entrypoints,
-    ...plugins.length > 0 ? { plugins } : {}
-  };
+/**
+ * Assembles a complete server-oriented {@link BuildOptions} before scheduling.
+ *
+ * @remarks
+ * Defaults `profile` to `'route-module'`. Plugins are
+ * {@link resolveServerAppBuildPlugins} plus unique caller contributions via
+ * {@link mergeCallerBuildPlugins}. Does not attach `sourceTransforms` — unlike
+ * {@link createBrowserBuildRequest}. Pass the returned options into
+ * {@link BuildRuntime.getProfile}; the runtime no longer injects plugins.
+ */
+export function createServerBuildRequest(appConfig, input) {
+    const profile = input.profile ?? 'route-module';
+    const plugins = mergeCallerBuildPlugins(resolveServerAppBuildPlugins(appConfig), input.plugins);
+    const { plugins: _callerPlugins, profile: _profile, ...overrides } = input;
+    return {
+        ...resolveBuildProfileOptions(profile, appConfig, overrides),
+        ...overrides,
+        entrypoints: input.entrypoints,
+        ...(plugins.length > 0 ? { plugins } : {}),
+    };
 }
-function createBrowserBuildRequest(appConfig, input) {
-  const { profile, excludeAppBuildPlugins, plugins: callerPlugins, ...overrides } = input;
-  const appBrowserPlugins = getAppBrowserBuildPlugins(appConfig);
-  const filteredAppBrowserPlugins = excludeAppBuildPlugins && excludeAppBuildPlugins.length > 0 ? appBrowserPlugins.filter((plugin) => !excludeAppBuildPlugins.includes(plugin.name)) : appBrowserPlugins;
-  const plugins = mergeCallerBuildPlugins(filteredAppBrowserPlugins, callerPlugins);
-  return {
-    ...resolveBuildProfileOptions("browser-hmr", appConfig, overrides),
-    ...getAppTranspileOptions(appConfig, profile),
-    ...overrides,
-    entrypoints: input.entrypoints,
-    plugins,
-    sourceTransforms: getAppSourceTransforms(appConfig)
-  };
+/**
+ * Assembles a complete browser-oriented {@link BuildOptions} before scheduling.
+ *
+ * @remarks
+ * Applies {@link resolveBuildProfileOptions} for `'browser-hmr'` (browser target
+ * defaults), then overlays {@link getAppTranspileOptions} for the given
+ * {@link BuildTranspileProfile}. That profile selects transpile settings only;
+ * the {@link BuildRuntime} executor slot is chosen later by the caller
+ * (typically {@link BrowserBundleService}).
+ *
+ * Always attaches {@link getAppSourceTransforms}. App browser plugins come from
+ * {@link getAppBrowserBuildPlugins} (includes JSX ownership); use
+ * `excludeAppBuildPlugins` to omit app-owned names. Caller plugins cannot
+ * replace app-manifest names ({@link mergeCallerBuildPlugins}).
+ */
+export function createBrowserBuildRequest(appConfig, input) {
+    const { profile, excludeAppBuildPlugins, plugins: callerPlugins, ...overrides } = input;
+    const appBrowserPlugins = getAppBrowserBuildPlugins(appConfig);
+    const filteredAppBrowserPlugins = excludeAppBuildPlugins && excludeAppBuildPlugins.length > 0
+        ? appBrowserPlugins.filter((plugin) => !excludeAppBuildPlugins.includes(plugin.name))
+        : appBrowserPlugins;
+    const plugins = mergeCallerBuildPlugins(filteredAppBrowserPlugins, callerPlugins);
+    return {
+        ...resolveBuildProfileOptions('browser-hmr', appConfig, overrides),
+        ...getAppTranspileOptions(appConfig, profile),
+        ...overrides,
+        entrypoints: input.entrypoints,
+        plugins,
+        sourceTransforms: getAppSourceTransforms(appConfig),
+    };
 }
-export {
-  createBrowserBuildRequest,
-  createServerBuildRequest,
-  mergeCallerBuildPlugins,
-  resolveServerAppBuildPlugins
-};

package/src/build/runtime/build-runtime.js

@@ -1,65 +1,75 @@
-import { availableParallelism } from "node:os";
-import { getAppBuildAdapter } from "../build-adapter.js";
-import { ParallelBuildExecutor } from "./parallel-build-executor.js";
-import { DedupingBuildExecutor } from "./deduping-build-executor.js";
-import { SerializedBuildExecutor } from "./serialized-build-executor.js";
+import { availableParallelism } from 'node:os';
+import { getAppBuildAdapter } from '../build-adapter.js';
+import { ParallelBuildExecutor } from './parallel-build-executor.js';
+import { DedupingBuildExecutor } from './deduping-build-executor.js';
+import { SerializedBuildExecutor } from './serialized-build-executor.js';
 function resolveParallelismLimit() {
-  return Math.max(1, availableParallelism() - 1);
+    return Math.max(1, availableParallelism() - 1);
 }
 function patchAppRuntime(appConfig, patch) {
-  appConfig.runtime = {
-    ...appConfig.runtime ?? {},
-    ...patch
-  };
+    appConfig.runtime = {
+        ...(appConfig.runtime ?? {}),
+        ...patch,
+    };
 }
 class AppBuildRuntime {
-  serverEntryExecutor;
-  routeModuleExecutor;
-  hmrExecutor;
-  constructor(appConfig) {
-    const adapter = getAppBuildAdapter(appConfig);
-    const limit = resolveParallelismLimit();
-    this.serverEntryExecutor = new SerializedBuildExecutor(adapter);
-    this.routeModuleExecutor = new DedupingBuildExecutor(new ParallelBuildExecutor(adapter, limit));
-    this.hmrExecutor = new DedupingBuildExecutor(new ParallelBuildExecutor(adapter, Math.min(3, limit)));
-  }
-  getProfile(profile) {
-    switch (profile) {
-      case "server-entry":
-        return this.serverEntryExecutor;
-      case "route-module":
-        return this.routeModuleExecutor;
-      case "browser-hmr":
-        return this.hmrExecutor;
+    serverEntryExecutor;
+    routeModuleExecutor;
+    hmrExecutor;
+    constructor(appConfig) {
+        const adapter = getAppBuildAdapter(appConfig);
+        const limit = resolveParallelismLimit();
+        this.serverEntryExecutor = new SerializedBuildExecutor(adapter);
+        this.routeModuleExecutor = new DedupingBuildExecutor(new ParallelBuildExecutor(adapter, limit));
+        this.hmrExecutor = new DedupingBuildExecutor(new ParallelBuildExecutor(adapter, Math.min(3, limit)));
     }
-  }
-  /**
-   * No-op for profile teardown.
-   *
-   * @remarks
-   * Rolldown-owned profiles use one-shot builds, and Vite-host profiles only
-   * wrap a boundary marker. Neither owns a long-lived engine to close.
-   */
-  async dispose() {
-  }
+    getProfile(profile) {
+        switch (profile) {
+            case 'server-entry':
+                return this.serverEntryExecutor;
+            case 'route-module':
+                return this.routeModuleExecutor;
+            case 'browser-hmr':
+                return this.hmrExecutor;
+        }
+    }
+    /**
+     * No-op for profile teardown.
+     *
+     * @remarks
+     * Rolldown-owned profiles use one-shot builds, and Vite-host profiles only
+     * wrap a boundary marker. Neither owns a long-lived engine to close.
+     */
+    async dispose() { }
 }
-function getBuildRuntime(appConfig) {
-  return appConfig.runtime?.buildRuntime;
+export function getBuildRuntime(appConfig) {
+    return appConfig.runtime?.buildRuntime;
 }
-function requireBuildRuntime(appConfig) {
-  return getBuildRuntime(appConfig) ?? installBuildRuntime(appConfig);
+/**
+ * Returns the installed build runtime, installing it when missing.
+ */
+export function requireBuildRuntime(appConfig) {
+    return getBuildRuntime(appConfig) ?? installBuildRuntime(appConfig);
 }
-function installBuildRuntime(appConfig) {
-  const buildRuntime = new AppBuildRuntime(appConfig);
-  patchAppRuntime(appConfig, { buildRuntime });
-  return buildRuntime;
+/**
+ * Installs profile-based build executors for one app instance.
+ *
+ * @remarks
+ * For Rolldown ownership, server-entry stays serialized single-flight while
+ * route-module and browser-HMR run in parallel one-shot builds. Vite-host
+ * profiles wrap a boundary marker and reject framework-owned build attempts.
+ * Profiles do not inject app plugins — assemble complete
+ * {@link BuildOptions} with {@link createServerBuildRequest} /
+ * {@link createBrowserBuildRequest} (or {@link BrowserBundleService}) first.
+ */
+export function installBuildRuntime(appConfig) {
+    const buildRuntime = new AppBuildRuntime(appConfig);
+    patchAppRuntime(appConfig, { buildRuntime });
+    return buildRuntime;
 }
-async function disposeAppBuildRuntime(appConfig) {
-  await getBuildRuntime(appConfig)?.dispose();
+/**
+ * No-op retained for server teardown call sites.
+ */
+export async function disposeAppBuildRuntime(appConfig) {
+    await getBuildRuntime(appConfig)?.dispose();
 }
-export {
-  disposeAppBuildRuntime,
-  getBuildRuntime,
-  installBuildRuntime,
-  requireBuildRuntime
-};

package/src/build/runtime/deduping-build-executor.js

@@ -1,34 +1,40 @@
-import { createBuildRequestIdentity } from "./build-request-identity.js";
-import { createBuildRequestIdentity as createBuildRequestIdentity2, createBuildOptionsDedupeKey } from "./build-request-identity.js";
-class DedupingBuildExecutor {
-  inner;
-  inFlight = /* @__PURE__ */ new Map();
-  constructor(inner) {
-    this.inner = inner;
-  }
-  build(options) {
-    const key = createBuildRequestIdentity(options);
-    const existing = this.inFlight.get(key);
-    if (existing) {
-      return existing;
+import { createBuildRequestIdentity } from './build-request-identity.js';
+export { createBuildRequestIdentity, createBuildOptionsDedupeKey } from './build-request-identity.js';
+/**
+ * In-flight build coalescing wrapper.
+ *
+ * @remarks
+ * `ParallelBuildExecutor` allows concurrent builds but does not dedupe identical
+ * options. Multiple callers (route import, browser bundles, HMR) can hit the
+ * same Rolldown request before disk or import caches warm. This wrapper shares
+ * one inner `build()` promise per dedupe key until it settles.
+ *
+ * Wrap route-module and browser-hmr profiles only; server-entry stays serialized.
+ */
+export class DedupingBuildExecutor {
+    inner;
+    inFlight = new Map();
+    constructor(inner) {
+        this.inner = inner;
+    }
+    build(options) {
+        const key = createBuildRequestIdentity(options);
+        const existing = this.inFlight.get(key);
+        if (existing) {
+            return existing;
+        }
+        const buildPromise = this.inner.build(options).finally(() => {
+            if (this.inFlight.get(key) === buildPromise) {
+                this.inFlight.delete(key);
+            }
+        });
+        this.inFlight.set(key, buildPromise);
+        return buildPromise;
+    }
+    unwrap() {
+        return this.inner;
+    }
+    getInFlightCountForTests() {
+        return this.inFlight.size;
     }
-    const buildPromise = this.inner.build(options).finally(() => {
-      if (this.inFlight.get(key) === buildPromise) {
-        this.inFlight.delete(key);
-      }
-    });
-    this.inFlight.set(key, buildPromise);
-    return buildPromise;
-  }
-  unwrap() {
-    return this.inner;
-  }
-  getInFlightCountForTests() {
-    return this.inFlight.size;
-  }
 }
-export {
-  DedupingBuildExecutor,
-  createBuildOptionsDedupeKey,
-  createBuildRequestIdentity2 as createBuildRequestIdentity
-};

package/src/build/runtime/parallel-build-executor.js

@@ -1,52 +1,60 @@
-class ParallelBuildExecutor {
-  inner;
-  limit;
-  activeCount = 0;
-  queue = [];
-  constructor(inner, limit) {
-    this.inner = inner;
-    this.limit = Math.max(1, limit);
-  }
-  build(options) {
-    return new Promise((resolve, reject) => {
-      this.queue.push({
-        operation: () => this.inner.build(options),
-        resolve,
-        reject
-      });
-      this.drainQueue();
-    });
-  }
-  drainQueue() {
-    while (this.activeCount < this.limit && this.queue.length > 0) {
-      const next = this.queue.shift();
-      if (!next) {
-        return;
-      }
-      this.activeCount += 1;
-      void next.operation().then((result) => {
-        next.resolve(result);
-      }).catch((error) => {
-        next.reject(error);
-      }).finally(() => {
-        this.activeCount -= 1;
-        this.drainQueue();
-      });
+/**
+ * Concurrency-limited build executor wrapper.
+ *
+ * @remarks
+ * Runs up to `limit` builds concurrently. Use for independent route-module
+ * and HMR browser builds where FIFO serialization is unnecessary.
+ */
+export class ParallelBuildExecutor {
+    inner;
+    limit;
+    activeCount = 0;
+    queue = [];
+    constructor(inner, limit) {
+        this.inner = inner;
+        this.limit = Math.max(1, limit);
+    }
+    build(options) {
+        return new Promise((resolve, reject) => {
+            this.queue.push({
+                operation: () => this.inner.build(options),
+                resolve,
+                reject,
+            });
+            this.drainQueue();
+        });
+    }
+    drainQueue() {
+        while (this.activeCount < this.limit && this.queue.length > 0) {
+            const next = this.queue.shift();
+            if (!next) {
+                return;
+            }
+            this.activeCount += 1;
+            void next
+                .operation()
+                .then((result) => {
+                next.resolve(result);
+            })
+                .catch((error) => {
+                next.reject(error);
+            })
+                .finally(() => {
+                this.activeCount -= 1;
+                this.drainQueue();
+            });
+        }
+    }
+    unwrap() {
+        return this.inner;
+    }
+    getConcurrencyLimitForTests() {
+        return this.limit;
+    }
+    getActiveCountForTests() {
+        return this.activeCount;
+    }
+    getQueuedCountForTests() {
+        return this.queue.length;
     }
-  }
-  unwrap() {
-    return this.inner;
-  }
-  getConcurrencyLimitForTests() {
-    return this.limit;
-  }
-  getActiveCountForTests() {
-    return this.activeCount;
-  }
-  getQueuedCountForTests() {
-    return this.queue.length;
-  }
 }
-export {
-  ParallelBuildExecutor
-};