fix(operator): re-queue approved withdrawals for execute after restart #166

Merged

PlasticDigits merged 1 commit from fix/operator-execute-approved-queue into main

2026-09-07 16:49:57 +00:00

PlasticDigits commented

2026-09-07 16:46:26 +00:00

Owner

Summary

Operator enumeration skipped already-approved destination withdrawals, so withdrawExecute* never ran after restart or pending_executions TTL eviction.
EVM and Terra writers now re-queue approved-not-executed hashes from on-chain state (INV-OP-W11) and execute them once the cancel window has elapsed.

Test plan

cargo test writers:: in packages/operator
Redeploy operator; confirm BSC pending CL8Y hashes execute (0x8daea9f5…, 0xd2350e21…)

## Summary - Operator enumeration skipped already-approved destination withdrawals, so `withdrawExecute*` never ran after restart or `pending_executions` TTL eviction. - EVM and Terra writers now re-queue approved-not-executed hashes from on-chain state (INV-OP-W11) and execute them once the cancel window has elapsed. ## Test plan - [x] `cargo test writers::` in `packages/operator` - [ ] Redeploy operator; confirm BSC pending CL8Y hashes execute (`0x8daea9f5…`, `0xd2350e21…`)

PlasticDigits added 1 commit

2026-09-07 16:46:26 +00:00

fix(operator): re-queue approved withdrawals for execute after restart 179a401850

Enumeration skipped already-approved hashes, so in-memory execute state lost on restart or TTL never recovered and transfers stayed approved-not-executed.

PlasticDigits requested review from maintainers

2026-09-07 16:46:26 +00:00

PlasticDigits merged commit 60c372a891 into main

2026-09-07 16:49:57 +00:00

PlasticDigits referenced this pull request from a commit

2026-09-07 16:49:59 +00:00

Merge pull request 'fix(operator): re-queue approved withdrawals for execute after restart' (#166) from fix/operator-execute-approved-queue into main

PlasticDigits deleted branch fix/operator-execute-approved-queue

2026-09-07 16:51:08 +00:00

PlasticDigits referenced this pull request

2026-09-12 02:39:25 +00:00

fix(operator): dest-approved Terra→EVM withdrawals stay unexecuted #170