reviewing-changes

What problem does it solve?

Manual code reviews are time-consuming, inconsistent, and prone to human error, especially in complex, security-critical applications like Bitwarden iOS. This Skill automates the rigorous verification process, ensuring every code change adheres to strict architectural, style, security, and testing standards, thereby saving developer time and enhancing code quality and reliability.

Core Features & Use Cases

• Comprehensive Compliance Checks: Automatically verifies adherence to Bitwarden iOS architecture patterns (e.g., unidirectional data flow, dependency injection), code style guidelines, compilation safety, appropriate test coverage, and critical security requirements (e.g., sensitive data handling, secure storage, encryption).
• Actionable Feedback & Prioritization: Provides precise feedback with file:line_number references for violations, actionable recommendations for improvements, and clearly distinguishes between critical blockers and suggestions, streamlining the review process.
• Use Case: When a developer submits a pull request for a Bitwarden iOS project, this Skill can be automatically invoked by a CI pipeline or manually for interactive reviews. It will perform a thorough, objective analysis, flagging issues related to security, architecture, and testing before a human reviewer even begins, significantly accelerating the review cycle and ensuring high-quality, secure code.

Quick Start

Review the attached pull request for the Bitwarden iOS project, focusing on architecture compliance, security implementations, and test coverage.