clementine-agent 1.0.59 → 1.0.61

package/dist/agent/assistant.js

@@ -2571,6 +2571,18 @@ You have a cost budget per message — not a hard turn limit. Work until the tas
                 if (!contradictionRetried && attempt < PersonalAssistant.RATE_LIMIT_MAX_RETRIES && responseText.trim()) {
                     try {
                         const toolCallRecords = collectToolCalls(collectedSdkMessages);
+                        // Diagnostic — emits once per turn so we can see what the
+                        // validator is working with even when it doesn't fire. Without
+                        // this we're blind to the "regex missed the phrasing" case.
+                        if (toolCallRecords.length > 0) {
+                            logger.info({
+                                sessionKey,
+                                sdkMessagesCaptured: collectedSdkMessages.length,
+                                toolCallsPaired: toolCallRecords.length,
+                                resultClasses: toolCallRecords.map(r => `${r.name}:${r.resultClass}`),
+                                replyPreview: responseText.slice(0, 200).replace(/\n/g, ' '),
+                            }, 'Contradiction validator pass');
+                        }
                         const finding = detectContradiction(responseText, toolCallRecords);
                         if (finding) {
                             contradictionRetried = true;

package/dist/agent/contradiction-validator.js

@@ -15,7 +15,7 @@
 const ARG_ERROR_RE = /\b(invalid|unknown field|required|missing parameter|schema|unrecognized|unexpected property)\b/i;
 const AUTH_ERROR_RE = /\b(unauthori[sz]ed|401|not authenticated|token expired|token has expired|invalid[_ ]?token|access denied)\b/i;
 /** Regex matching reply phrasings that claim a connector-wide failure. */
-export const CONTRADICTION_RE = /(dead\s*end|doesn'?t exist|not in (the |my )?schema|schema[- ]level|not available|isn'?t loaded|tools array is empty|MCP server still connecting|connector is (a )?dead|no such tool available|tool doesn't exist)/i;
+export const CONTRADICTION_RE = /(dead\s*end|doesn'?t exist|not in (the |my )?schema|schema[- ]level|aren'?t loading into|(not|isn'?t|aren'?t|wasn'?t) (loaded|wired|available|connected|coming through|responding|reachable|working)|connector[^.]{0,40}(dropped|is (a )?dead)|tools? array is empty|MCP server (still connecting|dropped|not responding|just isn'?t connected|isn'?t connected)|no such tool available|tool doesn'?t exist|both directions are blocked|(restart|close and reopen|reconnect) Claude Code)/i;
 export function classifyResult(content, isError) {
     if (!isError)
         return 'success';
@@ -101,7 +101,13 @@ export function detectContradiction(reply, calls) {
     const match = reply.match(CONTRADICTION_RE);
     if (!match)
         return null;
-    const connectorCalls = calls.filter(c => c.name.startsWith('mcp__claude_ai_'));
+    // Cover every connector — claude_ai_* (remote), imessage/figma/hostinger/etc.
+    // (Desktop Extensions + stdio servers), everything except Clementine's own
+    // tools server and plugins. Earlier versions only filtered claude_ai_*,
+    // which let "isn't loaded" replies slip through for iMessage etc.
+    const connectorCalls = calls.filter(c => c.name.startsWith('mcp__') &&
+        !c.name.startsWith('mcp__clementine-tools__') &&
+        !c.name.startsWith('mcp__plugin_'));
     const recoverable = connectorCalls.find(c => c.resultClass === 'success' || c.resultClass === 'arg_error');
     if (!recoverable)
         return null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "clementine-agent",
-  "version": "1.0.59",
+  "version": "1.0.61",
   "description": "Clementine — Personal AI Assistant (TypeScript)",
   "type": "module",
   "main": "dist/index.js",