@xagent-ai/cli 1.3.5 → 1.3.6

package/src/ai-client/providers/openai.ts

@@ -103,6 +103,10 @@ export class OpenAIProvider implements AIProvider {
       return this.convertResponse(response.data, model);
     }, retryConfig);
 
+    if (showDebug && result.data) {
+      this.debugResponse(result.data);
+    }
+
     if (result.success) {
       return result.data!;
     }
@@ -304,9 +308,61 @@ export class OpenAIProvider implements AIProvider {
     console.log(`📦 Model: ${model}`);
     console.log(`💬 Messages: ${messages.length}`);
     console.log('─'.repeat(60));
+    
+    // Print each message content
+    messages.forEach((msg, idx) => {
+      const role = msg.role || 'unknown';
+      const content = msg.content || '';
+      const reasoning = msg.reasoning_content ? `\n  [Reasoning] ${msg.reasoning_content}` : '';
+      const toolCalls = msg.tool_calls ? `\n  [Tool Calls] ${JSON.stringify(msg.tool_calls, null, 2)}` : '';
+      const toolCallId = msg.tool_call_id ? `\n  [Tool Call ID] ${msg.tool_call_id}` : '';
+      
+      console.log(`\n[${idx + 1}] Role: ${role}${reasoning}${toolCalls}${toolCallId}`);
+      if (content) {
+        // No truncation for system messages or any content
+        console.log(`    Content: ${content}`);
+      }
+    });
+    
     console.log('\n📤 Sending request...\n');
   }
 
+  /**
+   * Debug response output
+   */
+  private debugResponse(response: CompletionResponse): void {
+    console.log(`\n╔══════════════════════════════════════════════════════════╗`);
+    console.log(`║             AI RESPONSE DEBUG (OpenAI)                    ║`);
+    console.log(`╚══════════════════════════════════════════════════════════╝`);
+    console.log(`🆔 ID: ${response.id}`);
+    console.log(`📦 Model: ${response.model}`);
+    console.log(`⏱️  Created: ${response.created}`);
+    console.log('─'.repeat(60));
+    
+    // Print each choice
+    response.choices?.forEach((choice, idx) => {
+      const msg = choice.message;
+      const content = msg?.content || '';
+      const reasoning = msg?.reasoning_content ? `\n  [Reasoning] ${msg.reasoning_content}` : '';
+      const toolCalls = msg?.tool_calls ? `\n  [Tool Calls] ${JSON.stringify(msg.tool_calls, null, 2)}` : '';
+      
+      console.log(`\n[Choice ${idx}] Finish Reason: ${choice.finish_reason}${reasoning}${toolCalls}`);
+      if (content) {
+        // No truncation
+        console.log(`    Content: ${content}`);
+      }
+    });
+    
+    if (response.usage) {
+      console.log('\n📊 Usage:');
+      console.log(`   Prompt tokens: ${response.usage.prompt_tokens}`);
+      console.log(`   Completion tokens: ${response.usage.completion_tokens}`);
+      console.log(`   Total tokens: ${response.usage.total_tokens}`);
+    }
+    
+    console.log('\n📥 Response received.\n');
+  }
+
   /**
    * Handle errors
    */

package/src/ai-client/providers/remote.ts

@@ -275,6 +275,22 @@ export class RemoteProvider implements AIProvider {
     console.log(`🌐 Base URL: ${this.webBaseUrl}`);
     console.log(`💬 Messages: ${messages.length}`);
     console.log('─'.repeat(60));
+    
+    // Print each message content
+    messages.forEach((msg, idx) => {
+      const role = msg.role || 'unknown';
+      const content = msg.content || '';
+      const reasoning = msg.reasoning_content ? `\n  [Reasoning] ${msg.reasoning_content}` : '';
+      const toolCalls = msg.tool_calls ? `\n  [Tool Calls] ${JSON.stringify(msg.tool_calls, null, 2)}` : '';
+      const toolCallId = msg.tool_call_id ? `\n  [Tool Call ID] ${msg.tool_call_id}` : '';
+      
+      console.log(`\n[${idx + 1}] Role: ${role}${reasoning}${toolCalls}${toolCallId}`);
+      if (content) {
+        // No truncation
+        console.log(`    Content: ${content}`);
+      }
+    });
+    
     console.log('\n📤 Sending request to Remote API...\n');
   }
 
@@ -285,11 +301,34 @@ export class RemoteProvider implements AIProvider {
     console.log('\n╔══════════════════════════════════════════════════════════╗');
     console.log('║             AI RESPONSE DEBUG (REMOTE)                   ║');
     console.log('╚══════════════════════════════════════════════════════════╝');
-    console.log(`🆔 ID: remote-${Date.now()}`);
-    console.log(`📏 Content length: ${data.content?.length || 0}`);
-    if (data.tool_calls?.length) {
-      console.log(`🔧 Tool calls: ${data.tool_calls.length}`);
+    console.log(`🆔 ID: ${data.id || `remote-${Date.now()}`}`);
+    
+    const content = data?.message || data?.content || '';
+    const reasoning = data?.reasoning_content || data?.reasoningContent || '';
+    const toolCalls = data?.tool_calls || data?.toolCalls || [];
+    
+    if (reasoning) {
+      console.log(`\n💭 Reasoning: ${reasoning}`);
+    }
+    
+    if (content) {
+      console.log(`\n📝 Content: ${content}`);
+    }
+    
+    if (toolCalls?.length) {
+      console.log(`\n🔧 Tool calls: ${toolCalls.length}`);
+      toolCalls.forEach((tc: any, idx: number) => {
+        console.log(`   [${idx + 1}] ${tc.function?.name || tc.name}: ${tc.function?.arguments || tc.arguments}`);
+      });
+    }
+    
+    if (data?.usage) {
+      console.log('\n📊 Usage:');
+      console.log(`   Prompt tokens: ${data.usage.prompt_tokens}`);
+      console.log(`   Completion tokens: ${data.usage.completion_tokens}`);
+      console.log(`   Total tokens: ${data.usage.total_tokens}`);
     }
+    
     console.log('\n📥 Response received.\n');
   }
 

package/src/ai-client-factory.ts

@@ -150,6 +150,7 @@ export function createAIClient(authConfig: AuthConfig): AIClientInterface {
       type: 'remote',
       authToken: authConfig.apiKey || '',
       baseUrl: authConfig.xagentApiBaseUrl || 'https://app.xagent.cn',
+      showDebugInfo: authConfig.showAIDebugInfo,
     };
   } else if (authConfig.baseUrl?.includes('anthropic') || 
              authConfig.baseUrl?.includes('minimax') ||
@@ -160,6 +161,7 @@ export function createAIClient(authConfig: AuthConfig): AIClientInterface {
       apiKey: authConfig.apiKey || '',
       baseUrl: authConfig.baseUrl || 'https://api.anthropic.com',
       model: authConfig.modelName || 'claude-sonnet-4-20250514',
+      showDebugInfo: authConfig.showAIDebugInfo,
     };
   } else {
     // Default to OpenAI compatible mode
@@ -168,6 +170,7 @@ export function createAIClient(authConfig: AuthConfig): AIClientInterface {
       apiKey: authConfig.apiKey || '',
       baseUrl: authConfig.baseUrl || 'https://api.openai.com/v1',
       model: authConfig.modelName || 'gpt-4o',
+      showDebugInfo: authConfig.showAIDebugInfo,
     };
   }
 

package/src/smart-approval.ts

@@ -80,7 +80,8 @@ export class WhitelistChecker {
     'ask_user_question',
     'save_memory',
     'xml_escape',
-    'Skill',
+    'InvokeSkill',
+    'task',
   ]);
 
   /**

package/src/tools.ts

@@ -3525,6 +3525,14 @@ export class InvokeSkillTool implements Tool {
   }> {
     const { skillId, taskDescription, inputFile, outputFile, options } = params;
 
+    // Validate required parameters
+    if (!skillId || skillId.trim() === '') {
+      throw new Error('Missing required parameter: skillId is required to invoke a skill');
+    }
+    if (!taskDescription || taskDescription.trim() === '') {
+      throw new Error('Missing required parameter: taskDescription is required to invoke a skill');
+    }
+
     try {
       const { getSkillInvoker } = await import('./skill-invoker.js');
       const skillInvoker = getSkillInvoker();
@@ -4470,44 +4478,6 @@ export class ToolRegistry {
     if (executionMode === ExecutionMode.SMART) {
       const debugMode = process.env.DEBUG === 'smart-approval';
 
-      // task tool bypasses smart approval entirely
-      if (toolName === 'task') {
-        if (debugMode) {
-          const { getLogger } = await import('./logger.js');
-          const logger = getLogger();
-          logger.debug(
-            `[SmartApprovalEngine] Tool '${toolName}' bypassed smart approval completely`
-          );
-        }
-        return await cancellationManager.withCancellation(
-          tool.execute(params, executionMode),
-          `tool-${toolName}`
-        );
-      }
-
-      // Remote mode (OAuth XAGENT): remote LLM has already approved the tool
-      // Auto-approve InvokeSkill tools without local AI review
-      const { getConfigManager } = await import('./config.js');
-      const configManager = getConfigManager();
-      const authConfig = configManager.getAuthConfig();
-      const isRemoteMode = authConfig.type === AuthType.OAUTH_XAGENT;
-      if (isRemoteMode && toolName === 'InvokeSkill') {
-        console.log('');
-        if (isSdkMode && sdkOutputAdapter) {
-          sdkOutputAdapter.outputInfo(`[Smart Mode] Remote mode: tool '${toolName}' auto-approved (remote LLM already approved)`);
-        } else {
-          console.log('');
-          console.log(
-            `${indent}${colors.success(`✅ [Smart Mode] Remote mode: tool '${toolName}' auto-approved (remote LLM already approved)`)}`
-          );
-          console.log('');
-        }
-        return await cancellationManager.withCancellation(
-          tool.execute(params, executionMode),
-          `tool-${toolName}`
-        );
-      }
-
       const { getSmartApprovalEngine } = await import('./smart-approval.js');
 
       const approvalEngine = getSmartApprovalEngine(debugMode);