@weareikko/code-review 0.8.6 → 0.9.1

package/dist/{cli-BbR1oftT.js → cli-BGn8g23f.js}

@@ -560,7 +560,7 @@ function buildSummaryBody(summary, costFooter, options = {}) {
 	return `${withFooter}\n\n${buildSummaryHistoryBlock(historyEntries)}`;
 }
 function buildReviewedCommitFooter(commitSha) {
-	return `Reviewed by ${PRODUCT_LINK} v0.8.6 for commit ${commitSha}.`;
+	return `Reviewed by ${PRODUCT_LINK} v0.9.1 for commit ${commitSha}.`;
 }
 function extractReviewedCommitSha(body) {
 	return REVIEWED_COMMIT_FOOTER_PATTERN.exec(body)?.[1] ?? null;
@@ -1359,6 +1359,7 @@ function createDiagnosticContext(phase, config, runId, overrides = {}) {
 		project: config.project,
 		mr: config.mr,
 		gitlabUrl: config.gitlabUrl,
+		platform: config.platform,
 		cwd: config.cwd,
 		model: config.model,
 		minSeverity: config.minSeverity,
@@ -4354,54 +4355,59 @@ async function runReview(config, options) {
 		attachTelemetry: options.attachTelemetry,
 		verifyStaged: diskMode ? retrievableSkipped : void 0
 	};
+	const buildUsage = () => ({
+		model: config.model,
+		thinkingLevel: config.thinkingLevel,
+		tokens: aggregated.tokens,
+		cost: aggregated.cost,
+		byModel: buildByModelUsage(aggregated),
+		skills: context.skills.map((s) => s.name),
+		sizeNotice
+	});
 	let outputText;
-	if (config.reviewDepth === "full") {
-		const { findings, summary } = await runMultiAngleFind(context, minSeverity, userPrompt, deps);
-		outputText = await verifyAndSynthesize(findings, summary, diff, options.commitLog, deps);
-	} else {
-		const findAgent = createAgent({
-			systemPrompt,
-			model: primary.model,
-			tools,
-			thinkingLevel: config.thinkingLevel,
-			getApiKey: primary.getApiKey
-		});
-		const detachTelemetry = options.attachTelemetry?.(findAgent);
-		let turnCount = 0;
-		let toolCallCount = 0;
-		let finalText;
-		try {
-			finalText = await runAgentToCompletion(findAgent, userPrompt, {
-				timeoutMs,
-				onAssistantMessage: (message) => accumulateUsage(aggregated, message, primary.id),
-				onTurnStart: (turn) => {
-					turnCount = turn;
-					logger.debug(`Turn ${turn} started`);
-				},
-				onToolStart: (toolName, args) => {
-					toolCallCount += 1;
-					logger.debug(`  → ${toolName}${formatToolArgs(toolName, args)}`);
-				}
+	try {
+		if (config.reviewDepth === "full") {
+			const { findings, summary } = await runMultiAngleFind(context, minSeverity, userPrompt, deps);
+			outputText = await verifyAndSynthesize(findings, summary, diff, options.commitLog, deps);
+		} else {
+			const findAgent = createAgent({
+				systemPrompt,
+				model: primary.model,
+				tools,
+				thinkingLevel: config.thinkingLevel,
+				getApiKey: primary.getApiKey
 			});
-		} finally {
-			detachTelemetry?.();
+			const detachTelemetry = options.attachTelemetry?.(findAgent);
+			let turnCount = 0;
+			let toolCallCount = 0;
+			let finalText;
+			try {
+				finalText = await runAgentToCompletion(findAgent, userPrompt, {
+					timeoutMs,
+					onAssistantMessage: (message) => accumulateUsage(aggregated, message, primary.id),
+					onTurnStart: (turn) => {
+						turnCount = turn;
+						logger.debug(`Turn ${turn} started`);
+					},
+					onToolStart: (toolName, args) => {
+						toolCallCount += 1;
+						logger.debug(`  → ${toolName}${formatToolArgs(toolName, args)}`);
+					}
+				});
+			} finally {
+				detachTelemetry?.();
+			}
+			logger.debug(`Agent finished: ${turnCount} turn(s), ${toolCallCount} tool call(s)`);
+			outputText = config.reviewDepth === "verify" ? await runVerifyStage(finalText, diff, options.commitLog, deps) : finalText;
 		}
-		logger.debug(`Agent finished: ${turnCount} turn(s), ${toolCallCount} tool call(s)`);
-		outputText = config.reviewDepth === "verify" ? await runVerifyStage(finalText, diff, options.commitLog, deps) : finalText;
+	} finally {
+		options.onUsage?.(buildUsage());
 	}
 	const reviewPath = resolve(cwd, config.reviewFile);
 	await mkdir(dirname(reviewPath), { recursive: true });
 	await writeFile(reviewPath, outputText, "utf8");
 	if (retrievableSkipped.length > 0) await cleanupSkippedDiffs(cwd);
-	return {
-		model: config.model,
-		thinkingLevel: config.thinkingLevel,
-		tokens: aggregated.tokens,
-		cost: aggregated.cost,
-		byModel: buildByModelUsage(aggregated),
-		skills: context.skills.map((s) => s.name),
-		sizeNotice
-	};
+	return buildUsage();
 }
 /**
 * Convert the per-model usage buckets into the public {@link ModelUsage} array,
@@ -4758,13 +4764,13 @@ async function startOtelBridge(options = {}) {
 		unit: "{token}",
 		advice: { explicitBucketBoundaries: TOKEN_BUCKETS }
 	});
-	const operationCost = meter.createHistogram("gen_ai.client.cost", {
-		description: "GenAI operation cost in USD",
+	const operationCost = meter.createHistogram("code_review_llm_cost_usd", {
+		description: "LLM cost in USD per turn, by token type (non-standard extension)",
 		unit: "{usd}",
 		advice: { explicitBucketBoundaries: COST_BUCKETS_USD }
 	});
-	const timeToFirstToken = meter.createHistogram("gen_ai.client.time_to_first_token", {
-		description: "Time to first token from the LLM",
+	const timeToFirstToken = meter.createHistogram("gen_ai.client.operation.time_to_first_chunk", {
+		description: "Time to first chunk from the LLM",
 		unit: "s",
 		advice: { explicitBucketBoundaries: TTFT_BUCKETS_S }
 	});
@@ -4804,6 +4810,9 @@ async function startOtelBridge(options = {}) {
 				ciAttrs,
 				ciSpanAttrs,
 				model: ctx.model,
+				dryRun: ctx.dryRun,
+				platform: ctx.platform,
+				serverAddress: hostOf(ctx.gitlabUrl),
 				rootSpanCtx
 			});
 			emitReviewStartedLog(logger, ctx, ciAttrs, ciSpanAttrs, rootSpanCtx);
@@ -4835,50 +4844,55 @@ async function startOtelBridge(options = {}) {
 		entry.span.end();
 		entry.closed = true;
 		const status = resolveRunStatus(ctx, isError);
-		const projectPath = ciAttrs["gitlab.project_path"] ?? "";
+		const projectPath = ciAttrs["vcs.repository.name"] ?? "";
+		const vcs = vcsAttrs(ctx.platform, ctx.gitlabUrl);
 		if (typeof ctx.durationMs === "number") reviewPhaseDuration.record(ctx.durationMs / 1e3, {
 			...REVIEW_SERVICE_ATTRS,
-			"gitlab.project_path": projectPath,
-			"gitlab_review.phase": ctx.phase,
-			"gitlab_review.status": status
+			...vcs,
+			"vcs.repository.name": projectPath,
+			"code_review.phase": ctx.phase,
+			"code_review.status": status
 		});
 		if (ctx.phase === ROOT_PHASE) {
 			const meta = runMeta.get(ctx.runId);
-			const pipelineSource = ciAttrs["gitlab.pipeline_source"] ?? "";
+			const pipelineSource = ciAttrs["cicd.pipeline.source"] ?? "";
 			const runMetricBase = {
 				...REVIEW_SERVICE_ATTRS,
-				"gitlab.project_path": projectPath,
-				"gitlab_review.dry_run": ctx.dryRun
+				...vcs,
+				"vcs.repository.name": projectPath,
+				"code_review.dry_run": ctx.dryRun,
+				...ctx.firstReview !== void 0 ? { "code_review.first_review": ctx.firstReview } : {}
 			};
 			const usage = meta?.usage ?? ctx.usage;
 			const runModelAttrs = genAiModelAttrs(void 0, splitModel(usage?.model ?? "").modelId);
 			reviewRunsTotal.add(1, {
 				...runMetricBase,
-				"gitlab.pipeline_source": pipelineSource,
-				"gitlab_review.status": status
+				"cicd.pipeline.source": pipelineSource,
+				"code_review.status": status
 			});
 			if (isError) reviewErrorsTotal.add(1, {
 				...runMetricBase,
-				"gitlab_review.status": status,
+				"code_review.status": status,
 				"error.type": errorTypeOf(ctx)
 			});
 			if (typeof ctx.durationMs === "number") reviewRunDuration.record(ctx.durationMs / 1e3, {
 				...runMetricBase,
 				...runModelAttrs,
-				"gitlab.pipeline_source": pipelineSource,
-				"gitlab_review.status": status
+				"cicd.pipeline.source": pipelineSource,
+				"code_review.status": status
 			});
 			const totalCostUsd = usage?.cost.total;
 			if (totalCostUsd !== void 0) reviewTotalCost.record(totalCostUsd, {
 				...runMetricBase,
 				...runModelAttrs,
-				"gitlab_review.status": status
+				"code_review.status": status
 			});
 			if (usage) {
 				const tokenAttrs = {
 					...REVIEW_SERVICE_ATTRS,
 					...runModelAttrs,
-					"gitlab.project_path": projectPath
+					...vcs,
+					"vcs.repository.name": projectPath
 				};
 				for (const [field, type] of [
 					["input", "input"],
@@ -4894,7 +4908,7 @@ async function startOtelBridge(options = {}) {
 			if (bySeverity) {
 				for (const [severity, count] of Object.entries(bySeverity)) if (count && count > 0) reviewCommentsTotal.add(count, {
 					...runMetricBase,
-					"gitlab_review.comment.severity": severity
+					"code_review.comment.severity": severity
 				});
 			} else {
 				const posted = ctx.posted ?? 0;
@@ -4940,16 +4954,17 @@ async function startOtelBridge(options = {}) {
 					context: meta?.rootSpanCtx,
 					attributes: {
 						"service.name": SERVICE_NAME,
-						"event.name": "gitlab_review.comment",
-						"gitlab_review.run_id": runId,
-						"gitlab_review.comment.file": comment.file,
-						"gitlab_review.comment.line": comment.line,
-						"gitlab_review.comment.severity": comment.severity,
-						"gitlab_review.comment.is_duplicate": duplicate,
+						"event.name": "code_review.comment",
+						"code_review.run_id": runId,
+						"code_review.comment.file": comment.file,
+						"code_review.comment.line": comment.line,
+						"code_review.comment.severity": comment.severity,
+						"code_review.comment.is_duplicate": duplicate,
 						...meta && {
-							"gitlab.project_id": meta.project,
-							"gitlab.mr_iid": meta.mr,
-							"gitlab.server_url": meta.gitlabUrl,
+							"vcs.repository.id": meta.project,
+							"vcs.change.id": meta.mr,
+							...meta.platform ? { "vcs.provider.name": meta.platform } : {},
+							...meta.serverAddress ? { "server.address": meta.serverAddress } : {},
 							...meta.ciAttrs,
 							...meta.ciSpanAttrs
 						}
@@ -4964,20 +4979,30 @@ async function startOtelBridge(options = {}) {
 				ciAttrs,
 				runId,
 				configuredModel: runMeta.get(runId)?.model,
+				dryRun: runMeta.get(runId)?.dryRun,
+				platform: runMeta.get(runId)?.platform,
+				serverAddress: runMeta.get(runId)?.serverAddress,
 				captureContent
 			});
 		}
 	};
 }
 /**
-* Builds the dynamic `gen_ai.system` / `gen_ai.request.model` metric labels
-* shared by the per-turn and per-phase GenAI metric emitters. Owning these in
-* one place keeps the two emission sites from drifting into separate Prometheus
-* series (the double-count `recordGenAiMetrics` documents).
+* Builds the dynamic `gen_ai.provider.name` / `gen_ai.request.model` metric
+* labels shared by the per-turn and per-phase GenAI metric emitters. Owning
+* these in one place keeps the two emission sites from drifting into separate
+* Prometheus series (the double-count `recordGenAiMetrics` documents).
+*
+* `gen_ai.provider.name` is the current GenAI-semconv discriminator; the
+* deprecated `gen_ai.system` is emitted alongside it during the transition so
+* backends still keyed on the old attribute keep working.
 */
 function genAiModelAttrs(provider, modelId) {
 	return {
-		...provider ? { "gen_ai.system": provider } : {},
+		...provider ? {
+			"gen_ai.provider.name": provider,
+			"gen_ai.system": provider
+		} : {},
 		...modelId ? { "gen_ai.request.model": modelId } : {}
 	};
 }
@@ -5033,9 +5058,9 @@ function recordTurnUsage(span, u, metricAttrs, tokenUsage, operationCost) {
 			...metricAttrs,
 			"gen_ai.token.type": "cache_creation"
 		});
-		span.setAttribute("gen_ai.usage.cost.input_usd", u.cost.input);
-		span.setAttribute("gen_ai.usage.cost.output_usd", u.cost.output);
-		span.setAttribute("gen_ai.usage.cost.total_usd", u.cost.total);
+		span.setAttribute("code_review.cost.input_usd", u.cost.input);
+		span.setAttribute("code_review.cost.output_usd", u.cost.output);
+		span.setAttribute("code_review.cost.total_usd", u.cost.total);
 	}
 }
 /**
@@ -5099,12 +5124,15 @@ function extractOutputMessages(msg) {
 	}]);
 }
 function buildAgentSubscriber(tracer, tokenUsage, operationCost, timeToFirstToken, reviewerSpanCtx, options = {}) {
-	const { ciAttrs = {}, runId, configuredModel, captureContent = false } = options;
+	const { ciAttrs = {}, runId, configuredModel, dryRun, platform, serverAddress, captureContent = false } = options;
 	const configuredProvider = configuredModel ? splitModel(configuredModel).provider : void 0;
 	const baseMetricAttrs = {
 		"gen_ai.operation.name": "invoke_agent",
 		...REVIEW_SERVICE_ATTRS,
-		...ciAttrs
+		...ciAttrs,
+		...dryRun !== void 0 ? { "code_review.dry_run": dryRun } : {},
+		...platform ? { "vcs.provider.name": platform } : {},
+		...serverAddress ? { "server.address": serverAddress } : {}
 	};
 	return (agent) => {
 		let currentTurn;
@@ -5138,13 +5166,16 @@ function buildAgentSubscriber(tracer, tokenUsage, operationCost, timeToFirstToke
 					...baseMetricAttrs,
 					...genAiModelAttrs(provider, modelId)
 				};
-				if (provider) span.setAttribute("gen_ai.system", provider);
+				if (provider) {
+					span.setAttribute("gen_ai.provider.name", provider);
+					span.setAttribute("gen_ai.system", provider);
+				}
 				if (modelId) span.setAttribute("gen_ai.response.model", modelId);
 				if (msg.stopReason) span.setAttribute("gen_ai.response.stop_reason", msg.stopReason);
 				if (firstTokenMs !== void 0) {
 					const ttftS = (firstTokenMs - startMs) / 1e3;
 					timeToFirstToken.record(ttftS, metricAttrs);
-					span.setAttribute("gen_ai.client.time_to_first_token_s", ttftS);
+					span.setAttribute("gen_ai.client.operation.time_to_first_chunk_s", ttftS);
 				}
 				if (msg.usage) recordTurnUsage(span, msg.usage, metricAttrs, tokenUsage, operationCost);
 				if (captureContent) {
@@ -5205,6 +5236,26 @@ function buildAgentSubscriber(tracer, tokenUsage, operationCost, timeToFirstToke
 		});
 	};
 }
+/**
+* Applies our OTLP exporter env defaults, only when the caller hasn't set them
+* (explicit choices, including `none`, always win).
+*
+* Metrics temporality defaults to **delta** because code-review runs as a
+* short-lived CI job. With the OTel SDK's default cumulative temporality, each
+* ephemeral job's metric series starts mid-flight in the backend (its first
+* export already carries accumulated counts, and per-run histograms are a single
+* observation), so range aggregation of cost/token totals is unreliable —
+* `sum_over_time` over-counts and `increase` under-counts or returns nothing.
+* Delta makes each job's metrics self-contained deltas that sum correctly across
+* runs. Override with `OTEL_EXPORTER_OTLP_METRICS_TEMPORALITY_PREFERENCE=cumulative`
+* for long-running hosts, and confirm your backend ingests delta (Grafana
+* Cloud/Mimir converts to cumulative or stores it natively) before relying on it.
+*/
+function applyOtelExporterDefaults(env = process.env) {
+	env.OTEL_METRICS_EXPORTER = env.OTEL_METRICS_EXPORTER ?? "otlp";
+	env.OTEL_LOGS_EXPORTER = env.OTEL_LOGS_EXPORTER ?? "otlp";
+	env.OTEL_EXPORTER_OTLP_METRICS_TEMPORALITY_PREFERENCE = env.OTEL_EXPORTER_OTLP_METRICS_TEMPORALITY_PREFERENCE ?? "delta";
+}
 async function loadDefaultRuntime() {
 	let modules;
 	try {
@@ -5215,10 +5266,9 @@ async function loadDefaultRuntime() {
 	const [sdkNode, resources, semconv] = modules;
 	const serviceResource = resources.resourceFromAttributes({
 		[semconv.ATTR_SERVICE_NAME ?? "service.name"]: SERVICE_NAME,
-		[semconv.ATTR_SERVICE_VERSION ?? "service.version"]: "0.8.6"
+		[semconv.ATTR_SERVICE_VERSION ?? "service.version"]: "0.9.1"
 	});
-	process.env.OTEL_METRICS_EXPORTER = process.env.OTEL_METRICS_EXPORTER ?? "otlp";
-	process.env.OTEL_LOGS_EXPORTER = process.env.OTEL_LOGS_EXPORTER ?? "otlp";
+	applyOtelExporterDefaults(process.env);
 	const sdk = new sdkNode.NodeSDK({ resource: resources.defaultResource().merge(serviceResource) });
 	sdk.start();
 	return {
@@ -5237,14 +5287,14 @@ function emitReviewStartedLog(logger, ctx, ciAttrs, ciSpanAttrs, rootSpanCtx) {
 		context: rootSpanCtx,
 		attributes: {
 			"service.name": SERVICE_NAME,
-			"event.name": "gitlab_review.started",
-			"gitlab.project_id": ctx.project,
-			"gitlab.mr_iid": ctx.mr,
-			"gitlab.server_url": ctx.gitlabUrl,
+			"event.name": "code_review.started",
+			"vcs.repository.id": ctx.project,
+			"vcs.change.id": ctx.mr,
+			...vcsAttrs(ctx.platform, ctx.gitlabUrl),
 			...ciAttrs,
 			...ciSpanAttrs,
-			"gitlab_review.run_id": ctx.runId,
-			"gitlab_review.dry_run": ctx.dryRun,
+			"code_review.run_id": ctx.runId,
+			"code_review.dry_run": ctx.dryRun,
 			...modelId !== void 0 && { "gen_ai.request.model": modelId }
 		}
 	});
@@ -5264,10 +5314,10 @@ function emitReviewCompletedLog(logger, ctx, meta, isError) {
 		context: meta?.rootSpanCtx,
 		attributes: {
 			"service.name": SERVICE_NAME,
-			"event.name": isError ? "gitlab_review.failed" : "gitlab_review.completed",
-			"gitlab.project_id": ctx.project,
-			"gitlab.mr_iid": ctx.mr,
-			"gitlab.server_url": ctx.gitlabUrl,
+			"event.name": isError ? "code_review.failed" : "code_review.completed",
+			"vcs.repository.id": ctx.project,
+			"vcs.change.id": ctx.mr,
+			...vcsAttrs(ctx.platform, ctx.gitlabUrl),
 			...meta?.ciAttrs,
 			...meta?.ciSpanAttrs,
 			...isError && {
@@ -5275,15 +5325,15 @@ function emitReviewCompletedLog(logger, ctx, meta, isError) {
 				...ctx.errorInfo && { "error.message": ctx.errorInfo.message },
 				...typeof ctx.errorInfo?.status === "number" && { "http.response.status_code": ctx.errorInfo.status }
 			},
-			"gitlab_review.run_id": ctx.runId,
-			"gitlab_review.duration_ms": ctx.durationMs ?? 0,
-			"gitlab_review.dry_run": ctx.dryRun,
-			"gitlab_review.comments.generated": ctx.generated ?? 0,
-			"gitlab_review.comments.new": ctx.newComments ?? 0,
-			"gitlab_review.comments.duplicate": ctx.duplicateComments ?? 0,
-			"gitlab_review.comments.posted": ctx.posted ?? 0,
+			"code_review.run_id": ctx.runId,
+			"code_review.duration_ms": ctx.durationMs ?? 0,
+			"code_review.dry_run": ctx.dryRun,
+			"code_review.comments.generated": ctx.generated ?? 0,
+			"code_review.comments.new": ctx.newComments ?? 0,
+			"code_review.comments.duplicate": ctx.duplicateComments ?? 0,
+			"code_review.comments.posted": ctx.posted ?? 0,
 			...modelId !== void 0 && { "gen_ai.request.model": modelId },
-			...cost !== void 0 && { "gen_ai.usage.cost.total_usd": cost },
+			...cost !== void 0 && { "code_review.cost.total_usd": cost },
 			...usage?.tokens.input !== void 0 && { "gen_ai.usage.input_tokens": usage.tokens.input + (usage.tokens.cacheRead ?? 0) },
 			...usage?.tokens.cacheRead && {
 				"gen_ai.usage.input_tokens.cached": usage.tokens.cacheRead,
@@ -5302,39 +5352,39 @@ function spanNameFor(phase) {
 /** Creates the review-level OTel metric instruments on the given meter. */
 function createReviewInstruments(meter) {
 	return {
-		reviewRunsTotal: meter.createCounter("gitlab_review_runs_total", { description: "Total number of code-review runs, labelled by terminal status" }),
-		reviewErrorsTotal: meter.createCounter("gitlab_review_errors_total", { description: "Total number of failed code-review runs, labelled by error type" }),
+		reviewRunsTotal: meter.createCounter("code_review_runs_total", { description: "Total number of code-review runs, labelled by terminal status" }),
+		reviewErrorsTotal: meter.createCounter("code_review_errors_total", { description: "Total number of failed code-review runs, labelled by error type" }),
 		reviewLlmTokens: {
-			input: meter.createCounter("gitlab_review_llm_input_tokens_total", {
+			input: meter.createCounter("code_review_llm_input_tokens_total", {
 				description: "Total non-cached LLM input tokens consumed across code-review runs",
 				unit: "{token}"
 			}),
-			output: meter.createCounter("gitlab_review_llm_output_tokens_total", {
+			output: meter.createCounter("code_review_llm_output_tokens_total", {
 				description: "Total LLM output tokens generated across code-review runs",
 				unit: "{token}"
 			}),
-			cache_read: meter.createCounter("gitlab_review_llm_cache_read_tokens_total", {
+			cache_read: meter.createCounter("code_review_llm_cache_read_tokens_total", {
 				description: "Total LLM cache-read input tokens across code-review runs",
 				unit: "{token}"
 			}),
-			cache_creation: meter.createCounter("gitlab_review_llm_cache_creation_tokens_total", {
+			cache_creation: meter.createCounter("code_review_llm_cache_creation_tokens_total", {
 				description: "Total LLM cache-creation input tokens across code-review runs",
 				unit: "{token}"
 			})
 		},
-		reviewRunDuration: meter.createHistogram("gitlab_review_run_duration_seconds", {
+		reviewRunDuration: meter.createHistogram("code_review_run_duration_seconds", {
 			description: "Duration of a complete code-review run",
 			unit: "s",
 			advice: { explicitBucketBoundaries: REVIEW_RUN_DURATION_BUCKETS_S }
 		}),
-		reviewTotalCost: meter.createHistogram("gitlab_review_total_cost_usd", {
+		reviewTotalCost: meter.createHistogram("code_review_total_cost_usd", {
 			description: "Total LLM cost in USD for a complete code-review run",
 			unit: "{usd}",
 			advice: { explicitBucketBoundaries: REVIEW_TOTAL_COST_BUCKETS_USD }
 		}),
-		reviewCommentsTotal: meter.createCounter("gitlab_review_comments_total", { description: "Total number of MR comments posted by code-review" }),
-		reviewDraftsPublishedTotal: meter.createCounter("gitlab_review_drafts_published_total", { description: "Total number of draft notes published by code-review" }),
-		reviewPhaseDuration: meter.createHistogram("gitlab_review_phase_duration_seconds", {
+		reviewCommentsTotal: meter.createCounter("code_review_comments_total", { description: "Total number of MR comments posted by code-review" }),
+		reviewDraftsPublishedTotal: meter.createCounter("code_review_drafts_published_total", { description: "Total number of draft notes published by code-review" }),
+		reviewPhaseDuration: meter.createHistogram("code_review_phase_duration_seconds", {
 			description: "Duration of individual code-review workflow phases",
 			unit: "s",
 			advice: { explicitBucketBoundaries: REVIEW_PHASE_DURATION_BUCKETS_S }
@@ -5358,7 +5408,7 @@ function errorTypeOf(ctx) {
 	return base;
 }
 /**
-* Derives the `gitlab_review.status` label used by review-level OTel metrics.
+* Derives the `code_review.status` label used by review-level OTel metrics.
 * Distinguishes timeouts (AbortError / ETIMEDOUT) from generic errors so
 * Grafana alerts can treat deadline-exceeded runs separately.
 */
@@ -5369,55 +5419,91 @@ function resolveRunStatus(ctx, isError) {
 	return "error";
 }
 /**
-* Extracts GitLab CI environment variables that add project/pipeline context
-* to every metric, span, and log record. Only populated when running inside a
-* GitLab CI pipeline; callers spread the result so missing vars add nothing.
+* Extracts low-cardinality CI project/pipeline context (repository, owner, base
+* branch, pipeline trigger) as platform-neutral `vcs.*` / `cicd.*` attributes
+* added to every metric, span, and log record. Sourced from GitLab CI or GitHub
+* Actions variables — a run is one or the other, so `??` picks whichever is set;
+* callers spread the result so missing vars add nothing.
 */
 function buildCiAttrs(env) {
 	const attrs = {};
-	if (env.CI_PROJECT_PATH) attrs["gitlab.project_path"] = env.CI_PROJECT_PATH;
-	if (env.CI_PROJECT_NAMESPACE) attrs["gitlab.project_namespace"] = env.CI_PROJECT_NAMESPACE;
-	if (env.CI_MERGE_REQUEST_TARGET_BRANCH_NAME) attrs["gitlab.mr_target_branch"] = env.CI_MERGE_REQUEST_TARGET_BRANCH_NAME;
-	if (env.CI_PIPELINE_SOURCE) attrs["gitlab.pipeline_source"] = env.CI_PIPELINE_SOURCE;
+	const repository = env.CI_PROJECT_PATH ?? env.GITHUB_REPOSITORY;
+	if (repository) attrs["vcs.repository.name"] = repository;
+	const owner = env.CI_PROJECT_NAMESPACE ?? env.GITHUB_REPOSITORY_OWNER;
+	if (owner) attrs["vcs.owner.name"] = owner;
+	const baseRef = env.CI_MERGE_REQUEST_TARGET_BRANCH_NAME ?? env.GITHUB_BASE_REF;
+	if (baseRef) attrs["vcs.ref.base.name"] = baseRef;
+	const pipelineSource = env.CI_PIPELINE_SOURCE ?? env.GITHUB_EVENT_NAME;
+	if (pipelineSource) attrs["cicd.pipeline.source"] = pipelineSource;
 	return attrs;
 }
 /**
-* Extracts high-cardinality GitLab CI identifiers that should appear on spans
-* and log records but NOT on metric data points (to avoid label explosion in
-* Prometheus/Mimir). Spread results via `ciSpanAttrs` stored in RunMeta.
+* Extracts high-cardinality CI identifiers that should appear on spans and log
+* records but NOT on metric data points (to avoid label explosion in
+* Prometheus/Mimir). Sourced from GitLab CI or GitHub Actions. GitHub exposes no
+* per-job run id in the environment, so the job's config-key (`GITHUB_JOB`) is
+* used for the task identifier. Spread via `ciSpanAttrs` stored in RunMeta.
 */
 function buildCiSpanAttrs(env) {
 	const attrs = {};
-	if (env.CI_JOB_ID) attrs["gitlab.ci_job_id"] = env.CI_JOB_ID;
-	if (env.CI_PIPELINE_ID) attrs["gitlab.ci_pipeline_id"] = env.CI_PIPELINE_ID;
+	const taskRunId = env.CI_JOB_ID ?? env.GITHUB_JOB;
+	if (taskRunId) attrs["cicd.pipeline.task.run.id"] = taskRunId;
+	const pipelineRunId = env.CI_PIPELINE_ID ?? env.GITHUB_RUN_ID;
+	if (pipelineRunId) attrs["cicd.pipeline.run.id"] = pipelineRunId;
+	const repositoryUrl = env.CI_PROJECT_URL ?? (env.GITHUB_SERVER_URL && env.GITHUB_REPOSITORY ? `${env.GITHUB_SERVER_URL}/${env.GITHUB_REPOSITORY}` : void 0);
+	if (repositoryUrl) attrs["vcs.repository.url.full"] = repositoryUrl;
 	return attrs;
 }
+/** Host of a server URL, for the low-cardinality `server.address` instance label. */
+function hostOf(url) {
+	if (!url) return void 0;
+	try {
+		return new URL(url).host;
+	} catch {
+		return;
+	}
+}
+/**
+* Low-cardinality VCS discriminators added to every metric, span, and log:
+* `vcs.provider.name` (gitlab | github) to filter by platform, and
+* `server.address` (the instance host) to distinguish multiple GitLab/GitHub
+* instances (e.g. gitlab.com vs a self-hosted gitlab.example.com).
+*/
+function vcsAttrs(platform, serverUrl) {
+	const host = hostOf(serverUrl);
+	return {
+		...platform ? { "vcs.provider.name": platform } : {},
+		...host ? { "server.address": host } : {}
+	};
+}
 function baseAttributes(ctx) {
+	const host = hostOf(ctx.gitlabUrl);
 	return {
-		"gitlab_review.run_id": ctx.runId,
+		"code_review.run_id": ctx.runId,
 		"gen_ai.conversation.id": ctx.runId,
-		"gitlab_review.phase": ctx.phase,
-		"gitlab.project_id": ctx.project,
-		"gitlab.mr_iid": ctx.mr,
-		"gitlab.server_url": ctx.gitlabUrl,
-		"gitlab_review.dry_run": ctx.dryRun,
-		"gitlab_review.no_post": ctx.noPost,
-		"gitlab_review.min_severity": ctx.minSeverity
+		"code_review.phase": ctx.phase,
+		"vcs.repository.id": ctx.project,
+		"vcs.change.id": ctx.mr,
+		...ctx.platform ? { "vcs.provider.name": ctx.platform } : {},
+		...host ? { "server.address": host } : {},
+		"code_review.dry_run": ctx.dryRun,
+		"code_review.no_post": ctx.noPost,
+		"code_review.min_severity": ctx.minSeverity
 	};
 }
 var NUMERIC_RESULT_ATTRIBUTES = [
-	["durationMs", "gitlab_review.duration_ms"],
-	["generated", "gitlab_review.comments.generated"],
-	["newComments", "gitlab_review.comments.new"],
-	["duplicateComments", "gitlab_review.comments.duplicate"],
-	["posted", "gitlab_review.comments.posted"],
-	["draftsPublished", "gitlab_review.drafts.published"],
-	["draftsCreated", "gitlab_review.drafts.created"],
-	["summaryNoteId", "gitlab_review.summary.note_id"],
-	["warnings", "gitlab_review.warnings"],
-	["draftsAbandoned", "gitlab_review.drafts.abandoned"],
-	["draftsDeletedPrePublish", "gitlab_review.drafts.deleted_pre_publish"],
-	["draftsPublishFailed", "gitlab_review.drafts.publish_failed"],
+	["durationMs", "code_review.duration_ms"],
+	["generated", "code_review.comments.generated"],
+	["newComments", "code_review.comments.new"],
+	["duplicateComments", "code_review.comments.duplicate"],
+	["posted", "code_review.comments.posted"],
+	["draftsPublished", "code_review.drafts.published"],
+	["draftsCreated", "code_review.drafts.created"],
+	["summaryNoteId", "code_review.summary.note_id"],
+	["warnings", "code_review.warnings"],
+	["draftsAbandoned", "code_review.drafts.abandoned"],
+	["draftsDeletedPrePublish", "code_review.drafts.deleted_pre_publish"],
+	["draftsPublishFailed", "code_review.drafts.publish_failed"],
 	["diffFilesChanged", "diff.files_changed"],
 	["diffLinesAdded", "diff.lines_added"],
 	["diffLinesRemoved", "diff.lines_removed"],
@@ -5425,7 +5511,7 @@ var NUMERIC_RESULT_ATTRIBUTES = [
 	["httpResponseBodySize", "http.response.body.size"]
 ];
 var STRING_RESULT_ATTRIBUTES = [
-	["summaryAction", "gitlab_review.summary.action"],
+	["summaryAction", "code_review.summary.action"],
 	["httpRequestMethod", "http.request.method"],
 	["httpUrl", "url.full"],
 	["serverAddress", "server.address"]
@@ -5442,7 +5528,10 @@ function applyResultAttributes(span, ctx) {
 }
 function applyGenAiAttributes(span, ctx) {
 	const { provider, modelId } = splitModel(ctx.model ?? "");
-	if (provider) span.setAttribute("gen_ai.system", provider);
+	if (provider) {
+		span.setAttribute("gen_ai.provider.name", provider);
+		span.setAttribute("gen_ai.system", provider);
+	}
 	if (modelId) {
 		span.setAttribute("gen_ai.request.model", modelId);
 		span.setAttribute("gen_ai.response.model", modelId);
@@ -5452,16 +5541,16 @@ function applyGenAiAttributes(span, ctx) {
 	const usage = ctx.usage;
 	if (!usage) return;
 	setTokenUsageSpanAttributes(span, usage.tokens);
-	span.setAttribute("gen_ai.usage.cost.input_usd", usage.cost.input);
-	span.setAttribute("gen_ai.usage.cost.output_usd", usage.cost.output);
-	span.setAttribute("gen_ai.usage.cost.cache_read_usd", usage.cost.cacheRead);
-	span.setAttribute("gen_ai.usage.cost.cache_creation_usd", usage.cost.cacheWrite);
-	span.setAttribute("gen_ai.usage.cost.total_usd", usage.cost.total);
+	span.setAttribute("code_review.cost.input_usd", usage.cost.input);
+	span.setAttribute("code_review.cost.output_usd", usage.cost.output);
+	span.setAttribute("code_review.cost.cache_read_usd", usage.cost.cacheRead);
+	span.setAttribute("code_review.cost.cache_creation_usd", usage.cost.cacheWrite);
+	span.setAttribute("code_review.cost.total_usd", usage.cost.total);
 }
 /**
 * Records `gen_ai.client.operation.duration` for the `reviewer.run` phase.
 *
-* Token usage (`gen_ai.client.token.usage`) and cost (`gen_ai.client.cost`) are
+* Token usage (`gen_ai.client.token.usage`) and cost (`code_review_llm_cost_usd`) are
 * intentionally NOT recorded here. They are emitted per-turn by
 * `buildAgentSubscriber` from the live agent event stream. Keeping a single
 * emission point for each metric prevents the double-count that previously
@@ -5474,7 +5563,9 @@ function recordGenAiMetrics(durationHist, ctx, isError, ciAttrs = {}) {
 		"gen_ai.operation.name": "invoke_agent",
 		...REVIEW_SERVICE_ATTRS,
 		...ciAttrs,
-		...genAiModelAttrs(provider, modelId)
+		...vcsAttrs(ctx.platform, ctx.gitlabUrl),
+		...genAiModelAttrs(provider, modelId),
+		"code_review.dry_run": ctx.dryRun
 	};
 	if (isError) attrs["error.type"] = errorTypeOf(ctx);
 	if (typeof ctx.durationMs === "number") durationHist.record(ctx.durationMs / 1e3, attrs);
@@ -5647,7 +5738,7 @@ function boldCommentTitle(body) {
 */
 function buildCommentBody(body, commitSha, confidence) {
 	const confidenceLine = `_Confidence: ${confidence}._`;
-	const footer = `<sub>Reviewed by ${PRODUCT_LINK} v0.8.6 for commit ${commitSha}.</sub>`;
+	const footer = `<sub>Reviewed by ${PRODUCT_LINK} v0.9.1 for commit ${commitSha}.</sub>`;
 	return `${boldCommentTitle(body.trim())}\n\n${confidenceLine}\n\n---\n\n${footer}`;
 }
 function buildPayload(comment, body, refs, resolved) {
@@ -6442,6 +6533,7 @@ async function run(config, bridges) {
 		const refs = await tracedRead("scm.get_latest_version", () => platform.getRefs());
 		const initialDiscussions = await tracedRead("scm.get_discussions", () => platform.getDiscussions());
 		const reviewedCommitSha = findExistingReviewedCommitSha(initialDiscussions);
+		runContext.firstReview = findExistingSummaryNote(initialDiscussions) === null;
 		if (!config.forceReview && !config.dryRun && !config.noPost && reviewedCommitSha === refs.head_sha) {
 			const usage = zeroReviewUsage(config.model, config.thinkingLevel);
 			runContext.usage = usage;
@@ -6484,27 +6576,36 @@ async function run(config, bridges) {
 		if (priorThreads.length > 0) logger.info(`Found ${priorThreads.length} prior thread(s) with developer replies — including as context.`);
 		logger.info("Running review...");
 		let usage;
+		let partialUsage;
 		try {
 			usage = await traceDiagnosticPhase("reviewer.run", config, runId, async (context) => {
-				const result = await runReview(config, {
-					cwd: config.cwd,
-					diff,
-					commitLog,
-					priorThreads,
-					intent: {
-						title: mr.title,
-						description: mr.description
-					},
-					logger,
-					attachTelemetry: bridges?.otel?.createAgentTelemetry(runId),
-					sinceRef: config.inputMode === "commits" && reviewedCommitSha && reviewedCommitSha !== refs.head_sha ? reviewedCommitSha : void 0
-				});
-				context.usage = result;
-				return result;
+				try {
+					const result = await runReview(config, {
+						cwd: config.cwd,
+						diff,
+						commitLog,
+						priorThreads,
+						intent: {
+							title: mr.title,
+							description: mr.description
+						},
+						logger,
+						attachTelemetry: bridges?.otel?.createAgentTelemetry(runId),
+						onUsage: (u) => {
+							partialUsage = u;
+						},
+						sinceRef: config.inputMode === "commits" && reviewedCommitSha && reviewedCommitSha !== refs.head_sha ? reviewedCommitSha : void 0
+					});
+					context.usage = result;
+					return result;
+				} catch (error) {
+					if (partialUsage) context.usage = partialUsage;
+					throw error;
+				}
 			});
 		} catch (error) {
 			if (!isQuotaExceededError(error)) throw error;
-			const skipUsage = zeroReviewUsage(config.model, config.thinkingLevel);
+			const skipUsage = partialUsage ?? zeroReviewUsage(config.model, config.thinkingLevel);
 			runContext.usage = skipUsage;
 			runContext.generated = 0;
 			runContext.newComments = 0;
@@ -6744,10 +6845,10 @@ async function main(argv = process.argv.slice(2)) {
 		return;
 	}
 	if (argv.includes("--version") || argv.includes("-v")) {
-		console.log("0.8.6");
+		console.log("0.9.1");
 		return;
 	}
-	process.stderr.write(`[code-review] @weareikko/code-review v0.8.6\n`);
+	process.stderr.write(`[code-review] @weareikko/code-review v0.9.1\n`);
 	assertNodeVersion();
 	applyCodeReviewEnvPrefix();
 	applyDefaultCacheRetention();
@@ -6770,4 +6871,4 @@ if (isDirectRun()) main().catch((error) => {
 //#endregion
 export { normalizeBody as $, SUMMARY_HISTORY_END as A, buildSummaryHistoryEntries as B, createDiagnosticContext as C, traceDiagnosticPhase as D, traceDiagnostic as E, SUMMARY_MARKER as F, findExistingSummaryNoteId as G, extractSummaryHistoryEntries as H, buildArchivedSummaryEntry as I, upsertSummaryNote as J, stripSummaryHistory as K, buildReviewedCommitFooter as L, SUMMARY_HISTORY_ENTRY_START as M, SUMMARY_HISTORY_LIMIT as N, normalizeSeverity as O, SUMMARY_HISTORY_START as P, fingerprints as Q, buildSizeNoticeBlock as R, DIAGNOSTIC_CHANNEL_PREFIX as S, diagnosticChannels as T, findExistingReviewedCommitSha as U, extractReviewedCommitSha as V, findExistingSummaryNote as W, extractDiffHunkContext as X, appendFingerprintMarkers as Y, extractExistingFingerprints as Z, resolveNpmSkillDir as _, main as a, parseReviewMarkdownWithWarnings as b, buildGeneratedComments as c, startOtelBridge as d, sha256 as et, filterDiff as f, parseSkillSpec as g, loadNamedSkill as h, formatUsageLine as i, SUMMARY_HISTORY_ENTRY_END as j, toGitLabReviewSeverity as k, buildPayload as l, gitSkillCacheKey as m, formatPerModelUsage as n, run as o, runReview as p, stripSummaryMarker as q, formatSkillsFooter as r, withHttpStamping as s, countPostedBySeverity as t, isOtelEnabled as u, resolveSkillCacheDir as v, createDiagnosticRunId as w, DIAGNOSTIC_CHANNEL_NAMES as x, parseReviewMarkdown as y, buildSummaryBody as z };
 
-//# sourceMappingURL=cli-BbR1oftT.js.map
+//# sourceMappingURL=cli-BGn8g23f.js.map