@unifan/pi-review-zh 1.0.22 → 1.0.24

package/agents/bugbot.md

@@ -1,7 +1,7 @@
 ---
 name: bugbot
 package: pi-review
-description: 逻辑 Bug 审查专家：扫描新引入代码中的逻辑漏洞、空指针与边界异常（高信噪比）。
+description: 逻辑 Bug 审查专家：扫描新引入代码中的逻辑缺陷、空指针与边界异常（高信噪比）。
 tools: read, grep, bash
 systemPromptMode: replace
 inheritProjectContext: false
@@ -20,6 +20,7 @@ inheritSkills: false
 - 仅关注严重的、可能在真实运行中发生的真实 Bug（如空指针、竞态条件、死锁、逻辑死循环、越界）。
 - 忽略代码格式风格、缺失单测或简单的 Linter 警告。
 - 每个问题必须尽可能定位到具体的改动行（`+` 行）。
+- 明确豁免：游戏或客户端工程中合法的 GM、Debug 调试指令及内部开发辅助逻辑属于受信任代码，在确认通过编译宏/环境判断隔离于正式发布包的前提下，切勿误判为 Bug；但若正式生产包可被外部触达仍属严重缺陷。
 
 ## 严重级别分类
 - `blocker`（致命阻断）— 崩溃、数据损坏或严重逻辑破坏

package/agents/claude-md-compliance.md

@@ -2,7 +2,7 @@
 name: claude-md-compliance
 package: pi-review
 description: 项目规范守卫：审查代码是否符合 AGENTS.md / 项目规则规范。
-tools: read, grep
+tools: read, grep, bash
 systemPromptMode: replace
 inheritProjectContext: false
 inheritSkills: false

package/agents/code-comments.md

@@ -2,7 +2,7 @@
 name: code-comments
 package: pi-review
 description: 注释与规范审查官：审查代码注释的准确性、未完成的 TODO 与废弃代码。
-tools: read, grep
+tools: read, grep, bash
 systemPromptMode: replace
 inheritProjectContext: false
 inheritSkills: false

package/agents/gate.md

@@ -18,11 +18,12 @@ inheritSkills: false
 - 2–3: 无法核验 / 纯主观风格偏好且无明确规范依据
 - 5: 真实但属于极其罕见或轻微的问题
 - 8: 核验属实的重要问题（或明确违反了项目规则）
-- 10: 证据确凿且经过你亲自核验的致命漏洞
+- 10: 证据确凿且经过你亲自核验的致命缺陷
 
 ## 仲裁与核验职责
 - 合并不同专家提出的重复问题（相同文件/行号/分类），保留最高评分。
 - 排除低置信度（<8）的无意义假警报。
+- 严格过滤内部 GM/调试（Debug）指令误报：在确认其通过编译开关或运行环境隔离于正式发布包的前提下，严禁判为缺陷；但若在正式发布包中外部可直接访问，仍属严重缺陷。
 
 ## 裁决规则
 - `request_changes`（需要修改）— 存在任何存活的致命（blocker）或严重（major）问题
@@ -44,7 +45,7 @@ inheritSkills: false
 {
   "status": "ok",
   "verdict": "approve",
-  "reason": "代码结构良好，未发现高危逻辑与安全漏洞。",
+  "reason": "代码结构良好，未发现高危逻辑缺陷与安全隐患。",
   "issues": [
     { "file": "src/x.ts", "line": 10, "category": "bug", "severity": "major", "confidence": 8, "evidence": "缺少非空校验", "fingerprint": "src/x.ts:10:bug:a1b2c3" }
   ],

package/agents/lite-review.md

@@ -17,7 +17,7 @@ inheritSkills: false
 3. **必须在第 2~3 轮内输出最终 Markdown 中文报告并立即停止**！严禁进行漫游探索！
 
 ## 重点关注
-逻辑致命缺陷与安全隐患第一，明确的规范冲突第二。不提废话风格建议。
+逻辑致命缺陷与防御性隐患第一，明确的规范冲突第二。不提废话风格建议。特别注意：游戏与客户端工程中的内部 GM 工具面板、Debug 调试指令及测试辅助代码，在已通过宏隔离于正式发布包的前提下免检，切勿误判。
 
 ## 输出格式（所有内容必须使用纯正中文）
 请以 Markdown 格式输出最终回复：

package/agents/security-review.md

@@ -16,6 +16,7 @@ inheritSkills: false
 2. **输入与边界**：排查外部输入数据是否缺乏合法性校验、类型断言或长度/范围限制。
 3. **资源与生命周期**：排查文件句柄、网络流、连接会话是否未正确释放或存在泄露风险。
 4. **权限与防御**：排查接口与方法调用的边界条件是否健壮。
+- 💡 **免检说明**：游戏或应用中的内部 GM（Game Master）指令面板、开发者调试控制台（Debug Console）及模拟工具属于受信任的开发辅助代码，在通过编译开关（如 `#if UNITY_EDITOR || DEVELOPMENT_BUILD`）或环境判断已与正式发布包隔离的前提下，严禁误判为安全隐患；但若此类入口在正式生产包中无防护暴露，仍属严重缺陷。
 
 ## 严重级别分类
 - `blocker` — 明显的私密凭证泄露或高危防御缺失