Skip to content

chore(packaging): ignore local memory files (CLAUDE.local.md, .mb/, mb/)#2962

Merged
marioevz merged 2 commits into
ethereum:forks/amsterdamfrom
leolara:leolara/gitignore-local-memory-files
Jun 8, 2026
Merged

chore(packaging): ignore local memory files (CLAUDE.local.md, .mb/, mb/)#2962
marioevz merged 2 commits into
ethereum:forks/amsterdamfrom
leolara:leolara/gitignore-local-memory-files

Conversation

@leolara

@leolara leolara commented Jun 8, 2026

Copy link
Copy Markdown
Member

🗒️ Description

Add the local-only AI memory and notes files to .gitignore so they do not show up as untracked in working trees:

  • CLAUDE.local.md, the Claude Code local project memory file. The memory docs describe it as the gitignored local instructions file.
  • .mb/ and mb/, local memory-bank directories for per-task documentation and scratch notes.

Also add the same files to the codespell skip list in pyproject.toml. They can exist as symlinks in a working tree, and codespell does not read .gitignore. Without the skip entries, codespell scans them and reports false typos coming from personal notes.

🔗 Related Issues or PRs

N/A.

✅ Checklist

  • All: Ran fast static checks to avoid unnecessary CI fails, see also Code Standards and Enabling Pre-commit Checks:
    just static
  • All: PR title adheres to the repo standard - it will be used as the squash commit message and should start type(scope):.
  • All: Considered updating the online docs in the ./docs/ directory.
  • All: Set appropriate labels for the changes (only maintainers can apply labels).

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

These are local-only AI memory and notes files; ignore them so they do not show up as untracked in working trees.
@leolara

leolara commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

to show how this works, this is my personal CLAUDE.local.md : https://hackmd.io/@leolara/rJmO5eEWze

.mb/ and mb/ are memory bank directory to store information for the AI agent working on a task

@leolara leolara marked this pull request as ready for review June 8, 2026 08:18
Skip CLAUDE.local.md and the .mb/mb memory-bank dirs so codespell does not scan local-only AI memory and notes files (which may be present as symlinks).
@codecov

codecov Bot commented Jun 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.50%. Comparing base (6137bb3) to head (ed9a645).

Additional details and impacted files
@@               Coverage Diff                @@
##           forks/amsterdam    #2962   +/-   ##
================================================
  Coverage            90.50%   90.50%           
================================================
  Files                  535      535           
  Lines                32407    32407           
  Branches              3011     3011           
================================================
  Hits                 29331    29331           
  Misses                2559     2559           
  Partials               517      517           
Flag Coverage Δ
unittests 90.50% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 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.

@leolara leolara requested a review from danceratopz June 8, 2026 11:00

@marioevz marioevz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thanks!

@marioevz marioevz changed the title chore: ignore local memory files (CLAUDE.local.md, .mb/, mb/) chore(packaging): ignore local memory files (CLAUDE.local.md, .mb/, mb/) Jun 8, 2026
@marioevz marioevz merged commit 21c515a into ethereum:forks/amsterdam Jun 8, 2026
24 checks passed
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.

2 participants