Skip to content

fix: revert unauthorized CLAUDE.md change from #1081#1511

Merged
ericksoa merged 2 commits intomainfrom
fix/revert-claude-md-unauthorized-change
Apr 5, 2026
Merged

fix: revert unauthorized CLAUDE.md change from #1081#1511
ericksoa merged 2 commits intomainfrom
fix/revert-claude-md-unauthorized-change

Conversation

@ericksoa
Copy link
Copy Markdown
Contributor

@ericksoa ericksoa commented Apr 5, 2026

Summary

Test plan

Summary by CodeRabbit

  • Chores
    • Removed obsolete internal documentation entries.

The security credential injection PR (#1081) inadvertently included
an unrelated CLAUDE.md modification that added a "Claude Behavior
Rules" section. This was not part of the reviewed security change
and was never requested. Revert CLAUDE.md to its pre-#1081 state.

Signed-off-by: Aaron Erickson <aerickson@nvidia.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 5, 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: 2c900001-3c6b-42b6-85cf-6ef330286331

📥 Commits

Reviewing files that changed from the base of the PR and between 39b0761 and d7586c6.

📒 Files selected for processing (1)
  • CLAUDE.md
💤 Files with no reviewable changes (1)
  • CLAUDE.md

📝 Walkthrough

Walkthrough

Removal of the "Claude Behavior Rules" section from CLAUDE.md, which previously contained instructions for triggering nightly E2E test runs via a specific GitHub CLI dispatch command. No functional code changes.

Changes

Cohort / File(s) Summary
Documentation Update
CLAUDE.md
Removed 4 lines containing the "Claude Behavior Rules" section that documented a trigger for dispatching nightly E2E tests with a GitHub API command.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 Hop, hop, away went the rules of old,
Behavior directives no longer told,
E2E triggers removed from the file,
Cleaner docs now, with a rabbit's smile!

🚥 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 accurately reflects the main objective: reverting an unauthorized change to CLAUDE.md from a previous PR.
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 fix/revert-claude-md-unauthorized-change

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

@ericksoa ericksoa merged commit 77ede29 into main Apr 5, 2026
4 checks passed
tranzmatt pushed a commit to tranzmatt/NemoClaw that referenced this pull request Apr 6, 2026
## Summary

- Reverts the unrelated `CLAUDE.md` modification that was inadvertently
included in the security credential injection PR (NVIDIA#1081)
- The added "Claude Behavior Rules" section was never reviewed as part
of that security change and was not requested

## Test plan

- [x] Verify `CLAUDE.md` matches its pre-NVIDIA#1081 state
- [x] No other files affected

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->

## Summary by CodeRabbit

* **Chores**
  * Removed obsolete internal documentation entries.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Signed-off-by: Aaron Erickson <aerickson@nvidia.com>
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