Skip to content

Conversation

@ecPablo
Copy link
Contributor

@ecPablo ecPablo commented Jan 9, 2026

This pull request introduces improved error handling and validation for the proposal analysis context and EVM registry initialization across the MCMSv2 CLI and EVM analyzer. These changes help prevent misconfiguration and provide clearer guidance to developers when required dependencies are missing.

Validation and error messaging improvements:

  • Added a check in buildMCMSv2AnalyzeProposalCmd to ensure proposalCtxProvider is not nil, returning a descriptive error if missing.
  • Enhanced error handling in newCfgv2 to verify that the proposal context returned by the provider is not nil, with a clear error message for initialization issues.

EVM registry initialization guidance:

  • Improved the error message in AnalyzeEVMTransaction to instruct developers to provide an EVM registry via the ProposalContextProvider with the WithEVMABIMappings() option if it is missing.

@changeset-bot
Copy link

changeset-bot bot commented Jan 9, 2026

⚠️ No Changeset found

Latest commit: 1ed00f7

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@cl-sonarqube-production
Copy link

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.

1 participant