Bump aztec version to v4.1.0-rc4

[JanKuczma]

Summary by CodeRabbit

• Chores

  • Updated Aztec toolchain and dependencies from v4.0.0-devnet.2-patch.0 to v4.1.0-rc.4 across Docker environments, CI/CD workflows, package managers, and documentation.
  • Updated Solidity remappings to align with new Aztec version.

• Tests

  • Refactored e2e test patterns to align with updated contract simulation return values.
  • Enhanced test utilities with checkpoint-based block readiness validation.

• Refactor

  • Updated Solidity linting pragma configuration.
  • Removed deprecated Solidity payload contract file.

[coderabbitai]

Walkthrough

This PR updates the Aztec framework version from v4.0.0-devnet.2-patch.0 to v4.1.0-rc.4 across Docker images, CI workflows, documentation, and dependency manifests. It also refactors e2e tests and test utilities to properly destructure .simulate() and .deploy() return values, and updates migration bridge logic to use checkpoint-based L1-to-L2 message readiness checks.

Changes

Cohort / File(s)	Summary
Docker & CI Configuration .devcontainer/development/Dockerfile, .devcontainer/mac-arm/Dockerfile, .devcontainer/testing/Dockerfile, .github/workflows/ci.yml, .github/workflows/publish.yml, .mcp.json	Updated AZTEC_VERSION environment variable from v4.0.0-devnet.2-patch.0 to v4.1.0-rc.4 across all container and workflow configurations.
Documentation CLAUDE.md, README.md	Updated Aztec version references and manual CLI installation command from v4.0.0-devnet.2-patch.0 to v4.1.0-rc.4; revised version compatibility guidance.
Noir Package Dependencies noir/aztec-state-migration/Nargo.toml, noir/contracts/migration-{archive-registry,key-registry}/Nargo.toml, noir/test-contracts/{example-app,minimal-benchmark,nft-migration-app,token-migration-app}/v{1,2}/Nargo.toml, noir/tests/Nargo.toml	Updated Aztec dependency Git tags from v4.0.0-devnet.2-patch.0 to v4.1.0-rc.4 across all Noir contract packages.
TypeScript & Solidity Package Dependencies e2e-tests/package.json, ts/aztec-state-migration/package.json, solidity/foundry.toml, solidity/remappings.txt	Updated Aztec package versions from v4.0.0-devnet.2-patch.0 to v4.1.0-rc.4 in npm dependencies and Foundry mappings.
E2E Test Pattern Updates (simulate/deploy destructuring) e2e-tests/migration-{key-registry,mode-a,mode-b,public-mode-b}.test.ts, e2e-tests/nft-migration-mode-{a,b}.test.ts, e2e-tests/token-migration-{mode-a,mode-b,public-mode-b}.test.ts	Refactored contract .simulate() calls to extract results via { result: variable } destructuring instead of direct await assignment; updated .deploy() calls to destructure contract instances from { contract: name }.
Test Utilities & Logic e2e-tests/test-utils.ts, ts/aztec-state-migration/proofs.ts	Enhanced helper functions: updated simulate() and deploy() patterns; modified bridgeBlock to detect and produce additional blocks on old rollup if needed; replaced L1-to-L2 message block comparison with checkpoint-based readiness check in waitForL1ToL2Message.
Solidity Contracts solidity/contracts/Poseidon2Deploy.sol, solidity/contracts/RegisterNewRollupVersionPayload.sol	Updated linting pragma in Poseidon2Deploy.sol from solhint-disable-next-line to forge-lint: disable-start; removed RegisterNewRollupVersionPayload.sol file.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~50 minutes

Possibly related PRs

• Restructure TypeScript SDK as publishable npm package #43: Modifies the same bridge/migration logic (buildArchiveProof, bridgeBlock, waitForL1ToL2Message) and test-utils patterns, indicating coordinated updates to migration infrastructure.
• tests and some mac devcontainer fixes #41: Updates the same e2e test files and test-utils for simulate()/deploy() return value handling, suggesting shared refactoring of SDK API integration.

Poem

🐰 Aztec springs forward to RC.4,
With checkpoints guiding L1→L2 lore,
Simulations yield their .result prize,
Contracts deploy before our eyes,
One version pins the whole warren's skies! 🌟

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 38.89% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Bump aztec version to v4.1.0-rc4' directly and clearly summarizes the main change: updating the Aztec version across the entire codebase from v4.0.0-devnet.2-patch.0 to v4.1.0-rc.4.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch bump-aztec-version

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

Tip

CodeRabbit can enforce grammar and style rules using `languagetool`.

Configure the reviews.tools.languagetool setting to enable/disable rules and categories. Refer to the LanguageTool Community to learn more.

[coderabbitai]

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@e2e-tests/package.json`:
- Around line 8-22: The listed `@aztec/`* dependency entries (e.g.,
"@aztec/accounts", "@aztec/wallets", "@aztec/aztec.js", "@aztec/constants",
"@aztec/foundation", "@aztec/pxe", "@aztec/stdlib", "@aztec/world-state",
"@aztec/aztec-node", "@aztec/cli", "@aztec/ethereum", "@aztec/l1-artifacts",
"@aztec/noir-protocol-circuits-types", "@aztec/protocol-contracts",
"@aztec/telemetry-client") are pinned to v4.1.0-rc.4; change each of these
package versions to the required v4.0.0-devnet.2-patch.0 in package.json and
update the lockfile (run npm/yarn install) so the lockfile matches the new
versions.

In `@solidity/contracts/Poseidon2Deploy.sol`:
- Line 5: The forge-lint directive in Poseidon2Deploy.sol is using regular
comment syntax; replace the line containing "// forge-lint:
disable-start(unused-import)" with a doc-comment variant so Foundry recognizes
it—i.e., change the leading "//" to "///" on the directive line (the line that
currently disables unused-import warnings).

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 2f8de6a5-e3f3-4a7b-a7aa-f2fcc520eee6

📥 Commits

Reviewing files that changed from the base of the PR and between 67b25f9 and b45c05c.

⛔ Files ignored due to path filters (2)

• solidity/soldeer.lock is excluded by !**/*.lock
• yarn.lock is excluded by !**/yarn.lock, !**/*.lock

📒 Files selected for processing (36)

• .devcontainer/development/Dockerfile
• .devcontainer/mac-arm/Dockerfile
• .devcontainer/testing/Dockerfile
• .github/workflows/ci.yml
• .github/workflows/publish.yml
• .mcp.json
• CLAUDE.md
• README.md
• e2e-tests/migration-key-registry.test.ts
• e2e-tests/migration-mode-a.test.ts
• e2e-tests/migration-mode-b.test.ts
• e2e-tests/migration-public-mode-b.test.ts
• e2e-tests/nft-migration-mode-a.test.ts
• e2e-tests/nft-migration-mode-b.test.ts
• e2e-tests/package.json
• e2e-tests/test-utils.ts
• e2e-tests/token-migration-mode-a.test.ts
• e2e-tests/token-migration-mode-b.test.ts
• e2e-tests/token-migration-public-mode-b.test.ts
• noir/aztec-state-migration/Nargo.toml
• noir/contracts/migration-archive-registry/Nargo.toml
• noir/contracts/migration-key-registry/Nargo.toml
• noir/test-contracts/example-app/v1/Nargo.toml
• noir/test-contracts/example-app/v2/Nargo.toml
• noir/test-contracts/minimal-benchmark/Nargo.toml
• noir/test-contracts/nft-migration-app/v1/Nargo.toml
• noir/test-contracts/nft-migration-app/v2/Nargo.toml
• noir/test-contracts/token-migration-app/v1/Nargo.toml
• noir/test-contracts/token-migration-app/v2/Nargo.toml
• noir/tests/Nargo.toml
• solidity/contracts/Poseidon2Deploy.sol
• solidity/contracts/RegisterNewRollupVersionPayload.sol
• solidity/foundry.toml
• solidity/remappings.txt
• ts/aztec-state-migration/package.json
• ts/aztec-state-migration/proofs.ts

💤 Files with no reviewable changes (1)

• solidity/contracts/RegisterNewRollupVersionPayload.sol