feat: add Optimism provider

[Wodann]

A draft PR for adding Optimism support in EDR using the @ignored/edr-optimism package, intended for the alpha.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
hardhat	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 22, 2024 8:22am

[changeset-bot]

⚠️ No Changeset found

Latest commit: b318a4b

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

[schaable]

@Wodann I've merged this PR into the feat/edr-provider branch, so we can close this. There are a couple of enums, like StackTraceEntryType and MineOrdering, that still need to be converted to constants. I've casted the values for now to get things working, but we should revisit this in the future to clean it up.

[Wodann]

@Wodann I've merged this PR into the feat/edr-provider branch, so we can close this. There are a couple of enums, like StackTraceEntryType and MineOrdering, that still need to be converted to constants. I've casted the values for now to get things working, but we should revisit this in the future to clean it up.

Are enums prohibited?

Could you provide an example of what you'd like to be generated instead of an enum?

[schaable]

@Wodann I've merged this PR into the feat/edr-provider branch, so we can close this. There are a couple of enums, like StackTraceEntryType and MineOrdering, that still need to be converted to constants. I've casted the values for now to get things working, but we should revisit this in the future to clean it up.

Are enums prohibited?

Could you provide an example of what you'd like to be generated instead of an enum?

The issue is that we have isolatedModules set to true to ensure compatibility with single-file transpilation tools like tsx. With isolatedModules, const enums can’t be inlined, since this setting assumes each file compiles independently rather than together.

Instead of using enums, it would be better to generate constants, similar to what was done for SpecId.

[Wodann]

The issue is that we have isolatedModules set to true to ensure compatibility with single-file transpilation tools like tsx. With isolatedModules, const enums can’t be inlined, since this setting assumes each file compiles independently rather than together.

Instead of using enums, it would be better to generate constants, similar to what was done for SpecId.

Thanks for explaining! I'll add a task to do that and add you as a reviewer.

[alcuadrado]

Thanks a ton for this PR, @Wodann :) @schaable incorporated these changes into #5811.

I'm closing this PR to clean up the repo.

[Wodann]

Issue added