chore: add CodeRabbit AI review config - #136
Conversation
Configure CodeRabbit for automated PR review on main: - chill profile, auto-review on PRs to main (drafts excluded) - path filters for target/, lockfiles, generated artifacts - path instructions encoding the repo's Rust/WASM/control-plane/UI conventions - enable clippy, eslint, hadolint, actionlint, yamllint, markdownlint, shellcheck, gitleaks Signed-off-by: Nicolas Dreno <nicolas.dreno@daveo.fr>
📝 WalkthroughWalkthroughAdds a complete ChangesCodeRabbit configuration
Estimated code review effort: 3 (Moderate) | ~15–30 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
.coderabbit.yaml (1)
62-66: 🔒 Security & Privacy | 🔵 Trivial | ⚡ Quick winAdd dedicated GitHub Actions security analysis.
The workflow guidance covers action pinning, expression injection, and token permissions, but
actionlintis primarily a static checker. Enable CodeRabbit’s supportedzizmoranalyzer, or document the equivalent security check elsewhere. (docs.coderabbit.ai)Suggested configuration
actionlint: enabled: true + zizmor: + enabled: trueAlso applies to: 75-76
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @.coderabbit.yaml around lines 62 - 66, Update the GitHub Actions configuration in .coderabbit.yaml to enable CodeRabbit’s supported zizmor security analyzer, or add an equivalent dedicated workflow-security check alongside the existing actionlint-related guidance. Keep the current checks for unpinned actions, expression injection, and GITHUB_TOKEN permissions intact.Source: MCP tools
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Nitpick comments:
In @.coderabbit.yaml:
- Around line 62-66: Update the GitHub Actions configuration in .coderabbit.yaml
to enable CodeRabbit’s supported zizmor security analyzer, or add an equivalent
dedicated workflow-security check alongside the existing actionlint-related
guidance. Keep the current checks for unpinned actions, expression injection,
and GITHUB_TOKEN permissions intact.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: 267c2a31-28fd-45ed-96fd-def379a9876c
📒 Files selected for processing (1)
.coderabbit.yaml
What
Adds a
.coderabbit.yamlat the repo root to enable CodeRabbit automated AI review on pull requests.Config highlights
chill— less noisy, skips nitpicks already enforced by rustfmt/clippy/eslint.main(drafts excluded).target/, lockfiles,ui/dist,*.bca,schemas/,book/, snapshots,testdata/.CLAUDE.md:unwrap()/panic!()in prod,thiserrorvsanyhow,parking_lot,is_some_and,HeaderValue::from_static, no unjustified#[allow(dead_code)].barbacane-wasm: sandbox-escape / host↔guest trust boundary focus.barbacane-control: SQLi, authz, input validation.any, unhandled promises, XSS.${{ }}injection, broadGITHUB_TOKENperms.Note
Requires the CodeRabbit GitHub App to be installed on the
barbacane-devorg (free for public repos). Once installed, this PR itself becomes the first review target.Summary by CodeRabbit