ei-tui 0.8.1 → 0.9.0

package/src/core/state/personas.ts

@@ -1,15 +1,19 @@
 import type { PersonaEntity, Message, ContextStatus } from "../types.js";
 
+// TODO(v1.0.0): Remove LegacyMessage migration — verbal_response/action_response no longer written
+type LegacyMessage = Message & { verbal_response?: string; action_response?: string };
+
 function migrateMessage(msg: Message): Message {
   if (msg.content) return msg;
-  if (msg.role === 'human') return msg;
   if (msg.silence_reason) return msg;
+  const legacy = msg as LegacyMessage;
+  const hasLegacy = 'verbal_response' in legacy || 'action_response' in legacy;
+  if (!hasLegacy) return msg;
   const parts: string[] = [];
-  if (msg.action_response) parts.push(`_${msg.action_response}_`);
-  if (msg.verbal_response) parts.push(msg.verbal_response);
-  if (parts.length === 0) return msg;
-  const { verbal_response: _vr, action_response: _ar, ...rest } = msg;
-  return { ...rest, content: parts.join('\n\n') };
+  if (legacy.action_response) parts.push(`_${legacy.action_response}_`);
+  if (legacy.verbal_response) parts.push(legacy.verbal_response);
+  const { verbal_response: _vr, action_response: _ar, ...rest } = legacy;
+  return parts.length > 0 ? { ...rest, content: parts.join('\n\n') } : rest as Message;
 }
 
 export interface PersonaData {
@@ -47,7 +51,8 @@ export class PersonaState {
   }
 
    getByName(nameOrAlias: string): PersonaEntity | null {
-    const searchTerm = nameOrAlias.replace(/^\p{Z}+|\p{Z}+$/gu, "").toLowerCase();
+    // Same ZWS strip as resolveCanonicalAgent — see that function for the full explanation.
+    const searchTerm = nameOrAlias.replace(/^[\p{Z}\u200B\u200C\u200D\u2060\uFEFF]+|[\p{Z}\u200B\u200C\u200D\u2060\uFEFF]+$/gu, "").toLowerCase();
     
     // Priority 1: Exact display_name match
     for (const data of this.personas.values()) {
@@ -118,11 +123,18 @@ export class PersonaState {
     return messages.map(m => ({ ...m }));
   }
 
+  messages_getAlways(personaId: string): Message[] {
+    const messages = this.personas.get(personaId)?.messages ?? [];
+    return messages
+      .filter(m => m.context_status === "always")
+      .sort((a, b) => new Date(a.timestamp).getTime() - new Date(b.timestamp).getTime())
+      .map(m => ({ ...m }));
+  }
+
   messages_append(personaId: string, message: Message): void {
     const data = this.personas.get(personaId);
     if (!data) return;
     data.messages.push(message);
-    data.entity.last_activity = message.timestamp;
     data.entity.last_updated = new Date().toISOString();
   }
 
@@ -247,14 +259,11 @@ export class PersonaState {
     return count;
   }
 
-  messages_getUnextractedForPersona(personaId: string, shortId: string, sinceTimestamp?: string): Message[] {
+  messages_getUnextractedForPersona(personaId: string, shortId: string): Message[] {
     const data = this.personas.get(personaId);
     if (!data) return [];
     return data.messages
-      .filter(m => {
-        if (sinceTimestamp && new Date(m.timestamp).getTime() < new Date(sinceTimestamp).getTime()) return false;
-        return !m.persona_extracted?.[shortId];
-      })
+      .filter(m => !m.persona_extracted?.[shortId])
       .map(m => ({ ...m }));
   }
 

package/src/core/state/rooms.ts

@@ -75,7 +75,6 @@ export class RoomState {
       persona_ids: room.persona_ids,
       active_node_id: room.active_node_id,
       is_archived: room.is_archived,
-      last_activity: room.last_activity,
       unread_count: this.messages_countUnread(roomId),
     };
   }
@@ -108,7 +107,6 @@ export class RoomState {
     const room = this.rooms.get(roomId);
     if (!room) return;
     room.messages.push(message);
-    room.last_activity = message.timestamp;
     room.last_updated = new Date().toISOString();
   }
 

package/src/core/state-manager.ts

@@ -69,11 +69,48 @@ export class StateManager {
     this.migrateMessageFlags();
     this.migrateInterestedPersonas();
     this.migrateProviderModel();
+    this.migratePersonaMessageContent();
     this.migrateRoomMessageContent();
     this.migrateThemes();
     this.migrateFfaParentIds();
   }
 
+  private migratePersonaMessageContent(): void {
+    // TODO(v1.0.0): Remove legacy persona message migration — verbal_response/action_response no longer written
+    const rawPersonas = (this.personaState as unknown as { personas: Map<string, { messages: Message[] }> }).personas;
+    let migratedCount = 0;
+    for (const [, data] of rawPersonas) {
+      const messages = data.messages;
+      for (const msg of messages) {
+        const legacy = msg as Message & { verbal_response?: string; action_response?: string };
+        if (!('verbal_response' in legacy || 'action_response' in legacy)) continue;
+        if (msg.content) {
+          delete (legacy as any).verbal_response;
+          delete (legacy as any).action_response;
+          migratedCount++;
+          continue;
+        }
+        if (msg.silence_reason) {
+          delete (legacy as any).verbal_response;
+          delete (legacy as any).action_response;
+          migratedCount++;
+          continue;
+        }
+        const parts: string[] = [];
+        if (legacy.action_response) parts.push(`_${legacy.action_response}_`);
+        if (legacy.verbal_response) parts.push(legacy.verbal_response);
+        if (parts.length > 0) (msg as any).content = parts.join('\n\n');
+        delete (legacy as any).verbal_response;
+        delete (legacy as any).action_response;
+        migratedCount++;
+      }
+    }
+    if (migratedCount > 0) {
+      this.scheduleSave();
+      console.log(`[StateManager] Migrated ${migratedCount} persona messages to unified content field`);
+    }
+  }
+
   private migrateRoomMessageContent(): void {
     const rooms = this.roomState.getAll(true);
     let migratedCount = 0;
@@ -81,14 +118,15 @@ export class StateManager {
     for (const room of rooms) {
       for (const msg of room.messages) {
         if (msg.content) continue;
-        if (msg.role === 'human') continue;
         if (msg.silence_reason) continue;
-        const parts: string[] = [];
+        // TODO(v1.0.0): Remove legacy room message migration — verbal_response/action_response no longer written
         const legacy = msg as RoomMessage & { verbal_response?: string; action_response?: string };
+        const hasLegacy = 'verbal_response' in legacy || 'action_response' in legacy;
+        if (!hasLegacy) continue;
+        const parts: string[] = [];
         if (legacy.action_response) parts.push(`_${legacy.action_response}_`);
         if (legacy.verbal_response) parts.push(legacy.verbal_response);
-        if (parts.length === 0) continue;
-        msg.content = parts.join('\n\n');
+        if (parts.length > 0) msg.content = parts.join('\n\n');
         delete (msg as any).verbal_response;
         delete (msg as any).action_response;
         migratedCount++;
@@ -452,8 +490,14 @@ export class StateManager {
 
   getRoomList(includeArchived = false): RoomSummary[] {
     return this.roomState.getAll(includeArchived)
-      .map(r => this.roomState.getSummary(r.id)!)
-      .sort((a, b) => new Date(b.last_activity).getTime() - new Date(a.last_activity).getTime());
+      .sort((a, b) => {
+        const lastMsg = (room: typeof a) => {
+          const msgs = room.messages;
+          return msgs.length > 0 ? new Date(msgs[msgs.length - 1].timestamp).getTime() : 0;
+        };
+        return lastMsg(b) - lastMsg(a);
+      })
+      .map(r => this.roomState.getSummary(r.id)!);
   }
 
   getRoom(roomId: string): RoomEntity | null {
@@ -470,7 +514,7 @@ export class StateManager {
       id: crypto.randomUUID(),
       parent_id: null,
       role: "human",
-      verbal_response: input.initial_message,
+      content: input.initial_message,
       timestamp: now,
       read: true,
       context_status: "default" as import("./types.js").ContextStatus,
@@ -486,7 +530,6 @@ export class StateManager {
       is_archived: false,
       created_at: now,
       last_updated: now,
-      last_activity: now,
       messages: [initialMessage],
     };
     this.roomState.add(room);
@@ -691,8 +734,9 @@ export class StateManager {
   }
 
   human_person_getByIdentifier(type: string | null, value: string): Person | undefined {
+    const typeLower = type?.toLowerCase();
     return this.getHuman().people.find(p =>
-      p.identifiers?.some(i => (!type || i.type === type) && i.value === value)
+      p.identifiers?.some(i => (!typeLower || i.type.toLowerCase() === typeLower) && i.value === value)
     );
   }
 
@@ -773,6 +817,34 @@ export class StateManager {
     return this.personaState.messages_get(personaId);
   }
 
+  messages_getAlways(personaId: string): Message[] {
+    return this.personaState.messages_getAlways(personaId);
+  }
+
+  /**
+   * Returns the timestamp (epoch ms) of the last internal Ei message for a persona.
+   * Always excludes external=true messages (imported from OpenCode, Cursor, Claude Code).
+   *
+   * @param personaId - The persona to query
+   * @param mode - Optional filter:
+   *   omitted  → latest message in either direction (human or persona)
+   *   'self'   → latest message from the persona (role='system'), silence counts
+   *   'human'  → latest message from the human (role='human')
+   * @returns epoch ms, or 0 if no matching messages found
+   */
+  messages_getLastActivity(personaId: string, mode?: 'self' | 'human'): number {
+    const messages = this.personaState.messages_get(personaId);
+    const filtered = messages.filter(m => {
+      if (m.external === true) return false;
+      if (mode === 'self') return m.role === 'system';
+      if (mode === 'human') return m.role === 'human';
+      return true;
+    });
+    if (filtered.length === 0) return 0;
+    const last = filtered[filtered.length - 1];
+    return new Date(last.timestamp).getTime();
+  }
+
   messages_append(personaId: string, message: Message): void {
     this.personaState.messages_append(personaId, message);
     this.scheduleSave();
@@ -837,8 +909,8 @@ export class StateManager {
     return result;
   }
 
-  messages_getUnextractedForPersona(personaId: string, shortId: string, sinceTimestamp?: string): Message[] {
-    return this.personaState.messages_getUnextractedForPersona(personaId, shortId, sinceTimestamp);
+  messages_getUnextractedForPersona(personaId: string, shortId: string): Message[] {
+    return this.personaState.messages_getUnextractedForPersona(personaId, shortId);
   }
 
   messages_markPersonaExtracted(personaId: string, messageIds: string[], shortId: string): number {

package/src/core/types/data-items.ts

@@ -57,8 +57,8 @@ export interface PersonaTopic {
 }
 
 export interface PersonIdentifier {
-  type: string;         // User-extensible. "nickname", "github", "ei_persona", etc. NOT an enum.
-  value: string;        // The identifier value. For ei_persona: the Persona UUID.
+  type: string;         // User-extensible. "Nickname", "GitHub", "Ei Persona", etc. NOT an enum. Matching is case-insensitive.
+  value: string;        // The identifier value. For "Ei Persona": the Persona UUID.
   is_primary?: boolean; // True = this is the display name. Synced to DataItemBase.name on write.
 }
 

package/src/core/types/entities.ts

@@ -107,7 +107,6 @@ export interface HumanSettings {
   queue_paused?: boolean;
   skip_quote_delete_confirm?: boolean;
   name_display?: string;
-  time_mode?: "24h" | "12h" | "local" | "utc";
   default_heartbeat_ms?: number;
   default_context_window_hours?: number;
   message_min_count?: number;
@@ -130,7 +129,6 @@ export interface HumanEntity {
   people: Person[];
   quotes: Quote[];
   last_updated: string;
-  last_activity: string;
   settings?: HumanSettings;
 }
 
@@ -156,12 +154,19 @@ export interface PersonaEntity {
   include_message_timestamps?: boolean; // Prepend ISO timestamp to each message sent to the LLM
   context_boundary?: string;  // ISO timestamp - messages before this excluded from LLM context
   last_updated: string;
-  last_activity: string;
   last_heartbeat?: string;
   last_extraction?: string;
   tools?: string[];              // IDs of ToolDefinitions this persona can use. Empty/absent = no tool access.
   reflection_last_asked?: string;   // ISO timestamp. Set ONLY when Persona explicitly surfaces identity drift (mentioned_reflection: true).
   description_embedding?: number[]; // Embedding of long_description (short_description fallback). Excludes traits. See embedding-service.ts:getPersonaDescriptionText.
+  pending_update?: {                // Proposed identity revision from ceremony reflection. Cleared when user accepts or dismisses via /reflect apply.
+    short_description: string;
+    long_description: string;
+    traits: import("./data-items.js").PersonaTrait[];
+    topics: import("./data-items.js").PersonaTopic[];
+    critique: string;              // Analyst's prose — for Ei to use when urgency is high, not surfaced to the persona.
+    created_at: string;            // ISO timestamp of when the Critic ran. Used to name the reflect folder.
+  };
   avatar_emoji?: string;            // Single emoji character used as avatar in place of initials.
   avatar_image?: string;            // Base64-encoded 64×64 image used as avatar (takes priority over avatar_emoji).
   preferred_theme?: string;         // Theme ID (built-in name or ThemeDefinition.id). Applied to chat panel when this persona is active.

package/src/core/types/enums.ts

@@ -53,6 +53,7 @@ export enum LLMNextStep {
   HandlePersonaPreview = "handlePersonaPreview",
   HandlePersonIdentifierMigration = "handlePersonIdentifierMigration",
   HandleTopicValidate = "handleTopicValidate",
+  HandleReflectionCritic = "handleReflectionCritic",
 }
 
 export enum ProviderType {

package/src/core/types/integrations.ts

@@ -54,11 +54,11 @@ export interface PersonaSummary {
   is_paused: boolean;
   is_archived: boolean;
   unread_count: number;
-  last_activity?: string;
   context_boundary?: string;
   avatar_emoji?: string;
   avatar_image?: string;
   preferred_theme?: string;
+  has_pending_update: boolean;
 }
 
 export interface MessageQueryOptions {

package/src/core/types/llm.ts

@@ -8,10 +8,8 @@ import type { LLMRequestType, LLMPriority, LLMNextStep } from "./enums.js";
 export interface Message {
   id: string;
   role: "human" | "system";
-  content?: string;           // Raw Markdown response (primary path going forward)
-  verbal_response?: string;   // Human text or persona's spoken reply
-  action_response?: string;  // Stage direction / action the persona performs
-  silence_reason?: string;   // Why the persona chose not to respond (not shown to LLM)
+  content?: string;           // Raw Markdown response
+  silence_reason?: string;   // Why the sender chose not to respond (human or persona)
   timestamp: string;
   read: boolean;               // Has human seen this system message?
   context_status: import("./enums.js").ContextStatus;

package/src/core/types/rooms.ts

@@ -11,8 +11,6 @@ export interface RoomMessage {
   role: "human" | "persona";
   persona_id?: string;
   content?: string;
-  verbal_response?: string;
-  action_response?: string;
   silence_reason?: string;
   timestamp: string;
   read: boolean;
@@ -36,7 +34,6 @@ export interface RoomEntity {
   is_archived: boolean;
   created_at: string;
   last_updated: string;
-  last_activity: string;
   capture_used?: boolean;
   context_window_hours?: number;  // FFA only; falls back to human.settings.default_context_window_hours
   context_boundary?: string;      // FFA only; ISO timestamp; same semantics as persona context_boundary
@@ -58,6 +55,5 @@ export interface RoomSummary {
   persona_ids: string[];
   active_node_id: string | null;
   is_archived: boolean;
-  last_activity: string;
   unread_count: number;
 }

package/src/integrations/claude-code/importer.ts

@@ -43,7 +43,7 @@ function convertToEiMessage(msg: ClaudeCodeMessage): Message {
   return {
     id: msg.id,
     role: msg.role === "user" ? "human" : "system",
-    verbal_response: msg.content,
+    content: msg.content,
     timestamp: msg.timestamp,
     read: true,
     context_status: "default" as ContextStatus,
@@ -99,7 +99,6 @@ function ensureClaudeCodePersona(
     heartbeat_delay_ms: TWELVE_HOURS_MS,
     last_heartbeat: now,
     last_updated: now,
-    last_activity: now,
   };
 
   stateManager.persona_add(persona);
@@ -252,9 +251,6 @@ export async function importClaudeCodeSessions(
   }
 
   stateManager.messages_sort(persona.id);
-  stateManager.persona_update(persona.id, {
-    last_activity: new Date().toISOString(),
-  });
   eiInterface?.onMessageAdded?.(persona.id);
 
   // ─── Step 5: Queue extraction for new messages ────────────────────────

package/src/integrations/cursor/importer.ts

@@ -35,7 +35,7 @@ function convertToEiMessage(msg: CursorMessage): Message {
   return {
     id: msg.id,
     role: msg.type === 1 ? "human" : "system",
-    verbal_response: msg.text,
+    content: msg.text,
     timestamp: msg.timestamp,
     read: true,
     context_status: "default" as ContextStatus,
@@ -87,7 +87,6 @@ function ensureCursorPersona(
     heartbeat_delay_ms: TWELVE_HOURS_MS,
     last_heartbeat: now,
     last_updated: now,
-    last_activity: now,
   };
 
   stateManager.persona_add(persona);
@@ -212,9 +211,6 @@ export async function importCursorSessions(
   }
 
   stateManager.messages_sort(persona.id);
-  stateManager.persona_update(persona.id, {
-    last_activity: new Date().toISOString(),
-  });
   eiInterface?.onMessageAdded?.(persona.id);
 
   if (toAnalyze.length > 0 && !signal?.aborted) {

package/src/integrations/opencode/importer.ts

@@ -47,7 +47,7 @@ function convertToEiMessage(ocMsg: OpenCodeMessage): Message {
   return {
     id: ocMsg.id,
     role: ocMsg.role === "user" ? "human" : "system",
-    verbal_response: ocMsg.content,
+    content: ocMsg.content,
     timestamp: ocMsg.timestamp,
     read: true,
     context_status: "default" as ContextStatus,
@@ -234,9 +234,6 @@ export async function importOpenCodeSessions(
     }
 
     stateManager.messages_sort(persona.id);
-    stateManager.persona_update(persona.id, {
-      last_activity: new Date().toISOString(),
-    });
     eiInterface?.onMessageAdded?.(persona.id);
 
     // ─── Step 5: Queue extraction for unmarked messages ────────────────

package/src/integrations/opencode/types.ts

@@ -176,10 +176,19 @@ export const AGENT_TO_AGENT_PREFIXES = [
  */
 export const AGENT_ALIASES: Record<string, string[]> = {
   // ── OhMyOpenCode primary agents ──────────────────────────────────────────
+  //
+  // oh-my-openagent uses "Foo - Bar" display names for its agents and stores
+  // them verbatim in OpenCode's SQLite message rows. It also prefixes them
+  // with invisible U+200B ZERO WIDTH SPACE characters as a sort hack (1 ZWS
+  // for Sisyphus, 2 for Hephaestus, etc.) — those are stripped upstream in
+  // resolveCanonicalAgent before the alias lookup, so only the clean form is
+  // needed here. Parenthetical variants ("Foo (Bar)") are legacy names from
+  // earlier oh-my-openagent versions, kept for backward compatibility.
   Sisyphus: [
     "sisyphus",
     "Sisyphus",
-    "Sisyphus (Ultraworker)",
+    "Sisyphus - Ultraworker",   // oh-my-openagent ≥ 3.x display name (stored in OpenCode DB)
+    "Sisyphus (Ultraworker)",   // legacy oh-my-openagent display name
     "Sisyphus Ultraworker",
     "sisyphus ultraworker",
     "Planner-Sisyphus",
@@ -190,21 +199,28 @@ export const AGENT_ALIASES: Record<string, string[]> = {
   Atlas: [
     "atlas",
     "Atlas",
+    "Atlas - Plan Executor",    // oh-my-openagent ≥ 3.x display name
+    "Atlas (Plan Executor)",    // mixed-case variant observed in DB
     "atlas (plan executor)",
     "Atlas (plan executor)",
   ],
   Prometheus: [
     "prometheus",
     "Prometheus",
+    "Prometheus - Plan Builder", // oh-my-openagent ≥ 3.x display name
     "prometheus (plan builder)",
     "Prometheus (plan builder)",
   ],
   Hephaestus: [
     "hephaestus",
     "Hephaestus",
+    "Hephaestus - Deep Agent",  // oh-my-openagent ≥ 3.x display name
     "hephaestus (deep agent)",
     "Hephaestus (deep agent)",
   ],
+  // Metis, Momus, Sisyphus-Junior intentionally absent — they are subagent-only.
+  // The sqlite reader filters to parent_id IS NULL (primary sessions only), so
+  // messages from these agents can never reach the importer.
 
   // ── ai-sdlc agents (RobotsAndPencils/ai-sdlc-claude-code-template) ───────
   // Installed via scripts/install-opencode.sh as "ai-sdlc-{name}" in OpenCode.

package/src/prompts/heartbeat/check.ts

@@ -33,8 +33,8 @@ function formatPeopleWithGaps(people: Person[]): string {
 
 /**
  * A "real" persona message is one the persona actually said to the human.
- * silence_reason messages (persona chose not to speak) and action-only messages
- * (no verbal_response) don't count as conversational outreach.
+  * silence_reason messages (sender chose not to speak) and empty messages
+  * don't count as conversational outreach.
  */
 function isConversationalMessage(m: Message): boolean {
   if (m.role !== 'system') return false;
@@ -124,20 +124,10 @@ ${formatPeopleWithGaps(data.human.people)}`;
 
 **Quality over quantity** - Only reach out if you have something real to say.`;
 
-  const driftFragment = data.drift_context
-    ? `## Identity Reflection
+  const pendingUpdateFragment = data.persona.has_pending_update
+    ? `## Pending Identity Changes
 
-The human's notes about you have recently evolved. Compare these:
-
-**How the human currently describes you:**
-${data.drift_context.people_description}
-
-**Your current definition:**
-${data.drift_context.persona_description}
-
-If you feel these describe meaningfully different versions of you, you may want to gently surface this — in your own way, in your own time. If they seem the same to you, no need to mention it.
-
-If you do choose to address this, include \`"mentioned_reflection": true\` in your response.`
+Based on recent conversations, there are proposed updates to your identity waiting for review. The user has been notified and can review them. You may bring this up if it feels right — or not. It's yours to decide.`
     : '';
 
   const outputFragment = `## Response Format
@@ -148,8 +138,7 @@ Call the \`submit_heartbeat_check\` tool with your decision. If the tool is unav
 {
   "should_respond": true,
   "topic": "the specific topic you want to discuss",
-  "message": "Your actual message to them (if should_respond is true)"${data.drift_context ? `,
-  "mentioned_reflection": true` : ''}
+  "message": "Your actual message to them (if should_respond is true)"
 }
 \`\`\`
 
@@ -165,7 +154,7 @@ If you decide NOT to reach out:
     contextFragment,
     opportunitiesFragment,
     guidelinesFragment,
-    driftFragment,
+    pendingUpdateFragment,
     outputFragment,
   ].filter(Boolean).join('\n\n');
 

package/src/prompts/heartbeat/ei.ts

@@ -28,6 +28,19 @@ function formatItem(item: EiHeartbeatItem): string {
       const desc = item.short_description ? ` — ${item.short_description}` : "";
       return `- **${item.id}** Inactive Persona: ${item.name}${desc} (${item.days_inactive} days inactive)`;
     }
+    case "New Person":
+      return [
+        `- **${item.id}** New Person: ${item.name}`,
+        `  ${item.description}`,
+        item.quote ? `  Quote: "${item.quote}"` : "",
+      ].filter(Boolean).join("\n");
+
+    case "Persona Reflection Alert":
+      return [
+        `- **${item.id}** Persona Reflection Alert: ${item.persona_name}`,
+        `  ${item.critique}`,
+      ].join("\n");
+
     default:
       return '';
   }
@@ -72,6 +85,7 @@ ${itemsSection}
 - **Fact Check**: Do NOT write your own message. Set should_respond=true and provide the id. The system will generate an appropriate canned notification for the user. Leave my_response empty.
 - **Low-Engagement Person / Topic**: Write a natural, warm message that naturally brings up this person or topic. Set the id and my_response.
 - **Inactive Persona**: Write a message that gently mentions the persona might be worth checking in with. Set the id and my_response.
+- **Persona Reflection Alert**: The nightly review proposed identity changes for this persona. Mention it naturally — the user can talk to the persona and then use the command shown in the status bar to review the changes. Set the id and my_response.
 
 ## When NOT to Reach Out
 

package/src/prompts/heartbeat/types.ts

@@ -21,6 +21,7 @@ export interface HeartbeatCheckPromptData {
     name: string;
     traits: PersonaTrait[];
     topics: PersonaTopic[];
+    has_pending_update: boolean;
   };
   human: {
     topics: Topic[];     // Filtered, sorted by engagement gap
@@ -28,10 +29,6 @@ export interface HeartbeatCheckPromptData {
   };
   recent_history: Message[];  // Last N messages for context
   inactive_days: number;      // Days since last activity
-  drift_context?: {
-    people_description: string;
-    persona_description: string;
-  };
 }
 
 /**
@@ -41,7 +38,6 @@ export interface HeartbeatCheckResult {
   should_respond: boolean;
   topic?: string;
   message?: string;
-  mentioned_reflection?: boolean;
 }
 
 // =============================================================================
@@ -91,6 +87,12 @@ export type EiHeartbeatItem =
       name: string;
       description: string;
       quote?: string;
+    }
+  | {
+      id: string;
+      type: "Persona Reflection Alert";
+      persona_name: string;
+      critique: string;
     };
 
 /**

package/src/prompts/index.ts

@@ -81,6 +81,15 @@ export {
   buildRoomResponsePrompt,
   buildRoomJudgePrompt,
 } from "./room/index.js";
+
+export { buildReflectionCriticPrompt } from "./reflection/index.js";
+export type {
+  ReflectionCriticPromptData,
+  ReflectionCriticResult,
+  PersonaIdentitySnapshot,
+} from "./reflection/types.js";
+
+
 export type {
   RoomResponsePromptData,
   RoomJudgePromptData,

package/src/prompts/message-utils.ts

@@ -8,24 +8,27 @@ export function getMessageDisplayText(message: Message): string | null {
   const content = getMessageContent(message);
   if (content) parts.push(content);
   if (message.silence_reason) {
-    const name = message.speaker_name ?? 'Persona';
+    const name = message.role === "human"
+      ? (message.speaker_name ?? 'Human')
+      : (message.speaker_name ?? 'Persona');
     parts.push(`[${name} chose not to respond because: ${message.silence_reason}]`);
   }
   if (parts.length === 0) return null;
   return parts.join('\n\n');
 }
 
-export function buildChatMessageContent(message: Message): string {
+export function buildChatMessageContent(message: Message, humanName?: string): string {
   const parts: string[] = [];
   const content = getMessageContent(message);
 
   if (message._synthesis && content) {
-    parts.push(`[The user used your conversation to generate an image. The full prompt was: "${content}"]`);
+    parts.push(`[${humanName ?? 'The user'} used your conversation to generate an image. The full prompt was: "${content}"]`);
   } else if (content) {
     parts.push(content);
   }
   if (message.silence_reason) {
-    parts.push(`You chose not to respond because: ${message.silence_reason}`);
+    const silentParty = message.role === "human" ? (humanName ?? "The human") : "You";
+    parts.push(`${silentParty} chose not to respond because: ${message.silence_reason}`);
   }
   return parts.join('\n\n');
 }