chore: add CodeRabbit configuration file#165
Open
Muneerali199 wants to merge 2 commits intoAOSSIE-Org:devfrom
Open
chore: add CodeRabbit configuration file#165Muneerali199 wants to merge 2 commits intoAOSSIE-Org:devfrom
Muneerali199 wants to merge 2 commits intoAOSSIE-Org:devfrom
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Author
|
please sir @M4dhav review it when you are free and let me know if further changes require |
M4dhav
requested changes
Feb 22, 2026
Comment on lines
+24
to
+31
| planning: | ||
| enabled: true | ||
| auto_planning: | ||
| enabled: true | ||
| labels: | ||
| - "plan-me" # Auto-plan issues with this label | ||
| - "feature" # Also auto-plan these | ||
| - "!no-plan" # Never auto-plan issues with this label |
Contributor
There was a problem hiding this comment.
Turn off Planning. AI should not be used for Architecture and Feature Planning in it's entirety.
.coderabbit.yaml
Outdated
| # Only auto-review PRs targeting these branches | ||
| base_branches: | ||
| - main | ||
| - develop |
Contributor
There was a problem hiding this comment.
We use the dev branch, not develop
.coderabbit.yaml
Outdated
| sequence_diagrams: true | ||
|
|
||
| # Include poems in reviews | ||
| poem: true |
Contributor
There was a problem hiding this comment.
We don't need poems, can set this to false.
.coderabbit.yaml
Outdated
| review_status: true | ||
|
|
||
| # Keep the walkthrough section expanded by default | ||
| collapse_walkthrough: false |
.coderabbit.yaml
Outdated
| pre_merge_checks: | ||
| description: | ||
| # Validate that PR has a proper description | ||
| mode: warning # Options: off, warning, error |
.coderabbit.yaml
Outdated
Comment on lines
+202
to
+214
| # HTML files | ||
| - path: "**/*.html" | ||
| instructions: | | ||
| Review the HTML code against the google html style guide and point out any mismatches. Ensure that: | ||
| - The code adheres to best practices recommended by lighthouse or similar tools for performance | ||
|
|
||
| # CSS files | ||
| - path: "**/*.css" | ||
| instructions: | | ||
| Review the CSS code against the google css style guide and point out any mismatches. Ensure that: | ||
| - The code adheres to best practices associated with CSS. | ||
| - The code adheres to best practices recommended by lighthouse or similar tools for performance. | ||
| - The code adheres to similar naming conventions for classes, ids. |
.coderabbit.yaml
Outdated
Comment on lines
+222
to
+245
| # Solidity Smart Contract files | ||
| - path: "**/*.sol" | ||
| instructions: | | ||
| Solidity: | ||
| - Review the Solidity contracts for security vulnerabilities and adherence to best practices. | ||
| - Ensure immutability is used appropriately (e.g., `immutable` and `constant` where applicable). | ||
| - Ensure there are no unbounded loops that could lead to gas exhaustion. | ||
| - Verify correct and explicit visibility modifiers for all state variables and functions. | ||
| - Flag variables that are declared but used only once or are unnecessary. | ||
| - Identify potential gas optimization opportunities without compromising readability or security. | ||
| - Verify that any modification to contract logic includes corresponding updates to automated tests. | ||
| - Ensure failure paths and revert scenarios are explicitly handled and validated. | ||
| - Validate proper access control enforcement (e.g., Ownable, RBAC, role checks). | ||
| - Ensure consistent and correct event emission for all state-changing operations. | ||
| - Confirm architectural consistency with existing contracts (no unintended storage layout changes unless clearly documented). | ||
| - Flag major feature additions or architectural changes that were implemented without prior design discussion (if applicable). | ||
| - Flag pull requests that mix unrelated changes or multiple concerns in a single submission. | ||
| - Ensure security-sensitive logic changes are not introduced without adequate test coverage. | ||
| - Review for common smart contract vulnerabilities, including but not limited to: | ||
| - Reentrancy | ||
| - Improper input validation | ||
| - Access control bypass | ||
| - Integer overflows/underflows (if using unchecked blocks) | ||
| - Front-running risks where applicable |
.coderabbit.yaml
Outdated
| instructions: | | ||
| Review test files for: | ||
| - Comprehensive coverage of component behavior | ||
| - Proper use of @testing-library/react-native |
.coderabbit.yaml
Outdated
Comment on lines
+259
to
+270
| # Solidity test files | ||
| - path: "**/*.test.{sol}" | ||
| instructions: | | ||
| Review test files for: | ||
| - Comprehensive coverage of contract behavior. | ||
| - Coverage of success paths, edge cases, and failure/revert scenarios. | ||
| - Proper validation of access control restrictions. | ||
| - Verification of event emissions where applicable. | ||
| - Explicit validation of state changes after each relevant function call. | ||
| - Adequate test updates whenever contract logic is modified. | ||
| - Deterministic behavior (tests should not rely on implicit execution order or shared mutable state). | ||
| - Clear and descriptive test names that reflect the intended behavior being validated. |
.coderabbit.yaml
Outdated
|
|
||
|
|
||
| # Asset files (images, fonts, etc.) | ||
| - path: "assets/**/*" |
Contributor
There was a problem hiding this comment.
Our assets aren't inside a subfolder
Author
|
sir @M4dhav , I Applied all requested changes to .coderabbit.yaml and cleaned up irrelevant configurations. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds the standardized CodeRabbit configuration file (.coderabbit.yaml)
as discussed here :https://discord.com/channels/1022871757289422898/1235508903450775622/1474646668183081010