Update CodeBuddy commands and rules.

Author: domchen

.codebuddy/commands/cr.md

@@ -137,9 +137,11 @@ gh pr view {pr_number} --comments
 - 问题列表：按序号列出真正存在的问题，每个问题包含位置、描述、修复建议
 - 若无问题则输出"无问题"
 
-**!! IMPORTANT - 输出禁令**：
-- 禁止输出任何已排除的问题，包括禁止以"排除"、"不是问题"、"二次验证后确认正确"等形式提及
-- 禁止输出分析推理过程，只输出最终结论
+**!! CRITICAL - 输出禁令**：
+- 只输出最终确认存在的问题，其他一切不输出
+- 不输出分析过程、排除理由、"经确认"、"不是问题"等任何解释性文字
+- 不输出曾经考虑过但被排除的问题
+- 违反此规则视为审查失败
 
 ---
 
@@ -229,4 +231,5 @@ echo "已清理临时审查环境"
 7. **接口变更**：公开接口变更需关注必要性和兼容性
 8. **测试覆盖**：变更是否有对应测试，边界情况是否覆盖
 9. **潜在风险**：是否引入回归风险或影响其他模块
-10. **整体设计**：结合关联代码评估修改后的整体合理性，必要时建议扩大修改范围
+10. **模块架构**：模块职责是否清晰，依赖方向是否合理（如核心模块不应反向依赖平台特定实现）
+11. **整体设计**：结合关联代码评估修改后的整体合理性，必要时建议扩大修改范围

.codebuddy/rules/Code.md

@@ -12,10 +12,12 @@ alwaysApply: true
 - 重构时审查关联代码合理性，顺带清理冗余，不考虑向后兼容
 - 版权声明里的年份对新增的文件要使用当前年份（如 `Copyright (C) 2026 Tencent`），已有文件保持原年份不变
 - 驼峰命名法：大写开头（类静态方法、全局函数/变量）、小写开头（成员方法/变量、局部变量）、全大写下划线（静态常量）
+- 枚举值和常量不加 `k` 前缀，直接使用大写开头的驼峰命名（如 `GlyphPositioning::Default`）
 - 变量命名避免缩写，简短且语义明确
 - 变量声明时一律赋初始值（即使是 `={}`），智能指针初始值使用 nullptr
 - 避免 lambda 表达式，改用显式方法或函数
 - 禁止使用 `dynamic_cast` 和 C++ 异常（`throw`/`try`/`catch`）
+- 尽量避免声明 mutable 声明变量，有线程安全问题，应该把调用的方法去掉 const
 - CPP 文件里的函数实现顺序与头文件中定义的顺序尽可能一致
 - `include/` 目录 API 需详细注释含参数描述，其他公开方法一段话描述主要功能，私有方法不加注释
 - 函数内代码不加行注释，除非只看代码无法理解设计意图

.codebuddy/rules/Test.md

@@ -24,7 +24,9 @@ cmake --build cmake-build-debug --target PAGFullTest
 
 - 使用 `Baseline::Compare(pixels, key)` 比较截图，key 格式为 `{folder}/{name}`，例如 `PAGSurfaceTest/Mask`
 - 截图输出到 `test/out/{folder}/{name}.webp`，基准图为同目录下 `{name}_base.webp`
-- 比较机制：对比 `test/baseline/version.json`（仓库）与 `test/baseline/.cache/version.json`（本地）中同一 key 的版本号，一致时才进行基准图对比，不一致则跳过返回成功，以此接受截图变更
+- 比较机制：对比 `test/baseline/version.json`（仓库）与 `test/baseline/.cache/version.json`（本地）中同一 key 的版本号
+    - 两边 key 都存在且版本号不同：跳过比较并返回成功（用于接受截图变更）
+    - 其他情况：正常比较基准图，基准图不存在或不匹配则测试失败
 
 **!! IMPORTANT - 截图基准变更限制**：
 - **NEVER** 自动接受截图基准变更，包括禁止自动运行 `UpdateBaseline` target、禁止修改或覆盖 `version.json` 文件