Skip to content

chore(root): CodeRabbit のレビュー方針を調整#51

Merged
HappyOnigiri merged 3 commits intomainfrom
chore/update-coderabbit-tone
Mar 11, 2026
Merged

chore(root): CodeRabbit のレビュー方針を調整#51
HappyOnigiri merged 3 commits intomainfrom
chore/update-coderabbit-tone

Conversation

@HappyOnigiri
Copy link
Owner

@HappyOnigiri HappyOnigiri commented Mar 11, 2026

Summary

  • CodeRabbit のレビュー方針(tone_instructions)を調整し、より重要な指摘に集中するように設定を更新しました。

Related issues

  • なし

Changes

  • .coderabbit.yamltone_instructions を更新し、以下の制限を追加しました:
    • 将来的な対応を促すコメント(「今のところは許容できるが〜」など)の禁止。
    • 本当に問題がある事項のみを指摘する。
    • 低優先度のアドバイスや軽微なリファクタリング、フォーマットの指摘を避ける。
    • 重要な洞察がある場合は Nitpick として許可する。

Testing

  • make ci (pytest) を実行し、既存のテストがパスすることを確認しました。

Checklist

  • I verified the change locally or in CI as appropriate.
  • I added or updated tests when needed.
  • I updated documentation for user-facing changes.
  • I confirmed there are no unintended breaking changes, or documented them clearly.

Summary by CodeRabbit

  • その他の改善
    • レビュー指示の表現がより詳細になり、コメントのトーンや優先度に関する制約が強化されました。
    • レビューのワークフローや出力(要約、状態、詩、シーケンス図、ウォークスルーの折り畳み等)を細かく制御できる設定が多数追加されました。
    • 一部ツールの設定が削除され、利用可能なツール構成が調整されました。

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 11, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 6694fbb2-47a0-4edb-b671-7ec5962fddaa

📥 Commits

Reviewing files that changed from the base of the PR and between c4cf0bc and e2af003.

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

📝 Walkthrough

Walkthrough

.coderabbit.yamlが更新され、tone_instructionsが単一行から複数行ブロックへ変更されました。reviewsに多数のブールフラグとpath_filtersが追加され、toolsからast-grepエントリが削除されました。レビュープロンプト設定全体が拡張されています。

Changes

Cohort / File(s) Summary
Configuration File
​.coderabbit.yaml
tone_instructionsを単一行から複数行ブロックへ変更。reviewsrequest_changes_workflowhigh_level_summarypoemsequence_diagramscollapse_walkthroughreview_statusenable_prompt_for_ai_agentsin_progress_fortuneなど多数のブールフラグとpath_filtersリストを追加。toolsからast-grepエントリを削除し、レビュー促進設定を拡張。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed タイトルは .coderabbit.yaml のレビュー方針調整という実際の変更内容を正確に反映しており、プルリクエストの主要な目的を明確に示しています。
Description check ✅ Passed プルリクエスト説明はテンプレートのすべての必須セクション(Summary、Related issues、Changes、Testing、Checklist)を完備し、変更内容の詳細な説明とテスト確認を含んでいます。
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
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/update-coderabbit-tone

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: 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 9: The line containing the block scalar text that ends with an extraneous
double quote (the line starting with
"ただし、重要な洞察や論理的な問題が含まれる場合、「Nitpick」としての指摘は許可します。\"") should have the trailing
quote removed; since this value is inside a YAML block scalar (|) the quotes are
unnecessary and the stray `"` is being included in the string—edit that line to
delete the final `"` so the text ends with the Japanese punctuation only.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: b71edba1-43fe-40fc-9165-ffc67ddc6c85

📥 Commits

Reviewing files that changed from the base of the PR and between 9304a3d and e249b4d.

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

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.

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
.coderabbit.yaml (1)

38-39: ⚠️ Potential issue | 🟠 Major

ast-grepenabled 設定はスキーマに対応していません。

CodeRabbit のスキーマでは ast-grep は他のツール(shellcheck など)と異なり、enabled プロパティをサポートしていません。代わりに rule_dirsutil_dirsessential_rulespackages で設定します。現在のこの設定は認識されず、ast-grep は実際には無効化されません。

削除するか、以下のいずれかで置き換えてください:

  • reviews.tools.ast-grep 全体を削除、または
  • essential_rules: false に設定して rule_dirs/util_dirs/packages を空にする
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In @.coderabbit.yaml around lines 38 - 39, The ast-grep block currently uses an
unsupported property `enabled`; update the `reviews.tools.ast-grep`
configuration by removing the `enabled: false` line (or delete the entire
`reviews.tools.ast-grep` block), or replace it with schema-compliant settings
such as setting `essential_rules: false` and clearing `rule_dirs`, `util_dirs`,
and `packages` so ast-grep is effectively disabled; locate the `ast-grep` entry
in the .coderabbit.yaml and apply one of these schema-compatible fixes.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Outside diff comments:
In @.coderabbit.yaml:
- Around line 38-39: The ast-grep block currently uses an unsupported property
`enabled`; update the `reviews.tools.ast-grep` configuration by removing the
`enabled: false` line (or delete the entire `reviews.tools.ast-grep` block), or
replace it with schema-compliant settings such as setting `essential_rules:
false` and clearing `rule_dirs`, `util_dirs`, and `packages` so ast-grep is
effectively disabled; locate the `ast-grep` entry in the .coderabbit.yaml and
apply one of these schema-compatible fixes.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 0f6c67f9-70ba-400e-8975-a31dcc7bf14b

📥 Commits

Reviewing files that changed from the base of the PR and between e249b4d and c4cf0bc.

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

@HappyOnigiri HappyOnigiri merged commit 5b81fb9 into main Mar 11, 2026
2 checks passed
@HappyOnigiri HappyOnigiri deleted the chore/update-coderabbit-tone branch March 11, 2026 20:59
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