chores: remove suffix for the main assets#268
Conversation
Totalspud
left a comment
There was a problem hiding this comment.
On echelon-1, removing the suffix, would then leave you with two separate INIT tokens (one OP bridged, one IBC, bridged) with different denoms.
On embrmainnet-1 — INIT, USDC and iUSD each have multiple versions
with different decimals (ERC20 = 18, IBC = 6). Without the suffix creates ambiguity and routing issues.
The other changes look fine
|
I only removed the suffix for one token per asset. Each asset can still have multiple tokens depending on the bridge method or decimals, but only the official one uses |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Note
Medium Risk
Mostly metadata changes, but renaming
symbol/recommended_symbolacross multiple chains can break downstream integrations that rely on the previous suffix-based identifiers.Overview
Standardizes asset registry entries across multiple chains by removing
.erc20/.op/.ibcsuffixes fromrecommended_symbolfor primary assets, while updating severalsymbolfields (notably IBC variants likeINIT.ibc,USDC.ibc,iUSD.ibc) to keep representations distinguishable.Also removes the deprecated
chains/embr-1/chain.jsonentry, adds the missingiUSDEVM asset torave-1, and renames Yominet’s wrapped ONYX entry toONYX.wrapped(including its display name andrecommended_symbol).Written by Cursor Bugbot for commit 7ae13d6. This will update automatically on new commits. Configure here.