Opus refactor

frontend/src/pages/relays/RelayDetail.tsx

@@ -1,12 +1,12 @@
 import { useParams } from "@solidjs/router"
-import { createMemo, createResource, createSignal, Show } from "solid-js"
-import type { PlanId } from "@/lib/api"
+import { createMemo, createResource, Show } from "solid-js"
 import BackLink from "@/components/BackLink"
 import PageContainer from "@/components/PageContainer"
 import RelayDetailCard from "@/components/RelayDetailCard"
 import ResourceState from "@/components/ResourceState"
 import useMinLoading from "@/components/useMinLoading"
-import { deactivateRelayById, getRelayMembers, updateRelayById, updateRelayPlanById, useRelay, type Relay } from "@/lib/hooks"
+import { getRelayMembers, useRelay } from "@/lib/hooks"
+import useRelayToggles from "@/lib/useRelayToggles"
 
 export default function RelayDetail() {
   const params = useParams()
@@ -17,158 +17,13 @@ export default function RelayDetail() {
     return subdomain ? `wss://${subdomain}.spaces.coracle.social` : undefined
   })
   const [members] = createResource(relayUrl, getRelayMembers)
-  const [busy, setBusy] = createSignal(false)
-  const [error, setError] = createSignal("")
   const loading = useMinLoading(() => relay.loading && !relay())
-
-  async function handleDeactivate() {
-    if (busy()) return
-    setError("")
-    setBusy(true)
-    try {
-      await deactivateRelayById(relayId())
-      await refetch()
-    } catch (e) {
-      setError(e instanceof Error ? e.message : "Failed to deactivate relay")
-    } finally {
-      setBusy(false)
-    }
-  }
-
-  function toBool(value: number | undefined, fallback: boolean): boolean {
-    if (value === 0) return false
-    if (value === 1) return true
-    return fallback
-  }
-
-  function toInt(value: boolean): number {
-    return value ? 1 : 0
-  }
-
-  async function updateRelay(next: Relay, previous: Relay) {
-    const current = relay()
-    if (!current) return
-
-    setError("")
-    mutate(next)
-
-    try {
-      await updateRelayById(relayId(), next)
-      await refetch()
-    } catch (e) {
-      mutate(previous)
-      setError(e instanceof Error ? e.message : "Failed to update relay settings")
-    }
-  }
-
-  function togglePublicJoin() {
-    const current = relay()
-    if (!current) return
-    const next = {
-      ...current,
-      policy_public_join: toInt(!toBool(current.policy_public_join, false)),
-    }
-    void updateRelay(next, current)
-  }
-
-  function toggleStripSignatures() {
-    const current = relay()
-    if (!current) return
-    const next = {
-      ...current,
-      policy_strip_signatures: toInt(!toBool(current.policy_strip_signatures, false)),
-    }
-    void updateRelay(next, current)
-  }
-
-  function toggleGroups() {
-    const current = relay()
-    if (!current) return
-    const next = {
-      ...current,
-      groups_enabled: toInt(!toBool(current.groups_enabled, true)),
-    }
-    void updateRelay(next, current)
-  }
-
-  function toggleManagement() {
-    const current = relay()
-    if (!current) return
-    const next = {
-      ...current,
-      management_enabled: toInt(!toBool(current.management_enabled, true)),
-    }
-    void updateRelay(next, current)
-  }
-
-  function toggleMediaStorage() {
-    const current = relay()
-    if (!current) return
-    const next = {
-      ...current,
-      blossom_enabled: toInt(!toBool(current.blossom_enabled, current.plan !== "free")),
-    }
-    void updateRelay(next, current)
-  }
-
-  function togglePushNotifications() {
-    const current = relay()
-    if (!current) return
-    const next = {
-      ...current,
-      push_enabled: toInt(!toBool(current.push_enabled, true)),
-    }
-    void updateRelay(next, current)
-  }
-
-  function toggleLivekitSupport() {
-    const current = relay()
-    if (!current) return
-    const next = {
-      ...current,
-      livekit_enabled: toInt(!toBool(current.livekit_enabled, current.plan !== "free")),
-    }
-    void updateRelay(next, current)
-  }
-
-  async function handleUpdatePlan(plan: PlanId) {
-    const current = relay()
-    if (!current) return
-
-    const previous = current
-    setError("")
-
-    const next = { ...current }
-    if (plan === "free") {
-      next.blossom_enabled = 0
-      next.livekit_enabled = 0
-    }
-
-    next.plan = plan
-    mutate(next)
-
-    try {
-      await updateRelayPlanById(relayId(), plan)
-      await refetch()
-    } catch (e) {
-      mutate(previous)
-      setError(e instanceof Error ? e.message : "Failed to update relay plan")
-      throw e
-    }
-  }
+  const { busy, handleDeactivate, handleUpdatePlan, toggles } = useRelayToggles(relayId, relay, { refetch, mutate })
 
   return (
     <PageContainer>
       <BackLink href="/relays" label="Relays" />
-
-      <ResourceState
-        loading={loading()}
-        error={relay.error}
-        loadingText="Loading relay..."
-        errorText="Failed to load relay."
-        class="mb-4"
-      />
-
+      <ResourceState loading={loading()} error={relay.error} loadingText="Loading relay..." errorText="Failed to load relay." class="mb-4" />
       <Show when={!loading() && relay()}>
         {(r) => (
           <div class="mb-6">
@@ -178,21 +33,12 @@ export default function RelayDetail() {
               editHref={`/relays/${params.id}/edit`}
               onDeactivate={handleDeactivate}
               deactivating={busy()}
-              onTogglePublicJoin={togglePublicJoin}
-              onToggleStripSignatures={toggleStripSignatures}
-              onToggleGroups={toggleGroups}
-              onToggleManagement={toggleManagement}
-              onToggleMediaStorage={toggleMediaStorage}
-              onToggleLivekitSupport={toggleLivekitSupport}
-              onTogglePushNotifications={togglePushNotifications}
               onUpdatePlan={handleUpdatePlan}
+              {...toggles}
             />
           </div>
         )}
       </Show>
-      <Show when={error()}>
-        <p class="mt-3 text-sm text-red-600">{error()}</p>
-      </Show>
     </PageContainer>
   )
 }