Skip to content

chore(claude): add Claude Code support for maintainers#312

Merged
mwbrooks merged 5 commits intomainfrom
mwbrooks-claude
Jan 28, 2026
Merged

chore(claude): add Claude Code support for maintainers#312
mwbrooks merged 5 commits intomainfrom
mwbrooks-claude

Conversation

@mwbrooks
Copy link
Member

@mwbrooks mwbrooks commented Jan 28, 2026

Summary

This pull request adds a basic, shared Claude Code configuration for consistent AI-assisted development across all maintainers. 🤖

File structure:

  .claude/                                                                                                                                           
  ├── CLAUDE.md        # Project context, conventions, and coding guidelines
  └── settings.json    # Shared permissions for common commands (go test, make build, etc.)

Extra notes:

  • .claude/CLAUDE.md was generated with /init
    • I've confirmed (with Claude) that it can live inside of .claude/ instead of the project root
    • Claude discourages separating the file into .claude/rules/*.md because our file is still small and organized
  • .claude/settings.json auto-approves a few common bash commands
    • I tried to keep it to common, less destructive commands
  • .gitignore ignores local Claude Code files

✨ Hoping this is a place that we continue to add more clarity and direction to Claude around our desired coding style and goals!

Requirements

@mwbrooks mwbrooks added this to the Next Release milestone Jan 28, 2026
@mwbrooks mwbrooks self-assigned this Jan 28, 2026
@mwbrooks mwbrooks requested a review from a team as a code owner January 28, 2026 17:10
@mwbrooks mwbrooks added code health M-T: Test improvements and anything that improves code health semver:patch Use on pull requests to describe the release version increment labels Jan 28, 2026
mwbrooks
mwbrooks commented Jan 28, 2026
View reviewed changes
Copy link
Member Author

@mwbrooks mwbrooks left a comment

Choose a reason for hiding this comment

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

Comments for the kind human reviewers

.claude/CLAUDE.md
@@ -0,0 +1,242 @@
# CLAUDE.md
Copy link
Member Author

@mwbrooks mwbrooks Jan 28, 2026

Choose a reason for hiding this comment

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

note: Auto-generated with /init

@codecov
Copy link

codecov bot commented Jan 28, 2026 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.88%. Comparing base (114e011) to head (44973f5).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #312      +/-   ##
==========================================
- Coverage   64.89%   64.88%   -0.02%     
==========================================
  Files         212      212              
  Lines       17538    17538              
==========================================
- Hits        11382    11380       -2     
- Misses       5089     5092       +3     
+ Partials     1067     1066       -1

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

zimeg
zimeg approved these changes Jan 28, 2026
View reviewed changes
Copy link
Member

@zimeg zimeg left a comment

Choose a reason for hiding this comment

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

@mwbrooks LGTM! Such a welcomed change too! I look forward to enhancing available tooling in follow up PRs but this is scoped well to start IMO 👾 ✨

I'm finding also that a nested CLAUDE.md finds the right context. Clever setups:

/context
└ ~/.claude/CLAUDE.md: 101 tokens
└ .claude/CLAUDE.md: 2.3k tokens

mwbrooks and others added 4 commits January 28, 2026 12:10
@mwbrooks
chore(claude): prefer map pattern table tests
ea4a490
@mwbrooks
chore(claude): remove auto approving go and golangci-lint tests
a42bca5
@mwbrooks @zimeg
Update .claude/CLAUDE.md
e537819 
Co-authored-by: Eden Zimbelman <eden.zimbelman@salesforce.com>
@mwbrooks @zimeg
Update .claude/CLAUDE.md
44973f5 
Co-authored-by: Eden Zimbelman <eden.zimbelman@salesforce.com>
@mwbrooks
Copy link
Member Author

mwbrooks commented Jan 28, 2026

Thanks for the collab on ideas @zimeg! 👥 I'm going to merge this but let's continue to refine the Claude configuration files to work well with this codebase ✨

@mwbrooks mwbrooks merged commit 6b64a28 into main Jan 28, 2026
8 checks passed
@mwbrooks mwbrooks deleted the mwbrooks-claude branch January 28, 2026 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@zimeg zimeg zimeg approved these changes

Assignees

@mwbrooks mwbrooks

Labels

code health M-T: Test improvements and anything that improves code health semver:patch Use on pull requests to describe the release version increment

Projects

None yet

Milestone

v3.12.0

Development

Successfully merging this pull request may close these issues.

2 participants

@mwbrooks @zimeg

Comments