fabro(01KTM9H228G0Z1ATDMZ10GGS4W): simplify_gpt (succeeded)

Fabro-Run: 01KTM9H228G0Z1ATDMZ10GGS4W
Fabro-Completed: 7
Fabro-Checkpoint: e9691ee0b4

⚒️ Generated with [Fabro](https://fabro.sh)

apps/fabro-web/app/components/runs-list/run-table-row.tsx

@@ -12,6 +12,18 @@ import { RowActionsMenu } from "./row-actions-menu";
 import { SelectionCheckbox } from "./selection-checkbox";
 import type { ToggleableColumn } from "./toggleable-column";
 
+function CreatedByCell({ createdBy }: { createdBy: RunWithStatus["createdBy"] }) {
+  const display = principalDisplay(createdBy);
+
+  return (
+    <td className="relative z-10 w-8 whitespace-nowrap px-3 py-2.5">
+      <Tooltip label={display.label}>
+        <span aria-label={`Created by ${display.label}`}>{display.glyph}</span>
+      </Tooltip>
+    </td>
+  );
+}
+
 function listLifecycleStatusLabel(
   run: Pick<RunWithStatus, "status" | "statusLabel" | "lifecycleStatusLabel">,
 ): string | null {
@@ -35,7 +47,6 @@ export function RunTableRow({
 }) {
   const lifecycleLabel = listLifecycleStatusLabel(run);
   const statusDisplay = columnStatusDisplay[run.status];
-  const createdByDisplay = principalDisplay(run.createdBy);
   const show = (col: ToggleableColumn) => !hiddenColumns.has(col);
 
   return (
@@ -53,13 +64,7 @@ export function RunTableRow({
           <span className={`font-mono text-xs ${statusDisplay.text}`}>{run.statusLabel}</span>
         </span>
       </td>
-      {show("created_by") && (
-        <td className="relative z-10 w-8 whitespace-nowrap px-3 py-2.5">
-          <Tooltip label={createdByDisplay.label}>
-            <span aria-label={`Created by ${createdByDisplay.label}`}>{createdByDisplay.glyph}</span>
-          </Tooltip>
-        </td>
-      )}
+      {show("created_by") && <CreatedByCell createdBy={run.createdBy} />}
       {show("repo") && (
         <td className="whitespace-nowrap px-3 py-2.5 font-mono text-xs font-medium text-teal-500">
           {run.repo}

apps/fabro-web/app/lib/run-actions.test.ts

@@ -24,6 +24,7 @@ import {
   unarchiveRuns,
 } from "./run-actions";
 import { generatedAxios } from "./api-client";
+import { TEST_PRINCIPAL } from "./test-fixtures";
 
 type StubResponseInit = {
   status: number;
@@ -37,8 +38,6 @@ type CapturedRequest = {
   data?: unknown;
 };
 
-import { TEST_PRINCIPAL } from "./test-fixtures";
-
 const originalAdapter = generatedAxios.defaults.adapter;
 
 function makeRun(status: RunStatus, archived = false): Run {

apps/fabro-web/app/lib/test-fixtures.ts

@@ -1,8 +1,5 @@
 import type { Principal } from "@qltysh/fabro-api-client";
 
-/**
- * Shared frontend test fixture mirroring `fabro_types::test_support::test_principal()`.
- */
 export const TEST_PRINCIPAL: Principal = {
   kind:        "user",
   identity:    { issuer: "fabro:test", subject: "test-user" },