job51-gitlab-cr-node-skill-prompt-optimize 1.0.6 → 1.0.8

package/mr-review-template.md

@@ -32,20 +32,20 @@
 
 ### 🔴 严重问题
 
-  **文件行号**: [问题所在文件相关代码块具体开始行号，带有完整类路径的具体行路径，格式类似：com/job51/dev/anteros/service/impl/ApplyPushServiceImpl.java:176]<br/>
+  **文件及行号**: [问题所在文件相关代码块具体开始至结束行号，带有完整类路径的具体行路径，格式类似：com/job51/dev/anteros/service/impl/ApplyPushServiceImpl.java:176-180]<br/>
   **问题描述**: [具体问题描述]<br/>
   **修改建议**: [修复建议和示例代码]
 
 ### 🟡 警告
 
-  **文件行号**: [问题所在文件相关代码块具体开始行号，带有完整类路径的具体行路径，格式类似：com/job51/dev/anteros/service/impl/ApplyPushServiceImpl.java:176]<br/>
+  **文件及行号**: [问题所在文件相关代码块具体开始至结束行号，带有完整类路径的具体行路径，格式类似：com/job51/dev/anteros/service/impl/ApplyPushServiceImpl.java:176-180]<br/>
   **问题描述**: [具体问题描述]<br/>
   **修改建议**: [改进建议]
 
 ### 🟢 优化建议
 
-  **文件行号**: [问题所在文件相关代码块具体开始行号，带有完整类路径的具体行路径，格式类似：com/job51/dev/anteros/service/impl/ApplyPushServiceImpl.java:176]<br/>
-  **优化建议**: [优化建议]<br/>
+  **文件及行号**: [问题所在文件相关代码块具体开始至结束行号，带有完整类路径的具体行路径，格式类似：com/job51/dev/anteros/service/impl/ApplyPushServiceImpl.java:176-180]<br/>
+  **优化建议**: [优化建议及可以直接使用的修改提示词]<br/>
   **示例代码**: [优化后的代码示例]
 
 ### 🔄 上下文影响分析（需涵盖依赖关系、功能关联性、兼容性风险三个维度）

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "job51-gitlab-cr-node-skill-prompt-optimize",
-  "version": "1.0.6",
+  "version": "1.0.8",
   "description": "GitLab merge request code review tool with AI-powered analysis",
   "main": "index.js",
   "bin": {
@@ -12,7 +12,7 @@
     "example": "node example.js"
   },
   "keywords": [],
-  "author": "Linton Cao",
+  "author": "tao.jing",
   "license": "MIT",
   "dependencies": {
     "axios": "^1.6.0"

package/analyze_filepath.js

@@ -1,66 +0,0 @@
-// 详细分析Claude命令的文件路径问题
-const fs = require('fs');
-const path = require('path');
-
-console.log('=== 分析Claude命令文件路径问题 ===\n');
-
-// 1. 查看当前工作目录
-console.log(`当前工作目录: ${process.cwd()}`);
-
-// 2. 分析临时文件路径创建逻辑
-const timestamp = Date.now();
-const blockIndex = 0;
-const relativeTmpPath = `./temp-diff-block-${timestamp}-${blockIndex}.diff`;
-const absoluteTmpPath = path.resolve(relativeTmpPath);
-
-console.log(`相对路径: ${relativeTmpPath}`);
-console.log(`绝对路径: ${absoluteTmpPath}`);
-
-// 3. 检查index.js中的实际实现逻辑
-console.log('\n=== index.js中实际的文件路径逻辑 ===');
-
-console.log('在index.js的第120行:');
-console.log('  const tmpFileName = `./temp-diff-block-${Date.now()}-${blockIndex}.diff`;');
-
-console.log('在index.js的第124行:');
-console.log('  fs.writeFileSync(tmpFileName, diffObject.diff);');
-
-console.log('在index.js的第127行:');
-console.log('  const review_result = await this.reviewDiffWithClaudeUsingFile(tmpFileName);');
-
-console.log('在index.js的第198行:');
-console.log('  const prompt = \'/simple-code-review\' + \' \' + filePath;');
-
-console.log('这形成了如下操作:');
-console.log(`  1. 创建文件: ${relativeTmpPath}`);
-console.log(`  2. 调用: claude -p "/simple-code-review ${relativeTmpPath}"`);
-
-console.log('\n=== 关键问题 ===');
-console.log('1. 临时文件是在当前工作目录创建的');
-console.log('2. 使用相对路径传递给Claude命令');
-console.log('3. Claude命令是否能在其执行环境中找到这个相对路径的文件？');
-
-// 检查Claude在调用时的工作目录是否与临时文件创建目录一致
-console.log('\n=== 可能的解决方案 ===');
-console.log('1. 使用绝对路径而不是相对路径传递给Claude命令');
-console.log('2. 确保Claude命令在相同的工作目录中执行');
-console.log('3. 或者修改Claude技能以接受文件内容而不是文件路径');
-
-console.log('\n在第525行的runClaudeCommand函数中:');
-console.log('  cwd: __dirname,   // 设置工作目录为当前脚本所在目录');
-console.log('这应该确保Claude命令在正确的目录中执行，但可能在CI环境中仍有问题');
-
-// 模拟index.js中确切的代码流程
-console.log('\n=== 模拟精确的执行流程 ===');
-console.log('1. 创建临时文件 (使用相对路径):');
-console.log('   fs.writeFileSync("./temp-diff-block-xxx.diff", diff_content)');
-
-console.log('2. 生成prompt:');
-console.log('   prompt = "/simple-code-review ./temp-diff-block-xxx.diff"');
-
-console.log('3. 调用Claude命令:');
-console.log('   claude -p "/simple-code-review ./temp-diff-block-xxx.diff"');
-
-console.log('4. Claude的/simple-code-review技能可能需要读取该文件');
-
-console.log('\n问题: 如果Claude在其自己的工作目录中运行，可能找不到这个相对路径的文件。');

package/analyze_issue.js

@@ -1,68 +0,0 @@
-// 专门测试在GitLab CI中可能遇到的问题
-const { spawn, exec } = require('child_process');
-
-console.log('=== 分析GitLab CI中可能遇到的问题 ===');
-
-// 首先，测试在移除CLAUDECODE变量后直接执行claude命令
-console.log('1. 检查系统中是否安装了claude命令（移除CLAUDECODE环境变量后）...');
-
-const envWithoutClaudeCode = { ...process.env };
-delete envWithoutClaudeCode.CLAUDECODE;
-delete envWithoutClaudeCode.CLAUDE_CODE_ENTRYPOINT;
-
-// 使用exec同步检查命令是否存在
-exec('which claude', { env: envWithoutClaudeCode }, (error, stdout, stderr) => {
-  if (error) {
-    console.log('  -> Claude命令未找到或不可执行 (exit code: ' + error.code + ')');
-    console.log('  -> 这很可能是GitLab CI环境中的问题根源');
-
-    // 测试在GitLab CI中常见的情况：缺少claude CLI
-    console.log('\n2. 这就是GitLab CI环境中的典型问题：');
-    console.log('   - 没有安装Claude CLI');
-    console.log('   - Claude CLI无法访问或执行');
-    console.log('   - 权限不足');
-    console.log('   - PATH环境变量不同');
-
-    console.log('\n3. 因此，runClaudeCommand函数在GitLab CI中会：');
-    console.log('   - spawn执行claude命令失败');
-    console.log('   - 触发error事件');
-    console.log('   - 如果错误处理不当，可能返回空字符串');
-    console.log('   - 导致claudeResult为空');
-
-    console.log('\n4. 解决方案可能包括：');
-    console.log('   - 在GitLab CI中安装Claude CLI');
-    console.log('   - 修改脚本使用其他方式调用Claude（如API调用）');
-    console.log('   - 添加适当的错误处理和回退机制');
-  } else {
-    console.log('  -> Claude命令存在: ' + stdout.trim());
-
-    // 如果claude命令存在，测试执行它
-    console.log('\n5. Claude命令存在，尝试执行简单的Claude命令...');
-    const claudeTest = spawn('claude', ['-p', 'hello'], {
-      env: envWithoutClaudeCode,
-      shell: true
-    });
-
-    let testOutput = '';
-    let testError = '';
-
-    claudeTest.stdout.on('data', (data) => {
-      testOutput += data.toString();
-      console.log('  -> Claude输出: ' + data.toString().substring(0, 100) + '...');
-    });
-
-    claudeTest.stderr.on('data', (data) => {
-      testError += data.toString();
-      console.log('  -> Claude错误: ' + data.toString());
-    });
-
-    claudeTest.on('close', (code) => {
-      console.log('  -> Claude命令结束，退出码: ' + code);
-      if (code === 0) {
-        console.log('  -> Claude命令执行成功');
-      } else {
-        console.log('  -> Claude命令执行失败，这可能是另一个问题原因');
-      }
-    });
-  }
-});

package/debug_test.js

@@ -1,89 +0,0 @@
-const { spawn } = require('child_process');
-
-// 测试在不同环境下 claude 命令的行为
-function testClaudeCommand() {
-  console.log('开始测试Claude命令...');
-
-  // 测试1: 直接执行 claude 命令
-  console.log('\n--- 测试1: 检查claude命令 ---');
-  const checkCmd = spawn('which', ['claude'], {
-    shell: true,
-    stdio: ['ignore', 'pipe', 'pipe']
-  });
-
-  let checkStdout = '';
-  let checkStderr = '';
-
-  checkCmd.stdout.on('data', (data) => {
-    checkStdout += data.toString();
-    console.log('which claude stdout:', data.toString());
-  });
-
-  checkCmd.stderr.on('data', (data) => {
-    checkStderr += data.toString();
-    console.log('which claude stderr:', data.toString());
-  });
-
-  checkCmd.on('close', (code) => {
-    console.log('which claude exit code:', code);
-    console.log('which claude result:', checkStdout.trim() || 'Command not found');
-
-    // 测试2: 执行 claude --version
-    console.log('\n--- 测试2: 执行claude --version ---');
-    const versionCmd = spawn('claude', ['--version'], {
-      shell: true,
-      stdio: ['ignore', 'pipe', 'pipe']
-    });
-
-    let versionStdout = '';
-    let versionStderr = '';
-
-    versionCmd.stdout.on('data', (data) => {
-      versionStdout += data.toString();
-      console.log('claude --version stdout:', data.toString());
-    });
-
-    versionCmd.stderr.on('data', (data) => {
-      versionStderr += data.toString();
-      console.log('claude --version stderr:', data.toString());
-    });
-
-    versionCmd.on('close', (code) => {
-      console.log('claude --version exit code:', code);
-
-      // 测试3: 执行 claude 命令带简单提示
-      console.log('\n--- 测试3: 执行claude命令带简单提示 ---');
-      const claudeCmd = spawn('claude', ['-p', 'hello world'], {
-        shell: true,
-        stdio: ['ignore', 'pipe', 'pipe']
-      });
-
-      let claudeStdout = '';
-      let claudeStderr = '';
-
-      claudeCmd.stdout.on('data', (data) => {
-        claudeStdout += data.toString();
-        console.log('claude [-p hello world] stdout chunk:', data.toString().substring(0, 100) + '...');
-      });
-
-      claudeCmd.stderr.on('data', (data) => {
-        claudeStderr += data.toString();
-        console.log('claude [-p hello world] stderr:', data.toString());
-      });
-
-      claudeCmd.on('close', (code) => {
-        console.log('claude command exit code:', code);
-        console.log('claude command final stdout:', claudeStdout ? claudeStdout.substring(0, 200) + '...' : '(empty)');
-        console.log('claude command final stderr:', claudeStderr || '(empty)');
-
-        console.log('\n--- 环境信息 ---');
-        console.log('NODE_ENV:', process.env.NODE_ENV);
-        console.log('SHELL:', process.env.SHELL);
-        console.log('PROCESSOR_ARCHITECTURE:', process.env.PROCESSOR_ARCHITECTURE);
-        console.log('OSTYPE:', process.env.OSTYPE || 'Not set');
-      });
-    });
-  });
-}
-
-testClaudeCommand();

package/env_check.js

@@ -1,32 +0,0 @@
-// 检查当前运行环境
-console.log('=== 环境检测 ===');
-console.log('CLAUDECODE 环境变量:', process.env.CLAUDECODE);
-console.log('是否在 Claude Code 环境中:', !!process.env.CLAUDECODE);
-
-// 检查其他可能的环境变量
-console.log('TERM:', process.env.TERM);
-console.log('COLORTERM:', process.env.COLORTERM);
-console.log('TERM_PROGRAM:', process.env.TERM_PROGRAM);
-
-// 检查是否有其他指示运行环境的变量
-console.log('CI:', process.env.CI);
-console.log('GITLAB_CI:', process.env.GITLAB_CI);
-
-// 尝试执行一些基本的系统命令
-const { spawn } = require('child_process');
-
-console.log('\n=== 尝试运行简单命令 ===');
-
-// 测试环境变量
-const envTest = spawn('env');
-
-let envOutput = '';
-envTest.stdout.on('data', (data) => {
-  envOutput += data.toString();
-});
-
-envTest.on('close', () => {
-  console.log('所有环境变量:');
-  const claudeVars = envOutput.split('\n').filter(line => line.includes('CLAUDE') || line.includes('CODE'));
-  console.log('包含CLAUDE/CODE的环境变量:', claudeVars);
-});