ei-tui 1.0.1 → 1.2.0

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

@@ -85,6 +85,10 @@ export async function handleTopicUpdate(response: LLMResponse, state: StateManag
   const primaryId = personaIds[0] ?? personaId;
 
   const now = new Date().toISOString();
+  const { messages_analyze } = resolveMessageWindow(response, state);
+  const earliestMessageTimestamp = messages_analyze.length > 0
+    ? messages_analyze.reduce((a, b) => a.timestamp < b.timestamp ? a : b).timestamp
+    : now;
   const human = state.getHuman();
 
   const resolveItemId = (): string => {
@@ -135,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,
@@ -144,7 +150,7 @@ export async function handleTopicUpdate(response: LLMResponse, state: StateManag
     exposure_current: calculateExposureCurrent(exposureImpact, existingTopic?.exposure_current ?? 0),
     exposure_desired: result.exposure_desired ?? 0.5,
     last_updated: now,
-    learned_on: isNewItem ? now : existingTopic?.learned_on,
+    learned_on: isNewItem ? earliestMessageTimestamp : existingTopic?.learned_on,
     last_mentioned: now,
     learned_by: isNewItem ? primaryId : existingTopic?.learned_by,
     last_changed_by: primaryId,
@@ -152,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);
 
@@ -168,6 +175,26 @@ export async function handleTopicUpdate(response: LLMResponse, state: StateManag
   console.log(`[handleTopicUpdate] ${isNewItem ? "Created" : "Updated"} topic "${resolvedName}"`);
 }
 
+function ensureEiPersonaHasNickname(identifiers: PersonIdentifier[], state: StateManager): PersonIdentifier[] {
+  const eiPersonaId = identifiers.find(i => i.type === 'Ei Persona')?.value;
+  if (!eiPersonaId) return identifiers;
+
+  const persona = state.persona_getById(eiPersonaId);
+  if (!persona) return identifiers;
+
+  const hasNickname = identifiers.some(i => i.type === 'Nickname' && i.value === persona.display_name);
+  if (hasNickname) return identifiers;
+
+  const withoutPrimary = identifiers.map(i =>
+    i.type === 'Ei Persona' ? { ...i, is_primary: undefined } : i
+  ).map(({ is_primary, ...rest }) => is_primary ? { ...rest, is_primary } : rest);
+
+  return [
+    { type: 'Nickname', value: persona.display_name, is_primary: true as const },
+    ...withoutPrimary.map(i => i.type === 'Ei Persona' ? { type: i.type, value: i.value } : i),
+  ];
+}
+
 export async function handlePersonUpdate(response: LLMResponse, state: StateManager): Promise<void> {
   const result = response.parsed as (PersonUpdateResult & {
     identifiers?: PersonIdentifier[];
@@ -194,6 +221,10 @@ export async function handlePersonUpdate(response: LLMResponse, state: StateMana
   const primaryId = personaIds[0] ?? personaId;
 
   const now = new Date().toISOString();
+  const { messages_analyze } = resolveMessageWindow(response, state);
+  const earliestMessageTimestamp = messages_analyze.length > 0
+    ? messages_analyze.reduce((a, b) => a.timestamp < b.timestamp ? a : b).timestamp
+    : now;
   const human = state.getHuman();
 
   const resolveItemId = (): string => {
@@ -264,7 +295,7 @@ export async function handlePersonUpdate(response: LLMResponse, state: StateMana
         deduped.push(id);
       }
     }
-    resolvedIdentifiers = deduped;
+    resolvedIdentifiers = ensureEiPersonaHasNickname(deduped, state);
   } else {
     const base = [...(existingPerson?.identifiers ?? [])];
     const sanitizedToAdd = sanitizeEiPersonaIdentifiers(
@@ -279,12 +310,16 @@ export async function handlePersonUpdate(response: LLMResponse, state: StateMana
         base.push({ type: id.type, value: id.value, ...(id.is_primary ? { is_primary: id.is_primary } : {}) });
       }
     }
-    resolvedIdentifiers = base;
+    resolvedIdentifiers = ensureEiPersonaHasNickname(base, state);
   }
 
+  const personName = resolvedIdentifiers.find(i => i.is_primary && i.type !== 'Ei Persona')?.value
+    ?? resolvedIdentifiers.find(i => i.type !== 'Ei Persona')?.value
+    ?? candidateName;
+
   const person: Person = {
     id: itemId,
-    name: candidateName,
+    name: personName,
     description: resolvedDescription,
     sentiment: resolvedSentiment,
     relationship: result.relationship ?? candidateRelationship ?? existingPerson?.relationship ?? "Unknown",
@@ -293,7 +328,7 @@ export async function handlePersonUpdate(response: LLMResponse, state: StateMana
     identifiers: resolvedIdentifiers,
     validated_date: isNewItem ? '' : (existingPerson?.validated_date ?? ''),
     last_updated: now,
-    learned_on: isNewItem ? now : existingPerson?.learned_on,
+    learned_on: isNewItem ? earliestMessageTimestamp : existingPerson?.learned_on,
     last_mentioned: now,
     learned_by: isNewItem ? primaryId : existingPerson?.learned_by,
     last_changed_by: primaryId,
@@ -301,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);
 
@@ -323,14 +363,13 @@ function normalizeText(text: string): string {
     .replace(/[\u2018\u2019\u0060\u00B4]/g, "'")  // curly single, backtick, acute accent
     .replace(/[\u2014\u2013\u2012]/g, '-')         // em-dash, en-dash, figure dash
     .replace(/\u00A0/g, ' ')                       // non-breaking space
-    .replace(/[\u2000-\u200F]/g, ' ');              // unicode space variants
+    .replace(/[\u2000-\u200F]/g, ' ')              // unicode space variants
+    .replace(/[*_`~]/g, '');                       // Markdown emphasis/code chars
 }
 
 function stripPunctuation(text: string): string {
-  // Remove characters LLMs commonly mangle, keep spaces and alphanumeric
-  // Strip: punctuation, unicode punctuation variants, curly quotes, dashes, etc.
-  // Keep: letters, digits, spaces
   return text
+    .replace(/[*_`~]/g, ' ')     // Markdown chars (kept by \w, must strip explicitly)
     .replace(/[^\w\s]/gu, ' ')   // replace non-word, non-space with space
     .replace(/\s+/g, ' ')        // collapse multiple spaces
     .trim()
@@ -406,6 +445,10 @@ async function validateAndStoreQuotes(
   if (!candidates || candidates.length === 0) return;
   
   for (const candidate of candidates) {
+    if (!candidate.text) {
+      console.warn('[extraction] Skipping quote candidate with missing text field');
+      continue;
+    }
     let found = false;
     for (const message of messages) {
       const msgText = getMessageText(message);
@@ -511,7 +554,7 @@ async function validateAndStoreQuotes(
       break;
     }
     if (!found) {
-      console.warn(`[extraction] Quote not found in messages (both levels), skipping: "${candidate.text?.slice(0, 50)}..."`);
+      console.warn(`[extraction] Quote not found in messages (both levels), skipping: "${candidate.text}"`);
     }
   }
 }

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,
       };
@@ -450,6 +460,7 @@ const JSON_REPAIR_PATTERNS: Array<{ pattern: RegExp; replacement: string }> = [
   { pattern: /:\s*(\d{4}-\d{2}-\d{2}T[^"}\],\n]+)/g, replacement: ': "$1"' },
   { pattern: /:\s*0([1-9][0-9]*)([,\s\n\r\]}])/g, replacement: ": 0.$1$2" },
   { pattern: /,(\s*[\]}])/g, replacement: "$1" },
+  { pattern: /"(\s*\n[ \t]+"[a-zA-Z_][a-zA-Z0-9_]*"\s*:)/g, replacement: '",$1' },
 ];
 
 export function repairJSON(jsonStr: string): string {
@@ -529,6 +540,41 @@ export function rescueGemmaToolCalls(content: string): unknown[] {
   return rescued;
 }
 
+function findOutermostObject(str: string): string | null {
+  const start = str.indexOf('{');
+  if (start === -1) return null;
+
+  let depth = 0;
+  let inString = false;
+  let escaped = false;
+
+  for (let i = start; i < str.length; i++) {
+    const ch = str[i];
+
+    if (escaped) {
+      escaped = false;
+      continue;
+    }
+    if (ch === '\\' && inString) {
+      escaped = true;
+      continue;
+    }
+    if (ch === '"') {
+      inString = !inString;
+      continue;
+    }
+    if (inString) continue;
+
+    if (ch === '{') depth++;
+    else if (ch === '}') {
+      depth--;
+      if (depth === 0) return str.slice(start, i + 1);
+    }
+  }
+
+  return null;
+}
+
 export function parseJSONResponse(content: string): unknown {
   const jsonMatch = content.match(/```(?:json)?\s*([\s\S]*?)```/);
   const jsonStr = jsonMatch ? jsonMatch[1].trim() : content.trim();
@@ -541,10 +587,10 @@ export function parseJSONResponse(content: string): unknown {
       return JSON.parse(repaired);
     } catch {
       // Last resort: extract the outermost {...} block from mixed prose/JSON content.
-      // Handles 'thinking prose...\n{...json...}' responses from extended-thinking models.
-      const outerMatch = jsonStr.match(/\{[\s\S]*\}/);
-      if (outerMatch) {
-        const extracted = outerMatch[0];
+      // Bracket-depth scan (not greedy regex) stops at the first valid close so extra
+      // trailing braces from models like Gemma are excluded from the extracted slice.
+      const extracted = findOutermostObject(jsonStr);
+      if (extracted) {
         try {
           return JSON.parse(extracted);
         } catch {

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})`
     );
   }
 }