ei-tui 0.1.8 → 0.1.10

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ei-tui",
-  "version": "0.1.8",
+  "version": "0.1.10",
   "author": "Flare576",
   "repository": {
     "type": "git",

package/src/core/handlers/index.ts

@@ -233,6 +233,7 @@ function handleEiHeartbeat(response: LLMResponse, state: StateManager): void {
     timestamp: now,
     read: false,
     context_status: ContextStatus.Default,
+    f: true, r: true, p: true, o: true,
   });
 
   if (found.type === "fact") {
@@ -672,7 +673,7 @@ async function handleHumanItemUpdate(response: LLMResponse, state: StateManager)
   const isEi = personaDisplayName.toLowerCase() === "ei";
 
   const human = state.getHuman();
-  const getExistingItem = (): { learned_by?: string; persona_groups?: string[] } | undefined => {
+  const getExistingItem = (): { learned_by?: string; last_changed_by?: string; persona_groups?: string[] } | undefined => {
     if (isNewItem) return undefined;
     switch (candidateType) {
       case "fact": return human.facts.find(f => f.id === existingItemId);
@@ -710,7 +711,8 @@ async function handleHumanItemUpdate(response: LLMResponse, state: StateManager)
         validated: ValidationLevel.None,
         validated_date: now,
         last_updated: now,
-        learned_by: isNewItem ? personaDisplayName : existingItem?.learned_by,
+        learned_by: isNewItem ? personaId : existingItem?.learned_by,
+        last_changed_by: personaId,
         persona_groups: mergeGroups(existingItem?.persona_groups),
         embedding,
       };
@@ -725,7 +727,8 @@ async function handleHumanItemUpdate(response: LLMResponse, state: StateManager)
         sentiment: result.sentiment,
         strength: (result as any).strength ?? 0.5,
         last_updated: now,
-        learned_by: isNewItem ? personaDisplayName : existingItem?.learned_by,
+        learned_by: isNewItem ? personaId : existingItem?.learned_by,
+        last_changed_by: personaId,
         persona_groups: mergeGroups(existingItem?.persona_groups),
         embedding,
       };
@@ -745,7 +748,8 @@ async function handleHumanItemUpdate(response: LLMResponse, state: StateManager)
         exposure_current: calculateExposureCurrent(exposureImpact),
         exposure_desired: (result as any).exposure_desired ?? 0.5,
         last_updated: now,
-        learned_by: isNewItem ? personaDisplayName : existingItem?.learned_by,
+        learned_by: isNewItem ? personaId : existingItem?.learned_by,
+        last_changed_by: personaId,
         persona_groups: mergeGroups(existingItem?.persona_groups),
         embedding,
       };
@@ -763,7 +767,8 @@ async function handleHumanItemUpdate(response: LLMResponse, state: StateManager)
         exposure_current: calculateExposureCurrent(exposureImpact),
         exposure_desired: (result as any).exposure_desired ?? 0.5,
         last_updated: now,
-        learned_by: isNewItem ? personaDisplayName : existingItem?.learned_by,
+        learned_by: isNewItem ? personaId : existingItem?.learned_by,
+        last_changed_by: personaId,
         persona_groups: mergeGroups(existingItem?.persona_groups),
         embedding,
       };

package/src/core/processor.ts

@@ -565,7 +565,7 @@ export class Processor {
     const items: EiHeartbeatItem[] = [];
 
     const unverifiedFacts = human.facts
-      .filter(f => f.validated === ValidationLevel.None && f.learned_by !== "Ei")
+      .filter(f => f.validated === ValidationLevel.None && f.learned_by !== "ei" && (f.last_changed_by === undefined || f.last_changed_by !== "ei"))
       .slice(0, 5);
     for (const fact of unverifiedFacts) {
       const quote = human.quotes.find(q => q.data_item_ids.includes(fact.id));

package/src/core/state-manager.ts

@@ -36,11 +36,60 @@ export class StateManager {
       this.humanState.load(state.human);
       this.personaState.load(state.personas);
       this.queueState.load(state.queue);
+      this.migrateLearnedByToIds();
     } else {
       this.humanState.load(createDefaultHumanEntity());
     }
   }
 
+  /**
+   * Migration: learned_by used to store display names; now stores persona IDs.
+   * On load, attempt to resolve display names -> IDs using current persona map.
+   * Unresolvable values (renamed/deleted personas) are cleared to avoid stale display.
+   * No-op for already-migrated data (UUIDs or "ei" won't match display names).
+   */
+  private migrateLearnedByToIds(): void {
+    const personas = this.personaState.getAll();
+    const nameToId = new Map<string, string>();
+    for (const p of personas) {
+      nameToId.set(p.display_name.toLowerCase(), p.id);
+      for (const alias of p.aliases ?? []) {
+        nameToId.set(alias.toLowerCase(), p.id);
+      }
+    }
+    // "Ei" display name -> "ei" id (hardcoded, always valid)
+    nameToId.set("ei", "ei");
+
+    const human = this.humanState.get();
+    let dirty = false;
+    const migrateItem = (item: { learned_by?: string; last_changed_by?: string }) => {
+      if (item.learned_by && !this.isPersonaId(item.learned_by)) {
+        const resolved = nameToId.get(item.learned_by.toLowerCase());
+        item.learned_by = resolved ?? undefined;  // clear if unresolvable
+        dirty = true;
+      }
+      if (item.last_changed_by && !this.isPersonaId(item.last_changed_by)) {
+        const resolved = nameToId.get(item.last_changed_by.toLowerCase());
+        item.last_changed_by = resolved ?? undefined;
+        dirty = true;
+      }
+    };
+    [...human.facts, ...human.traits, ...human.topics, ...human.people].forEach(migrateItem);
+    if (dirty) {
+      this.humanState.set(human);
+      console.log("[StateManager] Migrated learned_by fields from display names to persona IDs");
+    }
+  }
+
+  /**
+   * Returns true if value looks like a persona ID (UUID or the special "ei" id).
+   * Display names are free-form strings that won't match UUID format.
+   */
+  private isPersonaId(value: string): boolean {
+    if (value === "ei") return true;
+    return /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i.test(value);
+  }
+
   private buildStorageState(): StorageState {
     return {
       version: 1,

package/src/core/types.ts

@@ -62,7 +62,8 @@ export interface DataItemBase {
   description: string;
   sentiment: number;
   last_updated: string;
-  learned_by?: string;           // Persona ID that learned this item
+  learned_by?: string;           // Persona ID that originally learned this item (stable UUID)
+  last_changed_by?: string;      // Persona ID that most recently updated this item (stable UUID)
   persona_groups?: string[];
   embedding?: number[];
 }

package/src/integrations/opencode/importer.ts

@@ -272,7 +272,8 @@ async function ensureSessionTopic(
   const existingTopic = human.topics.find((t) => t.id === session.id);
 
   const firstAgent = await reader.getFirstAgent(session.id);
-  const learnedBy = firstAgent ?? "build";
+  const firstPersona = firstAgent ? stateManager.persona_getByName(firstAgent) : null;
+  const learnedBy = firstPersona?.id ?? firstAgent ?? "build";
 
   if (existingTopic) {
     if (existingTopic.name !== session.title) {

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

@@ -350,7 +350,10 @@ export function humanToYAML(human: HumanEntity): string {
   
   return YAML.stringify(data, {
     lineWidth: 0,
-  }).replace(/^(\s+validated:\s+\S+)$/mg, '$1 # none | ei | human');
+  })
+  .replace(/^(\s+validated:\s+\S+)$/mg, '$1 # none | ei | human')
+  .replace(/^(\s+)(learned_by: .+)$/mg, '$1# [read-only] $2')
+  .replace(/^(\s+)(last_changed_by: .+)$/mg, '$1# [read-only] $2');
 }
 
 export interface HumanYAMLResult {
@@ -365,7 +368,12 @@ export interface HumanYAMLResult {
 }
 
 export function humanFromYAML(yamlContent: string): HumanYAMLResult {
-  const data = YAML.parse(yamlContent) as EditableHumanData;
+  // Strip read-only comment lines before parsing so users can't accidentally corrupt them
+  const stripped = yamlContent
+    .split('\n')
+    .filter(line => !/^\s*#\s*\[read-only\]/.test(line))
+    .join('\n');
+  const data = YAML.parse(stripped) as EditableHumanData;
   
   const deletedFactIds: string[] = [];
   const deletedTraitIds: string[] = [];