Skip to content

BOTでも問答無用にCodeRabbitがレヴューを始めるようにした#4548

Merged
TinyKitten merged 3 commits intodevfrom
feature/crai-for-bots
Nov 2, 2025
Merged

BOTでも問答無用にCodeRabbitがレヴューを始めるようにした#4548
TinyKitten merged 3 commits intodevfrom
feature/crai-for-bots

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Nov 2, 2025

Summary by CodeRabbit

  • Chores
    • Configured automated review workflows: automatic review triggering enabled.
    • Review workflow now enforces a "request changes" path for flagged items rather than auto-approving.

@TinyKitten TinyKitten self-assigned this Nov 2, 2025
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 2, 2025

Walkthrough

Adds a new .coderabbit.yml configuration file that enables automated review workflows and enables the request-changes workflow flag; no code or public API declarations changed.

Changes

Cohort / File(s) Summary
CI / Review configuration
/.coderabbit.yml
New YAML config introducing auto_review.enabled: true, auto_review.ignore_usernames: [], and reviews.request_changes_workflow: true to drive automated review behavior.

Sequence Diagram(s)

sequenceDiagram
    autonumber
    participant Dev as Developer
    participant Git as Git/PR
    participant CR as CodeRabbit (config)
    participant Bot as Auto-review Bot

    Dev->>Git: Open or update Pull Request
    Note right of Git #f9f2d7: PR events (open/synchronize)
    Git->>CR: Read `/.coderabbit.yml`
    CR-->>Bot: auto_review.enabled = true\nignore_usernames = []\nrequest_changes_workflow = true
    Bot->>Git: Run automated review (may request changes per workflow)
    Note right of Bot #e6f7ff: Review complete (request-changes process possible)
Loading

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

  • Verify YAML syntax and schema keys (auto_review.*, reviews.request_changes_workflow).
  • Confirm auto_review.ignore_usernames: [] and request-changes behavior match project policy.

Poem

🐇 A tiny YAML hops anew,
Tells the bot just what to do.
Reviews will run, requests may spring,
Quiet rules that make PRs sing. ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The pull request title, when translated from Japanese, describes enabling CodeRabbit to start reviews unconditionally even for bot-created pull requests. The changeset modifies .coderabbit.yml to enable auto_review functionality and set auto_review.ignore_usernames to an empty list, which directly implements this intent by removing any username exclusions from automated reviews. The title is specific, clearly communicates the primary change, and accurately reflects the configuration modifications being made.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/crai-for-bots

📜 Recent review details

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between eccd4d8 and 58c13d2.

📒 Files selected for processing (1)
  • .coderabbit.yml (1 hunks)
🧰 Additional context used
🧠 Learnings (2)
📓 Common learnings
Learnt from: TinyKitten
Repo: TrainLCD/MobileApp PR: 0
File: :0-0
Timestamp: 2025-08-11T21:13:39.083Z
Learning: TinyKitten prefers concise code reviews for the TrainLCD/MobileApp repository. Keep review comments brief and to the point.
📚 Learning: 2025-08-11T21:13:39.083Z
Learnt from: TinyKitten
Repo: TrainLCD/MobileApp PR: 0
File: :0-0
Timestamp: 2025-08-11T21:13:39.083Z
Learning: TinyKitten prefers concise code reviews for the TrainLCD/MobileApp repository. Keep review comments brief and to the point.

Applied to files:

  • .coderabbit.yml
🔇 Additional comments (1)
.coderabbit.yml (1)

1-6: Configuration looks good.

The setup correctly enables automatic reviews for all PRs (including bot-created ones, as intended) and enables the request-changes workflow. All properties conform to the CodeRabbit schema.


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

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

📜 Review details

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a17e2f0 and 0980edc.

📒 Files selected for processing (1)
  • .coderabbit.yml (1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: TinyKitten
Repo: TrainLCD/MobileApp PR: 0
File: :0-0
Timestamp: 2025-08-11T21:13:39.083Z
Learning: TinyKitten prefers concise code reviews for the TrainLCD/MobileApp repository. Keep review comments brief and to the point.
🔇 Additional comments (1)
.coderabbit.yml (1)

11-12: The auto_approve property is present in the .coderabbit.yml file.

The review comment claims auto_approve is "not found in the CodeRabbit review schema," but the actual .coderabbit.yml configuration file in this repository actively uses this property under the review: section. While CodeRabbit documentation is not available in the repository, the property is demonstrably configured in the codebase and should not be flagged as invalid based solely on absence of documentation.

Likely an incorrect or invalid review comment.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0980edc and eccd4d8.

📒 Files selected for processing (1)
  • .coderabbit.yml (1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: TinyKitten
Repo: TrainLCD/MobileApp PR: 0
File: :0-0
Timestamp: 2025-08-11T21:13:39.083Z
Learning: TinyKitten prefers concise code reviews for the TrainLCD/MobileApp repository. Keep review comments brief and to the point.

@TinyKitten TinyKitten merged commit 0646492 into dev Nov 2, 2025
4 checks passed
@TinyKitten TinyKitten deleted the feature/crai-for-bots branch November 2, 2025 15:54
TinyKitten added a commit that referenced this pull request Nov 2, 2025
* Add automated version bump workflow for canary PRs (#4547)

* Initial plan

* Add GitHub Actions workflow for auto version bump on canary PRs

Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

* Fix security issues in version bump workflow

Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

* Improve git diff check logic for better readability

Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

* Use git add -A for better maintainability

Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

* Add explicit error handling for version bump script

Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

* Add location accuracy history chart to DevOverlay (#4543)

* Initial plan

* Add location accuracy history chart to DevOverlay

Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

* Fix tests after location store refactoring

Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

* Address code review feedback: improve accuracy handling and use reduce instead of spread

Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

* Merge dev branch into accuracy chart feature

Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

* Push merge commit to remote

Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

* Remove temporary file

Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

* Add robust validation for accuracy values in location store and chart generation

Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

* BOTでも問答無用にCodeRabbitがレヴューを始めるようにした (#4548)

* BOTでも問答無用にCodeRabbitがレヴューを始めるようにした

* 不正なフィールドの削除と置換

* Update review configuration to enable request changes workflow

* Remove unnecessary submodule checkout from bump_version_on_pr workflow (#4554)

* Initial plan

* Fix: Add SSH key for private submodule access in bump_version_on_pr workflow

Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

* Remove unnecessary submodule checkout from bump_version_on_pr workflow

Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

* Replace direct push with PR creation in version bump workflow (#4556)

* Initial plan

* Change version bump workflow from push to PR creation

Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

* Bump version for canary release (#4557)

Co-authored-by: TinyKitten <32848922+TinyKitten@users.noreply.github.com>

* CodeRabbitは日本語で喋ってもろて (#4560)

* CodeRabbitは日本語で喋ってもろて

* Fix language declaration in .coderabbit.yml configuration

* 一つのPRに対してbump versionのPR自動作成は1回だけにする (#4559)

* Update bump_version_on_pr workflow to trigger only on opened pull requests

* Remove workflow_dispatch trigger from bump_version_on_pr workflow

* GitHub Copilotの活用に関する指示書を新規作成 (#4562)

---------

Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@coderabbitai coderabbitai bot mentioned this pull request Dec 20, 2025
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.

1 participant