ei-tui 1.1.0 → 1.2.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ei-tui",
-  "version": "1.1.0",
+  "version": "1.2.0",
   "author": "Flare576",
   "repository": {
     "type": "git",
@@ -51,28 +51,7 @@
     "test:e2e:tui": "cd tui && npm run test:e2e",
     "test:e2e:ui": "playwright test --ui",
     "test:e2e:debug": "playwright test --debug",
-    "test:evals": "vite-node tests/evals/reflection-critic.eval.ts",
-    "test:evals:observe": "vite-node tests/evals/reflection-critic.observe.ts",
-    "test:evals:fact-find": "vite-node tests/evals/fact-find.eval.ts",
-    "test:evals:topic-scan": "vite-node tests/evals/topic-scan.eval.ts",
-    "test:evals:topic-match": "vite-node tests/evals/topic-match.eval.ts",
-    "test:evals:topic-update": "vite-node tests/evals/topic-update.eval.ts",
-    "test:evals:topic-technical": "vite-node tests/evals/topic-technical.eval.ts",
-    "test:evals:rewrite-scan": "vite-node tests/evals/rewrite-scan.eval.ts",
-    "test:evals:rewrite-rewrite": "vite-node tests/evals/rewrite-rewrite.eval.ts",
-    "test:evals:rewrite-real-data": "vite-node tests/evals/rewrite-real-data.eval.ts",
-    "test:evals:topic-validate": "vite-node tests/evals/topic-validate.eval.ts",
-    "test:evals:person-scan": "vite-node tests/evals/person-scan.eval.ts",
-    "test:evals:person-scan-confidence": "vite-node tests/evals/person-scan-confidence.eval.ts",
-
-    "test:evals:person-update": "vite-node tests/evals/person-update.eval.ts",
-    "test:evals:persona-trait": "vite-node tests/evals/persona-trait-extraction.eval.ts",
-    "test:evals:dedup": "vite-node tests/evals/dedup-tool-calls.eval.ts",
-    "test:evals:response-read-memory": "vite-node tests/evals/response-read-memory.eval.ts",
-    "test:evals:response-pending-update": "vite-node tests/evals/response-pending-update.eval.ts",
-    "test:evals:heartbeat-pending-update": "vite-node tests/evals/heartbeat-pending-update.eval.ts",
-    "test:evals:real-data": "vite-node tests/evals/real-data-example.eval.ts",
-    "test:evals:persona-data-check": "vite-node tests/evals/persona-data-check.eval.ts",
+    "test:evals": "vite-node tests/evals/run.ts",
     "test:all": "npm run test && npm run test:e2e && npm run test:e2e:tui",
     "typecheck": "tsc --noEmit",
     "web": "cd web && npm run dev",

package/src/core/handlers/dedup.ts

@@ -25,8 +25,7 @@ export async function handleDedupCurate(
   
   // Validate entity_type
   if (!entity_type || !['topic', 'person'].includes(entity_type)) {
-    console.error(`[Dedup] Invalid entity_type: "${entity_type}" (from request data)`, response.request.data);
-    return;
+    throw new Error(`[Dedup] Invalid entity_type: "${entity_type}" (from request data)`);
   }
   
   // Parse Opus response
@@ -37,14 +36,12 @@ export async function handleDedupCurate(
       throw new Error("Invalid response format");
     }
   } catch (err) {
-    console.error(`[Dedup] Failed to parse Opus response:`, err);
-    return;
+    throw new Error(`[Dedup] Failed to parse Opus response: ${err instanceof Error ? err.message : String(err)}`);
   }
   
   // Validate response structure
   if (!Array.isArray(decisions.update) || !Array.isArray(decisions.remove) || !Array.isArray(decisions.add)) {
-    console.error(`[Dedup] Invalid response structure - missing update/remove/add arrays`);
-    return;
+    throw new Error(`[Dedup] Invalid response structure - missing update/remove/add arrays`);
   }
   
   console.log(`[Dedup] Processing cluster: ${decisions.update.length} updates, ${decisions.remove.length} removals, ${decisions.add.length} additions`);
@@ -63,15 +60,7 @@ export async function handleDedupCurate(
   
   // Validate entityList exists
   if (!entityList || !Array.isArray(entityList)) {
-    console.error(`[Dedup] entityList is ${entityList === undefined ? 'undefined' : 'not an array'} for entity_type="${entity_type}" (looking for state.${entity_type}s)`, {
-      entity_type,
-      entity_ids,
-      stateKeys: Object.keys(state),
-      factsExists: !!state.facts,
-      topicsExists: !!state.topics,
-      peopleExists: !!state.people
-    });
-    return;
+    throw new Error(`[Dedup] entityList is ${entityList === undefined ? 'undefined' : 'not an array'} for entity_type="${entity_type}" (looking for state.${entity_type}s)`);
   }
   const entities = entity_ids
     .map((id: string) => entityList.find((e: Fact | Topic | Person) => e.id === id))

package/src/core/handlers/document-segmentation.ts

@@ -30,8 +30,7 @@ export function handleDocumentSegmentation(response: LLMResponse, state: StateMa
   };
 
   if (!batchId || !filename) {
-    console.error("[handleDocumentSegmentation] Missing batchId or filename in request data");
-    return;
+    throw new Error("[handleDocumentSegmentation] Missing batchId or filename in request data");
   }
 
   let segments: string[];
@@ -103,7 +102,7 @@ export function finishDocumentBatch(batchId: string, filename: string, state: St
         ...updatedHuman.settings?.document,
         processed_documents: {
           ...(updatedHuman.settings?.document?.processed_documents ?? {}),
-          [filename]: new Date().toISOString(),
+          [filename]: { created_at: new Date().toISOString(), type: "imported" },
         },
       },
     },

package/src/core/handlers/heartbeat.ts

@@ -13,14 +13,12 @@ export function handleHeartbeatCheck(response: LLMResponse, state: StateManager)
   const personaId = response.request.data.personaId as string;
   const personaDisplayName = response.request.data.personaDisplayName as string;
   if (!personaId) {
-    console.error("[handleHeartbeatCheck] No personaId in request data");
-    return;
+    throw new Error("[handleHeartbeatCheck] No personaId in request data");
   }
 
   const result = response.parsed as HeartbeatCheckResult | undefined;
   if (!result) {
-    console.error(`[HeartbeatCheck ${personaDisplayName}] No parsed result`);
-    return;
+    throw new Error(`[HeartbeatCheck ${personaDisplayName}] No parsed result`);
   }
   console.log(`[HeartbeatCheck ${personaDisplayName}] Parsed result - should_respond: ${result.should_respond}, topic: ${result.topic ?? '(none)'}, message: ${result.message ? '(present)' : '(none)'}`);
 
@@ -52,8 +50,7 @@ export function handleHeartbeatCheck(response: LLMResponse, state: StateManager)
 export function handleEiHeartbeat(response: LLMResponse, state: StateManager): void {
   const result = response.parsed as EiHeartbeatResult | undefined;
   if (!result) {
-    console.error("[EiHeartbeat] No parsed result");
-    return;
+    throw new Error("[EiHeartbeat] No parsed result");
   }
   console.log(`[EiHeartbeat] Parsed result - should_respond: ${result.should_respond}, id: ${result.id ?? '(none)'}, my_response: ${result.my_response ? '(present)' : '(none)'}`);
   const now = new Date().toISOString();
@@ -127,8 +124,7 @@ export function handleReflectionCritic(response: LLMResponse, state: StateManage
 
   const result = response.parsed as ReflectionCriticResult | undefined;
   if (!result?.critique) {
-    console.error(`[ReflectionCritic ${personaDisplayName}] Invalid or missing parsed result`);
-    return;
+    throw new Error(`[ReflectionCritic ${personaDisplayName}] Invalid or missing parsed result`);
   }
 
   const personRecord = state.human_person_getByIdentifier("Ei Persona", personaId);
@@ -150,8 +146,7 @@ export function handleReflectionCritic(response: LLMResponse, state: StateManage
 
   const persona = state.persona_getById(personaId);
   if (!persona) {
-    console.error(`[ReflectionCritic ${personaDisplayName}] Persona not found after critic`);
-    return;
+    throw new Error(`[ReflectionCritic ${personaDisplayName}] Persona not found after critic`);
   }
 
   const mergedTopics = result.updated_identity.topics.map(updatedTopic => {

package/src/core/handlers/human-matching.ts

@@ -139,6 +139,8 @@ export async function handleTopicUpdate(response: LLMResponse, state: StateManag
     ? incomingSources
     : [...new Set([...(existingTopic?.sources ?? []), ...incomingSources])];
 
+  const newDescLen = resolvedDescription?.length ?? 0;
+  const existingFloor = existingTopic?.rewrite_length_floor;
   const topic: Topic = {
     id: itemId,
     name: resolvedName,
@@ -156,6 +158,7 @@ export async function handleTopicUpdate(response: LLMResponse, state: StateManag
     sources: sources.length > 0 ? sources : undefined,
     persona_groups: personaGroupsMerged,
     embedding,
+    rewrite_length_floor: existingFloor !== undefined && newDescLen < existingFloor ? existingFloor : undefined,
   };
   state.human_topic_upsert(topic);
 
@@ -333,6 +336,11 @@ export async function handlePersonUpdate(response: LLMResponse, state: StateMana
     sources: personSources.length > 0 ? personSources : undefined,
     persona_groups: personaGroupsMerged,
     embedding,
+    rewrite_length_floor: (() => {
+      const floor = existingPerson?.rewrite_length_floor;
+      const newLen = resolvedDescription?.length ?? 0;
+      return floor !== undefined && newLen < floor ? floor : undefined;
+    })(),
   };
   state.human_person_upsert(person);
 

package/src/core/handlers/index.ts

@@ -16,6 +16,7 @@ import { handleDedupCurate } from "./dedup.js";
 import { handleRoomResponse, handleRoomJudge } from "./rooms.js";
 import { handlePersonaPreview } from "./persona-preview.js";
 import { handleDocumentSegmentation } from "./document-segmentation.js";
+import { handleKnowledgeSynthesis } from "./knowledge-synthesis.js";
 
 export const handlers: Record<LLMNextStep, ResponseHandler> = {
   handlePersonaResponse,
@@ -43,4 +44,5 @@ export const handlers: Record<LLMNextStep, ResponseHandler> = {
   [LLMNextStep.HandleTopicValidate]: handleDedupCurate,
   [LLMNextStep.HandleReflectionCritic]: handleReflectionCritic,
   [LLMNextStep.HandleDocumentSegmentation]: handleDocumentSegmentation,
+  [LLMNextStep.HandleKnowledgeSynthesis]: handleKnowledgeSynthesis,
 };

package/src/core/handlers/knowledge-synthesis.ts

@@ -0,0 +1,50 @@
+import { ContextStatus } from "../types.js";
+import type { LLMResponse, Message } from "../types.js";
+import type { StateManager } from "../state-manager.js";
+
+export function handleKnowledgeSynthesis(response: LLMResponse, state: StateManager): void {
+  const { slug, subject } = response.request.data as {
+    slug: string;
+    subject: string;
+  };
+
+  if (!slug || !subject) {
+    throw new Error("[handleKnowledgeSynthesis] Missing slug or subject in request data");
+  }
+
+  const content = response.content?.trim() ?? "";
+  if (!content) {
+    throw new Error(`[handleKnowledgeSynthesis] Empty or null response content for slug "${slug}"`);
+  }
+
+  const now = new Date().toISOString();
+  const sourceTag = `generate:document:${slug}`;
+
+  const message: Message = {
+    id: crypto.randomUUID(),
+    role: "system",
+    content,
+    timestamp: now,
+    read: true,
+    context_status: ContextStatus.Always,
+    external: true,
+    source_tag: sourceTag,
+  };
+
+  state.messages_append("emmet", message);
+
+  const updatedHuman = state.getHuman();
+  state.setHuman({
+    ...updatedHuman,
+    settings: {
+      ...updatedHuman.settings,
+      document: {
+        ...updatedHuman.settings?.document,
+        processed_documents: {
+          ...(updatedHuman.settings?.document?.processed_documents ?? {}),
+          [slug]: { created_at: now, type: "generated", subject },
+        },
+      },
+    },
+  });
+}

package/src/core/handlers/persona-generation.ts

@@ -12,8 +12,7 @@ export function handlePersonaGeneration(response: LLMResponse, state: StateManag
   const personaId = response.request.data.personaId as string;
   const personaDisplayName = response.request.data.personaDisplayName as string;
   if (!personaId) {
-    console.error("[handlePersonaGeneration] No personaId in request data");
-    return;
+    throw new Error("[handlePersonaGeneration] No personaId in request data");
   }
 
   const result = response.parsed as PersonaGenerationResult | undefined;
@@ -115,14 +114,12 @@ export function handlePersonaTraitExtraction(response: LLMResponse, state: State
   const personaId = response.request.data.personaId as string;
   const personaDisplayName = response.request.data.personaDisplayName as string;
   if (!personaId) {
-    console.error("[handlePersonaTraitExtraction] No personaId in request data");
-    return;
+    throw new Error("[handlePersonaTraitExtraction] No personaId in request data");
   }
 
   const result = response.parsed as TraitResult[] | undefined;
   if (!result || !Array.isArray(result)) {
-    console.error("[handlePersonaTraitExtraction] Invalid parsed result");
-    return;
+    throw new Error("[handlePersonaTraitExtraction] Invalid parsed result");
   }
 
   if (result.length === 0) {
@@ -131,8 +128,7 @@ export function handlePersonaTraitExtraction(response: LLMResponse, state: State
 
   const persona = state.persona_getById(personaId);
   if (!persona) {
-    console.error(`[handlePersonaTraitExtraction] Persona ${personaId} not found`);
-    return;
+    throw new Error(`[handlePersonaTraitExtraction] Persona ${personaId} not found`);
   }
 
   const now = new Date().toISOString();

package/src/core/handlers/persona-response.ts

@@ -15,8 +15,7 @@ export function handlePersonaResponse(response: LLMResponse, state: StateManager
   const personaId = response.request.data.personaId as string;
   const personaDisplayName = response.request.data.personaDisplayName as string;
   if (!personaId) {
-    console.error("[handlePersonaResponse] No personaId in request data");
-    return;
+    throw new Error("[handlePersonaResponse] No personaId in request data");
   }
 
   state.messages_markPendingAsRead(personaId);
@@ -108,7 +107,7 @@ export function handleToolContinuation(response: LLMResponse, state: StateManage
   const originalStep = response.request.data.originalNextStep as LLMNextStep | undefined;
   
   if (!originalStep) {
-    console.error(`[handleToolContinuation] No originalNextStep in data, falling back to handlePersonaResponse`);
+    console.warn(`[handleToolContinuation] No originalNextStep in data, falling back to handlePersonaResponse`);
     handlePersonaResponse(response, state);
     return;
   }
@@ -118,7 +117,7 @@ export function handleToolContinuation(response: LLMResponse, state: StateManage
   const handler = handlers[originalStep];
   
   if (!handler) {
-    console.error(`[handleToolContinuation] No handler found for ${originalStep}, falling back to handlePersonaResponse`);
+    console.warn(`[handleToolContinuation] No handler found for ${originalStep}, falling back to handlePersonaResponse`);
     handlePersonaResponse(response, state);
     return;
   }

package/src/core/handlers/persona-topics.ts

@@ -12,8 +12,7 @@ export function handlePersonaTopicRating(response: LLMResponse, state: StateMana
   const personaId = response.request.data.personaId as string;
   const personaDisplayName = response.request.data.personaDisplayName as string;
   if (!personaId || !personaDisplayName) {
-    console.error("[handlePersonaTopicRating] Missing personaId or personaDisplayName in request data");
-    return;
+    throw new Error("[handlePersonaTopicRating] Missing personaId or personaDisplayName in request data");
   }
 
   const result = response.parsed as PersonaTopicRatingResult | undefined;
@@ -35,8 +34,7 @@ export function handlePersonaTopicRating(response: LLMResponse, state: StateMana
 
   const persona = state.persona_getById(personaId);
   if (!persona) {
-    console.error(`[handlePersonaTopicRating] Persona not found: ${personaDisplayName}`);
-    return;
+    throw new Error(`[handlePersonaTopicRating] Persona not found: ${personaDisplayName}`);
   }
 
   const now = new Date().toISOString();

package/src/core/handlers/rewrite.ts

@@ -20,6 +20,8 @@ import { getEmbeddingService, getItemEmbeddingText } from "../embedding-service.
 
 import { searchHumanData } from "../human-data-manager.js";
 
+const MIN_REWRITE_FLOOR = 750;
+
 /**
  * handleRewriteScan — Phase 1 of Rewrite.
  * LLM returns an array of subject strings found in the bloated item.
@@ -31,20 +33,25 @@ export async function handleRewriteScan(response: LLMResponse, state: StateManag
   const rewriteModel = response.request.data.rewriteModel as string;
 
   if (!itemId || !itemType) {
-    console.error("[handleRewriteScan] Missing itemId or itemType in request data");
-    return;
+    throw new Error("[handleRewriteScan] Missing itemId or itemType in request data");
   }
 
   const subjects = response.parsed as RewriteScanResult | undefined;
   if (!subjects || !Array.isArray(subjects) || subjects.length === 0) {
-    console.log(`[handleRewriteScan] No extra subjects found for ${itemType} "${itemId}" — marking rewrite_checked`);
+    console.log(`[handleRewriteScan] No extra subjects found for ${itemType} "${itemId}" — setting rewrite_length_floor`);
     const human = state.getHuman();
     if (itemType === "topic") {
       const topic = human.topics.find(t => t.id === itemId);
-      if (topic) state.human_topic_upsert({ ...topic, rewrite_checked: true });
+      if (topic) state.human_topic_upsert({
+        ...topic,
+        rewrite_length_floor: Math.max(MIN_REWRITE_FLOOR, Math.ceil((topic.description?.length ?? 0) * 1.1)),
+      });
     } else if (itemType === "person") {
       const person = human.people.find(p => p.id === itemId);
-      if (person) state.human_person_upsert({ ...person, rewrite_checked: true });
+      if (person) state.human_person_upsert({
+        ...person,
+        rewrite_length_floor: Math.max(MIN_REWRITE_FLOOR, Math.ceil((person.description?.length ?? 0) * 1.1)),
+      });
     }
     return;
   }
@@ -111,8 +118,7 @@ export async function handleRewriteRewrite(response: LLMResponse, state: StateMa
   const itemType = response.request.data.itemType as RewriteItemType;
 
   if (!itemId || !itemType) {
-    console.error("[handleRewriteRewrite] Missing itemId or itemType in request data");
-    return;
+    throw new Error("[handleRewriteRewrite] Missing itemId or itemType in request data");
   }
 
   const result = response.parsed as RewriteResult | undefined;
@@ -171,6 +177,7 @@ export async function handleRewriteRewrite(response: LLMResponse, state: StateMa
       console.warn(`[handleRewriteRewrite] Failed to compute embedding for existing ${resolvedType} "${item.name}":`, err);
     }
 
+    const existingFloor = Math.max(MIN_REWRITE_FLOOR, Math.ceil(item.description.length * 1.1));
     switch (resolvedType) {
       case "topic": {
         const existing = human.topics.find(t => t.id === item.id)!;
@@ -181,6 +188,7 @@ export async function handleRewriteRewrite(response: LLMResponse, state: StateMa
           sentiment: item.sentiment ?? existing.sentiment,
           last_updated: now,
           embedding,
+          rewrite_length_floor: existingFloor,
         });
         break;
       }
@@ -193,6 +201,7 @@ export async function handleRewriteRewrite(response: LLMResponse, state: StateMa
           sentiment: item.sentiment ?? existing.sentiment,
           last_updated: now,
           embedding,
+          rewrite_length_floor: existingFloor,
         });
         break;
       }
@@ -216,6 +225,7 @@ export async function handleRewriteRewrite(response: LLMResponse, state: StateMa
       console.warn(`[handleRewriteRewrite] Failed to compute embedding for new ${item.type} "${item.name}":`, err);
     }
 
+    const newFloor = Math.max(MIN_REWRITE_FLOOR, Math.ceil(item.description.length * 1.1));
     const baseFields = {
       id: crypto.randomUUID(),
       name: item.name,
@@ -227,6 +237,7 @@ export async function handleRewriteRewrite(response: LLMResponse, state: StateMa
       persona_groups: unionGroups,
       interested_personas: unionPersonas,
       embedding,
+      rewrite_length_floor: newFloor,
     };
 
     switch (item.type) {
@@ -267,10 +278,16 @@ export async function handleRewriteRewrite(response: LLMResponse, state: StateMa
   const updatedHuman = state.getHuman();
   if (itemType === "topic") {
     const original = updatedHuman.topics.find(t => t.id === itemId);
-    if (original) state.human_topic_upsert({ ...original, rewrite_checked: true });
+    if (original) state.human_topic_upsert({
+      ...original,
+      rewrite_length_floor: Math.max(MIN_REWRITE_FLOOR, Math.ceil((original.description?.length ?? 0) * 1.1)),
+    });
   } else if (itemType === "person") {
     const original = updatedHuman.people.find(p => p.id === itemId);
-    if (original) state.human_person_upsert({ ...original, rewrite_checked: true });
+    if (original) state.human_person_upsert({
+      ...original,
+      rewrite_length_floor: Math.max(MIN_REWRITE_FLOOR, Math.ceil((original.description?.length ?? 0) * 1.1)),
+    });
   }
 
   console.log(`[handleRewriteRewrite] Complete for ${itemType} "${itemId}": ${existingCount} existing updated, ${newCount} new created`);

package/src/core/handlers/rooms.ts

@@ -17,8 +17,7 @@ export function handleRoomResponse(response: LLMResponse, state: StateManager):
   const parentMessageId = response.request.data.parentMessageId as string | null ?? null;
 
   if (!roomId || !personaId) {
-    console.error("[handleRoomResponse] Missing roomId or personaId in request data");
-    return;
+    throw new Error("[handleRoomResponse] Missing roomId or personaId in request data");
   }
 
   const now = new Date().toISOString();
@@ -111,19 +110,16 @@ export async function handleRoomJudge(response: LLMResponse, state: StateManager
   const judgeDisplayName = response.request.data.judgePersonaDisplayName as string;
 
   if (!roomId) {
-    console.error("[handleRoomJudge] Missing roomId in request data");
-    return;
+    throw new Error("[handleRoomJudge] Missing roomId in request data");
   }
 
   if (!response.parsed) {
-    console.error(`[handleRoomJudge] No parsed result from judge ${judgeDisplayName}`);
-    return;
+    throw new Error(`[handleRoomJudge] No parsed result from judge ${judgeDisplayName}`);
   }
 
   const result = response.parsed as RoomJudgeResult;
   if (!result.winner_message_id) {
-    console.error(`[handleRoomJudge] Judge ${judgeDisplayName} returned no winner_message_id`);
-    return;
+    throw new Error(`[handleRoomJudge] Judge ${judgeDisplayName} returned no winner_message_id`);
   }
 
   const judgePersonaId = response.request.data.judgePersonaId as string;
@@ -131,16 +127,14 @@ export async function handleRoomJudge(response: LLMResponse, state: StateManager
   const allMessages = state.getRoomMessages(roomId);
   const winner = allMessages.find(m => m.id === result.winner_message_id);
   if (!winner) {
-    console.error(`[handleRoomJudge] Winner message ${result.winner_message_id} not found in room ${roomId}`);
-    return;
+    throw new Error(`[handleRoomJudge] Winner message ${result.winner_message_id} not found in room ${roomId}`);
   }
 
   const verdictParentId = winner.parent_id;
 
   const ok = state.setRoomActiveNode(roomId, result.winner_message_id);
   if (!ok) {
-    console.error(`[handleRoomJudge] Could not set active node ${result.winner_message_id} in room ${roomId}`);
-    return;
+    throw new Error(`[handleRoomJudge] Could not set active node ${result.winner_message_id} in room ${roomId}`);
   }
 
   const losers = allMessages

package/src/core/llm-client.ts

@@ -76,7 +76,17 @@ export interface LLMRawResponse {
 
 let llmCallCount = 0;
 
-
+function resolveApiKey(raw: string | undefined): string {
+  if (!raw || !raw.startsWith("$")) return raw ?? "";
+  const varName = raw.slice(1);
+  const resolved =
+    (typeof Bun !== "undefined" && (Bun as { env: Record<string, string> }).env?.[varName]) ||
+    (typeof process !== "undefined" && process.env?.[varName]);
+  if (!resolved) {
+    throw new Error(`Provider API key references env var $${varName}, but it is not set.`);
+  }
+  return resolved;
+}
 
 function isGuid(str: string): boolean {
   return /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i.test(str);
@@ -90,7 +100,7 @@ function buildResolvedModel(account: ProviderAccount, model: ModelConfig): Resol
     config: {
       name: account.name,
       baseURL: account.url,
-      apiKey: account.api_key || "",
+      apiKey: resolveApiKey(account.api_key),
     },
     extraHeaders: account.extra_headers,
   };
@@ -171,7 +181,7 @@ export function resolveModel(modelSpec?: string, accounts?: ProviderAccount[]):
         config: {
           name: matchingAccount.name,
           baseURL: matchingAccount.url,
-          apiKey: matchingAccount.api_key || "",
+          apiKey: resolveApiKey(matchingAccount.api_key),
         },
         extraHeaders: matchingAccount.extra_headers,
       };

package/src/core/message-manager.ts

@@ -255,8 +255,6 @@ export function checkAndQueueHumanExtraction(
   const unextractedPeople = sm.messages_getUnextracted(personaId, "p", undefined, "exclude");
   const peopleThreshold = Math.min(EXTRACTION_TAPER_CAP, human.people.length);
   if (unextractedPeople.length > 0 && unextractedPeople.length >= peopleThreshold) {
-    const personaForScan = sm.persona_getById(personaId);
-    const personScanOptions = personaForScan?.pending_update ? { reflection_progress: 1 } : undefined;
     const context: ExtractionContext = {
       personaId,
       channelDisplayName: personaDisplayName,
@@ -264,9 +262,9 @@ export function checkAndQueueHumanExtraction(
       messages_analyze: unextractedPeople,
       extraction_flag: "p",
     };
-    queuePersonScan(context, sm, personScanOptions);
+    queuePersonScan(context, sm);
     console.log(
-      `[Processor] Human Seed extraction: people (threshold: ${peopleThreshold}, unextracted: ${unextractedPeople.length}${personScanOptions ? ", reflection_progress=1" : ""})`
+      `[Processor] Human Seed extraction: people (threshold: ${peopleThreshold}, unextracted: ${unextractedPeople.length})`
     );
   }
 }

package/src/core/orchestrators/ceremony.ts

@@ -137,10 +137,7 @@ function queueExposurePhase(personaId: string, state: StateManager, options?: Ex
       messages_analyze: unextractedPeople,
       extraction_flag: "p",
     };
-    const personScanOptions = persona.pending_update
-      ? { ...options, reflection_progress: 1 }
-      : options;
-    queuePersonScan(context, state, personScanOptions);
+    queuePersonScan(context, state, options);
   }
   
   const totalUnextracted = unextractedFacts.length + unextractedTopics.length + unextractedPeople.length;
@@ -287,7 +284,7 @@ export function handleCeremonyProgress(state: StateManager, lastPhase: number):
   // Person Rewrite phase (phase 4): scan bloated Person records, extract Topics from them.
   // Gated via ceremony_progress so Topic Rewrite can run after — Topics created here
   // need to be visible before Topic Rewrite snapshots the threshold.
-  queuePersonRewritePhase(state);
+  queuePersonRewritePhase(state, { ceremonyProgress: 4 });
 
   // Zero-work guard: if no person rewrites queued, advance to topic rewrite immediately
   if (!state.queue_hasPendingCeremonies()) {
@@ -482,7 +479,7 @@ export function queueReflectionDrain(personaId: string, state: StateManager): vo
     messages_analyze: unextractedPeople,
     extraction_flag: "p",
   };
-  queuePersonScan(context, state, { reflection_progress: 1 });
+  queuePersonScan(context, state);
   console.log(`[reflection:drain] Queued Person scan for ${persona.display_name} (${unextractedPeople.length} messages) — clears on completion`);
 }
 
@@ -490,7 +487,7 @@ function getRewriteModel(state: StateManager): string | undefined {
   return state.getHuman().settings?.rewrite_model;
 }
 
-export function queuePersonRewritePhase(state: StateManager): void {
+export function queuePersonRewritePhase(state: StateManager, options?: { ceremonyProgress?: number }): void {
   const rewriteModel = getRewriteModel(state);
   if (!rewriteModel) {
     console.log("[ceremony:rewrite] rewrite_model not set — skipping person rewrite phase");
@@ -498,13 +495,30 @@ export function queuePersonRewritePhase(state: StateManager): void {
   }
 
   const human = state.getHuman();
-  const personsToScan = human.people.filter(person => {
+  const allCandidates = human.people.filter(person => {
     const isPersonaLinked = (person.identifiers ?? []).some(
       i => i.type.toLowerCase() === 'ei persona'
     );
     return !isPersonaLinked
-      && (person.description?.length ?? 0) > REWRITE_DESCRIPTION_THRESHOLD
-      && !person.rewrite_checked;
+      && (person.description?.length ?? 0) > REWRITE_DESCRIPTION_THRESHOLD;
+  });
+
+  const alreadyChecked = allCandidates.filter(p => {
+    const descLen = p.description?.length ?? 0;
+    return p.rewrite_length_floor !== undefined && descLen < p.rewrite_length_floor;
+  });
+  if (alreadyChecked.length > 0) {
+    for (const person of alreadyChecked) {
+      console.log(
+        `[ceremony:rewrite] Person "${person.name}" is ${person.description?.length ?? 0} chars ` +
+        `(floor: ${person.rewrite_length_floor}) — already reviewed, skipping`
+      );
+    }
+  }
+
+  const personsToScan = allCandidates.filter(p => {
+    if (p.rewrite_length_floor === undefined) return true;
+    return (p.description?.length ?? 0) >= p.rewrite_length_floor;
   });
 
   if (personsToScan.length === 0) {
@@ -527,7 +541,7 @@ export function queuePersonRewritePhase(state: StateManager): void {
         itemId: person.id,
         itemType: "person" as RewriteItemType,
         rewriteModel,
-        ceremony_progress: 4,
+        ...(options?.ceremonyProgress !== undefined && { ceremony_progress: options.ceremonyProgress }),
       },
     });
   }
@@ -543,11 +557,28 @@ export function queueTopicRewritePhase(state: StateManager): void {
   }
 
   const human = state.getHuman();
-  const topicsToScan = human.topics.filter(topic =>
+  const allCandidateTopics = human.topics.filter(topic =>
     (topic.description?.length ?? 0) > REWRITE_DESCRIPTION_THRESHOLD
-    && !topic.rewrite_checked
   );
 
+  const alreadyCheckedTopics = allCandidateTopics.filter(t => {
+    const descLen = t.description?.length ?? 0;
+    return t.rewrite_length_floor !== undefined && descLen < t.rewrite_length_floor;
+  });
+  if (alreadyCheckedTopics.length > 0) {
+    for (const topic of alreadyCheckedTopics) {
+      console.log(
+        `[ceremony:rewrite] Topic "${topic.name}" is ${topic.description?.length ?? 0} chars ` +
+        `(floor: ${topic.rewrite_length_floor}) — already reviewed, skipping`
+      );
+    }
+  }
+
+  const topicsToScan = allCandidateTopics.filter(t => {
+    if (t.rewrite_length_floor === undefined) return true;
+    return (t.description?.length ?? 0) >= t.rewrite_length_floor;
+  });
+
   if (topicsToScan.length === 0) {
     console.log("[ceremony:rewrite] No topics above threshold — skipping topic rewrite phase");
     return;