reviewing-changes

Review Bitwarden iOS code changes for architecture, style, security, and test compliance.

660|146|Updated Jul 14, 2023
One-click install
npx skills add https://github.com/bitwarden/ios --skill reviewing-changes
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: reviewing-changes
Source: https://github.com/bitwarden/ios/tree/main/.claude/skills/reviewing-changes
Command: npx skills add https://github.com/bitwarden/ios --skill reviewing-changes

SYSTEM DOCUMENTATION & REQUIREMENTS

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.

Frequently Asked Questions about reviewing-changes

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I automate code reviews for iOS projects?

Automate code reviews by running comprehensive checks that verify architecture compliance, style adherence, test coverage, and security requirements. This Skill analyzes pull requests and commits against Bitwarden iOS patterns—unidirectional data flow, dependency injection, and secure data handling—flagging violations with precise file:line references so reviewers focus on substantive feedback.

Can I use automated code review with SwiftUI and iOS CI pipelines?

Yes. This Skill integrates into iOS CI pipelines and manual workflows, enforcing SwiftUI architecture patterns, Coordinator + Processor design, ServiceContainer injection, and repository patterns specific to Bitwarden. It validates compilation safety and security requirements during pull request checks before human review.

What security issues does iOS code review catch?

Code review checks for secure handling of sensitive data, proper encryption implementation, and secure storage patterns required by Bitwarden iOS standards. It flags architectural deviations that could introduce security gaps, distinguishing critical blockers from suggestions.

How do I ensure test coverage and architecture compliance in iOS code?

Automated review verifies test coverage levels and enforces Bitwarden iOS architecture compliance—unidirectional data flow with Coordinators, dependency injection via ServiceContainer, and repository patterns. Violations receive actionable feedback tied to specific code locations.

What happens when code doesn't follow iOS architecture guidelines?

The review flags architecture violations—missing Coordinator patterns, incorrect dependency injection, or unidirectional data flow breaks—with precise locations and recommendations. Critical compliance issues are prioritized separately from style suggestions, helping developers prioritize fixes.