Skip to content

chore: add CodeRabbit configuration#576

Merged
t0k0sh1 merged 3 commits into
v0.0.6from
chore/add-coderabbit-config
Apr 2, 2026
Merged

chore: add CodeRabbit configuration#576
t0k0sh1 merged 3 commits into
v0.0.6from
chore/add-coderabbit-config

Conversation

@t0k0sh1

@t0k0sh1 t0k0sh1 commented Apr 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add .coderabbit.yaml for automated PR reviews
  • Reviews target PRs to vx.x.x release branches only
  • Assertive review profile with English comments
  • Excludes build artifacts (build/, build-asan/, *.pdf)

Test plan

  • Create a test PR targeting v0.0.6 and verify CodeRabbit posts review comments
  • Confirm excluded paths are not reviewed

Summary by CodeRabbit

  • Chores
    • Updated automated code review configuration: set review language to English and an assertive review profile, enable automatic review generation while disabling draft reviews, limit auto-reviews to release branches that follow version tagging, and exclude build artifact directories and PDF files from review processing.

@coderabbitai

coderabbitai Bot commented Apr 2, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

Adds a new .coderabbit.yaml configuration file that sets review language to English, uses the assertive review profile, enables automated (non-draft) reviews, restricts auto-review triggering to base branches matching ^v\d+\.\d+\.\d+$, and excludes build/**, build-asan/**, and **/*.pdf via path filters.

Changes

Cohort / File(s) Summary
Configuration
/.coderabbit.yaml
New CodeRabbit config: language: en, review_profile: assertive, auto: true, drafts: false, auto limited to base branches matching ^v\d+\.\d+\.\d+$, and path_filters excluding build/**, build-asan/**, and **/*.pdf.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

Poem

🐰 A small config hopped into view,
English checks and assertive too,
Drafts tucked out, autos on cue,
Builds and PDFs left for the dew. ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'chore: add CodeRabbit configuration' clearly and accurately summarizes the main change: adding a new .coderabbit.yaml configuration file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/add-coderabbit-config

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

@t0k0sh1 t0k0sh1 marked this pull request as ready for review April 2, 2026 12:01

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.coderabbit.yaml:
- Around line 13-14: Remove the unsupported global toggle by deleting the
invalid key "reviews.tools.enabled" and instead enable tools individually under
"reviews.tools.<tool>.enabled" (e.g., "reviews.tools.ruff.enabled",
"reviews.tools.eslint.enabled") where each tool has its own enabled flag; update
the YAML to remove the top-level "tools.enabled: true" entry and add per-tool
entries for any tools you intend to enable so the configuration conforms to the
schema.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 7cda5bf1-840e-4087-a21f-9c7918e7c981

📥 Commits

Reviewing files that changed from the base of the PR and between 9efbf24 and 5560b28.

📒 Files selected for processing (1)
  • .coderabbit.yaml

Comment thread .coderabbit.yaml Outdated
Copilot AI review requested due to automatic review settings April 2, 2026 12:06

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a CodeRabbit configuration file to enable automated PR reviews, scoped to release branches and excluding build artifacts.

Changes:

  • Introduces .coderabbit.yaml with English, assertive review profile and auto-review enabled (non-draft PRs only)
  • Restricts auto-reviews to vX.Y.Z-style base branches
  • Excludes build/, build-asan/, and *.pdf paths from review

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

Comment thread .coderabbit.yaml Outdated
enabled: true
drafts: false
base_branches:
- 'v[0-9]+\.[0-9]+\.[0-9]+'

Copilot AI Apr 2, 2026

Copy link

Choose a reason for hiding this comment

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

base_branches pattern is single-quoted YAML and currently contains double backslashes (\\.). In YAML single quotes, backslashes aren’t unescaped, so the regex passed to CodeRabbit will include \\. and may match a literal backslash + any character instead of a dot separator. Use a single backslash (\.) (or a clearer anchored pattern like ^v\d+\.\d+\.\d+$ if the intent is to match only exact release branch names).

Suggested change
- 'v[0-9]+\.[0-9]+\.[0-9]+'
- '^v\d+\.\d+\.\d+$'

Copilot uses AI. Check for mistakes.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.coderabbit.yaml:
- Around line 7-8: The base_branches regex currently uses partial matching and
will match branches containing the semver substring; update the base_branches
entry (the value for the base_branches key) to use an anchored regex
^v[0-9]+\.[0-9]+\.[0-9]+$ so only branches that exactly match the
vMAJOR.MINOR.PATCH pattern trigger auto-review.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 72f25ded-dc71-4bc3-a519-47b76bd8d9f5

📥 Commits

Reviewing files that changed from the base of the PR and between 5560b28 and 66d2e1e.

📒 Files selected for processing (1)
  • .coderabbit.yaml

Comment thread .coderabbit.yaml Outdated

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In @.coderabbit.yaml:
- Line 1: Update the language locale value from the generic "en" to the more
specific "en-US" in the configuration by replacing the entry "language: en" with
"language: en-US" so the file uses the schema-default, explicit locale.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 46c15b6e-aa21-46e7-9ada-65cd1ee00055

📥 Commits

Reviewing files that changed from the base of the PR and between 66d2e1e and ba95912.

📒 Files selected for processing (1)
  • .coderabbit.yaml

Comment thread .coderabbit.yaml
@@ -0,0 +1,12 @@
language: en

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🧹 Nitpick | 🔵 Trivial

Consider using a more specific language locale.

While en is valid, using en-US (the schema default) is more explicit and aligns with common practice in CodeRabbit configurations.

📝 Optional refinement
-language: en
+language: en-US
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In @.coderabbit.yaml at line 1, Update the language locale value from the
generic "en" to the more specific "en-US" in the configuration by replacing the
entry "language: en" with "language: en-US" so the file uses the schema-default,
explicit locale.

@t0k0sh1 t0k0sh1 merged commit 2eb4a1a into v0.0.6 Apr 2, 2026
9 of 10 checks passed
@t0k0sh1 t0k0sh1 deleted the chore/add-coderabbit-config branch April 2, 2026 12:27
@coderabbitai coderabbitai Bot mentioned this pull request Apr 3, 2026
2 tasks
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