@ecopages/core 0.2.0-alpha.25 → 0.2.0-alpha.26

package/src/services/module-loading/page-module-import.service.test.ts

@@ -1,504 +0,0 @@
-import { mkdtempSync, rmSync, writeFileSync } from 'node:fs';
-import assert from 'node:assert/strict';
-import { tmpdir } from 'node:os';
-import { join } from 'node:path';
-import { afterEach, beforeEach, describe, it } from 'vitest';
-import type { BuildResult } from '../../build/build-adapter.js';
-import { PageModuleImportService, type PageModuleImportDependencies } from './page-module-import.service.ts';
-
-function createBuildResult(overrides?: Partial<BuildResult>): BuildResult {
-	return {
-		success: true,
-		logs: [],
-		outputs: [],
-		...overrides,
-	};
-}
-
-function createFakeDependencies(): {
-	dependencies: PageModuleImportDependencies;
-	calls: {
-		hashFile: string[];
-		buildModule: Array<{
-			options: Parameters<PageModuleImportDependencies['buildModule']>[0];
-			buildExecutor: Parameters<PageModuleImportDependencies['buildModule']>[1];
-		}>;
-	};
-	setNextBuildResult(result: BuildResult): void;
-} {
-	const calls = {
-		hashFile: [] as string[],
-		buildModule: [] as Array<{
-			options: Parameters<PageModuleImportDependencies['buildModule']>[0];
-			buildExecutor: Parameters<PageModuleImportDependencies['buildModule']>[1];
-		}>,
-	};
-	let nextBuildResult = createBuildResult();
-
-	return {
-		dependencies: {
-			hashFile(filePath: string): string {
-				calls.hashFile.push(filePath);
-				return 'hash123';
-			},
-			async buildModule(options, buildExecutor): Promise<BuildResult> {
-				calls.buildModule.push({ options, buildExecutor });
-				return nextBuildResult;
-			},
-			canLoadSourceModuleFromHost(filePath: string): boolean {
-				return !filePath.endsWith('.mdx');
-			},
-			getHostModuleLoader(): undefined {
-				return undefined;
-			},
-		},
-		calls,
-		setNextBuildResult(result: BuildResult): void {
-			nextBuildResult = result;
-		},
-	};
-}
-
-describe('PageModuleImportService', () => {
-	let service: PageModuleImportService;
-	let fakeDependencies: ReturnType<typeof createFakeDependencies>;
-
-	beforeEach(() => {
-		fakeDependencies = createFakeDependencies();
-		service = new PageModuleImportService(fakeDependencies.dependencies);
-	});
-
-	afterEach(() => {
-		service.clearImportCache();
-		delete process.env.NODE_ENV;
-		delete (globalThis as typeof globalThis & { Bun?: unknown }).Bun;
-	});
-
-	it('should import the transpiled output in node runtimes', async () => {
-		const tempDir = mkdtempSync(join(tmpdir(), 'ecopages-page-module-import-'));
-		const compiledOutput = join(tempDir, 'page-hash123.js');
-		writeFileSync(compiledOutput, 'export const value = 42; export default { ok: true };', 'utf8');
-		fakeDependencies.setNextBuildResult(createBuildResult({ outputs: [{ path: compiledOutput }] }));
-
-		try {
-			const result = await service.importModule<{ default: { ok: boolean }; value: number }>({
-				filePath: '/app/pages/page.tsx',
-				rootDir: '/app',
-				outdir: tempDir,
-			});
-
-			assert.deepEqual(result.default, { ok: true });
-			assert.equal(result.value, 42);
-			assert.deepEqual(fakeDependencies.calls.hashFile, ['/app/pages/page.tsx']);
-			assert.deepEqual(fakeDependencies.calls.buildModule, [
-				{
-					options: {
-						entrypoints: ['/app/pages/page.tsx'],
-						root: '/app',
-						outdir: tempDir,
-						target: 'es2022',
-						format: 'esm',
-						sourcemap: 'none',
-						splitting: true,
-						minify: false,
-						naming: 'page-hash123.js',
-						externalPackages: true,
-						plugins: undefined,
-					},
-					buildExecutor: undefined,
-				},
-			]);
-		} finally {
-			rmSync(tempDir, { recursive: true, force: true });
-		}
-	});
-
-	it('should throw the provided transpile error when the build fails', async () => {
-		fakeDependencies.setNextBuildResult(
-			createBuildResult({
-				success: false,
-				logs: [{ message: 'Unexpected token' }],
-				outputs: [],
-			}),
-		);
-
-		await assert.rejects(
-			service.importModule({
-				filePath: '/app/pages/page.tsx',
-				rootDir: '/app',
-				outdir: '/tmp/out',
-				transpileErrorMessage: (details) => `transpile failed: ${details}`,
-			}),
-			/transpile failed: Unexpected token/,
-		);
-	});
-
-	it('should keep a stable node module path in development for unchanged files', async () => {
-		process.env.NODE_ENV = 'development';
-		const tempDir = mkdtempSync(join(tmpdir(), 'ecopages-page-module-import-dev-'));
-		const compiledOutput = join(tempDir, 'page-hash123.js');
-		writeFileSync(compiledOutput, 'export default { ok: true };', 'utf8');
-		fakeDependencies.setNextBuildResult(createBuildResult({ outputs: [{ path: compiledOutput }] }));
-
-		try {
-			await service.importModule({
-				filePath: '/app/pages/page.tsx',
-				rootDir: '/app',
-				outdir: tempDir,
-			});
-
-			assert.equal(fakeDependencies.calls.buildModule.length, 1);
-			assert.equal(fakeDependencies.calls.buildModule[0]?.options.naming, 'page-hash123.js');
-			assert.equal(fakeDependencies.calls.buildModule[0]?.options.splitting, true);
-			assert.equal(fakeDependencies.calls.buildModule[0]?.options.externalPackages, true);
-		} finally {
-			rmSync(tempDir, { recursive: true, force: true });
-		}
-	});
-
-	it('should import source modules through the host loader in node development when available', async () => {
-		process.env.NODE_ENV = 'development';
-		const hostLoaderCalls: string[] = [];
-		service = new PageModuleImportService({
-			...fakeDependencies.dependencies,
-			getHostModuleLoader: () => async (id: string) => {
-				hostLoaderCalls.push(id);
-				return { default: { ok: true }, value: 42 };
-			},
-		});
-
-		const result = await service.importModule<{ default: { ok: boolean }; value: number }>({
-			filePath: '/app/pages/page.tsx',
-			rootDir: '/app',
-			outdir: '/tmp/out',
-		});
-
-		assert.deepEqual(result.default, { ok: true });
-		assert.equal(result.value, 42);
-		assert.deepEqual(hostLoaderCalls, ['file:///app/pages/page.tsx?update=hash123-0']);
-		assert.equal(fakeDependencies.calls.buildModule.length, 0);
-	});
-
-	it('should delegate every host-loader import to the host without service-level caching', async () => {
-		process.env.NODE_ENV = 'development';
-		const hostLoaderCalls: string[] = [];
-		service = new PageModuleImportService({
-			...fakeDependencies.dependencies,
-			getHostModuleLoader: () => async (id: string) => {
-				hostLoaderCalls.push(id);
-				return { default: { ok: true } };
-			},
-		});
-
-		const first = await service.importModule<{ default: { ok: boolean } }>({
-			filePath: '/app/pages/page.tsx',
-			rootDir: '/app',
-			outdir: '/tmp/meta',
-		});
-
-		const second = await service.importModule<{ default: { ok: boolean } }>({
-			filePath: '/app/pages/page.tsx',
-			rootDir: '/app',
-			outdir: '/tmp/render',
-		});
-
-		assert.deepEqual(first.default, { ok: true });
-		assert.deepEqual(second.default, { ok: true });
-		assert.deepEqual(hostLoaderCalls, [
-			'file:///app/pages/page.tsx?update=hash123-0',
-			'file:///app/pages/page.tsx?update=hash123-0',
-		]);
-		assert.equal(fakeDependencies.calls.buildModule.length, 0);
-	});
-
-	it('should bypass the service cache when requested for bundled module imports', async () => {
-		const tempDir = mkdtempSync(join(tmpdir(), 'ecopages-page-module-import-bypass-'));
-		const compiledOutput = join(tempDir, 'page-hash123.js');
-		writeFileSync(compiledOutput, 'export default { ok: true };', 'utf8');
-		fakeDependencies.setNextBuildResult(createBuildResult({ outputs: [{ path: compiledOutput }] }));
-
-		try {
-			await service.importModule({
-				filePath: '/app/pages/page.tsx',
-				rootDir: '/app',
-				outdir: tempDir,
-				bypassCache: true,
-			});
-
-			await service.importModule({
-				filePath: '/app/pages/page.tsx',
-				rootDir: '/app',
-				outdir: tempDir,
-				bypassCache: true,
-			});
-
-			assert.equal(fakeDependencies.calls.buildModule.length, 2);
-		} finally {
-			rmSync(tempDir, { recursive: true, force: true });
-		}
-	});
-
-	it('should fall back to the build adapter when the host-loader policy rejects a source module', async () => {
-		process.env.NODE_ENV = 'development';
-		const tempDir = mkdtempSync(join(tmpdir(), 'ecopages-page-module-import-host-policy-'));
-		const compiledOutput = join(tempDir, 'page-hash123.js');
-		const hostLoaderCalls: string[] = [];
-		writeFileSync(compiledOutput, 'export default { ok: true };', 'utf8');
-		fakeDependencies.setNextBuildResult(createBuildResult({ outputs: [{ path: compiledOutput }] }));
-		service = new PageModuleImportService({
-			...fakeDependencies.dependencies,
-			canLoadSourceModuleFromHost: () => false,
-			getHostModuleLoader: () => async (id: string) => {
-				hostLoaderCalls.push(id);
-				return { default: { ok: false } };
-			},
-		});
-
-		try {
-			const result = await service.importModule<{ default: { ok: boolean } }>({
-				filePath: '/app/pages/page.tsx',
-				rootDir: '/app',
-				outdir: tempDir,
-			});
-
-			assert.deepEqual(result.default, { ok: true });
-			assert.deepEqual(hostLoaderCalls, []);
-			assert.equal(fakeDependencies.calls.buildModule.length, 1);
-		} finally {
-			rmSync(tempDir, { recursive: true, force: true });
-		}
-	});
-
-	it('should fall back to the build adapter for source types the host loader cannot import directly', async () => {
-		process.env.NODE_ENV = 'development';
-		const tempDir = mkdtempSync(join(tmpdir(), 'ecopages-page-module-import-mdx-'));
-		const compiledOutput = join(tempDir, 'page-hash123.js');
-		const hostLoaderCalls: string[] = [];
-		writeFileSync(compiledOutput, 'export default { ok: true };', 'utf8');
-		fakeDependencies.setNextBuildResult(createBuildResult({ outputs: [{ path: compiledOutput }] }));
-		service = new PageModuleImportService({
-			...fakeDependencies.dependencies,
-			getHostModuleLoader: () => async (id: string) => {
-				hostLoaderCalls.push(id);
-				return { default: { ok: false } };
-			},
-		});
-
-		try {
-			const result = await service.importModule<{ default: { ok: boolean } }>({
-				filePath: '/app/pages/page.mdx',
-				rootDir: '/app',
-				outdir: tempDir,
-			});
-
-			assert.deepEqual(result.default, { ok: true });
-			assert.deepEqual(hostLoaderCalls, []);
-			assert.equal(fakeDependencies.calls.buildModule.length, 1);
-		} finally {
-			rmSync(tempDir, { recursive: true, force: true });
-		}
-	});
-
-	it('should allow callers to opt back into bundled node imports explicitly', async () => {
-		process.env.NODE_ENV = 'development';
-		const tempDir = mkdtempSync(join(tmpdir(), 'ecopages-page-module-import-bundled-'));
-		const compiledOutput = join(tempDir, 'page-hash123.js');
-		writeFileSync(compiledOutput, 'export default { ok: true };', 'utf8');
-		fakeDependencies.setNextBuildResult(createBuildResult({ outputs: [{ path: compiledOutput }] }));
-
-		try {
-			await service.importModule({
-				filePath: '/app/pages/page.tsx',
-				rootDir: '/app',
-				outdir: tempDir,
-				externalPackages: false,
-			});
-
-			assert.equal(fakeDependencies.calls.buildModule.length, 1);
-			assert.equal(fakeDependencies.calls.buildModule[0]?.options.externalPackages, false);
-		} finally {
-			rmSync(tempDir, { recursive: true, force: true });
-		}
-	});
-
-	it('should allow callers to disable code splitting explicitly', async () => {
-		const tempDir = mkdtempSync(join(tmpdir(), 'ecopages-page-module-import-unsplit-'));
-		const compiledOutput = join(tempDir, 'page-hash123.js');
-		writeFileSync(compiledOutput, 'export default { ok: true };', 'utf8');
-		fakeDependencies.setNextBuildResult(createBuildResult({ outputs: [{ path: compiledOutput }] }));
-
-		try {
-			await service.importModule({
-				filePath: '/app/pages/page.tsx',
-				rootDir: '/app',
-				outdir: tempDir,
-				splitting: false,
-			});
-
-			assert.equal(fakeDependencies.calls.buildModule.length, 1);
-			assert.equal(fakeDependencies.calls.buildModule[0]?.options.splitting, false);
-		} finally {
-			rmSync(tempDir, { recursive: true, force: true });
-		}
-	});
-
-	it('should reuse cached node modules across different output directories', async () => {
-		const tempDir = mkdtempSync(join(tmpdir(), 'ecopages-page-module-import-cache-'));
-		const compiledOutput = join(tempDir, 'page-hash123.js');
-		writeFileSync(compiledOutput, 'export default { ok: true };', 'utf8');
-		fakeDependencies.setNextBuildResult(createBuildResult({ outputs: [{ path: compiledOutput }] }));
-
-		try {
-			const first = await service.importModule<{ default: { ok: boolean } }>({
-				filePath: '/app/pages/page.tsx',
-				rootDir: '/app',
-				outdir: join(tempDir, 'meta'),
-			});
-
-			const second = await service.importModule<{ default: { ok: boolean } }>({
-				filePath: '/app/pages/page.tsx',
-				rootDir: '/app',
-				outdir: join(tempDir, 'render'),
-			});
-
-			assert.deepEqual(first.default, { ok: true });
-			assert.deepEqual(second.default, { ok: true });
-			assert.equal(fakeDependencies.calls.buildModule.length, 1);
-		} finally {
-			rmSync(tempDir, { recursive: true, force: true });
-		}
-	});
-
-	it('should isolate node module identities across cache scopes', async () => {
-		const tempDir = mkdtempSync(join(tmpdir(), 'ecopages-page-module-import-scope-'));
-		const requestCompiledOutput = join(tempDir, 'page-hash123-request-metadata.js');
-		const renderCompiledOutput = join(tempDir, 'page-hash123-render.js');
-		writeFileSync(requestCompiledOutput, 'export const scope = "request"; export default { ok: true };', 'utf8');
-		writeFileSync(renderCompiledOutput, 'export const scope = "render"; export default { ok: true };', 'utf8');
-
-		service = new PageModuleImportService({
-			...fakeDependencies.dependencies,
-			async buildModule(options, buildExecutor) {
-				fakeDependencies.calls.buildModule.push({ options, buildExecutor });
-				return createBuildResult({
-					outputs: [
-						{
-							path:
-								options.naming === 'page-hash123-request-metadata.js'
-									? requestCompiledOutput
-									: renderCompiledOutput,
-						},
-					],
-				});
-			},
-		});
-
-		try {
-			const requestModule = await service.importModule<{ scope: string }>({
-				filePath: '/app/pages/page.tsx',
-				rootDir: '/app',
-				outdir: tempDir,
-				cacheScope: 'request-metadata',
-			});
-
-			const renderModule = await service.importModule<{ scope: string }>({
-				filePath: '/app/pages/page.tsx',
-				rootDir: '/app',
-				outdir: tempDir,
-				cacheScope: 'render',
-			});
-
-			assert.equal(requestModule.scope, 'request');
-			assert.equal(renderModule.scope, 'render');
-			assert.deepEqual(
-				fakeDependencies.calls.buildModule.map((call) => call.options.naming),
-				['page-hash123-request-metadata.js', 'page-hash123-render.js'],
-			);
-		} finally {
-			rmSync(tempDir, { recursive: true, force: true });
-		}
-	});
-
-	it('should reload the same node module path after development graph invalidation', async () => {
-		process.env.NODE_ENV = 'development';
-		const tempDir = mkdtempSync(join(tmpdir(), 'ecopages-page-module-import-invalidate-'));
-		const compiledOutput = join(tempDir, 'page-hash123.js');
-		writeFileSync(compiledOutput, 'export const version = 1; export default { ok: true };', 'utf8');
-		fakeDependencies.setNextBuildResult(createBuildResult({ outputs: [{ path: compiledOutput }] }));
-
-		try {
-			const first = await service.importModule<{ version: number }>({
-				filePath: '/app/pages/page.tsx',
-				rootDir: '/app',
-				outdir: tempDir,
-			});
-
-			writeFileSync(compiledOutput, 'export const version = 2; export default { ok: true };', 'utf8');
-			service.invalidateDevelopmentGraph();
-			fakeDependencies.setNextBuildResult(createBuildResult({ outputs: [{ path: compiledOutput }] }));
-
-			const second = await service.importModule<{ version: number }>({
-				filePath: '/app/pages/page.tsx',
-				rootDir: '/app',
-				outdir: tempDir,
-			});
-
-			assert.equal(first.version, 1);
-			assert.equal(second.version, 2);
-			assert.equal(fakeDependencies.calls.buildModule.length, 2);
-		} finally {
-			rmSync(tempDir, { recursive: true, force: true });
-		}
-	});
-
-	it('should invalidate the host-loader module identity in development', async () => {
-		process.env.NODE_ENV = 'development';
-		const hostLoaderCalls: string[] = [];
-		service = new PageModuleImportService({
-			...fakeDependencies.dependencies,
-			getHostModuleLoader: () => async (id: string) => {
-				hostLoaderCalls.push(id);
-				return { version: hostLoaderCalls.length };
-			},
-		});
-
-		const first = await service.importModule<{ version: number }>({
-			filePath: '/app/pages/page.tsx',
-			rootDir: '/app',
-			outdir: '/tmp/out',
-		});
-
-		service.invalidateDevelopmentGraph();
-
-		const second = await service.importModule<{ version: number }>({
-			filePath: '/app/pages/page.tsx',
-			rootDir: '/app',
-			outdir: '/tmp/out',
-		});
-
-		assert.equal(first.version, 1);
-		assert.equal(second.version, 2);
-		assert.deepEqual(hostLoaderCalls, [
-			'file:///app/pages/page.tsx?update=hash123-0',
-			'file:///app/pages/page.tsx?update=hash123-1',
-		]);
-		assert.equal(fakeDependencies.calls.buildModule.length, 0);
-	});
-
-	it('should use direct source imports for Bun development builds', async () => {
-		process.env.NODE_ENV = 'development';
-		(globalThis as typeof globalThis & { Bun?: unknown }).Bun = {};
-
-		try {
-			await service.importModule<unknown>({
-				filePath: '/app/pages/page.tsx',
-				rootDir: '/app',
-				outdir: '/tmp/out',
-			});
-		} catch {
-			// import will fail for the fake path, but we only care about the code path
-		}
-
-		assert.equal(fakeDependencies.calls.buildModule.length, 0);
-	});
-});