Skip to content

chore: remove CLAUDE.md now that Claude reads AGENTS.md - #4238

Merged
Xuanwo merged 1 commit into
mainfrom
remove-claude-md-files
Sep 21, 2026
Merged

Xuanwo merged 1 commit into
mainfrom
remove-claude-md-files

Conversation

@Xuanwo

@Xuanwo Xuanwo commented Sep 21, 2026

Copy link
Copy Markdown
Member

Summary

  • Remove the CLAUDE.md agent instruction files at the repo root, python/, and nodejs/.
  • Claude now reads AGENTS.md, and these files were only symlinks to the existing AGENTS.md copies.

Test plan

  • Confirm AGENTS.md remains at the repo root, python/, and nodejs/
  • Confirm no remaining in-repo references to CLAUDE.md

Made with Cursor

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.

Tip: disable this comment in your organization's Code Review settings.

@github-actions github-actions Bot added Python Python SDK chore Small non-consequential change. Hidden from changelogs. labels Sep 21, 2026

@lancedb-gatekeeper lancedb-gatekeeper Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Gate recommendation: approve with a non-blocking risk.

All three AGENTS.md sources remain intact, and the upstream default now loads them. The remaining compatibility gap is confined to contributor tooling and is recoverable, but merging this establishes Claude Code 2.1.277+ with the built-in agents-md behavior enabled as the supported automatic-instruction path.

Comment thread CLAUDE.md
@@ -1 +0,0 @@
AGENTS.md No newline at end of file

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing the root fallback also removes automatic project instructions for Claude Code versions before 2.1.277 and for current installations that disable the built-in agents-md mod or select claude-md. The merged upstream implementation makes AGENTS.md fallback the default, but its own README keeps those opt-out modes, and 2.1.278 has a current report of intermittent sessions loading no AGENTS.md. The effect is bounded to contributor-agent guidance and recovery is updating/configuring Claude Code or restoring the symlinks, so I’m treating it as a non-blocking adoption risk. Keeping these pointers through a transition would avoid the gap.

@lancedb-gatekeeper lancedb-gatekeeper Bot added K-approved Latest Gatekeeper recommendation permits acceptance. K-risk Latest Gatekeeper recommendation includes a non-blocking risk. labels Sep 21, 2026
@Xuanwo
Xuanwo merged commit 3be2922 into main Sep 21, 2026
23 checks passed
@Xuanwo
Xuanwo deleted the remove-claude-md-files branch September 21, 2026 12:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Small non-consequential change. Hidden from changelogs. K-approved Latest Gatekeeper recommendation permits acceptance. K-risk Latest Gatekeeper recommendation includes a non-blocking risk. Python Python SDK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants