If no arguments are passed, review the local changes by looking at the diff between the base branch - main by default - and the current branch.
If arguments are passed, review pull request #$ARGUMENTS by fetching it and seeing its details with gh pr view and gh pr diff.
When reviewing, analyze for:
- •
Code Quality
- •Rust idioms and Polkadot SDK patterns
- •Error handling and unwrap usage
- •Code clarity and maintainability
- •
Security
- •Unsafe code blocks
- •Input validation
- •Access control in pallets
- •
Performance
- •Weight/benchmark implications
- •Storage access patterns
- •Unnecessary allocations
- •
Testing
- •Test coverage for new code
- •Edge cases handled
- •
Breaking Changes
- •API compatibility
- •Migration requirements
Provide specific feedback with file paths and line numbers.