From 49e39c3a799c58a6883fa786dc797177c25b7b40 Mon Sep 17 00:00:00 2001
From: BigSimmo <87357024+BigSimmo@users.noreply.github.com>
Date: Fri, 3 Jul 2026 04:11:34 +0800
Subject: [PATCH 1/3] fix(review): address PR #209 third-pass Codex findings
 (4x P2)

- registry detail hook + [slug] API now carry governance; the loader
  reconciles the verified badge from the authoritative validation_status
  so a reviewed/downgraded record no longer shows the stale fixture
  verification state.
- DocumentSearchResultsPanel takes recordStatus and renders a loading/
  sign-in/error notice for Services/Forms record matches, so the main
  dashboard no longer silently shows zero cards when the registry fails.
- seed-registry-records preserves existing governance columns
  (source_status/validation_status/last_reviewed_at/review_due_at) on
  reseed, so a fixture-copy reseed can't downgrade reviewed rows.
- the 'apply default mode scope to shortcut searches' finding is moot:
  the mode-default scope was removed entirely in the prior commit, so
  executeSearch and shortcut searches are consistently unscoped.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
---
 scripts/seed-registry-records.ts              | 33 +++++++++++++-
 src/app/api/registry/records/[slug]/route.ts  |  9 +++-
 src/components/ClinicalDashboard.tsx          |  1 +
 .../document-search-results.tsx               | 43 ++++++++++++++++++-
 src/components/registry-record-loader.tsx     | 18 +++++++-
 src/lib/use-registry-records.ts               | 30 ++++++++++---
 6 files changed, 122 insertions(+), 12 deletions(-)

diff --git a/scripts/seed-registry-records.ts b/scripts/seed-registry-records.ts
index 6375ae04d9..8b0ee4abfb 100644
--- a/scripts/seed-registry-records.ts
+++ b/scripts/seed-registry-records.ts
@@ -97,7 +97,38 @@ async function main() {
   }
 
   const supabase = await loadAdminClient();
-  const { error } = await supabase.from("clinical_registry_records").upsert(rows, { onConflict: "owner_id,kind,slug" });
+
+  // Preserve governance that was reviewed after seeding: a reseed for fixture
+  // copy changes must not downgrade source_status / validation_status /
+  // last_reviewed_at / review_due_at back to the fixture-derived defaults.
+  const { data: existing, error: existingError } = await supabase
+    .from("clinical_registry_records")
+    .select("kind, slug, source_status, validation_status, last_reviewed_at, review_due_at")
+    .eq("owner_id", args.ownerId);
+  if (existingError) {
+    throw new Error(`Could not read existing governance: ${existingError.message}`);
+  }
+  const governanceByKey = new Map((existing ?? []).map((row) => [`${row.kind}:${row.slug}`, row] as const));
+  let preserved = 0;
+  const upsertRows = rows.map((row) => {
+    const prior = governanceByKey.get(`${row.kind}:${row.slug}`);
+    if (!prior) return row;
+    preserved += 1;
+    return {
+      ...row,
+      source_status: prior.source_status,
+      validation_status: prior.validation_status,
+      last_reviewed_at: prior.last_reviewed_at,
+      review_due_at: prior.review_due_at,
+    };
+  });
+  if (preserved > 0) {
+    console.log(`[registry:seed] Preserving reviewed governance on ${preserved} existing record(s).`);
+  }
+
+  const { error } = await supabase
+    .from("clinical_registry_records")
+    .upsert(upsertRows, { onConflict: "owner_id,kind,slug" });
   if (error) {
     throw new Error(`Upsert failed: ${error.message}`);
   }
diff --git a/src/app/api/registry/records/[slug]/route.ts b/src/app/api/registry/records/[slug]/route.ts
index 18ef828948..a63d82ff0b 100644
--- a/src/app/api/registry/records/[slug]/route.ts
+++ b/src/app/api/registry/records/[slug]/route.ts
@@ -6,6 +6,7 @@ import { isDemoMode, isLocalNoAuthMode } from "@/lib/env";
 import { jsonError } from "@/lib/http";
 import { getFormRecord } from "@/lib/forms";
 import {
+  deriveGovernanceColumns,
   normalizeRegistrySlug,
   rowGovernance,
   rowToServiceRecord,
@@ -42,7 +43,13 @@ export async function GET(request: Request, context: { params: Promise<{ slug: s
     if (isDemoMode()) {
       const record = kind === "form" ? getFormRecord(normalizedSlug) : getServiceRecord(normalizedSlug);
       if (!record) return notFoundResponse(normalizedSlug);
-      return registryResponse({ record, linkedDocuments: [], demoMode: true });
+      const derived = deriveGovernanceColumns(record);
+      return registryResponse({
+        record,
+        governance: { sourceStatus: derived.source_status, validationStatus: derived.validation_status },
+        linkedDocuments: [],
+        demoMode: true,
+      });
     }
 
     const supabase = createAdminClient();
diff --git a/src/components/ClinicalDashboard.tsx b/src/components/ClinicalDashboard.tsx
index 183755eff8..626508b726 100644
--- a/src/components/ClinicalDashboard.tsx
+++ b/src/components/ClinicalDashboard.tsx
@@ -5589,6 +5589,7 @@ export function ClinicalDashboard({
                       matches={documentMatches}
                       recordMatches={recordSearchMatches}
                       recordMode={recordSearchMode}
+                      recordStatus={registryRecords.status}
                       showRecordMatches={searchMode === "services" || searchMode === "forms"}
                       query={query}
                       loading={loading}
diff --git a/src/components/clinical-dashboard/document-search-results.tsx b/src/components/clinical-dashboard/document-search-results.tsx
index 1266fd2238..50f7552ec6 100644
--- a/src/components/clinical-dashboard/document-search-results.tsx
+++ b/src/components/clinical-dashboard/document-search-results.tsx
@@ -14,6 +14,8 @@ import {
   Filter,
   FolderOpen,
   ListChecks,
+  Loader2,
+  Shield,
   ShieldAlert,
   SlidersHorizontal,
   Sparkles,
@@ -54,6 +56,7 @@ import {
 import type { ServiceSearchMatch } from "@/lib/services";
 import type { FormSearchMatch } from "@/lib/forms";
 import type { ClinicalDocument, DocumentMatch, SearchResult } from "@/lib/types";
+import type { RegistryRequestStatus } from "@/lib/use-registry-records";
 import { documentRelevancePercent } from "./relevance-score";
 
 type SearchFacet = { value: string; count: number };
@@ -939,10 +942,42 @@ function SearchRecordResults({
   );
 }
 
+function RecordRegistryNotice({ status, mode }: { status: RegistryRequestStatus; mode: SearchRecordMode }) {
+  if (status === "ready") return null;
+  const noun = mode === "forms" ? "forms" : "services";
+  const config =
+    status === "loading"
+      ? { Icon: Loader2, spin: true, tone: "info" as const, text: `Loading your ${noun} registry...` }
+      : status === "unauthorized"
+        ? { Icon: Shield, spin: false, tone: "warning" as const, text: `Sign in to search your ${noun} registry.` }
+        : {
+            Icon: ShieldAlert,
+            spin: false,
+            tone: "danger" as const,
+            text: `Couldn't load the ${noun} registry. Try again shortly.`,
+          };
+  const toneClass =
+    config.tone === "danger"
+      ? "border-[color:var(--danger-border)] bg-[color:var(--danger-soft)]/50 text-[color:var(--danger)]"
+      : config.tone === "warning"
+        ? "border-[color:var(--warning-border)] bg-[color:var(--warning-soft)]/50 text-[color:var(--warning)]"
+        : "border-[color:var(--border)] bg-[color:var(--surface)] text-[color:var(--text-muted)]";
+  return (
+    <p
+      data-testid="dashboard-registry-status-notice"
+      className={cn("flex items-center gap-2 rounded-lg border px-3 py-2 text-sm font-semibold", toneClass)}
+    >
+      <config.Icon className={cn("h-4 w-4 shrink-0", config.spin && "animate-spin")} aria-hidden />
+      {config.text}
+    </p>
+  );
+}
+
 export function DocumentSearchResultsPanel({
   matches,
   recordMatches = [],
   recordMode = "services",
+  recordStatus = "ready",
   showRecordMatches = false,
   query,
   loading,
@@ -965,6 +1000,7 @@ export function DocumentSearchResultsPanel({
   matches: DocumentMatch[];
   recordMatches?: SearchRecordMatch[];
   recordMode?: SearchRecordMode;
+  recordStatus?: RegistryRequestStatus;
   showRecordMatches?: boolean;
   query: string;
   loading: boolean;
@@ -1066,7 +1102,12 @@ export function DocumentSearchResultsPanel({
         </div>
       ) : null}
 
-      {showRecordMatches ? <SearchRecordResults matches={recordMatches} query={query} mode={recordMode} /> : null}
+      {showRecordMatches ? (
+        <>
+          <RecordRegistryNotice status={recordStatus} mode={recordMode} />
+          <SearchRecordResults matches={recordMatches} query={query} mode={recordMode} />
+        </>
+      ) : null}
 
       {loading ? (
         <LoadingPanel label="Finding matching documents" />
diff --git a/src/components/registry-record-loader.tsx b/src/components/registry-record-loader.tsx
index eb274eacd0..108e202976 100644
--- a/src/components/registry-record-loader.tsx
+++ b/src/components/registry-record-loader.tsx
@@ -57,7 +57,7 @@ export function RegistryRecordLoader({
   slug: string;
   children: (record: ServiceRecord) => ReactNode;
 }) {
-  const { status, record } = useRegistryRecord(kind, slug);
+  const { status, record, governance } = useRegistryRecord(kind, slug);
   const copy = kindCopy[kind];
 
   if (status === "loading") {
@@ -108,5 +108,19 @@ export function RegistryRecordLoader({
     );
   }
 
-  return <>{children(record)}</>;
+  // Reconcile the verified badge with the authoritative governance column so a
+  // record reviewed/downgraded after seeding does not keep showing the stale
+  // fixture verification state.
+  const rendered = governance
+    ? {
+        ...record,
+        verification: {
+          ...record.verification,
+          locallyVerified:
+            governance.validationStatus === "locally_reviewed" || governance.validationStatus === "approved",
+        },
+      }
+    : record;
+
+  return <>{children(rendered)}</>;
 }
diff --git a/src/lib/use-registry-records.ts b/src/lib/use-registry-records.ts
index 3d050f8fbd..0cf8a09da9 100644
--- a/src/lib/use-registry-records.ts
+++ b/src/lib/use-registry-records.ts
@@ -2,7 +2,7 @@
 
 import { useEffect, useState } from "react";
 
-import type { RegistryRecordKind, RegistryValidationStatus } from "@/lib/registry-records";
+import type { RegistryRecordKind, RegistrySourceStatus, RegistryValidationStatus } from "@/lib/registry-records";
 import type { ServiceRecord } from "@/lib/services";
 import { useAuthSession } from "@/lib/supabase/client";
 
@@ -18,14 +18,28 @@ export type RegistryRecordsState = {
   governance: Record<string, RegistryValidationStatus>;
 };
 
+export type RegistryRecordGovernance = {
+  sourceStatus: RegistrySourceStatus;
+  validationStatus: RegistryValidationStatus;
+};
+
 export type RegistryRecordState = {
   status: RegistryRequestStatus;
   record: ServiceRecord | null;
   linkedDocuments: Array<{ id: string; title: string; file_name: string; status: string }>;
   demoMode: boolean;
+  /** Authoritative governance for the record from the API (null until ready),
+   *  so detail pages can render current badges rather than the fixture copy. */
+  governance: RegistryRecordGovernance | null;
 };
 
-const recordLoading: RegistryRecordState = { status: "loading", record: null, linkedDocuments: [], demoMode: false };
+const recordLoading: RegistryRecordState = {
+  status: "loading",
+  record: null,
+  linkedDocuments: [],
+  demoMode: false,
+  governance: null,
+};
 type RegistryRecordsKeyedState = RegistryRecordsState & { kind: RegistryRecordKind };
 
 function recordsState(
@@ -127,24 +141,25 @@ export function useRegistryRecord(kind: RegistryRecordKind, slug: string): Regis
         if (response.status === 401) {
           if (authStatus === "loading") return;
           if (authStatus === "authenticated") markSessionExpired();
-          setState({ status: "unauthorized", record: null, linkedDocuments: [], demoMode: false });
+          setState({ status: "unauthorized", record: null, linkedDocuments: [], demoMode: false, governance: null });
           return;
         }
         if (response.status === 404) {
-          setState({ status: "not_found", record: null, linkedDocuments: [], demoMode: false });
+          setState({ status: "not_found", record: null, linkedDocuments: [], demoMode: false, governance: null });
           return;
         }
         if (!response.ok) {
-          setState({ status: "error", record: null, linkedDocuments: [], demoMode: false });
+          setState({ status: "error", record: null, linkedDocuments: [], demoMode: false, governance: null });
           return;
         }
         const payload = (await response.json()) as {
           record?: ServiceRecord;
           linkedDocuments?: Array<{ id: string; title: string; file_name: string; status: string }>;
           demoMode?: boolean;
+          governance?: RegistryRecordGovernance;
         };
         if (!payload.record) {
-          setState({ status: "not_found", record: null, linkedDocuments: [], demoMode: false });
+          setState({ status: "not_found", record: null, linkedDocuments: [], demoMode: false, governance: null });
           return;
         }
         setState({
@@ -152,10 +167,11 @@ export function useRegistryRecord(kind: RegistryRecordKind, slug: string): Regis
           record: payload.record,
           linkedDocuments: payload.linkedDocuments ?? [],
           demoMode: Boolean(payload.demoMode),
+          governance: payload.governance ?? null,
         });
       })
       .catch(() => {
-        if (active) setState({ status: "error", record: null, linkedDocuments: [], demoMode: false });
+        if (active) setState({ status: "error", record: null, linkedDocuments: [], demoMode: false, governance: null });
       });
     return () => {
       active = false;

From eb37b76fad41050e7e71e5f10d131aa0ccffddfe Mon Sep 17 00:00:00 2001
From: BigSimmo <87357024+BigSimmo@users.noreply.github.com>
Date: Fri, 3 Jul 2026 04:30:35 +0800
Subject: [PATCH 2/3] fix(review): address late PR 209 registry states

---
 src/app/api/registry/records/route.ts         | 20 -------
 src/components/forms/forms-home-page.tsx      | 56 +++++++++++++++++--
 .../forms/forms-search-results-page.tsx       | 27 +++++++--
 src/components/mode-home-template.tsx         | 40 ++++++++++++-
 .../services/services-home-page.tsx           | 44 +++++++++++++--
 5 files changed, 149 insertions(+), 38 deletions(-)

diff --git a/src/app/api/registry/records/route.ts b/src/app/api/registry/records/route.ts
index 70f7cc79ef..6e33251361 100644
--- a/src/app/api/registry/records/route.ts
+++ b/src/app/api/registry/records/route.ts
@@ -96,31 +96,11 @@ export async function GET(request: Request) {
     const records = rows.map(rowToServiceRecord);
     const governanceBySlug = Object.fromEntries(rows.map((row) => [row.slug, rowGovernance(row)]));
 
-    const recordIds = rows.map((row) => row.id);
-    const linkedDocumentIdsByRecordId: Record<string, string[]> = {};
-    if (recordIds.length > 0) {
-      const { data: links, error: linksError } = await supabase
-        .from("clinical_registry_record_sources")
-        .select("record_id, document_id")
-        .eq("owner_id", user.id)
-        .in("record_id", recordIds);
-      if (linksError) throw new Error(linksError.message);
-      for (const link of links ?? []) {
-        const existing = linkedDocumentIdsByRecordId[link.record_id] ?? [];
-        existing.push(link.document_id);
-        linkedDocumentIdsByRecordId[link.record_id] = existing;
-      }
-    }
-    const linkedDocumentIdsBySlug = Object.fromEntries(
-      rows.map((row) => [row.slug, linkedDocumentIdsByRecordId[row.id] ?? []]),
-    );
-
     return registryResponse({
       records,
       matches: q ? matchesPayload(rankRecords(kind, records, q, limit)) : undefined,
       total: rows.length,
       governance: governanceBySlug,
-      linkedDocumentIds: linkedDocumentIdsBySlug,
     });
   } catch (error) {
     if (error instanceof AuthenticationError) {
diff --git a/src/components/forms/forms-home-page.tsx b/src/components/forms/forms-home-page.tsx
index 1d0bc5587d..eb3e040de3 100644
--- a/src/components/forms/forms-home-page.tsx
+++ b/src/components/forms/forms-home-page.tsx
@@ -1,9 +1,22 @@
 "use client";
 
-import { ArrowLeftRight, ClipboardCheck, FileText, Route, Search, ShieldCheck, Truck, UserRound } from "lucide-react";
+import {
+  ArrowLeftRight,
+  ClipboardCheck,
+  FileQuestion,
+  FileText,
+  Loader2,
+  Route,
+  Search,
+  ShieldAlert,
+  ShieldCheck,
+  Truck,
+  UserRound,
+} from "lucide-react";
 
 import {
   ModeHomeMain,
+  ModeHomeStatusNotice,
   ModeHomeTemplate,
   ModeHomeVerificationFooter,
   type ModeHomeAction,
@@ -65,6 +78,36 @@ const commonTasks: ModeHomePill[] = [
 export function FormsHomePage() {
   const registry = useRegistryRecords("form");
   const verifiedCount = countVerifiedRegistryRecords(registry);
+  const registryReady = registry.status === "ready";
+  const hasRegistryRecords = registryReady && registry.total > 0;
+  const registryNotice =
+    registry.status === "loading" ? (
+      <ModeHomeStatusNotice
+        icon={Loader2}
+        title="Loading forms registry"
+        body="Form tasks will appear once your private registry is ready."
+      />
+    ) : registry.status === "unauthorized" ? (
+      <ModeHomeStatusNotice
+        icon={ShieldAlert}
+        title="Sign in required"
+        body="Sign in to open private form records and pathways."
+        actionHref="/"
+        actionLabel="Go to sign in"
+      />
+    ) : registry.status === "error" ? (
+      <ModeHomeStatusNotice
+        icon={ShieldAlert}
+        title="Could not load forms"
+        body="The forms registry could not be loaded. Try again shortly."
+      />
+    ) : !hasRegistryRecords ? (
+      <ModeHomeStatusNotice
+        icon={FileQuestion}
+        title="No forms seeded yet"
+        body="Seed your forms registry before opening form detail shortcuts."
+      />
+    ) : null;
 
   return (
     <ModeHomeMain testId="forms-home">
@@ -75,18 +118,21 @@ export function FormsHomePage() {
         icon={FileText}
         desktopComposerSlotId={modeHomeDesktopComposerSlotId}
         actionsLabel="Forms tasks"
-        actions={taskCards}
+        actions={hasRegistryRecords ? taskCards : []}
         pillsTitle="Common tasks"
-        pills={commonTasks}
+        pills={hasRegistryRecords ? commonTasks : []}
         footer={
-          registry.status === "ready" ? (
+          hasRegistryRecords ? (
             <ModeHomeVerificationFooter
+              icon={ShieldCheck}
               label="Source verified"
               body="MHA 2014 forms"
               verifiedCount={verifiedCount}
               totalCount={registry.total}
             />
-          ) : null
+          ) : (
+            registryNotice
+          )
         }
       />
     </ModeHomeMain>
diff --git a/src/components/forms/forms-search-results-page.tsx b/src/components/forms/forms-search-results-page.tsx
index 641451b0bc..c4d7cf5dfc 100644
--- a/src/components/forms/forms-search-results-page.tsx
+++ b/src/components/forms/forms-search-results-page.tsx
@@ -841,14 +841,21 @@ function RegistryStatusNotice({ status }: { status: RegistryRequestStatus }) {
   if (status === "ready") return null;
   const notice =
     status === "loading"
-      ? { icon: Loader2, spin: true, tone: "info", text: "Loading your forms registry..." }
+      ? { icon: Loader2, spin: true, tone: "info", text: "Loading your forms registry...", action: null }
       : status === "unauthorized"
-        ? { icon: Shield, spin: false, tone: "warning", text: "Sign in to search your forms registry." }
+        ? {
+            icon: Shield,
+            spin: false,
+            tone: "warning",
+            text: "Sign in to search your forms registry.",
+            action: { href: "/", label: "Go to sign in" },
+          }
         : {
             icon: ShieldAlert,
             spin: false,
             tone: "danger",
             text: "Couldn't load the forms registry. Try again shortly.",
+            action: null,
           };
   const Icon = notice.icon;
   const toneClass =
@@ -858,13 +865,21 @@ function RegistryStatusNotice({ status }: { status: RegistryRequestStatus }) {
         ? "border-[color:var(--warning-border)] bg-[color:var(--warning-soft)]/50 text-[color:var(--warning)]"
         : "border-[color:var(--border)] bg-[color:var(--surface)] text-[color:var(--text-muted)]";
   return (
-    <p
+    <div
       data-testid="forms-registry-status-notice"
-      className={`flex items-center gap-2 rounded-lg border px-3 py-2 text-sm font-semibold ${toneClass}`}
+      className={`flex flex-wrap items-center gap-2 rounded-lg border px-3 py-2 text-sm font-semibold ${toneClass}`}
     >
       <Icon className={`h-4 w-4 shrink-0 ${notice.spin ? "animate-spin" : ""}`} aria-hidden />
-      {notice.text}
-    </p>
+      <span className="min-w-0 flex-1">{notice.text}</span>
+      {notice.action ? (
+        <Link
+          href={notice.action.href}
+          className="inline-flex min-h-8 items-center justify-center rounded-md bg-[color:var(--command)] px-3 text-xs font-bold text-[color:var(--command-contrast)] hover:bg-[color:var(--command-hover)]"
+        >
+          {notice.action.label}
+        </Link>
+      ) : null}
+    </div>
   );
 }
 
diff --git a/src/components/mode-home-template.tsx b/src/components/mode-home-template.tsx
index 42241bf689..25ea533887 100644
--- a/src/components/mode-home-template.tsx
+++ b/src/components/mode-home-template.tsx
@@ -1,6 +1,6 @@
 import Link from "next/link";
 import { type ReactNode } from "react";
-import { ArrowRight, ShieldCheck, type LucideIcon } from "lucide-react";
+import { ArrowRight, type LucideIcon } from "lucide-react";
 
 import { cn } from "@/components/ui-primitives";
 
@@ -111,11 +111,13 @@ export function ModeHomeMain({
 }
 
 export function ModeHomeVerificationFooter({
+  icon: Icon,
   label,
   body,
   verifiedCount,
   totalCount,
 }: {
+  icon: LucideIcon;
   label: string;
   body: string;
   verifiedCount: number;
@@ -124,7 +126,7 @@ export function ModeHomeVerificationFooter({
   return (
     <p className="flex flex-wrap items-center justify-center gap-x-3 gap-y-2 pt-1 text-xs font-medium leading-5 text-[color:var(--text-muted)] sm:text-sm">
       <span className="inline-flex items-center gap-2 font-semibold text-[color:var(--clinical-accent)]">
-        <ShieldCheck className="h-4 w-4" aria-hidden="true" />
+        <Icon className="h-4 w-4" aria-hidden="true" />
         {label}
       </span>
       <span aria-hidden="true">•</span>
@@ -136,6 +138,40 @@ export function ModeHomeVerificationFooter({
   );
 }
 
+export function ModeHomeStatusNotice({
+  icon: Icon,
+  title,
+  body,
+  actionHref,
+  actionLabel,
+}: {
+  icon: LucideIcon;
+  title: string;
+  body: string;
+  actionHref?: string;
+  actionLabel?: string;
+}) {
+  return (
+    <div className="mx-auto grid max-w-xl gap-3 rounded-lg border border-[color:var(--border)] bg-[color:var(--surface)] px-4 py-3 text-left shadow-[var(--shadow-inset)] sm:grid-cols-[2.25rem_minmax(0,1fr)_auto] sm:items-center">
+      <span className="grid h-9 w-9 place-items-center rounded-lg bg-[color:var(--clinical-accent-soft)] text-[color:var(--clinical-accent)]">
+        <Icon className="h-5 w-5" aria-hidden="true" />
+      </span>
+      <span className="grid gap-1">
+        <span className="text-sm font-bold text-[color:var(--text-heading)]">{title}</span>
+        <span className="text-sm leading-5 text-[color:var(--text-muted)]">{body}</span>
+      </span>
+      {actionHref && actionLabel ? (
+        <Link
+          href={actionHref}
+          className="inline-flex min-h-9 items-center justify-center rounded-lg bg-[color:var(--command)] px-3 text-sm font-semibold text-[color:var(--command-contrast)] hover:bg-[color:var(--command-hover)]"
+        >
+          {actionLabel}
+        </Link>
+      ) : null}
+    </div>
+  );
+}
+
 export function ModeHomeTemplate({
   testId,
   title,
diff --git a/src/components/services/services-home-page.tsx b/src/components/services/services-home-page.tsx
index 9eb5c6e745..657a488175 100644
--- a/src/components/services/services-home-page.tsx
+++ b/src/components/services/services-home-page.tsx
@@ -1,9 +1,10 @@
 "use client";
 
-import { FileSearch, MapPinned, Route, Users } from "lucide-react";
+import { FileQuestion, FileSearch, Loader2, MapPinned, Route, ShieldAlert, ShieldCheck, Users } from "lucide-react";
 
 import {
   ModeHomeMain,
+  ModeHomeStatusNotice,
   ModeHomeTemplate,
   ModeHomeVerificationFooter,
   type ModeHomeAction,
@@ -79,6 +80,36 @@ const commonPathways: ModeHomePill[] = [
 export function ServicesHomePage() {
   const registry = useRegistryRecords("service");
   const verifiedCount = countVerifiedRegistryRecords(registry);
+  const registryReady = registry.status === "ready";
+  const hasRegistryRecords = registryReady && registry.total > 0;
+  const registryNotice =
+    registry.status === "loading" ? (
+      <ModeHomeStatusNotice
+        icon={Loader2}
+        title="Loading services registry"
+        body="Service tasks will appear once your private registry is ready."
+      />
+    ) : registry.status === "unauthorized" ? (
+      <ModeHomeStatusNotice
+        icon={ShieldAlert}
+        title="Sign in required"
+        body="Sign in to open private service records and referral pathways."
+        actionHref="/"
+        actionLabel="Go to sign in"
+      />
+    ) : registry.status === "error" ? (
+      <ModeHomeStatusNotice
+        icon={ShieldAlert}
+        title="Could not load services"
+        body="The services registry could not be loaded. Try again shortly."
+      />
+    ) : !hasRegistryRecords ? (
+      <ModeHomeStatusNotice
+        icon={FileQuestion}
+        title="No services seeded yet"
+        body="Seed your services registry before opening service detail shortcuts."
+      />
+    ) : null;
 
   return (
     <ModeHomeMain testId="services-home">
@@ -89,18 +120,21 @@ export function ServicesHomePage() {
         icon={Users}
         desktopComposerSlotId={modeHomeDesktopComposerSlotId}
         actionsLabel="Service tasks"
-        actions={taskCards}
+        actions={hasRegistryRecords ? taskCards : []}
         pillsTitle="Common pathways"
-        pills={commonPathways}
+        pills={hasRegistryRecords ? commonPathways : []}
         footer={
-          registry.status === "ready" ? (
+          hasRegistryRecords ? (
             <ModeHomeVerificationFooter
+              icon={ShieldCheck}
               label="Catalogue service data"
               body="Confirm locally before use"
               verifiedCount={verifiedCount}
               totalCount={registry.total}
             />
-          ) : null
+          ) : (
+            registryNotice
+          )
         }
       />
     </ModeHomeMain>

From 9f3f873f2f2ddde2313a2f4398c31bb13f728468 Mon Sep 17 00:00:00 2001
From: BigSimmo <87357024+BigSimmo@users.noreply.github.com>
Date: Fri, 3 Jul 2026 08:28:46 +0800
Subject: [PATCH 3/3] fix(review): preserve only reviewed registry governance

---
 scripts/seed-registry-records.ts | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/scripts/seed-registry-records.ts b/scripts/seed-registry-records.ts
index 8b0ee4abfb..c4468b6f16 100644
--- a/scripts/seed-registry-records.ts
+++ b/scripts/seed-registry-records.ts
@@ -113,6 +113,11 @@ async function main() {
   const upsertRows = rows.map((row) => {
     const prior = governanceByKey.get(`${row.kind}:${row.slug}`);
     if (!prior) return row;
+    const hasReviewedGovernance =
+      Boolean(prior.last_reviewed_at) ||
+      prior.validation_status === "locally_reviewed" ||
+      prior.validation_status === "approved";
+    if (!hasReviewedGovernance) return row;
     preserved += 1;
     return {
       ...row,