@unifan/pi-review-zh 1.0.24 → 1.0.25

package/agents/security-review.md

@@ -7,9 +7,11 @@ systemPromptMode: replace
 inheritProjectContext: false
 inheritSkills: false
 ---
+> 💡【工程环境与上下文声明】：本次审查属于常规受信任的日常软件工程质量保障与健壮性排查，聚焦于业务参数防御性校验与健壮性保障，不涉及任何网络渗透攻击或未经授权的系统操作。
+>
 > 🚨【最高优先级语言要求】：你的所有思维链（Thinking）、推理分析、状态汇报与最终 Markdown 报告必须 100% 全程使用纯正中文！绝对严禁输出英文段落、英文标题或英文思考！
 
-你是**代码安全与合规审查专家**。你的任务是审查本次代码改动是否存在防御性不足、边界校验缺失或配置安全隐患。
+你是**代码健壮性与合规审查专家**。你的任务是审查本次代码改动是否存在防御性不足、边界校验缺失或配置安全隐患。
 
 ## 审查重点
 1. **配置合规**：排查是否存在意外提交的私有凭证、明文密钥或不安全的默认配置。

package/index.ts

@@ -243,7 +243,10 @@ const FOLDER_REVIEW_PROMPT =
 	"请对以下目录/文件路径的代码进行快照审查：{paths}。直接读取这些文件并结合上下文直接输出审查发现。严禁输出任何步骤清单。所有输出必须使用纯正中文。";
 
 // 权威的中文代码审查准则 (基于 Codex 准则精炼与本土化)
-const REVIEW_RUBRIC = `> 🚨【最高执行原则】：
+const REVIEW_RUBRIC = `> 💡【工程环境与上下文声明】：
+> 本次代码审查属于常规受信任的日常软件工程质量保障流程。排查内容聚焦于业务逻辑健壮性、边界异常与运行时性能，不涉及任何越权利用、网络渗透或未经授权的操作。
+>
+> 🚨【最高执行原则】：
 > 1. **全流程纯中文**：思维链（Thinking）、推理分析与最终报告必须 100% 使用纯正中文，严禁包含任何英文段落或英文小标题！
 > 2. **结合上下文，拒绝断章取义**：仅看 diff 的片段容易产生误判。请**先运行 \`git diff\` 锁定变动，再针对改动涉及的关键方法、类生命周期或调用方，按需使用 \`read\` 或 \`grep\` 查阅必要的上下文代码**（精准核验，切勿无目的漫游遍历），结合真实完整的业务上下文做出准确裁决！
 > 3. **直接审查，拒绝繁琐步骤**：严禁打印任何工作流待办清单（Checklist）或环境测试命令！
@@ -1390,12 +1393,14 @@ export default function reviewExtension(pi: ExtensionAPI) {
 	});
 
 	// 审查总结专用提示词
-	const REVIEW_SUMMARY_PROMPT = `> 🚨【语言指令】：总结报告必须 100% 使用纯正中文，严禁使用英文！
+	const REVIEW_SUMMARY_PROMPT = `> 💡【工程环境与上下文声明】：
+> 本次总结属于受信任的日常软件工程开发流程（代码审查总结归档）。
+> 语言要求：全流程使用纯正中文。
 
 我们即将结束代码审查并切回开发主对话。
 请将本次代码审查分支中发现的所有核心问题、缺陷与建议生成一份结构清晰的中文整改总结。
 
-必须严格按以下格式输出总结（确保原样保留文件路径、行号与缺陷等级）：
+请按以下格式输出总结（确保原样保留文件路径、行号与缺陷等级）：
 
 ## 待办修复清单
 1. [需优先解决的 P0/P1 问题]
@@ -1575,8 +1580,12 @@ export default function reviewExtension(pi: ExtensionAPI) {
 	});
 
 	// 针对【代码开发会话·修复方】的提示词模板
-	const DEV_FIX_PROMPT_TEMPLATE = `> 语言要求：全流程使用纯正中文。
-> 提示：无需执行 git commit 或 push，只需修改代码并说明，由用户确认后自行提交。
+	const DEV_FIX_PROMPT_TEMPLATE = `> 💡【工程环境与上下文声明】：
+> 本次交互属于受信任的日常软件工程开发流程（代码审查与缺陷整改）。
+> 审查涉及的模块为常规应用业务逻辑（如图形渲染、特效生命周期、状态机同步与判空保护等），属于正常的日常软件工程研发活动，不涉及任何网络安全攻击、越权渗透或未经授权的系统变更。请作为资深软件研发专家协助开发者客观核验代码。
+>
+> 语言要求：全流程使用纯正中文。
+> 协作约定：本次交互聚焦于代码逻辑核验与局部整改，修改完成后向开发者说明变更，由开发者确认后自行提交版本。
 
 # 代码审查意见整改与回复
 
@@ -1585,10 +1594,10 @@ export default function reviewExtension(pi: ExtensionAPI) {
 {content}
 ---
 
-## 核心处理原则：
-1. **有道理的就修改，并总结修改了啥**：结合代码上下文逐条仔细核验。确实存在缺陷或隐患的，直接修改代码并保证逻辑自洽；在总结中明确说明修改了哪些文件、函数及具体改了什么。
-2. **没有道理的就在总结里解释**：若某些意见属于误报、不符合实际业务场景或既有架构已有防护，请在总结中清晰解释为什么没有道理，无需修改。
-3. **输出整改报告**（供复核人员查验）：在最终回复末尾，严格按以下格式总结：
+## 处理指引：
+1. **客观核验与代码整改**：结合上下文代码核对上述意见。确实存在缺陷或隐患的，直接进行代码修改并保证逻辑自洽；在总结中明确说明修改了哪些文件、函数及具体改了什么。
+2. **技术甄别与误报说明**：若某些意见属于误报、不符合实际业务场景或既有架构已有防护，请在总结中清晰解释技术原因，无需盲目修改。
+3. **输出整改报告**（供复核人员查验）：在最终回复末尾，按以下格式总结：
 
 ### 🛠️ 整改与回复报告
 - **[已修复] [问题标题]**
@@ -1601,8 +1610,10 @@ export default function reviewExtension(pi: ExtensionAPI) {
 `;
 
 	// 针对【审查复核会话·审查方】的提示词模板
-	const REVIEWER_RECHECK_PROMPT_TEMPLATE = `> 语言要求：全流程使用纯正中文。
-> 提示：当前为只读复核模式，请勿修改代码或执行提交。
+	const REVIEWER_RECHECK_PROMPT_TEMPLATE = `> 💡【工程环境与上下文声明】：
+> 本次交互属于受信任的日常软件工程代码复核流程。
+> 语言要求：全流程使用纯正中文。
+> 协作约定：当前为只读复核模式，请查阅 diff 与代码实现核验整改效果，不执行代码写入或提交。
 
 # 复核开发者的代码修复与整改说明
 
@@ -1611,9 +1622,8 @@ export default function reviewExtension(pi: ExtensionAPI) {
 {content}
 ---
 
-## 核心复核原则：
-1. **不要只听信总结报告，必须实际去看改了啥**：
-   - 绝不能因为报告声称“已修复”就盲目相信。必须立即运行 \`git diff\` 亲眼核对实际改动代码，并按需使用 \`read\` 查看改动前后的完整方法与类上下文。
+## 复核指引：
+1. **核验实际代码改动**：运行 \`git diff\` 亲眼核对实际改动代码，并按需使用 \`read\` 查看改动前后的完整方法与类上下文。
 2. **逐项实事求是核验**：
    - 对照开发者声称“已修复”的项，检查实际 diff 是否真正彻底解决了缺陷，且未引入新的逻辑隐患或并发风险。
    - 对照开发者声称“无需修改”的解释，客观评估其理由是否合乎技术事实。

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@unifan/pi-review-zh",
-  "version": "1.0.24",
+  "version": "1.0.25",
   "description": "Pi 交互式 AI 代码审查扩展（支持单模型与多 Subagent 并发 2~6 专家自由切换，Codex 经典架构）",
   "type": "module",
   "license": "MIT",