address review comments

Signed-off-by: Chengxuan Xing <[email protected]>

Author: Chengxuan

internal/apiclient/eventstreams.go

@@ -18,6 +18,7 @@ package apiclient
 
 import (
 	"context"
+	"errors"
 	"fmt"
 	"regexp"
 
@@ -31,7 +32,7 @@ func (c *fftmClient) GetEventStreams(ctx context.Context) ([]apitypes.EventStrea
 		SetResult(&eventStreams).
 		Get("eventstreams")
 	if !resp.IsSuccess() {
-		return nil, fmt.Errorf("%s", string(resp.Body()))
+		return nil, errors.New(string(resp.Body()))
 	}
 	return eventStreams, err
 }
@@ -43,7 +44,7 @@ func (c *fftmClient) GetListeners(ctx context.Context, eventStreamID string) ([]
 		SetResult(&listeners).
 		Get(fmt.Sprintf("eventstreams/%s/listeners", eventStreamID))
 	if !resp.IsSuccess() {
-		return nil, fmt.Errorf("%s", string(resp.Body()))
+		return nil, errors.New(string(resp.Body()))
 	}
 	return listeners, err
 }
@@ -56,7 +57,7 @@ func (c *fftmClient) DeleteEventStream(ctx context.Context, eventStreamID string
 		return err
 	}
 	if !resp.IsSuccess() {
-		return fmt.Errorf("%s", string(resp.Body()))
+		return errors.New(string(resp.Body()))
 	}
 	return nil
 }
@@ -92,7 +93,7 @@ func (c *fftmClient) DeleteListener(ctx context.Context, eventStreamID, listener
 		return err
 	}
 	if !resp.IsSuccess() {
-		return fmt.Errorf("%s", string(resp.Body()))
+		return errors.New(string(resp.Body()))
 	}
 	return nil
 }

internal/persistence/postgres/transaction_writer.go

@@ -355,7 +355,7 @@ func (tw *transactionWriter) assignNonces(ctx context.Context, txInsertsByFrom m
 				}
 			}
 			log.L(ctx).Infof("Assigned nonce %s / %d to %s", signer, cacheEntry.nextNonce, op.txInsert.ID)
-			//nolint:gosec // Safe conversion as nextNonce is always positive
+			//nolint:gosec
 			op.txInsert.Nonce = fftypes.NewFFBigInt(int64(cacheEntry.nextNonce))
 			cacheEntry.nextNonce++
 			tw.nextNonceCache.Add(signer, cacheEntry)