Fixes type errors, cleans up code

src/components/CreateCampaignButton.tsx

@@ -1,12 +1,16 @@
-import { useState } from "react";
-import { pb } from "@/lib/pocketbase";
 import { useAuth } from "@/context/auth/AuthContext";
+import { pb } from "@/lib/pocketbase";
 import type { Campaign } from "@/lib/types";
+import { useState } from "react";
 
 /**
  * Button and form for creating a new campaign. Handles UI state and creation logic.
  */
-export function CreateCampaignButton({ onCreated }: { onCreated?: (campaign: Campaign) => void }) {
+export function CreateCampaignButton({
+  onCreated,
+}: {
+  onCreated?: (campaign: Campaign) => void;
+}) {
   const [creating, setCreating] = useState(false);
   const [name, setName] = useState("");
   const [loading, setLoading] = useState(false);
@@ -29,7 +33,7 @@ export function CreateCampaignButton({ onCreated }: { onCreated?: (campaign: Cam
       });
       setName("");
       setCreating(false);
-      if (onCreated) onCreated({ id: record.id, name: record.name });
+      if (onCreated) onCreated(record as Campaign);
     } catch (e: any) {
       setError(e?.message || "Failed to create campaign.");
     } finally {
@@ -55,7 +59,7 @@ export function CreateCampaignButton({ onCreated }: { onCreated?: (campaign: Cam
         className="px-3 py-2 rounded bg-slate-800 text-slate-100 border border-slate-700 focus:outline-none focus:ring-2 focus:ring-violet-500"
         placeholder="Campaign name"
         value={name}
-        onChange={e => setName(e.target.value)}
+        onChange={(e) => setName(e.target.value)}
         disabled={loading}
         autoFocus
       />
@@ -68,7 +72,11 @@ export function CreateCampaignButton({ onCreated }: { onCreated?: (campaign: Cam
       </button>
       <button
         className="px-2 py-2 rounded text-slate-400 hover:text-red-400"
-        onClick={() => { setCreating(false); setName(""); setError(null); }}
+        onClick={() => {
+          setCreating(false);
+          setName("");
+          setError(null);
+        }}
         disabled={loading}
         aria-label="Cancel"
       >

src/routes/__root.tsx

@@ -26,16 +26,6 @@ function RootHeader() {
         >
           Campaigns
         </Link>
-        <Link
-          to="/sessions"
-          className="no-underline text-slate-200 hover:text-violet-400 transition-colors font-medium border-b-2 border-transparent pb-1"
-          activeProps={{
-            className:
-              "no-underline text-violet-400 border-violet-400 border-b-2 pb-1",
-          }}
-        >
-          Sessions
-        </Link>
         <Link
           to="/about"
           className="no-underline text-slate-200 hover:text-violet-400 transition-colors font-medium border-b-2 border-transparent pb-1"

src/routes/_authenticated/document.$documentId.tsx

@@ -89,7 +89,9 @@ function RouteComponent() {
 
   async function handleToggleDiscovered(secret: Secret, checked: boolean) {
     // 1. Update the discovered field in the secret document
-    await pb.collection("documents").update(secret.id, {
+    const updatedSecret: Secret = await pb
+      .collection("documents")
+      .update(secret.id, {
         data: {
           ...secret.data,
           secret: {
@@ -115,17 +117,7 @@ function RouteComponent() {
     }
     // 4. Update local state
     setSecretList(
-      secretList.map((s: any) =>
+      secretList.map((s: any) => (s.id === secret.id ? updatedSecret : s)),
-        s.id === secret.id
-          ? {
-              ...s,
-              data: {
-                ...s.data,
-                secret: { ...s.data.secret, discovered: checked },
-              },
-            }
-          : s,
-      ),
     );
   }