autotel-cli 0.18.5 → 0.18.6

package/README.md

@@ -390,7 +390,7 @@ pnpm build
 pnpm test
 
 # Type check
-pnpm typecheck
+pnpm type-check
 ```
 
 ## License

package/dist/index.js

@@ -442,9 +442,10 @@ function sortImports(imports) {
 	const sideEffect = [];
 	const external = [];
 	const relative = [];
-	for (const imp of imports) if (imp.sideEffect) if (imp.source === "autotel/register") sideEffect.unshift(imp);
-	else sideEffect.push(imp);
-	else if (imp.source.startsWith(".") || imp.source.startsWith("/")) relative.push(imp);
+	for (const imp of imports) if (imp.sideEffect) {
+		if (imp.source === "autotel/register") sideEffect.unshift(imp);
+		else sideEffect.push(imp);
+	} else if (imp.source.startsWith(".") || imp.source.startsWith("/")) relative.push(imp);
 	else external.push(imp);
 	const sortBySource = (a, b) => a.source.localeCompare(b.source);
 	external.sort(sortBySource);
@@ -3024,32 +3025,34 @@ function applyPlan(args) {
 	}
 	const prod = plan.packagesToInstall.prod;
 	const dev = plan.packagesToInstall.dev;
-	if (prod.length > 0 || dev.length > 0) if (options.noInstall || options.printInstallCmd) {
-		if (prod.length > 0) result.printedInstalls.push(getInstallCommand(project.packageManager, prod));
-		if (dev.length > 0) result.printedInstalls.push(getInstallCommand(project.packageManager, dev, { dev: true }));
-	} else {
-		if (prod.length > 0) {
-			const cmd = getInstallCommand(project.packageManager, prod);
-			try {
-				execSync(cmd, {
-					cwd: project.packageRoot,
-					stdio: "pipe"
-				});
-				result.ranInstalls.push(cmd);
-			} catch {
-				result.installErrors.push(cmd);
+	if (prod.length > 0 || dev.length > 0) {
+		if (options.noInstall || options.printInstallCmd) {
+			if (prod.length > 0) result.printedInstalls.push(getInstallCommand(project.packageManager, prod));
+			if (dev.length > 0) result.printedInstalls.push(getInstallCommand(project.packageManager, dev, { dev: true }));
+		} else {
+			if (prod.length > 0) {
+				const cmd = getInstallCommand(project.packageManager, prod);
+				try {
+					execSync(cmd, {
+						cwd: project.packageRoot,
+						stdio: "pipe"
+					});
+					result.ranInstalls.push(cmd);
+				} catch {
+					result.installErrors.push(cmd);
+				}
 			}
-		}
-		if (dev.length > 0) {
-			const cmd = getInstallCommand(project.packageManager, dev, { dev: true });
-			try {
-				execSync(cmd, {
-					cwd: project.packageRoot,
-					stdio: "pipe"
-				});
-				result.ranInstalls.push(cmd);
-			} catch {
-				result.installErrors.push(cmd);
+			if (dev.length > 0) {
+				const cmd = getInstallCommand(project.packageManager, dev, { dev: true });
+				try {
+					execSync(cmd, {
+						cwd: project.packageRoot,
+						stdio: "pipe"
+					});
+					result.ranInstalls.push(cmd);
+				} catch {
+					result.installErrors.push(cmd);
+				}
 			}
 		}
 	}
@@ -3471,29 +3474,30 @@ async function runChecks(options, projectRoot) {
 		}
 	});
 	const config = detectConfig(project.packageRoot);
-	if (config.found) if (config.type === "cli-owned") checks.push({
-		id: "config-found",
-		title: "Configuration found",
-		level: "info",
-		status: "ok",
-		message: `CLI-owned instrumentation at ${config.path}`
-	});
-	else if (config.type === "user-created") checks.push({
-		id: "config-found",
-		title: "Configuration found",
-		level: "info",
-		status: "ok",
-		message: `User-created instrumentation at ${config.path}`,
-		details: ["Add CLI header to enable auto-updates"]
-	});
-	else checks.push({
-		id: "config-found",
-		title: "Configuration found",
-		level: "info",
-		status: "ok",
-		message: `Config found at ${config.path}`
-	});
-	else checks.push({
+	if (config.found) {
+		if (config.type === "cli-owned") checks.push({
+			id: "config-found",
+			title: "Configuration found",
+			level: "info",
+			status: "ok",
+			message: `CLI-owned instrumentation at ${config.path}`
+		});
+		else if (config.type === "user-created") checks.push({
+			id: "config-found",
+			title: "Configuration found",
+			level: "info",
+			status: "ok",
+			message: `User-created instrumentation at ${config.path}`,
+			details: ["Add CLI header to enable auto-updates"]
+		});
+		else checks.push({
+			id: "config-found",
+			title: "Configuration found",
+			level: "info",
+			status: "ok",
+			message: `Config found at ${config.path}`
+		});
+	} else checks.push({
 		id: "config-found",
 		title: "Configuration found",
 		level: "info",
@@ -3568,22 +3572,23 @@ async function runChecks(options, projectRoot) {
 		message: "No required environment variables for this backend"
 	});
 	const versionCheck = checkAutotelVersions(project.packageJson);
-	if (versionCheck.packages.length > 1) if (versionCheck.compatible) checks.push({
-		id: "version-compat",
-		title: "Version compatibility",
-		level: "warning",
-		status: "ok",
-		message: "All autotel packages are compatible"
-	});
-	else checks.push({
-		id: "version-compat",
-		title: "Version compatibility",
-		level: "warning",
-		status: "warn",
-		message: "Autotel packages have mismatched major versions",
-		details: versionCheck.packages.map((p) => `${p.name}@${p.version}`)
-	});
-	else checks.push({
+	if (versionCheck.packages.length > 1) {
+		if (versionCheck.compatible) checks.push({
+			id: "version-compat",
+			title: "Version compatibility",
+			level: "warning",
+			status: "ok",
+			message: "All autotel packages are compatible"
+		});
+		else checks.push({
+			id: "version-compat",
+			title: "Version compatibility",
+			level: "warning",
+			status: "warn",
+			message: "Autotel packages have mismatched major versions",
+			details: versionCheck.packages.map((p) => `${p.name}@${p.version}`)
+		});
+	} else checks.push({
 		id: "version-compat",
 		title: "Version compatibility",
 		level: "warning",
@@ -3600,41 +3605,42 @@ async function runChecks(options, projectRoot) {
 		details: esmCheck.details
 	});
 	const loggerCheck = checkLoggerInstrumentation(project.packageRoot, deps);
-	if (loggerCheck.hasLogger) if (loggerCheck.configuredInCode && !loggerCheck.hasInstrumentation) checks.push({
-		id: "logger-instrumentation",
-		title: "Logger instrumentation",
-		level: "warning",
-		status: "warn",
-		message: `${loggerCheck.logger} is configured but instrumentation package is missing`,
-		details: [`${loggerCheck.logger} is used in autoInstrumentations but ${loggerCheck.instrumentationPackage} is not installed`, `Install it: ${getInstallCommand(project.packageManager, [loggerCheck.instrumentationPackage])}`],
-		fix: {
-			cmd: getInstallCommand(project.packageManager, [loggerCheck.instrumentationPackage]),
-			description: `Install ${loggerCheck.instrumentationPackage}`
-		}
-	});
-	else if (loggerCheck.hasInstrumentation && loggerCheck.configuredInCode) checks.push({
-		id: "logger-instrumentation",
-		title: "Logger instrumentation",
-		level: "info",
-		status: "ok",
-		message: `${loggerCheck.logger} instrumentation is properly configured`
-	});
-	else if (loggerCheck.hasInstrumentation && !loggerCheck.configuredInCode) checks.push({
-		id: "logger-instrumentation",
-		title: "Logger instrumentation",
-		level: "info",
-		status: "ok",
-		message: `${loggerCheck.logger} instrumentation package is installed`,
-		details: [`Add '${loggerCheck.logger}' to autoInstrumentations in your init() call to enable trace context injection`]
-	});
-	else checks.push({
-		id: "logger-instrumentation",
-		title: "Logger instrumentation",
-		level: "info",
-		status: "skip",
-		message: `${loggerCheck.logger} is installed but not configured in code`
-	});
-	else checks.push({
+	if (loggerCheck.hasLogger) {
+		if (loggerCheck.configuredInCode && !loggerCheck.hasInstrumentation) checks.push({
+			id: "logger-instrumentation",
+			title: "Logger instrumentation",
+			level: "warning",
+			status: "warn",
+			message: `${loggerCheck.logger} is configured but instrumentation package is missing`,
+			details: [`${loggerCheck.logger} is used in autoInstrumentations but ${loggerCheck.instrumentationPackage} is not installed`, `Install it: ${getInstallCommand(project.packageManager, [loggerCheck.instrumentationPackage])}`],
+			fix: {
+				cmd: getInstallCommand(project.packageManager, [loggerCheck.instrumentationPackage]),
+				description: `Install ${loggerCheck.instrumentationPackage}`
+			}
+		});
+		else if (loggerCheck.hasInstrumentation && loggerCheck.configuredInCode) checks.push({
+			id: "logger-instrumentation",
+			title: "Logger instrumentation",
+			level: "info",
+			status: "ok",
+			message: `${loggerCheck.logger} instrumentation is properly configured`
+		});
+		else if (loggerCheck.hasInstrumentation && !loggerCheck.configuredInCode) checks.push({
+			id: "logger-instrumentation",
+			title: "Logger instrumentation",
+			level: "info",
+			status: "ok",
+			message: `${loggerCheck.logger} instrumentation package is installed`,
+			details: [`Add '${loggerCheck.logger}' to autoInstrumentations in your init() call to enable trace context injection`]
+		});
+		else checks.push({
+			id: "logger-instrumentation",
+			title: "Logger instrumentation",
+			level: "info",
+			status: "skip",
+			message: `${loggerCheck.logger} is installed but not configured in code`
+		});
+	} else checks.push({
 		id: "logger-instrumentation",
 		title: "Logger instrumentation",
 		level: "info",
@@ -5495,23 +5501,24 @@ function baselineError(source, reason) {
 }
 function readGitBaseline(cwd, ref) {
 	try {
+		const prefix = execFileSync("git", [
+			"-C",
+			cwd,
+			"rev-parse",
+			"--show-prefix"
+		], {
+			encoding: "utf8",
+			stdio: [
+				"ignore",
+				"pipe",
+				"ignore"
+			]
+		}).trim();
 		return execFileSync("git", [
 			"-C",
 			cwd,
 			"show",
-			`${ref}:${execFileSync("git", [
-				"-C",
-				cwd,
-				"rev-parse",
-				"--show-prefix"
-			], {
-				encoding: "utf8",
-				stdio: [
-					"ignore",
-					"pipe",
-					"ignore"
-				]
-			}).trim()}${MAP_FILE_NAME}`
+			`${ref}:${prefix}${MAP_FILE_NAME}`
 		], {
 			encoding: "utf8",
 			stdio: [
@@ -5519,7 +5526,7 @@ function readGitBaseline(cwd, ref) {
 				"pipe",
 				"ignore"
 			],
-			maxBuffer: 64 * 1024 * 1024
+			maxBuffer: 67108864
 		});
 	} catch {
 		return null;
@@ -6555,7 +6562,8 @@ async function resolveCodemodFiles(pathArg, cwd) {
 			return [];
 		}
 	}
-	return (await glob(path$1.isAbsolute(pathArg) ? pathArg : path$1.join(cwd, pathArg), {
+	const pattern = path$1.isAbsolute(pathArg) ? pathArg : path$1.join(cwd, pathArg);
+	return (await glob(pattern, {
 		cwd,
 		absolute: true,
 		ignore: ["**/node_modules/**", "**/*.d.ts"]
@@ -8266,12 +8274,13 @@ async function runDiagnoseAnomalies(flags) {
 	await runInvestigate("diagnose anomalies", flags, async (backend) => {
 		const lookback = flags.lookbackMinutes ?? 60;
 		const nowMs = Date.now();
-		return detectAnomalies((await backend.searchTraces({
+		const result = await backend.searchTraces({
 			service: flags.service,
 			startTimeUnixMs: nowMs - lookback * 60 * 1e3,
 			endTimeUnixMs: nowMs,
 			limit: 100
-		})).items, {
+		});
+		return detectAnomalies(result.items, {
 			service: flags.service,
 			operation: flags.operation
 		});
@@ -8414,12 +8423,13 @@ async function runExplainSlowdown(flags) {
 	await runInvestigate("correlate explain-slowdown", flags, async (backend) => {
 		const lookback = flags.lookbackMinutes ?? 60;
 		const nowMs = Date.now();
-		const anomalies = detectAnomalies((await backend.searchTraces({
+		const result = await backend.searchTraces({
 			service: flags.service,
 			startTimeUnixMs: nowMs - lookback * 60 * 1e3,
 			endTimeUnixMs: nowMs,
 			limit: 100
-		})).items, { service: flags.service });
+		});
+		const anomalies = detectAnomalies(result.items, { service: flags.service });
 		const findings = await Promise.all(anomalies.map(async (anomaly) => {
 			const sampleTraceId = anomaly.affectedTraceIds[0];
 			if (!sampleTraceId) return {
@@ -8486,7 +8496,8 @@ async function collectTracesForAnalytics(backend, input) {
 }
 async function runLlmUsage(flags) {
 	await runInvestigate("llm usage", flags, async (backend) => {
-		const report = collectUsage(await collectTracesForAnalytics(backend, flags));
+		const traces = await collectTracesForAnalytics(backend, flags);
+		const report = collectUsage(traces);
 		return {
 			period: {
 				startTime: flags.startTime ?? null,
@@ -8513,7 +8524,8 @@ async function runLlmUsage(flags) {
 }
 async function runLlmModels(flags) {
 	await runInvestigate("llm models", flags, async (backend) => {
-		const models = listModels(await collectTracesForAnalytics(backend, flags)).slice(0, flags.limit ?? 1e3);
+		const traces = await collectTracesForAnalytics(backend, flags);
+		const models = listModels(traces).slice(0, flags.limit ?? 1e3);
 		return {
 			count: models.length,
 			models
@@ -8522,14 +8534,16 @@ async function runLlmModels(flags) {
 }
 async function runLlmModelStats(flags) {
 	await runInvestigate("llm model-stats", flags, async (backend) => {
-		const stats = getModelStats(await collectTracesForAnalytics(backend, flags), flags.modelName);
+		const traces = await collectTracesForAnalytics(backend, flags);
+		const stats = getModelStats(traces, flags.modelName);
 		if (!stats) return { error: `No traces found for model '${flags.modelName}' in the specified time range` };
 		return stats;
 	});
 }
 async function runLlmExpensive(flags) {
 	await runInvestigate("llm expensive", flags, async (backend) => {
-		let ranked = rankExpensiveTraces(await collectTracesForAnalytics(backend, flags));
+		const traces = await collectTracesForAnalytics(backend, flags);
+		let ranked = rankExpensiveTraces(traces);
 		if (flags.minTokens !== void 0) ranked = ranked.filter((t) => t.tokens.total >= flags.minTokens);
 		ranked = ranked.slice(0, flags.limit ?? 10);
 		return {
@@ -8540,7 +8554,8 @@ async function runLlmExpensive(flags) {
 }
 async function runLlmSlow(flags) {
 	await runInvestigate("llm slow", flags, async (backend) => {
-		let ranked = rankSlowTraces(await collectTracesForAnalytics(backend, flags));
+		const traces = await collectTracesForAnalytics(backend, flags);
+		let ranked = rankSlowTraces(traces);
 		if (flags.minDurationMs !== void 0) ranked = ranked.filter((t) => t.durationMs >= flags.minDurationMs);
 		ranked = ranked.slice(0, flags.limit ?? 10);
 		return {
@@ -8551,7 +8566,8 @@ async function runLlmSlow(flags) {
 }
 async function runLlmTools(flags) {
 	await runInvestigate("llm tools", flags, async (backend) => {
-		const tools = listToolUsage(await collectTracesForAnalytics(backend, flags)).slice(0, flags.limit ?? 1e3);
+		const traces = await collectTracesForAnalytics(backend, flags);
+		const tools = listToolUsage(traces).slice(0, flags.limit ?? 1e3);
 		return {
 			count: tools.length,
 			totalCalls: tools.reduce((sum, t) => sum + t.usageCount, 0),
@@ -8741,7 +8757,8 @@ function assertVersion(version) {
 }
 async function runCollectorValidate(flags) {
 	await runStatic("collector validate", flags, async () => {
-		return validateOtlpReceiverConfig(readJsonFromStdinOrFile(flags.configFile));
+		const config = readJsonFromStdinOrFile(flags.configFile);
+		return validateOtlpReceiverConfig(config);
 	});
 }
 async function runCollectorSuggest(flags) {
@@ -9743,7 +9760,10 @@ async function run() {
 	const parseWithTelemetry = withCommanderTelemetry({
 		name: () => program.name(),
 		version: () => program.version() ?? "",
-		parseAsync: (argv) => program.parseAsync(argv)
+		async parseAsync(argv) {
+			await program.parseAsync(argv);
+			return this;
+		}
 	}, {
 		name: "autotel",
 		version: program.version()

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "autotel-cli",
-  "version": "0.18.5",
+  "version": "0.18.6",
   "description": "CLI for autotel - setup wizard, diagnostics, and incremental features",
   "type": "module",
   "bin": {
@@ -18,31 +18,25 @@
     "NOTICE"
   ],
   "dependencies": {
-    "@inquirer/prompts": "^8.5.2",
+    "@inquirer/prompts": "^8.7.2",
     "@opentelemetry/otlp-transformer": "^0.222.0",
-    "autotel-mcp": "0.9.1",
-    "chalk": "^5.6.2",
+    "autotel-mcp": "0.10.0",
+    "chalk": "^6.0.1",
     "commander": "^15.0.0",
     "glob": "^13.0.6",
     "ora": "^9.4.1",
     "semver": "^7.8.5",
     "ts-morph": "^28.0.0",
-    "yaml": "^2.9.0"
+    "yaml": "^2.9.1"
   },
   "devDependencies": {
-    "@eslint/js": "^10.0.1",
-    "@types/eslint-config-prettier": "^6.11.3",
-    "@types/node": "^26.1.2",
-    "@types/semver": "^7.7.1",
+    "@types/node": "^26.6.3",
+    "@types/semver": "^7.8.0",
     "autotel-telemetry": "0.2.0",
-    "eslint": "^10.7.0",
-    "eslint-config-prettier": "^10.1.8",
-    "eslint-plugin-unicorn": "^64.0.0",
-    "tsdown": "^0.22.14",
-    "tsx": "^4.23.1",
-    "typescript": "^6.0.3",
-    "typescript-eslint": "^8.65.0",
-    "vitest": "^4.1.10"
+    "tsdown": "^0.23.0",
+    "tsx": "^4.23.15",
+    "typescript": "^7.0.2",
+    "vitest": "^5.0.2"
   },
   "engines": {
     "node": ">=22.0.0"
@@ -69,7 +63,7 @@
     "dev": "tsdown --watch",
     "test": "vitest run --config vitest.unit.config.ts",
     "test:watch": "vitest",
-    "lint": "eslint src",
-    "typecheck": "tsc --noEmit"
+    "lint": "oxlint -c oxlint.json --disable-nested-config src",
+    "type-check": "tsc --noEmit"
   }
 }