sync: coreth PR #1197: style: enable staticcheck linter (#1769)

JonathanOppenheimer · web-flow · commit 5e716cfc554e · 2025-10-15T16:15:26.000Z
Signed-off-by: Jonathan Oppenheimer &lt;jonathan.oppenheimer@avalabs.org&gt;
diff --git a/.avalanche-golangci.yml b/.avalanche-golangci.yml
@@ -75,7 +75,7 @@ linters:
     - predeclared
     - revive
     - spancheck
-    # - staticcheck
+    - staticcheck
     - tagalign
     # - testifylint
     - unconvert
diff --git a/accounts/abi/abi_extra_test.go b/accounts/abi/abi_extra_test.go
@@ -15,10 +15,10 @@ import (
 
 // Note: This file contains tests in addition to those found in go-ethereum.
 
-const TEST_ABI = `[{"type":"function","name":"receive","inputs":[{"name":"sender","type":"address"},{"name":"amount","type":"uint256"},{"name":"memo","type":"bytes"}],"outputs":[{"internalType":"bool","name":"isAllowed","type":"bool"}]}]`
+const TestABI = `[{"type":"function","name":"receive","inputs":[{"name":"sender","type":"address"},{"name":"amount","type":"uint256"},{"name":"memo","type":"bytes"}],"outputs":[{"internalType":"bool","name":"isAllowed","type":"bool"}]}]`
 
 func TestUnpackInputIntoInterface(t *testing.T) {
-	abi, err := JSON(strings.NewReader(TEST_ABI))
+	abi, err := JSON(strings.NewReader(TestABI))
 	require.NoError(t, err)
 
 	type inputType struct {
@@ -35,7 +35,7 @@ func TestUnpackInputIntoInterface(t *testing.T) {
 	rawData, err := abi.Pack("receive", input.Sender, input.Amount, input.Memo)
 	require.NoError(t, err)
 
-	abi, err = JSON(strings.NewReader(TEST_ABI))
+	abi, err = JSON(strings.NewReader(TestABI))
 	require.NoError(t, err)
 
 	for _, test := range []struct {
@@ -97,7 +97,7 @@ func TestUnpackInputIntoInterface(t *testing.T) {
 }
 
 func TestPackOutput(t *testing.T) {
-	abi, err := JSON(strings.NewReader(TEST_ABI))
+	abi, err := JSON(strings.NewReader(TestABI))
 	require.NoError(t, err)
 
 	bytes, err := abi.PackOutput("receive", true)
diff --git a/cmd/simulator/load/loader.go b/cmd/simulator/load/loader.go
@@ -196,10 +196,8 @@ func ExecuteLoader(ctx context.Context, config config.Config) error {
 	log.Info("Distributed funds successfully", "time", time.Since(fundStart))
 
 	pks := make([]*ecdsa.PrivateKey, 0, len(keys))
-	senders := make([]common.Address, 0, len(keys))
 	for _, key := range keys {
 		pks = append(pks, key.PrivKey)
-		senders = append(senders, key.Address)
 	}
 
 	bigGwei := big.NewInt(params.GWei)
diff --git a/core/extstate/database.go b/core/extstate/database.go
@@ -26,7 +26,7 @@ func wrapIfFirewood(db state.Database) state.Database {
 	if !ok {
 		return db
 	}
-	return &firewoodAccessorDb{
+	return &firewoodAccessorDB{
 		Database: db,
 		fw:       fw,
 	}
diff --git a/core/extstate/firewood_database.go b/core/extstate/firewood_database.go
@@ -13,35 +13,35 @@ import (
 )
 
 var (
-	_ state.Database = (*firewoodAccessorDb)(nil)
+	_ state.Database = (*firewoodAccessorDB)(nil)
 	_ state.Trie     = (*firewood.AccountTrie)(nil)
 	_ state.Trie     = (*firewood.StorageTrie)(nil)
 )
 
-type firewoodAccessorDb struct {
+type firewoodAccessorDB struct {
 	state.Database
 	fw *firewood.Database
 }
 
 // OpenTrie opens the main account trie.
-func (db *firewoodAccessorDb) OpenTrie(root common.Hash) (state.Trie, error) {
+func (db *firewoodAccessorDB) OpenTrie(root common.Hash) (state.Trie, error) {
 	return firewood.NewAccountTrie(root, db.fw)
 }
 
 // OpenStorageTrie opens a wrapped version of the account trie.
 //
 //nolint:revive // removing names loses context.
-func (*firewoodAccessorDb) OpenStorageTrie(stateRoot common.Hash, addr common.Address, accountRoot common.Hash, self state.Trie) (state.Trie, error) {
+func (*firewoodAccessorDB) OpenStorageTrie(stateRoot common.Hash, addr common.Address, accountRoot common.Hash, self state.Trie) (state.Trie, error) {
 	accountTrie, ok := self.(*firewood.AccountTrie)
 	if !ok {
-		return nil, fmt.Errorf("Invalid account trie type: %T", self)
+		return nil, fmt.Errorf("invalid account trie type: %T", self)
 	}
 	return firewood.NewStorageTrie(accountTrie)
 }
 
 // CopyTrie returns a deep copy of the given trie.
 // It can be altered by the caller.
-func (*firewoodAccessorDb) CopyTrie(t state.Trie) state.Trie {
+func (*firewoodAccessorDB) CopyTrie(t state.Trie) state.Trie {
 	switch t := t.(type) {
 	case *firewood.AccountTrie:
 		return t.Copy()
diff --git a/network/waiting_handler.go b/network/waiting_handler.go
@@ -49,12 +49,12 @@ func (w *waitingResponseHandler) OnFailure() error {
 	return nil
 }
 
-func (waitingHandler *waitingResponseHandler) WaitForResult(ctx context.Context) ([]byte, error) {
+func (w *waitingResponseHandler) WaitForResult(ctx context.Context) ([]byte, error) {
 	select {
 	case <-ctx.Done():
 		return nil, ctx.Err()
-	case response := <-waitingHandler.responseChan:
-		if waitingHandler.failed {
+	case response := <-w.responseChan:
+		if w.failed {
 			return nil, errRequestFailed
 		}
 		return response, nil
diff --git a/params/extras/network_upgrades.go b/params/extras/network_upgrades.go
@@ -105,19 +105,19 @@ func (n *NetworkUpgrades) SetDefaults(agoUpgrades upgrade.Config) {
 func (n *NetworkUpgrades) verifyNetworkUpgrades(agoUpgrades upgrade.Config) error {
 	defaults := GetNetworkUpgrades(agoUpgrades)
 	if err := verifyWithDefault(n.SubnetEVMTimestamp, defaults.SubnetEVMTimestamp); err != nil {
-		return fmt.Errorf("SubnetEVM fork block timestamp is invalid: %w", err)
+		return fmt.Errorf("subnetEVM fork block timestamp is invalid: %w", err)
 	}
 	if err := verifyWithDefault(n.DurangoTimestamp, defaults.DurangoTimestamp); err != nil {
-		return fmt.Errorf("Durango fork block timestamp is invalid: %w", err)
+		return fmt.Errorf("durango fork block timestamp is invalid: %w", err)
 	}
 	if err := verifyWithDefault(n.EtnaTimestamp, defaults.EtnaTimestamp); err != nil {
-		return fmt.Errorf("Etna fork block timestamp is invalid: %w", err)
+		return fmt.Errorf("etna fork block timestamp is invalid: %w", err)
 	}
 	if err := verifyWithDefault(n.FortunaTimestamp, defaults.FortunaTimestamp); err != nil {
-		return fmt.Errorf("Fortuna fork block timestamp is invalid: %w", err)
+		return fmt.Errorf("fortuna fork block timestamp is invalid: %w", err)
 	}
 	if err := verifyWithDefault(n.GraniteTimestamp, defaults.GraniteTimestamp); err != nil {
-		return fmt.Errorf("Granite fork block timestamp is invalid: %w", err)
+		return fmt.Errorf("granite fork block timestamp is invalid: %w", err)
 	}
 	return nil
 }
diff --git a/plugin/evm/blockgascost/cost.go b/plugin/evm/blockgascost/cost.go
@@ -30,10 +30,10 @@ func BlockGasCost(
 	}
 
 	var (
-		minBlockGasCost uint64 = feeConfig.MinBlockGasCost.Uint64()
-		maxBlockGasCost uint64 = feeConfig.MaxBlockGasCost.Uint64()
-		op                     = safemath.Add[uint64]
-		defaultCost     uint64 = feeConfig.MaxBlockGasCost.Uint64()
+		minBlockGasCost = feeConfig.MinBlockGasCost.Uint64()
+		maxBlockGasCost = feeConfig.MaxBlockGasCost.Uint64()
+		op              = safemath.Add[uint64]
+		defaultCost     = feeConfig.MaxBlockGasCost.Uint64()
 	)
 	if timeElapsed > feeConfig.TargetBlockRate {
 		op = safemath.Sub
diff --git a/plugin/evm/client/client.go b/plugin/evm/client/client.go
@@ -49,8 +49,8 @@ type client struct {
 
 // NewClient returns a Client for interacting with EVM [chain]
 func NewClient(uri, chain string) Client {
-	requestUri := fmt.Sprintf("%s/ext/bc/%s", uri, chain)
-	return NewClientWithURL(requestUri)
+	requestURI := fmt.Sprintf("%s/ext/bc/%s", uri, chain)
+	return NewClientWithURL(requestURI)
 }
 
 // NewClientWithURL returns a Client for interacting with EVM [chain]
diff --git a/plugin/evm/config/config.go b/plugin/evm/config/config.go
@@ -173,7 +173,7 @@ type Config struct {
 	WarpOffChainMessages []hexutil.Bytes `json:"warp-off-chain-messages"`
 
 	// RPC settings
-	HttpBodyLimit        uint64 `json:"http-body-limit"`
+	HTTPBodyLimit        uint64 `json:"http-body-limit"`
 	BatchRequestLimit    uint64 `json:"batch-request-limit"`
 	BatchResponseMaxSize uint64 `json:"batch-response-max-size"`
 
diff --git a/plugin/evm/customrawdb/database_ext.go b/plugin/evm/customrawdb/database_ext.go
@@ -70,7 +70,7 @@ func ParseStateSchemeExt(provided string, disk ethdb.Database) (string, error) {
 	if provided == FirewoodScheme {
 		if diskScheme := rawdb.ReadStateScheme(disk); diskScheme != "" {
 			// Valid scheme on disk mismatched
-			return "", fmt.Errorf("State scheme %s already set on disk, can't use Firewood", diskScheme)
+			return "", fmt.Errorf("state scheme %s already set on disk, can't use Firewood", diskScheme)
 		}
 		// If no conflicting scheme is found, is valid.
 		return FirewoodScheme, nil
diff --git a/plugin/evm/database/wrapped_database.go b/plugin/evm/database/wrapped_database.go
@@ -11,37 +11,37 @@ import (
 )
 
 var (
-	_ ethdb.KeyValueStore = (*ethDbWrapper)(nil)
+	_ ethdb.KeyValueStore = (*ethDBWrapper)(nil)
 
 	ErrSnapshotNotSupported = errors.New("snapshot is not supported")
 )
 
 // ethDbWrapper implements ethdb.Database
-type ethDbWrapper struct{ database.Database }
+type ethDBWrapper struct{ database.Database }
 
-func WrapDatabase(db database.Database) ethdb.KeyValueStore { return ethDbWrapper{db} }
+func WrapDatabase(db database.Database) ethdb.KeyValueStore { return ethDBWrapper{db} }
 
 // Stat implements ethdb.Database
-func (ethDbWrapper) Stat(string) (string, error) { return "", database.ErrNotFound }
+func (ethDBWrapper) Stat(string) (string, error) { return "", database.ErrNotFound }
 
 // NewBatch implements ethdb.Database
-func (db ethDbWrapper) NewBatch() ethdb.Batch { return wrappedBatch{db.Database.NewBatch()} }
+func (db ethDBWrapper) NewBatch() ethdb.Batch { return wrappedBatch{db.Database.NewBatch()} }
 
 // NewBatchWithSize implements ethdb.Database
 // TODO: propagate size through avalanchego Database interface
-func (db ethDbWrapper) NewBatchWithSize(int) ethdb.Batch {
+func (db ethDBWrapper) NewBatchWithSize(int) ethdb.Batch {
 	return wrappedBatch{db.Database.NewBatch()}
 }
 
-func (ethDbWrapper) NewSnapshot() (ethdb.Snapshot, error) {
+func (ethDBWrapper) NewSnapshot() (ethdb.Snapshot, error) {
 	return nil, ErrSnapshotNotSupported
 }
 
 // NewIterator implements ethdb.Database
 //
 // Note: This method assumes that the prefix is NOT part of the start, so there's
 // no need for the caller to prepend the prefix to the start.
-func (db ethDbWrapper) NewIterator(prefix []byte, start []byte) ethdb.Iterator {
+func (db ethDBWrapper) NewIterator(prefix []byte, start []byte) ethdb.Iterator {
 	// avalanchego's database implementation assumes that the prefix is part of the
 	// start, so it is added here (if it is provided).
 	if len(prefix) > 0 {
@@ -54,7 +54,7 @@ func (db ethDbWrapper) NewIterator(prefix []byte, start []byte) ethdb.Iterator {
 }
 
 // NewIteratorWithStart implements ethdb.Database
-func (db ethDbWrapper) NewIteratorWithStart(start []byte) ethdb.Iterator {
+func (db ethDBWrapper) NewIteratorWithStart(start []byte) ethdb.Iterator {
 	return db.Database.NewIteratorWithStart(start)
 }
 
diff --git a/plugin/evm/vm.go b/plugin/evm/vm.go
@@ -153,7 +153,7 @@ var (
 
 // legacyApiNames maps pre geth v1.10.20 api names to their updated counterparts.
 // used in attachEthService for backward configuration compatibility.
-var legacyApiNames = map[string]string{
+var legacyAPINames = map[string]string{
 	"internal-public-eth":              "internal-eth",
 	"internal-public-blockchain":       "internal-blockchain",
 	"internal-public-transaction-pool": "internal-transaction",
@@ -1140,8 +1140,8 @@ func (vm *VM) CreateHandlers(context.Context) (map[string]http.Handler, error) {
 	if vm.config.BatchRequestLimit > 0 && vm.config.BatchResponseMaxSize > 0 {
 		handler.SetBatchLimits(int(vm.config.BatchRequestLimit), int(vm.config.BatchResponseMaxSize))
 	}
-	if vm.config.HttpBodyLimit > 0 {
-		handler.SetHTTPBodyLimit(int(vm.config.HttpBodyLimit))
+	if vm.config.HTTPBodyLimit > 0 {
+		handler.SetHTTPBodyLimit(int(vm.config.HTTPBodyLimit))
 	}
 
 	enabledAPIs := vm.config.EthAPIs()
@@ -1309,7 +1309,7 @@ func attachEthService(handler *rpc.Server, apis []rpc.API, names []string) error
 	for _, ns := range names {
 		// handle pre geth v1.10.20 api names as aliases for their updated values
 		// to allow configurations to be backwards compatible.
-		if newName, isLegacy := legacyApiNames[ns]; isLegacy {
+		if newName, isLegacy := legacyAPINames[ns]; isLegacy {
 			log.Info("deprecated api name referenced in configuration.", "deprecated", ns, "new", newName)
 			enabledServicesSet[newName] = struct{}{}
 			continue
diff --git a/precompile/contracts/feemanager/unpack_pack_test.go b/precompile/contracts/feemanager/unpack_pack_test.go
@@ -42,13 +42,10 @@ func FuzzPackGetFeeConfigOutputEqualTest(f *testing.F) {
 			MaxBlockGasCost:          bigIntVal,
 			BlockGasCostStep:         bigIntVal,
 		}
-		doCheckOutputs := true
 		// we can only check if outputs are correct if the value is less than MaxUint256
 		// otherwise the value will be truncated when packed,
 		// and thus unpacked output will not be equal to the value
-		if bigIntVal.Cmp(abi.MaxUint256) > 0 {
-			doCheckOutputs = false
-		}
+		doCheckOutputs := bigIntVal.Cmp(abi.MaxUint256) <= 0
 		testOldPackGetFeeConfigOutputEqual(t, feeConfig, doCheckOutputs)
 	})
 }
@@ -173,13 +170,10 @@ func FuzzPackGetLastChangedAtOutput(f *testing.F) {
 	f.Add(math.MaxBig256.Add(math.MaxBig256, common.Big1).Bytes())
 	f.Fuzz(func(t *testing.T, bigIntBytes []byte) {
 		bigIntVal := new(big.Int).SetBytes(bigIntBytes)
-		doCheckOutputs := true
 		// we can only check if outputs are correct if the value is less than MaxUint256
 		// otherwise the value will be truncated when packed,
 		// and thus unpacked output will not be equal to the value
-		if bigIntVal.Cmp(abi.MaxUint256) > 0 {
-			doCheckOutputs = false
-		}
+		doCheckOutputs := bigIntVal.Cmp(abi.MaxUint256) <= 0
 		testOldPackGetLastChangedAtOutputEqual(t, bigIntVal, doCheckOutputs)
 	})
 }
@@ -203,13 +197,10 @@ func FuzzPackSetFeeConfigEqualTest(f *testing.F) {
 			MaxBlockGasCost:          bigIntVal,
 			BlockGasCostStep:         bigIntVal,
 		}
-		doCheckOutputs := true
 		// we can only check if outputs are correct if the value is less than MaxUint256
 		// otherwise the value will be truncated when packed,
 		// and thus unpacked output will not be equal to the value
-		if bigIntVal.Cmp(abi.MaxUint256) > 0 {
-			doCheckOutputs = false
-		}
+		doCheckOutputs := bigIntVal.Cmp(abi.MaxUint256) <= 0
 		testOldPackSetFeeConfigInputEqual(t, feeConfig, doCheckOutputs)
 	})
 }
diff --git a/precompile/contracts/nativeminter/unpack_pack_test.go b/precompile/contracts/nativeminter/unpack_pack_test.go
@@ -32,13 +32,10 @@ func FuzzPackMintNativeCoinEqualTest(f *testing.F) {
 	f.Add(constants.BlackholeAddr.Bytes(), common.Big2.Bytes())
 	f.Fuzz(func(t *testing.T, b []byte, bigIntBytes []byte) {
 		bigIntVal := new(big.Int).SetBytes(bigIntBytes)
-		doCheckOutputs := true
 		// we can only check if outputs are correct if the value is less than MaxUint256
 		// otherwise the value will be truncated when packed,
 		// and thus unpacked output will not be equal to the value
-		if bigIntVal.Cmp(abi.MaxUint256) > 0 {
-			doCheckOutputs = false
-		}
+		doCheckOutputs := bigIntVal.Cmp(abi.MaxUint256) <= 0
 		testOldPackMintNativeCoinEqual(t, common.BytesToAddress(b), bigIntVal, doCheckOutputs)
 	})
 }
diff --git a/precompile/modules/module.go b/precompile/modules/module.go
@@ -25,12 +25,12 @@ type Module struct {
 
 type moduleArray []Module
 
-func (u moduleArray) Len() int {
-	return len(u)
+func (m moduleArray) Len() int {
+	return len(m)
 }
 
-func (u moduleArray) Swap(i, j int) {
-	u[i], u[j] = u[j], u[i]
+func (m moduleArray) Swap(i, j int) {
+	m[i], m[j] = m[j], m[i]
 }
 
 func (m moduleArray) Less(i, j int) bool {
diff --git a/sync/client/client.go b/sync/client/client.go
@@ -316,7 +316,7 @@ func (c *client) get(ctx context.Context, request message.Request, parseFn parse
 		var (
 			response []byte
 			nodeID   ids.NodeID
-			start    time.Time = time.Now()
+			start    = time.Now()
 		)
 		if len(c.stateSyncNodes) == 0 {
 			response, nodeID, err = c.networkClient.SendSyncedAppRequestAny(ctx, StateSyncVersion, requestBytes)
diff --git a/sync/handlers/leafs_request.go b/sync/handlers/leafs_request.go
diff --git a/sync/handlers/leafs_request_test.go b/sync/handlers/leafs_request_test.go
diff --git a/triedb/firewood/account_trie.go b/triedb/firewood/account_trie.go

Original file line number	Diff line number	Diff line change
`@@ -196,10 +196,8 @@ func ExecuteLoader(ctx context.Context, config config.Config) error {`
`196`	`196`	`log.Info("Distributed funds successfully", "time", time.Since(fundStart))`
`197`	`197`
`198`	`198`	`pks := make([]*ecdsa.PrivateKey, 0, len(keys))`
`199`		`- senders := make([]common.Address, 0, len(keys))`
`200`	`199`	`for _, key := range keys {`
`201`	`200`	`pks = append(pks, key.PrivKey)`
`202`		`- senders = append(senders, key.Address)`
`203`	`201`	`}`
`204`	`202`
`205`	`203`	`bigGwei := big.NewInt(params.GWei)`
Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,7 @@ func wrapIfFirewood(db state.Database) state.Database {`
`26`	`26`	`if !ok {`
`27`	`27`	`return db`
`28`	`28`	`}`
`29`		`- return &firewoodAccessorDb{`
	`29`	`+ return &firewoodAccessorDB{`
`30`	`30`	`Database: db,`
`31`	`31`	`fw: fw,`
`32`	`32`	`}`
Original file line number	Diff line number	Diff line change
`@@ -105,19 +105,19 @@ func (n *NetworkUpgrades) SetDefaults(agoUpgrades upgrade.Config) {`
`105`	`105`	`func (n *NetworkUpgrades) verifyNetworkUpgrades(agoUpgrades upgrade.Config) error {`
`106`	`106`	`defaults := GetNetworkUpgrades(agoUpgrades)`
`107`	`107`	`if err := verifyWithDefault(n.SubnetEVMTimestamp, defaults.SubnetEVMTimestamp); err != nil {`
`108`		`- return fmt.Errorf("SubnetEVM fork block timestamp is invalid: %w", err)`
	`108`	`+ return fmt.Errorf("subnetEVM fork block timestamp is invalid: %w", err)`
`109`	`109`	`}`
`110`	`110`	`if err := verifyWithDefault(n.DurangoTimestamp, defaults.DurangoTimestamp); err != nil {`
`111`		`- return fmt.Errorf("Durango fork block timestamp is invalid: %w", err)`
	`111`	`+ return fmt.Errorf("durango fork block timestamp is invalid: %w", err)`
`112`	`112`	`}`
`113`	`113`	`if err := verifyWithDefault(n.EtnaTimestamp, defaults.EtnaTimestamp); err != nil {`
`114`		`- return fmt.Errorf("Etna fork block timestamp is invalid: %w", err)`
	`114`	`+ return fmt.Errorf("etna fork block timestamp is invalid: %w", err)`
`115`	`115`	`}`
`116`	`116`	`if err := verifyWithDefault(n.FortunaTimestamp, defaults.FortunaTimestamp); err != nil {`
`117`		`- return fmt.Errorf("Fortuna fork block timestamp is invalid: %w", err)`
	`117`	`+ return fmt.Errorf("fortuna fork block timestamp is invalid: %w", err)`
`118`	`118`	`}`
`119`	`119`	`if err := verifyWithDefault(n.GraniteTimestamp, defaults.GraniteTimestamp); err != nil {`
`120`		`- return fmt.Errorf("Granite fork block timestamp is invalid: %w", err)`
	`120`	`+ return fmt.Errorf("granite fork block timestamp is invalid: %w", err)`
`121`	`121`	`}`
`122`	`122`	`return nil`
`123`	`123`	`}`
Original file line number	Diff line number	Diff line change
`@@ -30,10 +30,10 @@ func BlockGasCost(`
`30`	`30`	`}`
`31`	`31`
`32`	`32`	`var (`
`33`		`- minBlockGasCost uint64 = feeConfig.MinBlockGasCost.Uint64()`
`34`		`- maxBlockGasCost uint64 = feeConfig.MaxBlockGasCost.Uint64()`
`35`		`- op = safemath.Add[uint64]`
`36`		`- defaultCost uint64 = feeConfig.MaxBlockGasCost.Uint64()`
	`33`	`+ minBlockGasCost = feeConfig.MinBlockGasCost.Uint64()`
	`34`	`+ maxBlockGasCost = feeConfig.MaxBlockGasCost.Uint64()`
	`35`	`+ op = safemath.Add[uint64]`
	`36`	`+ defaultCost = feeConfig.MaxBlockGasCost.Uint64()`
`37`	`37`	`)`
`38`	`38`	`if timeElapsed > feeConfig.TargetBlockRate {`
`39`	`39`	`op = safemath.Sub`
Original file line number	Diff line number	Diff line change
`@@ -49,8 +49,8 @@ type client struct {`
`49`	`49`
`50`	`50`	`// NewClient returns a Client for interacting with EVM [chain]`
`51`	`51`	`func NewClient(uri, chain string) Client {`
`52`		`- requestUri := fmt.Sprintf("%s/ext/bc/%s", uri, chain)`
`53`		`- return NewClientWithURL(requestUri)`
	`52`	`+ requestURI := fmt.Sprintf("%s/ext/bc/%s", uri, chain)`
	`53`	`+ return NewClientWithURL(requestURI)`
`54`	`54`	`}`
`55`	`55`
`56`	`56`	`// NewClientWithURL returns a Client for interacting with EVM [chain]`