@ecopages/core 0.2.0-alpha.53 → 0.2.0-alpha.54

package/src/adapters/shared/hmr-html-response.ts

@@ -0,0 +1,54 @@
+import type { IHmrManager } from '../../types/public-types.ts';
+
+const HMR_RUNTIME_IMPORT = "import '/_hmr_runtime.js'";
+const HMR_RUNTIME_SCRIPT = `<script type="module">${HMR_RUNTIME_IMPORT};</script>`;
+
+/**
+ * Returns whether a response is HTML and therefore eligible for development HMR
+ * runtime injection.
+ */
+export function isHtmlResponse(response: Response): boolean {
+	const contentType = response.headers.get('Content-Type');
+	return contentType !== null && contentType.startsWith('text/html');
+}
+
+/**
+ * Returns whether HTML responses should receive the HMR runtime bootstrap.
+ *
+ * This is shared because filesystem page responses and adapter-level HTML
+ * responses flow through different layers, but both need identical injection
+ * behavior in watch mode.
+ */
+export function shouldInjectHmrHtmlResponse(watch: boolean, hmrManager?: Pick<IHmrManager, 'isEnabled'>): boolean {
+	return watch && hmrManager?.isEnabled() === true;
+}
+
+/**
+ * Injects the development HMR runtime script into an HTML response if it is not
+ * already present.
+ *
+ * The check is intentionally idempotent because an HTML response can pass
+ * through more than one development-layer wrapper before reaching the client.
+ */
+export async function injectHmrRuntimeIntoHtmlResponse(response: Response): Promise<Response> {
+	const html = await response.text();
+	const headers = new Headers(response.headers);
+	headers.set('Cache-Control', 'no-store, must-revalidate');
+	headers.delete('Content-Length');
+
+	if (html.includes(HMR_RUNTIME_IMPORT)) {
+		return new Response(html, {
+			status: response.status,
+			statusText: response.statusText,
+			headers,
+		});
+	}
+
+	const updatedHtml = html.replace(/<\/html>/i, `${HMR_RUNTIME_SCRIPT}</html>`);
+
+	return new Response(updatedHtml, {
+		status: response.status,
+		statusText: response.statusText,
+		headers,
+	});
+}

package/src/adapters/shared/hmr-manager.contract.test.ts

@@ -0,0 +1,228 @@
+import assert from 'node:assert/strict';
+import fs from 'node:fs';
+import os from 'node:os';
+import path from 'node:path';
+import { afterEach, describe, test, vi } from 'vitest';
+import { ConfigBuilder } from '../../config/config-builder.ts';
+import { resolveInternalWorkDir } from '../../utils/resolve-work-dir.ts';
+import { NodeHmrManager } from '../node/node-hmr-manager.ts';
+import { HmrManager as BunHmrManager } from '../bun/hmr-manager.ts';
+
+type SharedHmrManager = {
+	handleFileChange(filePath: string, options?: { broadcast?: boolean }): Promise<void>;
+	registerEntrypoint(entrypointPath: string): Promise<string>;
+	registerScriptEntrypoint(entrypointPath: string): Promise<string>;
+	getWatchedFiles(): Map<string, string>;
+	stop(): void;
+	appConfig: Awaited<ReturnType<ConfigBuilder['build']>>;
+};
+
+const tempRoots: string[] = [];
+
+function createTempRoot(prefix: string): string {
+	const root = fs.mkdtempSync(path.join(os.tmpdir(), `${prefix}-`));
+	tempRoots.push(root);
+	return root;
+}
+
+afterEach(() => {
+	for (const root of tempRoots.splice(0)) {
+		fs.rmSync(root, { recursive: true, force: true });
+	}
+	vi.restoreAllMocks();
+});
+
+const runtimes = [
+	{
+		name: 'node',
+		async create(rootDir: string): Promise<SharedHmrManager> {
+			const config = await new ConfigBuilder().setRootDir(rootDir).build();
+			return new NodeHmrManager({
+				appConfig: config,
+				bridge: {
+					subscriberCount: 0,
+					broadcast: () => {},
+				} as any,
+			});
+		},
+	},
+	{
+		name: 'bun',
+		async create(rootDir: string): Promise<SharedHmrManager> {
+			const config = await new ConfigBuilder().setRootDir(rootDir).build();
+			return new BunHmrManager({
+				appConfig: config,
+				bridge: {
+					subscriberCount: 0,
+					broadcast: () => {},
+					subscribe: () => {},
+					unsubscribe: () => {},
+				} as any,
+			});
+		},
+	},
+] as const;
+
+function getEntrypointOutputPath(manager: SharedHmrManager, entrypointPath: string): string {
+	const relativePathJs = path
+		.relative(manager.appConfig.absolutePaths.srcDir, entrypointPath)
+		.replace(/\.(tsx?|jsx?|mdx?)$/, '.js');
+	const encodedPathJs = relativePathJs.replace(/\[([^\]]+)\]/g, '_$1_');
+	return path.join(resolveInternalWorkDir(manager.appConfig), 'assets', '_hmr', encodedPathJs);
+}
+
+describe.each(runtimes)('shared HMR manager contract: $name', ({ create }) => {
+	test('shares one in-flight entrypoint registration across concurrent callers', async () => {
+		const rootDir = createTempRoot('ecopages-hmr-contract-register');
+		const pagesDir = path.join(rootDir, 'src', 'pages');
+		fs.mkdirSync(pagesDir, { recursive: true });
+
+		const entrypointPath = path.join(pagesDir, 'react-lab.tsx');
+		fs.writeFileSync(entrypointPath, 'export default function Page() { return null; }', 'utf8');
+
+		const manager = await create(rootDir);
+		const outputPath = getEntrypointOutputPath(manager, entrypointPath);
+
+		const handleFileChange = vi.spyOn(manager, 'handleFileChange').mockImplementation(async () => {
+			await new Promise((resolve) => setTimeout(resolve, 25));
+			fs.mkdirSync(path.dirname(outputPath), { recursive: true });
+			fs.writeFileSync(outputPath, 'export default 1;', 'utf8');
+		});
+
+		const [firstUrl, secondUrl] = await Promise.all([
+			manager.registerEntrypoint(entrypointPath),
+			manager.registerEntrypoint(entrypointPath),
+		]);
+
+		assert.equal(firstUrl, '/assets/_hmr/pages/react-lab.js');
+		assert.equal(secondUrl, '/assets/_hmr/pages/react-lab.js');
+		assert.equal(handleFileChange.mock.calls.length, 1);
+
+		manager.stop();
+	});
+
+	test('fails strict page registration when no integration emits output', async () => {
+		const rootDir = createTempRoot('ecopages-hmr-contract-strict-fail');
+		const pagesDir = path.join(rootDir, 'src', 'pages');
+		fs.mkdirSync(pagesDir, { recursive: true });
+
+		const entrypointPath = path.join(pagesDir, 'react-content.mdx');
+		fs.writeFileSync(entrypointPath, '# Hello', 'utf8');
+
+		const manager = await create(rootDir);
+		vi.spyOn(manager, 'handleFileChange').mockImplementation(async () => {});
+
+		await assert.rejects(() => manager.registerEntrypoint(entrypointPath), /Integration failed to emit entrypoint/);
+		assert.equal(manager.getWatchedFiles().has(path.resolve(entrypointPath)), false);
+
+		manager.stop();
+	});
+
+	test('uses the generic build path for explicit script entrypoints', async () => {
+		const rootDir = createTempRoot('ecopages-hmr-contract-script-register');
+		const srcDir = path.join(rootDir, 'src');
+		fs.mkdirSync(srcDir, { recursive: true });
+
+		const entrypointPath = path.join(srcDir, 'script.ts');
+		fs.writeFileSync(entrypointPath, 'console.log("hello");', 'utf8');
+
+		const manager = await create(rootDir);
+		const outputPath = path.join(resolveInternalWorkDir(manager.appConfig), 'assets', '_hmr', 'script.js');
+		const buildCalls: string[] = [];
+		manager.appConfig.runtime!.buildExecutor = {
+			build: vi.fn(async (options) => {
+				buildCalls.push(options.entrypoints[0] as string);
+				fs.mkdirSync(path.dirname(outputPath), { recursive: true });
+				fs.writeFileSync(outputPath, 'fresh-output', 'utf8');
+				return {
+					success: true,
+					logs: [],
+					outputs: [{ path: outputPath }],
+				};
+			}),
+		};
+
+		const outputUrl = await manager.registerScriptEntrypoint(entrypointPath);
+
+		assert.equal(outputUrl, '/assets/_hmr/script.js');
+		assert.deepEqual(buildCalls, [entrypointPath]);
+		assert.equal(fs.readFileSync(outputPath, 'utf8'), 'fresh-output');
+
+		manager.stop();
+	});
+
+	test('registering one script entrypoint does not rebuild previously watched script entrypoints', async () => {
+		const rootDir = createTempRoot('ecopages-hmr-contract-script-targeted-register');
+		const srcDir = path.join(rootDir, 'src');
+		fs.mkdirSync(srcDir, { recursive: true });
+
+		const firstEntrypoint = path.join(srcDir, 'first.script.ts');
+		const secondEntrypoint = path.join(srcDir, 'second.script.ts');
+		fs.writeFileSync(firstEntrypoint, 'console.log("first");', 'utf8');
+		fs.writeFileSync(secondEntrypoint, 'console.log("second");', 'utf8');
+
+		const manager = await create(rootDir);
+		const buildCalls: string[][] = [];
+		manager.appConfig.runtime!.buildExecutor = {
+			build: vi.fn(async (options) => {
+				const entrypoints = (options.entrypoints as string[]).map(String);
+				buildCalls.push(entrypoints);
+				for (const entrypoint of entrypoints) {
+					const relativePathJs = path
+						.relative(path.join(rootDir, 'src'), entrypoint)
+						.replace(/\.(tsx?|jsx?|mdx?)$/, '.js');
+					const outputPath = path.join(
+						resolveInternalWorkDir(manager.appConfig),
+						'assets',
+						'_hmr',
+						relativePathJs,
+					);
+					fs.mkdirSync(path.dirname(outputPath), { recursive: true });
+					fs.writeFileSync(outputPath, `output:${path.basename(entrypoint)}`, 'utf8');
+				}
+				return {
+					success: true,
+					logs: [],
+					outputs: entrypoints.map((entrypoint) => ({
+						path: path.join(
+							resolveInternalWorkDir(manager.appConfig),
+							'assets',
+							'_hmr',
+							path.relative(path.join(rootDir, 'src'), entrypoint).replace(/\.(tsx?|jsx?|mdx?)$/, '.js'),
+						),
+					})),
+				};
+			}),
+		};
+
+		await manager.registerScriptEntrypoint(firstEntrypoint);
+		await manager.registerScriptEntrypoint(secondEntrypoint);
+
+		assert.deepEqual(buildCalls, [[firstEntrypoint], [secondEntrypoint]]);
+
+		manager.stop();
+	});
+
+	test('stop clears retained registration state', async () => {
+		const rootDir = createTempRoot('ecopages-hmr-contract-stop');
+		const pagesDir = path.join(rootDir, 'src', 'pages');
+		fs.mkdirSync(pagesDir, { recursive: true });
+
+		const entrypointPath = path.join(pagesDir, 'react-content.tsx');
+		fs.writeFileSync(entrypointPath, 'export default function Page() { return null; }', 'utf8');
+
+		const manager = await create(rootDir);
+		const outputPath = getEntrypointOutputPath(manager, entrypointPath);
+
+		vi.spyOn(manager, 'handleFileChange').mockImplementation(async () => {
+			fs.mkdirSync(path.dirname(outputPath), { recursive: true });
+			fs.writeFileSync(outputPath, 'export default 1;', 'utf8');
+		});
+
+		await manager.registerEntrypoint(entrypointPath);
+
+		manager.stop();
+
+		assert.equal(manager.getWatchedFiles().size, 0);
+	});
+});

package/src/adapters/shared/hmr-manager.dispatch.test.ts

@@ -0,0 +1,229 @@
+import assert from 'node:assert/strict';
+import fs from 'node:fs';
+import os from 'node:os';
+import path from 'node:path';
+import { afterEach, describe, test } from 'vitest';
+import { ConfigBuilder } from '../../config/config-builder.ts';
+import { HmrStrategy, HmrStrategyType, type HmrAction } from '../../hmr/hmr-strategy.ts';
+import type { ClientBridgeEvent } from '../../types/public-types.ts';
+import { HmrManager as BunHmrManager } from '../bun/hmr-manager.ts';
+import { NodeHmrManager } from '../node/node-hmr-manager.ts';
+
+class FakeHmrStrategy extends HmrStrategy {
+	readonly type: HmrStrategyType;
+	private readonly _matches: (filePath: string) => boolean;
+	private readonly _action: HmrAction;
+
+	constructor(type: HmrStrategyType, matchFn: (f: string) => boolean, action: HmrAction) {
+		super();
+		this.type = type;
+		this._matches = matchFn;
+		this._action = action;
+	}
+
+	matches(filePath: string): boolean {
+		return this._matches(filePath);
+	}
+
+	async process(_filePath: string): Promise<HmrAction> {
+		return this._action;
+	}
+}
+
+type BridgeSpy = {
+	broadcasts: ClientBridgeEvent[];
+	bridge: {
+		subscriberCount: number;
+		broadcast(event: ClientBridgeEvent): void;
+		subscribe(): void;
+		unsubscribe(): void;
+	};
+};
+
+function createBridgeSpy(): BridgeSpy {
+	const broadcasts: ClientBridgeEvent[] = [];
+	const bridge = {
+		subscriberCount: 0,
+		broadcast(event: ClientBridgeEvent) {
+			broadcasts.push(event);
+		},
+		subscribe() {},
+		unsubscribe() {},
+	};
+	return { broadcasts, bridge };
+}
+
+const tempRoots: string[] = [];
+
+function createTempRoot(prefix: string): string {
+	const root = fs.mkdtempSync(path.join(os.tmpdir(), `${prefix}-`));
+	tempRoots.push(root);
+	return root;
+}
+
+afterEach(() => {
+	for (const root of tempRoots.splice(0)) {
+		fs.rmSync(root, { recursive: true, force: true });
+	}
+});
+
+const runtimes = [
+	{
+		name: 'node',
+		async create(rootDir: string, bridgeSpy: BridgeSpy) {
+			const config = await new ConfigBuilder().setRootDir(rootDir).build();
+			return new NodeHmrManager({ appConfig: config, bridge: bridgeSpy.bridge as any });
+		},
+	},
+	{
+		name: 'bun',
+		async create(rootDir: string, bridgeSpy: BridgeSpy) {
+			const config = await new ConfigBuilder().setRootDir(rootDir).build();
+			return new BunHmrManager({ appConfig: config, bridge: bridgeSpy.bridge as any });
+		},
+	},
+] as const;
+
+describe.each(runtimes)('handleFileChange dispatch: $name', ({ create }) => {
+	test('CSS file change routes to DefaultHmrStrategy and broadcasts reload', async () => {
+		const rootDir = createTempRoot('ecopages-dispatch-css');
+		fs.mkdirSync(path.join(rootDir, 'src'), { recursive: true });
+		const spy = createBridgeSpy();
+		const manager = await create(rootDir, spy);
+
+		const cssFile = path.join(rootDir, 'src', 'styles', 'main.css');
+		fs.mkdirSync(path.dirname(cssFile), { recursive: true });
+		fs.writeFileSync(cssFile, 'body {}\n', 'utf8');
+		await manager.handleFileChange(cssFile);
+
+		assert.equal(spy.broadcasts.length, 1);
+		assert.equal(spy.broadcasts[0].type, 'reload');
+		assert.equal(spy.broadcasts[0].path, cssFile);
+
+		manager.stop();
+	});
+
+	test('HTML file change routes to DefaultHmrStrategy and broadcasts reload', async () => {
+		const rootDir = createTempRoot('ecopages-dispatch-html');
+		fs.mkdirSync(path.join(rootDir, 'src'), { recursive: true });
+		const spy = createBridgeSpy();
+		const manager = await create(rootDir, spy);
+
+		const htmlFile = path.join(rootDir, 'src', 'pages', 'index.html');
+		fs.mkdirSync(path.dirname(htmlFile), { recursive: true });
+		fs.writeFileSync(htmlFile, '<div>Hello</div>\n', 'utf8');
+		await manager.handleFileChange(htmlFile);
+
+		assert.equal(spy.broadcasts.length, 1);
+		assert.equal(spy.broadcasts[0].type, 'reload');
+		assert.equal(spy.broadcasts[0].path, htmlFile);
+
+		manager.stop();
+	});
+
+	test('TS file with no registered entrypoints falls back to DefaultHmrStrategy reload', async () => {
+		const rootDir = createTempRoot('ecopages-dispatch-ts-fallback');
+		const srcDir = path.join(rootDir, 'src');
+		fs.mkdirSync(srcDir, { recursive: true });
+		const spy = createBridgeSpy();
+		const manager = await create(rootDir, spy);
+
+		const tsFile = path.join(srcDir, 'component.ts');
+		fs.writeFileSync(tsFile, 'export const component = true;\n', 'utf8');
+		await manager.handleFileChange(tsFile);
+
+		assert.equal(spy.broadcasts.length, 1);
+		assert.equal(spy.broadcasts[0].type, 'reload');
+
+		manager.stop();
+	});
+
+	test('broadcast:false suppresses events even when the strategy returns a broadcast action', async () => {
+		const rootDir = createTempRoot('ecopages-dispatch-no-broadcast');
+		fs.mkdirSync(path.join(rootDir, 'src'), { recursive: true });
+		const spy = createBridgeSpy();
+		const manager = await create(rootDir, spy);
+
+		const cssFile = path.join(rootDir, 'src', 'main.css');
+		fs.writeFileSync(cssFile, 'body {}\n', 'utf8');
+		await manager.handleFileChange(cssFile, { broadcast: false });
+
+		assert.equal(spy.broadcasts.length, 0);
+
+		manager.stop();
+	});
+
+	test('INTEGRATION strategy wins over DefaultHmrStrategy for matched files', async () => {
+		const rootDir = createTempRoot('ecopages-dispatch-integration-priority');
+		fs.mkdirSync(path.join(rootDir, 'src'), { recursive: true });
+		const spy = createBridgeSpy();
+		const manager = await create(rootDir, spy);
+
+		const customFile = path.join(rootDir, 'src', 'component.jsx');
+		fs.writeFileSync(customFile, 'export default null;\n', 'utf8');
+		const integrationEvent: ClientBridgeEvent = {
+			type: 'update',
+			path: '/assets/_hmr/component.js',
+			timestamp: 1,
+		};
+		manager.registerStrategy(
+			new FakeHmrStrategy(HmrStrategyType.INTEGRATION, (f) => f === customFile, {
+				type: 'broadcast',
+				events: [integrationEvent],
+			}),
+		);
+
+		await manager.handleFileChange(customFile);
+
+		assert.equal(spy.broadcasts.length, 1);
+		assert.deepEqual(spy.broadcasts[0], integrationEvent);
+
+		manager.stop();
+	});
+
+	test('action.type none suppresses broadcast even when shouldBroadcast is true', async () => {
+		const rootDir = createTempRoot('ecopages-dispatch-none-action');
+		fs.mkdirSync(path.join(rootDir, 'src'), { recursive: true });
+		const spy = createBridgeSpy();
+		const manager = await create(rootDir, spy);
+
+		const customFile = path.join(rootDir, 'src', 'silent.ts');
+		fs.writeFileSync(customFile, 'export const silent = true;\n', 'utf8');
+		manager.registerStrategy(
+			new FakeHmrStrategy(HmrStrategyType.INTEGRATION, (f) => f === customFile, { type: 'none' }),
+		);
+
+		await manager.handleFileChange(customFile);
+
+		assert.equal(spy.broadcasts.length, 0);
+
+		manager.stop();
+	});
+
+	test('all events returned by a strategy are each broadcast individually', async () => {
+		const rootDir = createTempRoot('ecopages-dispatch-multi-event');
+		fs.mkdirSync(path.join(rootDir, 'src'), { recursive: true });
+		const spy = createBridgeSpy();
+		const manager = await create(rootDir, spy);
+
+		const customFile = path.join(rootDir, 'src', 'multi.ts');
+		fs.writeFileSync(customFile, 'export const multi = true;\n', 'utf8');
+		const events: ClientBridgeEvent[] = [
+			{ type: 'update', path: '/assets/_hmr/a.js', timestamp: 1 },
+			{ type: 'update', path: '/assets/_hmr/b.js', timestamp: 2 },
+		];
+		manager.registerStrategy(
+			new FakeHmrStrategy(HmrStrategyType.INTEGRATION, (f) => f === customFile, {
+				type: 'broadcast',
+				events,
+			}),
+		);
+
+		await manager.handleFileChange(customFile);
+
+		assert.equal(spy.broadcasts.length, 2);
+		assert.deepEqual(spy.broadcasts, events);
+
+		manager.stop();
+	});
+});