Skip to content

docs: CLAUDE.md に判断の先送り防止ルールを追加#270

Merged
coji merged 1 commit intomainfrom
docs/claude-md-decision-rules
Mar 31, 2026
Merged

docs: CLAUDE.md に判断の先送り防止ルールを追加#270
coji merged 1 commit intomainfrom
docs/claude-md-decision-rules

Conversation

@coji
Copy link
Copy Markdown
Owner

@coji coji commented Mar 31, 2026

Summary

作業方針に2項目を追加:

  • 「やらない」なら理由を言い切る — 「スコープ外」「別PRで」は判断ではなく先送り。やらない理由を具体的に述べる
  • 「動く」と「正しい」を混同しない — テストが通る簡易実装で満足せず、設計書の方針と照合する

背景

今日のセッションで、レビュー指摘に対して「スコープ外」と言って判断を先送りにしたり、設計書に書いた状態機械を実装せず簡易版で済ませようとするパターンが繰り返し発生した。既存のルール(「スコープ外でも対応する」「本質的な対応を優先する」)だけでは防げなかったため、より具体的なガードレールを追加する。

🤖 Generated with Claude Code

Summary by CodeRabbit

リリースノート

  • ドキュメント
    • 開発ガイドラインを更新しました。問題特定時の具体的な理由の明記要件を追加し、実装の検証プロセスを強化しました。

「スコープ外」で判断を先送りにするパターンと、
簡易実装で済ませるパターンを防ぐためのルールを追加。

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 31, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ed9b0c06-b6c9-431f-a2b8-16a6ee9be7c2

📥 Commits

Reviewing files that changed from the base of the PR and between febd5a8 and 6bfec70.

📒 Files selected for processing (1)
  • CLAUDE.md

📝 Walkthrough

ウォークスルー

CLAUDE.mdに2つの新しい作業行動ガイドラインを追加しました。問題が特定されたが未対処の場合に具体的な理由の明示を要求し、テスト通過に加えて設計意図との整合性を検証することを求めています。

変更内容

コホート / ファイル 要約
ドキュメント更新
CLAUDE.md
問題の未対処理由の明示化と、実装が設計ドキュメント/イシュー意図と整合していることを検証する要件を追加

推定レビュー努力

🎯 1 (Trivial) | ⏱️ ~2 minutes

🐰 新しきガイドライン芽吹きて
問題の根をしっかり掘り
テスト通すだけでなく
設計の道と歩み寄り
正しさへと導く灯

🚥 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 PRタイトルは変更内容と完全に関連しており、CLAUDE.mdに判断の先送り防止ルールを追加することを明確に要約している。
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 docs/claude-md-decision-rules

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@coji coji merged commit 53a3329 into main Mar 31, 2026
6 checks passed
@coji coji deleted the docs/claude-md-decision-rules branch March 31, 2026 09:32
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