refactor(store): use map.Copy for cleaner map handling (#24076)

Co-authored-by: Alex | Interchain Labs <alex@interchainlabs.io>

store/cachemulti/store.go

@@ -3,6 +3,7 @@ package cachemulti
 import (
 	"fmt"
 	"io"
+	"maps"
 
 	dbm "github.com/cosmos/cosmos-db"
 
@@ -94,9 +95,7 @@ func (cms Store) SetTracer(w io.Writer) types.MultiStore {
 // necessary between tracing operations. It returns a modified MultiStore.
 func (cms Store) SetTracingContext(tc types.TraceContext) types.MultiStore {
 	if cms.traceContext != nil {
-		for k, v := range tc {
-			cms.traceContext[k] = v
-		}
+		maps.Copy(cms.traceContext, tc)
 	} else {
 		cms.traceContext = tc
 	}

store/rootmulti/store.go

@@ -5,6 +5,7 @@ import (
 	"errors"
 	"fmt"
 	"io"
+	"maps"
 	"math"
 	"sort"
 	"strings"
@@ -383,9 +384,7 @@ func (rs *Store) getTracingContext() types.TraceContext {
 	}
 
 	ctx := types.TraceContext{}
-	for k, v := range rs.traceContext {
-		ctx[k] = v
-	}
+	maps.Copy(ctx, rs.traceContext)
 
 	return ctx
 }

store/types/store.go

@@ -3,6 +3,7 @@ package types
 import (
 	"fmt"
 	"io"
+	"maps"
 	"slices"
 
 	"github.com/cometbft/cometbft/proto/tendermint/crypto"
@@ -454,9 +455,7 @@ type TraceContext map[string]interface{}
 // Clone clones tc into another instance of TraceContext.
 func (tc TraceContext) Clone() TraceContext {
 	ret := TraceContext{}
-	for k, v := range tc {
-		ret[k] = v
-	}
+	maps.Copy(ret, tc)
 
 	return ret
 }
@@ -467,9 +466,7 @@ func (tc TraceContext) Merge(newTc TraceContext) TraceContext {
 		tc = TraceContext{}
 	}
 
-	for k, v := range newTc {
-		tc[k] = v
-	}
+	maps.Copy(tc, newTc)
 
 	return tc
 }