@azure-net/cli 4.0.0-0 → 4.1.0

package/src/checks/checkPresenterNames.ts

@@ -0,0 +1,208 @@
+import fs from 'node:fs/promises';
+import path from 'node:path';
+
+type PresenterOccurrence = {
+	id: string;
+	file: string;
+	line: number;
+	callee: string;
+};
+
+const TARGET_EXTENSIONS = new Set(['.ts', '.js', '.svelte']);
+const IGNORED_DIRS = new Set(['node_modules', 'dist', '.svelte-kit', '.git']);
+
+const CREATE_FACTORY_PATTERN = '(?:export\\s+)?(?:const|let|var)\\s+([A-Za-z_$][A-Za-z0-9_$]*)\\s*=\\s*createPresenterFactory\\s*\\(';
+const IMPORT_FROM_PRESENTER_PATTERN = 'import\\s*\\{([^}]+)\\}\\s*from\\s*[\'"]([^\'"]+)[\'"]';
+const PRESENTER_CALL_PATTERN = '\\b([A-Za-z_$][A-Za-z0-9_$]*)\\s*\\(\\s*([\'"`])([^\'"`]*)\\2';
+
+const toPosixPath = (value: string): string => value.split(path.sep).join('/');
+
+const isPresenterImportPath = (importPath: string): boolean => {
+	const normalized = importPath.replace(/\\/g, '/');
+	return /(^|\/)presenter(\/|$)/.test(normalized);
+};
+
+const parseImportLocals = (rawImports: string): string[] => {
+	const locals: string[] = [];
+	for (const segment of rawImports.split(',')) {
+		const item = segment.trim();
+		if (!item.length) continue;
+		const aliasMatch = item.match(/^([A-Za-z_$][A-Za-z0-9_$]*)\s+as\s+([A-Za-z_$][A-Za-z0-9_$]*)$/);
+		if (aliasMatch?.[2]) {
+			locals.push(aliasMatch[2]);
+			continue;
+		}
+
+		const directMatch = item.match(/^([A-Za-z_$][A-Za-z0-9_$]*)$/);
+		if (directMatch?.[1]) {
+			locals.push(directMatch[1]);
+		}
+	}
+	return locals;
+};
+
+const listSourceFiles = async (rootDir: string): Promise<string[]> => {
+	const result: 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 fullPath = path.join(current, entry.name);
+			const extension = path.extname(fullPath);
+			if (!TARGET_EXTENSIONS.has(extension)) continue;
+			result.push(fullPath);
+		}
+	}
+
+	return result;
+};
+
+const getLineByIndex = (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 extractFactoryNamesFromContent = (content: string): string[] => {
+	const createFactoryRegex = new RegExp(CREATE_FACTORY_PATTERN, 'g');
+	const names: string[] = [];
+	let match: RegExpExecArray | null;
+	while ((match = createFactoryRegex.exec(content)) !== null) {
+		const name = match[1];
+		if (name) names.push(name);
+	}
+	return names;
+};
+
+const extractImportedPresenterFactories = (content: string): string[] => {
+	const importFromPresenterRegex = new RegExp(IMPORT_FROM_PRESENTER_PATTERN, 'g');
+	const names: string[] = [];
+	let match: RegExpExecArray | null;
+	while ((match = importFromPresenterRegex.exec(content)) !== null) {
+		const rawImports = match[1];
+		const importPath = match[2];
+		if (!rawImports || !importPath) continue;
+		if (!isPresenterImportPath(importPath)) continue;
+		names.push(...parseImportLocals(rawImports));
+	}
+	return names;
+};
+
+const collectPresenterOccurrences = (content: string, filePath: string, knownFactories: Set<string>): PresenterOccurrence[] => {
+	const presenterCallRegex = new RegExp(PRESENTER_CALL_PATTERN, 'g');
+	const occurrences: PresenterOccurrence[] = [];
+	const availableCallees = new Set<string>(['createPresenter']);
+
+	for (const name of extractFactoryNamesFromContent(content)) {
+		availableCallees.add(name);
+	}
+	for (const name of extractImportedPresenterFactories(content)) {
+		availableCallees.add(name);
+	}
+	for (const name of knownFactories) {
+		availableCallees.add(name);
+	}
+
+	let match: RegExpExecArray | null;
+	while ((match = presenterCallRegex.exec(content)) !== null) {
+		const callee = match[1];
+		const quote = match[2];
+		const presenterId = match[3];
+		if (!callee || !presenterId) continue;
+		if (!availableCallees.has(callee)) continue;
+		if (quote === '`' && presenterId.includes('${')) continue;
+
+		occurrences.push({
+			id: presenterId,
+			file: filePath,
+			line: getLineByIndex(content, match.index),
+			callee
+		});
+	}
+
+	return occurrences;
+};
+
+export default async function checkPresenterNames(): Promise<void> {
+	const srcPath = path.join(process.cwd(), 'src');
+	try {
+		await fs.access(srcPath);
+	} catch {
+		console.error('❌ src folder not found in current workspace.');
+		process.exitCode = 1;
+		return;
+	}
+
+	const files = await listSourceFiles(srcPath);
+	const fileContents = new Map<string, string>();
+
+	for (const filePath of files) {
+		const content = await fs.readFile(filePath, 'utf-8');
+		fileContents.set(filePath, content);
+	}
+
+	const globalFactoryNames = new Set<string>();
+	for (const [filePath, content] of fileContents.entries()) {
+		for (const name of extractFactoryNamesFromContent(content)) {
+			globalFactoryNames.add(name);
+		}
+		if (filePath.endsWith(path.join('core', 'presenter', 'index.ts')) || filePath.endsWith(path.join('core', 'presenter', 'index.js'))) {
+			for (const name of extractImportedPresenterFactories(content)) {
+				globalFactoryNames.add(name);
+			}
+		}
+	}
+
+	const occurrences: PresenterOccurrence[] = [];
+	for (const [filePath, content] of fileContents.entries()) {
+		occurrences.push(...collectPresenterOccurrences(content, filePath, globalFactoryNames));
+	}
+
+	if (!occurrences.length) {
+		console.log('✅ No presenter declarations found.');
+		return;
+	}
+
+	const grouped = new Map<string, PresenterOccurrence[]>();
+	for (const occurrence of occurrences) {
+		const list = grouped.get(occurrence.id) ?? [];
+		list.push(occurrence);
+		grouped.set(occurrence.id, list);
+	}
+
+	const duplicated = Array.from(grouped.entries()).filter(([, list]) => list.length > 1);
+	if (!duplicated.length) {
+		console.log(`✅ Presenter names are unique (${occurrences.length} found).`);
+		return;
+	}
+
+	console.error('❌ Duplicate presenter names found:');
+	for (const [presenterId, list] of duplicated) {
+		console.error(`\n- "${presenterId}"`);
+		for (const item of list) {
+			console.error(`  • ${toPosixPath(path.relative(process.cwd(), item.file))}:${item.line} (${item.callee})`);
+		}
+	}
+
+	process.exitCode = 1;
+}

package/src/checks/checkProject.js

@@ -0,0 +1,42 @@
+import { execSync } from 'node:child_process';
+import { loadUserConfig } from '../utils/loadConfig.js';
+import checkInternal from './checkInternal.js';
+
+const isPackageManager = (value) => typeof value === 'string' && ['pnpm', 'npm', 'yarn', 'bun'].includes(value);
+
+const runScript = (manager, script) => {
+	const command = `${manager} run ${script}`;
+	console.log(`\n▶ Running: ${command}`);
+	execSync(command, { stdio: 'inherit' });
+};
+
+const runInternalChecks = async () => {
+	console.log('\n▶ Running: azure-net check internal');
+	process.exitCode = 0;
+	await checkInternal();
+	if ((process.exitCode ?? 0) !== 0) {
+		throw new Error('azure-net check internal failed');
+	}
+};
+
+export default async function checkProject() {
+	const config = await loadUserConfig();
+	const manager = config.packageManager;
+
+	if (!isPackageManager(manager)) {
+		console.error('❌ packageManager is missing or invalid in azure-net.config.ts/js. Use one of: bun, pnpm, npm, yarn');
+		process.exitCode = 1;
+		return;
+	}
+
+	try {
+		runScript(manager, 'format');
+		await runInternalChecks();
+		runScript(manager, 'lint');
+		runScript(manager, 'check');
+		console.log('\n✅ Project checks passed.');
+	} catch (error) {
+		console.error(`\n❌ Project checks failed: ${error instanceof Error ? error.message : String(error)}`);
+		process.exitCode = 1;
+	}
+}

package/src/checks/checkProject.ts

@@ -0,0 +1,45 @@
+import { execSync } from 'node:child_process';
+import { loadUserConfig } from '../utils/loadConfig.js';
+import checkInternal from './checkInternal.js';
+
+type PackageManager = 'pnpm' | 'npm' | 'yarn' | 'bun';
+
+const isPackageManager = (value: unknown): value is PackageManager =>
+	typeof value === 'string' && ['pnpm', 'npm', 'yarn', 'bun'].includes(value);
+
+const runScript = (manager: PackageManager, script: string): void => {
+	const command = `${manager} run ${script}`;
+	console.log(`\n▶ Running: ${command}`);
+	execSync(command, { stdio: 'inherit' });
+};
+
+const runInternalChecks = async (): Promise<void> => {
+	console.log('\n▶ Running: azure-net check internal');
+	process.exitCode = 0;
+	await checkInternal();
+	if ((process.exitCode ?? 0) !== 0) {
+		throw new Error('azure-net check internal failed');
+	}
+};
+
+export default async function checkProject(): Promise<void> {
+	const config = await loadUserConfig();
+	const manager = config.packageManager;
+
+	if (!isPackageManager(manager)) {
+		console.error('❌ packageManager is missing or invalid in azure-net.config.ts/js. Use one of: bun, pnpm, npm, yarn');
+		process.exitCode = 1;
+		return;
+	}
+
+	try {
+		runScript(manager, 'format');
+		await runInternalChecks();
+		runScript(manager, 'lint');
+		runScript(manager, 'check');
+		console.log('\n✅ Project checks passed.');
+	} catch (error) {
+		console.error(`\n❌ Project checks failed: ${error instanceof Error ? error.message : String(error)}`);
+		process.exitCode = 1;
+	}
+}

package/src/checks/checkProviderGraph.js

@@ -0,0 +1,221 @@
+import fs from 'node:fs/promises';
+import path from 'node:path';
+
+const TARGET_EXTENSIONS = new Set(['.ts', '.js']);
+const IGNORED_DIRS = new Set(['node_modules', 'dist', '.svelte-kit', '.git']);
+
+const toPosixPath = (value) => value.split(path.sep).join('/');
+
+const getLineByIndex = (content, index) => {
+	let line = 1;
+	for (let i = 0; i < index; i += 1) {
+		if (content[i] === '\n') line += 1;
+	}
+	return line;
+};
+
+const listSourceFiles = 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 findMatchingBracket = (content, startIndex, openChar, closeChar) => {
+	let depth = 0;
+	for (let i = startIndex; i < content.length; i += 1) {
+		const ch = content[i];
+		if (ch === openChar) depth += 1;
+		if (ch === closeChar) {
+			depth -= 1;
+			if (depth === 0) return i;
+		}
+	}
+	return -1;
+};
+
+const extractDependsOn = (configBlock) => {
+	const dependsOnMatch = configBlock.match(/dependsOn\s*:\s*\{([\s\S]*?)\}/m);
+	if (!dependsOnMatch?.[1]) return [];
+
+	const block = dependsOnMatch[1];
+	const deps = new Set();
+	const keyRegex = /([A-Za-z_$][A-Za-z0-9_$]*)\s*:/g;
+	let match;
+	while ((match = keyRegex.exec(block)) !== null) {
+		const key = match[1];
+		if (key) deps.add(key);
+	}
+	return [...deps];
+};
+
+const parseProviderGraphFromFile = (content, filePath) => {
+	const nodes = [];
+	const token = 'createBoundaryProvider(';
+	let fromIndex = 0;
+
+	while (fromIndex < content.length) {
+		const start = content.indexOf(token, fromIndex);
+		if (start === -1) break;
+
+		const openParenIndex = start + token.length - 1;
+		const closeParenIndex = findMatchingBracket(content, openParenIndex, '(', ')');
+		if (closeParenIndex === -1) break;
+
+		const fullCall = content.slice(openParenIndex + 1, closeParenIndex);
+		const idMatch = fullCall.match(/^\s*(['"`])([^'"`]+)\1\s*,/m);
+		const providerName = idMatch?.[2];
+		if (providerName) {
+			const objectStartInCall = fullCall.indexOf('{');
+			if (objectStartInCall !== -1) {
+				const absoluteObjectStart = openParenIndex + 1 + objectStartInCall;
+				const absoluteObjectEnd = findMatchingBracket(content, absoluteObjectStart, '{', '}');
+				const configBlock =
+					absoluteObjectEnd !== -1 ? content.slice(absoluteObjectStart, absoluteObjectEnd + 1) : fullCall.slice(objectStartInCall);
+
+				nodes.push({
+					name: providerName,
+					file: filePath,
+					line: getLineByIndex(content, start),
+					dependsOn: extractDependsOn(configBlock)
+				});
+			} else {
+				nodes.push({
+					name: providerName,
+					file: filePath,
+					line: getLineByIndex(content, start),
+					dependsOn: []
+				});
+			}
+		}
+
+		fromIndex = closeParenIndex + 1;
+	}
+
+	return nodes;
+};
+
+const findCycles = (graph) => {
+	const cycles = [];
+	const visited = new Set();
+	const inStack = new Set();
+	const stack = [];
+
+	const dfs = (node) => {
+		visited.add(node);
+		inStack.add(node);
+		stack.push(node);
+
+		const neighbors = graph.get(node) ?? [];
+		for (const next of neighbors) {
+			if (!graph.has(next)) continue;
+			if (!visited.has(next)) {
+				dfs(next);
+				continue;
+			}
+			if (inStack.has(next)) {
+				const idx = stack.indexOf(next);
+				if (idx !== -1) cycles.push([...stack.slice(idx), next]);
+			}
+		}
+
+		stack.pop();
+		inStack.delete(node);
+	};
+
+	for (const node of graph.keys()) {
+		if (!visited.has(node)) dfs(node);
+	}
+
+	return cycles;
+};
+
+export default async function checkProviderGraph() {
+	const srcPath = path.join(process.cwd(), 'src');
+	try {
+		await fs.access(srcPath);
+	} catch {
+		console.error('❌ src folder not found in current workspace.');
+		process.exitCode = 1;
+		return;
+	}
+
+	const files = await listSourceFiles(srcPath);
+	const providers = [];
+
+	for (const filePath of files) {
+		const content = await fs.readFile(filePath, 'utf-8');
+		providers.push(...parseProviderGraphFromFile(content, filePath));
+	}
+
+	if (!providers.length) {
+		console.log('✅ No createBoundaryProvider declarations found.');
+		return;
+	}
+
+	const graph = new Map();
+	const issues = [];
+	const byName = new Map();
+
+	for (const provider of providers) {
+		if (byName.has(provider.name)) {
+			const existing = byName.get(provider.name);
+			issues.push({
+				file: provider.file,
+				line: provider.line,
+				message: `Provider graph ambiguity: duplicate provider id "${provider.name}" (first declared at ${toPosixPath(
+					path.relative(process.cwd(), existing.file)
+				)}:${existing.line})`
+			});
+			continue;
+		}
+		byName.set(provider.name, provider);
+		graph.set(provider.name, provider.dependsOn);
+	}
+
+	const cycles = findCycles(graph);
+	for (const cycle of cycles) {
+		const firstNode = byName.get(cycle[0] ?? '');
+		issues.push({
+			file: firstNode?.file ?? srcPath,
+			line: firstNode?.line ?? 1,
+			message: `Circular provider dependency detected: ${cycle.join(' -> ')}`
+		});
+	}
+
+	if (!issues.length) {
+		console.log(`✅ Provider graph check passed (${providers.length} providers).`);
+		return;
+	}
+
+	console.error('❌ Provider graph check failed:');
+	for (const issue of issues) {
+		console.error(`• ${toPosixPath(path.relative(process.cwd(), issue.file))}:${issue.line} — ${issue.message}`);
+	}
+
+	process.exitCode = 1;
+}