From a1d4ea9fda71a5c6e3a211fea84c226b6e08895e Mon Sep 17 00:00:00 2001
From: Boris Nagaev <bnagaev@gmail.com>
Date: Mon, 25 Aug 2025 23:37:42 -0300
Subject: [PATCH] loopin: no timeout tx if invoice was paid

This action is not needed, since the swap is complete.
Wait until the timeout anyway to be able to push the key on every block.
---
 loopin.go | 33 ++++++++++++++++++++++++++++-----
 1 file changed, 28 insertions(+), 5 deletions(-)

diff --git a/loopin.go b/loopin.go
index dc9bf3fbf..59157e5ac 100644
--- a/loopin.go
+++ b/loopin.go
@@ -854,9 +854,11 @@ func getTxFee(tx *wire.MsgTx, fee chainfee.SatPerKVByte) btcutil.Amount {
 	return fee.FeeForVSize(lntypes.VByte(vsize))
 }
 
-// waitForSwapComplete waits until a spending tx of the htlc gets confirmed and
-// the swap invoice is either settled or canceled. If the htlc times out, the
-// timeout tx will be published.
+// waitForSwapComplete waits until a spending tx of the HTLC gets confirmed and
+// the swap invoice is either settled or canceled. If the HTLC times out and the
+// invoice is still not paid, the timeout tx will be published. If the HTLC
+// times out and the invoice is paid, the function stops waiting and returns
+// clearly.
 func (s *loopInSwap) waitForSwapComplete(ctx context.Context,
 	htlcOutpoint *wire.OutPoint, htlcValue btcutil.Amount) error {
 
@@ -882,8 +884,18 @@ func (s *loopInSwap) waitForSwapComplete(ctx context.Context,
 	}
 
 	// publishTxOnTimeout publishes the timeout tx if the contract has
-	// expired.
+	// expired and invoice has not been settled.
 	publishTxOnTimeout := func() (btcutil.Amount, error) {
+		// Don't publish the timeout tx if the invoice was settled.
+		if s.state == loopdb.StateInvoiceSettled {
+			return 0, nil
+		}
+
+		// Don't publish the timeout tx if the swap succeeded.
+		if s.state == loopdb.StateSuccess {
+			return 0, nil
+		}
+
 		if s.height >= s.LoopInContract.CltvExpiry {
 			return s.publishTimeoutTx(ctx, htlcOutpoint, htlcValue)
 		}
@@ -902,7 +914,18 @@ func (s *loopInSwap) waitForSwapComplete(ctx context.Context,
 	htlcSpend := false
 	invoiceFinalized := false
 	htlcKeyRevealed := false
-	for !htlcSpend || !invoiceFinalized {
+	for {
+		// Check stop conditions.
+		if htlcSpend && invoiceFinalized {
+			break
+		}
+		if s.state == loopdb.StateInvoiceSettled {
+			if s.height >= s.LoopInContract.CltvExpiry {
+				s.setState(loopdb.StateSuccess)
+				break
+			}
+		}
+
 		select {
 		// If the client requested the swap to be abandoned, we override
 		// the status in the database.