Skip to content

Document the gas comparison workflow - #652

Open
duncancmt wants to merge 1 commit into
masterfrom
dcmt/agentsmd
Open

duncancmt wants to merge 1 commit into
masterfrom
dcmt/agentsmd

Conversation

@duncancmt

@duncancmt duncancmt commented Sep 14, 2026

Copy link
Copy Markdown
Collaborator

closes #651

AGENTS.md pointed at npm scripts `snapshot:main` and `diff:main`, which
package.json does not define, so `npm run` fails with "Missing script". The
comparison tool is `npm run compare_gas`, which reads `.forge-snapshots/` and
honors `COMPARE_GIT_SHA` to choose the commit to compare against. Snapshots are
only rewritten by the integration test profile, which needs the RPC URL
variables.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@duncancmt
duncancmt requested a review from e1Ru1o September 14, 2026 10:50
@duncancmt duncancmt self-assigned this Sep 14, 2026
@immunefi-magnus

Copy link
Copy Markdown

🛡️ Immunefi PR Reviews

We noticed that your project isn't set up for automatic code reviews. If you'd like this PR reviewed by the Immunefi team, you can request it manually using the link below:

🔗 Send this PR in for review

Once submitted, we'll take care of assigning a reviewer and follow up here.

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.

AGENTS.md tells agents to run npm run snapshot:main and npm run diff:main, neither of which exists

2 participants