feat: Upgrade v1.6.1 - #86
Merged
Merged
Conversation
v1.6.1 is a no-op state upgrade: it carries no state migration and exists only so the network can coordinate a version bump to ship binary-level fixes. The handler runs standard module migrations and returns. (Vesting schedule shift / account migration were dropped — not approved by the business/VC — so no vesting logic ships in this upgrade.)
Pulls the vesting-delegate fix into the tacchaind binary: bank emits a locked_amount attribute on coin_spent (cosmos-sdk tac.3) and the EVM balance handler subtracts only the spendable portion (amount - locked, evm tac.10). Delegating vesting-locked tokens via the staking precompile no longer silently burns (within spendable) or overflow-panics (beyond spendable). Ships in the v1.6.1 upgrade.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
v1.6.1 — upgrade handler + vesting-locked delegation fix
Coordinated version bump, binary fixes only (no state migration).
RunMigrations+ return) for the version bump.cosmos-sdk v0.53.6-tac.3+evm v0.6.0-tac.10.Fix: delegating vesting-locked tokens via the staking precompile (0x800) used to burn (within spendable) or overflow-panic / get stuck (beyond spendable), since EVM tracks only spendable balance. Bank now tags the locked portion on
coin_spentand the handler subtractsamount − locked→ delegation works, supply conserved.Tested: fork tests + localnet e2e (beyond 50k + within delegations pass, holdings conserved, no burn/panic).