@oh-my-pi/pi-coding-agent 12.18.3 → 12.19.0

package/src/web/scrapers/pubmed.ts

@@ -1,8 +1,12 @@
 /**
  * PubMed handler for web-fetch
  */
-import type { RenderResult, SpecialHandler } from "./types";
-import { finalizeOutput, loadPage } from "./types";
+import { buildResult, loadPage, type RenderResult, type SpecialHandler, tryParseJson } from "./types";
+
+const NCBI_HEADERS = {
+	Accept: "application/json, text/plain;q=0.9, */*;q=0.8",
+	"User-Agent": "CodingAgent/1.0 (web scraper)",
+};
 
 /**
  * Handle PubMed URLs - fetch article metadata, abstract, MeSH terms
@@ -39,14 +43,45 @@ export const handlePubMed: SpecialHandler = async (
 
 		const fetchedAt = new Date().toISOString();
 		const notes: string[] = [];
+		const buildFallback = (fallbackNotes: string[]) =>
+			buildResult(`# PubMed Article\n\n**PMID:** ${pmid}\n\n---\n\n## Abstract\n\nNo abstract available.\n`, {
+				url,
+				method: "pubmed",
+				fetchedAt,
+				notes: fallbackNotes,
+			});
+
+		const fetchWithRetry = async (requestUrl: string, acceptJson = true) => {
+			let response = await loadPage(requestUrl, {
+				timeout,
+				signal,
+				headers: {
+					...NCBI_HEADERS,
+					Accept: acceptJson ? "application/json" : "text/plain, */*;q=0.8",
+				},
+			});
+			if (!response.ok) {
+				response = await loadPage(requestUrl, {
+					timeout,
+					signal,
+					headers: {
+						...NCBI_HEADERS,
+						Accept: acceptJson ? "application/json" : "text/plain, */*;q=0.8",
+					},
+				});
+			}
+			return response;
+		};
 
 		// Fetch summary metadata
 		const summaryUrl = `https://eutils.ncbi.nlm.nih.gov/entrez/eutils/esummary.fcgi?db=pubmed&id=${pmid}&retmode=json`;
-		const summaryResult = await loadPage(summaryUrl, { timeout, signal });
+		const summaryResult = await fetchWithRetry(summaryUrl);
 
-		if (!summaryResult.ok) return null;
+		if (!summaryResult.ok) {
+			return buildFallback(["Failed to fetch PubMed summary metadata"]);
+		}
 
-		let summaryData: {
+		const summaryData = tryParseJson<{
 			result?: {
 				[pmid: string]: {
 					title?: string;
@@ -60,20 +95,19 @@ export const handlePubMed: SpecialHandler = async (
 					articleids?: Array<{ idtype: string; value: string }>;
 				};
 			};
-		};
-
-		try {
-			summaryData = JSON.parse(summaryResult.content);
-		} catch {
-			return null;
+		}>(summaryResult.content);
+		if (!summaryData) {
+			return buildFallback(["Failed to parse PubMed summary metadata"]);
 		}
 
 		const article = summaryData.result?.[pmid];
-		if (!article) return null;
+		if (!article) {
+			return buildFallback(["PubMed record unavailable from E-utilities summary endpoint"]);
+		}
 
 		// Fetch abstract
 		const abstractUrl = `https://eutils.ncbi.nlm.nih.gov/entrez/eutils/efetch.fcgi?db=pubmed&id=${pmid}&rettype=abstract&retmode=text`;
-		const abstractResult = await loadPage(abstractUrl, { timeout, signal });
+		const abstractResult = await fetchWithRetry(abstractUrl, false);
 
 		let abstractText = "";
 		if (abstractResult.ok) {
@@ -136,7 +170,11 @@ export const handlePubMed: SpecialHandler = async (
 		// Try to fetch MeSH terms
 		try {
 			const meshUrl = `https://eutils.ncbi.nlm.nih.gov/entrez/eutils/efetch.fcgi?db=pubmed&id=${pmid}&rettype=medline&retmode=text`;
-			const meshResult = await loadPage(meshUrl, { timeout: Math.min(timeout, 5), signal });
+			const meshResult = await loadPage(meshUrl, {
+				timeout: Math.min(timeout, 5),
+				signal,
+				headers: { ...NCBI_HEADERS, Accept: "text/plain, */*;q=0.8" },
+			});
 
 			if (meshResult.ok) {
 				const meshTerms: string[] = [];
@@ -160,17 +198,12 @@ export const handlePubMed: SpecialHandler = async (
 			// MeSH terms are optional
 		}
 
-		const output = finalizeOutput(md);
-		return {
+		return buildResult(md, {
 			url,
-			finalUrl: url,
-			contentType: "text/markdown",
 			method: "pubmed",
-			content: output.content,
 			fetchedAt,
-			truncated: output.truncated,
 			notes: notes.length > 0 ? notes : ["Fetched via NCBI E-utilities"],
-		};
+		});
 	} catch {
 		return null;
 	}

package/src/web/scrapers/pypi.ts

@@ -1,5 +1,4 @@
-import type { RenderResult, SpecialHandler } from "./types";
-import { finalizeOutput, formatCount, loadPage } from "./types";
+import { buildResult, formatNumber, loadPage, type RenderResult, type SpecialHandler, tryParseJson } from "./types";
 
 /**
  * Handle PyPI URLs via JSON API
@@ -35,13 +34,11 @@ export const handlePyPI: SpecialHandler = async (
 		// Parse download stats
 		let weeklyDownloads: number | null = null;
 		if (downloadsResult.ok) {
-			try {
-				const dlData = JSON.parse(downloadsResult.content) as { data?: { last_week?: number } };
-				weeklyDownloads = dlData.data?.last_week ?? null;
-			} catch {}
+			const dlData = tryParseJson<{ data?: { last_week?: number } }>(downloadsResult.content);
+			if (dlData) weeklyDownloads = dlData.data?.last_week ?? null;
 		}
 
-		let pkg: {
+		const pkg = tryParseJson<{
 			info: {
 				name: string;
 				version: string;
@@ -59,13 +56,8 @@ export const handlePyPI: SpecialHandler = async (
 			urls?: Array<{ filename: string; size: number; upload_time: string }>;
 			releases?: Record<string, unknown>;
 			requires_dist?: string[];
-		};
-
-		try {
-			pkg = JSON.parse(result.content);
-		} catch {
-			return null; // JSON parse failed
-		}
+		}>(result.content);
+		if (!pkg) return null;
 
 		const info = pkg.info;
 		let md = `# ${info.name}\n\n`;
@@ -76,7 +68,7 @@ export const handlePyPI: SpecialHandler = async (
 		md += "\n";
 
 		if (weeklyDownloads !== null) {
-			md += `**Weekly Downloads:** ${formatCount(weeklyDownloads)}\n`;
+			md += `**Weekly Downloads:** ${formatNumber(weeklyDownloads)}\n`;
 		}
 
 		md += "\n";
@@ -112,17 +104,7 @@ export const handlePyPI: SpecialHandler = async (
 			md += `\n---\n\n## Description\n\n${info.description}\n`;
 		}
 
-		const output = finalizeOutput(md);
-		return {
-			url,
-			finalUrl: url,
-			contentType: "text/markdown",
-			method: "pypi",
-			content: output.content,
-			fetchedAt,
-			truncated: output.truncated,
-			notes: ["Fetched via PyPI JSON API"],
-		};
+		return buildResult(md, { url, method: "pypi", fetchedAt, notes: ["Fetched via PyPI JSON API"] });
 	} catch {}
 
 	return null;

package/src/web/scrapers/rawg.ts

@@ -1,5 +1,11 @@
-import type { RenderResult, SpecialHandler } from "./types";
-import { finalizeOutput, htmlToBasicMarkdown, loadPage } from "./types";
+import {
+	buildResult,
+	htmlToBasicMarkdown,
+	loadPage,
+	type RenderResult,
+	type SpecialHandler,
+	tryParseJson,
+} from "./types";
 
 interface RawgPlatformEntry {
 	platform?: {
@@ -41,12 +47,8 @@ export const handleRawg: SpecialHandler = async (
 
 		if (!result.ok) return null;
 
-		let game: RawgGameResponse;
-		try {
-			game = JSON.parse(result.content);
-		} catch {
-			return null;
-		}
+		const game = tryParseJson<RawgGameResponse>(result.content);
+		if (!game) return null;
 
 		if (requiresApiKey(game)) return null;
 
@@ -72,17 +74,7 @@ export const handleRawg: SpecialHandler = async (
 			md += `## Description\n\n${description}\n`;
 		}
 
-		const output = finalizeOutput(md);
-		return {
-			url,
-			finalUrl: url,
-			contentType: "text/markdown",
-			method: "rawg",
-			content: output.content,
-			fetchedAt,
-			truncated: output.truncated,
-			notes: ["Fetched via RAWG API"],
-		};
+		return buildResult(md, { url, method: "rawg", fetchedAt, notes: ["Fetched via RAWG API"] });
 	} catch {}
 
 	return null;

package/src/web/scrapers/readthedocs.ts

@@ -2,8 +2,7 @@
  * Read the Docs handler for web-fetch
  */
 import { parse as parseHtml } from "node-html-parser";
-import type { RenderResult, SpecialHandler } from "./types";
-import { finalizeOutput, htmlToBasicMarkdown, loadPage } from "./types";
+import { buildResult, htmlToBasicMarkdown, loadPage, type RenderResult, type SpecialHandler } from "./types";
 
 export const handleReadTheDocs: SpecialHandler = async (
 	url: string,
@@ -110,16 +109,12 @@ export const handleReadTheDocs: SpecialHandler = async (
 		notes.push("Failed to extract content");
 	}
 
-	const { content: finalContent, truncated } = finalizeOutput(content);
-
-	return {
+	return buildResult(content, {
 		url,
 		finalUrl: result.finalUrl,
-		contentType: sourceUrl ? "text/plain" : "text/html",
 		method: "readthedocs",
-		content: finalContent,
 		fetchedAt,
-		truncated,
 		notes,
-	};
+		contentType: sourceUrl ? "text/plain" : "text/html",
+	});
 };

package/src/web/scrapers/reddit.ts

@@ -1,5 +1,4 @@
-import type { RenderResult, SpecialHandler } from "./types";
-import { finalizeOutput, loadPage } from "./types";
+import { buildResult, formatIsoDate, loadPage, type RenderResult, type SpecialHandler, tryParseJson } from "./types";
 
 interface RedditPost {
 	title: string;
@@ -44,7 +43,8 @@ export const handleReddit: SpecialHandler = async (
 		const result = await loadPage(jsonUrl, { timeout, signal });
 		if (!result.ok) return null;
 
-		const data = JSON.parse(result.content);
+		const data = tryParseJson<any>(result.content);
+		if (!data) return null;
 		let md = "";
 
 		// Handle different Reddit URL types
@@ -54,7 +54,7 @@ export const handleReddit: SpecialHandler = async (
 			if (postData) {
 				md = `# ${postData.title}\n\n`;
 				md += `**r/${postData.subreddit}** · u/${postData.author} · ${postData.score} points · ${postData.num_comments} comments\n`;
-				md += `*${new Date(postData.created_utc * 1000).toISOString().split("T")[0]}*\n\n`;
+				md += `*${formatIsoDate(postData.created_utc * 1000)}*\n\n`;
 
 				if (postData.is_self && postData.selftext) {
 					md += `---\n\n${postData.selftext}\n\n`;
@@ -87,17 +87,7 @@ export const handleReddit: SpecialHandler = async (
 
 		if (!md) return null;
 
-		const output = finalizeOutput(md);
-		return {
-			url,
-			finalUrl: url,
-			contentType: "text/markdown",
-			method: "reddit",
-			content: output.content,
-			fetchedAt,
-			truncated: output.truncated,
-			notes: ["Fetched via Reddit JSON API"],
-		};
+		return buildResult(md, { url, method: "reddit", fetchedAt, notes: ["Fetched via Reddit JSON API"] });
 	} catch {}
 
 	return null;

package/src/web/scrapers/repology.ts

@@ -1,5 +1,4 @@
-import type { RenderResult, SpecialHandler } from "./types";
-import { finalizeOutput, loadPage } from "./types";
+import { buildResult, loadPage, type RenderResult, type SpecialHandler, tryParseJson } from "./types";
 
 interface RepologyPackage {
 	repo: string;
@@ -122,18 +121,17 @@ export const handleRepology: SpecialHandler = async (
 		const apiUrl = `https://repology.org/api/v1/project/${encodeURIComponent(packageName)}`;
 		const result = await loadPage(apiUrl, {
 			timeout,
-			headers: { Accept: "application/json" },
+			headers: {
+				Accept: "application/json",
+				"User-Agent": "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36",
+			},
 			signal,
 		});
 
 		if (!result.ok) return null;
 
-		let packages: RepologyPackage[];
-		try {
-			packages = JSON.parse(result.content);
-		} catch {
-			return null;
-		}
+		const packages = tryParseJson<RepologyPackage[]>(result.content);
+		if (!packages) return null;
 
 		// Empty response means package not found
 		if (!Array.isArray(packages) || packages.length === 0) return null;
@@ -245,17 +243,7 @@ export const handleRepology: SpecialHandler = async (
 
 		md += `\n---\n\n[View on Repology](${url})\n`;
 
-		const output = finalizeOutput(md);
-		return {
-			url,
-			finalUrl: url,
-			contentType: "text/markdown",
-			method: "repology",
-			content: output.content,
-			fetchedAt,
-			truncated: output.truncated,
-			notes: ["Fetched via Repology API"],
-		};
+		return buildResult(md, { url, method: "repology", fetchedAt, notes: ["Fetched via Repology API"] });
 	} catch {}
 
 	return null;

package/src/web/scrapers/rfc.ts

@@ -1,5 +1,4 @@
-import type { RenderResult, SpecialHandler } from "./types";
-import { finalizeOutput, loadPage } from "./types";
+import { buildResult, loadPage, type RenderResult, type SpecialHandler, tryParseJson } from "./types";
 
 interface RfcMetadata {
 	doc_id: string;
@@ -118,12 +117,8 @@ export const handleRfc: SpecialHandler = async (
 
 		let metadata: RfcMetadata | null = null;
 		if (metaResult.ok) {
-			try {
-				metadata = JSON.parse(metaResult.content);
-				notes.push("Metadata from RFC Editor JSON API");
-			} catch {
-				// JSON parse failed, continue without metadata
-			}
+			metadata = tryParseJson<RfcMetadata>(metaResult.content);
+			if (metadata) notes.push("Metadata from RFC Editor JSON API");
 		}
 
 		// Build markdown output
@@ -192,17 +187,13 @@ export const handleRfc: SpecialHandler = async (
 		md += cleanRfcText(textResult.content);
 		md += "\n```\n";
 
-		const output = finalizeOutput(md);
-		return {
+		return buildResult(md, {
 			url,
 			finalUrl: `https://www.rfc-editor.org/rfc/rfc${rfcNumber}`,
-			contentType: "text/markdown",
 			method: "rfc",
-			content: output.content,
 			fetchedAt,
-			truncated: output.truncated,
 			notes: notes.length ? notes : ["Fetched from RFC Editor"],
-		};
+		});
 	} catch {}
 
 	return null;

package/src/web/scrapers/rubygems.ts

@@ -1,5 +1,4 @@
-import type { RenderResult, SpecialHandler } from "./types";
-import { finalizeOutput, formatCount, loadPage } from "./types";
+import { buildResult, formatNumber, loadPage, type RenderResult, type SpecialHandler, tryParseJson } from "./types";
 
 interface RubyGemsDependency {
 	name: string;
@@ -56,12 +55,8 @@ export const handleRubyGems: SpecialHandler = async (
 
 		if (!result.ok) return null;
 
-		let gem: RubyGemsResponse;
-		try {
-			gem = JSON.parse(result.content);
-		} catch {
-			return null;
-		}
+		const gem = tryParseJson<RubyGemsResponse>(result.content);
+		if (!gem) return null;
 
 		let md = `# ${gem.name}\n\n`;
 		if (gem.info) md += `${gem.info}\n\n`;
@@ -72,8 +67,8 @@ export const handleRubyGems: SpecialHandler = async (
 		md += "\n";
 
 		// Downloads
-		md += `**Total Downloads:** ${formatCount(gem.downloads)}`;
-		if (gem.version_downloads) md += ` · **Version Downloads:** ${formatCount(gem.version_downloads)}`;
+		md += `**Total Downloads:** ${formatNumber(gem.downloads)}`;
+		if (gem.version_downloads) md += ` · **Version Downloads:** ${formatNumber(gem.version_downloads)}`;
 		md += "\n\n";
 
 		// Links
@@ -100,17 +95,7 @@ export const handleRubyGems: SpecialHandler = async (
 			}
 		}
 
-		const output = finalizeOutput(md);
-		return {
-			url,
-			finalUrl: url,
-			contentType: "text/markdown",
-			method: "rubygems",
-			content: output.content,
-			fetchedAt,
-			truncated: output.truncated,
-			notes: ["Fetched via RubyGems API"],
-		};
+		return buildResult(md, { url, method: "rubygems", fetchedAt, notes: ["Fetched via RubyGems API"] });
 	} catch {}
 
 	return null;

package/src/web/scrapers/searchcode.ts

@@ -1,5 +1,5 @@
 import type { RenderResult, SpecialHandler } from "./types";
-import { finalizeOutput, formatCount, loadPage } from "./types";
+import { buildResult, formatNumber, loadPage, tryParseJson } from "./types";
 
 interface SearchcodeResult {
 	id?: number | string;
@@ -84,12 +84,8 @@ export const handleSearchcode: SpecialHandler = async (
 			const result = await loadPage(apiUrl, { timeout, signal, headers: { Accept: "application/json" } });
 			if (!result.ok) return null;
 
-			let data: SearchcodeResult;
-			try {
-				data = JSON.parse(result.content) as SearchcodeResult;
-			} catch {
-				return null;
-			}
+			const data = tryParseJson<SearchcodeResult>(result.content);
+			if (!data) return null;
 
 			const filename = data.filename || data.location || `Result ${id}`;
 			const lineNumbers = parseLineNumbers(data.lines);
@@ -116,17 +112,7 @@ export const handleSearchcode: SpecialHandler = async (
 				md += "\n\n_No snippet available._\n";
 			}
 
-			const output = finalizeOutput(md);
-			return {
-				url,
-				finalUrl: url,
-				contentType: "text/markdown",
-				method: "searchcode",
-				content: output.content,
-				fetchedAt,
-				truncated: output.truncated,
-				notes: ["Fetched via searchcode API"],
-			};
+			return buildResult(md, { url, method: "searchcode", fetchedAt, notes: ["Fetched via searchcode API"] });
 		}
 
 		const query = parsed.searchParams.get("q");
@@ -141,12 +127,8 @@ export const handleSearchcode: SpecialHandler = async (
 		const result = await loadPage(apiUrl, { timeout, signal, headers: { Accept: "application/json" } });
 		if (!result.ok) return null;
 
-		let data: SearchcodeSearchResponse;
-		try {
-			data = JSON.parse(result.content) as SearchcodeSearchResponse;
-		} catch {
-			return null;
-		}
+		const data = tryParseJson<SearchcodeSearchResponse>(result.content);
+		if (!data) return null;
 
 		const results = Array.isArray(data.results) ? data.results : [];
 		const total =
@@ -162,7 +144,7 @@ export const handleSearchcode: SpecialHandler = async (
 		md += `## Metadata\n\n`;
 		md += `**Query:** \`${query}\`\n`;
 		md += `**Page:** ${page}\n`;
-		if (total !== null) md += `**Total Results:** ${formatCount(total)}\n`;
+		if (total !== null) md += `**Total Results:** ${formatNumber(total)}\n`;
 		md += `**Result Count:** ${results.length}\n`;
 		if (typeof data.nextpage === "number") md += `**Next Page:** ${data.nextpage}\n`;
 
@@ -200,17 +182,7 @@ export const handleSearchcode: SpecialHandler = async (
 			}
 		}
 
-		const output = finalizeOutput(md);
-		return {
-			url,
-			finalUrl: url,
-			contentType: "text/markdown",
-			method: "searchcode",
-			content: output.content,
-			fetchedAt,
-			truncated: output.truncated,
-			notes: ["Fetched via searchcode API"],
-		};
+		return buildResult(md, { url, method: "searchcode", fetchedAt, notes: ["Fetched via searchcode API"] });
 	} catch {}
 
 	return null;

package/src/web/scrapers/sec-edgar.ts

@@ -1,5 +1,5 @@
 import type { RenderResult, SpecialHandler } from "./types";
-import { finalizeOutput, loadPage } from "./types";
+import { buildResult, loadPage, tryParseJson } from "./types";
 
 interface SecFiling {
 	accessionNumber: string;
@@ -184,12 +184,8 @@ export const handleSecEdgar: SpecialHandler = async (
 
 		if (!result.ok) return null;
 
-		let company: SecCompany;
-		try {
-			company = JSON.parse(result.content);
-		} catch {
-			return null;
-		}
+		const company = tryParseJson<SecCompany>(result.content);
+		if (!company) return null;
 
 		// Build markdown output
 		let md = `# ${company.name}\n\n`;
@@ -257,17 +253,7 @@ export const handleSecEdgar: SpecialHandler = async (
 		md += `- [SEC EDGAR Filings](https://www.sec.gov/cgi-bin/browse-edgar?action=getcompany&CIK=${cik}&type=&dateb=&owner=include&count=40)\n`;
 		md += `- [Company Search](https://www.sec.gov/cgi-bin/browse-edgar?company=${encodeURIComponent(company.name)}&CIK=&type=&owner=include&count=40&action=getcompany)\n`;
 
-		const output = finalizeOutput(md);
-		return {
-			url,
-			finalUrl: url,
-			contentType: "text/markdown",
-			method: "sec-edgar",
-			content: output.content,
-			fetchedAt,
-			truncated: output.truncated,
-			notes: ["Fetched via SEC EDGAR API"],
-		};
+		return buildResult(md, { url, method: "sec-edgar", fetchedAt, notes: ["Fetched via SEC EDGAR API"] });
 	} catch {}
 
 	return null;

package/src/web/scrapers/semantic-scholar.ts

@@ -1,5 +1,5 @@
 import type { SpecialHandler } from "./types";
-import { finalizeOutput, formatCount, loadPage } from "./types";
+import { buildResult, formatNumber, loadPage, tryParseJson } from "./types";
 
 interface SemanticScholarAuthor {
 	name: string;
@@ -48,16 +48,13 @@ export const handleSemanticScholar: SpecialHandler = async (url: string, timeout
 
 	const paperId = extractPaperId(url);
 	if (!paperId) {
-		return {
+		return buildResult("Failed to extract paper ID from Semantic Scholar URL", {
 			url,
-			finalUrl: url,
-			contentType: "text/plain",
 			method: "semantic-scholar",
-			content: "Failed to extract paper ID from Semantic Scholar URL",
 			fetchedAt: new Date().toISOString(),
-			truncated: false,
 			notes: ["Invalid URL format"],
-		};
+			contentType: "text/plain",
+		});
 	}
 
 	const fields = [
@@ -80,32 +77,26 @@ export const handleSemanticScholar: SpecialHandler = async (url: string, timeout
 	const { content, ok, finalUrl } = await loadPage(apiUrl, { timeout, signal });
 
 	if (!ok || !content) {
-		return {
+		return buildResult("Failed to fetch paper from Semantic Scholar API", {
 			url,
 			finalUrl: apiUrl,
-			contentType: "text/plain",
 			method: "semantic-scholar",
-			content: "Failed to fetch paper from Semantic Scholar API",
 			fetchedAt: new Date().toISOString(),
-			truncated: false,
 			notes: ["API request failed"],
-		};
+			contentType: "text/plain",
+		});
 	}
 
-	let paper: SemanticScholarPaper;
-	try {
-		paper = JSON.parse(content);
-	} catch {
-		return {
+	const paper = tryParseJson<SemanticScholarPaper>(content);
+	if (!paper) {
+		return buildResult("Failed to parse response from Semantic Scholar API", {
 			url,
 			finalUrl: apiUrl,
-			contentType: "text/plain",
 			method: "semantic-scholar",
-			content: "Failed to parse response from Semantic Scholar API",
 			fetchedAt: new Date().toISOString(),
-			truncated: false,
 			notes: ["JSON parse error"],
-		};
+			contentType: "text/plain",
+		});
 	}
 
 	const sections: string[] = [];
@@ -123,10 +114,10 @@ export const handleSemanticScholar: SpecialHandler = async (url: string, timeout
 	if (paper.year) metadata.push(`Year: ${paper.year}`);
 	if (paper.journal?.name) metadata.push(`Venue: ${paper.journal.name}`);
 	if (paper.citationCount !== undefined) {
-		metadata.push(`Citations: ${formatCount(paper.citationCount)}`);
+		metadata.push(`Citations: ${formatNumber(paper.citationCount)}`);
 	}
 	if (paper.referenceCount !== undefined) {
-		metadata.push(`References: ${formatCount(paper.referenceCount)}`);
+		metadata.push(`References: ${formatNumber(paper.referenceCount)}`);
 	}
 	if (metadata.length > 0) {
 		sections.push(metadata.join(" • "));
@@ -175,16 +166,5 @@ export const handleSemanticScholar: SpecialHandler = async (url: string, timeout
 	}
 
 	const fullContent = sections.join("\n");
-	const { content: finalContent, truncated } = finalizeOutput(fullContent);
-
-	return {
-		url,
-		finalUrl,
-		contentType: "text/markdown",
-		method: "semantic-scholar",
-		content: finalContent,
-		fetchedAt: new Date().toISOString(),
-		truncated,
-		notes: [],
-	};
+	return buildResult(fullContent, { url, finalUrl, method: "semantic-scholar", fetchedAt: new Date().toISOString() });
 };