chaimi-keep-mcp 3.1.44-beta.2 → 3.1.44-beta.4

package/README.md

@@ -141,7 +141,19 @@ AI 会自动调用 `save_income` 工具记录收入。
 
 ## 更新日志
 
-### v3.1.44-beta.1 (2026-04-20)
+### v3.1.44-beta.4 (2026-04-21)
+- **优化** 统一字段命名规范
+  - 将 `mcp_version` 改为 `mcpVersion`，统一使用驼峰命名法
+  - 符合项目 JavaScript 代码规范
+
+### v3.1.44-beta.3 (2026-04-21)
+- **修复** MCP 调用日志字段缺失问题（完整修复）
+  - 修复所有日志记录阶段（request_received/cloud_calling/cloud_success/cloud_failed/validation_failed/mcp_error）
+  - 补充缺失字段：agentType、apiProvider、mcp_version、params
+  - 修改 callMcpHubWithLogging 函数签名，添加 agentType/apiProvider 参数
+  - 更新所有调用处（8处），正确传递元数据字段
+
+### v3.1.44-beta.2 (2026-04-20)
 - **重构** `get_text_parse_prompt` 工具改为云端实现
   - 新增 `mcpPrompt` 云函数 `getTextParsePrompt` 工具
   - Prompt 包含43个标准分类（与手动记账分类完全一致）

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "chaimi-keep-mcp",
-  "version": "3.1.44-beta.2",
+  "version": "3.1.44-beta.4",
   "description": "柴米记账 MCP Server - 支持 Claude、Cursor、OpenClaw、WorkBuddy 等 AI 工具直接记账",
   "main": "server.js",
   "bin": {

package/server.js

@@ -170,7 +170,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
             agentType: { type: 'string', description: '【必填】Agent类型，如：claude-desktop、cursor、openclaw、workbuddy、trae' },
             apiProvider: { type: 'string', description: '【必填】AI服务提供商，如：anthropic、openai、doubao、aliyun' },
             rawInput: { type: 'string', description: '【必填】用户的原始输入内容，用于记录用户原始请求' },
-            mcp_version: { type: 'string', description: 'MCP Server 版本号（自动填充）' },
+            mcpVersion: { type: 'string', description: 'MCP Server 版本号（自动填充）' },
           },
           required: ['name', 'amount', 'category', 'agentType', 'apiProvider', 'rawInput'],
         },
@@ -214,7 +214,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
             agentType: { type: 'string', description: '【必填】Agent类型：claude-desktop、cursor、openclaw、workbuddy、trae' },
             apiProvider: { type: 'string', description: '【必填】AI服务提供商：anthropic、openai、doubao、aliyun' },
             rawInput: { type: 'string', description: '【必填】用户的原始输入内容' },
-            mcp_version: { type: 'string', description: '【自动填充】MCP Server版本号' },
+            mcpVersion: { type: 'string', description: '【自动填充】MCP Server版本号' },
           },
           required: ['store', 'items', 'storeCategory', 'storeSubCategory', 'agentType', 'apiProvider', 'rawInput'],
         },
@@ -315,7 +315,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
             agentType: { type: 'string', description: '【必填】Agent类型，如：claude-desktop、cursor、openclaw、workbuddy、trae' },
             apiProvider: { type: 'string', description: '【必填】AI服务提供商，如：anthropic、openai、doubao、aliyun' },
             rawInput: { type: 'string', description: '【必填】用户的原始输入内容，用于记录用户原始请求' },
-            mcp_version: { type: 'string', description: 'MCP Server 版本号（自动填充）' },
+            mcpVersion: { type: 'string', description: 'MCP Server 版本号（自动填充）' },
           },
           required: ['name', 'amount', 'category', 'agentType', 'apiProvider', 'rawInput'],
         },
@@ -538,16 +538,20 @@ function getOSInfo() {
 }
 
 // 调用 mcpHub 云函数（带日志和链路追踪）
-async function callMcpHubWithLogging(tool, params, token, traceId, startTime, osInfo) {
+async function callMcpHubWithLogging(tool, params, token, traceId, startTime, osInfo, agentType, apiProvider) {
   // 记录即将调用云函数
   logMcpCall({
     traceId,
     stage: 'cloud_calling',
     toolName: tool,
-    timestamp: new Date().toISOString(),
-    logSource: 'mcp',
+    params: sanitizeLogParams(params),
+    agentType: agentType || '',
+    apiProvider: apiProvider || '',
+    mcpVersion: MCP_VERSION,
     osType: osInfo?.osType,
-    osVersion: osInfo?.osVersion
+    osVersion: osInfo?.osVersion,
+    timestamp: new Date().toISOString(),
+    logSource: 'mcp'
   });
 
   try {
@@ -559,10 +563,13 @@ async function callMcpHubWithLogging(tool, params, token, traceId, startTime, os
       stage: 'cloud_success',
       toolName: tool,
       duration: Date.now() - startTime,
-      timestamp: new Date().toISOString(),
-      logSource: 'mcp',
+      agentType: agentType || '',
+      apiProvider: apiProvider || '',
+      mcpVersion: MCP_VERSION,
       osType: osInfo?.osType,
-      osVersion: osInfo?.osVersion
+      osVersion: osInfo?.osVersion,
+      timestamp: new Date().toISOString(),
+      logSource: 'mcp'
     });
 
     return result;
@@ -574,10 +581,13 @@ async function callMcpHubWithLogging(tool, params, token, traceId, startTime, os
       toolName: tool,
       error: cloudErr.message,
       duration: Date.now() - startTime,
-      timestamp: new Date().toISOString(),
-      logSource: 'mcp',
+      agentType: agentType || '',
+      apiProvider: apiProvider || '',
+      mcpVersion: MCP_VERSION,
       osType: osInfo?.osType,
-      osVersion: osInfo?.osVersion
+      osVersion: osInfo?.osVersion,
+      timestamp: new Date().toISOString(),
+      logSource: 'mcp'
     });
     throw cloudErr;
   }
@@ -659,6 +669,10 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
   const startTime = Date.now();
   const traceId = generateTraceId();
   const osInfo = getOSInfo();
+  
+  // 提取元数据字段
+  const agentType = args.agentType || process.env.AGENT_TYPE || process.env.MCP_AGENT_TYPE || '';
+  const apiProvider = args.apiProvider || process.env.API_PROVIDER || process.env.MCP_API_PROVIDER || '';
 
   // 记录请求到达
   logMcpCall({
@@ -668,7 +682,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
     params: sanitizeLogParams(args),
     agentType: args.agentType || process.env.AGENT_TYPE || process.env.MCP_AGENT_TYPE || '',
     apiProvider: args.apiProvider || process.env.API_PROVIDER || process.env.MCP_API_PROVIDER || '',
-    mcp_version: MCP_VERSION,
+    mcpVersion: MCP_VERSION,
     osType: osInfo.osType,
     osVersion: osInfo.osVersion,
     timestamp: new Date().toISOString(),
@@ -687,10 +701,13 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         toolName: name,
         error: 'SKILL_NOT_READ',
         duration: Date.now() - startTime,
-        timestamp: new Date().toISOString(),
-        logSource: 'mcp',
+        agentType: args.agentType || process.env.AGENT_TYPE || process.env.MCP_AGENT_TYPE || '',
+        apiProvider: args.apiProvider || process.env.API_PROVIDER || process.env.MCP_API_PROVIDER || '',
+        mcpVersion: MCP_VERSION,
         osType: osInfo.osType,
-        osVersion: osInfo.osVersion
+        osVersion: osInfo.osVersion,
+        timestamp: new Date().toISOString(),
+        logSource: 'mcp'
       });
       return {
         content: [
@@ -831,7 +848,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         }
         
         const mcpParams = convertParams('save_expense', processedArgs);
-        result = await callMcpHubWithLogging('addExpense', mcpParams, token, traceId, startTime, osInfo);
+        result = await callMcpHubWithLogging('addExpense', mcpParams, token, traceId, startTime, osInfo, agentType, apiProvider);
         
         if (result.success) {
           const displayName = processedArgs.name || '未知商品';
@@ -1098,7 +1115,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         }
         
         const mcpParams = convertParams('save_receipt', processedArgs);
-        result = await callMcpHubWithLogging('addReceipt', mcpParams, token, traceId, startTime, osInfo);
+        result = await callMcpHubWithLogging('addReceipt', mcpParams, token, traceId, startTime, osInfo, agentType, apiProvider);
         
         if (result.success) {
           const totalAmount = processedArgs.totalAmount || processedArgs.items?.reduce((sum, item) => sum + (item.amount || 0), 0) || 0;
@@ -1140,7 +1157,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
       case 'get_receipt_list': {
         const toolName = toolMapping[name];
         const mcpParams = convertParams(name, processedArgs);
-        result = await callMcpHubWithLogging(toolName, mcpParams, token, traceId, startTime, osInfo);
+        result = await callMcpHubWithLogging(toolName, mcpParams, token, traceId, startTime, osInfo, agentType, apiProvider);
 
         if (result.success) {
           const total = result.data?.total || 0;
@@ -1181,7 +1198,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
       case 'get_statistics': {
         const toolName = toolMapping[name];
         const mcpParams = convertParams(name, processedArgs);
-        result = await callMcpHubWithLogging(toolName, mcpParams, token, traceId, startTime, osInfo);
+        result = await callMcpHubWithLogging(toolName, mcpParams, token, traceId, startTime, osInfo, agentType, apiProvider);
 
         if (result.success) {
           const data = result.data || {};
@@ -1229,7 +1246,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
       case 'get_insights': {
         const toolName = toolMapping[name];
         const mcpParams = convertParams(name, processedArgs);
-        result = await callMcpHubWithLogging(toolName, mcpParams, token, traceId, startTime, osInfo);
+        result = await callMcpHubWithLogging(toolName, mcpParams, token, traceId, startTime, osInfo, agentType, apiProvider);
 
         if (result.success) {
           const data = result.data || {};
@@ -1275,7 +1292,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
       case 'export_data': {
         const toolName = toolMapping[name];
         const mcpParams = convertParams(name, processedArgs);
-        result = await callMcpHubWithLogging(toolName, mcpParams, token, traceId, startTime, osInfo);
+        result = await callMcpHubWithLogging(toolName, mcpParams, token, traceId, startTime, osInfo, agentType, apiProvider);
 
         if (result.success) {
           const summary = result.data?.summary || {};
@@ -1329,7 +1346,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         }
 
         const mcpParams = convertParams('save_income', processedArgs);
-        result = await callMcpHubWithLogging('addIncome', mcpParams, token, traceId, startTime, osInfo);
+        result = await callMcpHubWithLogging('addIncome', mcpParams, token, traceId, startTime, osInfo, agentType, apiProvider);
 
         if (result.success) {
           const displayName = processedArgs.name || '未知收入';
@@ -1397,11 +1414,11 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
           context: processedArgs.context || '',
           agentType: processedArgs.agentType || '',
           apiProvider: processedArgs.apiProvider || '',
-          mcp_version: MCP_VERSION,
+          mcpVersion: MCP_VERSION,
           userAgent: request.headers?.['user-agent'] || ''
         };
 
-        result = await callMcpHubWithLogging('addFeedback', feedbackData, token, traceId, startTime, osInfo);
+        result = await callMcpHubWithLogging('addFeedback', feedbackData, token, traceId, startTime, osInfo, agentType, apiProvider);
 
         if (result.success) {
           const typeText = {
@@ -1430,7 +1447,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
     const fullResponse = {
       userMessage: userMessage,
       result: result,
-      mcp_version: MCP_VERSION
+      mcpVersion: MCP_VERSION
     };
 
     return {
@@ -1449,10 +1466,13 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
       toolName: name,
       error: error.message,
       duration: Date.now() - startTime,
-      timestamp: new Date().toISOString(),
-      logSource: 'mcp',
+      agentType: agentType || '',
+      apiProvider: apiProvider || '',
+      mcpVersion: MCP_VERSION,
       osType: osInfo.osType,
-      osVersion: osInfo.osVersion
+      osVersion: osInfo.osVersion,
+      timestamp: new Date().toISOString(),
+      logSource: 'mcp'
     });
 
     // 处理授权错误
@@ -1689,7 +1709,7 @@ function convertParams(toolName, args) {
         rawInput: sanitizeString(args.rawInput, 1000) || '',
         agentType: args.agentType || '',
         apiProvider: args.apiProvider || '',
-        mcp_version: args.mcp_version || MCP_VERSION,
+        mcpVersion: MCP_VERSION,
         source: 'mcp_txt_expense',
       };
     }
@@ -1743,7 +1763,7 @@ function convertParams(toolName, args) {
         rawInput: sanitizeString(args.rawInput, 2000),
         agentType: args.agentType || '',
         apiProvider: args.apiProvider || '',
-        mcp_version: args.mcp_version || MCP_VERSION,
+        mcpVersion: MCP_VERSION,
         source: 'mcp_receipt',
         storeCategory: sanitizeString(args.storeCategory, 50) || '其他',
         storeSubCategory: sanitizeString(args.storeSubCategory, 50) || '其他',
@@ -1775,7 +1795,7 @@ function convertParams(toolName, args) {
         rawInput: sanitizeString(args.rawInput, 1000),
         agentType: args.agentType || '',
         apiProvider: args.apiProvider || '',
-        mcp_version: args.mcp_version || MCP_VERSION,
+        mcpVersion: MCP_VERSION,
         source: 'mcp_txt_income',
       };