tapfreighter: rename state SendStateLogCommit

Renamed the `ChainPorter` state `SendStateLogCommit` to
`SendStateStorePreBroadcast` for better clarity on the state's
functionality

Author: ffranr

itest/psbt_test.go

@@ -2968,7 +2968,7 @@ func testPsbtRelativeLockTimeSendProofFail(t *harnessTest) {
 
 	AssertSendEvents(
 		t.t, aliceScriptKeyBytes, sendEvents,
-		tapfreighter.SendStateLogCommit,
+		tapfreighter.SendStateStorePreBroadcast,
 		tapfreighter.SendStateWaitTxConf,
 	)
 

tapfreighter/chain_porter.go

@@ -1031,16 +1031,16 @@ func (p *ChainPorter) stateStep(currentPkg sendPackage) (*sendPackage, error) {
 				"package: %w", err)
 		}
 
-		currentPkg.SendState = SendStateLogCommit
+		currentPkg.SendState = SendStateStorePreBroadcast
 
 		return &currentPkg, nil
 
-	// At this state, we have a final PSBT transaction which is fully
-	// signed. We'll write this to disk (the point of no return), then
-	// broadcast this to the network.
-	case SendStateLogCommit:
-		// Before we can broadcast, we want to find out the current
-		// height to pass as a height hint.
+	// In this state, the parcel state is stored before the fully signed
+	// transaction is broadcast to the mempool.
+	case SendStateStorePreBroadcast:
+		// We won't broadcast in this state, but in preparation for
+		// broadcasting, we will find out the current height to use as
+		// a height hint.
 		ctx, cancel := p.WithCtxQuit()
 		defer cancel()
 		currentHeight, err := p.cfg.ChainBridge.CurrentHeight(ctx)

tapfreighter/parcel.go

@@ -38,11 +38,9 @@ const (
 	// then finalize to place the necessary signatures in the transaction.
 	SendStateAnchorSign
 
-	// SendStateLogCommit is the final in memory state. In this state,
-	// we'll extract the signed transaction from the PSBT and log the
-	// transfer information to disk. At this point, after a restart, the
-	// transfer can be resumed.
-	SendStateLogCommit
+	// SendStateStorePreBroadcast is the state in which the finalized fully
+	// signed transaction is written to persistent storage before broadcast.
+	SendStateStorePreBroadcast
 
 	// SendStateBroadcast broadcasts the transfer transaction to the
 	// network, and imports the taproot output back into the wallet to
@@ -78,8 +76,8 @@ func (s SendState) String() string {
 	case SendStateAnchorSign:
 		return "SendStateAnchorSign"
 
-	case SendStateLogCommit:
-		return "SendStateLogCommit"
+	case SendStateStorePreBroadcast:
+		return "SendStateStorePreBroadcast"
 
 	case SendStateBroadcast:
 		return "SendStateBroadcast"
@@ -381,7 +379,7 @@ func (p *PreAnchoredParcel) pkg() *sendPackage {
 	// commitment.
 	return &sendPackage{
 		Parcel:         p,
-		SendState:      SendStateLogCommit,
+		SendState:      SendStateStorePreBroadcast,
 		VirtualPackets: p.virtualPackets,
 		PassiveAssets:  p.passiveAssets,
 		AnchorTx:       p.anchorTx,