ei-tui 0.6.3 → 0.6.5

package/tui/src/components/HelpOverlay.tsx

@@ -1,10 +1,13 @@
 import { useKeyboard } from "@opentui/solid";
-import { For, onMount, onCleanup } from "solid-js";
-import { getAllCommands } from "../commands/registry";
+import { onMount, onCleanup } from "solid-js";
+import type { CliRenderer } from "@opentui/core";
 import { useKeyboardNav } from "../context/keyboard.js";
+import { spawnPager } from "../util/editor.js";
+import { buildManPage } from "../util/help-content.js";
 
 interface HelpOverlayProps {
   onDismiss: () => void;
+  renderer: CliRenderer;
 }
 
 export function HelpOverlay(props: HelpOverlayProps) {
@@ -14,11 +17,14 @@ export function HelpOverlay(props: HelpOverlayProps) {
 
   useKeyboard((event) => {
     event.preventDefault();
-    props.onDismiss();
+    if (event.name === "m") {
+      props.onDismiss();
+      void spawnPager(buildManPage(), props.renderer);
+    } else {
+      props.onDismiss();
+    }
   });
 
-  const commands = getAllCommands();
-
   return (
     <box
       position="absolute"
@@ -31,43 +37,67 @@ export function HelpOverlay(props: HelpOverlayProps) {
       justifyContent="center"
     >
       <box
-        width={60}
+        width={82}
         backgroundColor="#1a1a2e"
         borderStyle="single"
         borderColor="#586e75"
         padding={2}
         flexDirection="column"
+        gap={1}
       >
 
-        <text fg="#eee8d5">
-          Commands:
-        </text>
-        <For each={commands.sort()}>
-          {(cmd) => (
-            <text fg="#93a1a1">
-              /{cmd.name} - {cmd.description}
-            </text>
-          )}
-        </For>
-        <text> </text>
+        <box flexDirection="row" gap={2}>
+
+          <box flexDirection="column" gap={1} width={38}>
+            <box flexDirection="column">
+              <text fg="#eee8d5">Keybindings</text>
+              <text fg="#93a1a1">  Ctrl+E    Open $EDITOR (preserves input)</text>
+              <text fg="#93a1a1">  Ctrl+C    Clear input / exit</text>
+              <text fg="#93a1a1">  Ctrl+B    Toggle sidebar</text>
+              <text fg="#93a1a1">  Escape    Abort / resume queue</text>
+              <text fg="#93a1a1">  PgUp/Dn  Scroll messages</text>
+            </box>
+
+            <box flexDirection="column">
+              <text fg="#eee8d5">Core</text>
+              <text fg="#93a1a1">  /set          Edit global settings</text>
+              <text fg="#93a1a1">  /q  /q!       Quit  (! = skip sync)</text>
+              <text fg="#93a1a1">  /provider     Manage LLM providers</text>
+              <text fg="#93a1a1">  /me           Edit your data</text>
+              <text fg="#93a1a1">  /d  /d &lt;name&gt; Edit persona details</text>
+            </box>
+          </box>
+
+          <box flexDirection="column" gap={1} width={38}>
+            <box flexDirection="column">
+              <text fg="#eee8d5">Persona</text>
+              <text fg="#93a1a1">  /p  /p new  /p update</text>
+              <text fg="#93a1a1">  /context   Message context</text>
+              <text fg="#93a1a1">  /pause  /resume</text>
+            </box>
+
+            <box flexDirection="column">
+              <text fg="#eee8d5">Rooms</text>
+              <text fg="#93a1a1">  /r  /r new    Room picker / create</text>
+              <text fg="#93a1a1">  /activate     Advance active node</text>
+              <text fg="#93a1a1">  /silence      Pass your turn</text>
+            </box>
+
+            <box flexDirection="column">
+              <text fg="#eee8d5">Extended</text>
+              <text fg="#93a1a1">  /tools        Tool providers</text>
+              <text fg="#93a1a1">  /auth spotify Spotify OAuth</text>
+              <text fg="#93a1a1">  /queue  /dlq  Inspect queues</text>
+            </box>
+          </box>
+
+        </box>
 
-        <text fg="#eee8d5">
-          Keybindings:
-        </text>
-        <text fg="#93a1a1">Escape - Abort operation / Resume queue</text>
-        <text fg="#93a1a1">Ctrl+C - Clear input / Exit</text>
-        <text fg="#93a1a1">Ctrl+B - Toggle sidebar</text>
-        <text fg="#93a1a1">Ctrl+E - Edit in $EDITOR</text>
-        <text fg="#93a1a1">PageUp/Down - Scroll messages</text>
-        <text> </text>
+        <box flexDirection="column">
+          <text fg="#586e75">  m - full manual  |  any key - dismiss</text>
+          <text fg="#2a2a3e">  Ei - 永 (ei) - eternal</text>
+        </box>
 
-        <text fg="#586e75">
-          Press any key to dismiss
-        </text>
-        <text> </text>
-        <text fg="#2a2a3e">
-          Ei - 永 (ei) - eternal
-        </text>
       </box>
     </box>
   );

package/tui/src/context/overlay.tsx

@@ -33,8 +33,10 @@ export const OverlayProvider: ParentComponent = (props) => {
     const hideOverlay = () => {
       logger.debug("[overlay] hideOverlay called");
       setOverlayRenderer(null);
+      cliRenderer?.requestRender();
     };
     setOverlayRenderer(() => () => renderer(hideOverlay, hideForEditor));
+    queueMicrotask(() => cliRenderer?.requestRender());
   };
 
   const hideOverlay = () => {

package/tui/src/index.tsx

@@ -4,6 +4,13 @@ import { App } from "./app";
 
 import { InstanceLock } from "./util/instance-lock";
 import { FileStorage } from "./storage/file";
+import pkg from "../../package.json";
+
+const args = process.argv.slice(2);
+if (args.includes("--version") || args.includes("version") || args.includes("-v")) {
+  process.stdout.write(`${pkg.version}\n`);
+  process.exit(0);
+}
 
 const storage = new FileStorage(Bun.env.EI_DATA_PATH);
 const lock = new InstanceLock(storage.getDataPath());

package/tui/src/util/editor.ts

@@ -23,6 +23,41 @@ export interface EditorResult {
   aborted: boolean;
 }
 
+export async function spawnPager(content: string, renderer: CliRenderer): Promise<void> {
+  const pager = process.env.PAGER || "less";
+  const tmpDir = os.tmpdir();
+  const tmpFile = path.join(tmpDir, `ei-help-${Date.now()}.txt`);
+
+  fs.writeFileSync(tmpFile, content, "utf-8");
+
+  await new Promise(resolve => setTimeout(resolve, 50));
+
+  return new Promise((resolve) => {
+    renderer.suspend();
+    renderer.currentRenderBuffer.clear();
+
+    const child = spawn(pager, [tmpFile], {
+      stdio: "inherit",
+      shell: true,
+    });
+
+    child.on("error", () => {
+      try { fs.unlinkSync(tmpFile); } catch {}
+      renderer.resume();
+      renderer.requestRender();
+      resolve();
+    });
+
+    child.on("exit", () => {
+      try { fs.unlinkSync(tmpFile); } catch {}
+      renderer.currentRenderBuffer.clear();
+      renderer.resume();
+      renderer.requestRender();
+      resolve();
+    });
+  });
+}
+
 export async function spawnEditorRaw(options: EditorRawOptions): Promise<EditorResult> {
   const { initialContent, filename } = options;
   const editor = process.env.EDITOR || process.env.VISUAL || "vi";
@@ -152,9 +187,7 @@ export async function spawnEditor(options: EditorOptions): Promise<EditorResult>
         logger.debug("[editor] already suspended before spawn, skipping resume");
       }
       
-      queueMicrotask(() => {
-        renderer.requestRender();
-      });
+      renderer.requestRender();
       
       if (code !== 0) {
         try { fs.unlinkSync(tmpFile); } catch {}

package/tui/src/util/help-content.ts

@@ -0,0 +1,136 @@
+export function buildManPage(): string {
+  return `EI(1)                           Ei Terminal UI                          EI(1)
+
+NAME
+    ei - local-first AI companion with persistent personas
+
+KEYBINDINGS
+    Ctrl+E        Open current input in $EDITOR (preserves text)
+    Ctrl+C        Clear input field (second press exits)
+    Ctrl+B        Toggle sidebar
+    Escape        Abort operation / resume queue
+    PageUp/Down   Scroll message history
+
+CORE COMMANDS
+    /settings, /set
+        Edit global settings in $EDITOR. Configure default model,
+        heartbeat interval, context window, integrations, and more.
+
+    /quit, /q
+        Save, sync, and exit. Add ! to force quit without syncing: /q!
+
+    /provider
+        Open provider picker to select, edit, or create LLM providers.
+        /provider <Name>           Set provider on active persona
+        /provider <Name>:<Model>   Set provider and model explicitly
+        /provider new              Create a new provider
+
+    /me
+        Edit your personal data (facts, topics, people) in $EDITOR.
+        Each section includes a commented stub — uncomment and fill it in
+        to create a new entry. No UUID required; one is generated for you.
+
+        /me fact               Edit only facts (stub included)
+        /me topic              Edit only topics (stub included)
+        /me person             Edit only people (stub included)
+        /me fact new           Open with just the new-fact stub
+        /me fact coffee        Filter facts whose name contains "coffee"
+        /me person "New York"  Quoted search for multi-word names
+
+    /details, /d
+        Edit the current persona's details in $EDITOR.
+        /d <name>     Edit a specific persona by name
+
+PERSONA COMMANDS
+    /persona, /p
+        Open persona picker. Switch, list, or create personas.
+        /p new <name>              Create a new persona
+        /p update <name> [person]  Regenerate persona from a person record
+
+    /context, /messages
+        Edit which messages are included in LLM context.
+
+    /pause
+        Pause the current persona indefinitely.
+        /pause <duration>   Pause for a duration: 2h, 1d, 1w
+
+    /resume, /unpause
+        Resume the current paused persona.
+        /resume <name>   Resume a specific persona
+
+    /new
+        Toggle a context boundary — starts a fresh conversation thread
+        without deleting history.
+
+    /quotes, /quote
+        Manage quotes attached to messages.
+        /quotes <N>            View quotes from message N
+        /quotes me             View your own quotes
+        /quotes search "term"  Search quotes by keyword
+        /quotes <persona>      View a persona's quotes
+
+ROOM COMMANDS
+    /room, /r
+        Open room picker. Switch or create rooms.
+        /room new         Create a new room (FFA, CYP, or MAP mode)
+        /room new <name>  Create with a pre-filled name
+
+    /activate, /a
+        Advance the active node in a CYP or MAP room.
+        /activate <num>   Activate a specific response by number
+
+    /silence
+        Pass your turn in a room with an optional reason.
+        /silence [reason]
+
+    /capture
+        Force-extract quotes, topics, and people from the current chat now.
+
+EXTENDED COMMANDS
+    /tools
+        Manage tool providers — enable or disable tools per persona.
+
+    /auth
+        Authenticate with an external service.
+        /auth spotify   Connect your Spotify account
+
+    /queue
+        Pause the queue and inspect or edit active items in $EDITOR.
+
+    /dlq
+        Inspect and recover failed (dead-letter) queue items in $EDITOR.
+
+    /dedupe
+        Find and merge duplicate people or topics.
+        /dedupe person Flare "Jeremy Scherer"
+        /dedupe topic AI "artificial intelligence"
+
+    /archive
+        Archive a persona or room. Lists archived items if no name given.
+        /archive <name>   Archive by name
+
+    /unarchive
+        Restore an archived persona or room and switch to it.
+        /unarchive <name>
+
+    /delete, /del
+        Permanently delete a persona. Cannot be undone.
+        /delete <name>
+
+    /setsync, /ss
+        Set sync credentials (triggers restart).
+        /setsync <username> <passphrase>
+
+    /editor, /e, /edit
+        Open $EDITOR with the current input field contents.
+        Note: Ctrl+E does the same thing without clearing the input first.
+
+TIPS
+    - Append ! to any command as shorthand for --force: /quit!
+    - Duration strings: 30m, 2h, 1d, 1w (used by /pause, /settings)
+    - All editor fields that say "null" inherit from your global settings
+    - $EDITOR and $PAGER are respected throughout
+
+Ei - 永 (ei) - eternal
+`;
+}

package/tui/src/util/yaml-human.ts

@@ -51,6 +51,19 @@ function readOnlyToEnd<T extends WithReadOnlyFields>(item: T): T {
   return { ...rest, learned_on, learned_by, validated_date, last_mentioned, last_updated, last_changed_by } as T;
 }
 
+const FIELD_ORDER = ['id', 'name', 'description', 'sentiment', 'relationship', 'category', 'exposure_current', 'exposure_desired'];
+
+function canonicalFieldOrder<T extends object>(item: T): T {
+  const ordered: Record<string, unknown> = {};
+  for (const key of FIELD_ORDER) {
+    if (key in item) ordered[key] = (item as Record<string, unknown>)[key];
+  }
+  for (const [key, val] of Object.entries(item)) {
+    if (!(key in ordered)) ordered[key] = val;
+  }
+  return ordered as T;
+}
+
 function buildGroupCheckboxMap(itemGroups: string[], allGroups: string[]): Record<string, boolean>[] {
   const activeSet = new Set(itemGroups);
   return [...new Set([...allGroups, ...itemGroups])].map(g => ({ [g]: activeSet.has(g) }));
@@ -65,8 +78,10 @@ function toYAMLIdentifiers(identifiers: PersonIdentifier[], personaLookup?: Map<
   });
 }
 
-function knownTypesComment(personaLookup?: Map<string, string>): string {
-  const lines = [`# Valid types: ${BUILT_IN_IDENTIFIER_TYPES.join(', ')}`];
+function knownTypesComment(people: Person[], personaLookup?: Map<string, string>): string {
+  const userTypes = people.flatMap(p => (p.identifiers ?? []).map(i => i.type));
+  const allTypes = [...new Set([...BUILT_IN_IDENTIFIER_TYPES, ...userTypes])];
+  const lines = [`# Valid types: ${allTypes.join(', ')}`];
   if (personaLookup && personaLookup.size > 0) {
     lines.push(`# Personas: ${Array.from(personaLookup.values()).join(', ')}`);
   }
@@ -84,11 +99,68 @@ function parseGroupCheckboxMap(groups: Record<string, boolean>[] | undefined): s
   return result;
 }
 
-export function humanToYAML(human: HumanEntity, personaLookup?: Map<string, string>, allGroups: string[] = []): string {
-  const data: EditableHumanData = {
-    facts: human.facts.map(f => { const { interested_personas: _ip, persona_groups, ...rest } = readOnlyToEnd(f); return { ...rest, persona_groups: buildGroupCheckboxMap(persona_groups ?? [], allGroups), _delete: false }; }),
-    topics: human.topics.map(t => { const { interested_personas: _ip, persona_groups, ...rest } = readOnlyToEnd(t); return { ...rest, persona_groups: buildGroupCheckboxMap(persona_groups ?? [], allGroups), _delete: false }; }),
-    people: human.people.map(p => {
+function sectionStub(type: "facts" | "topics" | "people", people: Person[], personaLookup?: Map<string, string>): string {
+  if (type === "facts") {
+    return [
+      `  # --- New Fact (uncomment to create) ---`,
+      `  # - name: ''`,
+      `  #   description: ''`,
+      `  #   sentiment: 0`,
+    ].join('\n');
+  }
+
+  if (type === "topics") {
+    return [
+      `  # --- New Topic (uncomment to create) ---`,
+      `  # - name: ''`,
+      `  #   description: ''`,
+      `  #   category: ''  # Interest, Goal, Dream, Conflict, Concern, Fear, Hope, Plan, Project`,
+      `  #   exposure_desired: 0.5`,
+      `  #   sentiment: 0`,
+    ].join('\n');
+  }
+
+  const userTypes = people.flatMap(p => (p.identifiers ?? []).map(i => i.type));
+  const allTypes = [...new Set([...BUILT_IN_IDENTIFIER_TYPES, ...userTypes])];
+  const identifierTypeHint = allTypes.join(', ');
+  const personaNames = personaLookup && personaLookup.size > 0
+    ? Array.from(personaLookup.values()).join(', ')
+    : null;
+
+  return [
+    `  # --- New Person (uncomment to create) ---`,
+    `  # - name: ''`,
+    `  #   description: ''`,
+    `  #   relationship: ''`,
+    `  #   exposure_desired: 0.5`,
+    `  #   sentiment: 0`,
+    `  #   identifiers:`,
+    `  #     # Valid types: ${identifierTypeHint}`,
+    ...(personaNames ? [`  #     # Personas: ${personaNames}`] : []),
+    `  #     - type: ''`,
+    `  #       value: ''`,
+    `  #       primary: true`,
+  ].join('\n');
+}
+
+export function humanToYAML(
+  human: HumanEntity,
+  personaLookup?: Map<string, string>,
+  allGroups: string[] = [],
+  sections?: Set<"facts" | "topics" | "people">,
+): string {
+  const activeSections = sections ?? new Set<"facts" | "topics" | "people">(["facts", "topics", "people"]);
+
+  const data: Partial<EditableHumanData> = {};
+
+  if (activeSections.has("facts") && human.facts.length > 0) {
+    data.facts = human.facts.map(f => { const { interested_personas: _ip, persona_groups, ...rest } = readOnlyToEnd(f); return { ...rest, persona_groups: buildGroupCheckboxMap(persona_groups ?? [], allGroups), _delete: false }; });
+  }
+  if (activeSections.has("topics") && human.topics.length > 0) {
+    data.topics = human.topics.map(t => { const { interested_personas: _ip, persona_groups, ...rest } = readOnlyToEnd(t); return { ...rest, persona_groups: buildGroupCheckboxMap(persona_groups ?? [], allGroups), _delete: false }; });
+  }
+  if (activeSections.has("people") && human.people.length > 0) {
+    data.people = human.people.map(p => {
       const { identifiers, interested_personas: _ip, persona_groups, ...rest } = readOnlyToEnd(p);
       return {
         ...rest,
@@ -96,31 +168,51 @@ export function humanToYAML(human: HumanEntity, personaLookup?: Map<string, stri
         identifiers: toYAMLIdentifiers(identifiers ?? [], personaLookup),
         _delete: false as const,
       };
-    }),
+    });
+  }
+
+  const personComment = knownTypesComment(human.people, personaLookup);
+
+  const applyReadOnlyMarkers = (yaml: string): string =>
+    yaml
+      .replace(/^(\s+)(learned_on: .+)$/mg, '$1# [read-only] $2')
+      .replace(/^(\s+)(learned_by: )(.+)$/mg, (_, indent, key, val) => {
+        const trimmed = val.trim();
+        const displayName = personaLookup?.get(trimmed) ?? trimmed;
+        return `${indent}# [read-only] ${key}${displayName}`;
+      })
+      .replace(/^(\s+)(validated_date: .+)$/mg, '$1# [read-only] $2')
+      .replace(/^(\s+)(last_mentioned: .+)$/mg, '$1# [read-only] $2')
+      .replace(/^(\s+)(last_updated: .+)$/mg, '$1# [read-only] $2')
+      .replace(/^(\s+)(last_changed_by: )(.+)$/mg, (_, indent, key, val) => {
+        const trimmed = val.trim();
+        const displayName = personaLookup?.get(trimmed) ?? trimmed;
+        return `${indent}# [read-only] ${key}${displayName}`;
+      })
+      .replace(/^(\s+)(identifiers:)/mg, (_, indent, _key) => {
+        return `${indent}${personComment}\n${indent}identifiers:`;
+      });
+
+  const serializeSection = (key: "facts" | "topics" | "people", items: unknown[] | undefined): string => {
+    const stub = sectionStub(key, human.people, personaLookup);
+    if (!items || items.length === 0) {
+      return `${key}:\n${stub}`;
+    }
+    const ordered = (items as object[]).map(canonicalFieldOrder);
+    const itemsYaml = YAML.stringify(ordered, { lineWidth: 0 })
+      .split('\n')
+      .map(line => `  ${line}`)
+      .join('\n')
+      .trimEnd();
+    return `${key}:\n${applyReadOnlyMarkers(itemsYaml)}\n${stub}`;
   };
 
-  const personComment = knownTypesComment(personaLookup);
-
-  return YAML.stringify(data, {
-    lineWidth: 0,
-  })
-  .replace(/^(\s+)(learned_on: .+)$/mg, '$1# [read-only] $2')
-  .replace(/^(\s+)(learned_by: )(.+)$/mg, (_, indent, key, val) => {
-    const trimmed = val.trim();
-    const displayName = personaLookup?.get(trimmed) ?? trimmed;
-    return `${indent}# [read-only] ${key}${displayName}`;
-  })
-  .replace(/^(\s+)(validated_date: .+)$/mg, '$1# [read-only] $2')
-  .replace(/^(\s+)(last_mentioned: .+)$/mg, '$1# [read-only] $2')
-  .replace(/^(\s+)(last_updated: .+)$/mg, '$1# [read-only] $2')
-  .replace(/^(\s+)(last_changed_by: )(.+)$/mg, (_, indent, key, val) => {
-    const trimmed = val.trim();
-    const displayName = personaLookup?.get(trimmed) ?? trimmed;
-    return `${indent}# [read-only] ${key}${displayName}`;
-  })
-  .replace(/^(\s+)(identifiers:)/mg, (_, indent, _key) => {
-    return `${indent}${personComment}\n${indent}identifiers:`;
-  });
+  const parts: string[] = [];
+  if (activeSections.has("facts")) parts.push(serializeSection("facts", data.facts));
+  if (activeSections.has("topics")) parts.push(serializeSection("topics", data.topics));
+  if (activeSections.has("people")) parts.push(serializeSection("people", data.people));
+
+  return parts.join('\n') + '\n';
 }
 
 export interface HumanYAMLResult {
@@ -192,7 +284,7 @@ export function humanFromYAML(yamlContent: string, original?: HumanEntity): Huma
     .split('\n')
     .filter(line => !/^\s*#\s*\[read-only\]/.test(line))
     .join('\n');
-  const data = YAML.parse(stripped) as EditableHumanData;
+  const data = (YAML.parse(stripped) ?? {}) as EditableHumanData;
 
   const deletedFactIds: string[] = [];
   const deletedTopicIds: string[] = [];
@@ -207,10 +299,11 @@ export function humanFromYAML(yamlContent: string, original?: HumanEntity): Huma
       deletedFactIds.push(f.id);
     } else {
       const { _delete, persona_groups: groupMap, ...parsed } = f;
+      if (!parsed.id) parsed.id = crypto.randomUUID();
       const originalFact = original?.facts.find(of => of.id === parsed.id);
       const fact: Fact = originalFact
         ? { ...originalFact, ...parsed, persona_groups: parseGroupCheckboxMap(groupMap) }
-        : { ...parsed, persona_groups: parseGroupCheckboxMap(groupMap) };
+        : { ...parsed, last_updated: new Date().toISOString(), persona_groups: parseGroupCheckboxMap(groupMap) };
       facts.push(fact);
       if (!originalFact || factChanged(fact, originalFact)) {
         if (fact.description && !originalFact?.validated_date) {
@@ -227,10 +320,11 @@ export function humanFromYAML(yamlContent: string, original?: HumanEntity): Huma
       deletedTopicIds.push(t.id);
     } else {
       const { _delete, persona_groups: groupMap, ...parsed } = t;
+      if (!parsed.id) parsed.id = crypto.randomUUID();
       const originalTopic = original?.topics.find(ot => ot.id === parsed.id);
       const topic: Topic = originalTopic
         ? { ...originalTopic, ...parsed, persona_groups: parseGroupCheckboxMap(groupMap) }
-        : { ...parsed, persona_groups: parseGroupCheckboxMap(groupMap) };
+        : { ...parsed, last_updated: new Date().toISOString(), persona_groups: parseGroupCheckboxMap(groupMap) };
       topics.push(topic);
       if (!originalTopic || topicChanged(topic, originalTopic)) {
         changedTopicIds.add(topic.id);
@@ -244,6 +338,7 @@ export function humanFromYAML(yamlContent: string, original?: HumanEntity): Huma
       deletedPersonIds.push(p.id);
     } else {
       const { _delete, identifiers: yamlIdentifiers, persona_groups: groupMap, ...parsed } = p;
+      if (!parsed.id) parsed.id = crypto.randomUUID();
       const identifiers: PersonIdentifier[] = (yamlIdentifiers ?? []).map(({ type, value, primary }) => ({
         type,
         value,
@@ -252,7 +347,7 @@ export function humanFromYAML(yamlContent: string, original?: HumanEntity): Huma
       const originalPerson = original?.people.find(op => op.id === parsed.id);
       const person: Person = originalPerson
         ? { ...originalPerson, ...parsed, identifiers, persona_groups: parseGroupCheckboxMap(groupMap) }
-        : { ...parsed, identifiers, persona_groups: parseGroupCheckboxMap(groupMap) };
+        : { ...parsed, last_updated: new Date().toISOString(), identifiers, persona_groups: parseGroupCheckboxMap(groupMap) };
       people.push(person);
       if (!originalPerson || personChanged(person, originalPerson)) {
         changedPersonIds.add(person.id);

package/tui/src/util/yaml-persona.ts

@@ -7,6 +7,7 @@ import type {
   ProviderAccount,
 } from "../../../src/core/types.js";
 import { modelGuidToDisplay, displayToModelGuid } from "./yaml-shared.js";
+import { parseDuration, formatDuration } from "./duration.js";
 
 const PLACEHOLDER_LONG_DESC = "Detailed description of this persona's personality, background, and role";
 
@@ -36,8 +37,8 @@ interface EditablePersonaData {
   groups_visible?: Record<string, boolean>[];
   traits: YAMLTrait[];
   topics: YAMLPersonaTopic[];
-  heartbeat_delay_ms?: string | number;
-  context_window_hours?: number;
+  heartbeat_delay_ms?: string | null;
+  context_window_hours?: number | null;
   is_paused?: boolean;
   pause_until?: string;
   is_static?: boolean;
@@ -175,10 +176,10 @@ export function newPersonaFromYAML(yamlContent: string, allTools?: ToolDefinitio
     groups_visible: groupsVisible.length > 0 ? groupsVisible : ["General"],
     traits,
     topics,
-    heartbeat_delay_ms: data.heartbeat_delay_ms === 'default' || data.heartbeat_delay_ms === undefined
+    heartbeat_delay_ms: data.heartbeat_delay_ms == null
       ? undefined
-      : Number(data.heartbeat_delay_ms) || undefined,
-    context_window_hours: data.context_window_hours,
+      : parseDuration(data.heartbeat_delay_ms) ?? undefined,
+    context_window_hours: data.context_window_hours ?? undefined,
     tools: resolvePersonaToolsFromMap(data.tools, allTools ?? [], allProviders ?? []),
   };
 }
@@ -216,8 +217,8 @@ export function personaToYAML(persona: PersonaEntity, allGroups?: string[], allT
       : persona.topics.map(({ name, perspective, approach, personal_stake, exposure_current, exposure_desired }) => ({
           name, perspective, approach, personal_stake, exposure_current, exposure_desired
         })),
-    heartbeat_delay_ms: persona.heartbeat_delay_ms || 'default',
-    context_window_hours: persona.context_window_hours,
+    heartbeat_delay_ms: persona.heartbeat_delay_ms ? formatDuration(persona.heartbeat_delay_ms) : null,
+    context_window_hours: persona.context_window_hours ?? null,
     is_paused: persona.is_paused || undefined,
     pause_until: persona.pause_until,
     is_static: persona.is_static || undefined,
@@ -324,10 +325,10 @@ export function personaFromYAML(yamlContent: string, original: PersonaEntity, al
     groups_visible: groupsVisible,
     traits,
     topics,
-    heartbeat_delay_ms: data.heartbeat_delay_ms === 'default' || data.heartbeat_delay_ms === undefined
+    heartbeat_delay_ms: data.heartbeat_delay_ms == null
       ? undefined
-      : Number(data.heartbeat_delay_ms) || undefined,
-    context_window_hours: data.context_window_hours,
+      : parseDuration(data.heartbeat_delay_ms) ?? undefined,
+    context_window_hours: data.context_window_hours ?? undefined,
     is_paused: data.is_paused ?? false,
     pause_until: data.pause_until,
     is_static: data.is_static ?? false,