gitlab-ai-review 3.2.0 → 3.2.1
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/lib/prompt-tools.js +52 -15
- package/package.json +1 -1
package/lib/prompt-tools.js
CHANGED
|
@@ -67,12 +67,24 @@ export function buildFileReviewPrompt(fileName, meaningfulChanges) {
|
|
|
67
67
|
prompt += `\`\`\`\n`;
|
|
68
68
|
});
|
|
69
69
|
|
|
70
|
-
prompt += `\n##
|
|
70
|
+
prompt += `\n## ⚠️ 重要说明：如何区分删除和新增
|
|
71
|
+
|
|
72
|
+
**请务必注意以下标记：**
|
|
73
|
+
- 标记为 **"类型: 删除"** 且代码以 **"-"** 开头的，是**被删除的代码**（不再存在）
|
|
74
|
+
- 标记为 **"类型: 新增"** 且代码以 **"+"** 开头的，是**新增的代码**（新添加的）
|
|
75
|
+
|
|
76
|
+
**特别注意：**
|
|
77
|
+
- 如果看到 **"类型: 删除"** 和 **"-"**，这意味着该代码已被移除，不存在了
|
|
78
|
+
- 不要把"删除"理解成"存在"或"新增"
|
|
79
|
+
- 删除的代码需要重点关注是否仍被其他地方使用
|
|
80
|
+
|
|
81
|
+
## 审查要求
|
|
71
82
|
|
|
72
83
|
1. 仔细审查每一行变更，判断是否存在问题
|
|
73
|
-
2.
|
|
74
|
-
3.
|
|
75
|
-
4.
|
|
84
|
+
2. **优先检查"类型: 删除"的代码**，确认是否有其他地方仍在使用
|
|
85
|
+
3. **只对有问题的行提出审查意见**，没有问题的行不需要评论
|
|
86
|
+
4. **必须使用中文**返回审查意见
|
|
87
|
+
5. 返回 JSON 格式的结果，格式如下：
|
|
76
88
|
|
|
77
89
|
\`\`\`json
|
|
78
90
|
{
|
|
@@ -90,9 +102,14 @@ export function buildFileReviewPrompt(fileName, meaningfulChanges) {
|
|
|
90
102
|
}
|
|
91
103
|
\`\`\`
|
|
92
104
|
|
|
93
|
-
|
|
94
|
-
|
|
95
|
-
|
|
105
|
+
6. 如果所有代码都没有问题，返回空的 reviews 数组：\`{"reviews": []}\`
|
|
106
|
+
7. 只返回 JSON，不要包含其他文字说明
|
|
107
|
+
8. comment 字段必须使用中文，要简洁明了
|
|
108
|
+
|
|
109
|
+
**再次强调：**
|
|
110
|
+
- 看到 "类型: 删除" 和 "-" → 代码被移除了
|
|
111
|
+
- 看到 "类型: 新增" 和 "+" → 代码是新加的
|
|
112
|
+
- 不要混淆删除和新增！`;
|
|
96
113
|
|
|
97
114
|
return prompt;
|
|
98
115
|
}
|
|
@@ -255,18 +272,33 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
|
|
|
255
272
|
prompt += `\`\`\`\n`;
|
|
256
273
|
});
|
|
257
274
|
|
|
258
|
-
prompt += `\n##
|
|
275
|
+
prompt += `\n## ⚠️ 重要说明：如何区分删除和新增
|
|
259
276
|
|
|
260
|
-
|
|
261
|
-
|
|
277
|
+
**请务必正确理解以下标记：**
|
|
278
|
+
1. **"类型: 删除"** + 代码以 **"-"** 开头 = **代码已被移除**（不存在了）
|
|
279
|
+
2. **"类型: 新增"** + 代码以 **"+"** 开头 = **代码是新加的**（新存在的）
|
|
280
|
+
|
|
281
|
+
**错误示例（请避免）：**
|
|
282
|
+
❌ 看到 "类型: 删除" 却说"新增了XXX"
|
|
283
|
+
❌ 看到 "-" 号却理解为"添加了"
|
|
284
|
+
❌ 把删除当成存在
|
|
285
|
+
|
|
286
|
+
**正确理解：**
|
|
287
|
+
✅ "类型: 删除" → 这段代码被删掉了 → 需要检查是否还有地方在用
|
|
288
|
+
✅ "类型: 新增" → 这段代码是新加的 → 检查新代码的质量
|
|
289
|
+
|
|
290
|
+
## 审查要求
|
|
291
|
+
|
|
292
|
+
1. **首先确认变更类型**：看清楚是"删除"还是"新增"
|
|
293
|
+
2. **🚨 最高优先级**：检查"类型: 删除"的符号是否仍被使用（文件内部或其他文件）
|
|
262
294
|
3. **特别关注**：
|
|
263
|
-
-
|
|
295
|
+
- **删除的函数/类/组件**是否在其他地方被调用（这会导致错误）
|
|
264
296
|
- 修改的函数签名是否会导致现有调用失效
|
|
265
|
-
-
|
|
297
|
+
- 新增的代码质量是否符合规范
|
|
266
298
|
4. 检查函数签名是否改变，所有调用方式是否需要更新
|
|
267
299
|
5. 检查是否有潜在的 API 不兼容问题
|
|
268
|
-
6.
|
|
269
|
-
7.
|
|
300
|
+
6. 如果发现文件内部使用了**被删除**的符号，这是**严重错误**
|
|
301
|
+
7. 如果发现其他文件使用了**被删除/修改**的符号，需要提醒可能的影响
|
|
270
302
|
8. **只对有问题的行提出审查意见**，没有问题的行不需要评论
|
|
271
303
|
9. **必须使用中文**返回审查意见
|
|
272
304
|
10. 返回 JSON 格式的结果，格式如下：
|
|
@@ -306,7 +338,12 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
|
|
|
306
338
|
|
|
307
339
|
11. 如果所有代码都没有问题，返回空的 reviews 数组：\`{"reviews": []}\`
|
|
308
340
|
12. 只返回 JSON，不要包含其他文字说明
|
|
309
|
-
13. comment
|
|
341
|
+
13. comment 字段必须使用中文，要简洁明了，重点指出影响范围
|
|
342
|
+
|
|
343
|
+
**最后再次确认：**
|
|
344
|
+
- 看到 **"类型: 删除"** 和 **"-"** → 代码**被移除了**，需要检查影响
|
|
345
|
+
- 看到 **"类型: 新增"** 和 **"+"** → 代码**是新的**，检查代码质量
|
|
346
|
+
- **千万不要把删除理解成新增！**`;
|
|
310
347
|
|
|
311
348
|
return prompt;
|
|
312
349
|
}
|