@weareikko/code-review 0.9.4 → 0.9.6

package/dist/{cli-C3kNr0rX.js → cli-D6vNV3kA.js}

@@ -2,7 +2,7 @@ import { mkdir, readFile, readdir, realpath, rename, rm, unlink, writeFile } fro
 import { dirname, join, relative, resolve, sep } from "node:path";
 import { fileURLToPath, pathToFileURL } from "node:url";
 import nodeFs, { existsSync, readFileSync } from "node:fs";
-import { getEnvApiKey, getModel } from "@earendil-works/pi-ai";
+import { getEnvApiKey, streamSimple } from "@earendil-works/pi-ai/compat";
 import { createHash, randomUUID } from "node:crypto";
 import { homedir } from "node:os";
 import { parse } from "yaml";
@@ -11,6 +11,7 @@ import { promisify } from "node:util";
 import { tracingChannel } from "node:diagnostics_channel";
 import { performance } from "node:perf_hooks";
 import { Agent } from "@earendil-works/pi-agent-core";
+import { getBuiltinModel } from "@earendil-works/pi-ai/providers/all";
 import { createReadOnlyTools } from "@earendil-works/pi-coding-agent";
 import { createTwoFilesPatch } from "diff";
 import * as git from "isomorphic-git";
@@ -540,6 +541,10 @@ function summarizeDiff(diff) {
 }
 //#endregion
 //#region src/skills.ts
+/** Normalize a filesystem-relative path to the POSIX form used in URLs. */
+function toPosixPath(path) {
+	return sep === "/" ? path : path.split(sep).join("/");
+}
 var SKILL_DIRS = [".agents/skills", ".claude/skills"];
 var RESOURCE_DIRS = ["references"];
 function parseFrontmatter(content) {
@@ -562,7 +567,7 @@ function parseFrontmatter(content) {
 		description: trimmedDescription
 	};
 }
-async function loadSkillFromDir(dirPath, source) {
+async function loadSkillFromDir(dirPath, origin) {
 	const skillMdPath = join(dirPath, "SKILL.md");
 	let content;
 	try {
@@ -579,14 +584,17 @@ async function loadSkillFromDir(dirPath, source) {
 		filePath: skillMdPath,
 		rootDir: dirPath,
 		resourceDirs,
-		source
+		origin
 	};
 }
 function resolveBuiltinSkillsDir() {
 	return join(dirname(fileURLToPath(import.meta.url)), "..", "skills");
 }
 async function loadBuiltinSkill(name) {
-	return loadSkillFromDir(join(resolveBuiltinSkillsDir(), name), "builtin");
+	return loadSkillFromDir(join(resolveBuiltinSkillsDir(), name), {
+		kind: "builtin",
+		name
+	});
 }
 async function loadAutoDiscoveredSkills(cwd, gitRoot, warn) {
 	const dirs = [];
@@ -609,7 +617,10 @@ async function loadAutoDiscoveredSkills(cwd, gitRoot, warn) {
 		}
 		for (const entry of entries) {
 			const entryPath = join(skillsPath, entry);
-			const skill = await loadSkillFromDir(entryPath, "project");
+			const skill = await loadSkillFromDir(entryPath, {
+				kind: "project",
+				path: toPosixPath(relative(gitRoot, entryPath))
+			});
 			if (skill) found.set(skill.name, skill);
 			else if (warn && existsSync(join(entryPath, "SKILL.md"))) warn(`Skill at ${entryPath} has a SKILL.md but is missing required frontmatter fields (name, description) — skill not loaded.`);
 		}
@@ -897,13 +908,20 @@ async function loadNamedSkill(spec, cwd, options = {}) {
 			const pkgRef = parsed.subpath ? `${parsed.packageName} (subpath "${parsed.subpath}")` : parsed.packageName;
 			throw new ConfigError(`Cannot load skill: "${spec}"`, { hint: `Package ${pkgRef} was not found in node_modules. Run \`npm install ${parsed.packageName}\` in the project.` });
 		}
-		const skill = await loadSkillFromDir(dir, "npm");
+		const skill = await loadSkillFromDir(dir, {
+			kind: "npm",
+			packageName: parsed.packageName,
+			subpath: parsed.subpath
+		});
 		if (!skill) throw new ConfigError(`Cannot load skill: "${spec}"`, { hint: `The package at ${dir} does not contain a valid SKILL.md.` });
 		return skill;
 	}
 	if (parsed.protocol === "file") {
 		const resolvedPath = parsed.path.startsWith("/") ? parsed.path : join(cwd, parsed.path);
-		const skill = await loadSkillFromDir(resolvedPath, "file");
+		const skill = await loadSkillFromDir(resolvedPath, {
+			kind: "file",
+			path: resolvedPath
+		});
 		if (!skill) throw new ConfigError(`Cannot load skill: "${spec}"`, { hint: `No valid SKILL.md was found at "${resolvedPath}". Check that the path points to a skill directory.` });
 		return skill;
 	}
@@ -921,7 +939,12 @@ async function loadNamedSkill(spec, cwd, options = {}) {
 			hint: `Failed to clone "${redactUrl(parsed.url)}"${atRef}. Check the URL, the ref, and your git credentials. For GitLab, prefer the SSH form: git+ssh://git@host/group/project.git`
 		});
 	}
-	const skill = await loadSkillFromDir(parsed.subpath ? join(repoDir, parsed.subpath) : repoDir, "git");
+	const skill = await loadSkillFromDir(parsed.subpath ? join(repoDir, parsed.subpath) : repoDir, {
+		kind: "git",
+		url: parsed.url,
+		ref: parsed.ref,
+		path: parsed.subpath
+	});
 	if (!skill) throw new ConfigError(`Cannot load skill: "${spec}"`, { hint: parsed.subpath ? `The cloned repository has no valid SKILL.md at subpath "${parsed.subpath}".` : "The cloned repository has no valid SKILL.md at its root. If the skill lives in a subdirectory, point at it with \"#<ref>/<subpath>\"." });
 	return skill;
 }
@@ -1060,7 +1083,7 @@ async function resolveAnthropicSkill(repoDir, mp, spec) {
 	const pluginDir = resolvePluginDir(repoDir, manifest, entry.source, mp, spec);
 	const realPluginDir = await resolveInside(realRoot, pluginDir, ref);
 	const bases = computeSkillBases(repoDir, pluginDir, entry, entry.strict !== false && realPluginDir ? await readPluginManifest(realRoot, realPluginDir, ref) : null, ref);
-	const skill = await findSkillInBases(realRoot, bases, pluginDir, spec.skill, ref);
+	const skill = await findSkillInBases(realRoot, bases, pluginDir, spec, ref, mp);
 	if (!skill) throw new ConfigError(`Cannot load skill: "${ref}"`, { hint: `No skill "${spec.skill}" found in plugin "${spec.plugin}". Looked under the default skills/ directory${bases.length > 1 ? " and the plugin's custom \"skills\" paths" : ""}. Check the skill name against the marketplace.` });
 	return skill;
 }
@@ -1120,14 +1143,15 @@ function resolveUnderPlugin(repoDir, pluginDir, rel, ref) {
 * symlinks and reject any directory that escapes the cloned marketplace before a
 * file is read.
 */
-async function findSkillInBases(realRoot, bases, pluginDir, skillName, ref) {
+async function findSkillInBases(realRoot, bases, pluginDir, spec, ref, mp) {
+	const skillName = spec.skill;
 	for (const base of bases) {
-		const container = await loadSkillIfInside(realRoot, join(base, skillName), ref);
+		const container = await loadSkillIfInside(realRoot, join(base, skillName), ref, mp, spec);
 		if (container) return container;
-		const single = await loadSkillIfInside(realRoot, base, ref);
+		const single = await loadSkillIfInside(realRoot, base, ref, mp, spec);
 		if (single && single.name === skillName) return single;
 	}
-	const root = await loadSkillIfInside(realRoot, pluginDir, ref);
+	const root = await loadSkillIfInside(realRoot, pluginDir, ref, mp, spec);
 	return root && root.name === skillName ? root : null;
 }
 /**
@@ -1172,11 +1196,18 @@ async function readTextInside(realRoot, filePath, ref) {
 * resolve within `realRoot`. Guarding the file (not just the directory) closes
 * the case where a legitimate in-repo dir holds a `SKILL.md` symlinked outside.
 */
-async function loadSkillIfInside(realRoot, dir, ref) {
+async function loadSkillIfInside(realRoot, dir, ref, mp, spec) {
 	const real = await resolveInside(realRoot, dir, ref);
 	if (!real) return null;
 	if (await readTextInside(realRoot, join(real, "SKILL.md"), ref) === null) return null;
-	return loadSkillFromDir(real, "marketplace");
+	return loadSkillFromDir(real, {
+		kind: "marketplace",
+		marketplace: mp.name,
+		plugin: spec.plugin,
+		url: mp.url,
+		ref: mp.ref,
+		path: toPosixPath(relative(realRoot, real))
+	});
 }
 /**
 * Resolve a plugin entry's `source` to an absolute directory inside the cloned
@@ -1303,13 +1334,24 @@ function extractExistingFingerprints(discussions) {
 	for (const discussion of discussions) for (const note of discussion.notes ?? []) for (const match of String(note.body ?? "").matchAll(FINGERPRINT_MARKER_RE$2)) set.add(match[1]);
 	return set;
 }
+//#endregion
+//#region src/product.ts
+/**
+* The published package name, shown in review footers. Independent of the review
+* platform: the same tool posts to GitLab and GitHub, so the footer identifies
+* the tool, not the backend. Kept as a single source of truth so the inline and
+* summary footers can never drift apart.
+*/
+var PRODUCT_NAME = "@weareikko/code-review";
+/** Canonical project URL used in the footer's markdown link. */
+var PRODUCT_URL = "https://github.com/weareikko/code-review";
 /**
 * The `[name](url)` markdown link used verbatim in both the inline comment footer
 * ({@link buildCommentBody}) and the reviewed-commit summary footer
 * ({@link buildReviewedCommitFooter}). Changing this changes the reviewed-commit
 * footer format, which is guarded by a migration test.
 */
-var PRODUCT_LINK = `[@weareikko/code-review](https://github.com/weareikko/code-review)`;
+var PRODUCT_LINK = `[${PRODUCT_NAME}](${PRODUCT_URL})`;
 //#endregion
 //#region src/posting.ts
 var SUMMARY_MARKER = "<!-- code-review:summary -->";
@@ -1389,7 +1431,7 @@ function buildSummaryBody(summary, costFooter, options = {}) {
 	return `${withFooter}\n\n${buildSummaryHistoryBlock(historyEntries)}`;
 }
 function buildReviewedCommitFooter(commitSha) {
-	return `Reviewed by ${PRODUCT_LINK} v0.9.4 for commit ${commitSha}.`;
+	return `Reviewed by ${PRODUCT_LINK} v0.9.6 for commit ${commitSha}.`;
 }
 function extractReviewedCommitSha(body) {
 	return REVIEWED_COMMIT_FOOTER_PATTERN.exec(body)?.[1] ?? null;
@@ -2034,6 +2076,16 @@ function detectPlatform(args, env, readEventFile = readEventFileSync) {
 	if (hasGitLab && !hasGitHub) return "gitlab";
 	throw new ConfigError(hasGitHub && hasGitLab ? "Ambiguous review platform: both GitHub and GitLab identifiers are present." : "Could not detect the review platform from the environment.", { hint: `Set --platform (or CODE_REVIEW_PLATFORM) to one of: ${PLATFORMS.join(", ")}.` });
 }
+/**
+* Web URL of the repository under review, as the CI environment reports it:
+* GitLab exposes it directly, GitHub composes it from the server and the
+* `owner/repo` slug. Returns `undefined` outside CI, where no such URL exists.
+* Shared by the config (skill source links) and the OTel attribute builder.
+*/
+function resolveProjectWebUrl(env = process.env) {
+	if (env.CI_PROJECT_URL) return env.CI_PROJECT_URL.replace(/\/$/, "");
+	if (env.GITHUB_SERVER_URL && env.GITHUB_REPOSITORY) return `${env.GITHUB_SERVER_URL.replace(/\/$/, "")}/${env.GITHUB_REPOSITORY}`;
+}
 function resolveConfig(argv = process.argv.slice(2), env = process.env) {
 	const args = parseArgs(argv);
 	const platform = detectPlatform(args, env);
@@ -2041,6 +2093,7 @@ function resolveConfig(argv = process.argv.slice(2), env = process.env) {
 	const token = resolveGitLabToken(args, env);
 	const githubApiUrl = String(args.githubApiUrl ?? env.GITHUB_API_URL ?? "https://api.github.com").replace(/\/$/, "");
 	const githubServerUrl = String(args.githubServerUrl ?? env.GITHUB_SERVER_URL ?? "https://github.com").replace(/\/$/, "");
+	const githubRepository = String(args.githubRepository ?? env.GITHUB_REPOSITORY ?? "");
 	const model = String(args.model ?? env.CODE_REVIEW_MODEL ?? "");
 	const apiKey = String(args.apiKey ?? resolveProviderApiKey(model) ?? "");
 	const baseUrl = String(args.baseUrl ?? first(env.CODE_REVIEW_BASE_URL, resolveOllamaBaseUrl(model, env)) ?? "");
@@ -2059,11 +2112,12 @@ function resolveConfig(argv = process.argv.slice(2), env = process.env) {
 		gitlabUrl,
 		gitlabToken: token.token,
 		gitlabAuthHeader: token.header,
-		githubRepository: String(args.githubRepository ?? env.GITHUB_REPOSITORY ?? ""),
+		githubRepository,
 		githubPr: resolveGitHubPr(args, env),
 		githubToken: String(args.githubToken ?? env.GITHUB_TOKEN ?? ""),
 		githubApiUrl,
 		githubServerUrl,
+		projectWebUrl: resolveProjectWebUrl(env) ?? "",
 		model,
 		modelPool: resolveModelPool(args, env),
 		minSeverity: normalizeChoice(args.minSeverity ?? env.CODE_REVIEW_MIN_SEVERITY ?? "info"),
@@ -3961,6 +4015,26 @@ var SEVERITY_RULE = {
 	CRITICAL: "- Only report CRITICAL issues — skip WARN and INFO"
 };
 var exec = promisify(execFile);
+/**
+* Stream function for the reviewer agent.
+*
+* pi-ai >=0.82 requires an explicit stream function (earlier versions built one
+* internally from model + getApiKey); `streamSimple` is the drop-in. Critically,
+* 0.83 also moved cloudflare-ai-gateway base-URL substitution — the
+* `{CLOUDFLARE_ACCOUNT_ID}` / `{CLOUDFLARE_GATEWAY_ID}` placeholders — from a
+* direct `process.env` read to an explicit `env` on the stream options. Without
+* threading `env` through, the gateway URL keeps its literal placeholders and
+* every request fails with Cloudflare 401 2035 ("Invalid request path"). Passing
+* `env` is harmless for providers whose base URL has no placeholders.
+*
+* `stream` is injectable so the env threading can be unit-tested without a live call.
+*/
+function createReviewStreamFn(stream = streamSimple) {
+	return (model, context, options) => stream(model, context, {
+		...options,
+		env: process.env
+	});
+}
 function defaultCreateAgent(params) {
 	return new Agent({
 		initialState: {
@@ -3969,7 +4043,8 @@ function defaultCreateAgent(params) {
 			tools: params.tools,
 			thinkingLevel: params.thinkingLevel
 		},
-		getApiKey: params.getApiKey
+		getApiKey: params.getApiKey,
+		streamFn: createReviewStreamFn()
 	});
 }
 async function findGitRoot(cwd) {
@@ -4447,7 +4522,7 @@ function resolveModel(modelString, baseUrl, maxTokens) {
 	const { provider, modelId } = splitModel(modelString);
 	if (provider === void 0 || modelId === void 0) throw new ReviewerError(`Invalid model format "${modelString}". Expected "provider/modelId" (e.g. "anthropic/claude-sonnet-4-5").`);
 	if (provider === "ollama") return buildOllamaModel(modelId, baseUrl || "http://localhost:11434/v1", maxTokens);
-	const model = getModel(provider, modelId);
+	const model = getBuiltinModel(provider, modelId);
 	if (!model) throw new ReviewerError(`Unknown model "${modelString}".`, { hint: `Check that "${provider}" is a valid provider and "${modelId}" is a registered model ID.` });
 	if (baseUrl || maxTokens > 0) return {
 		...model,
@@ -4712,7 +4787,10 @@ async function runReview(config, options) {
 		tokens: aggregated.tokens,
 		cost: aggregated.cost,
 		byModel: buildByModelUsage(aggregated),
-		skills: context.skills.map((s) => s.name),
+		skills: context.skills.map((s) => ({
+			name: s.name,
+			origin: s.origin
+		})),
 		sizeNotice
 	});
 	let outputText;
@@ -5616,7 +5694,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.9.4"
+		[semconv.ATTR_SERVICE_VERSION ?? "service.version"]: "0.9.6"
 	});
 	applyOtelExporterDefaults(process.env);
 	const sdk = new sdkNode.NodeSDK({ resource: resources.defaultResource().merge(serviceResource) });
@@ -5800,7 +5878,7 @@ function buildCiSpanAttrs(env) {
 	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);
+	const repositoryUrl = resolveProjectWebUrl(env);
 	if (repositoryUrl) attrs["vcs.repository.url.full"] = repositoryUrl;
 	return attrs;
 }
@@ -6088,7 +6166,7 @@ function boldCommentTitle(body) {
 */
 function buildCommentBody(body, commitSha, confidence) {
 	const confidenceLine = `_Confidence: ${confidence}._`;
-	const footer = `<sub>Reviewed by ${PRODUCT_LINK} v0.9.4 for commit ${commitSha}.</sub>`;
+	const footer = `<sub>Reviewed by ${PRODUCT_LINK} v0.9.6 for commit ${commitSha}.</sub>`;
 	return `${boldCommentTitle(body.trim())}\n\n${confidenceLine}\n\n---\n\n${footer}`;
 }
 function buildPayload(comment, body, refs, resolved) {
@@ -6661,6 +6739,99 @@ function createPlatform(config) {
 	return new GitLabPlatform(config);
 }
 //#endregion
+//#region src/skill-links.ts
+/**
+* Hosts that serve blobs at `/<repo>/blob/<ref>/<path>`. Everything else is
+* assumed to be GitLab (`/<repo>/-/blob/<ref>/<path>`), which covers both
+* gitlab.com and the self-hosted instances this tool mostly runs against.
+*/
+var GITHUB_HOSTS = new Set(["github.com", "www.github.com"]);
+/** Percent-encode each path segment while keeping the `/` separators intact. */
+function encodePath(path) {
+	return path.split("/").filter(Boolean).map((segment) => encodeURIComponent(segment)).join("/");
+}
+/**
+* Build a blob URL for `path` at `ref` inside the repository served at
+* `repoWebUrl`. An empty ref resolves to `HEAD`, which both GitHub and GitLab
+* accept, so a skill pinned to a remote's default branch still links.
+*/
+function blobUrl(repoWebUrl, ref, path) {
+	let parsed;
+	try {
+		parsed = new URL(repoWebUrl);
+	} catch {
+		return;
+	}
+	return `${`${parsed.origin}${parsed.pathname.replace(/\/+$/, "")}`}/${GITHUB_HOSTS.has(parsed.host) ? "blob" : "-/blob"}/${encodeURIComponent(ref || "HEAD")}/${encodePath(path)}`;
+}
+/**
+* Convert a git clone URL to the repository's web URL: drop any transport
+* marker, embedded credentials, and the `.git` suffix, and serve it over HTTPS.
+* Accepts `https://`, `ssh://`, `git+<transport>://`, and scp-style
+* (`git@host:group/repo.git`) forms. Returns `undefined` for anything it cannot
+* parse, so an exotic remote simply yields an unlinked skill.
+*/
+function gitRepoWebUrl(cloneUrl) {
+	let raw = cloneUrl.trim();
+	if (!raw) return void 0;
+	if (raw.startsWith("git+")) raw = raw.slice(4);
+	const scp = /^(?:[^@/]+@)?([^/:]+):(?!\/)(.+)$/.exec(raw);
+	if (scp) raw = `ssh://${scp[1]}/${scp[2]}`;
+	let parsed;
+	try {
+		parsed = new URL(raw);
+	} catch {
+		return;
+	}
+	if (!parsed.host) return void 0;
+	const path = parsed.pathname.replace(/\.git\/?$/, "").replace(/\/+$/, "");
+	if (!path || path === "/") return void 0;
+	return `https://${parsed.host}${path}`;
+}
+/**
+* Resolve a link to a skill's `SKILL.md`, or `undefined` when the source is not
+* reachable from a browser (a `file:` path, or an in-repo skill on a run with no
+* CI project URL). Built-in skills link to the published tag of this package, so
+* the link always shows the skill exactly as the run used it.
+*/
+function skillSourceUrl(origin, context = {}) {
+	switch (origin.kind) {
+		case "builtin": return blobUrl(PRODUCT_URL, "0.9.6", `skills/${origin.name}/SKILL.md`);
+		case "project":
+			if (!context.projectWebUrl || !context.commitSha) return void 0;
+			return blobUrl(context.projectWebUrl, context.commitSha, `${origin.path}/SKILL.md`);
+		case "npm": return `https://www.npmjs.com/package/${origin.packageName}`;
+		case "file": return;
+		case "git":
+		case "marketplace": {
+			const repoWebUrl = gitRepoWebUrl(origin.url);
+			if (!repoWebUrl) return void 0;
+			const path = origin.path ? `${origin.path}/SKILL.md` : "SKILL.md";
+			return blobUrl(repoWebUrl, origin.ref, path);
+		}
+	}
+}
+/**
+* The name a skill is shown under in the footer. Marketplace skills carry their
+* full selector (`<marketplace>:<plugin>/<skill>`) because a bare skill name says
+* nothing about which marketplace and plugin it came from — and that selector is
+* exactly what a developer would put in `CODE_REVIEW_SKILLS` to use it. Every
+* other source shows the skill's own name.
+*/
+function skillDisplayName(skill) {
+	return skill.origin.kind === "marketplace" ? `${skill.origin.marketplace}:${skill.origin.plugin}/${skill.name}` : skill.name;
+}
+/**
+* Render one skill for the summary footer: a link to its source when one can be
+* resolved, otherwise the bare name. The name keeps its code span either way so
+* linked and unlinked skills read the same.
+*/
+function formatSkillLink(skill, context = {}) {
+	const name = skillDisplayName(skill);
+	const url = skillSourceUrl(skill.origin, context);
+	return url ? `[\`${name}\`](${url})` : `\`${name}\``;
+}
+//#endregion
 //#region src/summary-carryover.ts
 var FINGERPRINT_MARKER_GLOBAL_RE = new RegExp(FINGERPRINT_MARKER_PATTERN, "gi");
 var RISK_RANK = {
@@ -7036,7 +7207,10 @@ async function run(config, bridges) {
 				const summaryBody = withCarriedOverFindings(parsed.summary, discussions, currentFingerprints);
 				const result = await platform.upsertSummary(summaryBody, discussions, {
 					costFooter: [formatUsageLine(usage), formatPerModelUsage(usage)].filter(Boolean).join("\n\n"),
-					skillsFooter: formatSkillsFooter(usage.skills),
+					skillsFooter: formatSkillsFooter(usage.skills, {
+						projectWebUrl: config.projectWebUrl,
+						commitSha: refs.head_sha
+					}),
 					reviewedCommitSha: refs.head_sha,
 					runId,
 					sizeNotice: usage.sizeNotice
@@ -7103,9 +7277,15 @@ function zeroReviewUsage(model, thinkingLevel) {
 		sizeNotice: { sizeSkippedFiles: [] }
 	};
 }
-function formatSkillsFooter(skills) {
+/**
+* The summary footer's skills line. Each skill links to its `SKILL.md` source so
+* a developer reading the review can check what the reviewer was told; skills
+* with no reachable source (a `file:` path, or an in-repo skill on a run without
+* CI project coordinates) stay as plain names.
+*/
+function formatSkillsFooter(skills, context = {}) {
 	if (skills.length === 0) return void 0;
-	return `Skills: ${skills.map((s) => `\`${s}\``).join(", ")}`;
+	return `Skills: ${skills.map((s) => formatSkillLink(s, context)).join(", ")}`;
 }
 function formatUsageLine(usage) {
 	const formatter = new Intl.NumberFormat("en-US");
@@ -7195,10 +7375,10 @@ async function main(argv = process.argv.slice(2)) {
 		return;
 	}
 	if (argv.includes("--version") || argv.includes("-v")) {
-		console.log("0.9.4");
+		console.log("0.9.6");
 		return;
 	}
-	process.stderr.write(`[code-review] @weareikko/code-review v0.9.4\n`);
+	process.stderr.write(`[code-review] @weareikko/code-review v0.9.6\n`);
 	assertNodeVersion();
 	applyCodeReviewEnvPrefix();
 	applyDefaultCacheRetention();
@@ -7219,6 +7399,6 @@ if (isDirectRun()) main().catch((error) => {
 	process.exitCode = 1;
 });
 //#endregion
-export { resolveNpmSkillDir as $, SUMMARY_MARKER as A, findExistingSummaryNoteId as B, normalizeSeverity as C, SUMMARY_HISTORY_ENTRY_START as D, SUMMARY_HISTORY_ENTRY_END as E, buildSummaryHistoryEntries as F, extractDiffHunkContext as G, stripSummaryMarker as H, extractReviewedCommitSha as I, normalizeBody as J, extractExistingFingerprints as K, extractSummaryHistoryEntries as L, buildReviewedCommitFooter as M, buildSizeNoticeBlock as N, SUMMARY_HISTORY_LIMIT as O, buildSummaryBody as P, parseSkillSpec as Q, findExistingReviewedCommitSha as R, traceDiagnosticPhase as S, SUMMARY_HISTORY_END as T, upsertSummaryNote as U, stripSummaryHistory as V, appendFingerprintMarkers as W, gitSkillCacheKey as X, sha256 as Y, loadNamedSkill as Z, DIAGNOSTIC_CHANNEL_PREFIX as _, main as a, diagnosticChannels as b, buildGeneratedComments as c, startOtelBridge as d, resolveSkillCacheDir as et, filterDiff as f, DIAGNOSTIC_CHANNEL_NAMES as g, parseReviewMarkdownWithWarnings as h, formatUsageLine as i, buildArchivedSummaryEntry as j, SUMMARY_HISTORY_START as k, buildPayload as l, parseReviewMarkdown as m, formatPerModelUsage as n, run as o, runReview as p, fingerprints as q, formatSkillsFooter as r, withHttpStamping as s, countPostedBySeverity as t, isOtelEnabled as u, createDiagnosticContext as v, toGitLabReviewSeverity as w, traceDiagnostic as x, createDiagnosticRunId as y, findExistingSummaryNote as z };
+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, filterDiff as _, main as a, resolveSkillCacheDir as at, parseReviewMarkdownWithWarnings as b, blobUrl as c, skillDisplayName as d, sha256 as et, skillSourceUrl as f, startOtelBridge as g, isOtelEnabled as h, formatUsageLine as i, resolveNpmSkillDir as it, SUMMARY_HISTORY_ENTRY_END as j, toGitLabReviewSeverity as k, formatSkillLink as l, buildPayload as m, formatPerModelUsage as n, loadNamedSkill as nt, run as o, buildGeneratedComments as p, stripSummaryMarker as q, formatSkillsFooter as r, parseSkillSpec as rt, withHttpStamping as s, countPostedBySeverity as t, gitSkillCacheKey as tt, gitRepoWebUrl as u, runReview as v, createDiagnosticRunId as w, DIAGNOSTIC_CHANNEL_NAMES as x, parseReviewMarkdown as y, buildSummaryBody as z };
 
-//# sourceMappingURL=cli-C3kNr0rX.js.map
+//# sourceMappingURL=cli-D6vNV3kA.js.map