refactor(release): extract shared fmtPct/fmtCount to format.ts

Address Cursor Bugbot review — deduplicate the percentage and count
formatting helpers into a shared module imported by both list and view.

Author: BYK

src/commands/release/format.ts

@@ -0,0 +1,32 @@
+/**
+ * Shared formatting utilities for release commands.
+ *
+ * Small helpers used by both `list.ts` and `view.ts` to format
+ * health/adoption metrics consistently.
+ */
+
+/**
+ * Format a percentage value with one decimal place, or "—" when absent.
+ *
+ * @example fmtPct(42.3) // "42.3%"
+ * @example fmtPct(null) // "—"
+ */
+export function fmtPct(value: number | null | undefined): string {
+  if (value === null || value === undefined) {
+    return "—";
+  }
+  return `${value.toFixed(1)}%`;
+}
+
+/**
+ * Format an integer count with thousands separators, or "—" when absent.
+ *
+ * @example fmtCount(52000) // "52,000"
+ * @example fmtCount(null)  // "—"
+ */
+export function fmtCount(value: number | null | undefined): string {
+  if (value === null || value === undefined) {
+    return "—";
+  }
+  return value.toLocaleString("en-US");
+}

src/commands/release/list.ts

@@ -15,6 +15,7 @@ import {
   type OrgListCommandDocs,
 } from "../../lib/list-command.js";
 import type { OrgListConfig } from "../../lib/org-list.js";
+import { fmtPct } from "./format.js";
 
 export const PAGINATION_KEY = "release-list";
 
@@ -30,14 +31,6 @@ function getHealthData(release: OrgReleaseResponse) {
   return release.projects?.find((p) => p.healthData?.hasHealthData)?.healthData;
 }
 
-/** Format a percentage value with one decimal, or "—" when absent. */
-function fmtPct(value: number | null | undefined): string {
-  if (value === null || value === undefined) {
-    return "—";
-  }
-  return `${value.toFixed(1)}%`;
-}
-
 const RELEASE_COLUMNS: Column<ReleaseWithOrg>[] = [
   { header: "ORG", value: (r) => r.orgSlug || "" },
   {

src/commands/release/view.ts

@@ -27,16 +27,9 @@ import {
   FRESH_FLAG,
 } from "../../lib/list-command.js";
 import { resolveOrg } from "../../lib/resolve-target.js";
+import { fmtCount, fmtPct } from "./format.js";
 import { parseReleaseArg } from "./parse.js";
 
-/** Format a percentage with one decimal, colorized by threshold. */
-function fmtPct(value: number | null | undefined): string {
-  if (value === null || value === undefined) {
-    return "—";
-  }
-  return `${value.toFixed(1)}%`;
-}
-
 /** Format a crash-free rate with color coding (green ≥ 99, yellow ≥ 95, red < 95). */
 function fmtCrashFree(value: number | null | undefined): string {
   if (value === null || value === undefined) {
@@ -52,14 +45,6 @@ function fmtCrashFree(value: number | null | undefined): string {
   return colorTag("red", formatted);
 }
 
-/** Format a count with thousands separators, or "—" when absent. */
-function fmtCount(value: number | null | undefined): string {
-  if (value === null || value === undefined) {
-    return "—";
-  }
-  return value.toLocaleString("en-US");
-}
-
 /**
  * Build a markdown table of per-project health data.
  *