hedgequantx 2.9.22 → 2.9.24

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hedgequantx",
-  "version": "2.9.22",
+  "version": "2.9.24",
   "description": "HedgeQuantX - Prop Futures Trading CLI",
   "main": "src/app.js",
   "bin": {

package/src/pages/ai-agents-ui.js

@@ -334,17 +334,22 @@ const drawConnectionTest = async (agents, boxWidth, clearWithBanner) => {
   
   const W = boxWidth - 2;
   
-  // Show loading state with spinner
+  // Show loading state with spinner inside closed box
   clearWithBanner();
   console.log(chalk.cyan('╔' + '═'.repeat(W) + '╗'));
   console.log(chalk.cyan('║') + chalk.yellow.bold(centerText('AI AGENTS CONNECTION TEST', W)) + chalk.cyan('║'));
   console.log(chalk.cyan('╠' + '═'.repeat(W) + '╣'));
+  
+  // Centered spinner text inside box
+  const spinnerText = 'Testing connections...';
+  const paddedText = centerText(spinnerText, W);
+  console.log(chalk.cyan('║') + chalk.yellow(paddedText) + chalk.cyan('║'));
   console.log(chalk.cyan('║') + ' '.repeat(W) + chalk.cyan('║'));
+  console.log(chalk.cyan('╚' + '═'.repeat(W) + '╝'));
   
-  // Start spinner on the loading line
-  const spinnerText = 'Testing connections... Please wait';
+  // Start spinner below the box
   const spinner = ora({
-    text: spinnerText,
+    text: '',
     spinner: 'dots',
     color: 'yellow'
   }).start();

package/src/services/ai-supervision/health.js

@@ -10,7 +10,7 @@
  */
 
 const cliproxy = require('../cliproxy');
-const https = require('https');
+const { extractJSON } = require('./parser');
 
 /** Test prompt to verify agent understands directive format */
 const TEST_PROMPT = `You are being tested. Respond ONLY with this exact JSON, nothing else:
@@ -161,6 +161,7 @@ const testAgentConnection = async (agent) => {
 
 /**
  * Validate that response matches expected JSON format
+ * Uses extractJSON from parser.js to handle MiniMax <think> tags and other formats
  * @param {string} content - Response content from agent
  * @returns {Object} { valid, error }
  */
@@ -169,53 +170,40 @@ const validateResponseFormat = (content) => {
     return { valid: false, error: 'Empty response' };
   }
   
-  try {
-    // Try to extract JSON from response
-    let json;
-    
-    // Direct parse
-    try {
-      json = JSON.parse(content.trim());
-    } catch (e) {
-      // Try to find JSON in response
-      const match = content.match(/\{[\s\S]*\}/);
-      if (match) {
-        json = JSON.parse(match[0]);
-      } else {
-        return { valid: false, error: 'No JSON in response' };
-      }
-    }
-    
-    // Check required fields
-    if (!json.decision) {
-      return { valid: false, error: 'Missing "decision" field' };
-    }
-    
-    if (json.confidence === undefined) {
-      return { valid: false, error: 'Missing "confidence" field' };
-    }
-    
-    if (!json.reason) {
-      return { valid: false, error: 'Missing "reason" field' };
-    }
-    
-    // Validate decision value
-    const validDecisions = ['approve', 'reject', 'modify'];
-    if (!validDecisions.includes(json.decision)) {
-      return { valid: false, error: `Invalid decision: ${json.decision}` };
-    }
-    
-    // Validate confidence is number 0-100
-    const conf = Number(json.confidence);
-    if (isNaN(conf) || conf < 0 || conf > 100) {
-      return { valid: false, error: `Invalid confidence: ${json.confidence}` };
-    }
-    
-    return { valid: true, error: null };
-    
-  } catch (error) {
-    return { valid: false, error: `Parse error: ${error.message}` };
+  // Use robust JSON extraction from parser.js
+  // Handles: direct JSON, markdown code blocks, JSON with extra text (MiniMax <think> tags)
+  const json = extractJSON(content);
+  
+  if (!json) {
+    return { valid: false, error: 'No valid JSON found in response' };
+  }
+  
+  // Check required fields
+  if (!json.decision) {
+    return { valid: false, error: 'Missing "decision" field' };
+  }
+  
+  if (json.confidence === undefined) {
+    return { valid: false, error: 'Missing "confidence" field' };
   }
+  
+  if (!json.reason) {
+    return { valid: false, error: 'Missing "reason" field' };
+  }
+  
+  // Validate decision value
+  const validDecisions = ['approve', 'reject', 'modify'];
+  if (!validDecisions.includes(json.decision)) {
+    return { valid: false, error: `Invalid decision: ${json.decision}` };
+  }
+  
+  // Validate confidence is number 0-100
+  const conf = Number(json.confidence);
+  if (isNaN(conf) || conf < 0 || conf > 100) {
+    return { valid: false, error: `Invalid confidence: ${json.confidence}` };
+  }
+  
+  return { valid: true, error: null };
 };
 
 /**
@@ -338,7 +326,13 @@ const formatPreflightResults = (results, boxWidth) => {
       lines.push(dottedLine('Format', chalk.green('✓ VALID'), 9));
     } else {
       lines.push(dottedLine('Connection', chalk.red('✗ FAILED'), 9));
-      lines.push(chalk.red(`         Error: ${agent.error}`));
+      // Truncate error message to fit in box
+      const maxErrorLen = W - 16; // Account for "         Error: " prefix
+      let errorMsg = agent.error || 'Unknown error';
+      if (errorMsg.length > maxErrorLen) {
+        errorMsg = errorMsg.substring(0, maxErrorLen - 3) + '...';
+      }
+      lines.push(chalk.red(`         Error: ${errorMsg}`));
     }
     
     lines.push('');