Merge pull request #3605 from nspcc-dev/fix-linter

AnnaShaleva · web-flow · commit a9242535db75 · 2024-10-11T14:16:44.000+03:00
Enable linter for every module
diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
@@ -14,8 +14,33 @@ on:
 
 jobs:
   lint:
-    name: Lint
+    name: 'Lint: NeoGo'
     uses: nspcc-dev/.github/.github/workflows/go-linter.yml@master
+
+  lint_examples:
+    name: 'Lint: examples (${{ matrix.contract }})'
+    uses: nspcc-dev/.github/.github/workflows/go-linter.yml@master
+
+    strategy:
+      fail-fast: false
+      matrix:
+        contract: [ 'engine', 'events', 'iterator', 'nft-d', 'nft-nd', 'nft-nd-nns', 'oracle',
+                    'runtime', 'storage', 'timer', 'token', 'zkp/cubic_circuit', 'zkp/xor_compat']
+    with:
+      workdir: examples/${{ matrix.contract }}
+
+  lint_scripts:
+    name: 'Lint: scripts'
+    uses: nspcc-dev/.github/.github/workflows/go-linter.yml@master
+    with:
+      workdir: scripts
+
+  lint_interops:
+    name: 'Lint: interop'
+    uses: nspcc-dev/.github/.github/workflows/go-linter.yml@master
+    with:
+      workdir: pkg/interop
+
   gomodcheck:
     name: Check internal dependencies
     runs-on: ubuntu-latest
diff --git a/Makefile b/Makefile
@@ -24,6 +24,9 @@ IMAGE_REPO=nspccdev/neo-go
 
 DISABLE_NEOTEST_COVER=1
 
+ROOT_DIR:=$(dir $(realpath $(firstword $(MAKEFILE_LIST))))
+GOMODDIRS=$(dir $(shell find $(ROOT_DIR) -name go.mod))
+
 # All of the targets are phony here because we don't really use make dependency
 # tracking for files
 .PHONY: build $(BINARY) deps image docker/$(BINARY) image-latest image-push image-push-latest clean-cluster \
@@ -113,7 +116,9 @@ vet:
 	curl -L -o $@ https://github.com/nspcc-dev/.github/raw/master/.golangci.yml
 
 lint: .golangci.yml
-	@golangci-lint run
+	@for dir in $(GOMODDIRS); do \
+		(cd "$$dir" && golangci-lint run --config $(ROOT_DIR)/$< | sed -r "s,^,$$dir," | sed -r "s,^$(ROOT_DIR),,") \
+	done
 
 fmt:
 	@gofmt -l -w -s $$(find . -type f -name '*.go'| grep -v "/vendor/")
diff --git a/examples/engine/engine.go b/examples/engine/engine.go
@@ -4,25 +4,25 @@ import (
 	"github.com/nspcc-dev/neo-go/pkg/interop/runtime"
 )
 
-// NotifyScriptContainer sends runtime notification with script container hash
+// NotifyScriptContainer sends runtime notification with script container hash.
 func NotifyScriptContainer() {
 	tx := runtime.GetScriptContainer()
 	runtime.Notify("Tx", tx.Hash)
 }
 
-// NotifyCallingScriptHash sends runtime notification with calling script hash
+// NotifyCallingScriptHash sends runtime notification with calling script hash.
 func NotifyCallingScriptHash() {
 	callingScriptHash := runtime.GetCallingScriptHash()
 	runtime.Notify("Calling", callingScriptHash)
 }
 
-// NotifyExecutingScriptHash sends runtime notification about executing script hash
+// NotifyExecutingScriptHash sends runtime notification about executing script hash.
 func NotifyExecutingScriptHash() {
 	execScriptHash := runtime.GetExecutingScriptHash()
 	runtime.Notify("Executing", execScriptHash)
 }
 
-// NotifyEntryScriptHash sends notification about entry script hash
+// NotifyEntryScriptHash sends notification about entry script hash.
 func NotifyEntryScriptHash() {
 	entryScriptHash := runtime.GetEntryScriptHash()
 	runtime.Notify("Entry", entryScriptHash)
diff --git a/examples/iterator/iterator.go b/examples/iterator/iterator.go
@@ -7,7 +7,7 @@ import (
 )
 
 // _deploy primes contract's storage with some data to be used later.
-func _deploy(_ any, _ bool) {
+func _deploy(_ any, _ bool) { // nolint: unused
 	ctx := storage.GetContext() // RW context.
 	storage.Put(ctx, "foo1", "1")
 	storage.Put(ctx, "foo2", "2")
diff --git a/examples/nft-nd-nns/nns.go b/examples/nft-nd-nns/nns.go
@@ -60,7 +60,7 @@ const (
 // Other constants.
 const (
 	// defaultRegisterPrice is the default price for new domain registration.
-	defaultRegisterPrice = 10_0000_0000
+	defaultRegisterPrice = 10_0000_0000 // nolint: unused
 	// millisecondsInYear is amount of milliseconds per year.
 	millisecondsInYear = 365 * 24 * 3600 * 1000
 )
@@ -79,7 +79,7 @@ func Update(nef []byte, manifest string) {
 }
 
 // _deploy initializes defaults (total supply and registration price) on contract deploy.
-func _deploy(data any, isUpdate bool) {
+func _deploy(data any, isUpdate bool) { // nolint: unused
 	if isUpdate {
 		return
 	}
@@ -491,7 +491,7 @@ func getRecordKey(tokenId []byte, name string, typ RecordType) []byte {
 
 // isValid returns true if the provided address is a valid Uint160.
 func isValid(address interop.Hash160) bool {
-	return address != nil && len(address) == 20
+	return address != nil && len(address) == 20 // nolint: gosimple
 }
 
 // checkCommittee panics if the script container is not signed by the committee.
@@ -555,7 +555,7 @@ func splitAndCheck(name string, allowMultipleFragments bool) []string {
 	if l > 2 && !allowMultipleFragments {
 		return nil
 	}
-	for i := 0; i < l; i++ {
+	for i := range fragments {
 		if !checkFragment(fragments[i], i == l-1) {
 			return nil
 		}
diff --git a/examples/nft-nd-nns/nns_test.go b/examples/nft-nd-nns/nns_test.go
@@ -418,11 +418,9 @@ func testTokensOf(t *testing.T, c *neotest.ContractInvoker, result [][]byte, arg
 	}
 	require.NoError(t, err)
 	iter := s.Pop().Interop().Value().(*storage.Iterator)
-	arr := make([]stackitem.Item, 0, len(result))
 	for i := range result {
 		require.True(t, iter.Next())
 		require.Equal(t, result[i], iter.Value().Value())
-		arr = append(arr, stackitem.Make(result[i]))
 	}
 	require.False(t, iter.Next())
 }
diff --git a/examples/nft-nd/nft.go b/examples/nft-nd/nft.go
@@ -26,7 +26,7 @@ const (
 	totalSupplyPrefix = "s"
 	// balancePrefix contains map from addresses to balances.
 	balancePrefix = "b"
-	// accountPrefix contains map from address + token id to tokens
+	// accountPrefix contains map from address + token id to tokens.
 	accountPrefix = "a"
 	// tokenPrefix contains map from token id to it's owner.
 	tokenPrefix = "t"
diff --git a/examples/runtime/runtime.go b/examples/runtime/runtime.go
@@ -7,7 +7,7 @@ import (
 )
 
 var (
-	// Check if the invoker of the contract is the specified owner
+	// Check if the invoker of the contract is the specified owner.
 	owner = address.ToHash160("NbrUYaZgyhSkNoRo9ugRyEMdUZxrhkNaWB")
 )
 
@@ -22,7 +22,7 @@ func init() {
 
 // _deploy is called after contract deployment or update, it'll be called
 // in deployment transaction and if call update method of this contract.
-func _deploy(_ any, isUpdate bool) {
+func _deploy(_ any, isUpdate bool) { // nolint: unused
 	if isUpdate {
 		Log("_deploy method called after contract update")
 		return
diff --git a/examples/storage/storage.go b/examples/storage/storage.go
@@ -5,13 +5,13 @@ import (
 	"github.com/nspcc-dev/neo-go/pkg/interop/storage"
 )
 
-// ctx holds storage context for contract methods
+// ctx holds storage context for contract methods.
 var ctx storage.Context
 
 // defaultKey represents the default key.
 var defaultKey = []byte("default")
 
-// init inits storage context before any other contract method is called
+// init inits storage context before any other contract method is called.
 func init() {
 	ctx = storage.GetContext()
 }
diff --git a/examples/timer/timer.go b/examples/timer/timer.go
@@ -9,23 +9,23 @@ import (
 	"github.com/nspcc-dev/neo-go/pkg/interop/storage"
 )
 
-const defaultTicks = 3
+const defaultTicks = 3 // nolint: unused
 const mgmtKey = "mgmt"
 
 var (
-	// ctx holds storage context for contract methods
+	// ctx holds storage context for contract methods.
 	ctx storage.Context
-	// Check if the invoker of the contract is the specified owner
+	// Check if the invoker of the contract is the specified owner.
 	owner = address.ToHash160("NbrUYaZgyhSkNoRo9ugRyEMdUZxrhkNaWB")
-	// ticksKey is a storage key for ticks counter
+	// ticksKey is a storage key for ticks counter.
 	ticksKey = []byte("ticks")
 )
 
 func init() {
 	ctx = storage.GetContext()
 }
 
-func _deploy(_ any, isUpdate bool) {
+func _deploy(_ any, isUpdate bool) { // nolint: unused
 	if isUpdate {
 		ticksLeft := storage.Get(ctx, ticksKey).(int) + 1
 		storage.Put(ctx, ticksKey, ticksLeft)
diff --git a/examples/token/nep17/nep17.go b/examples/token/nep17/nep17.go
@@ -8,7 +8,7 @@ import (
 	"github.com/nspcc-dev/neo-go/pkg/interop/storage"
 )
 
-// Token holds all token info
+// Token holds all token info.
 type Token struct {
 	// Token name
 	Name string
@@ -35,17 +35,17 @@ func getIntFromDB(ctx storage.Context, key []byte) int {
 	return res
 }
 
-// GetSupply gets the token totalSupply value from VM storage
+// GetSupply gets the token totalSupply value from VM storage.
 func (t Token) GetSupply(ctx storage.Context) int {
 	return getIntFromDB(ctx, []byte(t.CirculationKey))
 }
 
-// BalanceOf gets the token balance of a specific address
+// BalanceOf gets the token balance of a specific address.
 func (t Token) BalanceOf(ctx storage.Context, holder []byte) int {
 	return getIntFromDB(ctx, holder)
 }
 
-// Transfer token from one user to another
+// Transfer token from one user to another.
 func (t Token) Transfer(ctx storage.Context, from, to interop.Hash160, amount int, data any) bool {
 	amountFrom := t.CanTransfer(ctx, from, to, amount)
 	if amountFrom == -1 {
@@ -74,7 +74,7 @@ func (t Token) Transfer(ctx storage.Context, from, to interop.Hash160, amount in
 	return true
 }
 
-// CanTransfer returns the amount it can transfer
+// CanTransfer returns the amount it can transfer.
 func (t Token) CanTransfer(ctx storage.Context, from []byte, to []byte, amount int) int {
 	if len(to) != 20 || !IsUsableAddress(from) {
 		return -1
@@ -94,10 +94,9 @@ func (t Token) CanTransfer(ctx storage.Context, from []byte, to []byte, amount i
 	return amountFrom
 }
 
-// IsUsableAddress checks if the sender is either the correct Neo address or SC address
+// IsUsableAddress checks if the sender is either the correct Neo address or SC address.
 func IsUsableAddress(addr []byte) bool {
 	if len(addr) == 20 {
-
 		if runtime.CheckWitness(addr) {
 			return true
 		}
@@ -112,13 +111,13 @@ func IsUsableAddress(addr []byte) bool {
 	return false
 }
 
-// Mint initial supply of tokens
+// Mint initial supply of tokens.
 func (t Token) Mint(ctx storage.Context, to interop.Hash160) bool {
 	if !IsUsableAddress(t.Owner) {
 		return false
 	}
 	minted := storage.Get(ctx, []byte("minted"))
-	if minted != nil && minted.(bool) == true {
+	if minted != nil && minted.(bool) {
 		return false
 	}
 
diff --git a/examples/token/token.go b/examples/token/token.go
@@ -19,7 +19,7 @@ var (
 )
 
 // init initializes Token Interface and storage context for the Smart
-// Contract to operate with
+// Contract to operate with.
 func init() {
 	token = nep17.Token{
 		Name:           "Awesome NEO Token",
@@ -32,32 +32,32 @@ func init() {
 	ctx = storage.GetContext()
 }
 
-// Symbol returns the token symbol
+// Symbol returns the token symbol.
 func Symbol() string {
 	return token.Symbol
 }
 
-// Decimals returns the token decimals
+// Decimals returns the token decimals.
 func Decimals() int {
 	return token.Decimals
 }
 
-// TotalSupply returns the token total supply value
+// TotalSupply returns the token total supply value.
 func TotalSupply() int {
 	return token.GetSupply(ctx)
 }
 
-// BalanceOf returns the amount of token on the specified address
+// BalanceOf returns the amount of token on the specified address.
 func BalanceOf(holder interop.Hash160) int {
 	return token.BalanceOf(ctx, holder)
 }
 
-// Transfer token from one user to another
+// Transfer token from one user to another.
 func Transfer(from interop.Hash160, to interop.Hash160, amount int, data any) bool {
 	return token.Transfer(ctx, from, to, amount, data)
 }
 
-// Mint initial supply of tokens
+// Mint initial supply of tokens.
 func Mint(to interop.Hash160) bool {
 	return token.Mint(ctx, to)
 }
diff --git a/examples/zkp/cubic_circuit/main.go b/examples/zkp/cubic_circuit/main.go
@@ -51,7 +51,7 @@ func (circuit *CubicCircuit) Define(api frontend.API) error {
 
 // main demonstrates how to build the proof and verify it with the help of gnark
 // library. Error handling omitted intentionally to simplify the example.
-func main() {
+func main() { // nolint: unused
 	var (
 		circuit    CubicCircuit
 		assignment = CubicCircuit{X: 3, Y: 35}
diff --git a/examples/zkp/cubic_circuit/main_test.go b/examples/zkp/cubic_circuit/main_test.go
@@ -315,7 +315,7 @@ func setup(t *testing.T, ccs constraint.ConstraintSystem, phase1ResponsePath str
 	// receive a []byte, deserialize it, add his contribution and send back to
 	// coordinator, like it is done in https://github.com/bnb-chain/zkbnb-setup
 	// for BN254 elliptic curve.
-	for i := range nContributionsPhase2 {
+	for range nContributionsPhase2 {
 		srs2.Contribute()
 	}
 
diff --git a/examples/zkp/xor_compat/verify.go b/examples/zkp/xor_compat/verify.go
@@ -43,7 +43,7 @@ var (
 // proving system and is taken from the
 // https://github.com/neo-project/neo/issues/2647#issuecomment-1002893109 without
 // changes. The verification process checks the following equality:
-// A * B = alpha * beta + sum(pub_input[i] * (beta * u_i(x) + alpha * v_i(x) + w_i(x)) / gamma) * gamma + C * delta
+// A * B = alpha * beta + sum(pub_input[i] * (beta * u_i(x) + alpha * v_i(x) + w_i(x)) / gamma) * gamma + C * delta.
 func VerifyProof(a []byte, b []byte, c []byte, publicInput [][]byte) bool {
 	alphaPoint := crypto.Bls12381Deserialize(alpha)
 	betaPoint := crypto.Bls12381Deserialize(beta)
@@ -68,11 +68,11 @@ func VerifyProof(a []byte, b []byte, c []byte, publicInput [][]byte) bool {
 		panic("error: inputlen or iclen")
 	}
 	icPoints := make([]crypto.Bls12381Point, iclen)
-	for i := 0; i < iclen; i++ {
+	for i := range icPoints {
 		icPoints[i] = crypto.Bls12381Deserialize(ic[i])
 	}
 	acc := icPoints[0]
-	for i := 0; i < inputlen; i++ {
+	for i := range publicInput {
 		scalar := publicInput[i] // 32-bytes LE field element.
 		temp := crypto.Bls12381Mul(icPoints[i+1], scalar, false)
 		acc = crypto.Bls12381Add(acc, temp)
diff --git a/pkg/interop/native/ledger/transaction_signer.go b/pkg/interop/native/ledger/transaction_signer.go
@@ -88,7 +88,7 @@ type WitnessCondition struct {
 // WitnessConditionType represents the type of rule-based witness condition.
 type WitnessConditionType byte
 
-// Various witness condition types
+// Various witness condition types.
 const (
 	// WitnessBoolean is a generic boolean condition.
 	WitnessBoolean WitnessConditionType = 0x00
diff --git a/pkg/interop/native/neo/neo.go b/pkg/interop/native/neo/neo.go
@@ -130,5 +130,4 @@ func GetAccountState(addr interop.Hash160) *AccountState {
 // GetCommitteeAddress represents `getCommitteeAddress` method of NEO native contract.
 func GetCommitteeAddress() interop.Hash160 {
 	return neogointernal.CallWithToken(Hash, "getCommitteeAddress", int(contract.ReadStates)).(interop.Hash160)
-
 }
diff --git a/scripts/go.mod b/scripts/go.mod
diff --git a/scripts/go.sum b/scripts/go.sum

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ import (`
`7`	`7`	`)`
`8`	`8`
`9`	`9`	`// _deploy primes contract's storage with some data to be used later.`
`10`		`-func _deploy(_ any, _ bool) {`
	`10`	`+func _deploy(_ any, _ bool) { // nolint: unused`
`11`	`11`	`ctx := storage.GetContext() // RW context.`
`12`	`12`	`storage.Put(ctx, "foo1", "1")`
`13`	`13`	`storage.Put(ctx, "foo2", "2")`
Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ const (`
`60`	`60`	`// Other constants.`
`61`	`61`	`const (`
`62`	`62`	`// defaultRegisterPrice is the default price for new domain registration.`
`63`		`- defaultRegisterPrice = 10_0000_0000`
	`63`	`+ defaultRegisterPrice = 10_0000_0000 // nolint: unused`
`64`	`64`	`// millisecondsInYear is amount of milliseconds per year.`
`65`	`65`	`millisecondsInYear = 365 * 24 * 3600 * 1000`
`66`	`66`	`)`
`@@ -79,7 +79,7 @@ func Update(nef []byte, manifest string) {`
`79`	`79`	`}`
`80`	`80`
`81`	`81`	`// _deploy initializes defaults (total supply and registration price) on contract deploy.`
`82`		`-func _deploy(data any, isUpdate bool) {`
	`82`	`+func _deploy(data any, isUpdate bool) { // nolint: unused`
`83`	`83`	`if isUpdate {`
`84`	`84`	`return`
`85`	`85`	`}`
`@@ -491,7 +491,7 @@ func getRecordKey(tokenId []byte, name string, typ RecordType) []byte {`
`491`	`491`
`492`	`492`	`// isValid returns true if the provided address is a valid Uint160.`
`493`	`493`	`func isValid(address interop.Hash160) bool {`
`494`		`- return address != nil && len(address) == 20`
	`494`	`+ return address != nil && len(address) == 20 // nolint: gosimple`
`495`	`495`	`}`
`496`	`496`
`497`	`497`	`// checkCommittee panics if the script container is not signed by the committee.`
`@@ -555,7 +555,7 @@ func splitAndCheck(name string, allowMultipleFragments bool) []string {`
`555`	`555`	`if l > 2 && !allowMultipleFragments {`
`556`	`556`	`return nil`
`557`	`557`	`}`
`558`		`- for i := 0; i < l; i++ {`
	`558`	`+ for i := range fragments {`
`559`	`559`	`if !checkFragment(fragments[i], i == l-1) {`
`560`	`560`	`return nil`
`561`	`561`	`}`
Original file line number	Diff line number	Diff line change
`@@ -418,11 +418,9 @@ func testTokensOf(t testing.T, c neotest.ContractInvoker, result [][]byte, arg`
`418`	`418`	`}`
`419`	`419`	`require.NoError(t, err)`
`420`	`420`	`iter := s.Pop().Interop().Value().(*storage.Iterator)`
`421`		`- arr := make([]stackitem.Item, 0, len(result))`
`422`	`421`	`for i := range result {`
`423`	`422`	`require.True(t, iter.Next())`
`424`	`423`	`require.Equal(t, result[i], iter.Value().Value())`
`425`		`- arr = append(arr, stackitem.Make(result[i]))`
`426`	`424`	`}`
`427`	`425`	`require.False(t, iter.Next())`
`428`	`426`	`}`
Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ import (`
`7`	`7`	`)`
`8`	`8`
`9`	`9`	`var (`
`10`		`- // Check if the invoker of the contract is the specified owner`
	`10`	`+ // Check if the invoker of the contract is the specified owner.`
`11`	`11`	`owner = address.ToHash160("NbrUYaZgyhSkNoRo9ugRyEMdUZxrhkNaWB")`
`12`	`12`	`)`
`13`	`13`
`@@ -22,7 +22,7 @@ func init() {`
`22`	`22`
`23`	`23`	`// _deploy is called after contract deployment or update, it'll be called`
`24`	`24`	`// in deployment transaction and if call update method of this contract.`
`25`		`-func _deploy(_ any, isUpdate bool) {`
	`25`	`+func _deploy(_ any, isUpdate bool) { // nolint: unused`
`26`	`26`	`if isUpdate {`
`27`	`27`	`Log("_deploy method called after contract update")`
`28`	`28`	`return`
Original file line number	Diff line number	Diff line change
`@@ -5,13 +5,13 @@ import (`
`5`	`5`	`"github.com/nspcc-dev/neo-go/pkg/interop/storage"`
`6`	`6`	`)`
`7`	`7`
`8`		`-// ctx holds storage context for contract methods`
	`8`	`+// ctx holds storage context for contract methods.`
`9`	`9`	`var ctx storage.Context`
`10`	`10`
`11`	`11`	`// defaultKey represents the default key.`
`12`	`12`	`var defaultKey = []byte("default")`
`13`	`13`
`14`		`-// init inits storage context before any other contract method is called`
	`14`	`+// init inits storage context before any other contract method is called.`
`15`	`15`	`func init() {`
`16`	`16`	`ctx = storage.GetContext()`
`17`	`17`	`}`