newpr 1.0.2 → 1.0.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "newpr",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "AI-powered large PR review tool - understand PRs with 1000+ lines of changes",
   "module": "src/cli/index.ts",
   "type": "module",

package/src/telemetry/index.ts

@@ -0,0 +1,115 @@
+import { randomUUID } from "node:crypto";
+import { homedir } from "node:os";
+import { join } from "node:path";
+import { mkdirSync } from "node:fs";
+
+const GA_ID = "G-L3SL6T6JQ1";
+const GA_SECRET = "Sier1nbXS2-eX2TR3j1kZQ";
+const MP_ENDPOINT = `https://www.google-analytics.com/mp/collect?measurement_id=${GA_ID}&api_secret=${GA_SECRET}`;
+
+const CONFIG_DIR = join(homedir(), ".newpr");
+const TELEMETRY_FILE = join(CONFIG_DIR, "telemetry.json");
+
+interface TelemetryConfig {
+	client_id: string;
+	consent: "granted" | "denied" | "pending";
+}
+
+let config: TelemetryConfig | null = null;
+
+function ensureDir(): void {
+	mkdirSync(CONFIG_DIR, { recursive: true });
+}
+
+async function loadConfig(): Promise<TelemetryConfig> {
+	if (config) return config;
+	try {
+		const file = Bun.file(TELEMETRY_FILE);
+		if (await file.exists()) {
+			const data = JSON.parse(await file.text()) as Partial<TelemetryConfig>;
+			config = {
+				client_id: data.client_id || randomUUID(),
+				consent: data.consent === "granted" || data.consent === "denied" ? data.consent : "pending",
+			};
+		} else {
+			config = { client_id: randomUUID(), consent: "pending" };
+		}
+	} catch {
+		config = { client_id: randomUUID(), consent: "pending" };
+	}
+	await saveConfig();
+	return config;
+}
+
+async function saveConfig(): Promise<void> {
+	if (!config) return;
+	ensureDir();
+	await Bun.write(TELEMETRY_FILE, `${JSON.stringify(config, null, 2)}\n`);
+}
+
+export async function getTelemetryConsent(): Promise<"granted" | "denied" | "pending"> {
+	const cfg = await loadConfig();
+	return cfg.consent;
+}
+
+export async function setTelemetryConsent(consent: "granted" | "denied"): Promise<void> {
+	const cfg = await loadConfig();
+	cfg.consent = consent;
+	await saveConfig();
+}
+
+async function sendEvents(events: Array<{ name: string; params?: Record<string, string | number | boolean> }>): Promise<void> {
+	const cfg = await loadConfig();
+	if (cfg.consent !== "granted") return;
+
+	const body = {
+		client_id: cfg.client_id,
+		events,
+	};
+
+	try {
+		await fetch(MP_ENDPOINT, {
+			method: "POST",
+			headers: { "Content-Type": "application/json" },
+			body: JSON.stringify(body),
+		});
+	} catch {
+		// Fire-and-forget — network failures are silent
+	}
+}
+
+function track(name: string, params?: Record<string, string | number | boolean>): void {
+	sendEvents([{ name, params }]);
+}
+
+export const telemetry = {
+	serverStarted: (version: string) =>
+		track("server_started", { version, platform: process.platform, arch: process.arch }),
+
+	analysisStarted: (fileCount: number) =>
+		track("analysis_started", { file_count: fileCount }),
+
+	analysisCompleted: (fileCount: number, durationSec: number) =>
+		track("analysis_completed", { file_count: fileCount, duration_sec: durationSec }),
+
+	analysisError: (errorType: string) =>
+		track("analysis_error", { error_type: errorType.slice(0, 100) }),
+
+	chatSent: () =>
+		track("chat_sent"),
+
+	chatCompleted: (durationSec: number, hasTools: boolean) =>
+		track("chat_completed", { duration_sec: durationSec, has_tools: hasTools }),
+
+	reviewSubmitted: (event: string) =>
+		track("review_submitted", { review_event: event }),
+
+	stackStarted: () =>
+		track("stack_started"),
+
+	stackCompleted: (groupCount: number) =>
+		track("stack_completed", { group_count: groupCount }),
+
+	stackPublished: (prCount: number) =>
+		track("stack_published", { pr_count: prCount }),
+};

package/src/web/client/components/AnalyticsConsent.tsx

@@ -7,13 +7,23 @@ export function AnalyticsConsent({ onDone }: { onDone: () => void }) {
 
 	if (state !== "pending") return null;
 
+	const syncServer = (consent: "granted" | "denied") => {
+		fetch("/api/config", {
+			method: "PUT",
+			headers: { "Content-Type": "application/json" },
+			body: JSON.stringify({ telemetry_consent: consent }),
+		}).catch(() => {});
+	};
+
 	const handleAccept = () => {
 		setConsent("granted");
+		syncServer("granted");
 		onDone();
 	};
 
 	const handleDecline = () => {
 		setConsent("denied");
+		syncServer("denied");
 		onDone();
 	};
 

package/src/web/client/components/SettingsPanel.tsx

@@ -285,6 +285,11 @@ function AnalyticsToggle() {
 		setConsent(next);
 		setLocal(next);
 		analytics.settingsChanged("analytics");
+		fetch("/api/config", {
+			method: "PUT",
+			headers: { "Content-Type": "application/json" },
+			body: JSON.stringify({ telemetry_consent: next }),
+		}).catch(() => {});
 	};
 
 	return (

package/src/web/server/routes.ts

@@ -17,6 +17,7 @@ import { detectAgents, runAgent } from "../../workspace/agent.ts";
 import { randomBytes } from "node:crypto";
 import { publishStack } from "../../stack/publish.ts";
 import { startStack, getStackState, cancelStack, subscribeStack, restoreCompletedStacks } from "./stack-manager.ts";
+import { getTelemetryConsent, setTelemetryConsent, telemetry } from "../../telemetry/index.ts";
 
 function json(data: unknown, status = 200): Response {
 	return new Response(JSON.stringify(data), {
@@ -610,6 +611,7 @@ Before posting an inline comment, ALWAYS call \`get_file_diff\` first to find th
 			const pluginList = getAllPlugins().map((p) => ({ id: p.id, name: p.name }));
 			const enabledPlugins = stored.enabled_plugins ?? pluginList.map((p) => p.id);
 			const agents = await detectAgents();
+			const telemetryConsent = await getTelemetryConsent();
 			return json({
 				model: config.model,
 				agent: config.agent ?? null,
@@ -622,6 +624,7 @@ Before posting an inline comment, ALWAYS call \`get_file_diff\` first to find th
 				has_github_token: !!token,
 				enabled_plugins: enabledPlugins,
 				available_plugins: pluginList,
+				telemetry_consent: telemetryConsent,
 				defaults: {
 					model: DEFAULT_CONFIG.model,
 					language: DEFAULT_CONFIG.language,
@@ -673,6 +676,11 @@ Before posting an inline comment, ALWAYS call \`get_file_diff\` first to find th
 				update.enabled_plugins = (body as Record<string, unknown>).enabled_plugins as string[];
 			}
 
+			const telemetryConsentVal = (body as Record<string, unknown>).telemetry_consent as string | undefined;
+			if (telemetryConsentVal === "granted" || telemetryConsentVal === "denied") {
+				await setTelemetryConsent(telemetryConsentVal);
+			}
+
 			await writeStoredConfig(update);
 			return json({ ok: true });
 		},
@@ -748,6 +756,7 @@ Before posting an inline comment, ALWAYS call \`get_file_diff\` first to find th
 					return json({ error: errBody.message ?? `GitHub API error: ${res.status}` }, res.status);
 				}
 				const data = await res.json() as { id: number; state: string; html_url: string };
+				telemetry.reviewSubmitted(body.event);
 				return json({ ok: true, id: data.id, state: data.state, html_url: data.html_url });
 			} catch (err) {
 				return json({ error: err instanceof Error ? err.message : String(err) }, 500);

package/src/web/server/session-manager.ts

@@ -4,6 +4,7 @@ import type { ProgressEvent } from "../../analyzer/progress.ts";
 import { analyzePr } from "../../analyzer/pipeline.ts";
 import { parsePrInput } from "../../github/parse-pr.ts";
 import { saveSession, savePatchesSidecar } from "../../history/store.ts";
+import { telemetry } from "../../telemetry/index.ts";
 
 type SessionStatus = "running" | "done" | "error" | "canceled";
 
@@ -65,6 +66,7 @@ export function startAnalysis(
 	};
 	sessions.set(id, session);
 
+	telemetry.analysisStarted(0);
 	runPipeline(session, prInput, token, config);
 
 	return { sessionId: id };
@@ -100,6 +102,9 @@ async function runPipeline(
 		session.result = result;
 		session.finishedAt = Date.now();
 
+		const durationSec = Math.round((session.finishedAt - session.startedAt) / 1000);
+		telemetry.analysisCompleted(result.files?.length ?? 0, durationSec);
+
 		for (const sub of session.subscribers) {
 			sub({ type: "done" });
 		}
@@ -118,6 +123,8 @@ async function runPipeline(
 		session.error = msg;
 		session.finishedAt = Date.now();
 
+		telemetry.analysisError(msg);
+
 		for (const sub of session.subscribers) {
 			sub({ type: "error", data: msg });
 		}

package/src/web/server.ts

@@ -5,6 +5,7 @@ import index from "./index.html";
 
 import type { PreflightResult } from "../cli/preflight.ts";
 import { getVersion } from "../version.ts";
+import { telemetry } from "../telemetry/index.ts";
 
 interface WebServerOptions {
 	port: number;
@@ -211,6 +212,8 @@ export async function startWebServer(options: WebServerOptions): Promise<void> {
 
 	const url = `http://localhost:${server.port}`;
 
+	telemetry.serverStarted(getVersion());
+
 	const dim = (s: string) => `\x1b[2m${s}\x1b[0m`;
 	const bold = (s: string) => `\x1b[1m${s}\x1b[0m`;
 	const cyan = (s: string) => `\x1b[36m${s}\x1b[0m`;