fix(twister): type Goal/TodayItem id fields as Uuid, not string

Goal.id/focusId, TodayItem.id/threadIds/focusId/goalId, and the
corresponding Plot-tool method signatures (archiveGoal, updateTodayItem,
getTodayThreadId) now use the branded Uuid type consistently with every
other id field in the SDK (Thread.id, Focus.id, Link.id, etc.), instead
of plain string.

Author: KrisBraun

twister/src/goal.ts

@@ -15,6 +15,8 @@
  * cleared (`null`) ones.
  */
 
+import { type Uuid } from "./utils/uuid";
+
 /**
  * Lifecycle status of a goal.
  * - `active`: live; shapes day planning.
@@ -40,7 +42,7 @@ export type GoalCadence = {
  */
 export type Goal = {
   /** Unique identifier for the goal */
-  id: string;
+  id: Uuid;
   /** Short imperative, e.g. "Ship the Today feature" */
   title: string;
   /** Freeform elaboration / agent notes */
@@ -54,7 +56,7 @@ export type Goal = {
   /** Recurring effort budget */
   cadence: GoalCadence | null;
   /** Linked focus id, or null when the goal isn't tied to a focus */
-  focusId: string | null;
+  focusId: Uuid | null;
 };
 
 /** Type for creating a new goal: `title` is required, all else optional. */

twister/src/plot.ts

@@ -1495,7 +1495,7 @@ export type TodayItemKind = "todo" | "event" | "urgent" | "goal" | "update";
  * server-authored and read-only from the SDK.
  */
 export type TodayItem = {
-  id: string;
+  id: Uuid;
   /** The user-local day this item belongs to, as an ISO date ("YYYY-MM-DD"). */
   day: string;
   section: TodayItemSection;
@@ -1507,11 +1507,11 @@ export type TodayItem = {
   /** 1–2 sentence support text, mainly for "updates" items. */
   detail: string | null;
   /** Linked threads; the first is the primary open target. May be empty (e.g. goal items). */
-  threadIds: string[];
+  threadIds: Uuid[];
   /** The focus this item is labeled with, if any. */
-  focusId: string | null;
+  focusId: Uuid | null;
   /** The goal this item derives from, if any. */
-  goalId: string | null;
+  goalId: Uuid | null;
   /** Event start (events only). */
   startsAt: Date | string | null;
   /** Event end (events only). */

twister/src/tools/plot.ts

@@ -690,7 +690,7 @@ export abstract class Plot extends ITool {
    * @throws Error if the goal does not exist
    */
   // eslint-disable-next-line @typescript-eslint/no-unused-vars
-  abstract archiveGoal(id: string): Promise<void>;
+  abstract archiveGoal(id: Uuid): Promise<void>;
 
   /**
    * Lists the user's current Today snapshot: all non-archived items for the
@@ -716,7 +716,7 @@ export abstract class Plot extends ITool {
    */
   // eslint-disable-next-line @typescript-eslint/no-unused-vars
   abstract updateTodayItem(update: {
-    id: string;
+    id: Uuid;
     rank?: number;
     checked?: boolean;
     dismissed?: boolean;
@@ -728,7 +728,7 @@ export abstract class Plot extends ITool {
    *
    * Requires `today` access ({@link TodayAccess.Read} or higher).
    */
-  abstract getTodayThreadId(): Promise<string | null>;
+  abstract getTodayThreadId(): Promise<Uuid | null>;
 
   /**
    * Retrieves links from connected source channels.