multi: enforce 80-char line length and add nolint:lll comments

- Updated `.golangci.yml` to enforce a maximum line length of 80
  characters.
- Reformatted code comments and function parameters to adhere to the new
  line-length restriction.
- Added `nolint: lll` comments where line wrapping was not applicable.

Author: ffranr

.golangci.yml

@@ -20,6 +20,9 @@ linters-settings:
   gofmt:
     # simplify code: gofmt with `-s` option, true by default
     simplify: true
+  lll:
+    # Max line length, lines longer will be reported.
+    line-length: 80
   tagliatelle:
     case:
       rules:

accounts/tlv.go

@@ -227,7 +227,9 @@ func PaymentEntryMapEncoder(w io.Writer, val any, buf *[8]byte) error {
 }
 
 // PaymentEntryMapDecoder decodes a map of payment entries.
-func PaymentEntryMapDecoder(r io.Reader, val any, buf *[8]byte, _ uint64) error {
+func PaymentEntryMapDecoder(r io.Reader, val any, buf *[8]byte,
+	_ uint64) error {
+
 	if typ, ok := val.(*AccountPayments); ok {
 		numItems, err := tlv.ReadVarInt(r, buf)
 		if err != nil {

autopilotserver/mock/server.go

@@ -436,7 +436,9 @@ func rulesToRPC(rulesMap map[string]*RuleRanges) (
 	return res, nil
 }
 
-func permissionsToRPC(ps map[string][]bakery.Op) []*autopilotserverrpc.Permissions {
+func permissionsToRPC(
+	ps map[string][]bakery.Op) []*autopilotserverrpc.Permissions {
+
 	res := make([]*autopilotserverrpc.Permissions, len(ps))
 
 	for method, ops := range ps {

config.go

@@ -583,13 +583,16 @@ func loadAndValidateConfig(interceptor signal.Interceptor) (*Config, error) {
 
 	defaultTapCfg := tapcfg.DefaultConfig()
 	if cfg.tapRemote && cfg.Network != DefaultNetwork {
-		if cfg.Remote.TaprootAssets.MacaroonPath == defaultTapCfg.RpcConf.MacaroonPath {
+		if cfg.Remote.TaprootAssets.MacaroonPath ==
+			defaultTapCfg.RpcConf.MacaroonPath {
+
 			macaroonPath := cfg.TaprootAssets.RpcConf.MacaroonPath
 			cfg.Remote.TaprootAssets.MacaroonPath = macaroonPath
 		}
-		if cfg.Remote.TaprootAssets.TLSCertPath == defaultTapCfg.RpcConf.TLSCertPath {
-			tlsCertPath := cfg.TaprootAssets.RpcConf.TLSCertPath
+		if cfg.Remote.TaprootAssets.TLSCertPath ==
+			defaultTapCfg.RpcConf.TLSCertPath {
 
+			tlsCertPath := cfg.TaprootAssets.RpcConf.TLSCertPath
 			cfg.Remote.TaprootAssets.TLSCertPath = tlsCertPath
 		}
 	}

firewall/privacy_mapper_test.go

@@ -27,17 +27,20 @@ func TestPrivacyMapper(t *testing.T) {
 	}
 
 	// Define some transaction outpoints used for mapping.
-	clearTxID := "abcdefabcdefabcdefabcdefabcdefabcdefabcdefabcdefabcdefabcdefabcd"
+	clearTxID :=
+		"abcdefabcdefabcdefabcdefabcdefabcdefabcdefabcdefabcdefabcdefabcd"
 	clearTxIDReveresed, err := chainhash.NewHashFromStr(clearTxID)
 	require.NoError(t, err)
 
-	obfusTxID0 := "097ef666a61919ff3413b3b701eae3a5cbac08f70c0ca567806e1fa6acbfe384"
+	obfusTxID0 :=
+		"097ef666a61919ff3413b3b701eae3a5cbac08f70c0ca567806e1fa6acbfe384"
 	require.NoError(t, err)
 	obfusOut0 := uint32(2161781494)
 	obfusTxID0Reversed, err := chainhash.NewHashFromStr(obfusTxID0)
 	require.NoError(t, err)
 
-	obfusTxID1 := "45ec471bfccb0b7b9a8bc4008248931c59ad994903e07b54f54821ea3ef5cc5c"
+	obfusTxID1 :=
+		"45ec471bfccb0b7b9a8bc4008248931c59ad994903e07b54f54821ea3ef5cc5c"
 	obfusOut1 := uint32(1642614131)
 
 	clearPending := &lnrpc.PendingChannelsResponse_PendingChannel{
@@ -554,6 +557,7 @@ func TestPrivacyMapper(t *testing.T) {
 			name:    "PendingChannels Response",
 			uri:     "/lnrpc.Lightning/PendingChannels",
 			msgType: rpcperms.TypeResponse,
+			// nolint: lll
 			msg: &lnrpc.PendingChannelsResponse{
 				PendingOpenChannels: []*lnrpc.PendingChannelsResponse_PendingOpenChannel{
 					{
@@ -595,6 +599,7 @@ func TestPrivacyMapper(t *testing.T) {
 					},
 				},
 			},
+			//nolint:lll
 			expectedReplacement: &lnrpc.PendingChannelsResponse{
 				PendingOpenChannels: []*lnrpc.PendingChannelsResponse_PendingOpenChannel{
 					{

firewalldb/actions.go

@@ -318,9 +318,9 @@ func (s *groupActionsReadDB) ListActions(ctx context.Context) ([]*RuleAction,
 	return actions, nil
 }
 
-// groupFeatureActionsReadDB is an implementation of the rules.ActionsListDB that
-// will provide read access to all the Actions of a feature within a particular
-// group.
+// groupFeatureActionsReadDB is an implementation of the rules.ActionsListDB
+// that will provide read access to all the Actions of a feature within a
+// particular group.
 type groupFeatureActionsReadDB struct {
 	*allActionsReadDB
 }

firewalldb/actions_kvdb.go

@@ -32,6 +32,7 @@ const (
 	typeLocatorActionID  tlv.Type = 2
 )
 
+// nolint: lll
 /*
 	The Actions are stored in the following structure in the KV db:
 

firewalldb/kvstores_kvdb.go

@@ -8,6 +8,7 @@ import (
 	"go.etcd.io/bbolt"
 )
 
+// nolint:lll
 /*
 The KVStores are stored in the following structure in the KV db. Note that
 the `perm` and `temp` buckets are identical in structure. The only difference

itest/litd_node.go

@@ -637,7 +637,8 @@ func (hn *HarnessNode) InvoiceMacPath() string {
 	return hn.Cfg.InvoiceMacPath
 }
 
-// renameFile is a helper to rename (log) files created during integration tests.
+// renameFile is a helper to rename (log) files created during integration
+// tests.
 func renameFile(fromFileName, toFileName string) {
 	err := os.Rename(fromFileName, toFileName)
 	if err != nil {
@@ -1803,7 +1804,9 @@ func (hn *HarnessNode) WaitForBlockchainSync(ctx context.Context) error {
 
 // WaitForBalance waits until the node sees the expected confirmed/unconfirmed
 // balance within their wallet.
-func (hn *HarnessNode) WaitForBalance(expectedBalance btcutil.Amount, confirmed bool) error {
+func (hn *HarnessNode) WaitForBalance(expectedBalance btcutil.Amount,
+	confirmed bool) error {
+
 	ctx := context.Background()
 	req := &lnrpc.WalletBalanceRequest{}
 

itest/network_harness.go

@@ -1010,10 +1010,10 @@ func (n *NetworkHarness) OpenChannel(srcNode, destNode *HarnessNode,
 	}
 }
 
-// OpenPendingChannel attempts to open a channel between srcNode and destNode with the
-// passed channel funding parameters. If the passed context has a timeout, then
-// if the timeout is reached before the channel pending notification is
-// received, an error is returned.
+// OpenPendingChannel attempts to open a channel between srcNode and destNode
+// with the passed channel funding parameters. If the passed context has a
+// timeout, then if the timeout is reached before the channel pending
+// notification is received, an error is returned.
 func (n *NetworkHarness) OpenPendingChannel(srcNode, destNode *HarnessNode,
 	amt btcutil.Amount,
 	pushAmt btcutil.Amount) (*lnrpc.PendingUpdate, error) {
@@ -1080,7 +1080,8 @@ func (n *NetworkHarness) OpenPendingChannel(srcNode, destNode *HarnessNode,
 // has a timeout, then if the timeout is reached before the channel has been
 // opened, then an error is returned.
 func (n *NetworkHarness) WaitForChannelOpen(
-	openChanStream lnrpc.Lightning_OpenChannelClient) (*lnrpc.ChannelPoint, error) {
+	openChanStream lnrpc.Lightning_OpenChannelClient) (*lnrpc.ChannelPoint,
+	error) {
 
 	ctxb := context.Background()
 	ctx, cancel := context.WithTimeout(ctxb, wait.ChannelOpenTimeout)