@azure-net/cli 5.0.0-1 → 5.0.0-3

package/bin/azure-net.js

@@ -257,6 +257,14 @@ checkCommand
 		await handler();
 	});
 
+checkCommand
+	.command('layer-imports')
+	.description('Check forbidden imports between layers inside each context')
+	.action(async () => {
+		const handler = (await import('../src/checks/checkLayerImports.js')).default;
+		await handler();
+	});
+
 checkCommand
 	.command('layer-boundaries')
 	.description('Check layer boundaries between contexts by aliases')

package/bin/azure-net.ts

@@ -257,6 +257,14 @@ checkCommand
 		await handler();
 	});
 
+checkCommand
+	.command('layer-imports')
+	.description('Check forbidden imports between layers inside each context')
+	.action(async () => {
+		const handler = (await import('../src/checks/checkLayerImports.js')).default;
+		await handler();
+	});
+
 checkCommand
 	.command('layer-boundaries')
 	.description('Check layer boundaries between contexts by aliases')

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@azure-net/cli",
-  "version": "5.0.0-1",
+  "version": "5.0.0-3",
   "type": "module",
   "bin": {
     "azure-net": "bin/azure-net.js"

package/src/checks/checkInternal.js

@@ -6,6 +6,7 @@ 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';
 const tasks = [
 	{ name: 'presenter-names', run: checkPresenterNames },
 	{ name: 'provider-names', run: checkProviderNames },
@@ -13,6 +14,7 @@ const tasks = [
 	{ 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 }
 ];

package/src/checks/checkInternal.ts

@@ -6,6 +6,7 @@ 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;
@@ -19,6 +20,7 @@ const tasks: CheckTask[] = [
 	{ 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 }
 ];

package/src/checks/checkLayerImports.js

@@ -0,0 +1,136 @@
+import fs from 'node:fs/promises';
+import path from 'node:path';
+import { normalizeContexts } from '../init/initAliases.js';
+import { loadUserConfig } from '../utils/loadConfig.js';
+const TARGET_EXTENSIONS = new Set(['.ts', '.js', '.svelte']);
+const IGNORED_DIRS = new Set(['node_modules', 'dist', '.svelte-kit', '.git']);
+const LAYERS = ['domain', 'application', 'infrastructure', 'presentation'];
+const FORBIDDEN_LAYER_IMPORTS = {
+	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) => value.split(path.sep).join('/');
+const findLineByIndex = (content, index) => {
+	let line = 1;
+	for (let i = 0; i < index; i += 1) {
+		if (content[i] === '\n') line += 1;
+	}
+	return line;
+};
+const getRelativePath = (filePath) => toPosixPath(path.relative(process.cwd(), filePath));
+const listFiles = async (rootDir) => {
+	const files = [];
+	const stack = [rootDir];
+	while (stack.length) {
+		const current = stack.pop();
+		if (!current) continue;
+		let entries = [];
+		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) => {
+	const matches = [];
+	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;
+		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, contextRoot) => {
+	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, context) => {
+	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, importPath, contextRoot) => {
+	if (!importPath.startsWith('.')) return null;
+	const absoluteImportPath = path.resolve(path.dirname(filePath), importPath);
+	return getLayerFromPath(absoluteImportPath, contextRoot);
+};
+const createMessage = (sourceLayer, targetLayer, context) =>
+	`Layer import violation: "${context.name}" ${sourceLayer} layer cannot import from ${targetLayer} layer`;
+export default async function checkLayerImports() {
+	const config = await loadUserConfig();
+	const contexts = normalizeContexts(config.contexts);
+	if (!contexts.length) {
+		console.error('❌ No contexts found in azure-net.config.ts/js. Fill `contexts` first.');
+		process.exitCode = 1;
+		return;
+	}
+	const issues = [];
+	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;
+}

package/src/checks/checkLayerImports.ts

@@ -0,0 +1,185 @@
+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;
+}

package/src/checks/index.js

@@ -4,6 +4,7 @@ export { default as checkProviderGraph } from './checkProviderGraph.js';
 export { default as checkDomain } from './checkDomain.js';
 export { default as checkImports } from './checkImports.js';
 export { default as checkHooks } from './checkHooks.js';
+export { default as checkLayerImports } from './checkLayerImports.js';
 export { default as checkLayerBoundaries } from './checkLayerBoundaries.js';
 export { default as checkFoldersStructure } from './checkFoldersStructure.js';
 export { default as checkInternal } from './checkInternal.js';

package/src/checks/index.ts

@@ -4,6 +4,7 @@ export { default as checkProviderGraph } from './checkProviderGraph.js';
 export { default as checkDomain } from './checkDomain.js';
 export { default as checkImports } from './checkImports.js';
 export { default as checkHooks } from './checkHooks.js';
+export { default as checkLayerImports } from './checkLayerImports.js';
 export { default as checkLayerBoundaries } from './checkLayerBoundaries.js';
 export { default as checkFoldersStructure } from './checkFoldersStructure.js';
 export { default as checkInternal } from './checkInternal.js';