Skip to content

Add root CLAUDE.md importing local AGENTS.md - #522

Merged
rickdijk merged 3 commits into
developfrom
chore/add-claude-md
Sep 14, 2026
Merged

rickdijk merged 3 commits into
developfrom
chore/add-claude-md

Conversation

@rickdijk

@rickdijk rickdijk commented Sep 14, 2026 •

Copy link
Copy Markdown
Contributor

✨ What kind of change does this PR introduce? (Bug fix, feature, docs update...)

Docs / tooling. Adds a root CLAUDE.md for contributors using Claude Code.

⤵️ What is the current behavior?

Agent guidance lives in AGENTS.md. Claude Code does not read AGENTS.md — the documentation is explicit that it reads CLAUDE.md, and there is no setting to add other context filenames — so every Claude Code session in this repo starts with no project guidance at all, silently and with no warning.

🆕 What is the new behavior (if this is a feature change)?

CLAUDE.md is added at the repo root. It does two things:

  • Imports AGENTS.md via @AGENTS.md, so all tool-neutral guidance stays in one place and is not duplicated.
  • Holds the Claude-specific bits that don't belong in a tool-neutral file: driving Flutter through FVM, preferring the file tools over shell equivalents, which review slash commands to run for lib/src/web/ and lib/src/storage/ changes, and the commit/PR attribution lines.

Note on sequencing: at the time this PR was opened, AGENTS.md was git-ignored, which made the @AGENTS.md import a no-op in a fresh clone and in CI. That is fixed in the follow-up PR, which un-ignores and commits AGENTS.md — merge that one for the import to actually resolve for anyone but the local checkout.

💥 Does this PR introduce a breaking change?

No. Documentation only — no package code is touched.

🐛 Recommendations for testing

Nothing to test. flutter_cache_manager/CHANGELOG.md gains one ## [Unreleased] entry.

📝 Links to relevant issues/docs

🤔 Checklist before submitting

  • All projects build (no code changes)
  • Follows style guide lines (code style guide)
  • Relevant documentation was updated
  • Rebased onto current develop

🤖 Generated with Claude Code

rickdijk and others added 2 commits September 14, 2026 15:35
Claude Code reads CLAUDE.md, not AGENTS.md, so import the (git-ignored,
local) AGENTS.md from it and keep only Claude-specific notes here: FVM
usage, file-tool preferences, review slash commands, and commit/PR
attribution.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
AGENTS.md was git-ignored (6292965), so "the best documentation anywhere
in this scope" was absent from every clone and from CI, and CLAUDE.md's
@AGENTS.md import resolved to nothing for anyone but the local checkout.
Un-ignore it and commit it.

Trim 205 lines to 188, splitting the content by what it does. Technical
reference stays: architecture, directory map, where-to-change map, fork
workflow, commands, testing expectations, platform notes, PR checklist.
Seven lines leave, each with a destination rather than a deletion:

* the "do not force-push / do not bump versions / do not commit unless
  asked" rules become PreToolUse hooks (FPL-8), where they block instead
  of being advice an agent may or may not honour
* the Notion link and internal Baseflow strategy move to the private
  maintenance repo (FPL-12)

Also drop CLAUDE.md's note that the import is a no-op, which this change
makes untrue.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Publish AGENTS.md and trim it to technical content
@rickdijk
rickdijk merged commit 3e0864d into develop Sep 14, 2026
8 checks passed
@rickdijk
rickdijk deleted the chore/add-claude-md branch September 14, 2026 23:49
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