@unifan/pi-review-zh 1.0.18 → 1.0.20
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/index.ts +25 -40
- package/package.json +1 -1
package/index.ts
CHANGED
|
@@ -1355,62 +1355,47 @@ export default function reviewExtension(pi: ExtensionAPI) {
|
|
|
1355
1355
|
});
|
|
1356
1356
|
|
|
1357
1357
|
// 针对【代码开发会话·修复方】的提示词模板
|
|
1358
|
-
const DEV_FIX_PROMPT_TEMPLATE = `>
|
|
1358
|
+
const DEV_FIX_PROMPT_TEMPLATE = `> 语言要求:全流程使用纯正中文。
|
|
1359
|
+
> 提示:无需执行 git commit 或 push,只需修改代码并说明,由用户确认后自行提交。
|
|
1359
1360
|
|
|
1360
|
-
#
|
|
1361
|
+
# 代码审查意见整改与回复
|
|
1361
1362
|
|
|
1362
|
-
|
|
1363
|
+
外部审查专家提出了以下代码审查意见:
|
|
1363
1364
|
---
|
|
1364
1365
|
{content}
|
|
1365
1366
|
---
|
|
1366
1367
|
|
|
1367
|
-
##
|
|
1368
|
-
1.
|
|
1369
|
-
2.
|
|
1370
|
-
|
|
1371
|
-
|
|
1372
|
-
|
|
1373
|
-
|
|
1374
|
-
|
|
1375
|
-
|
|
1376
|
-
|
|
1377
|
-
###
|
|
1378
|
-
|
|
1379
|
-
- **修改说明**:说明修改了哪些文件与函数,采取了什么具体修复方案。
|
|
1380
|
-
- **[无需修改/已说明] [问题标题]**
|
|
1381
|
-
- **原因解释**:说明为什么无需修改或为什么既有实现是合理的。
|
|
1382
|
-
|
|
1383
|
-
### 📝 本次整改提交摘要
|
|
1384
|
-
一句话总结本次修复涉及的模块与核心变动。
|
|
1368
|
+
## 执行建议:
|
|
1369
|
+
1. 结合代码上下文逐条核验上述审查意见。
|
|
1370
|
+
2. 合理且存在隐患的问题:修改对应代码并保证逻辑自洽。
|
|
1371
|
+
3. 有争议或误报的问题:简要说明理由与考量。
|
|
1372
|
+
4. 在最终回复末尾,整理一份简明报告供复核:
|
|
1373
|
+
|
|
1374
|
+
### 🛠️ 整改与回复报告
|
|
1375
|
+
- **[已修复] [问题标题]**:说明修改文件与修复方案
|
|
1376
|
+
- **[无需修改/已说明] [问题标题]**:说明理由
|
|
1377
|
+
|
|
1378
|
+
### 📝 整改总结
|
|
1379
|
+
一句话总结本次变动。
|
|
1385
1380
|
`;
|
|
1386
1381
|
|
|
1387
1382
|
// 针对【审查复核会话·审查方】的提示词模板
|
|
1388
|
-
const REVIEWER_RECHECK_PROMPT_TEMPLATE = `>
|
|
1389
|
-
>
|
|
1383
|
+
const REVIEWER_RECHECK_PROMPT_TEMPLATE = `> 语言要求:全流程使用纯正中文。
|
|
1384
|
+
> 提示:当前为只读复核模式,请勿修改代码或执行提交,仅对照 diff 进行复核。
|
|
1390
1385
|
|
|
1391
|
-
#
|
|
1386
|
+
# 复核开发者的代码修复与整改说明
|
|
1392
1387
|
|
|
1393
1388
|
开发者提交了针对此前代码审查的整改回复:
|
|
1394
1389
|
---
|
|
1395
1390
|
{content}
|
|
1396
1391
|
---
|
|
1397
1392
|
|
|
1398
|
-
##
|
|
1399
|
-
1.
|
|
1400
|
-
2.
|
|
1401
|
-
|
|
1402
|
-
-
|
|
1403
|
-
-
|
|
1404
|
-
3. **严禁修改任何源码文件**(只读复核)。
|
|
1405
|
-
4. **给出最终复核裁决**:
|
|
1406
|
-
- **若所有关键问题均已圆满解决,代码质量达标**:
|
|
1407
|
-
请在报告末尾明确输出绿色的终审裁决大标题:
|
|
1408
|
-
## ✅【审核通过,可以提交代码】
|
|
1409
|
-
并附上一句话总评。
|
|
1410
|
-
- **若仍有未解决或新引入的问题**:
|
|
1411
|
-
请在报告末尾明确输出:
|
|
1412
|
-
## ❌【仍有阻塞问题需继续修改】
|
|
1413
|
-
列出具体的剩余问题、所在文件行号与修改要求,以便用户再次复制给开发者继续修复。
|
|
1393
|
+
## 执行建议:
|
|
1394
|
+
1. 运行 \`git diff\` 查看最新改动,按需使用 \`read\` 查阅关键代码上下文。
|
|
1395
|
+
2. 对照上述整改说明,验证问题是否已被妥善解决,确认未引入新隐患。
|
|
1396
|
+
3. 给出复核结论:
|
|
1397
|
+
- 若全部解决且质量良好:输出 **## ✅【审核通过,可以提交代码】**
|
|
1398
|
+
- 若仍有问题:输出 **## ❌【仍有阻塞问题需继续修改】** 并简要说明剩余问题。
|
|
1414
1399
|
`;
|
|
1415
1400
|
|
|
1416
1401
|
function detectReviewSyncRole(
|