Skip to content

Commit

Permalink
Add Wasmd Fork ADR
Browse files Browse the repository at this point in the history
  • Loading branch information
joelsmith-2019 committed Nov 6, 2024
1 parent 3e73176 commit 4d4853a
Showing 1 changed file with 19 additions and 0 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
# 6. Transfer Wasmd Fork Logic to Wormchain Ante Handler

Check warning on line 1 in wormchain/docs/architecture/0006-transfer-wasmd-fork-logic-to-wormchain.md

View workflow job for this annotation

GitHub Actions / spellcheck

Unknown word (Wasmd)

Date: 2024-07-29

## Status

Accepted

## Context

The Wormchain repo dependeds on a fork of `wasmd` to prevent certain types of Wasm transactions from being accepted by the chain. The Strangelove team proposed migrating the logic to an Ante Handler on chain to remove the need to maintain a fork of `wasmd`.

Check warning on line 11 in wormchain/docs/architecture/0006-transfer-wasmd-fork-logic-to-wormchain.md

View workflow job for this annotation

GitHub Actions / spellcheck

Unknown word (dependeds)

Check warning on line 11 in wormchain/docs/architecture/0006-transfer-wasmd-fork-logic-to-wormchain.md

View workflow job for this annotation

GitHub Actions / spellcheck

Unknown word (wasmd)

Check warning on line 11 in wormchain/docs/architecture/0006-transfer-wasmd-fork-logic-to-wormchain.md

View workflow job for this annotation

GitHub Actions / spellcheck

Unknown word (Strangelove)

Check warning on line 11 in wormchain/docs/architecture/0006-transfer-wasmd-fork-logic-to-wormchain.md

View workflow job for this annotation

GitHub Actions / spellcheck

Unknown word (wasmd)

## Decision

After some discussion and investigation, the Strangelove team has decided to transfer the fork logic to an Ante Handler on Wormchain. This will allow the Wormchain to be more in line with the Cosmos SDK and reduce the maintenance overhead of maintaining a fork of `wasmd`.

Check warning on line 15 in wormchain/docs/architecture/0006-transfer-wasmd-fork-logic-to-wormchain.md

View workflow job for this annotation

GitHub Actions / spellcheck

Unknown word (Strangelove)

Check warning on line 15 in wormchain/docs/architecture/0006-transfer-wasmd-fork-logic-to-wormchain.md

View workflow job for this annotation

GitHub Actions / spellcheck

Unknown word (wasmd)

## Consequences

The Wormchain will no longer depend on a fork of `wasmd` and will be able to upgrade to newer versions of the Cosmos SDK more easily. The Strangelove team will need to implement the Ante Handler and test it to ensure that it works as expected. As of now, the Strangelove team has not identified any major risks associated with this decision.

Check warning on line 19 in wormchain/docs/architecture/0006-transfer-wasmd-fork-logic-to-wormchain.md

View workflow job for this annotation

GitHub Actions / spellcheck

Unknown word (wasmd)

Check warning on line 19 in wormchain/docs/architecture/0006-transfer-wasmd-fork-logic-to-wormchain.md

View workflow job for this annotation

GitHub Actions / spellcheck

Unknown word (Strangelove)

Check warning on line 19 in wormchain/docs/architecture/0006-transfer-wasmd-fork-logic-to-wormchain.md

View workflow job for this annotation

GitHub Actions / spellcheck

Unknown word (Strangelove)

0 comments on commit 4d4853a

Please sign in to comment.