Skip to content

ci: add CodeRabbit.ai review configuration#30

Open
don-petry wants to merge 3 commits intooneirosoft:mainfrom
don-petry:ci/coderabbit
Open

ci: add CodeRabbit.ai review configuration#30
don-petry wants to merge 3 commits intooneirosoft:mainfrom
don-petry:ci/coderabbit

Conversation

@don-petry
Copy link
Copy Markdown
Contributor

@don-petry don-petry commented Mar 31, 2026

Why?

Manual code review is time-consuming and easy to miss subtle issues in. CodeRabbit provides AI-powered review with domain-specific context (e.g., understanding that store operations need atomicity, or that CLI output must be deterministic). This complements human review rather than replacing it.

Summary

  • Add .coderabbit.yaml with path-specific review instructions for store, git, and CLI modules
  • Enable clippy and shellcheck review tools
  • Set review profile to "chill" with auto-review on non-draft PRs

Refs #11 (item 10)

Note for maintainer

The CodeRabbit GitHub App must be installed on the oneirosoft/dagger repository for this configuration to take effect. Install it at https://coderabbit.ai — it is free for open-source repos.

🤖 Generated with Claude Code

Add .coderabbit.yaml with path-specific review instructions for
store (atomicity/crash safety), git.rs (command injection), and
CLI (thin adapter pattern). Enables clippy and shellcheck tools.

Refs oneirosoft#11 (item 10)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 31, 2026 02:48
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds CodeRabbit configuration to enable automated AI reviews with module-specific guidance and linting tool integrations.

Changes:

  • Add .coderabbit.yaml with path-specific review instructions for src/core/store/**, src/core/git.rs, and src/cli/**
  • Enable CodeRabbit review tools (clippy, shellcheck) and set the review profile to chill with auto-review on non-draft PRs
  • Enable chat auto-replies

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

.coderabbit.yaml Outdated
Comment on lines +20 to +21
Verify CLI layer is a thin adapter. Business logic should live in
src/core/. Check that error messages are user-friendly.
Copy link

Copilot AI Mar 31, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the path instruction text, src/core/. looks like a typo (extra .). Consider changing it to src/core/ to avoid confusing readers and to match the actual directory path.

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Addressed in the latest push.

DJ and others added 2 commits March 30, 2026 20:09
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants