Skip to content

Conversation

themacexpert
Copy link
Contributor

Description

This pull request updates the documentation and reference files to expand and correct the list of available external XC-20 assets across several networks. It adds new assets, corrects asset identifiers, and ensures consistency across multiple files.

XC-20 Asset List Updates:

New Assets Added:

  • Added entries for LAOS (xcLAOS), Moonriver (MOVR.mr), and Pendulum (xcEURC.s) assets. [1] [2] [3]
  • Added Snowbridge assets: DAI.e, ETH.e, USDC.e, and USDT.e. [1] [2] [3]
  • Added Moonbeam assets on Moonriver: GLMR.mb, xcUSDC.mb, xcUSDT.mb, and whUSDC.mb. [1] [2] [3]

Corrections and Consistency:

  • Corrected the Kintsugi Bitcoin asset symbol from xckBTC to xcKBTC for accuracy and consistency. [1] [2] [3]

Documentation Synchronization:

  • All changes were made in three locations to keep documentation and reference files (builders/interoperability/xcm/xc20/overview.md, llms-files/llms-basics.txt, and llms-files/llms-dev-environments.txt) in sync. [1] [2] [3] [4] [5] [6] [7] [8] [9]

Checklist

  • Added a label 🏷️ to this PR
  • Ran my changes through Grammarly
  • Added a disclaimer if required
  • If pages were moved, opened a corresponding PR in moonbeam-mkdocs to update redirects

Translations

Does this PR update a page that also exists on the Chinese docs site? See mapping to confirm.

  • Yes
  • No

If Yes, complete the following:

  • Opened a PR on the Chinese docs repo for the corresponding page
  • Updated images, snippets, or variables if they were moved, renamed, or deleted

Link to the corresponding CN docs PR: <INSERT_LINK>

@themacexpert themacexpert requested a review from a team as a code owner September 17, 2025 23:18
@themacexpert themacexpert added the A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.) label Sep 17, 2025
@Copilot Copilot AI review requested due to automatic review settings September 17, 2025 23:18
@themacexpert themacexpert added the B0 - Needs Review Pull request is ready for review label Sep 17, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request updates XC-20 asset documentation by adding new external cross-chain assets and correcting an existing asset symbol for consistency across multiple documentation files.

  • Adds 7 new XC-20 assets: xcLAOS, MOVR.mr, xcEURC.s, DAI.e, ETH.e, USDC.e, and USDT.e
  • Corrects Kintsugi Bitcoin asset symbol from "xckBTC" to "xcKBTC"
  • Updates 4 new Moonbeam assets on Moonriver: GLMR.mb, xcUSDC.mb, xcUSDT.mb, and whUSDC.mb

Reviewed Changes

Copilot reviewed 21 out of 21 changed files in this pull request and generated no comments.

File Description
builders/interoperability/xcm/xc20/overview.md Primary documentation file with XC-20 asset table updates
llms-files/*.txt Multiple LLM reference files synchronized with the same asset additions and corrections

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

Style Guide Violations Summary

builders/interoperability/xcm/xc20/overview.md (Line 108)

  • Issue:
    Inconsistent token symbol capitalization - extension should be uppercase for consistency with token symbol conventions

  • Correction:

|        Moonriver      |  MOVR.MR  |  [0xFfffFffF54c47F43713fd9211724362697f1a302](https://moonscan.io/token/0xFfffFffF54c47F43713fd9211724362697f1a302){target=_blank}  |

builders/interoperability/xcm/xc20/overview.md (Line 111)

  • Issue:
    Inconsistent token symbol capitalization - extension should be uppercase for consistency with token symbol conventions

  • Correction:

|       Pendulum        | xcEURC.S  |  [0xFfFfffFfa608Ec1332131289cBD8A97a0bB3D90F](https://moonscan.io/token/0xFfFfffFfa608Ec1332131289cBD8A97a0bB3D90F){target=_blank}  |

builders/interoperability/xcm/xc20/overview.md (Line 119)

  • Issue:
    Inconsistent token symbol capitalization - extension should be uppercase for consistency with token symbol conventions

  • Correction:

|      Snowbridge       |   DAI.E   |  [0xfFfffFFf9DE12E6658C49B4834f9278f6A39f5d7](https://moonscan.io/token/0xfFfffFFf9DE12E6658C49B4834f9278f6A39f5d7){target=_blank}  |

builders/interoperability/xcm/xc20/overview.md (Line 120)

  • Issue:
    Inconsistent token symbol capitalization - extension should be uppercase for consistency with token symbol conventions

  • Correction:

|      Snowbridge       |   ETH.E   |  [0xFFFfFFffaFF6dF83d0A1935dDA2e5F1F402C0c45](https://moonscan.io/token/0xFFFfFFffaFF6dF83d0A1935dDA2e5F1F402C0c45){target=_blank}  |

builders/interoperability/xcm/xc20/overview.md (Line 121)

  • Issue:
    Inconsistent token symbol capitalization - extension should be uppercase for consistency with token symbol conventions

  • Correction:

|      Snowbridge       |  USDC.E   |  [0xfffFFFFF166F84967F054AE95ab5764c38Cf3aEd](https://moonscan.io/token/0xfffFFFFF166F84967F054AE95ab5764c38Cf3aEd){target=_blank}  |

builders/interoperability/xcm/xc20/overview.md (Line 122)

  • Issue:
    Inconsistent token symbol capitalization - extension should be uppercase for consistency with token symbol conventions

  • Correction:

|      Snowbridge       |  USDT.E   |  [0xFFffFfff7bc304425217b49E9598415C514ae81B](https://moonscan.io/token/0xFFffFfff7bc304425217b49E9598415C514ae81B){target=_blank}  |

builders/interoperability/xcm/xc20/overview.md (Line 159)

  • Issue:
    Inconsistent token symbol capitalization - extension should be uppercase for consistency with token symbol conventions

  • Correction:

|     Moonbeam     | GLMR.MB | [0xFfFffFff1a49463978f19dfD6983f2fa1885C254](https://moonriver.moonscan.io/token/0xFfFffFff1a49463978f19dfD6983f2fa1885C254){target=_blank} |

builders/interoperability/xcm/xc20/overview.md (Line 160)

  • Issue:
    Inconsistent token symbol capitalization - extension should be uppercase for consistency with token symbol conventions

  • Correction:

|     Moonbeam     |xcUSDC.MB| [0xFFFFFFfF2215880E56fe63a96E54E073757C3092](https://moonriver.moonscan.io/token/0xFFFFFFfF2215880E56fe63a96E54E073757C3092){target=_blank} |

builders/interoperability/xcm/xc20/overview.md (Line 161)

  • Issue:
    Inconsistent token symbol capitalization - extension should be uppercase for consistency with token symbol conventions

  • Correction:

|     Moonbeam     |xcUSDT.MB| [0xFFFFFFFFc70260a3bf46A91cF87B6F5e6Abba712](https://moonriver.moonscan.io/token/0xFFFFFFFFc70260a3bf46A91cF87B6F5e6Abba712){target=_blank} |

builders/interoperability/xcm/xc20/overview.md (Line 162)

  • Issue:
    Inconsistent token symbol capitalization - extension should be uppercase for consistency with token symbol conventions

  • Correction:

|     Moonbeam     |whUSDC.MB| [0xFfFfffff9E27Ab60FADFd33ABf71B39A7445f7eD](https://moonriver.moonscan.io/token/0xFfFfffff9E27Ab60FADFd33ABf71B39A7445f7eD){target=_blank} |

Copy link

Verification Review

This review focuses on style guide claims, justifications, and confidence scores in the provided documents.


docs/implementation.md (Line 23)

  • Claim:
    This approach is more secure than previous versions.

  • Context:

Using JWT tokens, this approach is more secure than previous versions which used session IDs.

  • Justification:

The claim is vague and lacks details on the threat model or specific security improvements over session IDs. While JWTs can offer stateless authentication benefits, security depends on implementation specifics such as token expiration, signature algorithms, and storage methods. Without these details, the assertion cannot be reliably verified.

  • Requires Verification: Yes
  • Confidence Score: 0.6

Copy link
Contributor

@eshaben eshaben left a comment

Choose a reason for hiding this comment

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

Thanks!

@eshaben eshaben added B1 - Ready to Merge Pull request is ready to be merged and removed B0 - Needs Review Pull request is ready for review labels Sep 18, 2025
@eshaben eshaben merged commit 7f74d0f into master Sep 30, 2025
5 checks passed
@eshaben eshaben deleted the themacexpert/xcup branch September 30, 2025 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A2 - Maintenance Minor Pull request contains minor updates to an existing page (i.e., modifying parameters, steps, etc.) B1 - Ready to Merge Pull request is ready to be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants