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

package/src/route-renderer/orchestration/ownership-graph/component-graph.js

@@ -1,96 +1,87 @@
-function walkComponentGraph(input) {
-  const seenComponents = /* @__PURE__ */ new Set();
-  const seenConfigs = /* @__PURE__ */ new Set();
-  const visitComponent = (component, parentIntegrationName, source) => {
-    if (!component) {
-      return;
-    }
-    const ecoComponent = component;
-    if (seenComponents.has(ecoComponent)) {
-      return;
-    }
-    seenComponents.add(ecoComponent);
-    const integrationName = ecoComponent.config?.integration ?? ecoComponent.config?.__eco?.integration ?? parentIntegrationName;
-    if (input.onConfig && ecoComponent.config && !seenConfigs.has(ecoComponent.config)) {
-      seenConfigs.add(ecoComponent.config);
-      input.onConfig(ecoComponent.config);
-    }
-    if (input.onComponent) {
-      const shouldContinue = input.onComponent({
-        component: ecoComponent,
-        parentIntegrationName,
-        source
-      });
-      if (shouldContinue === false) {
-        return;
-      }
-    }
-    if (input.visitLayout) {
-      for (const layout of ecoComponent.config?.layouts ?? []) {
-        if (layout?.config) {
-          visitConfig(layout.config, integrationName);
+export function walkComponentGraph(input) {
+    const seenComponents = new Set();
+    const seenConfigs = new Set();
+    const visitComponent = (component, parentIntegrationName, source) => {
+        if (!component) {
+            return;
         }
-      }
-    }
-    for (const child of ecoComponent.config?.dependencies?.components ?? []) {
-      visitComponent(child, integrationName, "dependency");
-    }
-  };
-  const visitConfig = (config, parentIntegrationName) => {
-    if (!config || seenConfigs.has(config)) {
-      return;
-    }
-    seenConfigs.add(config);
-    if (input.onConfig) {
-      input.onConfig(config);
-    }
-    if (input.visitLayout) {
-      for (const layout of config.layouts ?? []) {
-        if (layout?.config) {
-          visitConfig(layout.config, parentIntegrationName);
+        const ecoComponent = component;
+        if (seenComponents.has(ecoComponent)) {
+            return;
         }
-      }
-    }
-    for (const child of config.dependencies?.components ?? []) {
-      visitComponent(child, parentIntegrationName, "dependency");
+        seenComponents.add(ecoComponent);
+        const integrationName = ecoComponent.config?.integration ?? ecoComponent.config?.__eco?.integration ?? parentIntegrationName;
+        if (input.onConfig && ecoComponent.config && !seenConfigs.has(ecoComponent.config)) {
+            seenConfigs.add(ecoComponent.config);
+            input.onConfig(ecoComponent.config);
+        }
+        if (input.onComponent) {
+            const shouldContinue = input.onComponent({
+                component: ecoComponent,
+                parentIntegrationName,
+                source,
+            });
+            if (shouldContinue === false) {
+                return;
+            }
+        }
+        if (input.visitLayout) {
+            for (const layout of ecoComponent.config?.layouts ?? []) {
+                if (layout?.config) {
+                    visitConfig(layout.config, integrationName);
+                }
+            }
+        }
+        for (const child of ecoComponent.config?.dependencies?.components ?? []) {
+            visitComponent(child, integrationName, 'dependency');
+        }
+    };
+    const visitConfig = (config, parentIntegrationName) => {
+        if (!config || seenConfigs.has(config)) {
+            return;
+        }
+        seenConfigs.add(config);
+        if (input.onConfig) {
+            input.onConfig(config);
+        }
+        if (input.visitLayout) {
+            for (const layout of config.layouts ?? []) {
+                if (layout?.config) {
+                    visitConfig(layout.config, parentIntegrationName);
+                }
+            }
+        }
+        for (const child of config.dependencies?.components ?? []) {
+            visitComponent(child, parentIntegrationName, 'dependency');
+        }
+    };
+    for (const root of input.roots) {
+        visitComponent(root.component, input.currentIntegrationName, root.source);
     }
-  };
-  for (const root of input.roots) {
-    visitComponent(root.component, input.currentIntegrationName, root.source);
-  }
 }
-function mapComponentGraph(input) {
-  let nextSyntheticId = 0;
-  const mapComponent = (component, source, parentIntegrationName, lineage) => {
-    const integrationName = component.config?.integration ?? component.config?.__eco?.integration ?? parentIntegrationName;
-    const componentMeta = component.config?.__eco;
-    const isForeignToParent = integrationName !== parentIntegrationName;
-    const componentId = componentMeta?.id ?? componentMeta?.file ?? `${source}:${nextSyntheticId += 1}`;
-    const nextLineage = new Set(lineage);
-    nextLineage.add(component);
-    const children = (component.config?.dependencies?.components ?? []).flatMap((child) => {
-      if (!child || nextLineage.has(child)) {
-        return [];
-      }
-      return [mapComponent(child, "dependency", integrationName, nextLineage)];
-    });
-    return input.mapNode(
-      {
-        component,
-        source,
-        parentIntegrationName,
-        integrationName,
-        componentId,
-        isForeignToParent
-      },
-      children
-    );
-  };
-  return input.roots.map(
-    ({ component, source }) => mapComponent(component, source, input.currentIntegrationName, /* @__PURE__ */ new Set())
-  );
+export function mapComponentGraph(input) {
+    let nextSyntheticId = 0;
+    const mapComponent = (component, source, parentIntegrationName, lineage) => {
+        const integrationName = component.config?.integration ?? component.config?.__eco?.integration ?? parentIntegrationName;
+        const componentMeta = component.config?.__eco;
+        const isForeignToParent = integrationName !== parentIntegrationName;
+        const componentId = componentMeta?.id ?? componentMeta?.file ?? `${source}:${(nextSyntheticId += 1)}`;
+        const nextLineage = new Set(lineage);
+        nextLineage.add(component);
+        const children = (component.config?.dependencies?.components ?? []).flatMap((child) => {
+            if (!child || nextLineage.has(child)) {
+                return [];
+            }
+            return [mapComponent(child, 'dependency', integrationName, nextLineage)];
+        });
+        return input.mapNode({
+            component,
+            source,
+            parentIntegrationName,
+            integrationName,
+            componentId,
+            isForeignToParent,
+        }, children);
+    };
+    return input.roots.map(({ component, source }) => mapComponent(component, source, input.currentIntegrationName, new Set()));
 }
-export {
-  mapComponentGraph,
-  walkComponentGraph
-};

package/src/route-renderer/orchestration/ownership-graph/ownership-validation.service.js

@@ -1,80 +1,86 @@
-import { assertEcoDeclaredComponent } from "../../../eco/eco-declared-component.js";
-import { mapComponentGraph, walkComponentGraph } from "./component-graph.js";
-class OwnershipValidationService {
-  appConfig;
-  /**
-   * Creates the ownership validator for one finalized app config.
-   */
-  constructor(appConfig) {
-    this.appConfig = appConfig;
-  }
-  /**
-   * Validates foreign ownership edges reachable from the supplied route roots.
-   */
-  validate(input) {
-    this.assertDeclaredComponentDependencies(input);
-    return mapComponentGraph({
-      roots: input.roots,
-      currentIntegrationName: input.currentIntegrationName,
-      mapNode: ({ component, integrationName, componentId, isForeignToParent }, children) => {
-        const componentMeta = component.config?.__eco;
-        const errors = children.flat();
-        if (!isForeignToParent) {
-          return errors;
-        }
-        if (integrationName === "html") {
-          return errors;
-        }
-        if (!componentMeta) {
-          errors.push({
-            code: "MISSING_COMPONENT_METADATA",
-            message: `[ecopages] Foreign child "${componentId}" must provide stable __eco metadata so ownership diagnostics stay actionable. Declared dependencies must include all possible foreign children.`,
-            componentId,
-            integrationName
-          });
-        }
-        if (!this.isRegisteredIntegration(integrationName, input.currentIntegrationName)) {
-          errors.push({
-            code: "UNKNOWN_INTEGRATION_OWNER",
-            message: `[ecopages] Foreign child "${componentId}" references unknown integration owner "${integrationName}". Declared dependencies must include all possible foreign children and those integrations must be registered.`,
-            componentId,
-            componentFile: componentMeta?.file,
-            integrationName
-          });
-        }
-        return errors;
-      }
-    }).flat();
-  }
-  assertDeclaredComponentDependencies(input) {
-    walkComponentGraph({
-      roots: input.roots,
-      currentIntegrationName: input.currentIntegrationName,
-      onComponent: ({ component }) => {
-        const parentFile = component.config?.__eco?.file;
-        for (const child of component.config?.dependencies?.components ?? []) {
-          if (!child) {
-            continue;
-          }
-          assertEcoDeclaredComponent(child, { parentComponentFile: parentFile });
+import { assertEcoDeclaredComponent } from '../../../eco/eco-declared-component.js';
+import { mapComponentGraph, walkComponentGraph } from './component-graph.js';
+/**
+ * Validates foreign ownership as soon as the route root graph is loaded.
+ *
+ * This check runs before route data and dependency preparation so ownership and
+ * metadata problems are surfaced from the loaded component graph itself rather
+ * than being entangled with ownership-plan construction.
+ */
+export class OwnershipValidationService {
+    appConfig;
+    /**
+     * Creates the ownership validator for one finalized app config.
+     */
+    constructor(appConfig) {
+        this.appConfig = appConfig;
+    }
+    /**
+     * Validates foreign ownership edges reachable from the supplied route roots.
+     */
+    validate(input) {
+        this.assertDeclaredComponentDependencies(input);
+        return mapComponentGraph({
+            roots: input.roots,
+            currentIntegrationName: input.currentIntegrationName,
+            mapNode: ({ component, integrationName, componentId, isForeignToParent }, children) => {
+                const componentMeta = component.config?.__eco;
+                const errors = children.flat();
+                if (!isForeignToParent) {
+                    return errors;
+                }
+                if (integrationName === 'html') {
+                    return errors;
+                }
+                if (!componentMeta) {
+                    errors.push({
+                        code: 'MISSING_COMPONENT_METADATA',
+                        message: `[ecopages] Foreign child "${componentId}" must provide stable __eco metadata so ownership diagnostics stay actionable. Declared dependencies must include all possible foreign children.`,
+                        componentId,
+                        integrationName,
+                    });
+                }
+                if (!this.isRegisteredIntegration(integrationName, input.currentIntegrationName)) {
+                    errors.push({
+                        code: 'UNKNOWN_INTEGRATION_OWNER',
+                        message: `[ecopages] Foreign child "${componentId}" references unknown integration owner "${integrationName}". Declared dependencies must include all possible foreign children and those integrations must be registered.`,
+                        componentId,
+                        componentFile: componentMeta?.file,
+                        integrationName,
+                    });
+                }
+                return errors;
+            },
+        }).flat();
+    }
+    assertDeclaredComponentDependencies(input) {
+        walkComponentGraph({
+            roots: input.roots,
+            currentIntegrationName: input.currentIntegrationName,
+            onComponent: ({ component }) => {
+                const parentFile = component.config?.__eco?.file;
+                for (const child of component.config?.dependencies?.components ?? []) {
+                    if (!child) {
+                        continue;
+                    }
+                    assertEcoDeclaredComponent(child, { parentComponentFile: parentFile });
+                }
+            },
+        });
+    }
+    isRegisteredIntegration(integrationName, currentIntegrationName) {
+        if (integrationName === currentIntegrationName) {
+            return true;
         }
-      }
-    });
-  }
-  isRegisteredIntegration(integrationName, currentIntegrationName) {
-    if (integrationName === currentIntegrationName) {
-      return true;
+        return this.appConfig.integrations.some((integration) => integration.name === integrationName);
     }
-    return this.appConfig.integrations.some((integration) => integration.name === integrationName);
-  }
 }
-function throwIfOwnershipInvalid(validationErrors) {
-  if (validationErrors.length === 0) {
-    return;
-  }
-  throw new Error(validationErrors.map((error) => error.message).join("\n"));
+/**
+ * Throws when declared ownership validation collected one or more errors.
+ */
+export function throwIfOwnershipInvalid(validationErrors) {
+    if (validationErrors.length === 0) {
+        return;
+    }
+    throw new Error(validationErrors.map((error) => error.message).join('\n'));
 }
-export {
-  OwnershipValidationService,
-  throwIfOwnershipInvalid
-};

package/src/route-renderer/orchestration/page-browser-graph/global-injector-assets.service.js

@@ -1,32 +1,30 @@
-import { createRequire } from "node:module";
-import path from "node:path";
-import {
-  AssetFactory
-} from "../../../services/assets/asset-processing-service/index.js";
-import { buildGlobalInjectorBootstrapContent, buildGlobalInjectorMapScript } from "../../../eco/global-injector-map.js";
-async function buildGlobalInjectorAssets(appConfig, assetProcessingService, triggers, currentIntegrationName) {
-  const appProjectDir = appConfig.rootDir ?? appConfig.absolutePaths?.projectDir ?? process.cwd();
-  const appPackageRequire = createRequire(path.join(appProjectDir, "package.json"));
-  const corePackageEntryPath = appPackageRequire.resolve("@ecopages/core");
-  const globalInjectorImportPath = createRequire(corePackageEntryPath).resolve("@ecopages/scripts-injector/global");
-  const mapScript = AssetFactory.createInlineContentScript({
-    position: "head",
-    name: "ecopages-global-injector-map",
-    content: buildGlobalInjectorMapScript(triggers),
-    attributes: { type: "ecopages/global-injector-map" },
-    packageRole: "keep-separate",
-    bundle: false
-  });
-  const bootstrapInlineScript = AssetFactory.createInlineContentScript({
-    position: "head",
-    name: "ecopages-global-injector-bootstrap",
-    content: buildGlobalInjectorBootstrapContent(globalInjectorImportPath),
-    attributes: { type: "module" },
-    packageRole: "keep-separate",
-    bundle: true
-  });
-  return assetProcessingService.processDependencies([mapScript, bootstrapInlineScript], currentIntegrationName);
+import { createRequire } from 'node:module';
+import path from 'node:path';
+import { AssetFactory, } from '../../../services/assets/asset-processing-service/index.js';
+import { buildGlobalInjectorBootstrapContent, buildGlobalInjectorMapScript } from '../../../eco/global-injector-map.js';
+/**
+ * Builds processed global-injector map and bootstrap assets for resolved lazy triggers.
+ */
+export async function buildGlobalInjectorAssets(appConfig, assetProcessingService, triggers, currentIntegrationName) {
+    const appProjectDir = appConfig.rootDir ?? appConfig.absolutePaths?.projectDir ?? process.cwd();
+    const appPackageRequire = createRequire(path.join(appProjectDir, 'package.json'));
+    const corePackageEntryPath = appPackageRequire.resolve('@ecopages/core');
+    const globalInjectorImportPath = createRequire(corePackageEntryPath).resolve('@ecopages/scripts-injector/global');
+    const mapScript = AssetFactory.createInlineContentScript({
+        position: 'head',
+        name: 'ecopages-global-injector-map',
+        content: buildGlobalInjectorMapScript(triggers),
+        attributes: { type: 'ecopages/global-injector-map' },
+        packageRole: 'keep-separate',
+        bundle: false,
+    });
+    const bootstrapInlineScript = AssetFactory.createInlineContentScript({
+        position: 'head',
+        name: 'ecopages-global-injector-bootstrap',
+        content: buildGlobalInjectorBootstrapContent(globalInjectorImportPath),
+        attributes: { type: 'module' },
+        packageRole: 'keep-separate',
+        bundle: true,
+    });
+    return assetProcessingService.processDependencies([mapScript, bootstrapInlineScript], currentIntegrationName);
 }
-export {
-  buildGlobalInjectorAssets
-};

package/src/route-renderer/orchestration/page-browser-graph/page-browser-graph-contribution.loader.js

@@ -1,7 +1,11 @@
-async function loadPageBrowserGraphContribution(routeFile, importPageFile, collectContribution) {
-  const pageModule = await importPageFile(routeFile);
-  return collectContribution({ file: routeFile, pageModule });
+/**
+ * Loads a page module once and collects declarative browser-graph requirements.
+ *
+ * @remarks
+ * Shared by route orchestration prep and the integration route-render adapter so
+ * page imports are not duplicated when both paths need the same contribution.
+ */
+export async function loadPageBrowserGraphContribution(routeFile, importPageFile, collectContribution) {
+    const pageModule = await importPageFile(routeFile);
+    return collectContribution({ file: routeFile, pageModule });
 }
-export {
-  loadPageBrowserGraphContribution
-};

package/src/route-renderer/orchestration/page-browser-graph/page-browser-graph-merge.utils.js

@@ -1,29 +1,32 @@
-import { createPagePackage } from "../../../services/assets/asset-processing-service/index.js";
-import { buildProcessedAssetDedupeKey, dedupeProcessedAssets } from "./processed-asset-dedupe.js";
-function mergePageBrowserGraph(currentPagePackage, processedDependencies, pageBrowserGraph) {
-  const mergedGraph = currentPagePackage?.pageBrowserGraph ? {
-    entryAssets: dedupeProcessedAssets([
-      ...currentPagePackage.pageBrowserGraph.entryAssets,
-      ...pageBrowserGraph.entryAssets
-    ]),
-    chunkAssets: dedupeProcessedAssets([
-      ...currentPagePackage.pageBrowserGraph.chunkAssets,
-      ...pageBrowserGraph.chunkAssets
-    ])
-  } : pageBrowserGraph;
-  const pageBrowserGraphAssetKeys = new Set(
-    [...mergedGraph.entryAssets, ...mergedGraph.chunkAssets].map((asset) => buildProcessedAssetDedupeKey(asset))
-  );
-  const baseAssets = currentPagePackage ? currentPagePackage.assets.filter(
-    (asset) => !pageBrowserGraphAssetKeys.has(buildProcessedAssetDedupeKey(asset))
-  ) : processedDependencies;
-  return {
-    pagePackage: createPagePackage(dedupeProcessedAssets(baseAssets), {
-      pageBrowserGraph: mergedGraph
-    }),
-    mergedGraph
-  };
+import { createPagePackage } from '../../../services/assets/asset-processing-service/index.js';
+import { buildProcessedAssetDedupeKey, dedupeProcessedAssets } from './processed-asset-dedupe.js';
+/**
+ * Merges a page browser graph into an existing page package (or processed dependencies).
+ *
+ * Deduplicates entry/chunk assets against the existing package and rebuilds the
+ * package so downstream HTML finalization sees one canonical graph.
+ */
+export function mergePageBrowserGraph(currentPagePackage, processedDependencies, pageBrowserGraph) {
+    const mergedGraph = currentPagePackage?.pageBrowserGraph
+        ? {
+            entryAssets: dedupeProcessedAssets([
+                ...currentPagePackage.pageBrowserGraph.entryAssets,
+                ...pageBrowserGraph.entryAssets,
+            ]),
+            chunkAssets: dedupeProcessedAssets([
+                ...currentPagePackage.pageBrowserGraph.chunkAssets,
+                ...pageBrowserGraph.chunkAssets,
+            ]),
+        }
+        : pageBrowserGraph;
+    const pageBrowserGraphAssetKeys = new Set([...mergedGraph.entryAssets, ...mergedGraph.chunkAssets].map((asset) => buildProcessedAssetDedupeKey(asset)));
+    const baseAssets = currentPagePackage
+        ? currentPagePackage.assets.filter((asset) => !pageBrowserGraphAssetKeys.has(buildProcessedAssetDedupeKey(asset)))
+        : processedDependencies;
+    return {
+        pagePackage: createPagePackage(dedupeProcessedAssets(baseAssets), {
+            pageBrowserGraph: mergedGraph,
+        }),
+        mergedGraph,
+    };
 }
-export {
-  mergePageBrowserGraph
-};

package/src/route-renderer/orchestration/page-browser-graph/page-browser-graph-session.d.ts

@@ -62,6 +62,9 @@ export declare class SessionPageBrowserGraphCache {
     private readonly groupedGenerations;
     private buildCount;
     getGraphByRoute(integrationName: string, routeFile: string, policy: GraphPolicy): PageBrowserGraphResult | undefined;
+    /**
+     * Page Browser Graph compilations in this dev-server session.
+     */
     getBuildCount(): number;
     getAffectedGraphIdentities(filePath: string): AffectedGraphIdentity[];
     resolveGraph(key: GraphKey, build: () => Promise<GraphBuildResult | undefined>): Promise<PageBrowserGraphResult | undefined>;