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

package/CHANGELOG.md

@@ -65,6 +65,8 @@ All notable changes to `@ecopages/core` are documented here.
 - Fixed invariant checks for route paths with improved error messaging in `AbstractApplicationAdapter` (`9c2a6242`).
 - Fixed dependency import name extraction in `extractEcopagesVirtualImports` (`39bbc472`).
 - Removed an invalid npm export entry that pointed to a non-existent `utils/ecopages-url-resolver` declaration target.
+- Kept source module HMR active when stylesheet processors also watch TSX and JSX files for Tailwind-driven CSS rebuilds.
+- Triggered HMR current-page refreshes instead of fallback reload suppression for processor-watched TSX and JSX changes that are not client entrypoints.
 
 ### Tests
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ecopages/core",
-  "version": "0.2.0-alpha.1",
+  "version": "0.2.0-alpha.3",
   "description": "Core package for Ecopages",
   "keywords": [
     "ecopages",
@@ -17,7 +17,7 @@
     "directory": "packages/core"
   },
   "dependencies": {
-    "@ecopages/file-system": "0.2.0-alpha.1",
+    "@ecopages/file-system": "0.2.0-alpha.3",
     "@ecopages/logger": "latest",
     "@ecopages/scripts-injector": "^0.1.3",
     "@worker-tools/html-rewriter": "0.1.0-pre.19",

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

@@ -58,6 +58,7 @@ 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.

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

@@ -3,6 +3,7 @@ import path from "node:path";
 import { RESOLVED_ASSETS_DIR } from "../../constants";
 import { defaultBuildAdapter } 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";
@@ -121,6 +122,18 @@ 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.

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

@@ -7,6 +7,7 @@ import type { DefaultHmrContext, EcoPagesAppConfig, IHmrManager } from '../../in
 import type { EcoBuildPlugin } from '../../build/build-types.ts';
 import { fileSystem } from '@ecopages/file-system';
 import type { HmrStrategy } from '../../hmr/hmr-strategy';
+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';
@@ -158,6 +159,20 @@ export class HmrManager implements IHmrManager {
 		this.bridge.broadcast(event);
 	}
 
+	public canHandleFileChange(filePath: string): boolean {
+		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 as Error);
+				return false;
+			}
+		});
+
+		return strategy !== undefined && strategy.type !== HmrStrategyType.FALLBACK;
+	}
+
 	/**
 	 * Handles file changes using registered HMR strategies.
 	 * Strategies are evaluated in priority order until one matches.

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

@@ -41,6 +41,7 @@ export declare class NodeHmrManager implements IHmrManager {
     buildRuntime(): Promise<void>;
     getRuntimePath(): string;
     broadcast(event: ClientBridgeEvent): void;
+    canHandleFileChange(filePath: string): boolean;
     handleFileChange(filePath: string): Promise<void>;
     getOutputUrl(entrypointPath: string): string | undefined;
     getWatchedFiles(): Map<string, string>;

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

@@ -4,6 +4,7 @@ import { fileURLToPath } from "node:url";
 import { RESOLVED_ASSETS_DIR } from "../../constants.js";
 import { defaultBuildAdapter } from "../../build/build-adapter.js";
 import { fileSystem } from "@ecopages/file-system";
+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";
@@ -94,6 +95,21 @@ class NodeHmrManager {
     );
     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(
+          `[NodeHmrManager] Error checking match for ${candidate.constructor.name}:`,
+          err
+        );
+        return false;
+      }
+    });
+    return strategy !== void 0 && strategy.type !== HmrStrategyType.FALLBACK;
+  }
   async handleFileChange(filePath) {
     const sorted = [...this.strategies].sort((a, b) => b.priority - a.priority);
     const strategy = sorted.find((s) => {

package/src/adapters/node/node-hmr-manager.ts

@@ -7,6 +7,7 @@ import type { DefaultHmrContext, EcoPagesAppConfig, IHmrManager, IClientBridge }
 import type { EcoBuildPlugin } from '../../build/build-types.ts';
 import { fileSystem } from '@ecopages/file-system';
 import type { HmrStrategy } from '../../hmr/hmr-strategy.ts';
+import { HmrStrategyType } from '../../hmr/hmr-strategy.ts';
 import { DefaultHmrStrategy } from '../../hmr/strategies/default-hmr-strategy.ts';
 import { JsHmrStrategy } from '../../hmr/strategies/js-hmr-strategy.ts';
 import { appLogger } from '../../global/app-logger.ts';
@@ -121,6 +122,23 @@ export class NodeHmrManager implements IHmrManager {
 		this.bridge.broadcast(event);
 	}
 
+	public canHandleFileChange(filePath: string): boolean {
+		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(
+					`[NodeHmrManager] Error checking match for ${candidate.constructor.name}:`,
+					err as Error,
+				);
+				return false;
+			}
+		});
+
+		return strategy !== undefined && strategy.type !== HmrStrategyType.FALLBACK;
+	}
+
 	public async handleFileChange(filePath: string): Promise<void> {
 		const sorted = [...this.strategies].sort((a, b) => b.priority - a.priority);
 		const strategy = sorted.find((s) => {

package/src/hmr/strategies/js-hmr-strategy.js

@@ -26,7 +26,16 @@ class JsHmrStrategy extends HmrStrategy {
     if (watchedFiles.size === 0) {
       return false;
     }
-    return isJsTs && isInSrc;
+    if (!isJsTs || !isInSrc) {
+      return false;
+    }
+    if (watchedFiles.has(filePath)) {
+      return true;
+    }
+    if (this.context.getDependencyEntrypoints) {
+      return this.context.getDependencyEntrypoints(filePath).size > 0;
+    }
+    return true;
   }
   /**
    * Processes a file change by rebuilding affected entrypoints.

package/src/hmr/strategies/js-hmr-strategy.ts

@@ -121,7 +121,19 @@ export class JsHmrStrategy extends HmrStrategy {
 			return false;
 		}
 
-		return isJsTs && isInSrc;
+		if (!isJsTs || !isInSrc) {
+			return false;
+		}
+
+		if (watchedFiles.has(filePath)) {
+			return true;
+		}
+
+		if (this.context.getDependencyEntrypoints) {
+			return this.context.getDependencyEntrypoints(filePath).size > 0;
+		}
+
+		return true;
 	}
 
 	/**

package/src/public-types.d.ts

@@ -178,6 +178,10 @@ export interface IHmrManager {
      * Returns whether HMR is enabled.
      */
     isEnabled(): boolean;
+    /**
+     * Returns true when a changed file matches a non-fallback HMR strategy.
+     */
+    canHandleFileChange(path: string): boolean;
     /**
      * Broadcasts an HMR event to connected clients.
      */

package/src/public-types.ts

@@ -209,6 +209,11 @@ export interface IHmrManager {
 	 */
 	isEnabled(): boolean;
 
+	/**
+	 * Returns true when a changed file matches a non-fallback HMR strategy.
+	 */
+	canHandleFileChange(path: string): boolean;
+
 	/**
 	 * Broadcasts an HMR event to connected clients.
 	 */

package/src/watchers/project-watcher.d.ts

@@ -77,6 +77,11 @@ export declare class ProjectWatcher {
      * Checks if file path matches any additionalWatchPaths patterns.
      */
     private matchesAdditionalWatchPaths;
+    /**
+     * Checks whether a file is watched by any processor, even if that processor
+     * does not own the file as a primary asset.
+     */
+    private isWatchedByProcessor;
     /**
      * Checks if a file is handled by a processor.
      * Processors that declare extensions own those file types.

package/src/watchers/project-watcher.js

@@ -97,6 +97,10 @@ class ProjectWatcher {
       if (this.isHandledByProcessor(filePath)) {
         return;
       }
+      if (this.isWatchedByProcessor(filePath) && !this.hmrManager.canHandleFileChange(filePath)) {
+        this.hmrManager.broadcast({ type: "layout-update" });
+        return;
+      }
       await this.hmrManager.handleFileChange(filePath);
     } catch (error) {
       if (error instanceof Error) {
@@ -127,12 +131,35 @@ class ProjectWatcher {
     }
     return false;
   }
+  /**
+   * Checks whether a file is watched by any processor, even if that processor
+   * does not own the file as a primary asset.
+   */
+  isWatchedByProcessor(filePath) {
+    for (const processor of this.appConfig.processors.values()) {
+      const watchConfig = processor.getWatchConfig();
+      if (!watchConfig) continue;
+      const { extensions = [] } = watchConfig;
+      if (extensions.length && extensions.some((ext) => filePath.endsWith(ext))) {
+        return true;
+      }
+    }
+    return false;
+  }
   /**
    * Checks if a file is handled by a processor.
    * Processors that declare extensions own those file types.
    */
   isHandledByProcessor(filePath) {
     for (const processor of this.appConfig.processors.values()) {
+      const capabilities = processor.getAssetCapabilities?.() ?? [];
+      if (capabilities.length > 0) {
+        const matchesConfiguredAsset = typeof processor.matchesFileFilter !== "function" || processor.matchesFileFilter(filePath);
+        if (matchesConfiguredAsset && capabilities.some((capability) => processor.canProcessAsset?.(capability.kind, filePath))) {
+          return true;
+        }
+        continue;
+      }
       const watchConfig = processor.getWatchConfig();
       if (!watchConfig) continue;
       const { extensions = [] } = watchConfig;

package/src/watchers/project-watcher.test-helpers.js

@@ -14,6 +14,7 @@ const createMockHmrManager = () => ({
   registerStrategy: vi.fn(() => {
   }),
   isEnabled: vi.fn(() => true),
+  canHandleFileChange: vi.fn(() => true),
   getOutputUrl: vi.fn(() => void 0),
   getWatchedFiles: vi.fn(() => /* @__PURE__ */ new Map()),
   getSpecifierMap: vi.fn(() => /* @__PURE__ */ new Map()),

package/src/watchers/project-watcher.test-helpers.ts

@@ -12,6 +12,7 @@ export const createMockHmrManager = (): IHmrManager =>
 		registerSpecifierMap: vi.fn(() => {}),
 		registerStrategy: vi.fn(() => {}),
 		isEnabled: vi.fn(() => true),
+		canHandleFileChange: vi.fn(() => true),
 		getOutputUrl: vi.fn(() => undefined),
 		getWatchedFiles: vi.fn(() => new Map()),
 		getSpecifierMap: vi.fn(() => new Map()),

package/src/watchers/project-watcher.ts

@@ -144,6 +144,11 @@ export class ProjectWatcher {
 				return;
 			}
 
+			if (this.isWatchedByProcessor(filePath) && !this.hmrManager.canHandleFileChange(filePath)) {
+				this.hmrManager.broadcast({ type: 'layout-update' });
+				return;
+			}
+
 			await this.hmrManager.handleFileChange(filePath);
 		} catch (error) {
 			if (error instanceof Error) {
@@ -178,12 +183,45 @@ export class ProjectWatcher {
 		return false;
 	}
 
+	/**
+	 * Checks whether a file is watched by any processor, even if that processor
+	 * does not own the file as a primary asset.
+	 */
+	private isWatchedByProcessor(filePath: string): boolean {
+		for (const processor of this.appConfig.processors.values()) {
+			const watchConfig = processor.getWatchConfig();
+			if (!watchConfig) continue;
+
+			const { extensions = [] } = watchConfig;
+			if (extensions.length && extensions.some((ext) => filePath.endsWith(ext))) {
+				return true;
+			}
+		}
+
+		return false;
+	}
+
 	/**
 	 * Checks if a file is handled by a processor.
 	 * Processors that declare extensions own those file types.
 	 */
 	private isHandledByProcessor(filePath: string): boolean {
 		for (const processor of this.appConfig.processors.values()) {
+			const capabilities = processor.getAssetCapabilities?.() ?? [];
+			if (capabilities.length > 0) {
+				const matchesConfiguredAsset =
+					typeof processor.matchesFileFilter !== 'function' || processor.matchesFileFilter(filePath);
+
+				if (
+					matchesConfiguredAsset &&
+					capabilities.some((capability) => processor.canProcessAsset?.(capability.kind, filePath))
+				) {
+					return true;
+				}
+
+				continue;
+			}
+
 			const watchConfig = processor.getWatchConfig();
 			if (!watchConfig) continue;