ei-tui 0.6.2 → 0.6.4

package/package.json

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

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

@@ -123,9 +123,9 @@ export async function handleFactFind(response: LLMResponse, state: StateManager)
       continue;
     }
 
-    // Skip if the LLM returned a null/empty value — don't store null descriptions
-    if (!factResult.value) {
-      console.log(`[handleFactFind] Skipping fact with null/empty value: "${factResult.name}"`);
+    // Skip if the LLM returned a null/empty/non-string value — don't store booleans or nulls
+    if (!factResult.value || typeof factResult.value !== 'string') {
+      console.log(`[handleFactFind] Skipping fact with null/empty/non-string value: "${factResult.name}" (got ${typeof factResult.value})`);
       continue;
     }
 

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

@@ -123,6 +123,7 @@ export async function handleTopicUpdate(response: LLMResponse, state: StateManag
   const roomId = response.request.data.roomId as string | undefined;
   const candidateCategory = response.request.data.candidateCategory as string | undefined;
   const candidateName = response.request.data.candidateName as string | undefined;
+  const candidateDescription = response.request.data.candidateDescription as string | undefined;
 
   const personaIds = personaId.split("|").filter(Boolean);
   const primaryId = personaIds[0] ?? personaId;
@@ -145,10 +146,15 @@ export async function handleTopicUpdate(response: LLMResponse, state: StateManag
   const existingTopic = isNewItem ? undefined : human.topics.find(t => t.id === existingItemId);
 
   const resolvedName = result.name || existingTopic?.name || candidateName;
-  const resolvedDescription = typeof result.description === 'string' ? result.description : existingTopic?.description;
+  const resolvedDescription = typeof result.description === 'string' ? result.description : existingTopic?.description ?? candidateDescription;
+  const resolvedSentiment = result.sentiment !== undefined ? result.sentiment : existingTopic?.sentiment ?? 0;
 
-  if (!resolvedName || !resolvedDescription || result.sentiment === undefined) {
-    throw new Error(`[handleTopicUpdate] Missing required fields: name=${resolvedName}, description=${!!resolvedDescription}, sentiment=${result.sentiment}`);
+  if (!resolvedName || !resolvedDescription) {
+    if (isNewItem) {
+      throw new Error(`[handleTopicUpdate] Cannot create new topic — missing required fields: name=${resolvedName}, description=${!!resolvedDescription}`);
+    }
+    console.log(`[handleTopicUpdate] Skipping update for "${resolvedName ?? existingItemId}" — no description available and existing record preserved`);
+    return;
   }
 
   let embedding: number[] | undefined;
@@ -173,7 +179,7 @@ export async function handleTopicUpdate(response: LLMResponse, state: StateManag
     id: itemId,
     name: resolvedName,
     description: resolvedDescription,
-    sentiment: result.sentiment,
+    sentiment: resolvedSentiment,
     category: result.category ?? candidateCategory ?? existingTopic?.category,
     exposure_current: calculateExposureCurrent(exposureImpact, existingTopic?.exposure_current ?? 0),
     exposure_desired: result.exposure_desired ?? 0.5,
@@ -216,11 +222,8 @@ export async function handlePersonUpdate(response: LLMResponse, state: StateMana
   const candidateRelationship = response.request.data.candidateRelationship as string | undefined;
   const candidateIdentifiers = (response.request.data.candidateIdentifiers ?? []) as PersonIdentifier[];
 
-  if (!result.description || result.sentiment === undefined) {
-    throw new Error(`[handlePersonUpdate] Missing required fields: description=${!!result.description}, sentiment=${result.sentiment}`);
-  }
-
   const candidateName = response.request.data.candidateName as string;
+  const candidateDescription = response.request.data.candidateDescription as string | undefined;
   const personaIds = personaId.split("|").filter(Boolean);
   const primaryId = personaIds[0] ?? personaId;
 
@@ -241,11 +244,22 @@ export async function handlePersonUpdate(response: LLMResponse, state: StateMana
 
   const existingPerson = isNewItem ? undefined : human.people.find(p => p.id === existingItemId);
 
+  const resolvedDescription = typeof result.description === 'string' ? result.description : existingPerson?.description ?? candidateDescription;
+  const resolvedSentiment = result.sentiment !== undefined ? result.sentiment : existingPerson?.sentiment ?? 0;
+
+  if (!resolvedDescription) {
+    if (isNewItem) {
+      throw new Error(`[handlePersonUpdate] Cannot create new person "${candidateName}" — no description available`);
+    }
+    console.log(`[handlePersonUpdate] Skipping update for "${candidateName}" — no description available and existing record preserved`);
+    return;
+  }
+
   let embedding: number[] | undefined;
   try {
     const embeddingService = getEmbeddingService();
     const relationship = result.relationship ?? candidateRelationship ?? existingPerson?.relationship;
-    const text = getPersonEmbeddingText({ name: candidateName, relationship, description: result.description });
+    const text = getPersonEmbeddingText({ name: candidateName, relationship, description: resolvedDescription });
     embedding = await embeddingService.embed(text);
   } catch (err) {
     console.warn(`[handlePersonUpdate] Failed to compute embedding for person "${candidateName}":`, err);
@@ -295,8 +309,8 @@ export async function handlePersonUpdate(response: LLMResponse, state: StateMana
   const person: Person = {
     id: itemId,
     name: candidateName,
-    description: result.description,
-    sentiment: result.sentiment,
+    description: resolvedDescription,
+    sentiment: resolvedSentiment,
     relationship: result.relationship ?? candidateRelationship ?? existingPerson?.relationship ?? "Unknown",
     exposure_current: calculateExposureCurrent(exposureImpact, existingPerson?.exposure_current ?? 0),
     exposure_desired: result.exposure_desired ?? 0.5,

package/src/core/orchestrators/human-extraction.ts

@@ -423,6 +423,7 @@ export function queueTopicUpdate(
         isNewItem,
         existingItemId: existingItem?.id,
         candidateName: isNewItem ? context.candidateName : undefined,
+        candidateDescription: isNewItem ? context.candidateDescription : undefined,
         candidateCategory: context.candidateCategory,
         analyze_from_timestamp: getAnalyzeFromTimestamp(chunk),
       },
@@ -586,6 +587,7 @@ export function queuePersonUpdate(
         isNewItem,
         existingItemId: existingItem?.id,
         candidateName: context.candidateName,
+        candidateDescription: isNewItem ? context.candidateDescription : undefined,
         candidateRelationship: context.candidateRelationship,
         candidateIdentifiers: isNewItem ? candidateIdentifiers : undefined,
         analyze_from_timestamp: getAnalyzeFromTimestamp(chunk),