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

package/test_path_recognition.js

@@ -0,0 +1,119 @@
+// 详细测试Claude命令能否识别传入的文件路径
+const fs = require('fs');
+const path = require('path');
+const { spawn } = require('child_process');
+
+console.log('=== 测试Claude命令的文件路径识别能力 ===\n');
+
+async function testPathRecognition() {
+  // 1. 创建一个临时文件
+  const tempFileName = `./test-path-recognition-${Date.now()}.txt`;
+  const sampleContent = 'This is a test file to verify path recognition.';
+
+  console.log(`1. 创建测试文件: ${tempFileName}`);
+  fs.writeFileSync(tempFileName, sampleContent);
+
+  // 2. 验证文件创建成功
+  const exists = fs.existsSync(tempFileName);
+  console.log(`2. 文件创建成功: ${exists}`);
+
+  // 3. 获取各种路径形式
+  const relativePath = tempFileName;                           // ./test-file.txt
+  const absolutePath = path.resolve(tempFileName);             // /full/path/to/test-file.txt
+  const simpleName = path.basename(tempFileName);              // test-file.txt
+
+  console.log(`3. 相对路径: ${relativePath}`);
+  console.log(`   绝对路径: ${absolutePath}`);
+  console.log(`   简单文件名: ${simpleName}`);
+
+  // 4. 检查各种路径形式在当前目录下是否可访问
+  console.log(`5. 当前工作目录: ${process.cwd()}`);
+  console.log(`   相对路径可访问: ${fs.existsSync(relativePath)}`);
+  console.log(`   绝对路径可访问: ${fs.existsSync(absolutePath)}`);
+  console.log(`   简单文件名可访问: ${fs.existsSync(simpleName)}`);
+
+  // 5. 测试runClaudeCommand函数的实际行为
+  console.log('\n6. 模拟runClaudeCommand函数的行为...');
+
+  // 在index.js中实际使用的代码流程
+  const prompt = '/simple-code-review' + ' ' + relativePath;
+  console.log(`   构造的prompt: ${prompt}`);
+
+  // 这对应index.js第523行开始的runClaudeCommand函数
+  console.log(`   实际执行: claude -p "${prompt}"`);
+  console.log(`   工作目录设置为: ${__dirname} (index.js所在目录)`);
+
+  // 检查在__dirname下该文件是否存在
+  const originalCwd = process.cwd();
+  try {
+    process.chdir(__dirname);
+    console.log(`   切换到__dirname: ${__dirname}`);
+    console.log(`   在__dirname下相对路径可访问: ${fs.existsSync(relativePath)}`);
+    console.log(`   在__dirname下绝对路径可访问: ${fs.existsSync(absolutePath)}`);
+  } finally {
+    process.chdir(originalCwd);
+  }
+
+  // 6. 分析index.js中的runClaudeCommand实现
+  console.log('\n7. index.js中的runClaudeCommand函数分析:');
+  console.log('   spawn(\'claude\', [\'-p\', promptContent], {');
+  console.log('     shell: true,');
+  console.log('     cwd: __dirname,    <- 这很重要！');
+  console.log('     env: process.env,');
+  console.log('     stdio: [\'ignore\', \'pipe\', \'pipe\']');
+  console.log('   });');
+  console.log('');
+  console.log('   关键点: cwd被设置为__dirname，这意味着Claude命令将在');
+  console.log('   index.js文件所在的目录中执行，而不是当前工作目录。');
+
+  // 7. 验证__dirname和process.cwd()的关系
+  console.log(`\n8. 目录关系验证:`);
+  console.log(`   __dirname (脚本目录): ${__dirname}`);
+  console.log(`   process.cwd() (当前工作目录): ${process.cwd()}`);
+  console.log(`   两者是否相同: ${__dirname === process.cwd()}`);
+
+  // 8. 针对index.js的精确场景进行测试
+  console.log('\n9. 模拟index.js中的精确场景:');
+
+  // 模拟第118-120行的代码
+  const blockIndex = 0;
+  const tmpFileNameFromIndex = `./temp-diff-block-${Date.now()}-${blockIndex}.diff`;
+  console.log(`   模拟创建临时文件: ${tmpFileNameFromIndex}`);
+
+  // 模拟第124行
+  fs.writeFileSync(tmpFileNameFromIndex, 'Sample diff content for testing');
+  console.log(`   文件写入成功`);
+
+  // 模拟第198行
+  const promptFromIndex = '/simple-code-review' + ' ' + tmpFileNameFromIndex;
+  console.log(`   模拟prompt: ${promptFromIndex}`);
+
+  // 检查在index.js所在目录中是否可以访问该文件
+  const fileAccessibleInScriptDir = fs.existsSync(path.join(__dirname, tmpFileNameFromIndex));
+  console.log(`   在脚本目录中可访问 (使用join): ${fileAccessibleInScriptDir}`);
+
+  // 清理测试文件
+  try {
+    if (fs.existsSync(tempFileName)) {
+      fs.unlinkSync(tempFileName);
+      console.log(`\n10. 清理测试文件: ${tempFileName}`);
+    }
+
+    if (fs.existsSync(tmpFileNameFromIndex)) {
+      fs.unlinkSync(tmpFileNameFromIndex);
+      console.log(`    清理临时文件: ${tmpFileNameFromIndex}`);
+    }
+  } catch (error) {
+    console.error(`清理文件时出错: ${error.message}`);
+  }
+
+  console.log('\n=== 结论 ===');
+  console.log('1. 在index.js的runClaudeCommand中，cwd设置为__dirname');
+  console.log('2. 临时文件使用相对路径创建，也位于__dirname目录中');
+  console.log('3. 因此Claude命令应该能够在设置的cwd中找到该文件');
+  console.log('4. 如果在GitLab CI中失败，可能是由于Claude命令本身不存在或');
+  console.log('   权限/环境问题，而不是文件路径问题');
+  console.log('5. 但路径问题仍可能存在，因为如果工作流程改变，可能会产生差异');
+}
+
+testPathRecognition().catch(console.error);

package/utils.js

@@ -0,0 +1,67 @@
+/**
+ * 工具函数集合
+ */
+
+const axios = require('axios');
+
+// 调试日志函数
+function debugLog(message) {
+  console.log('[DEBUG]', new Date().toISOString(), message);
+}
+
+/**
+ * 提取REPORT标签之间的内容
+ * @param {string} text 包含REPORT标签的文本
+ * @returns {string} 提取后的内容
+ */
+function extractReportContent(text) {
+  const reportRegex = /<REPORT>([\s\S]*?)<\/REPORT>/;
+  const match = text.match(reportRegex);
+  return match ? match[1].trim() : text;
+}
+
+// 创建一个可配置的GitLab API客户端
+class GitLabAPIClient {
+  constructor(gitlabToken, gitlabUrl = null) {
+    this.gitlabToken = gitlabToken;
+    this.gitlabUrl = gitlabUrl || 'https://gitdev.51job.com';
+    this.axiosInstance = axios.create({
+      headers: {
+        'PRIVATE-TOKEN': this.gitlabToken,
+      },
+    });
+  }
+
+  /**
+   * 通用的GitLab API调用函数
+   * @param {string} endpoint API端点
+   * @param {Object} options 额外选项（method, data等）
+   * @returns {Promise} API调用结果
+   */
+  async callGitLabAPI(endpoint, options = {}) {
+    const { method = 'GET', data = null, headers = {} } = options;
+
+    try {
+      const response = await this.axiosInstance({
+        method,
+        url: `${this.gitlabUrl}${endpoint}`,
+        headers: {
+          'Content-Type': 'application/json',
+          ...headers
+        },
+        data
+      });
+
+      return response.data;
+    } catch (error) {
+      console.error(`GitLab API调用失败: ${method} ${endpoint}`, error.message);
+      throw error;
+    }
+  }
+}
+
+module.exports = {
+  GitLabAPIClient,
+  debugLog,
+  extractReportContent
+};