Post-upgrade full review: Terra Classic bidirectional swaps (operator, canceler, contracts, bridge) before economic tokens #111
Labels
No labels
agent:implement
agent:ready
backend
bug
cannot-reproduce
confirmed
desktop
docs
documentation
duplicate
enhancement
feature
frontend
good first issue
help wanted
high-risk
in-review
invalid
mobile
needs-triage
P0-critical
P1-high
P2-medium
P3-low
qa
QA
question
ready
report
responsive
security
security-escalate
smart-contract
solana
tablet
test-pass
ux
wallet-issue
wallet:keplr
wallet:metamask
wallet:station
wallet:walletconnect
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
code/cl8y-bridge-monorepo#111
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Summary
The bridge is currently restricted to noneconomic (test) tokens only. After the bridge upgrade and the Terra Classic chain upgrade, we need a full end-to-end verification that the operator, canceler, smart contracts (EVM, CosmWasm, Solana), and bridge services behave correctly for bidirectional swaps involving Terra Classic to every other supported chain before re-enabling economic tokens.
Requested action: @brouie — please conduct a full review of test evidence, configuration, and on-chain/off-chain behavior against this checklist, and sign off (or file blocking items) before economic tokens are turned back on.
Terra Classic chain upgrade (today)
Public announcement context (validators / network upgrade):
Reference: X post (nuenghandsome)
Implications for testing: RPC/LCD endpoints may be unstable during the upgrade; schedule post-upgrade smoke tests and full matrix tests once
columbus-5is stable and finality behaves as expected.Pre-flight (post chain upgrade)
columbus-5and Tendermint / SDK behavior matches expectations (e.g. tx indexing, events, gas).CosmWasm (Terra Classic) contracts
EVM contracts (BSC, opBNB)
getCancelerCount() > 0on EVM bridges where cancelers are required (watchtower gap if zero).bytes4chain id for Terra Classic) and paired tokens are correct after upgrade.Solana program
Operator (relayer)
Canceler (watchtower)
Bidirectional swap matrix (Terra Classic)
Complete both directions for each pair (minimum noneconomic tokens; expand if economic dry-run is staged separately):
For each cell:
Economic tokens (reenable gate)
Automated tests & evidence
make test-operator/make test-canceler(or CI equivalent) pass on the release branch.make e2e-testsubsets: Terra↔EVM, canceler, etc.).Sign-off
Issue created to coordinate post–Terra Classic v4.0.0 / Cosmos SDK 0.53 upgrade validation before restoring economic-token bridging.
assigned to @Brouie
mentioned in issue #96
@PlasticDigits Post-upgrade verification progress on build v0.1.82 -
f227b6f, 4/20 mainnet.Pre-flight (post Terra chain upgrade)
EVM contracts
Solana program
Operator (relayer)
Canceler (watchtower)
Bidirectional swap matrix (Terra Classic)
Terra->Solana tested today. Solana->Terra outbound blocked by lack of SPL faucet on mainnet.
Automated tests
Sign-off
Still open: Solana->Terra outbound path (faucet blocker), operator RPC degradation staged test. Everything else covered by live mainnet activity this week.
Recommend NOT yet enabling economic tokens until:
Technical path matrix is healthy. UX polish items above are the remaining gates from my side.
For solana -> terra outbound path, we are NOT deploying faucets to mainnet. Instead tokens should be transferred from another chain to solana first, then test outbound transfers.
For RPC degradation, I have checked logs for operator and canceler and confirmed both are working in production, so those items are passing.
Agreed - once 101, 106, 108 are resolved I will reenable economic tokens.
Noted on all three points:
Solana -> Terra outbound path: Understood -- no mainnet faucet. Verification flow becomes: bridge tokens INTO Solana first (from BSC or Terra), then test outbound from Solana back to Terra. I'll plan for this when validating the Solana outbound leg.
RPC degradation: Operator and canceler production logs confirm working, so those checklist items pass. Removing from my watch list.
Economic tokens: Acknowledge plan -- re-enable once #101, #106, #108 all resolved. With #101 and #108 now verified fixed on v0.1.82 -
2a127e6(and #106 already closed), the dependency is cleared on the QA side.Updated checklist state:
Issue stays open as sign-off tracker until economic tokens are re-enabled and Solana outbound flow validated.
@PlasticDigits post-upgrade re-verification complete on v0.1.82 -
2a127e6.Pre-flight (Terra Classic v4.0.0 / Cosmos SDK 0.53)
columbus-5confirmedterra-classic-lcd.publicnode.com,lcd.terra-classic.hexxagon.io,terra-classic-fcd.publicnode.com)lcd.terra-classic.hexxagon.devreturns chain_idrebel-2(testnet), notcolumbus-5— flagged here for dev awareness, consider folding into #109terra18m02l2f43c2dagqnz3kfccpgz9pzzz5hk9l5mh5wvr6dcvv47zfqdfs7laqueryable post-upgrade, labelcl8y-bridge-v2, code_id 10971, created block 27,401,342 (pre-upgrade, survived intact)terra1xsecn4snv94ezcez0z3vq8an9j4h4kxxcydp8l, paused=false, min_signatures=1, fee_bps=30, fee_collectorterra1q7txczaxuvy923k4km9ya062dryk6mjwd6tmzmEVM + Solana registration (still valid from 4/8 and 4/20)
isChainRegistered(0x00000001)= true (Terra) — re-verified today post-upgradeCJGDL5pHawva98BGawAFkGwMhZLKr4RuvqfC7iUtYkQA, testb4dzQ7U7mwtsqZdPZGvwPi2PZbhiyZpj6KM5SinpNpgQJ, tdec3Af1LkQo2g9xasGhCcofVnRPWJxPjSyBHnMiFdzVNRm5Bidirectional swap matrix (post-upgrade, testa 1.99/1.994, 4/21)
0x4f2c4a6098c2f6dcd5f75d439eb467719c342ed8ad62b3786f0efdd6fb674c4c0xb0e1cb50a7011889085935ce53a136bdad65e34f9d590e54fd23b43085e764e20xeae361cd677d03e2532b3420c472f6a6958fbbc55c61863cfd09aab3f4a6b0bb0xe50cb835be68b655d72a10f7f397189fe050fb9ba18209181c54f2b537784d450xb1f9d51883c919d65d58a581196ea6c73fe92eb30525db1febc88541ffd7cd370x5e9c83b38218867858cecaa68b57c9db5d66dd2df2ce8cde19fd07d4e1d35d3c0x6829879ebc7320925081eefc222044384898d3e44721c5da023f86403d96c32e7/7 post-upgrade E2E PASS. All 4 steps (EVM↔Terra) / 5 steps (Solana) green. Balances delivered to destination. Phantom final balance 5.974 null after 3 Solana inbound transfers (1.99 + 1.994 + 1.99).
Operator signing, canceler visibility, and event indexing all still working on columbus-5 / Cosmos SDK 0.53.
Solana outbound
Not covered — no Solana-side noneconomic faucet exists. Per your 4/20 guidance: bridge IN to Solana first (done today), then outbound. Tokens now available in Phantom for outbound testing once economic tokens re-enable.
Sign-off
Bridge + chain post-upgrade state: VERIFIED.
Economic token re-enable: deferred to your call per the gate — my side is clear on the noneconomic matrix.
Outstanding items tracked separately: #97 (keypair blocker), #98, #100-bridge, #109 (adding hexxagon.dev testnet mislabel to this), #113 (new, below-MIN UX from #112).
mentioned in issue #109