Skip to content

Replace coreum-wasm-sdk with tx-wasm-sdk#4

Merged
TxCorpi0x merged 5 commits into
masterfrom
mehdi/tx-wasm-sdk-rename
Oct 17, 2025
Merged

Replace coreum-wasm-sdk with tx-wasm-sdk#4
TxCorpi0x merged 5 commits into
masterfrom
mehdi/tx-wasm-sdk-rename

Conversation

@TxCorpi0x

Copy link
Copy Markdown
Contributor

Description

This pull request removes support for legacy modules smart contracts from the codebase and integration tests. This includes deleting all related source code, configuration, and build logic for legacy contracts, and updating integration test dependencies to only use the current modules. The goal is to clean up deprecated functionality and prepare for future upgrades.

Removal of legacy modules smart contracts:

  • Deleted the entire integration-tests/contracts/legacy-modules directory, including all Rust contract code, configuration files, and documentation.

  • Removed the WasmLegacyModulesDir constant and all references to legacy modules in build/tx-chain/wasm.go.
    Integration tests cleanup:

  • Updated integration test dependency lists to remove compilation and usage of legacy modules smart contracts in build/tx-chain/tests.go.

Preparation for future upgrades:

  • Added TODO comments in app/app.go to indicate future removal of legacy message and query handlers.

Reviewers checklist:

  • Try to write more meaningful comments with clear actions to be taken.
  • Nit-picking should be unblocking. Focus on core issues.

Authors checklist

  • Provide a concise and meaningful description
  • Review the code yourself first, before making the PR.
  • Annotate your PR in places that require explanation.
  • Think and try to split the PR to smaller PR if it is big.

@TxCorpi0x TxCorpi0x force-pushed the mehdi/tx-wasm-sdk-rename branch 5 times, most recently from df98432 to 8af3f72 Compare October 10, 2025 13:11
@TxCorpi0x TxCorpi0x force-pushed the mehdi/tx-wasm-sdk-rename branch from 8af3f72 to 92d0036 Compare October 10, 2025 13:14
@TxCorpi0x TxCorpi0x requested a review from ysv October 10, 2025 15:07
@TxCorpi0x TxCorpi0x requested review from miladz68 and removed request for ysv October 16, 2025 13:29

@miladz68 miladz68 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@miladz68 reviewed 58 of 58 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @TxCorpi0x)

@TxCorpi0x TxCorpi0x merged commit f178235 into master Oct 17, 2025
14 of 15 checks passed
@ysv ysv deleted the mehdi/tx-wasm-sdk-rename branch December 19, 2025 14:41
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