reviewing-changes

变更审查 - Android 补充内容

此技能提供 Android 专属的工作流补充内容，用于完善基础的 bitwarden-code-reviewer Agent 标准。

说明

重要：请在提供反馈前，系统地完成每个步骤。每个检查清单文件都包含针对相应审查阶段的结构化思考指导。

步骤 1：获取更多详情

使用可用工具（JIRA MCP、GitHub API）获取与拉取请求相关的其他信息。

如果拉取请求的标题和说明未提供足够的上下文，请向审查者索取更多详情：

• 添加 JIRA 工单链接
• 关联 GitHub Issue
• 添加另一个拉取请求的链接
• 在 PR 标题或正文中补充更多详情

Android 元数据检查 — 如果缺少以下任一项，请标记为 ❓：

• PR 包含 *Screen.kt 或 Composable 变更，但没有截图
• PR 新增了 ViewModel 或 Repository，但没有测试计划或测试文件变更

步骤 2：结合 Android 特定规则检测变更类型

使用 Agent 的基础变更类型检测，并结合 Android 特定规则进行细化：

Android 特定模式：

• 功能新增：新增 ViewModel、Repository、@Composable 函数或 *Screen.kt 文件
• UI 优化：仅修改 *Screen.kt、*Composable.kt、ui/ 包中的文件
• 基础设施：修改 .github/workflows/、gradle/、build.gradle.kts、libs.versions.toml
• 依赖更新：仅修改 libs.versions.toml 或 build.gradle.kts，并更新版本

步骤 3：加载相应检查清单

根据检测到的类型，读取相关检查清单文件：

• 依赖更新 → checklists/dependency-update.md（加快审查）
• 缺陷修复 → checklists/bug-fix.md（重点审查）
• 功能新增 → checklists/feature-addition.md（全面审查）
• UI 优化 → checklists/ui-refinement.md（侧重设计的审查）
• 重构 → checklists/refactoring.md（侧重模式的审查）
• 基础设施 → checklists/infrastructure.md（侧重工具的审查）

检查清单提供：

• 多轮审查策略
• 针对不同类型的重点领域
• 需要检查和跳过的内容
• 结构化思考指导

步骤 4：按照检查清单执行审查

遵循检查清单中的多轮审查策略，系统地思考每一轮审查。

步骤 5：按需查阅 Android 参考资料

仅在需要解答具体问题时加载参考文件：

• 再次审查 → 调用 reviewing-incremental-changes Agent 技能；仅关注变更行，不要指出未变更代码中的新问题
• 问题优先级 → reference/priority-framework.md（严重 vs 建议 vs 可选）
• 反馈措辞 → reference/review-psychology.md（提问 vs 命令、第一人称陈述）
• 架构问题 → reference/architectural-patterns.md（MVVM、Hilt DI、模块组织、错误处理）
• 安全问题（快速参考） → reference/security-patterns.md（常见模式和反模式）
• 安全问题（全面参考） → docs/ARCHITECTURE.md#security（完整的零知识架构）
• 测试问题 → reference/testing-patterns.md（单元测试、模拟、空值安全）
• UI 问题 → reference/ui-patterns.md（Compose 模式、主题）
• 风格问题（项目特定） → reference/style-patterns.md（审查中执行的 Kotlin 规则）
• 风格问题（通用） → docs/STYLE_AND_BEST_PRACTICES.md

核心原则

• 优先级顺序：安全性 → 正确性 → 破坏性变更 → 性能 → 可维护性
• 恰当的深度：根据变更的复杂程度和风险调整评审的严格程度
• 具体引用：始终使用 file:line_number 格式标明精确位置
• 可执行的反馈：说明要做什么以及原因，而不只是指出问题
• 高效评审：采用多轮策略，跳过不相关的内容
• Android 模式：验证 MVVM、Hilt DI、Compose 约定和 Kotlin 惯用写法