Enhance CodeRabbit Solidity Review Instructions for Smart Contract Governance

[kumawatkaran523]

This PR enhances the Solidity and Solidity test review instructions in .coderabbit.yaml to align with the updated smart contract governance process.

Summary by CodeRabbit

• Chores
  • Expanded Solidity code review instructions with detailed security checks, architectural considerations, and common vulnerability detection guidelines.
  • Enhanced test file review requirements to ensure comprehensive coverage of success paths, edge cases, revert scenarios, and state validation.

[coderabbitai]

Warning

Ignoring CodeRabbit configuration file changes. For security, only the configuration from the base branch is applied for open source repositories.

Walkthrough

The .coderabbit.yaml configuration file has been expanded to include comprehensive Solidity review guidance, covering immutability patterns, loop safety, visibility modifiers, security vulnerabilities like reentrancy and access control, test coverage requirements, and state change validation.

Changes

Cohort / File(s)	Summary
Solidity Review Configuration .coderabbit.yaml	Enhanced with detailed checklists for contract reviews including security checks, vulnerability patterns, access control, event emissions, storage considerations, and extended test guidance covering success paths, edge cases, reverts, and state validation.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Suggested labels

Documentation

Poem

🐰 Hops with delight at the security sight!
Solidity contracts now reviewed with care,
From loops to reentrancy, checks everywhere,
Test paths and edge cases, handled with flair,
Better reviews make bunny code rare! ✨

🚥 Pre-merge checks | ✅ 2

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately summarizes the main change—enhancing Solidity review instructions in the CodeRabbit configuration file.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In @.coderabbit.yaml:
- Line 239: The glob "**/*.test.{sol}" is wrong (unnecessary brace and wrong
Foundry suffix); replace it with a pattern that matches both Foundry and the
.test convention—e.g., use "**/*.{t.sol,test.sol}" or two patterns "**/*.t.sol"
and "**/*.test.sol" to ensure files like Counter.t.sol are matched; update the
entry that currently contains the string "**/*.test.{sol}" accordingly.