Skip to content

update CLAUDE.md for the Foundry toolchain - #2980

Merged
sparrowDom merged 1 commit into
masterfrom
sparrowDom/claude-md-updates
Aug 19, 2026
Merged

update CLAUDE.md for the Foundry toolchain#2980
sparrowDom merged 1 commit into
masterfrom
sparrowDom/claude-md-updates

Conversation

@sparrowDom

Copy link
Copy Markdown
Member

Rewrite the Hardhat-era sections (setup, build/test/deploy commands, test organization, deployment scripts, roles, verification) around the Makefile, foundry.toml, tests/ and scripts/deploy/. Hardhat is documented as remaining only for the ops task CLI; the JS test suite and hardhat-deploy scripts are marked frozen. Fixes stale vault names and the VaultCore/VaultAdmin inheritance direction, and folds in the core-contract-stability design principle from the untracked contracts/CLAUDE.md.

Code Change Checklist

To be completed before internal review begins:

  • The contract code is complete
  • Executable deployment file
  • Fork tests that test after the deployment file runs
  • Unit tests *if needed
  • The owner has done a full checklist review of the code + tests

Internal review:

  • Two approvals by internal reviewers

Rewrite the Hardhat-era sections (setup, build/test/deploy commands, test
organization, deployment scripts, roles, verification) around the Makefile,
foundry.toml, tests/ and scripts/deploy/. Hardhat is documented as remaining
only for the ops task CLI; the JS test suite and hardhat-deploy scripts are
marked frozen. Fixes stale vault names and the VaultCore/VaultAdmin
inheritance direction, and folds in the core-contract-stability design
principle from the untracked contracts/CLAUDE.md.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@sparrowDom
sparrowDom merged commit 2da3a20 into master Aug 19, 2026
12 checks passed
@sparrowDom
sparrowDom deleted the sparrowDom/claude-md-updates branch August 19, 2026 13:02
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