Skip to content

Update AGENTS.md solc versions and EVM hardfork to match source#512

Open
duncancmt wants to merge 2 commits intomasterfrom
dcmt/agents
Open

Update AGENTS.md solc versions and EVM hardfork to match source#512
duncancmt wants to merge 2 commits intomasterfrom
dcmt/agents

Conversation

@duncancmt
Copy link
Collaborator

  • EVM version: cancun → osaka (matching foundry.toml)
  • Add auto_detect_solc = true, remove pinned solc_version
  • Split main contracts into core (^0.8.25) and chains (=0.8.33)
  • Add AllowanceHolder + Deployer row (CI-pinned =0.8.25)
  • Fix EulerSwapBUSL pragma to ^0.8.24
  • Update build commands to match CI (skip AllowanceHolder/Deployer)

- EVM version: cancun → osaka (matching foundry.toml)
- Add auto_detect_solc = true, remove pinned solc_version
- Split main contracts into core (^0.8.25) and chains (=0.8.33)
- Add AllowanceHolder + Deployer row (CI-pinned =0.8.25)
- Fix EulerSwapBUSL pragma to ^0.8.24
- Update build commands to match CI (skip AllowanceHolder/Deployer)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@immunefi-magnus
Copy link

🛡️ 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.

@duncancmt duncancmt requested review from e1Ru1o and jparklev February 27, 2026 22:40
@duncancmt duncancmt self-assigned this Feb 27, 2026
Copy link
Contributor

@jparklev jparklev left a comment

Choose a reason for hiding this comment

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

LGTM

| Main contracts (`src/`) | 0.8.25 | cancun | 2,000 |
| UniswapV4 (`lib/v4-core/`) | 0.8.26 | cancun | 2,000 |
| Core libraries (`src/core/`, `src/*.sol`) | `^0.8.25` (auto-detected) | osaka | 2,000 |
| Chain contracts (`src/chains/*/`) | `=0.8.33` | osaka | 2,000 |
Copy link
Contributor

@jparklev jparklev Feb 28, 2026

Choose a reason for hiding this comment

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

A few of the files in chains/ aren't pinned (eg Blast/Common.sol is ^0.8.25), in case that feels like a helpful to mention

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I think that was an oversight. Fixed.

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