autotel-cli 0.12.0 → 0.13.1

package/README.md

@@ -1,6 +1,6 @@
 # autotel-cli
 
-CLI for autotel — setup wizard, diagnostics, and incremental features.
+CLI for autotel. Setup wizard, diagnostics, and incremental features.
 
 Designed for **both humans and AI agents**: every command supports `--json`,
 errors are returned as a structured envelope with stable `AUTOTEL_E_*` codes,
@@ -46,34 +46,35 @@ individually.
 
 **Options:**
 
-| Flag | What it does |
-| --- | --- |
-| `--yes, -y` | Auto-apply detected items; no prompts |
-| `--preset <name>` | Use a named quick preset (skips detection) |
-| `--dry-run` | Print plan; write nothing |
-| `--no-install` | Generate files only; print install command |
-| `--force` | Overwrite a hand-edited instrumentation file (backup written) |
-| `--no-detect` | Skip auto-detection; requires `--plan`, `--input`, or `--preset` |
-| `--detect-only` | Print the detected plan and exit |
-| `--plan <path>` | Apply a pre-built `InitPlan` JSON file |
-| `--input -` / `--input <path>` | Read `InitPlan` JSON from stdin or a file |
-| `--scan-env` | Consent to reading `.env` / `.env.local` for backend detection |
-| `--json` | Machine-readable JSON instead of human output |
-| `--output-file <path>` | Persist JSON output to a file |
-| `--no-secrets-in-output` | Redact secret-shaped values (`*KEY*`, `*TOKEN*`, etc.) |
-| `--no-interactive` | Fail fast instead of prompting |
+| Flag                           | What it does                                                     |
+| ------------------------------ | ---------------------------------------------------------------- |
+| `--yes, -y`                    | Auto-apply detected items; no prompts                            |
+| `--preset <name>`              | Use a named quick preset (skips detection)                       |
+| `--dry-run`                    | Print plan; write nothing                                        |
+| `--no-install`                 | Generate files only; print install command                       |
+| `--force`                      | Overwrite a hand-edited instrumentation file (backup written)    |
+| `--no-detect`                  | Skip auto-detection; requires `--plan`, `--input`, or `--preset` |
+| `--detect-only`                | Print the detected plan and exit                                 |
+| `--plan <path>`                | Apply a pre-built `InitPlan` JSON file                           |
+| `--input -` / `--input <path>` | Read `InitPlan` JSON from stdin or a file                        |
+| `--scan-env`                   | Consent to reading `.env` / `.env.local` for backend detection   |
+| `--json`                       | Machine-readable JSON instead of human output                    |
+| `--output-file <path>`         | Persist JSON output to a file                                    |
+| `--no-secrets-in-output`       | Redact secret-shaped values (`*KEY*`, `*TOKEN*`, etc.)           |
+| `--no-interactive`             | Fail fast instead of prompting                                   |
 
 **Detection coverage:**
 
 - **Frameworks (auto-wired):** Hono, MCP servers/clients, TanStack Start
 - **Frameworks (via `auto-instrumentations-node`):** Express, Fastify, NestJS, Next.js, pg, mysql/2, redis, ioredis, GraphQL, AWS SDK
-- **Loggers:** Pino (first-class — `init({ logger })`), Winston/Bunyan (auto-instrumented)
+- **Loggers:** Pino (first-class: `init({ logger })`), Winston/Bunyan (auto-instrumented)
 - **Subscribers:** PostHog, Mixpanel, Amplitude, Segment, Slack
 - **Plugins:** Mongoose, Drizzle, Sentry
 - **Platforms:** Cloudflare (from `wrangler.toml`), AWS Lambda, Edge
 - **Backends from env vars:** `DD_API_KEY` → Datadog, `HONEYCOMB_API_KEY` → Honeycomb, `OTEL_EXPORTER_OTLP_ENDPOINT` → OTLP
 
 **Quick presets:**
+
 - `node-datadog-pino` - Node.js + Datadog + Pino logging
 - `node-datadog-agent` - Node.js + Datadog Agent (local development)
 - `node-honeycomb` - Node.js + Honeycomb
@@ -128,12 +129,14 @@ npx autotel doctor --list-checks
 ```
 
 **Options:**
+
 - `--json` - Output machine-readable JSON
 - `--fix` - Auto-fix resolvable issues
 - `--list-checks` - List all available checks
 - `--env-file <path>` - Specify env file to check
 
 **Exit codes:**
+
 - `0` - All checks passed
 - `1` - Warnings found
 - `2` - Errors found
@@ -163,12 +166,14 @@ npx autotel add backend datadog --help
 ```
 
 **Types:**
+
 - `backend` - Telemetry backends (Datadog, Honeycomb, OTLP, etc.)
 - `subscriber` - Event subscribers (PostHog, Mixpanel, Segment, Slack, etc.)
 - `plugin` - Database/ORM plugins (Mongoose, Drizzle, etc.)
 - `platform` - Platform support (AWS Lambda, Cloudflare Workers, etc.)
 
 **Options:**
+
 - `--list` - List available presets
 - `--dry-run` - Skip installation and print what would be done
 - `--force` - Overwrite non-CLI-owned config (creates backup first)

package/dist/index.d.ts

@@ -1,2 +1,2 @@
 
-export { };
+export {}

package/dist/index.js

@@ -3942,8 +3942,14 @@ function transformFile(content, filePath, options) {
 			skip(spanName, "already wrapped");
 			continue;
 		}
-		const mod = fn.getModifiers().map((m) => m.getText()).join(" ");
-		const newText = `${mod ? mod + " " : ""}const ${name} = trace('${spanName}', function ${name}${fn.getText().replace(/^function\s*\w*\s*/, "")};`;
+		const modifiers = fn.getModifiers().map((m) => m.getText());
+		const isAsync = modifiers.includes("async");
+		const declModifiers = modifiers.filter((m) => m !== "async");
+		const modPrefix = declModifiers.length > 0 ? declModifiers.join(" ") + " " : "";
+		const fnText = fn.getText();
+		const rest = fnText.replace(/^(?:\b(?:export|default|async|declare)\b\s*)*function\s*(\*?)\s*\w*\s*/, "");
+		const star = /^(?:\b(?:export|default|async|declare)\b\s*)*function\s*\*/.test(fnText) ? "*" : "";
+		const newText = `${modPrefix}const ${name} = trace('${spanName}', ${isAsync ? "async " : ""}function${star} ${name}${rest});`;
 		edits.push({
 			node: fn,
 			newText
@@ -3958,7 +3964,7 @@ function transformFile(content, filePath, options) {
 			if (shouldSkipName(spanName, options)) skip(spanName, "name match");
 			else {
 				const fn = defaultFn;
-				const full = `const ${name} = trace('${spanName}', function ${name}(${(fn.getParameters?.() ?? []).map((p) => p.getText()).join(", ")}) ${(fn.getBody?.())?.getText() ?? "{}"};
+				const full = `const ${name} = trace('${spanName}', function ${name}(${(fn.getParameters?.() ?? []).map((p) => p.getText()).join(", ")}) ${(fn.getBody?.())?.getText() ?? "{}"});
 export default ` + name + ";";
 				edits.push({
 					node: defaultFn,
@@ -3975,7 +3981,7 @@ export default ` + name + ";";
 		const isArrow = kind === SyntaxKind.ArrowFunction;
 		const isFnExpr = kind === SyntaxKind.FunctionExpression;
 		if (kind === SyntaxKind.CallExpression) {
-			if (init.getExpression().getText() === "trace") {
+			if (init.asKindOrThrow(SyntaxKind.CallExpression).getExpression().getText() === "trace") {
 				const name = decl.getName();
 				if (typeof name === "string") {
 					const spanName = getSpanName(name, filePath, options);
@@ -4050,7 +4056,7 @@ export default ` + name + ";";
 	for (const stmt of sourceFile.getVariableStatements()) for (const decl of stmt.getDeclarationList().getDeclarations()) {
 		const init = decl.getInitializer();
 		if (!init || init.getKind() !== SyntaxKind.ObjectLiteralExpression) continue;
-		const obj = init;
+		const obj = init.asKindOrThrow(SyntaxKind.ObjectLiteralExpression);
 		for (const prop of obj.getProperties()) {
 			if (prop.getKind() !== SyntaxKind.MethodDeclaration) continue;
 			const method = prop;
@@ -4764,30 +4770,60 @@ COMMANDS.push(...[
 		name: "telemetry",
 		description: "Manage opt-in CLI usage telemetry (parent)",
 		mutating: false,
+		network: false,
+		writesFiles: false,
+		supportsDryRun: false,
+		requiresPackageJson: false,
+		mayReadEnv: false,
+		supportsJson: false,
 		flags: []
 	},
 	{
 		name: "telemetry status",
 		description: "Show telemetry consent status",
 		mutating: false,
+		network: false,
+		writesFiles: false,
+		supportsDryRun: false,
+		requiresPackageJson: false,
+		mayReadEnv: false,
+		supportsJson: true,
 		flags: []
 	},
 	{
 		name: "telemetry enable",
 		description: "Enable telemetry for this tool",
 		mutating: true,
+		network: false,
+		writesFiles: true,
+		supportsDryRun: false,
+		requiresPackageJson: false,
+		mayReadEnv: false,
+		supportsJson: false,
 		flags: []
 	},
 	{
 		name: "telemetry disable",
 		description: "Disable telemetry and purge undelivered events",
 		mutating: true,
+		network: false,
+		writesFiles: true,
+		supportsDryRun: false,
+		requiresPackageJson: false,
+		mayReadEnv: false,
+		supportsJson: false,
 		flags: []
 	},
 	{
 		name: "telemetry disclosure",
 		description: "Print telemetry disclosure markdown",
 		mutating: false,
+		network: false,
+		writesFiles: false,
+		supportsDryRun: false,
+		requiresPackageJson: false,
+		mayReadEnv: false,
+		supportsJson: false,
 		flags: []
 	}
 ]);
@@ -6407,6 +6443,11 @@ function createDebugDrain() {
 		if (process.env.AUTOTEL_TELEMETRY_DEBUG === "1") for (const event of events) console.error("[autotel-telemetry]", JSON.stringify(event));
 	};
 }
+const RETRYABLE_STATUS_CODES = /* @__PURE__ */ new Set([
+	408,
+	425,
+	429
+]);
 function createHttpDrain(endpoint) {
 	return async (events) => {
 		if (events.length === 0) return;
@@ -6415,7 +6456,9 @@ function createHttpDrain(endpoint) {
 			headers: { "content-type": "application/json" },
 			body: JSON.stringify({ events })
 		});
-		if (!response.ok) throw new Error(`Telemetry delivery failed with HTTP ${response.status}`);
+		if (response.ok) return;
+		if (response.status >= 400 && response.status < 500 && !RETRYABLE_STATUS_CODES.has(response.status)) return;
+		throw new Error(`Telemetry delivery failed with HTTP ${response.status}`);
 	};
 }
 function composeDrains(...drains) {
@@ -6794,10 +6837,14 @@ function createProgram() {
 * which the top-level handler in `index.ts` converts to an envelope.
 */
 async function run() {
-	const base = createProgram();
-	const program = withCommanderTelemetry(base, {
+	const program = createProgram();
+	const parseWithTelemetry = withCommanderTelemetry({
+		name: () => program.name(),
+		version: () => program.version() ?? "",
+		parseAsync: (argv) => program.parseAsync(argv)
+	}, {
 		name: "autotel",
-		version: base.version()
+		version: program.version()
 	});
 	program.exitOverride();
 	const argvJoined = process.argv.slice(2).join(" ");
@@ -6810,7 +6857,7 @@ async function run() {
 		if (isJsonOnly) cmd.configureOutput({ writeErr: () => {} });
 		stack.push(...cmd.commands);
 	}
-	await program.parseAsync(process.argv);
+	await parseWithTelemetry.parseAsync(process.argv);
 }
 //#endregion
 //#region src/lib/commander-error.ts