CLAUDE.md: 非実行ファイルへの不要テスト防止ガードレール強化#518
Merged
clonable-eden merged 1 commit intomainfrom Mar 23, 2026
Merged
Conversation
…ble files Add review criterion for Reviewers to detect and reject content-based tests for non-executable files. Clarify TDD priority for Workers: the no-test rule for non-executable files takes precedence over TDD's RED step. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
clonable-eden
commented
Mar 23, 2026
Owner
Author
clonable-eden
left a comment
clonable-eden
left a comment
There was a problem hiding this comment.
Review: APPROVE ✅
All changes correctly implement what issue #517 requested. CI passes.
Correctness
- Testing Criteria (line 63): "No tests for non-executable files" criterion added to
## Review > ### Testing Criteria— Reviewers will now detect and reject content-based tests on non-executable files. - What to Test (lines 225-226): TDD precedence clarification added — Workers will now know that the no-test rule overrides TDD for non-executable file changes.
Conventions
- Documentation-only CLAUDE.md change — no tests required (per the very rule being added).
- PR body includes
closes #517. Branch naming follows conventions.
Scope
- Exactly 3 lines added across 2 sections. No unrelated modifications.
CI
- All checks pass (cekernel tests, auto-approve, test, changes).
LGTM — ready to merge.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
closes #517
Summary
## Review > ### Testing Criteriaに「No tests for non-executable files」基準を追加（Reviewer が content-based テストを検出・reject できるようにする）## Testing > ### What to Testに TDD との優先順位を明示（Worker が非実行ファイル変更で RED テストを書かないようにする）Test Plan
🤖 Generated with Claude Code