Skip to content

add xSLP to registry - #848

Merged
simcheolhwan merged 2 commits into
initia-labs:mainfrom
0xRadian:add/xSLP
Jul 26, 2026
Merged

simcheolhwan merged 2 commits into
initia-labs:mainfrom
0xRadian:add/xSLP

Conversation

@0xRadian

@0xRadian 0xRadian commented Jul 24, 2026 •

Copy link
Copy Markdown
Contributor

adds the liquid wrapper of the iUSD SLP position to the registry on Strat and on mainnet

Summary by CodeRabbit

  • New Features
    • Added the Cabal xSLP asset to the Initia, Strat, and Echelon networks.
    • Updated token details including denomination mapping, IBC routing metadata, and logo/branding assets for xSLP.

@coderabbitai

coderabbitai Bot commented Jul 24, 2026 •

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: ae443201-cb55-4f75-803f-f4c3ed084d79

📥 Commits

Reviewing files that changed from the base of the PR and between 2f85ade and 5f70637.

📒 Files selected for processing (1)
  • mainnets/echelon/assetlist.json

Walkthrough

Adds Cabal xSLP entries to the Initia, Strat, and Echelon asset lists, including denomination mappings, token identifiers, IBC trace metadata, and logo references.

Changes

Cabal xSLP registration

Layer / File(s) Summary
Cross-chain xSLP asset metadata
mainnets/initia/assetlist.json, mainnets/strat/assetlist.json, mainnets/echelon/assetlist.json
Adds Cabal xSLP definitions with denomination units, token metadata, IBC trace information, and xSLP.png image references.

Estimated code review effort: 2 (Simple) | ~10 minutes

Poem

I’m a rabbit with a registry key,
Hopping xSLP across the chain sea.
Initia, Strat, Echelon align,
Logos glow in a tidy line.
Three lists now know where tokens fly.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: adding xSLP to the registry.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@mainnets/initia/assetlist.json`:
- Around line 1315-1322: Fix the broken shared xSLP PNG reference by adding the
missing image at images/xSLP.png or updating the URLs to an existing PNG. Apply
the same correction in mainnets/initia/assetlist.json lines 1315-1322 and
mainnets/strat/assetlist.json lines 104-111 so both xSLP registrations use the
consistent valid image reference.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 30a10193-8b86-4966-9b35-492fc5b75c49

📥 Commits

Reviewing files that changed from the base of the PR and between 0e6018f and 2f85ade.

⛔ Files ignored due to path filters (1)
  • images/xSLP.png is excluded by !**/*.png
📒 Files selected for processing (2)
  • mainnets/initia/assetlist.json
  • mainnets/strat/assetlist.json

Comment thread mainnets/initia/assetlist.json
@simcheolhwan
simcheolhwan merged commit 0555e57 into initia-labs:main Jul 26, 2026
7 checks passed
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.

2 participants