Skip to content

fix: restore CLAUDE.md in .agents/skills/sync-agent-instructions#81

Open
xiaolai wants to merge 1 commit into
ReflexioAI:mainfrom
xiaolai:fix/nlpm-sync-claude-md
Open

fix: restore CLAUDE.md in .agents/skills/sync-agent-instructions#81
xiaolai wants to merge 1 commit into
ReflexioAI:mainfrom
xiaolai:fix/nlpm-sync-claude-md

Conversation

@xiaolai

@xiaolai xiaolai commented Jun 20, 2026

Copy link
Copy Markdown

Automated: drive-by fix from NLPM, an NL artifact linter. Reviewed and reproduced before submission.

Bug: .agents/skills/sync-agent-instructions/SKILL.md lists AGENTS.md twice (once where CLAUDE.md should be) in the frontmatter description, the h1, and the git diff command in Step 1 — CLAUDE.md is absent from all three places. The git diff HEAD -- AGENTS.md GEMINI.md AGENTS.md command in Step 1 will never detect changes to CLAUDE.md. The .claude/skills/sync-agent-instructions/SKILL.md copy is correct and unaffected.

Evidence: Line 3 frontmatter reads AGENTS.md, GEMINI.md, AGENTS.md; line 8 h1 reads AGENTS.md, GEMINI.md, and AGENTS.md; line 12 git diff lists AGENTS.md GEMINI.md AGENTS.mdCLAUDE.md absent in all three. The .claude/skills/ copy correctly lists CLAUDE.md in each location.

Fix: Replace the duplicate AGENTS.md with CLAUDE.md in the frontmatter description, h1, and Step 1 git diff command — matching the .claude/skills/ copy exactly.

Summary by CodeRabbit

  • Documentation
    • Corrected agent instructions documentation to ensure proper synchronization of configuration files and updated the workflow to accurately track all instruction sources, improving consistency across agent configurations.

The .agents/skills/ copy had AGENTS.md listed twice (once as the first
file and once where CLAUDE.md should be) in the frontmatter description,
h1, and the git diff command in Step 1. The .claude/skills/ copy is
correct. This brings the two copies back in sync.

Co-Authored-By: Claude Code <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jun 20, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 81f879e5-fa04-4e1c-ac7b-809dbeeee245

📥 Commits

Reviewing files that changed from the base of the PR and between 3d1612c and 21751ab.

📒 Files selected for processing (1)
  • .agents/skills/sync-agent-instructions/SKILL.md

📝 Walkthrough

Walkthrough

A documentation fix in .agents/skills/sync-agent-instructions/SKILL.md corrects a duplicated AGENTS.md reference. The file set is updated to correctly list CLAUDE.md, GEMINI.md, and AGENTS.md in the description, keep-in-sync statement, and the git diff example command.

Changes

Sync-Agent Instructions Documentation Fix

Layer / File(s) Summary
Correct file references in SKILL.md
.agents/skills/sync-agent-instructions/SKILL.md
Replaces the duplicated AGENTS.md entry with CLAUDE.md in the description, the keep-in-sync statement, and the git diff HEAD -- ... example command so all three distinct files are correctly named.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A bunny once listed the same file twice,
But caught the mistake — wasn't that nice?
CLAUDE.md hops in,
GEMINI.md grins,
And AGENTS.md nods with precise! 🐇

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'fix: restore CLAUDE.md in .agents/skills/sync-agent-instructions' directly and clearly summarizes the main change—restoring the missing CLAUDE.md file reference in the sync-agent instructions documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

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