job51-gitlab-cr-node-skill-prompt-optimize 1.0.0

package/README.md

@@ -0,0 +1,58 @@
+# GitLab Code Review AI Tool
+
+一个使用 AI 自动审查 GitLab 合并请求的工具，基于 Claude AI 进行代码分析。
+
+## 安装
+
+通过 npm 全局安装：
+
+```bash
+npm install -g gitlab-cr-node
+```
+
+## 使用方法
+
+### 1. 环境变量配置
+
+在使用前需要设置以下环境变量：
+
+```bash
+export CI_API_V4_URL="https://your-gitlab-instance.com/api/v4"
+export GITLAB_ACCESS_TOKEN="your_gitlab_access_token"
+export CI_PROJECT_ID="your_project_id"
+export CI_MERGE_REQUEST_IID="your_merge_request_iid"
+```
+
+或者在 Windows PowerShell 中：
+
+```powershell
+$env:CI_API_V4_URL="https://your-gitlab-instance.com/api/v4"
+$env:GITLAB_ACCESS_TOKEN="your_gitlab_access_token"
+$env:CI_PROJECT_ID="your_project_id"
+$env:CI_MERGE_REQUEST_IID="your_merge_request_iid"
+```
+
+### 2. 运行代码审查
+
+设置好环境变量后，运行：
+
+```bash
+gitlab-cr
+```
+
+## 功能特性
+
+- 自动获取 GitLab 合并请求的代码变更
+- 使用 Claude AI 进行代码审查
+- 生成结构化的审查报告
+- 将审查结果发布到 GitLab MR
+
+## 依赖要求
+
+- Node.js 10+
+- Claude CLI 工具
+- GitLab 访问令牌
+
+## 许可证
+
+MIT

package/analyze_filepath.js

@@ -0,0 +1,66 @@
+// 详细分析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

@@ -0,0 +1,68 @@
+// 专门测试在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

@@ -0,0 +1,89 @@
+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

@@ -0,0 +1,32 @@
+// 检查当前运行环境
+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);
+});

package/example.js

@@ -0,0 +1,29 @@
+// 示例文件，展示如何使用 GitLabCodeReviewer 类
+const GitLabCodeReviewer = require('./index.js');
+
+// 从环境变量获取配置
+const gitlabUrl = process.env.CI_API_V4_URL;
+const gitlabToken = process.env.GITLAB_ACCESS_TOKEN;
+const projectId = process.env.CI_PROJECT_ID;
+const mergeRequestIid = process.env.CI_MERGE_REQUEST_IID;
+
+if (!gitlabToken || !projectId || !mergeRequestIid) {
+    console.error('缺少必要的环境变量配置');
+    console.error('请设置: CI_API_V4_URL, GITLAB_ACCESS_TOKEN, CI_PROJECT_ID, CI_MERGE_REQUEST_IID');
+    process.exit(1);
+}
+
+const reviewer = new GitLabCodeReviewer(gitlabToken, gitlabUrl);
+
+async function runExample() {
+    try {
+        console.log('开始审查合并请求...');
+        const results = await reviewer.reviewMergeRequest(projectId, mergeRequestIid);
+        console.log('审查完成，结果已发布到 GitLab MR');
+    } catch (error) {
+        console.error('审查过程出错:', error);
+        process.exit(1);
+    }
+}
+
+runExample();