@ecopages/core 0.2.0-alpha.3 → 0.2.0-alpha.31

package/src/adapters/bun/hmr-manager.d.ts

@@ -1,27 +1,42 @@
 import type { WebSocketHandler } from 'bun';
-import type { DefaultHmrContext, EcoPagesAppConfig, IHmrManager } from '../../internal-types';
+import type { DefaultHmrContext, EcoPagesAppConfig, IHmrManager } from '../../types/internal-types.js';
 import type { EcoBuildPlugin } from '../../build/build-types.js';
-import type { HmrStrategy } from '../../hmr/hmr-strategy';
-import type { ClientBridge } from './client-bridge';
-import type { ClientBridgeEvent } from '../../public-types';
+import { type HmrStrategy } from '../../hmr/hmr-strategy.js';
+import type { ClientBridge } from './client-bridge.js';
+import type { ClientBridgeEvent } from '../../types/public-types.js';
 type BunSocketHandler = WebSocketHandler<unknown>;
 export interface HmrManagerParams {
     appConfig: EcoPagesAppConfig;
     bridge: ClientBridge;
 }
+type HandleFileChangeOptions = {
+    broadcast?: boolean;
+};
+/**
+ * Bun development HMR manager.
+ *
+ * @remarks
+ * Bun shares the same public contract as the Node manager: page entrypoints are
+ * strict integration-owned registrations, while generic script assets use their
+ * own explicit registration path.
+ */
 export declare class HmrManager implements IHmrManager {
+    private static readonly entrypointRegistrationTimeoutMs;
     readonly appConfig: EcoPagesAppConfig;
     private readonly bridge;
     /** Keep track of watchers */
     private watchers;
     /** entrypoint -> output path */
     private watchedFiles;
-    /** bare specifier -> runtime URL (e.g., 'react' -> '/assets/vendors/react.js') */
-    private specifierMap;
+    private entrypointRegistrations;
     private distDir;
     private plugins;
     private enabled;
     private strategies;
+    private readonly entrypointRegistrar;
+    private readonly browserBundleService;
+    private readonly entrypointDependencyGraph;
+    private readonly serverModuleTranspiler;
     private wsHandler;
     constructor({ appConfig, bridge }: HmrManagerParams);
     /**
@@ -31,6 +46,15 @@ export declare class HmrManager implements IHmrManager {
      * can share the same dist path during e2e runs.
      */
     private cleanDistDir;
+    /**
+     * Returns whether the generic JS strategy may rebuild an entrypoint.
+     *
+     * @remarks
+     * Integration-owned page entrypoints are excluded so a shared dependency
+     * invalidation cannot replace framework-owned browser output with a generic JS
+     * rebuild.
+     */
+    private shouldJsStrategyProcessEntrypoint;
     /**
      * Initializes core HMR strategies.
      * Strategies are evaluated in priority order (highest first).
@@ -45,12 +69,6 @@ export declare class HmrManager implements IHmrManager {
     setPlugins(plugins: EcoBuildPlugin[]): void;
     setEnabled(enabled: boolean): void;
     isEnabled(): boolean;
-    /**
-     * Registers a mapping from bare specifiers to vendor URLs.
-     * Used by integrations to provide their module resolution mappings.
-     * @param map - Object mapping bare specifiers to vendor URLs
-     */
-    registerSpecifierMap(map: Record<string, string>): void;
     getWebSocketHandler(): BunSocketHandler;
     /**
      * Builds the client-side HMR runtime script.
@@ -58,28 +76,56 @@ export declare class HmrManager implements IHmrManager {
     buildRuntime(): Promise<void>;
     getRuntimePath(): string;
     broadcast(event: ClientBridgeEvent): void;
-    canHandleFileChange(filePath: string): boolean;
-    /**
-     * Handles file changes using registered HMR strategies.
-     * Strategies are evaluated in priority order until one matches.
-     * @param filePath - Absolute path to the changed file
-     */
-    handleFileChange(filePath: string): Promise<void>;
-    /**
-     * Registers a client entrypoint to be built and watched by Bun.
-     */
+    handleFileChange(filePath: string, options?: HandleFileChangeOptions): Promise<void>;
     getOutputUrl(entrypointPath: string): string | undefined;
     getWatchedFiles(): Map<string, string>;
-    getSpecifierMap(): Map<string, string>;
     getDistDir(): string;
     getPlugins(): EcoBuildPlugin[];
     getDefaultContext(): DefaultHmrContext;
+    private clearFailedEntrypointRegistration;
+    /**
+     * Registers one integration-owned page entrypoint.
+     *
+     * @remarks
+     * Concurrent callers share one in-flight registration. The registration is
+     * cleared from the dedupe map when it settles so later callers cannot inherit a
+     * stale promise.
+     */
     registerEntrypoint(entrypointPath: string): Promise<string>;
     /**
-     * Encodes dynamic route segments (brackets) in file paths.
-     * Converts `[slug]` to `_slug_` to avoid filesystem/URL issues.
+     * Registers one generic script entrypoint.
+     *
+     * @remarks
+     * This explicit path keeps the page-entrypoint contract strict while still
+     * allowing generic script assets to use the fallback build path.
+     */
+    registerScriptEntrypoint(entrypointPath: string): Promise<string>;
+    /**
+     * Performs strict integration-owned registration for one normalized path.
+     *
+     * @remarks
+     * The manager reserves the output URL, removes any stale emitted file, runs
+     * strategy processing without broadcasting, and then verifies that the owning
+     * integration emitted the expected file.
+     */
+    private emitStrictEntrypoint;
+    /**
+     * Performs registration for a generic script asset.
+     *
+     * @remarks
+     * This path performs a targeted browser bundle for the requested script
+     * entrypoint only. The resulting dependency graph is retained so later file
+     * changes can invalidate just the affected script entrypoints.
+     */
+    private emitScriptEntrypoint;
+    /**
+     * Stops active watchers and releases retained registration state.
+     *
+     * @remarks
+     * Emitted `_hmr` files remain on disk because parallel app processes may share
+     * the same dist directory. The in-memory indexes are cleared so stale
+     * entrypoints cannot leak through a reused manager object.
      */
-    private encodeDynamicSegments;
     stop(): void;
 }
 export {};

package/src/adapters/bun/hmr-manager.js

@@ -1,30 +1,56 @@
 import fs from "node:fs";
 import path from "node:path";
-import { RESOLVED_ASSETS_DIR } from "../../constants";
-import { defaultBuildAdapter } from "../../build/build-adapter.js";
+import { RESOLVED_ASSETS_DIR } from "../../config/constants.js";
+import { getAppBuildExecutor } from "../../build/build-adapter.js";
 import { fileSystem } from "@ecopages/file-system";
-import { HmrStrategyType } from "../../hmr/hmr-strategy";
-import { DefaultHmrStrategy } from "../../hmr/strategies/default-hmr-strategy";
-import { JsHmrStrategy } from "../../hmr/strategies/js-hmr-strategy";
-import { appLogger } from "../../global/app-logger";
+import { HmrStrategyType } from "../../hmr/hmr-strategy.js";
+import { DefaultHmrStrategy } from "../../hmr/strategies/default-hmr-strategy.js";
+import { JsHmrStrategy } from "../../hmr/strategies/js-hmr-strategy.js";
+import { appLogger } from "../../global/app-logger.js";
+import { HmrEntrypointRegistrar } from "../shared/hmr-entrypoint-registrar.js";
+import { BrowserBundleService } from "../../services/assets/browser-bundle.service.js";
+import { getAppServerModuleTranspiler } from "../../services/module-loading/app-server-module-transpiler.service.js";
+import {
+  getAppEntrypointDependencyGraph,
+  NoopEntrypointDependencyGraph,
+  setAppEntrypointDependencyGraph
+} from "../../services/runtime-state/entrypoint-dependency-graph.service.js";
+import { resolveInternalExecutionDir, resolveInternalWorkDir } from "../../utils/resolve-work-dir.js";
 class HmrManager {
+  static entrypointRegistrationTimeoutMs = 4e3;
   appConfig;
   bridge;
   /** Keep track of watchers */
   watchers = /* @__PURE__ */ new Map();
   /** entrypoint -> output path */
   watchedFiles = /* @__PURE__ */ new Map();
-  /** bare specifier -> runtime URL (e.g., 'react' -> '/assets/vendors/react.js') */
-  specifierMap = /* @__PURE__ */ new Map();
+  entrypointRegistrations = /* @__PURE__ */ new Map();
   distDir;
   plugins = [];
   enabled = true;
   strategies = [];
+  entrypointRegistrar;
+  browserBundleService;
+  entrypointDependencyGraph;
+  serverModuleTranspiler;
   wsHandler;
   constructor({ appConfig, bridge }) {
     this.appConfig = appConfig;
     this.bridge = bridge;
-    this.distDir = path.join(this.appConfig.absolutePaths.distDir, RESOLVED_ASSETS_DIR, "_hmr");
+    this.distDir = path.join(resolveInternalWorkDir(this.appConfig), RESOLVED_ASSETS_DIR, "_hmr");
+    this.entrypointRegistrar = new HmrEntrypointRegistrar({
+      srcDir: this.appConfig.absolutePaths.srcDir,
+      distDir: this.distDir,
+      entrypointRegistrations: this.entrypointRegistrations,
+      watchedFiles: this.watchedFiles,
+      clearFailedRegistration: (entrypointPath) => this.clearFailedEntrypointRegistration(entrypointPath),
+      registrationTimeoutMs: HmrManager.entrypointRegistrationTimeoutMs
+    });
+    this.browserBundleService = new BrowserBundleService(appConfig);
+    const existingEntrypointDependencyGraph = getAppEntrypointDependencyGraph(appConfig);
+    this.entrypointDependencyGraph = existingEntrypointDependencyGraph instanceof NoopEntrypointDependencyGraph ? existingEntrypointDependencyGraph : new NoopEntrypointDependencyGraph();
+    setAppEntrypointDependencyGraph(this.appConfig, this.entrypointDependencyGraph);
+    this.serverModuleTranspiler = getAppServerModuleTranspiler(this.appConfig);
     this.cleanDistDir();
     this.initializeStrategies();
   }
@@ -37,6 +63,27 @@ class HmrManager {
   cleanDistDir() {
     fileSystem.ensureDir(this.distDir);
   }
+  /**
+   * Returns whether the generic JS strategy may rebuild an entrypoint.
+   *
+   * @remarks
+   * Integration-owned page entrypoints are excluded so a shared dependency
+   * invalidation cannot replace framework-owned browser output with a generic JS
+   * rebuild.
+   */
+  shouldJsStrategyProcessEntrypoint(entrypointPath) {
+    return !this.strategies.some((strategy) => {
+      if (strategy.type !== HmrStrategyType.INTEGRATION || strategy.priority <= HmrStrategyType.SCRIPT) {
+        return false;
+      }
+      try {
+        return strategy.matches(entrypointPath);
+      } catch (error) {
+        appLogger.error(error);
+        return false;
+      }
+    });
+  }
   /**
    * Initializes core HMR strategies.
    * Strategies are evaluated in priority order (highest first).
@@ -44,10 +91,15 @@ class HmrManager {
   initializeStrategies() {
     const jsContext = {
       getWatchedFiles: () => this.watchedFiles,
-      getSpecifierMap: () => this.specifierMap,
       getDistDir: () => this.distDir,
       getPlugins: () => this.plugins,
-      getSrcDir: () => this.appConfig.absolutePaths.srcDir
+      getSrcDir: () => this.appConfig.absolutePaths.srcDir,
+      getPagesDir: () => this.appConfig.absolutePaths.pagesDir,
+      getLayoutsDir: () => this.appConfig.absolutePaths.layoutsDir,
+      getTemplateExtensions: () => this.appConfig.templatesExt,
+      getBrowserBundleService: () => this.browserBundleService,
+      getEntrypointDependencyGraph: () => this.entrypointDependencyGraph,
+      shouldProcessEntrypoint: (entrypointPath) => this.shouldJsStrategyProcessEntrypoint(entrypointPath)
     };
     this.strategies = [new JsHmrStrategy(jsContext), new DefaultHmrStrategy()];
   }
@@ -68,16 +120,6 @@ class HmrManager {
   isEnabled() {
     return this.enabled;
   }
-  /**
-   * Registers a mapping from bare specifiers to vendor URLs.
-   * Used by integrations to provide their module resolution mappings.
-   * @param map - Object mapping bare specifiers to vendor URLs
-   */
-  registerSpecifierMap(map) {
-    for (const [specifier, url] of Object.entries(map)) {
-      this.specifierMap.set(specifier, url);
-    }
-  }
   getWebSocketHandler() {
     const open = (ws) => {
       this.bridge.subscribe(ws);
@@ -101,12 +143,12 @@ class HmrManager {
    */
   async buildRuntime() {
     const runtimeSource = path.resolve(import.meta.dirname, "../../hmr/client/hmr-runtime.js");
-    const result = await defaultBuildAdapter.build({
+    const result = await this.browserBundleService.bundle({
+      profile: "hmr-runtime",
       entrypoints: [runtimeSource],
       outdir: this.distDir,
       naming: "_hmr_runtime.js",
       minify: false,
-      ...defaultBuildAdapter.getTranspileOptions("hmr-runtime"),
       plugins: this.plugins
     });
     if (!result.success) {
@@ -122,30 +164,13 @@ class HmrManager {
     );
     this.bridge.broadcast(event);
   }
-  canHandleFileChange(filePath) {
-    const sorted = [...this.strategies].sort((a, b) => b.priority - a.priority);
-    const strategy = sorted.find((candidate) => {
-      try {
-        return candidate.matches(filePath);
-      } catch (err) {
-        appLogger.error(`[HmrManager] Error checking match for ${candidate.constructor.name}:`, err);
-        return false;
-      }
-    });
-    return strategy !== void 0 && strategy.type !== HmrStrategyType.FALLBACK;
-  }
-  /**
-   * Handles file changes using registered HMR strategies.
-   * Strategies are evaluated in priority order until one matches.
-   * @param filePath - Absolute path to the changed file
-   */
-  async handleFileChange(filePath) {
+  async handleFileChange(filePath, options = {}) {
     const sorted = [...this.strategies].sort((a, b) => b.priority - a.priority);
     const strategy = sorted.find((s) => {
       try {
         return s.matches(filePath);
       } catch (err) {
-        appLogger.error(`[HmrManager] Error checking match for ${s.constructor.name}:`, err);
+        appLogger.error(err);
         return false;
       }
     });
@@ -155,7 +180,8 @@ class HmrManager {
     }
     appLogger.debug(`[HmrManager] Selected strategy: ${strategy.constructor.name}`);
     const action = await strategy.process(filePath);
-    if (action.type === "broadcast") {
+    const shouldBroadcast = options.broadcast ?? true;
+    if (shouldBroadcast && action.type === "broadcast") {
       if (action.events) {
         for (const event of action.events) {
           this.broadcast(event);
@@ -163,18 +189,12 @@ class HmrManager {
       }
     }
   }
-  /**
-   * Registers a client entrypoint to be built and watched by Bun.
-   */
   getOutputUrl(entrypointPath) {
     return this.watchedFiles.get(entrypointPath);
   }
   getWatchedFiles() {
     return this.watchedFiles;
   }
-  getSpecifierMap() {
-    return this.specifierMap;
-  }
   getDistDir() {
     return this.distDir;
   }
@@ -184,55 +204,107 @@ class HmrManager {
   getDefaultContext() {
     return {
       getWatchedFiles: () => this.watchedFiles,
-      getSpecifierMap: () => this.specifierMap,
       getDistDir: () => this.distDir,
       getPlugins: () => this.plugins,
       getSrcDir: () => this.appConfig.absolutePaths.srcDir,
       getLayoutsDir: () => this.appConfig.absolutePaths.layoutsDir,
-      getPagesDir: () => this.appConfig.absolutePaths.pagesDir
+      getPagesDir: () => this.appConfig.absolutePaths.pagesDir,
+      getBuildExecutor: () => getAppBuildExecutor(this.appConfig),
+      getBrowserBundleService: () => this.browserBundleService,
+      importServerModule: async (filePath) => await this.serverModuleTranspiler.importModule({
+        filePath,
+        outdir: path.join(resolveInternalExecutionDir(this.appConfig), ".server-modules"),
+        externalPackages: true
+      })
     };
   }
+  clearFailedEntrypointRegistration(entrypointPath) {
+    this.watchedFiles.delete(entrypointPath);
+  }
+  /**
+   * Registers one integration-owned page entrypoint.
+   *
+   * @remarks
+   * Concurrent callers share one in-flight registration. The registration is
+   * cleared from the dedupe map when it settles so later callers cannot inherit a
+   * stale promise.
+   */
   async registerEntrypoint(entrypointPath) {
-    if (this.watchedFiles.has(entrypointPath)) {
-      return this.watchedFiles.get(entrypointPath);
+    return await this.entrypointRegistrar.registerEntrypoint(entrypointPath, {
+      emit: async (normalizedEntrypoint) => await this.emitStrictEntrypoint(normalizedEntrypoint),
+      getMissingOutputError: (normalizedEntrypoint, outputPath) => new Error(
+        `[HMR] Integration failed to emit entrypoint ${normalizedEntrypoint} to ${outputPath}. Page entrypoints must be produced by their owning integration.`
+      )
+    });
+  }
+  /**
+   * Registers one generic script entrypoint.
+   *
+   * @remarks
+   * This explicit path keeps the page-entrypoint contract strict while still
+   * allowing generic script assets to use the fallback build path.
+   */
+  async registerScriptEntrypoint(entrypointPath) {
+    return await this.entrypointRegistrar.registerEntrypoint(entrypointPath, {
+      emit: async (normalizedEntrypoint, outputPath) => await this.emitScriptEntrypoint(normalizedEntrypoint, outputPath),
+      getMissingOutputError: (normalizedEntrypoint) => new Error(`[HMR] Failed to register script entrypoint: ${normalizedEntrypoint}`)
+    });
+  }
+  /**
+   * Performs strict integration-owned registration for one normalized path.
+   *
+   * @remarks
+   * The manager reserves the output URL, removes any stale emitted file, runs
+   * strategy processing without broadcasting, and then verifies that the owning
+   * integration emitted the expected file.
+   */
+  async emitStrictEntrypoint(entrypointPath) {
+    await this.handleFileChange(entrypointPath, { broadcast: false });
+  }
+  /**
+   * Performs registration for a generic script asset.
+   *
+   * @remarks
+   * This path performs a targeted browser bundle for the requested script
+   * entrypoint only. The resulting dependency graph is retained so later file
+   * changes can invalidate just the affected script entrypoints.
+   */
+  async emitScriptEntrypoint(entrypointPath, outputPath) {
+    const naming = path.relative(this.distDir, outputPath).split(path.sep).join("/");
+    const buildResult = await this.browserBundleService.bundle({
+      profile: "hmr-entrypoint",
+      entrypoints: [entrypointPath],
+      outdir: this.distDir,
+      naming,
+      minify: false,
+      plugins: this.plugins
+    });
+    if (!buildResult.success) {
+      appLogger.error(`[HMR] Generic script entrypoint build failed for ${entrypointPath}:`, buildResult.logs);
+      return;
     }
-    const srcDir = this.appConfig.absolutePaths.srcDir;
-    const relativePath = path.relative(srcDir, entrypointPath);
-    const relativePathJs = relativePath.replace(/\.(tsx?|jsx?|mdx?)$/, ".js");
-    const encodedPathJs = this.encodeDynamicSegments(relativePathJs);
-    const urlPath = encodedPathJs.split(path.sep).join("/");
-    const outputUrl = `/${path.join(RESOLVED_ASSETS_DIR, "_hmr", urlPath)}`;
-    const outputPath = path.join(this.distDir, urlPath);
-    this.watchedFiles.set(entrypointPath, outputUrl);
-    await this.handleFileChange(entrypointPath);
-    if (!fileSystem.exists(outputPath)) {
-      const fallback = await defaultBuildAdapter.build({
-        entrypoints: [entrypointPath],
-        outdir: this.distDir,
-        naming: encodedPathJs,
-        minify: false,
-        external: Array.from(this.specifierMap.keys()),
-        ...defaultBuildAdapter.getTranspileOptions("hmr-entrypoint"),
-        plugins: this.plugins
-      });
-      if (!fallback.success) {
-        appLogger.error(`[HMR] Fallback build failed for ${entrypointPath}:`, fallback.logs);
-      }
+    const entrypointDependencies = buildResult.dependencyGraph?.entrypoints?.[entrypointPath];
+    if (entrypointDependencies) {
+      this.entrypointDependencyGraph.setEntrypointDependencies(entrypointPath, entrypointDependencies);
     }
-    return outputUrl;
   }
   /**
-   * Encodes dynamic route segments (brackets) in file paths.
-   * Converts `[slug]` to `_slug_` to avoid filesystem/URL issues.
+   * Stops active watchers and releases retained registration state.
+   *
+   * @remarks
+   * Emitted `_hmr` files remain on disk because parallel app processes may share
+   * the same dist directory. The in-memory indexes are cleared so stale
+   * entrypoints cannot leak through a reused manager object.
    */
-  encodeDynamicSegments(filepath) {
-    return filepath.replace(/\[([^\]]+)\]/g, "_$1_");
-  }
   stop() {
+    this.entrypointRegistrations.clear();
     for (const watcher of this.watchers.values()) {
       watcher.close();
     }
     this.watchers.clear();
+    this.watchedFiles.clear();
+    this.entrypointDependencyGraph.reset();
+    this.plugins = [];
   }
 }
 export {

package/src/adapters/bun/index.d.ts

@@ -1,3 +1,2 @@
-export { EcopagesApp, createApp } from './create-app.js';
-export { defineApiHandler, defineGroupHandler } from './define-api-handler.js';
-export type { GroupHandler } from './define-api-handler.js';
+export { BunEcopagesApp, createApp } from './create-app.js';
+export { defineApiHandler, defineGroupHandler, type GroupHandler } from '../shared/define-api-handler.js';

package/src/adapters/bun/index.js

@@ -1,7 +1,7 @@
-import { EcopagesApp, createApp } from "./create-app.js";
-import { defineApiHandler, defineGroupHandler } from "./define-api-handler.js";
+import { BunEcopagesApp, createApp } from "./create-app.js";
+import { defineApiHandler, defineGroupHandler } from "../shared/define-api-handler.js";
 export {
-  EcopagesApp,
+  BunEcopagesApp,
   createApp,
   defineApiHandler,
   defineGroupHandler

package/src/adapters/bun/server-adapter.d.ts

@@ -1,12 +1,12 @@
 import type { Server, WebSocketHandler } from 'bun';
-import type { EcoPagesAppConfig } from '../../internal-types.js';
-import type { ApiHandler, ErrorHandler, StaticRoute } from '../../public-types.js';
+import type { EcoPagesAppConfig } from '../../types/internal-types.js';
+import type { ApiHandler, ErrorHandler, StaticRoute } from '../../types/public-types.js';
 import { SharedServerAdapter } from '../shared/server-adapter.js';
 import type { ServerAdapterResult } from '../abstract/server-adapter.js';
 import { ServerRouteHandler, type ServerRouteHandlerParams } from '../shared/server-route-handler.js';
-import { ServerStaticBuilder, type ServerStaticBuilderParams } from '../shared/server-static-builder';
-import { ClientBridge } from './client-bridge';
-import { HmrManager } from './hmr-manager';
+import { ServerStaticBuilder, type ServerStaticBuilderParams } from '../shared/server-static-builder.js';
+import { ClientBridge } from './client-bridge.js';
+import { HmrManager } from './hmr-manager.js';
 import { ServerLifecycle } from './server-lifecycle.js';
 type BunServerInstance = Server<unknown>;
 type BunNativeServeOptions = Bun.Serve.Options<unknown>;