cleanup

common/txmgr/address_state.go

@@ -172,9 +172,6 @@ func NewAddressState[
 }
 
 func (as *AddressState[CHAIN_ID, ADDR, TX_HASH, BLOCK_HASH, R, SEQ, FEE]) close() {
-	as.Lock()
-	defer as.Unlock()
-
 	as.unstarted.Close()
 	as.unstarted = nil
 	as.inprogress = nil

common/txmgr/inmemory_store.go

@@ -20,9 +20,9 @@ import (
 )
 
 // BIG TODO LIST
-// TODO: make sure that all state transitions are handled by the address state to ensure that the in-memory store is always in a consistent state
 // TODO: figure out if multiple tx attempts are actually stored in the db for each tx
 // TODO: need a way to get id for a tx attempt. since there are some methods where the persistent store creates a tx attempt and doesnt returns it
+// TODO: should txAttempt state transitions be handled by the address state manager?
 
 var (
 	// ErrInvalidChainID is returned when the chain ID is invalid