Fix Review

Overview

When to Use This Skill

• Reviewing commits that address security audit findings
• Verifying that fixes don't introduce new vulnerabilities
• Ensuring code changes properly resolve identified issues
• Validating that remediation efforts are complete and correct

Instructions

1. Review Fix Commits: Analyze commits that claim to fix audit findings
2. Verify Resolution: Ensure the original issue is properly addressed
3. Check for Regressions: Verify no new bugs or vulnerabilities are introduced
4. Validate Completeness: Ensure all aspects of the finding are resolved

Review Process

1. Compare the fix against the original audit finding
2. Verify the fix addresses the root cause, not just symptoms
3. Check for potential side effects or new issues
4. Validate that tests cover the fixed scenario
5. Ensure no similar vulnerabilities exist elsewhere

Best Practices

• Review fixes in context of the full codebase
• Verify test coverage for the fixed issue
• Check for similar patterns that might need fixing
• Ensure fixes follow security best practices
• Document the resolution approach

Resources