chore(groups): retire the /groups index in favor of the profile Groups tab

src/app/__tests__/public-endorsements-groups.test.tsx

@@ -2,20 +2,20 @@ import { describe, it, expect, vi, beforeEach } from "vitest"
 import { render, screen, fireEvent, cleanup } from "@testing-library/react"
 
 /**
- * P1: /endorsements and /groups used to redirect anonymous (signed-out)
- * visitors to /welcome via the route-level <AuthGuard>. That guard was
- * removed; instead each page renders a public sign-in prompt in place.
+ * P1: /endorsements used to redirect anonymous (signed-out) visitors to
+ * /welcome via the route-level <AuthGuard>. That guard was removed;
+ * instead the page renders a public sign-in prompt in place.
  *
- * Both pages are personal, owner-scoped surfaces (a personal inbox and
- * the viewer's own membership list, keyed on the viewer's DID), so there
- * is no public listing to show — the "best available public view" is an
- * explanation + a sign-in CTA. These tests pin that:
+ * /endorsements is a personal, owner-scoped surface (a personal inbox
+ * keyed on the viewer's DID), so there is no public listing to show —
+ * the "best available public view" is an explanation + a sign-in CTA.
+ * These tests pin that:
  *   - signed-out renders the prompt (no crash, no error state),
  *   - the CTA invokes the shared openSignIn() flow,
  *   - signed-in still renders the real management UI.
  *
- * The shared SignedOutPrompt lets us assert one openSignIn() spy across
- * both routes.
+ * (The sibling /groups index has since been retired — it now redirects
+ * to /home — so only /endorsements remains covered here.)
  */
 
 const openSignIn = vi.fn().mockResolvedValue(undefined)
@@ -41,7 +41,7 @@ vi.mock("next/navigation", () => ({
   useSearchParams: () => new URLSearchParams(""),
 }))
 
-// --- /groups dependencies -------------------------------------------------
+// org-context is pulled in transitively by the endorsements page chrome.
 vi.mock("@/lib/groups/org-context", () => ({
   useOrg: () => ({
     activeOrg: null,
@@ -52,12 +52,6 @@ vi.mock("@/lib/groups/org-context", () => ({
   }),
 }))
 
-vi.mock("@/lib/groups/api", () => ({
-  putMembership: vi.fn().mockResolvedValue(undefined),
-  deleteMembership: vi.fn().mockResolvedValue(undefined),
-  removeOrgMember: vi.fn().mockResolvedValue(undefined),
-}))
-
 // --- /endorsements dependencies ------------------------------------------
 vi.mock("@/hooks/use-endorsements", () => ({
   useGivenEndorsements: () => ({
@@ -119,30 +113,3 @@ describe("/endorsements public view (signed out)", () => {
     expect(screen.getByRole("tab", { name: "Given" })).toBeTruthy()
   })
 })
-
-describe("/groups public view (signed out)", () => {
-  it("renders the sign-in prompt instead of redirecting", async () => {
-    const { default: GroupsPage } = await import("../../app/groups/page")
-    render(<GroupsPage />)
-
-    expect(screen.getByText("Sign in to see your groups")).toBeTruthy()
-    expect(replace).not.toHaveBeenCalled()
-  })
-
-  it("invokes openSignIn when the CTA is clicked", async () => {
-    const { default: GroupsPage } = await import("../../app/groups/page")
-    render(<GroupsPage />)
-
-    fireEvent.click(screen.getByRole("button", { name: "Sign in" }))
-    expect(openSignIn).toHaveBeenCalledTimes(1)
-  })
-
-  it("renders the real Groups UI when authenticated", async () => {
-    authState = { did: "did:plc:me", isAuthenticated: true, isLoading: false }
-    const { default: GroupsPage } = await import("../../app/groups/page")
-    render(<GroupsPage />)
-
-    expect(screen.queryByText("Sign in to see your groups")).toBeNull()
-    expect(screen.getByRole("heading", { name: "Membership" })).toBeTruthy()
-  })
-})

src/app/groups/create/page.tsx

@@ -274,7 +274,7 @@ export default function CreateGroupPage() {
       // in the account switcher + group list.
       await putMembership(did, groupDid, "owner")
       await refetchOrgs()
-      router.push("/groups")
+      router.push("/home")
     } catch (err) {
       console.error("[groups/create] failed", err)
       if (
@@ -343,9 +343,9 @@ export default function CreateGroupPage() {
             >
               <Button
                 variant="secondary"
-                onClick={() => router.push("/groups")}
+                onClick={() => router.push("/home")}
               >
-                Back to groups
+                Back to home
               </Button>
             </EmptyState>
           </div>
@@ -592,7 +592,7 @@ export default function CreateGroupPage() {
             <Button
               type="button"
               variant="ghost"
-              onClick={() => router.push("/groups")}
+              onClick={() => router.push("/home")}
               disabled={isCreating}
             >
               Cancel

src/app/groups/import/page.tsx

@@ -73,7 +73,7 @@ export default function ImportGroupPage() {
           console.error("[groups/import] putMembership failed", err)
         }
         await refetchOrgs()
-        router.push("/groups")
+        router.push("/home")
       } catch (err) {
         if (err instanceof RegisterGroupError && err.code === "GroupAlreadyRegistered") {
           setError("This account is already registered as a group.")