From 855d73630bf6dd0c96229db0f4cc135b1867170d Mon Sep 17 00:00:00 2001
From: Bryan Helmkamp <bryan@brynary.com>
Date: Mon, 25 May 2026 13:39:17 -0400
Subject: [PATCH] refactor(web): remove unused RunRow + RUNS_LIST_GRID_TEMPLATE
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The grid-based RunRow was the only Children-tab consumer of the
runs-list module's row primitive. When Children adopted the full
RunsListView (table layout) in 4dfcbc0e0, RunRow became unused — the
re-export in runs.tsx was preserved for a release as a precaution, but
nothing imports it. Same for RUNS_LIST_GRID_TEMPLATE, which only the
grid RunRow needed.

Note: automation-runs.tsx still defines its own local RunRow with the
same name; that one is unaffected.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 .../components/runs-list/run-table-row.tsx    | 63 -------------------
 apps/fabro-web/app/routes/runs.tsx            |  1 -
 2 files changed, 64 deletions(-)

diff --git a/apps/fabro-web/app/components/runs-list/run-table-row.tsx b/apps/fabro-web/app/components/runs-list/run-table-row.tsx
index 79f11ec47..52efd1152 100644
--- a/apps/fabro-web/app/components/runs-list/run-table-row.tsx
+++ b/apps/fabro-web/app/components/runs-list/run-table-row.tsx
@@ -9,8 +9,6 @@ import { RowActionsMenu } from "./row-actions-menu";
 import { SelectionCheckbox } from "./selection-checkbox";
 import type { ToggleableColumn } from "./toggleable-column";
 
-export const RUNS_LIST_GRID_TEMPLATE = "auto 5rem auto 1fr auto auto 8rem auto";
-
 function listLifecycleStatusLabel(
   run: Pick<RunWithStatus, "status" | "statusLabel" | "lifecycleStatusLabel">,
 ): string | null {
@@ -21,67 +19,6 @@ function listLifecycleStatusLabel(
   return run.lifecycleStatusLabel;
 }
 
-export function RunRow({ run }: { run: RunWithStatus }) {
-  const lifecycleLabel = listLifecycleStatusLabel(run);
-  const statusDisplay = columnStatusDisplay[run.status];
-
-  return (
-    <div className="grid items-center rounded-md border border-line bg-panel/80 px-4 py-3 transition-all duration-200 hover:border-line-strong hover:bg-panel" style={{ gridColumn: "1 / -1", gridTemplateColumns: "subgrid" }}>
-      <Link to={`/runs/${run.id}`} className="contents">
-      <span className="flex items-center gap-2 pr-2">
-        <span className={`size-1.5 shrink-0 rounded-full ${statusDisplay.dot}`} aria-hidden="true" />
-        <span className={`font-mono text-xs ${statusDisplay.text}`}>{run.statusLabel}</span>
-      </span>
-
-      <span className="font-mono text-xs pr-2 text-fg-muted">
-        {run.elapsed}
-      </span>
-
-      <span className="truncate font-mono text-xs font-medium text-teal-500 pr-2">{run.repo}</span>
-
-      <span className="flex items-center gap-2 min-w-0">
-        <InlineMarkdown content={run.title} className="truncate text-sm text-fg-2" />
-        {lifecycleLabel != null && (
-          <span className="rounded-full border border-line px-1.5 py-0.5 font-mono text-[11px] uppercase tracking-wide text-fg-muted">
-            {lifecycleLabel}
-          </span>
-        )}
-        {run.comments != null && run.comments > 0 && (
-          <span className="inline-flex shrink-0 items-center gap-1 font-mono text-xs text-fg-muted">
-            <svg viewBox="0 0 16 16" fill="currentColor" className="size-3" aria-hidden="true">
-              <path d="M1 2.75C1 1.784 1.784 1 2.75 1h10.5c.966 0 1.75.784 1.75 1.75v7.5A1.75 1.75 0 0 1 13.25 12H9.06l-2.573 2.573A1.458 1.458 0 0 1 4 13.543V12H2.75A1.75 1.75 0 0 1 1 10.25Zm1.75-.25a.25.25 0 0 0-.25.25v7.5c0 .138.112.25.25.25h2a.75.75 0 0 1 .75.75v2.19l2.72-2.72a.749.749 0 0 1 .53-.22h4.5a.25.25 0 0 0 .25-.25v-7.5a.25.25 0 0 0-.25-.25Z" />
-            </svg>
-            {run.comments}
-          </span>
-        )}
-      </span>
-
-      <span className="truncate font-mono text-xs text-fg-3 pr-2">{run.workflow}</span>
-
-      <span
-        className="font-mono text-xs text-fg-muted pr-2"
-        title={run.createdAt}
-      >
-        {run.createdAt != null ? formatRelativeTime(run.createdAt) : ""}
-      </span>
-
-      <span className="flex items-center justify-end gap-2 pr-4 font-mono text-xs tabular-nums">
-        {run.additions != null && <span className="text-mint">+{run.additions.toLocaleString()}</span>}
-        {run.deletions != null && <span className="text-coral">-{run.deletions.toLocaleString()}</span>}
-      </span>
-      </Link>
-
-      <span className="inline-flex items-center justify-end gap-1.5 font-mono text-xs text-fg-muted">
-        {run.pullRequestUrl && run.number != null && (
-          <PullRequestChip number={run.number} url={run.pullRequestUrl}>
-            {run.checks != null && <span className={`size-1.5 rounded-full ${ciConfig[deriveCiStatus(run.checks)].dot}`} />}
-          </PullRequestChip>
-        )}
-      </span>
-    </div>
-  );
-}
-
 export function RunTableRow({
   run,
   hiddenColumns,
diff --git a/apps/fabro-web/app/routes/runs.tsx b/apps/fabro-web/app/routes/runs.tsx
index f34af3c37..d3c6e159b 100644
--- a/apps/fabro-web/app/routes/runs.tsx
+++ b/apps/fabro-web/app/routes/runs.tsx
@@ -73,7 +73,6 @@ export {
   RUNS_PREFERENCES_STORAGE_KEY,
 } from "../components/runs-list/preferences";
 export { summarizeBatchLifecycleAction } from "../components/runs-list/bulk-action-toolbar";
-export { RUNS_LIST_GRID_TEMPLATE, RunRow } from "../components/runs-list/run-table-row";
 
 export function meta({}: any) {
   return [{ title: "Runs — Fabro" }];