Integrate AI code reviewer

[ylwu-amzn]

Is your feature request related to a problem? Please describe

The current code review process can be time-consuming and may miss certain issues that an AI could potentially catch. Human reviewers may have inconsistent standards or overlook minor details due to fatigue or time constraints. Additionally, there's a need for faster initial feedback on code changes, especially for large repositories with high commit frequencies.

Describe the solution you'd like

We propose integrating an AI code reviewer into our GitHub workflow. The AI reviewer would:

1. Automatically analyze pull requests and provide feedback on code quality, style, and potential bugs
2. Suggest optimizations and best practices
3. Identify security vulnerabilities
4. Check for consistency with project-specific coding standards
5. Provide explanations for its suggestions to help developers learn and improve
6. Work alongside human reviewers, not replace them, to enhance the overall code review process
7. The AI reviewer should be customizable to fit our project's specific needs and should integrate seamlessly with GitHub's existing code review features.

Describe alternatives you've considered

1. Using static code analysis tools: While useful, they lack the contextual understanding and learning capabilities of AI
2. Implementing stricter code linting rules: This can catch some issues but may not provide the depth of analysis an AI could offer
3. Increasing the number of human reviewers: This could be costly and may not necessarily improve consistency or speed

Additional context

One example AI code reviewer action: https://github.com/marketplace/actions/ai-code-review-action

[brianf-aws]

I agree that we should leverage AI to do some time saving and mental effort. To add on to the use cases I think it would be great if they could also do this

• A summary of what the PR does; this lowers the mental overhead to understand what they are reviewing Here is a scenario that i thought the AI model could do. (This PR introduces feature X that interacts with classes a,b,c. Its most likely trying to do this... Based on the UTs wrote a sample input and output scenario looks like this.)
• Be able to to tell what scenarios you didn't test (We have a hard time thinking of edge cases until they happen)

[owaiskazi19]

This is nice and thanks @ylwu-amzn for the proposal. The only question I have is that the action requires an OpenAI key, are we fine with registering one for the action?

[ylwu-amzn]

This is nice and thanks @ylwu-amzn for the proposal. The only question I have is that the action requires an OpenAI key, are we fine with registering one for the action?

I think should be ok. Open to discuss this.