Skip to content

feat: Add Avalanche as supported chain#3290

Draft
dijanin-brat wants to merge 5 commits into
masterfrom
dijanin-brat/supportAvalanche
Draft

feat: Add Avalanche as supported chain#3290
dijanin-brat wants to merge 5 commits into
masterfrom
dijanin-brat/supportAvalanche

Conversation

@dijanin-brat
Copy link
Copy Markdown
Contributor

We don't want to merge this until we bump contracts repo

@dijanin-brat dijanin-brat self-assigned this Apr 24, 2026
@dijanin-brat dijanin-brat added the do not merge Don't merge until label is removed label Apr 24, 2026
pxrl
pxrl previously approved these changes Apr 27, 2026
Comment thread src/adapter/BaseChainAdapter.ts Outdated
Comment on lines +499 to +500
const { BSC, HYPEREVM, MAINNET, PLASMA, MONAD, AVALANCHE } = CHAIN_IDs;
const nativeTokenChains = [BSC, HYPEREVM, MAINNET, PLASMA, MONAD, AVALANCHE];
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

These seem to be sorted:

Suggested change
const { BSC, HYPEREVM, MAINNET, PLASMA, MONAD, AVALANCHE } = CHAIN_IDs;
const nativeTokenChains = [BSC, HYPEREVM, MAINNET, PLASMA, MONAD, AVALANCHE];
const { AVALANCHE, BSC, HYPEREVM, MAINNET, PLASMA, MONAD } = CHAIN_IDs;
const nativeTokenChains = [AVALANCHE, BSC, HYPEREVM, MAINNET, PLASMA, MONAD];

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

they are not, but i agree that they should be 👍
1ea1d88

Comment thread src/common/Constants.ts Outdated

// Chain IDs using the Succinct/Helios SP1 messaging bridge.
export const UNIVERSAL_CHAINS = [
CHAIN_IDs.AVALANCHE, // @TODO: Should be universal chain
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Yeah, Universal. We could in theory support it via CCTP but we likely lack some implementation for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Don't merge until label is removed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants