chore!: changed execution reverted code from 11 to 3

[LesnyRumcajs]

Summary of changes

Changes introduced in this pull request:

• porting fix(eth): use error code 3 for EExecutionReverted filecoin-project/lotus#13467, don't merge until the reference is merged as well.

Reference issue to close (if applicable)

Closes

Other information and links

Change checklist

• I have performed a self-review of my own code,
• I have made corresponding changes to the documentation. All new code adheres to the team's documentation standards,
• I have added tests that prove my fix is effective or that my feature works (if possible),
• I have made sure the CHANGELOG is up-to-date. All user-facing changes should be reflected in this document.

Summary by CodeRabbit

• Breaking Changes

  • Error code returned for execution-reverted responses changed from 11 to 3 — integrations relying on the previous code should be updated.

• Documentation

  • Changelog and related docs updated to reflect the new error code and its ecosystem impact.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

Changed the execution-reverted RPC error code constant from 11 to 3 and updated the unreleased CHANGELOG entry to document this breaking change. Two files modified: src/rpc/methods/eth/errors.rs and CHANGELOG.md.

Changes

Cohort / File(s)	Summary
Error Code Update src/rpc/methods/eth/errors.rs	Updated pub const EXECUTION_REVERTED_CODE: i32 from 11 to 3; adjusted inline documentation to reflect the new code value.
Changelog CHANGELOG.md	Added a breaking changelog entry documenting the execution-reverted error code change from 11 to 3 in the unreleased section.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested labels

RPC

Suggested reviewers

• hanabi1224
• akaladarshi

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title directly and accurately summarizes the main change: updating the execution reverted error code from 11 to 3, which is the primary modification across both CHANGELOG.md and src/rpc/methods/eth/errors.rs.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

---

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

[hanabi1224]

Upstream PR has been merged. Should we merge this or hold until calbinet F3 reset releases are published?

[LesnyRumcajs]

I think we can merge it already, we'll have eventual consistency.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.73%. Comparing base (b2c3068) to head (4fbb725).
⚠️ Report is 4 commits behind head on main.

Additional details and impacted files

Files with missing lines	Coverage Δ
src/rpc/methods/eth/errors.rs	0.00% <ø> (ø)

... and 3 files with indirect coverage changes

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b2c3068...4fbb725. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[sudo-shashank]

@LesnyRumcajs changelog needs a fix accidentally you removed Forest v0.31.1 "Quadrantids" section during merge

[LesnyRumcajs]

@LesnyRumcajs changelog needs a fix accidentally you removed Forest v0.31.1 "Quadrantids" section during merge

Good catch! I should stick to vim and in-terminal rebasing and stop touching those shiny github buttons...