@weareikko/code-review 0.8.5 → 0.9.0

package/dist/{cli-DmzA9PxS.js → cli-DK0kteLL.js}

@@ -166,6 +166,7 @@ var REVIEW_THREADS_QUERY = `
           pageInfo { hasNextPage endCursor }
           nodes {
             isResolved
+            isOutdated
             comments(first: 100) { nodes { databaseId } }
           }
         }
@@ -338,14 +339,22 @@ var GitHubClient = class {
 		return parsed.data;
 	}
 	/**
-	* Return the database IDs of review comments that belong to a **resolved**
-	* review thread. GitHub's REST comment endpoints omit thread-resolution state;
-	* it is only exposed via GraphQL `reviewThreads.isResolved`. Callers use this
-	* set to mark normalized notes resolved so resolved threads are excluded from
-	* summary carry-over and prior-thread context. Paginates over threads.
+	* Return the database IDs of review comments that belong to a **settled**
+	* review thread — one that is either resolved or outdated. GitHub's REST
+	* comment endpoints omit both states; they are only exposed via GraphQL
+	* `reviewThreads.isResolved` / `isOutdated`. Callers use this set to mark
+	* normalized notes resolved so settled threads are excluded from summary
+	* carry-over and prior-thread context. Paginates over threads.
+	*
+	* Outdated counts as settled because GitHub, unlike GitLab, does not
+	* auto-resolve a thread when the line it anchors to changes: fixing a finding
+	* flips the thread to outdated but leaves `isResolved` false until someone
+	* manually resolves it. Treating outdated as settled mirrors GitLab's
+	* "automatically resolve outdated diff discussions" behaviour, so a fixed
+	* finding stops being re-listed under "Still open from earlier reviews" (#133).
 	*/
-	async listResolvedReviewCommentIds(owner, repo, pull) {
-		const resolved = /* @__PURE__ */ new Set();
+	async listSettledReviewCommentIds(owner, repo, pull) {
+		const settled = /* @__PURE__ */ new Set();
 		let cursor = null;
 		let hasNext = true;
 		while (hasNext) {
@@ -357,14 +366,14 @@ var GitHubClient = class {
 			})).repository?.pullRequest?.reviewThreads;
 			if (!threads) break;
 			for (const thread of threads.nodes ?? []) {
-				if (!thread.isResolved) continue;
-				for (const comment of thread.comments?.nodes ?? []) if (typeof comment.databaseId === "number") resolved.add(comment.databaseId);
+				if (!thread.isResolved && !thread.isOutdated) continue;
+				for (const comment of thread.comments?.nodes ?? []) if (typeof comment.databaseId === "number") settled.add(comment.databaseId);
 			}
 			hasNext = threads.pageInfo?.hasNextPage ?? false;
 			cursor = threads.pageInfo?.endCursor ?? null;
 			if (!cursor) hasNext = false;
 		}
-		return resolved;
+		return settled;
 	}
 };
 //#endregion
@@ -551,7 +560,7 @@ function buildSummaryBody(summary, costFooter, options = {}) {
 	return `${withFooter}\n\n${buildSummaryHistoryBlock(historyEntries)}`;
 }
 function buildReviewedCommitFooter(commitSha) {
-	return `Reviewed by ${PRODUCT_LINK} v0.8.5 for commit ${commitSha}.`;
+	return `Reviewed by ${PRODUCT_LINK} v0.9.0 for commit ${commitSha}.`;
 }
 function extractReviewedCommitSha(body) {
 	return REVIEWED_COMMIT_FOOTER_PATTERN.exec(body)?.[1] ?? null;
@@ -1350,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,
@@ -3681,8 +3691,15 @@ async function loadReviewContext(cwd, skillNames = [], warn, options = {}) {
 	]);
 	const skills = [...discovered];
 	const discoveredNames = new Set(discovered.map((s) => s.name));
-	const named = await Promise.all(skillNames.filter((n) => !discoveredNames.has(n)).map((n) => loadNamedSkill(n, cwd, { refresh: options.refreshGitSkills })));
-	skills.push(...named);
+	const named = await Promise.all(skillNames.filter((n) => !discoveredNames.has(n)).map(async (n) => {
+		try {
+			return await loadNamedSkill(n, cwd, { refresh: options.refreshGitSkills });
+		} catch (error) {
+			warn?.(`Skipping skill "${n}": ${formatError(error)}`);
+			return null;
+		}
+	}));
+	skills.push(...named.filter((s) => s !== null));
 	return {
 		conventions,
 		reviewRules,
@@ -4338,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,
@@ -4742,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 }
 	});
@@ -4788,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);
@@ -4819,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"],
@@ -4878,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;
@@ -4924,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
 						}
@@ -4948,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 } : {}
 	};
 }
@@ -5017,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);
 	}
 }
 /**
@@ -5083,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;
@@ -5122,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) {
@@ -5199,7 +5246,7 @@ 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.5"
+		[semconv.ATTR_SERVICE_VERSION ?? "service.version"]: "0.9.0"
 	});
 	process.env.OTEL_METRICS_EXPORTER = process.env.OTEL_METRICS_EXPORTER ?? "otlp";
 	process.env.OTEL_LOGS_EXPORTER = process.env.OTEL_LOGS_EXPORTER ?? "otlp";
@@ -5221,14 +5268,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 }
 		}
 	});
@@ -5248,10 +5295,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 && {
@@ -5259,15 +5306,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,
@@ -5286,39 +5333,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 }
@@ -5342,7 +5389,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.
 */
@@ -5353,55 +5400,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"],
@@ -5409,7 +5492,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"]
@@ -5426,7 +5509,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);
@@ -5436,16 +5522,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
@@ -5458,7 +5544,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);
@@ -5631,7 +5719,7 @@ function boldCommentTitle(body) {
 */
 function buildCommentBody(body, commitSha, confidence) {
 	const confidenceLine = `_Confidence: ${confidence}._`;
-	const footer = `<sub>Reviewed by ${PRODUCT_LINK} v0.8.5 for commit ${commitSha}.</sub>`;
+	const footer = `<sub>Reviewed by ${PRODUCT_LINK} v0.9.0 for commit ${commitSha}.</sub>`;
 	return `${boldCommentTitle(body.trim())}\n\n${confidenceLine}\n\n---\n\n${footer}`;
 }
 function buildPayload(comment, body, refs, resolved) {
@@ -5795,11 +5883,14 @@ function reviewCommentPosition(comment) {
 * comments that render identically on GitHub, so `extractExistingFingerprints`,
 * `findExistingSummaryNote`, and the reviewed-commit scan all work as-is.
 *
-* `resolvedCommentIds` carries the database ids of comments in resolved review
-* threads (from the GraphQL `reviewThreads` query, since REST omits resolution),
-* so each note gets a `resolved` flag mirroring GitLab's per-note field.
+* `settledCommentIds` carries the database ids of comments in settled review
+* threads — resolved or outdated (from the GraphQL `reviewThreads` query, since
+* REST omits both). Each such note gets a `resolved` flag mirroring GitLab's
+* per-note field: an outdated GitHub thread maps to `resolved: true` because
+* GitHub, unlike GitLab, does not auto-resolve a thread when its anchored line
+* changes, so treating outdated as resolved matches GitLab's behaviour (#133).
 */
-function normalizeGitHubDiscussions(reviewComments, issueComments, resolvedCommentIds = /* @__PURE__ */ new Set()) {
+function normalizeGitHubDiscussions(reviewComments, issueComments, settledCommentIds = /* @__PURE__ */ new Set()) {
 	const threads = /* @__PURE__ */ new Map();
 	const order = [];
 	for (const comment of reviewComments) {
@@ -5813,7 +5904,7 @@ function normalizeGitHubDiscussions(reviewComments, issueComments, resolvedComme
 		notes.push({
 			id: comment.id,
 			body: comment.body ?? "",
-			resolved: resolvedCommentIds.has(comment.id),
+			resolved: settledCommentIds.has(comment.id),
 			position: reviewCommentPosition(comment)
 		});
 	}
@@ -5880,12 +5971,12 @@ var GitHubPlatform = class {
 		return refs;
 	}
 	async getDiscussions() {
-		const [reviewComments, issueComments, resolvedCommentIds] = await Promise.all([
+		const [reviewComments, issueComments, settledCommentIds] = await Promise.all([
 			this.client.listReviewComments(this.owner, this.repo, this.pull),
 			this.client.listIssueComments(this.owner, this.repo, this.pull),
-			this.client.listResolvedReviewCommentIds(this.owner, this.repo, this.pull)
+			this.client.listSettledReviewCommentIds(this.owner, this.repo, this.pull)
 		]);
-		return normalizeGitHubDiscussions(reviewComments, issueComments, resolvedCommentIds);
+		return normalizeGitHubDiscussions(reviewComments, issueComments, settledCommentIds);
 	}
 	buildComments(comments, diff, refs, existingFingerprints) {
 		this.commitId = refs.head_sha;
@@ -6423,6 +6514,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;
@@ -6465,27 +6557,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;
@@ -6725,10 +6826,10 @@ async function main(argv = process.argv.slice(2)) {
 		return;
 	}
 	if (argv.includes("--version") || argv.includes("-v")) {
-		console.log("0.8.5");
+		console.log("0.9.0");
 		return;
 	}
-	process.stderr.write(`[code-review] @weareikko/code-review v0.8.5\n`);
+	process.stderr.write(`[code-review] @weareikko/code-review v0.9.0\n`);
 	assertNodeVersion();
 	applyCodeReviewEnvPrefix();
 	applyDefaultCacheRetention();
@@ -6751,4 +6852,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-DmzA9PxS.js.map
+//# sourceMappingURL=cli-DK0kteLL.js.map