@azure-net/cli 5.1.1 → 6.0.0-next.3

package/src/checks/checkImports.ts

@@ -1,193 +0,0 @@
-import fs from 'node:fs/promises';
-import path from 'node:path';
-
-type ImportIssue = {
-	file: string;
-	line: number;
-	message: string;
-};
-
-type ImportKind = 'value' | 'type' | 'side-effect';
-
-type ImportEntry = {
-	kind: ImportKind;
-	line: number;
-	source: string;
-	raw: string;
-};
-
-const TARGET_EXTENSIONS = new Set(['.ts', '.js', '.svelte']);
-const IGNORED_DIRS = new Set(['node_modules', 'dist', '.svelte-kit', '.git']);
-const IMPORT_DECLARATION_PATTERN = /(^|\n)\s*import\s+(?:[\s\S]*?\s+from\s*)?['"]([^'"]+)['"]\s*;?/g;
-const EXPORT_DECLARATION_PATTERN = /(^|\n)\s*export\s+(?:type\s+)?(?:[\s\S]*?\s+from\s*)['"]([^'"]+)['"]\s*;?/g;
-const FORBIDDEN_IMPORT_EXTENSION_PATTERN = /\.(?:ts|js)(?:$|[?#])/;
-
-const toPosixPath = (value: string): string => value.split(path.sep).join('/');
-
-const getRelativePath = (filePath: string): string => toPosixPath(path.relative(process.cwd(), filePath));
-
-const findLineByIndex = (content: string, index: number): number => {
-	let line = 1;
-	for (let i = 0; i < index; i += 1) {
-		if (content[i] === '\n') line += 1;
-	}
-	return line;
-};
-
-const listFiles = async (rootDir: string): Promise<string[]> => {
-	const files: string[] = [];
-	const stack: string[] = [rootDir];
-
-	while (stack.length) {
-		const current = stack.pop();
-		if (!current) continue;
-
-		let entries: import('node:fs').Dirent[] = [];
-		try {
-			entries = await fs.readdir(current, { withFileTypes: true });
-		} catch {
-			continue;
-		}
-
-		for (const entry of entries) {
-			if (entry.isDirectory()) {
-				if (!IGNORED_DIRS.has(entry.name)) {
-					stack.push(path.join(current, entry.name));
-				}
-				continue;
-			}
-
-			if (!entry.isFile()) continue;
-			const filePath = path.join(current, entry.name);
-			if (!TARGET_EXTENSIONS.has(path.extname(filePath))) continue;
-			files.push(filePath);
-		}
-	}
-
-	return files;
-};
-
-const getImportKind = (raw: string): ImportKind => {
-	const normalized = raw.trim();
-	if (/^import\s+type\b/.test(normalized)) return 'type';
-	if (/^import\s*['"]/.test(normalized)) return 'side-effect';
-	return 'value';
-};
-
-const getKindOrder = (kind: ImportKind): number => {
-	if (kind === 'value') return 0;
-	if (kind === 'type') return 1;
-	return 2;
-};
-
-const extractImports = (content: string): ImportEntry[] => {
-	const imports: ImportEntry[] = [];
-	const regex = new RegExp(IMPORT_DECLARATION_PATTERN.source, 'g');
-	let match: RegExpExecArray | null;
-
-	while ((match = regex.exec(content)) !== null) {
-		const raw = match[0].trim();
-		const source = match[2];
-		if (!source) continue;
-
-		imports.push({
-			kind: getImportKind(raw),
-			line: findLineByIndex(content, match.index + (match[1] === '\n' ? 1 : 0)),
-			source,
-			raw
-		});
-	}
-
-	return imports.sort((a, b) => a.line - b.line);
-};
-
-const extractExportSources = (content: string): Array<{ source: string; line: number }> => {
-	const exports: Array<{ source: string; line: number }> = [];
-	const regex = new RegExp(EXPORT_DECLARATION_PATTERN.source, 'g');
-	let match: RegExpExecArray | null;
-
-	while ((match = regex.exec(content)) !== null) {
-		const source = match[2];
-		if (!source) continue;
-		exports.push({
-			source,
-			line: findLineByIndex(content, match.index + (match[1] === '\n' ? 1 : 0))
-		});
-	}
-
-	return exports;
-};
-
-const hasForbiddenExtension = (source: string): boolean => {
-	const sourceWithoutQuery = source.split('?')[0] ?? source;
-	return FORBIDDEN_IMPORT_EXTENSION_PATTERN.test(sourceWithoutQuery);
-};
-
-const collectIssuesForFile = (filePath: string, content: string): ImportIssue[] => {
-	const issues: ImportIssue[] = [];
-	const imports = extractImports(content);
-	let lastKindOrder = 0;
-
-	for (const importEntry of imports) {
-		if (hasForbiddenExtension(importEntry.source)) {
-			issues.push({
-				file: filePath,
-				line: importEntry.line,
-				message: `Import "${importEntry.source}" must not include .ts/.js extension`
-			});
-		}
-
-		const currentOrder = getKindOrder(importEntry.kind);
-		if (currentOrder < lastKindOrder) {
-			issues.push({
-				file: filePath,
-				line: importEntry.line,
-				message: 'Import order violation: value imports first, then type imports, then side-effect imports'
-			});
-		}
-		lastKindOrder = Math.max(lastKindOrder, currentOrder);
-	}
-
-	for (const exportEntry of extractExportSources(content)) {
-		if (!hasForbiddenExtension(exportEntry.source)) continue;
-		issues.push({
-			file: filePath,
-			line: exportEntry.line,
-			message: `Export source "${exportEntry.source}" must not include .ts/.js extension`
-		});
-	}
-
-	return issues;
-};
-
-export default async function checkImports(): Promise<void> {
-	const srcRoot = path.join(process.cwd(), 'src');
-
-	try {
-		await fs.access(srcRoot);
-	} catch {
-		console.error('❌ src folder not found.');
-		process.exitCode = 1;
-		return;
-	}
-
-	const issues: ImportIssue[] = [];
-	const files = await listFiles(srcRoot);
-
-	for (const filePath of files) {
-		const content = await fs.readFile(filePath, 'utf-8');
-		issues.push(...collectIssuesForFile(filePath, content));
-	}
-
-	if (!issues.length) {
-		console.log('✅ Imports check passed: extensions and import groups are valid.');
-		return;
-	}
-
-	console.error('❌ Imports check failed:');
-	for (const issue of issues) {
-		console.error(`• ${getRelativePath(issue.file)}:${issue.line} — ${issue.message}`);
-	}
-
-	process.exitCode = 1;
-}

package/src/checks/checkInternal.ts

@@ -1,48 +0,0 @@
-import checkPresenterNames from './checkPresenterNames.js';
-import checkProviderNames from './checkProviderNames.js';
-import checkProviderGraph from './checkProviderGraph.js';
-import checkDomain from './checkDomain.js';
-import checkLayerBoundaries from './checkLayerBoundaries.js';
-import checkFoldersStructure from './checkFoldersStructure.js';
-import checkImports from './checkImports.js';
-import checkHooks from './checkHooks.js';
-import checkLayerImports from './checkLayerImports.js';
-
-type CheckTask = {
-	name: string;
-	run: () => Promise<void>;
-};
-
-const tasks: CheckTask[] = [
-	{ name: 'presenter-names', run: checkPresenterNames },
-	{ name: 'provider-names', run: checkProviderNames },
-	{ name: 'provider-graph', run: checkProviderGraph },
-	{ name: 'domain', run: checkDomain },
-	{ name: 'imports', run: checkImports },
-	{ name: 'hooks', run: checkHooks },
-	{ name: 'layer-imports', run: checkLayerImports },
-	{ name: 'layer-boundaries', run: checkLayerBoundaries },
-	{ name: 'folders-structure', run: checkFoldersStructure }
-];
-
-const runTask = async (task: CheckTask): Promise<void> => {
-	console.log(`\n▶ Running check ${task.name}...`);
-	process.exitCode = 0;
-	await task.run();
-
-	if ((process.exitCode ?? 0) !== 0) {
-		throw new Error(`Check "${task.name}" failed`);
-	}
-};
-
-export default async function checkInternal(): Promise<void> {
-	try {
-		for (const task of tasks) {
-			await runTask(task);
-		}
-		console.log('\n✅ Internal checks passed.');
-	} catch (error) {
-		console.error(`\n❌ Internal checks failed: ${error instanceof Error ? error.message : String(error)}`);
-		process.exitCode = 1;
-	}
-}

package/src/checks/checkLayerBoundaries.ts

@@ -1,255 +0,0 @@
-import fs from 'node:fs/promises';
-import path from 'node:path';
-import { loadUserConfig } from '../utils/loadConfig.js';
-import { normalizeContexts } from '../init/initAliases.js';
-import { SHARED_ESSENTIALS_DIR_NAME } from '../utils/sharedFoundation.js';
-
-type BoundaryIssue = {
-	file: string;
-	line: number;
-	message: string;
-};
-
-type ContextConfig = {
-	name: string;
-	alias: string;
-};
-
-const TARGET_EXTENSIONS = new Set(['.ts', '.js', '.svelte']);
-const IGNORED_DIRS = new Set(['node_modules', 'dist', '.svelte-kit', '.git']);
-
-const IMPORT_FROM_PATTERN = /\bimport\s+(?:type\s+)?[\s\S]*?\sfrom\s*['"]([^'"]+)['"]/g;
-const EXPORT_FROM_PATTERN = /\bexport\s+(?:type\s+)?\{[\s\S]*?\}\sfrom\s*['"]([^'"]+)['"]/g;
-const DYNAMIC_IMPORT_PATTERN = /\bimport\s*\(\s*['"]([^'"]+)['"]\s*\)/g;
-
-const toPosixPath = (value: string): string => value.split(path.sep).join('/');
-
-const normalizeAlias = (value: unknown, fallback = ''): string => {
-	const raw = String(value ?? '').trim();
-	if (!raw.length) return fallback;
-	return raw.startsWith('$') ? raw : `$${raw}`;
-};
-
-const findLineByIndex = (content: string, index: number): number => {
-	let line = 1;
-	for (let i = 0; i < index; i += 1) {
-		if (content[i] === '\n') line += 1;
-	}
-	return line;
-};
-
-const getRelativePath = (filePath: string): string => toPosixPath(path.relative(process.cwd(), filePath));
-
-const listFiles = async (rootDir: string): Promise<string[]> => {
-	const files: string[] = [];
-	const stack: string[] = [rootDir];
-
-	while (stack.length) {
-		const current = stack.pop();
-		if (!current) continue;
-
-		let entries: import('node:fs').Dirent[] = [];
-		try {
-			entries = await fs.readdir(current, { withFileTypes: true });
-		} catch {
-			continue;
-		}
-
-		for (const entry of entries) {
-			if (entry.isDirectory()) {
-				if (!IGNORED_DIRS.has(entry.name)) {
-					stack.push(path.join(current, entry.name));
-				}
-				continue;
-			}
-
-			if (!entry.isFile()) continue;
-			const filePath = path.join(current, entry.name);
-			if (!TARGET_EXTENSIONS.has(path.extname(filePath))) continue;
-			files.push(filePath);
-		}
-	}
-
-	return files;
-};
-
-const extractImports = (content: string): Array<{ value: string; line: number }> => {
-	const matches: Array<{ value: string; line: number }> = [];
-	const patterns = [IMPORT_FROM_PATTERN, EXPORT_FROM_PATTERN, DYNAMIC_IMPORT_PATTERN];
-
-	for (const pattern of patterns) {
-		const regex = new RegExp(pattern.source, 'g');
-		let match: RegExpExecArray | null;
-		while ((match = regex.exec(content)) !== null) {
-			const value = match[1];
-			if (!value) continue;
-			matches.push({ value, line: findLineByIndex(content, match.index) });
-		}
-	}
-
-	return matches;
-};
-
-const resolveAliasFromImport = (importPath: string, aliases: string[]): string | null => {
-	const sorted = [...aliases].sort((a, b) => b.length - a.length);
-	for (const alias of sorted) {
-		if (importPath === alias || importPath.startsWith(`${alias}/`)) {
-			return alias;
-		}
-	}
-	return null;
-};
-
-const resolveContextFromRelativeImport = (filePath: string, importPath: string, contexts: ContextConfig[]): ContextConfig | null => {
-	if (!importPath.startsWith('.')) return null;
-
-	const absoluteImportPath = path.resolve(path.dirname(filePath), importPath);
-	const appRoot = path.join(process.cwd(), 'src', 'app');
-
-	for (const context of contexts) {
-		const contextRoot = path.join(appRoot, context.name);
-		if (absoluteImportPath === contextRoot || absoluteImportPath.startsWith(`${contextRoot}${path.sep}`)) {
-			return context;
-		}
-	}
-
-	return null;
-};
-
-const isInside = (targetPath: string, rootPath: string): boolean =>
-	targetPath === rootPath || targetPath.startsWith(`${rootPath}${path.sep}`);
-
-const resolveAbsoluteRelativeImport = (filePath: string, importPath: string): string | null => {
-	if (!importPath.startsWith('.')) return null;
-	return path.resolve(path.dirname(filePath), importPath);
-};
-
-export default async function checkLayerBoundaries(): Promise<void> {
-	const config = await loadUserConfig();
-	const contexts = normalizeContexts(config.contexts) as ContextConfig[];
-
-	if (!contexts.length) {
-		console.error('❌ No contexts found in azure-net.config.ts/js. Fill `contexts` first.');
-		process.exitCode = 1;
-		return;
-	}
-
-	const sharedAliasRaw = normalizeAlias(config.sharedAlias, '');
-	const sharedAlias = sharedAliasRaw.length ? sharedAliasRaw : null;
-	if (!sharedAlias) {
-		console.error('❌ sharedAlias is missing in azure-net.config.ts/js. Set it explicitly to the shared context alias.');
-		process.exitCode = 1;
-		return;
-	}
-
-	const aliasToContext = new Map<string, ContextConfig>();
-	for (const context of contexts) {
-		aliasToContext.set(context.alias, context);
-	}
-
-	const knownAliases = [...contexts.map((context) => context.alias)];
-	const sharedContext = aliasToContext.get(sharedAlias);
-	if (!sharedContext) {
-		console.error(`❌ sharedAlias "${sharedAlias}" does not point to any configured context.`);
-		process.exitCode = 1;
-		return;
-	}
-
-	const issues: BoundaryIssue[] = [];
-
-	for (const context of contexts) {
-		const contextRoot = path.join(process.cwd(), 'src', 'app', context.name);
-		const isSharedContext = context.alias === sharedAlias;
-		const sharedEssentialsRoot = isSharedContext ? path.join(contextRoot, SHARED_ESSENTIALS_DIR_NAME) : null;
-
-		try {
-			await fs.access(contextRoot);
-		} catch {
-			continue;
-		}
-
-		const files = await listFiles(contextRoot);
-		const allowedAliases = new Set<string>([context.alias]);
-		if (context.alias !== sharedAlias) {
-			allowedAliases.add(sharedAlias);
-		}
-
-		for (const filePath of files) {
-			const content = await fs.readFile(filePath, 'utf-8');
-			const imports = extractImports(content);
-			const isSharedEssentialsFile = Boolean(sharedEssentialsRoot && isInside(filePath, sharedEssentialsRoot));
-
-			for (const entry of imports) {
-				const targetAlias = resolveAliasFromImport(entry.value, knownAliases);
-				if (!targetAlias) continue;
-
-				if (isSharedEssentialsFile && targetAlias === sharedAlias) {
-					const importRest = entry.value.slice(sharedAlias.length).replace(/^\/+/, '');
-					if (importRest.length && !importRest.startsWith(`${SHARED_ESSENTIALS_DIR_NAME}/`)) {
-						issues.push({
-							file: filePath,
-							line: entry.line,
-							message: `Layer boundary violation: shared essentials cannot import from shared "${importRest}". Essentials may depend only on shared essentials code.`
-						});
-						continue;
-					}
-				}
-
-				if (allowedAliases.has(targetAlias)) continue;
-
-				const targetContext = aliasToContext.get(targetAlias);
-				const baseMessage = targetContext
-					? `Layer boundary violation: context "${context.name}" cannot import from context "${targetContext.name}" (${targetAlias})`
-					: `Layer boundary violation: context "${context.name}" cannot import from "${targetAlias}"`;
-
-				issues.push({
-					file: filePath,
-					line: entry.line,
-					message: baseMessage
-				});
-			}
-
-			for (const entry of imports) {
-				const absoluteRelativeImport = resolveAbsoluteRelativeImport(filePath, entry.value);
-				if (
-					isSharedEssentialsFile &&
-					absoluteRelativeImport &&
-					sharedEssentialsRoot &&
-					isInside(absoluteRelativeImport, contextRoot) &&
-					!isInside(absoluteRelativeImport, sharedEssentialsRoot)
-				) {
-					issues.push({
-						file: filePath,
-						line: entry.line,
-						message: 'Layer boundary violation: shared essentials cannot import from shared layers/ui via relative path.'
-					});
-					continue;
-				}
-
-				const relativeTargetContext = resolveContextFromRelativeImport(filePath, entry.value, contexts);
-				if (!relativeTargetContext || relativeTargetContext.name === context.name) continue;
-				if (context.alias !== sharedAlias && relativeTargetContext.alias === sharedAlias) continue;
-
-				const baseMessage = `Layer boundary violation: context "${context.name}" cannot import from context "${relativeTargetContext.name}" via relative path`;
-
-				issues.push({
-					file: filePath,
-					line: entry.line,
-					message: baseMessage
-				});
-			}
-		}
-	}
-
-	if (!issues.length) {
-		console.log('✅ Layer boundaries check passed.');
-		return;
-	}
-
-	console.error('❌ Layer boundaries check failed:');
-	for (const issue of issues) {
-		console.error(`• ${getRelativePath(issue.file)}:${issue.line} — ${issue.message}`);
-	}
-
-	process.exitCode = 1;
-}

package/src/checks/checkLayerImports.ts

@@ -1,185 +0,0 @@
-import fs from 'node:fs/promises';
-import path from 'node:path';
-import { normalizeContexts } from '../init/initAliases.js';
-import { loadUserConfig } from '../utils/loadConfig.js';
-
-type LayerName = 'domain' | 'application' | 'infrastructure' | 'presentation';
-
-type ContextConfig = {
-	name: string;
-	alias: string;
-};
-
-type LayerImportIssue = {
-	file: string;
-	line: number;
-	message: string;
-};
-
-const TARGET_EXTENSIONS = new Set(['.ts', '.js', '.svelte']);
-const IGNORED_DIRS = new Set(['node_modules', 'dist', '.svelte-kit', '.git']);
-const LAYERS: LayerName[] = ['domain', 'application', 'infrastructure', 'presentation'];
-const FORBIDDEN_LAYER_IMPORTS: Record<LayerName, LayerName[]> = {
-	infrastructure: ['application', 'presentation'],
-	application: ['presentation'],
-	domain: ['application', 'infrastructure', 'presentation'],
-	presentation: []
-};
-
-const IMPORT_FROM_PATTERN = /\bimport\s+(?:type\s+)?[\s\S]*?\sfrom\s*['"]([^'"]+)['"]/g;
-const EXPORT_FROM_PATTERN = /\bexport\s+(?:type\s+)?\{[\s\S]*?\}\sfrom\s*['"]([^'"]+)['"]/g;
-const DYNAMIC_IMPORT_PATTERN = /\bimport\s*\(\s*['"]([^'"]+)['"]\s*\)/g;
-
-const toPosixPath = (value: string): string => value.split(path.sep).join('/');
-
-const findLineByIndex = (content: string, index: number): number => {
-	let line = 1;
-	for (let i = 0; i < index; i += 1) {
-		if (content[i] === '\n') line += 1;
-	}
-	return line;
-};
-
-const getRelativePath = (filePath: string): string => toPosixPath(path.relative(process.cwd(), filePath));
-
-const listFiles = async (rootDir: string): Promise<string[]> => {
-	const files: string[] = [];
-	const stack: string[] = [rootDir];
-
-	while (stack.length) {
-		const current = stack.pop();
-		if (!current) continue;
-
-		let entries: import('node:fs').Dirent[] = [];
-		try {
-			entries = await fs.readdir(current, { withFileTypes: true });
-		} catch {
-			continue;
-		}
-
-		for (const entry of entries) {
-			if (entry.isDirectory()) {
-				if (!IGNORED_DIRS.has(entry.name)) {
-					stack.push(path.join(current, entry.name));
-				}
-				continue;
-			}
-
-			if (!entry.isFile()) continue;
-			const filePath = path.join(current, entry.name);
-			if (!TARGET_EXTENSIONS.has(path.extname(filePath))) continue;
-			files.push(filePath);
-		}
-	}
-
-	return files;
-};
-
-const extractImports = (content: string): Array<{ value: string; line: number }> => {
-	const matches: Array<{ value: string; line: number }> = [];
-	const patterns = [IMPORT_FROM_PATTERN, EXPORT_FROM_PATTERN, DYNAMIC_IMPORT_PATTERN];
-
-	for (const pattern of patterns) {
-		const regex = new RegExp(pattern.source, 'g');
-		let match: RegExpExecArray | null;
-		while ((match = regex.exec(content)) !== null) {
-			const value = match[1];
-			if (!value) continue;
-			matches.push({ value, line: findLineByIndex(content, match.index) });
-		}
-	}
-
-	return matches;
-};
-
-const getLayerFromPath = (filePath: string, contextRoot: string): LayerName | null => {
-	for (const layer of LAYERS) {
-		const layerRoot = path.join(contextRoot, 'layers', layer);
-		if (filePath === layerRoot || filePath.startsWith(`${layerRoot}${path.sep}`)) {
-			return layer;
-		}
-	}
-
-	return null;
-};
-
-const getLayerFromAliasImport = (importPath: string, context: ContextConfig): LayerName | null => {
-	if (importPath !== context.alias && !importPath.startsWith(`${context.alias}/`)) return null;
-
-	const withoutAlias = importPath.slice(context.alias.length).replace(/^\/+/, '');
-	for (const layer of LAYERS) {
-		if (withoutAlias === `layers/${layer}` || withoutAlias.startsWith(`layers/${layer}/`)) return layer;
-		if (withoutAlias === layer || withoutAlias.startsWith(`${layer}/`)) return layer;
-	}
-
-	return null;
-};
-
-const getLayerFromRelativeImport = (filePath: string, importPath: string, contextRoot: string): LayerName | null => {
-	if (!importPath.startsWith('.')) return null;
-
-	const absoluteImportPath = path.resolve(path.dirname(filePath), importPath);
-	return getLayerFromPath(absoluteImportPath, contextRoot);
-};
-
-const createMessage = (sourceLayer: LayerName, targetLayer: LayerName, context: ContextConfig): string =>
-	`Layer import violation: "${context.name}" ${sourceLayer} layer cannot import from ${targetLayer} layer`;
-
-export default async function checkLayerImports(): Promise<void> {
-	const config = await loadUserConfig();
-	const contexts = normalizeContexts(config.contexts) as ContextConfig[];
-
-	if (!contexts.length) {
-		console.error('❌ No contexts found in azure-net.config.ts/js. Fill `contexts` first.');
-		process.exitCode = 1;
-		return;
-	}
-
-	const issues: LayerImportIssue[] = [];
-
-	for (const context of contexts) {
-		const contextRoot = path.join(process.cwd(), 'src', 'app', context.name);
-
-		try {
-			await fs.access(contextRoot);
-		} catch {
-			continue;
-		}
-
-		const files = await listFiles(path.join(contextRoot, 'layers'));
-
-		for (const filePath of files) {
-			const sourceLayer = getLayerFromPath(filePath, contextRoot);
-			if (!sourceLayer) continue;
-
-			const forbiddenLayers = FORBIDDEN_LAYER_IMPORTS[sourceLayer];
-			if (!forbiddenLayers.length) continue;
-
-			const content = await fs.readFile(filePath, 'utf-8');
-			const imports = extractImports(content);
-
-			for (const entry of imports) {
-				const targetLayer = getLayerFromAliasImport(entry.value, context) ?? getLayerFromRelativeImport(filePath, entry.value, contextRoot);
-				if (!targetLayer || !forbiddenLayers.includes(targetLayer)) continue;
-
-				issues.push({
-					file: filePath,
-					line: entry.line,
-					message: createMessage(sourceLayer, targetLayer, context)
-				});
-			}
-		}
-	}
-
-	if (!issues.length) {
-		console.log('✅ Layer imports check passed.');
-		return;
-	}
-
-	console.error('❌ Layer imports check failed:');
-	for (const issue of issues) {
-		console.error(`• ${getRelativePath(issue.file)}:${issue.line} — ${issue.message}`);
-	}
-
-	process.exitCode = 1;
-}