mcp-osp-prompt 1.0.0 → 1.0.2

package/server.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 import process from 'process';
 import { initializePrompts, getToolsConfiguration } from './prompt-manager.js';
-import { handleDevTool, handleDevManual, handleDevFeedback, handleLoadResource } from './tools.js';
+import { handleDevTool, handleDevManual, handleDevFeedback } from './tools.js';
 import { CFG } from './config.js';
 import { 
   createSuccessResponse, 
@@ -43,12 +43,12 @@ if (debug) {
 
 // 🔄 优雅关闭处理
 process.on('SIGTERM', () => {
-  console.log('📴 [MCP-Server] Received SIGTERM, shutting down gracefully');
+  console.error('📴 [MCP-Server] Received SIGTERM, shutting down gracefully');
   process.exit(0);
 });
 
 process.on('SIGINT', () => {
-  console.log('📴 [MCP-Server] Received SIGINT, shutting down gracefully');
+  console.error('📴 [MCP-Server] Received SIGINT, shutting down gracefully');
   process.exit(0);
 });
 
@@ -58,18 +58,35 @@ let isServerReady = false;
 
 // 同步初始化函数 - 阻塞式等待初始化完成
 async function initializeServerSync() {
+  // ✅ 添加keepAlive定时器，防止客户端超时
+  let keepAliveInterval;
+  
   try {
     console.error('[MCP-Server] Starting synchronous initialization...');
     
+    // 每10秒输出一次日志，保持连接活跃
+    keepAliveInterval = setInterval(() => {
+      console.error('[MCP-KeepAlive] Initializing tools, please wait...');
+    }, 10000);
+    
     // 同步初始化prompt管理器
     globalToolsConfig = await initializePrompts();
     
+    // 清理keepAlive定时器
+    clearInterval(keepAliveInterval);
+    keepAliveInterval = null;
+    
     // 标记服务器就绪
     isServerReady = true;
     console.error(`[MCP-Server] ✅ Server ready with ${globalToolsConfig.length} tools`);
     
     return true;
   } catch (error) {
+    // 清理keepAlive定时器（如果还在运行）
+    if (keepAliveInterval) {
+      clearInterval(keepAliveInterval);
+    }
+    
     console.error('[MCP-Server] ❌ Initialization failed:', error.message);
     console.error('[MCP-Server] Server will not accept requests');
     isServerReady = false;
@@ -216,9 +233,6 @@ async function handleToolCall(req) {
 ⚠️ **MANDATORY NEXT STEP REMINDER:** 在完成下一个实施步骤后，必须再次调用dev-feedback工具汇报进度并获取确认。每个步骤完成后都需要调用feedback工具 - 这是强制要求！`;
           }
           break;
-        case 'handleLoadResource':
-          resultText = await handleLoadResource(args);
-          break;
         default:
           throw new Error(`Unknown handler: ${tool.handler}`);
       }
@@ -226,15 +240,44 @@ async function handleToolCall(req) {
       // Handle prompt-based tools
       const promptResult = await handleDevTool(name, args);
       
-      // Add mandatory feedback reminder to all prompt-based tools
-      resultText = `${promptResult}
+      // 根据AUTO模式附加不同的工作流指令
+      const isAutomatedMode = (process.env.AUTOMATED_MODE || 'false').toLowerCase() === 'true';
+      
+      if (isAutomatedMode) {
+        // AUTO=true: 简洁提醒
+        resultText = `${promptResult}
 
-⚠️ **CRITICAL WORKFLOW REQUIREMENT:**
-1. 在制定好plan后，第一步开始执行前，必须调用dev-feedback工具将计划展示给用户确认
-2. 每个实施步骤完成后，必须调用dev-feedback工具汇报进度并获取确认后再继续下一步
-3. 这是强制要求，不可跳过！每次调用feedback后会自动提醒下一步也需要再次调用feedback
+**工作流提醒:** 按计划执行，完成后汇报结果。`;
+      } else {
+        // AUTO=false: 强制分步确认+自检
+        resultText = `${promptResult}
 
-**请严格遵守此工作流程，确保每个步骤都有用户确认。**`;
+---
+## ⚠️ **CRITICAL WORKFLOW REQUIREMENT:**
+
+**每完成一步，必须调用 \`dev-feedback\` 并展示以下信息：**
+
+### 展示内容
+1. **当前阶段**：完成的步骤/阶段名称
+2. **工作摘要**：完成的主要工作内容
+3. **自检结果**：
+   - 是否有多方案未选择
+   - 是否有信息不明确或需要澄清的内容
+   - 是否发现冲突（代码/逻辑/信息）
+   - 是否偏离计划需要调整
+   - 是否即将执行重要操作（删除代码、修改核心逻辑、数据库变更等）
+   - 是否存在"TBD"、"待确认"、"Question"等需要用户确认的内容
+   - 是否涉及项目/系统范围确认（无论已确认还是待确认，都必须调用 dev-feedback 让用户确认）
+4. **下一步**：下一步计划（如有）
+
+**最后一步结束后，确认：是否满足所有需求和完成了计划？**
+
+### 规则
+- 宁可多确认，不可漏确认
+- 严禁将待确认内容留到最终文档
+
+**⚠️ 立即调用 dev-feedback 开始确认！**`;
+      }
     } else {
       throw new Error(`Unknown tool type: ${tool.type}`);
     }
@@ -248,7 +291,7 @@ async function handleToolCall(req) {
     
     // 特殊处理：某些"错误"实际上是正常的用户交互结果
     if (error.message.includes('用户要求调整计划')) {
-      console.log(`[MCP信息] 用户计划调整请求: ${error.message}`);
+      console.error(`[MCP信息] 用户计划调整请求: ${error.message}`);
       return createSuccessResponse(req.id, {
         content: [{ type: 'text', text: error.message }]
       });
@@ -260,7 +303,7 @@ async function handleToolCall(req) {
     // 针对常见错误场景提供更清晰的信息
     if (error.message.includes('fetch fail') || error.message.includes('API')) {
       errorMsg = `网络请求失败 - ${error.message}`;
-    } else if (error.message.includes('RESOURCE_PATH')) {
+    } else if (error.message.includes('PROMPT_PATH')) {
       errorMsg = `配置错误 - ${error.message}`;
     } else if (error.message.includes('Unknown prompt file')) {
       errorMsg = `文件未找到 - ${error.message}`;

package/test.js

@@ -9,7 +9,7 @@ console.log('🧪 Starting comprehensive MCP server tests...');
 (async () => {
   // Set automated mode to skip GUI dialogs and required environment variables
   process.env.AUTOMATED_MODE = 'true';
-  process.env.RESOURCE_PATH = '../dev-arch'; // Use local test path (parent directory)
+  process.env.PROMPT_PATH = '../dev-arch'; // Use local test path (parent directory)
   process.env.GIT_TOKEN = 'test-token'; // Dummy token for testing
   
   const proc = spawn('node', ['server.js'], { 
@@ -18,7 +18,7 @@ console.log('🧪 Starting comprehensive MCP server tests...');
     env: { 
       ...process.env, 
       AUTOMATED_MODE: 'true',
-      RESOURCE_PATH: '../dev-arch',
+      PROMPT_PATH: '../dev-arch',
       GIT_TOKEN: 'test-token'
     }
   });

package/tools.js

@@ -50,9 +50,6 @@ export function processPromptTemplate(content, context = {}) {
   const sourceText = context.source || '';
   processedContent = processedContent.replace(/\$\{input:source[^}]*}/g, sourceText);
   
-  // Replace requirements variable (used in design.prompt.md)
-  processedContent = processedContent.replace(/\$\{input:requirements[^}]*}/g, sourceText);
-  
   // Inject project context if available
   if (context.projectInfo) {
     processedContent = processedContent.replace(/\$\{project:([^}]+)}/g, (match, key) => {
@@ -84,41 +81,13 @@ export function processPromptTemplate(content, context = {}) {
 export function generateStepModeInstructions(context = {}) {
   const sourceText = context.source || '未指定需求';
   
+  // 简化版：主要逻辑已统一移到server.js handleToolCall处理
+  // 这里只提供基本的需求上下文，避免重复
   return `
 
-## 🔄 分步交互模式已启用
-当前需求：${sourceText}
-
-### 🎯 分步执行指南
-1. **计划阶段**: 先分析需求并制定实施计划，然后调用 \`dev-feedback\` 工具请求用户确认
-2. **实施阶段**: 每完成一个重要步骤后，使用 \`dev-feedback\` 工具询问用户是否继续
-3. **实施阶段**: 连续出现3次执行错误，则使用 \`dev-feedback\` 工具请求用户确认是否需要继续或修改计划
-4. **实施阶段**: 如果执行的过程中，需要使用不在计划中的新方案或新思路，则使用 \`dev-feedback\` 工具请求用户确认是否需要继续或修改计划
-5. **确认选项**: 
-   - "继续" - 按计划继续执行
-   - "修改" - 调整当前计划或步骤
-   - "暂停" - 暂停开发等待进一步指示
-
-### 📞 如何调用确认工具
-
-**调用方式** :
-统一的弹窗选项，所有阶段都使用相同的简洁选择：
-
-- **所有阶段**: 统一使用 ["继续", "修改计划"] 两个选项
-- **用户体验**: 选择"继续"将按当前计划执行，选择"修改计划"将弹出输入框供用户提供调整意见
-- **简化设计**: 移除了复杂的多选项，采用二元选择 + 输入框的设计模式
-
-**示例调用**:
-\`\`\`
-调用 dev-feedback 工具，参数：
-{
-  "title": "开发计划确认",
-  "message": "已分析需求并制定开发计划，包含以下步骤：1)设计数据模型 2)实现API接口 3)编写单元测试 4)集成测试。是否开始实施？",
-  "phase": "planning"
-}
-\`\`\`
-
-**重要**: 在每个关键节点都要主动请求用户确认，确保开发方向正确。优先使用阶段特定的feedback调用方式。`;
+---
+**当前需求：** ${sourceText}
+---`;
 }
 
 /**
@@ -276,8 +245,6 @@ export async function enhancedDevFeedback(feedbackInput = {}) {
   // 简化逻辑：直接传入options，函数内部自动判断场景
   const options = getStandardButtons(feedbackInput.options);
   
-  console.log('📋 [ENHANCED-FEEDBACK] Applied unified standard buttons:', options);
-  
   // Standard message processing - no version info in intermediate feedback
   let enhancedMessage = feedbackInput.message;
   
@@ -434,7 +401,7 @@ export async function handleDevTool(toolName, args) {
     let versionInfo = '';
     try {
       const info = await getLocalPromptInfo(fileName);
-      versionInfo = `\n\n📌 V: ${info.version}`;
+      versionInfo = `\n\n📌 Prompt版本: ${info.version}`;
     } catch (error) {
       console.warn(`[Version] Failed to get version for ${fileName}:`, error.message);
     }
@@ -501,80 +468,6 @@ When creating directories or referencing dates, always use ${currentDate}.
   // Prepend date context to the beginning of the content
   content = dateContextHeader + content;
   
-  // 🟢 GREEN: 统一注入完整的resources列表（projects + rules）
-  try {
-    // 动态导入避免循环依赖
-    const { formatResourcesList } = await import('./resource-manager.js');
-    const { UNIFIED_RESOURCES } = await import('./config.js');
-    
-    // 统一加载完整的projects和rules列表
-    const resourcesList = await formatResourcesList(UNIFIED_RESOURCES);
-    
-    // 构建前置内容
-    let prefixContent = '---\n\n';
-    prefixContent += resourcesList + '\n';
-    
-    if (!automatedMode) {
-      // 手动模式：要求通过feedback确认资源选择
-      prefixContent += `
-**📋 执行规则 (MANDATORY EXECUTION RULES):**
-
-1. **每阶段完成必须反馈**: 完成每个阶段/步骤后，必须调用 \`dev-feedback\` 工具汇报进度并获取确认。
-2. **等待用户确认**: 每次feedback后必须等待用户确认才能继续下一步。
-3. **这是强制要求**: 不是可选项，每个阶段都必须执行。
-
----
-
-**🎯 资源确认流程 (RESOURCE CONFIRMATION - auto=false 模式):**
-
-在开始详细分析或实施工作之前：
-
-1. **审查可用资源**: 仔细查看上方提供的projects和rules资源列表
-2. **分析需求**: 根据需求/任务，判断需要使用哪些资源
-3. **选择资源**: 
-   - **项目文档 (Projects)**: 选择相关的系统wiki文档
-   - **开发规则 (Rules)**: 选择相关的编码规范和标准
-4. **强制确认**: 必须调用 \`dev-feedback\` 工具向用户确认：
-   - 需求范围总结
-   - 选定的项目文档列表（说明原因）
-   - 选定的规则文档列表（说明原因）
-   - 询问："确认范围正确吗？可以继续吗？"
-5. **等待批准**: 在用户确认前不得继续
-6. **加载资源**: 确认后，使用 \`load-resource\` 工具加载已批准的资源
-
-**此确认步骤是强制性的且阻塞性的 - 不能跳过。**
-
----
-
-`;
-    } else {
-      // 自动模式：AI自行选择并加载resources
-      prefixContent += `
-**📋 AUTO 模式 - 自动资源管理:**
-
-系统已提供完整的项目文档和开发规则列表。
-
-**工作流程:**
-1. 根据任务需求，自行判断并选择需要的resources
-2. 使用 \`load-resource\` 工具加载选定的资源内容
-3. 所有projects和rules都可用，按需加载
-
-**可用资源:**
-- **Projects**: 所有系统项目文档
-- **Rules**: 所有开发规范和标准
-
-直接开始分析和实施，根据需要使用 \`load-resource\` 工具。
-
----
-
-`;
-    }
-    
-    content = prefixContent + content;
-  } catch (error) {
-    console.warn('[handleDevTool] Failed to inject resources list:', error.message);
-  }
-  
   // 版本信息现在在弹窗中显示，不再添加到response中
   return content;
 }
@@ -602,9 +495,10 @@ export async function handleDevFeedback(args) {
   // 简化逻辑：直接传入options，函数内部自动判断场景
   const finalOptions = getStandardButtons(options.length > 0 ? options : null);
   
-  console.log('🔧 [BUTTON-MANAGEMENT] Applied unified standard buttons:', finalOptions);
-  
-  console.log('📞 [DEV-FEEDBACK] Showing dialog:', { title, message, options: finalOptions });
+  // 🆕 添加保活机制：在等待用户交互期间定期输出日志
+  const keepAliveInterval = setInterval(() => {
+    console.error('[MCP-KeepAlive] Waiting for user dialog interaction...');
+  }, 15000);
   
   try {
     const result = await showConfirmDialog({ 
@@ -613,9 +507,10 @@ export async function handleDevFeedback(args) {
       buttons: finalOptions
     });
     
+    clearInterval(keepAliveInterval); // 用户选择后清理定时器
+    
     // If user selects a "modify" option, automatically show input dialog
     if (result && (result.includes('修改') || result.includes('调整') || result.includes('建议'))) {
-      console.log('📝 [INPUT REQUEST] User wants to provide input, showing input dialog...');
       
       const userInput = await showInputDialog({
         title: '输入修改建议',
@@ -624,8 +519,6 @@ export async function handleDevFeedback(args) {
         rows: 5  // Use 5-row input for modification suggestions
       });
       
-      console.log('✅ [USER INPUT]:', userInput);
-      
       // Return structured feedback with clear indication for AI processing
       return {
         action: result,
@@ -635,9 +528,9 @@ export async function handleDevFeedback(args) {
         requiresAdjustment: true  // Flag to indicate AI should adjust execution
       };
     }
-    console.log('✅ [USER RESPONSE]:', result);
     return result;
   } catch (error) {
+    clearInterval(keepAliveInterval); // 出错时也要清理定时器
     console.error('❌ [FEEDBACK ERROR]:', error.message);
     
     // Check if this is a dialog system failure
@@ -672,30 +565,4 @@ export async function handleDevFeedback(args) {
 }
 
 // handleDevReload removed - configuration now loads automatically from environment and mcp.json
-// No manual reload needed as per user requirements
-
-/**
- * 🟢 GREEN: Handle load-resource tool
- * 加载resource的完整内容
- */
-export async function handleLoadResource(args) {
-  const { resourceName } = args;
-  
-  if (!resourceName) {
-    return `❌ Error: resourceName is required`;
-  }
-  
-  try {
-    // 动态导入避免循环依赖
-    const { loadResourceContent } = await import('./resource-manager.js');
-    const result = await loadResourceContent(resourceName);
-    
-    if (!result.success) {
-      return `❌ Error loading resource "${resourceName}": ${result.error}`;
-    }
-    
-    return `# Resource: ${result.filename} (${result.type})\n\n${result.content}`;
-  } catch (error) {
-    return `❌ Error: ${error.message}`;
-  }
-}
+// No manual reload needed as per user requirements