- Added a secondary pass to clean-up JSON parsing mistakes, improving robustness, in particular with mixed dialogue/narration content.

- Improved error handling when parsing character profiles via Wikia.
- Added more loading messages.

Author: Rhystic1

src/components/views/ChatInterface.vue

@@ -1589,31 +1589,43 @@ const searchForCharactersViaWikiFetch = async (characterNames: string[]): Promis
       { role: 'user', content: summarizePrompt }
     ]
     
-    try {
-      // Call OpenRouter WITHOUT web search - we already have the content
-      const result = await callOpenRouter(messages, false)
-      
-      let jsonStr = result.replace(/```json\n?|\n?```/g, '').trim()
-      const start = jsonStr.indexOf('{')
-      const end = jsonStr.lastIndexOf('}')
-      if (start !== -1 && end !== -1) {
-        jsonStr = jsonStr.substring(start, end + 1)
-      }
-      
-      const profiles = JSON.parse(jsonStr)
-      
-      // Add character IDs
-      for (const charName of Object.keys(profiles)) {
-        const char = l2d.find((c) => c.name.toLowerCase() === charName.toLowerCase())
-        if (char) {
-          profiles[charName].id = char.id
+    let attempts = 0
+    const maxAttempts = 3
+    let success = false
+    
+    while (attempts < maxAttempts && !success) {
+      try {
+        // Call OpenRouter WITHOUT web search - we already have the content
+        const result = await callOpenRouter(messages, false)
+        
+        let jsonStr = result.replace(/```json\n?|\n?```/g, '').trim()
+        const start = jsonStr.indexOf('{')
+        const end = jsonStr.lastIndexOf('}')
+        if (start !== -1 && end !== -1) {
+          jsonStr = jsonStr.substring(start, end + 1)
+        }
+        
+        const profiles = JSON.parse(jsonStr)
+        
+        // Add character IDs
+        for (const charName of Object.keys(profiles)) {
+          const char = l2d.find((c) => c.name.toLowerCase() === charName.toLowerCase())
+          if (char) {
+            profiles[charName].id = char.id
+          }
+        }
+        
+        characterProfiles.value = { ...characterProfiles.value, ...profiles }
+        logDebug(`[searchForCharactersViaWikiFetch] Added profile for ${name}:`, profiles)
+        success = true
+      } catch (e) {
+        attempts++
+        if (attempts >= maxAttempts) {
+          console.error(`[searchForCharactersViaWikiFetch] Failed to process ${name} after ${maxAttempts} attempts:`, e)
+        } else {
+          console.warn(`[searchForCharactersViaWikiFetch] Attempt ${attempts} failed for ${name}, retrying...`)
         }
       }
-      
-      characterProfiles.value = { ...characterProfiles.value, ...profiles }
-      logDebug(`[searchForCharactersViaWikiFetch] Added profile for ${name}:`, profiles)
-    } catch (e) {
-      console.error(`[searchForCharactersViaWikiFetch] Failed to process ${name}:`, e)
     }
   }
 }
@@ -2236,6 +2248,10 @@ const processAIResponse = async (responseStr: string) => {
 
   logDebug('Parsed Action Sequence:', data)
 
+  // Ensure narration/dialogue separation even when the model returns a single mixed text step.
+  data = sanitizeActions(data)
+  logDebug('Sanitized Action Sequence:', data)
+
   isGenerating.value = false
   loadingStatus.value = "..."
 

src/utils/chatUtils.ts

@@ -187,20 +187,76 @@ export const parseAIResponse = (responseStr: string): any[] => {
 export const sanitizeActions = (actions: any[]): any[] => {
   const newActions: any[] = []
 
+  // Metadata fields that should not be duplicated if we split an action into multiple ones.
+  // These can trigger side-effects (search, profile updates, debug panels).
+  const nonDuplicatedFields = new Set([
+    'needs_search',
+    'memory',
+    'characterProgression',
+    'debug_info'
+  ])
+
+  const looksLikeNarrationWithoutQuotes = (rawText: string): boolean => {
+    const t = (rawText || '').trim()
+    if (!t) return false
+
+    const lower = t.toLowerCase()
+
+    for (const char of l2d) {
+      const name = (char as any).name as string
+      if (!name) continue
+
+      const nameLower = name.toLowerCase()
+      if (!lower.startsWith(nameLower)) continue
+
+      const after = t.slice(name.length)
+      const nextChar = after.charAt(0)
+
+      // Speaker label ("Name:")
+      if (nextChar === ':') return false
+
+      // Direct address ("Name,")
+      if (nextChar === ',') return false
+
+      // Possessive narration: "Name's ..." / "Name’s ..."
+      if (nextChar === '\'' || nextChar === '’') {
+        const poss = after.slice(0, 2)
+        if (poss === "'s" || poss === '’s') return true
+      }
+
+      // Strong narration clue: "Name ..., her/his/ ..." early in the sentence.
+      if (nextChar && /\s/.test(nextChar)) {
+        const rest = after.trimStart().slice(0, 80).toLowerCase()
+        if (rest.includes(', her ') || rest.includes(', his ')) {
+          return true
+        }
+      }
+
+      return false
+    }
+
+    return false
+  }
+
   for (const action of actions) {
     if (!action.text || typeof action.text !== 'string') {
       newActions.push(action)
       continue
     }
 
-    // Check if the text contains dialogue
+    // Check if the text contains dialogue.
+    // Supports straight quotes ("...") and curly quotes (open “ ... close ”).
+    // Curly quotes are asymmetric, so we match them as a pair explicitly.
     const text = action.text
-    const quotes = '[""“”]'
-    const dialogueRegex = new RegExp('(' + quotes + ')([^' + quotes + ']*)\\1', 'g')
-    
+    const dialogueRegex = /("[\s\S]*?"|“[\s\S]*?”)/g
+
     if (!dialogueRegex.test(text)) {
-      // No dialogue detected, keep original action
-      newActions.push(action)
+      // No quotes. If the model marked it as speaking, keep it UNLESS it looks like third-person narration.
+      if (action.speaking === true && looksLikeNarrationWithoutQuotes(text)) {
+        newActions.push({ ...action, speaking: false })
+      } else {
+        newActions.push(action)
+      }
       continue
     }
 
@@ -236,19 +292,30 @@ export const sanitizeActions = (actions: any[]): any[] => {
     for (let i = 0; i < parts.length; i++) {
       const part = parts[i]
       if (!part.text.trim()) continue
+
+      // Preserve all original fields by default, but avoid duplicating side-effect fields.
+      const base: any = { ...action }
+      if (i > 0) {
+        for (const key of Object.keys(base)) {
+          if (nonDuplicatedFields.has(key)) {
+            if (key === 'needs_search') base[key] = []
+            else delete base[key]
+          }
+        }
+      }
 
       if (part.isDialogue) {
         newActions.push({
+          ...base,
           text: part.text.trim(),
-          character: action.character,
-          animation: action.animation,
           speaking: true
         })
       } else {
+        // Keep the original character for narration (the narration is *about* that character).
+        // This prevents narration like from being treated/displayed as spoken dialogue.
         newActions.push({
+          ...base,
           text: part.text.trim(),
-          character: 'narrator',
-          animation: 'idle',
           speaking: false
         })
       }
@@ -265,8 +332,11 @@ export const sanitizeActions = (actions: any[]): any[] => {
     // Check if previous is narration, current is dialogue
     if (prev.speaking === false &&
         curr.speaking === true &&
-        prev.animation && prev.animation !== 'idle') {
-      // Copy the narration animation to the dialogue
+        prev.animation && prev.animation !== 'idle' &&
+        prev.character && curr.character &&
+        prev.character === curr.character &&
+        (!curr.animation || curr.animation === 'idle')) {
+      // Only carry over animation for the SAME character, and only when the dialogue didn't specify one.
       curr.animation = prev.animation
     }
   }

src/utils/json/loadingMessages.json

@@ -13,5 +13,9 @@
   "Drinking soda with Anis...",
   "Plotting the next conspiracy...",
   "Cleaning Soda's latest mess...",
-  "Smashing another window..."
+  "Smashing another window...",
+  "Riding the AZX...",
+  "Receiving funds for the Admire...",
+  "Exploring the surface...",
+  "Debating whether coffee is best with milk or sugar..."
 ]