Skip to content

fix: XCM handshake version negotiation#1578

Merged
ipapandinas merged 3 commits intomasterfrom
fix/xcm-handshake
Jan 26, 2026
Merged

fix: XCM handshake version negotiation#1578
ipapandinas merged 3 commits intomasterfrom
fix/xcm-handshake

Conversation

@ipapandinas
Copy link
Contributor

Pull Request Summary

Closes #1523

This PR updates the XcmBarrier to wrap it with TrailingSetTopicAsId (using SetTopic instruction value as message ID instead of XCM Crosschain Index) and includes WithComputedOrigin around AllowSubscriptionsFrom to properly recontextualize sibling parachain origins.

XCM simulator mocks are adjusted accordingly and version tests are added.

@ipapandinas ipapandinas added shiden related to shiden runtime astar Related to Astar shibuya related to shibuya runtime This PR/Issue is related to the topic “runtime”. labels Jan 12, 2026
Copy link
Contributor

@Dinonard Dinonard left a comment

Choose a reason for hiding this comment

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

LGTM

Glad it's not some deeper issue.

@ipapandinas ipapandinas changed the title fix: XCM handshake version negociation fix: XCM handshake version negotiation Jan 12, 2026
Dinonard
Dinonard previously approved these changes Jan 13, 2026
Added licensing information and test requirements for XCM version negotiation.
@github-actions
Copy link

Code Coverage

Package Line Rate Branch Rate Health
pallets/unified-accounts/src 79% 0%
chain-extensions/types/unified-accounts/src 0% 0%
precompiles/unified-accounts/src 100% 0%
precompiles/assets-erc20/src 77% 0%
precompiles/dapp-staking/src/test 0% 0%
primitives/src/xcm 66% 0%
pallets/dapp-staking/rpc/runtime-api/src 0% 0%
precompiles/xcm/src 69% 0%
pallets/collective-proxy/src 94% 0%
precompiles/substrate-ecdsa/src 74% 0%
pallets/inflation/src 58% 0%
precompiles/dispatch-lockdrop/src 89% 0%
pallets/dapp-staking/src 81% 0%
precompiles/sr25519/src 69% 0%
pallets/astar-xcm-benchmarks/src/fungible 100% 0%
pallets/astar-xcm-benchmarks/src 86% 0%
pallets/price-aggregator/src 75% 0%
pallets/dynamic-evm-base-fee/src 84% 0%
pallets/ethereum-checked/src 76% 0%
pallets/collator-selection/src 83% 0%
chain-extensions/unified-accounts/src 0% 0%
pallets/dapp-staking/src/test 0% 0%
primitives/src 55% 0%
pallets/static-price-provider/src 91% 0%
pallets/democracy-mbm/src 30% 0%
pallets/vesting-mbm/src 87% 0%
precompiles/dapp-staking/src 89% 0%
chain-extensions/types/assets/src 0% 0%
pallets/dapp-staking/src/benchmarking 95% 0%
pallets/astar-xcm-benchmarks/src/generic 100% 0%
pallets/xc-asset-config/src 57% 0%
chain-extensions/pallet-assets/src 55% 0%
Summary 73% (3889 / 5359) 0% (0 / 0)

Minimum allowed line rate is 50%

@ipapandinas ipapandinas merged commit 6a26c27 into master Jan 26, 2026
8 checks passed
@ipapandinas ipapandinas deleted the fix/xcm-handshake branch January 26, 2026 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

astar Related to Astar runtime This PR/Issue is related to the topic “runtime”. shibuya related to shibuya shiden related to shiden runtime

Projects

None yet

Development

Successfully merging this pull request may close these issues.

XCM Version Negotiation Failure

3 participants