@azure-net/cli 5.1.0 → 6.0.0-next.2

package/src/checks/checkDomain.ts

@@ -1,223 +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';
-
-type DomainIssue = {
-	file: string;
-	line: number;
-	message: string;
-};
-
-type Declaration = {
-	kind: 'interface' | 'type';
-	name: string;
-	line: number;
-};
-
-const DOMAIN_FILE_EXTENSIONS = new Set(['.ts']);
-const IGNORED_DIRS = new Set(['node_modules', 'dist', '.svelte-kit', '.git']);
-
-const INTERFACE_DECLARATION_PATTERN = /(?:^|\n)\s*(?:export\s+)?interface\s+([A-Za-z_$][A-Za-z0-9_$]*)\b/g;
-const TYPE_DECLARATION_PATTERN = /(?:^|\n)\s*(?:export\s+)?type\s+([A-Za-z_$][A-Za-z0-9_$]*)\s*=/g;
-
-const EXPORT_STAR_PATTERN = /export\s+\*\s+from\s*['"][^'"]+['"]\s*;?/g;
-const EXPORT_FROM_PATTERN = /export\s+(?:type\s+)?\{[\s\S]*?\}\s+from\s*['"][^'"]+['"]\s*;?/g;
-const EXPORT_LOCAL_PATTERN = /export\s+(?:type\s+)?\{[\s\S]*?\}\s*;?/g;
-
-const toPosixPath = (value: string): string => value.split(path.sep).join('/');
-
-const stripComments = (content: string): string => content.replace(/\/\*[\s\S]*?\*\//g, '').replace(/(^|[^\\])\/\/.*$/gm, '$1');
-
-const stripExportStatements = (content: string): string =>
-	content.replace(EXPORT_STAR_PATTERN, '').replace(EXPORT_FROM_PATTERN, '').replace(EXPORT_LOCAL_PATTERN, '');
-
-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 getRelativeFilePath = (filePath: string): string => toPosixPath(path.relative(process.cwd(), filePath));
-
-const isEnumsPath = (filePath: string): boolean => {
-	const normalized = toPosixPath(filePath);
-	return normalized.includes('/enums/');
-};
-
-const isIndexFile = (filePath: string): boolean => path.basename(filePath) === 'index.ts';
-
-const startsWithInterfacePrefix = (filePath: string): boolean => /^I[A-Z]/.test(path.basename(filePath, '.ts'));
-
-const isExportOnlyIndex = (content: string): boolean => {
-	const withoutComments = stripComments(content);
-	const withoutExports = stripExportStatements(withoutComments);
-	return withoutExports.trim().length === 0;
-};
-
-const validateName = (declaration: Declaration): string | null => {
-	if (declaration.kind === 'interface') {
-		return /^I[A-Z]/.test(declaration.name) ? null : `Interface "${declaration.name}" must start with "I" and have uppercase second letter`;
-	}
-
-	return /^[IT][A-Z]/.test(declaration.name)
-		? null
-		: `Type "${declaration.name}" must start with "I" or "T" and have uppercase second letter`;
-};
-
-const extractDeclarations = (content: string): Declaration[] => {
-	const cleaned = stripComments(content);
-	const declarations: Declaration[] = [];
-
-	const interfaceRegex = new RegExp(INTERFACE_DECLARATION_PATTERN.source, 'g');
-	let interfaceMatch: RegExpExecArray | null;
-	while ((interfaceMatch = interfaceRegex.exec(cleaned)) !== null) {
-		const name = interfaceMatch[1];
-		if (!name) continue;
-		declarations.push({
-			kind: 'interface',
-			name,
-			line: findLineByIndex(cleaned, interfaceMatch.index)
-		});
-	}
-
-	const typeRegex = new RegExp(TYPE_DECLARATION_PATTERN.source, 'g');
-	let typeMatch: RegExpExecArray | null;
-	while ((typeMatch = typeRegex.exec(cleaned)) !== null) {
-		const name = typeMatch[1];
-		if (!name) continue;
-		declarations.push({
-			kind: 'type',
-			name,
-			line: findLineByIndex(cleaned, typeMatch.index)
-		});
-	}
-
-	return declarations.sort((a, b) => a.line - b.line);
-};
-
-const listDomainFiles = async (domainRoot: string): Promise<string[]> => {
-	const files: string[] = [];
-	const stack: string[] = [domainRoot];
-
-	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 (!DOMAIN_FILE_EXTENSIONS.has(path.extname(filePath))) continue;
-			files.push(filePath);
-		}
-	}
-
-	return files;
-};
-
-const collectIssuesForFile = (filePath: string, content: string): DomainIssue[] => {
-	const issues: DomainIssue[] = [];
-
-	if (isIndexFile(filePath) && !isExportOnlyIndex(content)) {
-		issues.push({
-			file: filePath,
-			line: 1,
-			message: 'index.ts must contain only exports. Move declarations to separate files'
-		});
-	}
-
-	if (isEnumsPath(filePath)) {
-		return issues;
-	}
-
-	if (!isIndexFile(filePath) && startsWithInterfacePrefix(filePath)) {
-		issues.push({
-			file: filePath,
-			line: 1,
-			message: 'Domain file names must not start with "I". Keep "I" only in interface/type names'
-		});
-	}
-
-	const declarations = extractDeclarations(content);
-	if (declarations.length > 5) {
-		issues.push({
-			file: filePath,
-			line: declarations[5]?.line ?? 1,
-			message: `Too many declarations in one file (${declarations.length}). Maximum is 5`
-		});
-	}
-
-	for (const declaration of declarations) {
-		const validationError = validateName(declaration);
-		if (!validationError) continue;
-		issues.push({
-			file: filePath,
-			line: declaration.line,
-			message: validationError
-		});
-	}
-
-	return issues;
-};
-
-export default async function checkDomain(): Promise<void> {
-	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: DomainIssue[] = [];
-
-	for (const context of contexts) {
-		const domainRoot = path.join(process.cwd(), 'src', 'app', context.name, 'layers', 'domain');
-
-		try {
-			await fs.access(domainRoot);
-		} catch {
-			issues.push({
-				file: domainRoot,
-				line: 1,
-				message: `Domain folder not found for context "${context.name}"`
-			});
-			continue;
-		}
-
-		const files = await listDomainFiles(domainRoot);
-		for (const filePath of files) {
-			const content = await fs.readFile(filePath, 'utf-8');
-			issues.push(...collectIssuesForFile(filePath, content));
-		}
-	}
-
-	if (!issues.length) {
-		console.log('✅ Domain check passed: naming, per-file limits and index exports are valid.');
-		return;
-	}
-
-	console.error('❌ Domain check failed:');
-	for (const issue of issues) {
-		console.error(`• ${getRelativeFilePath(issue.file)}:${issue.line} — ${issue.message}`);
-	}
-
-	process.exitCode = 1;
-}

package/src/checks/checkFoldersStructure.ts

@@ -1,85 +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';
-
-type StructureIssue = {
-	file: string;
-	message: string;
-};
-
-type ContextConfig = {
-	name: string;
-	alias: string;
-};
-
-const ALLOWED_LAYERS_FOLDERS = new Set(['domain', 'presentation', 'infrastructure', 'application']);
-const ALLOWED_UI_FOLDERS = new Set(['components', 'widgets', 'design', 'style']);
-const ALLOWED_COMPONENTS_FOLDERS = new Set(['page', 'layout', 'shared']);
-
-const toPosixPath = (value: string): string => value.split(path.sep).join('/');
-
-const getRelativePath = (filePath: string): string => toPosixPath(path.relative(process.cwd(), filePath));
-
-const getDirectories = async (dirPath: string): Promise<string[] | null> => {
-	try {
-		const entries = await fs.readdir(dirPath, { withFileTypes: true });
-		return entries.filter((entry) => entry.isDirectory()).map((entry) => entry.name);
-	} catch {
-		return null;
-	}
-};
-
-const checkOnlyAllowedFolders = async (
-	basePath: string,
-	allowed: Set<string>,
-	scopeLabel: string,
-	issues: StructureIssue[]
-): Promise<void> => {
-	const directories = await getDirectories(basePath);
-	if (!directories) return;
-
-	for (const dir of directories) {
-		if (allowed.has(dir)) continue;
-		issues.push({
-			file: basePath,
-			message: `${scopeLabel} can contain only: ${Array.from(allowed).join(', ')}. Found: ${dir}`
-		});
-	}
-};
-
-export default async function checkFoldersStructure(): 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: StructureIssue[] = [];
-
-	for (const context of contexts) {
-		const contextRoot = path.join(process.cwd(), 'src', 'app', context.name);
-		const layersPath = path.join(contextRoot, 'layers');
-		const uiPath = path.join(contextRoot, 'ui');
-		const componentsPath = path.join(uiPath, 'components');
-
-		await checkOnlyAllowedFolders(layersPath, ALLOWED_LAYERS_FOLDERS, `layers (${context.name})`, issues);
-		await checkOnlyAllowedFolders(uiPath, ALLOWED_UI_FOLDERS, `ui (${context.name})`, issues);
-		await checkOnlyAllowedFolders(componentsPath, ALLOWED_COMPONENTS_FOLDERS, `ui/components (${context.name})`, issues);
-	}
-
-	if (!issues.length) {
-		console.log('✅ Folders structure check passed.');
-		return;
-	}
-
-	console.error('❌ Folders structure check failed:');
-	for (const issue of issues) {
-		console.error(`• ${getRelativePath(issue.file)} — ${issue.message}`);
-	}
-
-	process.exitCode = 1;
-}

package/src/checks/checkHooks.ts

@@ -1,22 +0,0 @@
-import fs from 'node:fs';
-import path from 'node:path';
-
-const forbiddenHooks = ['hooks.server.ts', 'hooks.server.js', 'hooks.client.ts', 'hooks.client.js'];
-
-export default async function checkHooks(): Promise<void> {
-	const srcPath = path.join(process.cwd(), 'src');
-	const found = forbiddenHooks.map((filename) => path.join(srcPath, filename)).filter((filepath) => fs.existsSync(filepath));
-
-	if (!found.length) {
-		console.log('✅ No manual SvelteKit hooks found.');
-		return;
-	}
-
-	console.error('❌ Manual SvelteKit hooks are not allowed with @azure-net/kit AzureNetPlugin.');
-	console.error('Move lifecycle code to src/program.ts through createApp(). Found:');
-	for (const filepath of found) {
-		console.error(` - ${path.relative(process.cwd(), filepath)}`);
-	}
-
-	process.exitCode = 1;
-}

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;
-	}
-}