fix: check for all native assetIds in isNativeAddress util

[micaelae]

Explanation

This change updates the isNativeAddressutil so that it returnstrue` when the address is a native EVM assetId

References

Related to https://consensyssoftware.atlassian.net/browse/SWAPS-2601

Changelog

Checklist

• I've updated the test suite for new or updated code as appropriate
• I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
• I've communicated my changes to consumers by updating changelogs for packages I've changed, highlighting breaking changes as necessary
• I've prepared draft pull requests for clients and consumer packages to resolve any breaking changes

[cursor]

Bug: Address Matching Bug

The isNativeAddress function incorrectly identifies non-native addresses as native assets. The condition address.includes(reference) || reference.endsWith(address) is overly permissive. For example, an address like 'account60' would match a '60' SLIP44 reference, and '1' would match '501', leading to arbitrary strings being treated as native assets.

packages/bridge-controller/src/utils/bridge.ts#L162-L167

core/packages/bridge-controller/src/utils/bridge.ts

Lines 162 to 167 in 30608f3

	!address ||
	(!isStrictHexString(address) &&
	Object.values(SYMBOL_TO_SLIP44_MAP).some(
	// check if it matches any supported SLIP44 references
	(reference) => address.includes(reference) || reference.endsWith(address),
	));

Fix in Cursor • Fix in Web

---

Was this report helpful? Give feedback by reacting with 👍 or 👎