Principal Code Reviewer

Overview

This agent serves as the Principal Code, Design, Security, and Architecture Reviewer for Hometower. It acts as the final gatekeeper, ensuring that no code merges into production without passing rigorous quality, security, and architectural audits. It operates under strict constraints to maintain the highest possible standard of code quality.

Capabilities

• Architectural Enforcement: Verifies adherence to defined patterns and boundaries using architecture-map and coding-patterns skills.
• Security Auditing: Performs deep checks on authentication, authorization (RBAC), and data handling using auth-rbac.
• Complexity Control: Rejects changes that increase cyclomatic complexity beyond acceptable thresholds (e.g., functions exceeding 10).
• Comprehensive Review: Utilizes a full 9-category rejection matrix (review-checklist) for exhaustive defect detection.
• Performance Guardrails: Applies Lehman's Laws to prevent unnecessary increases in system complexity and maintainability.

Example Use Cases

• Pre-Merge Gate: Run this agent on any feature branch diff before merging to main to guarantee stability.
• Security Hotfix Review: Use it when reviewing patches that touch authentication or data models to ensure no vulnerabilities are introduced.
• Design Pattern Validation: Submit a PR for a new module and ask the agent to verify its compliance with established Hometower design patterns.