feat(control-panel): add repo owner registration workspace

apps/gittensory-ui/src/components/site/app-panels/owner-panel.tsx

@@ -1,162 +1,270 @@
 import { useMemo, useState } from "react";
 
-import { DiffBlock, StatusPill, type Status } from "@/components/site/control-primitives";
+import {
+  BoundaryBadge,
+  DiffBlock,
+  StatusPill,
+  type Status,
+} from "@/components/site/control-primitives";
 import { StateBoundary } from "@/components/site/state-views";
 import { Input } from "@/components/ui/input";
 import { useApiResource } from "@/lib/api/use-api-resource";
+import {
+  buildRegistrationWorkspaceView,
+  splitRepoFullName,
+  type GittensorConfigRecommendationPayload,
+  type RegistrationReadinessPayload,
+  type RegistrationWorkspaceView,
+  type WorkspaceLaneStatus,
+} from "@/lib/registration-workspace";
 
-const STATUS_MAP: Record<string, Status> = { ok: "ready", warn: "warn", blocked: "blocked" };
-
-type RegistrationReadiness = {
-  repoFullName: string;
-  ready: boolean;
-  recommendedRegistrationMode: string;
-  issuePolicy: string;
-  blockers: string[];
-  warnings: string[];
+const LANE_STATUS_MAP: Record<WorkspaceLaneStatus, Status> = {
+  ready: "ready",
+  warn: "warn",
+  blocked: "blocked",
+  info: "info",
 };
 
-type ConfigRecommendation = {
-  current: Record<string, unknown> | null;
-  recommended: Record<string, unknown>;
-  reasons: string[];
-  warnings: string[];
+const FRESHNESS_STATUS_MAP: Record<RegistrationWorkspaceView["freshness"]["status"], Status> = {
+  complete: "ready",
+  degraded: "degraded",
+  stale: "stale",
+  unknown: "info",
 };
 
 export function OwnerPanel() {
   const [repo, setRepo] = useState("entrius/gittensor");
-  const [owner, name] = repo.split("/");
-  const repoPath =
-    owner && name
-      ? `/v1/repos/${encodeURIComponent(owner)}/${encodeURIComponent(name)}`
-      : "/v1/repos/entrius/gittensor";
-  const readiness = useApiResource<RegistrationReadiness>(
-    `${repoPath}/registration-readiness`,
+  const parts = splitRepoFullName(repo.trim());
+  const repoPath = parts
+    ? `/v1/repos/${encodeURIComponent(parts.owner)}/${encodeURIComponent(parts.repo)}`
+    : null;
+  const readiness = useApiResource<RegistrationReadinessPayload>(
+    `${repoPath ?? "/v1/repos/__invalid__/__invalid__"}/registration-readiness`,
     "Registration readiness",
+    undefined,
+    { enabled: Boolean(repoPath) },
   );
-  const config = useApiResource<ConfigRecommendation>(
-    `${repoPath}/gittensor-config-recommendation`,
+  const config = useApiResource<GittensorConfigRecommendationPayload>(
+    `${repoPath ?? "/v1/repos/__invalid__/__invalid__"}/gittensor-config-recommendation`,
     "Config recommendation",
+    undefined,
+    { enabled: Boolean(repoPath) },
   );
-  const liveSteps = useMemo(() => {
+
+  const workspace = useMemo(() => {
     if (readiness.status !== "ready") return null;
-    const steps = [
-      {
-        id: "mode",
-        title: "Registration mode",
-        detail: readiness.data.recommendedRegistrationMode,
-        status: readiness.data.ready ? "ok" : "warn",
-      },
-      {
-        id: "issue-policy",
-        title: "Issue policy",
-        detail: readiness.data.issuePolicy,
-        status: readiness.data.ready ? "ok" : "warn",
-      },
-      ...readiness.data.blockers.map((detail, i) => ({
-        id: `blocker-${i}`,
-        title: "Readiness blocker",
-        detail,
-        status: "blocked",
-      })),
-      ...readiness.data.warnings.map((detail, i) => ({
-        id: `warning-${i}`,
-        title: "Readiness warning",
-        detail,
-        status: "warn",
-      })),
-    ];
-    return steps.length > 0 ? steps : null;
-  }, [readiness]);
-  const steps = liveSteps ?? [];
-  const removed = config.status === "ready" ? recordLines(config.data.current ?? {}) : [];
-  const added = config.status === "ready" ? recordLines(config.data.recommended) : [];
+    const configPayload = config.status === "ready" ? config.data : null;
+    return buildRegistrationWorkspaceView(readiness.data, configPayload);
+  }, [readiness, config]);
+
   const refresh = () => {
     void readiness.reload();
     void config.reload();
   };
 
+  const invalidRepo = repo.trim().length > 0 && !parts;
+
   return (
-    <StateBoundary
-      isLoading={readiness.status === "loading" || config.status === "loading"}
-      isError={readiness.status === "error" && config.status === "error"}
-      isEmpty={steps.length === 0 && config.status !== "ready"}
-      onRetry={refresh}
-      onRefresh={refresh}
-      loadingTitle="Loading owner readiness…"
-      emptyTitle="No readiness checks yet"
-      emptyDescription="Registration readiness and config recommendations appear after repository analysis runs."
-    >
-      <div className="space-y-6">
-        <section className="rounded-token border-hairline bg-card p-5">
-          <div className="flex flex-wrap items-end justify-between gap-3">
-            <div>
-              <h2 className="font-display text-token-lg font-semibold">Registration readiness</h2>
-              <p className="mt-1 text-token-xs text-muted-foreground">
-                Live read-only API check from cached repository intelligence.
-              </p>
-            </div>
-            <div className="w-full sm:w-56">
-              <label className="font-mono text-token-2xs uppercase tracking-wider text-muted-foreground">
-                Repo
-              </label>
-              <Input
-                value={repo}
-                onChange={(e) => setRepo(e.target.value)}
-                className="mt-1 font-mono text-token-xs"
-              />
-            </div>
+    <div className="space-y-6">
+      <section className="rounded-token border-hairline bg-card p-5">
+        <div className="flex flex-wrap items-end justify-between gap-3">
+          <div>
+            <h2 className="font-display text-token-lg font-semibold">Registration workspace</h2>
+            <p className="mt-1 text-token-xs text-muted-foreground">
+              Readiness report with lane tradeoffs — not raw Gittensor telemetry.
+            </p>
           </div>
-          {readiness.status === "error" && (
-            <p className="mt-3 text-token-2xs text-muted-foreground">
-              Live readiness failed ({readiness.error}).
+          <div className="w-full sm:w-64">
+            <label className="font-mono text-token-2xs uppercase tracking-wider text-muted-foreground">
+              Repository
+            </label>
+            <Input
+              value={repo}
+              onChange={(e) => setRepo(e.target.value)}
+              className="mt-1 font-mono text-token-xs"
+              placeholder="owner/repo"
+            />
+            {invalidRepo ? (
+              <p className="mt-1 text-token-2xs text-danger">Use a valid owner/repo slug.</p>
+            ) : null}
+          </div>
+        </div>
+      </section>
+
+      <StateBoundary
+        isLoading={
+          Boolean(repoPath) && (readiness.status === "loading" || config.status === "loading")
+        }
+        isError={Boolean(repoPath) && readiness.status === "error" && config.status === "error"}
+        isEmpty={Boolean(repoPath) && readiness.status === "ready" && !workspace}
+        onRetry={refresh}
+        onRefresh={refresh}
+        loadingTitle="Loading registration workspace…"
+        emptyTitle="No readiness report yet"
+        emptyDescription="Registration readiness appears after repository intelligence has been generated for this repo."
+      >
+        {workspace ? (
+          <RegistrationWorkspace
+            workspace={workspace}
+            generatedLabel={formatGeneratedAt(workspace.generatedAt)}
+          />
+        ) : null}
+        {repoPath && readiness.status === "error" ? (
+          <p className="text-token-2xs text-muted-foreground">
+            Readiness failed ({readiness.error}).
+          </p>
+        ) : null}
+      </StateBoundary>
+    </div>
+  );
+}
+
+function RegistrationWorkspace({
+  workspace,
+  generatedLabel,
+}: {
+  workspace: RegistrationWorkspaceView;
+  generatedLabel: string;
+}) {
+  return (
+    <div className="space-y-6">
+      <section className="rounded-token border-hairline bg-card p-5">
+        <div className="flex flex-wrap items-start justify-between gap-3">
+          <div className="space-y-2">
+            <div className="flex flex-wrap items-center gap-2">
+              <h3 className="font-display text-token-md font-semibold">{workspace.repoFullName}</h3>
+              <StatusPill status={LANE_STATUS_MAP[workspace.summary.status]}>
+                {workspace.summary.ready ? "Ready" : "Not ready"}
+              </StatusPill>
+              <StatusPill status={FRESHNESS_STATUS_MAP[workspace.freshness.status]}>
+                {workspace.freshness.status}
+              </StatusPill>
+              <BoundaryBadge boundary="private-api" />
+            </div>
+            <p className="text-token-xs text-muted-foreground">{workspace.summary.headline}</p>
+            <p className="font-mono text-token-2xs text-muted-foreground">
+              Generated {generatedLabel}
             </p>
-          )}
-          <ul className="mt-4 divide-hairline">
-            {steps.map((s) => (
-              <li
-                key={s.id}
-                className="flex items-start justify-between gap-3 py-3 transition-colors hover:bg-muted/30"
-              >
-                <div>
-                  <div className="font-medium">{s.title}</div>
-                  <div className="mt-0.5 text-token-xs text-muted-foreground">{s.detail}</div>
-                </div>
-                <StatusPill status={STATUS_MAP[s.status]}>{s.status}</StatusPill>
-              </li>
+          </div>
+        </div>
+        <p className="mt-3 rounded-token border border-mint/20 bg-mint/5 px-3 py-2 text-token-xs text-muted-foreground">
+          {workspace.advisoryBanner}
+        </p>
+        {workspace.freshness.warnings.length > 0 ? (
+          <ul className="mt-3 space-y-1 text-token-2xs text-warning">
+            {workspace.freshness.warnings.map((warning) => (
+              <li key={warning}>{warning}</li>
             ))}
           </ul>
-        </section>
+        ) : null}
+        <dl className="mt-4 grid gap-3 sm:grid-cols-2">
+          <Metric label="Recommended mode" value={workspace.summary.recommendedMode} />
+          <Metric label="Issue policy" value={workspace.summary.issuePolicy} />
+        </dl>
+      </section>
 
-        <section className="rounded-token border-hairline bg-card p-5">
-          <h2 className="font-display text-token-lg font-semibold">Suggested .gittensor.yml</h2>
-          <p className="mt-1 text-token-xs text-muted-foreground">
-            Diff against the current configuration. Apply via PR when ready.
-          </p>
-          {config.status === "error" && (
-            <p className="mt-2 text-token-2xs text-muted-foreground">
-              Live recommendation failed ({config.error}).
+      <section className="grid gap-4 lg:grid-cols-2">
+        <WorkspaceSectionCard section={workspace.lanes.directPr} />
+        <WorkspaceSectionCard section={workspace.lanes.issueDiscovery} />
+        <WorkspaceSectionCard section={workspace.lanes.maintainerEconomics} />
+        <WorkspaceSectionCard section={workspace.lanes.minerGuidance} />
+      </section>
+
+      <section className="space-y-4">
+        <h3 className="font-display text-token-md font-semibold">Operations & policy</h3>
+        <div className="grid gap-4 lg:grid-cols-2">
+          {workspace.operations.map((section) => (
+            <WorkspaceSectionCard key={section.id} section={section} />
+          ))}
+        </div>
+        {workspace.policyWarnings.length > 0 ? (
+          <div className="rounded-token border-hairline bg-card p-5">
+            <h4 className="font-medium">Focus policy warnings</h4>
+            <ul className="mt-3 space-y-3">
+              {workspace.policyWarnings.map((warning) => (
+                <li key={`${warning.title}-${warning.detail}`} className="text-token-xs">
+                  <div className="flex items-center gap-2">
+                    <span className="font-medium">{warning.title}</span>
+                    <StatusPill status={warning.severity === "critical" ? "blocked" : "warn"}>
+                      {warning.severity}
+                    </StatusPill>
+                  </div>
+                  <p className="mt-1 text-muted-foreground">{warning.detail}</p>
+                  <p className="mt-1 text-muted-foreground">{warning.action}</p>
+                </li>
+              ))}
+            </ul>
+          </div>
+        ) : null}
+      </section>
+
+      {workspace.config ? (
+        <section className="rounded-token border-hairline bg-card p-5 space-y-4">
+          <div>
+            <h3 className="font-display text-token-md font-semibold">Suggested Gittensor config</h3>
+            <p className="mt-1 text-token-xs text-muted-foreground">
+              Tradeoffs below separate maintainer economics from contributor lanes. Apply via PR
+              when ready.
             </p>
-          )}
-          <div className="mt-3">
-            <DiffBlock removed={removed} added={added} />
           </div>
+          {workspace.config.tradeoffs.length > 0 ? (
+            <div>
+              <h4 className="text-token-xs font-medium uppercase tracking-wider text-muted-foreground">
+                Tradeoffs
+              </h4>
+              <ul className="mt-2 list-disc space-y-1 pl-5 text-token-xs text-muted-foreground">
+                {workspace.config.tradeoffs.map((entry) => (
+                  <li key={entry}>{entry}</li>
+                ))}
+              </ul>
+            </div>
+          ) : null}
+          <DiffBlock
+            removed={workspace.config.currentLines}
+            added={workspace.config.recommendedLines}
+          />
         </section>
+      ) : null}
+    </div>
+  );
+}
+
+function WorkspaceSectionCard({
+  section,
+}: {
+  section: RegistrationWorkspaceView["lanes"]["directPr"];
+}) {
+  return (
+    <article className="rounded-token border-hairline bg-card p-5">
+      <div className="flex items-start justify-between gap-2">
+        <h4 className="font-medium">{section.title}</h4>
+        <StatusPill status={LANE_STATUS_MAP[section.status]}>{section.status}</StatusPill>
       </div>
-    </StateBoundary>
+      <p className="mt-2 text-token-xs text-muted-foreground">{section.summary}</p>
+      {section.bullets.length > 0 ? (
+        <ul className="mt-3 list-disc space-y-1 pl-5 text-token-2xs text-muted-foreground">
+          {section.bullets.map((bullet) => (
+            <li key={bullet}>{bullet}</li>
+          ))}
+        </ul>
+      ) : null}
+    </article>
   );
 }
 
-function recordLines(record: Record<string, unknown>) {
-  const entries = Object.entries(record);
-  if (entries.length === 0) return ["{}"];
-  return entries.slice(0, 8).map(([key, value]) => `${key}: ${formatValue(value)}`);
+function Metric({ label, value }: { label: string; value: string }) {
+  return (
+    <div className="rounded-token border-hairline bg-muted/20 px-3 py-2">
+      <dt className="font-mono text-token-2xs uppercase tracking-wider text-muted-foreground">
+        {label}
+      </dt>
+      <dd className="mt-1 font-mono text-token-xs">{value}</dd>
+    </div>
+  );
 }
 
-function formatValue(value: unknown) {
-  if (value === null) return "null";
-  if (typeof value === "string" || typeof value === "number" || typeof value === "boolean") {
-    return String(value);
-  }
-  return JSON.stringify(value);
+function formatGeneratedAt(value: string) {
+  const parsed = Date.parse(value);
+  if (!Number.isFinite(parsed)) return value;
+  return new Date(parsed).toLocaleString();
 }