Merge pull request #3592 from nspcc-dev/org-linter

Linter: adapt org-wide linter

.github/workflows/tests.yml

@@ -15,19 +15,7 @@ on:
 jobs:
   lint:
     name: Lint
-    runs-on: ubuntu-latest
-
-    steps:
-      - uses: actions/checkout@v4
-      - uses: actions/setup-go@v5
-        with:
-          go-version-file: 'go.mod'
-      - name: golangci-lint
-        uses: golangci/golangci-lint-action@v4
-        with:
-          version: latest
-          skip-pkg-cache: true # golangci-lint can't work with this cache enabled, ref. https://github.com/golangci/golangci-lint-action/issues/135.
-
+    uses: nspcc-dev/.github/.github/workflows/go-linter.yml@master
   gomodcheck:
     name: Check internal dependencies
     runs-on: ubuntu-latest

.gitignore

@@ -55,3 +55,6 @@ testdata/
 pkg/vm/testdata/fuzz
 !pkg/vm/testdata
 !pkg/wallet/testdata
+
+# Linter
+.golangci.yml

Makefile

@@ -109,7 +109,10 @@ test:
 vet:
 	@go vet ./...
 
-lint:
+.golangci.yml:
+	curl -L -o $@ https://github.com/nspcc-dev/.github/raw/master/.golangci.yml
+
+lint: .golangci.yml
 	@golangci-lint run
 
 fmt:

cli/cmdargs/parser.go

@@ -224,6 +224,7 @@ func parseCosigner(c string) (transaction.Signer, error) {
 
 				res.AllowedGroups = append(res.AllowedGroups, pub)
 			}
+		default:
 		}
 	}
 	return res, nil

pkg/compiler/codegen.go

@@ -946,6 +946,7 @@ func (c *codegen) Visit(node ast.Node) ast.Visitor {
 					expectedLen = 20
 				case smartcontract.Hash256Type:
 					expectedLen = 32
+				default:
 				}
 				if expectedLen != -1 && expectedLen != len(n.Elts) {
 					c.prog.Err = fmt.Errorf("%s type must have size %d", tn.Obj().Name(), expectedLen)
@@ -1225,10 +1226,10 @@ func (c *codegen) Visit(node ast.Node) ast.Visitor {
 		case token.CONTINUE:
 			post := c.getLabelOffset(labelPost, label)
 			emit.Jmp(c.prog.BinWriter, opcode.JMPL, post)
+		default:
+			return nil
 		}
 
-		return nil
-
 	case *ast.LabeledStmt:
 		c.nextLabel = n.Label.Name
 
@@ -1722,6 +1723,7 @@ func getJumpForToken(tok token.Token, typ types.Type) (opcode.Opcode, bool) {
 			}
 			return opcode.JMPNEL, true
 		}
+	default:
 	}
 	return 0, false
 }
@@ -2393,6 +2395,7 @@ func (c *codegen) writeJumps(b []byte) ([]byte, error) {
 				return nil, fmt.Errorf("func '%s' has %d local variables (maximum is 255)", info.name, info.count)
 			}
 			b[nextIP-2] = byte(info.count)
+		default:
 		}
 	}
 
@@ -2502,6 +2505,7 @@ func removeNOPs(b []byte, nopOffsets []int, sequencePoints map[string][]DebugSeq
 			finallyOffset := int(int32(binary.LittleEndian.Uint32(arg)))
 			finallyOffset += calcOffsetCorrection(ip, ip+finallyOffset, nopOffsets)
 			binary.LittleEndian.PutUint32(arg, uint32(finallyOffset))
+		default:
 		}
 	}
 

pkg/compiler/inline_test.go

@@ -47,6 +47,7 @@ func checkInstrCount(t *testing.T, src string, expectedSSlotCount, expectedCall,
 			if ctx.IP() == mainStart && expectedLocalsMain >= 0 {
 				require.Equal(t, expectedLocalsMain, int(param[0]))
 			}
+		default:
 		}
 		if ctx.IP() == ctx.LenInstr() {
 			break

pkg/consensus/recovery_message.go

@@ -180,6 +180,7 @@ func (m *recoveryMessage) AddPayload(p dbft.ConsensusPayload[util.Uint256]) {
 			Signature:        p.GetCommit().(*commit).signature,
 			InvocationScript: p.(*Payload).Witness.InvocationScript,
 		})
+	default:
 	}
 }
 

pkg/core/native/designate.go

@@ -272,6 +272,7 @@ func (s *Designate) notifyRoleChanged(v *roleData, r noderoles.Role) {
 		if s.StateRootService != nil {
 			s.StateRootService.UpdateStateValidators(v.height, v.nodes.Copy())
 		}
+	default:
 	}
 }
 

pkg/core/native/native_test/cryptolib_test.go

@@ -468,5 +468,5 @@ func TestVerifyGroth16Proof(t *testing.T) {
 	publicWitness := make([]byte, fr.Bytes)
 
 	// Verify.
-	validatorInvoker.Invoke(t, true, "verifyProof", argA, argB, argC, []interface{}{publicWitness})
+	validatorInvoker.Invoke(t, true, "verifyProof", argA, argB, argC, []any{publicWitness})
 }

pkg/neorpc/rpcevent/filter.go

@@ -89,6 +89,7 @@ func Matches(f Comparator, r Container) bool {
 			}
 		}
 		return senderOk && signerOK && typeOk
+	default:
+		return false
 	}
-	return false
 }

pkg/network/server.go

@@ -1446,6 +1446,7 @@ func (s *Server) handleMessage(peer Peer, msg *Message) error {
 			return s.handlePong(peer, pong)
 		case CMDVersion, CMDVerack:
 			return fmt.Errorf("received '%s' after the handshake", msg.Command.String())
+		default:
 		}
 	} else {
 		switch msg.Command {

pkg/network/server_test.go

@@ -536,6 +536,7 @@ func (s *Server) testHandleGetData(t *testing.T, invType payload.InventoryType,
 		case CMDNotFound:
 			require.Equal(t, notFound, msg.Payload.(*payload.Inventory).Hashes)
 			recvNotFound.Store(true)
+		default:
 		}
 	}
 
@@ -818,6 +819,7 @@ func TestHandleGetMPTData(t *testing.T) {
 			case CMDMPTData:
 				require.Equal(t, found, msg.Payload)
 				recvResponse.Store(true)
+			default:
 			}
 		}
 		hs := []util.Uint256{r1, r2}

pkg/rpcclient/nns/contract_test.go

@@ -582,18 +582,18 @@ func TestSetAdmin(t *testing.T) {
 	testCases := []struct {
 		name     string
 		setup    func()
-		testFunc func() (interface{}, error)
-		want     interface{}
+		testFunc func() (any, error)
+		want     any
 		wantErr  bool
 	}{
 		{
 			name: "SetAdmin - Error",
 			setup: func() {
 				ta.err = errors.New("test error")
 			},
-			testFunc: func() (interface{}, error) {
+			testFunc: func() (any, error) {
 				txh, vub, err := c.SetAdmin(name, admin)
-				return []interface{}{txh, vub}, err
+				return []any{txh, vub}, err
 			},
 			wantErr: true,
 		},
@@ -604,19 +604,19 @@ func TestSetAdmin(t *testing.T) {
 				ta.txh = txhMock
 				ta.vub = 42
 			},
-			testFunc: func() (interface{}, error) {
+			testFunc: func() (any, error) {
 				txh, vub, err := c.SetAdmin(name, admin)
-				return []interface{}{txh, vub}, err
+				return []any{txh, vub}, err
 			},
-			want: []interface{}{txhMock, uint32(42)},
+			want: []any{txhMock, uint32(42)},
 		},
 		{
 			name: "SetAdminTransaction - Success",
 			setup: func() {
 				ta.err = nil
 				ta.tx = txMock
 			},
-			testFunc: func() (interface{}, error) {
+			testFunc: func() (any, error) {
 				return c.SetAdminTransaction(name, admin)
 			},
 			want: txMock,
@@ -626,7 +626,7 @@ func TestSetAdmin(t *testing.T) {
 			setup: func() {
 				ta.err = errors.New("test error")
 			},
-			testFunc: func() (interface{}, error) {
+			testFunc: func() (any, error) {
 				return c.SetAdminTransaction(name, admin)
 			},
 			wantErr: true,
@@ -637,7 +637,7 @@ func TestSetAdmin(t *testing.T) {
 				ta.err = nil
 				ta.tx = txMock
 			},
-			testFunc: func() (interface{}, error) {
+			testFunc: func() (any, error) {
 				return c.SetAdminUnsigned(name, admin)
 			},
 			want: txMock,
@@ -647,7 +647,7 @@ func TestSetAdmin(t *testing.T) {
 			setup: func() {
 				ta.err = errors.New("test error")
 			},
-			testFunc: func() (interface{}, error) {
+			testFunc: func() (any, error) {
 				return c.SetAdminUnsigned(name, admin)
 			},
 			wantErr: true,
@@ -681,18 +681,18 @@ func TestSetRecord(t *testing.T) {
 	testCases := []struct {
 		name     string
 		setup    func()
-		testFunc func() (interface{}, error)
-		want     interface{}
+		testFunc func() (any, error)
+		want     any
 		wantErr  bool
 	}{
 		{
 			name: "SetRecord - Error",
 			setup: func() {
 				ta.err = errors.New("test error")
 			},
-			testFunc: func() (interface{}, error) {
+			testFunc: func() (any, error) {
 				txh, vub, err := c.SetRecord(name, typev, data)
-				return []interface{}{txh, vub}, err
+				return []any{txh, vub}, err
 			},
 			wantErr: true,
 		},
@@ -703,19 +703,19 @@ func TestSetRecord(t *testing.T) {
 				ta.txh = txhMock
 				ta.vub = 42
 			},
-			testFunc: func() (interface{}, error) {
+			testFunc: func() (any, error) {
 				txh, vub, err := c.SetRecord(name, typev, data)
-				return []interface{}{txh, vub}, err
+				return []any{txh, vub}, err
 			},
-			want: []interface{}{txhMock, uint32(42)},
+			want: []any{txhMock, uint32(42)},
 		},
 		{
 			name: "SetRecordTransaction - Success",
 			setup: func() {
 				ta.err = nil
 				ta.tx = txMock
 			},
-			testFunc: func() (interface{}, error) {
+			testFunc: func() (any, error) {
 				return c.SetRecordTransaction(name, typev, data)
 			},
 			want: txMock,
@@ -725,7 +725,7 @@ func TestSetRecord(t *testing.T) {
 			setup: func() {
 				ta.err = errors.New("test error")
 			},
-			testFunc: func() (interface{}, error) {
+			testFunc: func() (any, error) {
 				return c.SetRecordTransaction(name, typev, data)
 			},
 			wantErr: true,
@@ -736,7 +736,7 @@ func TestSetRecord(t *testing.T) {
 				ta.err = nil
 				ta.tx = txMock
 			},
-			testFunc: func() (interface{}, error) {
+			testFunc: func() (any, error) {
 				return c.SetRecordUnsigned(name, typev, data)
 			},
 			want: txMock,
@@ -746,7 +746,7 @@ func TestSetRecord(t *testing.T) {
 			setup: func() {
 				ta.err = errors.New("test error")
 			},
-			testFunc: func() (interface{}, error) {
+			testFunc: func() (any, error) {
 				return c.SetRecordUnsigned(name, typev, data)
 			},
 			wantErr: true,

pkg/services/notary/core_test.go

@@ -232,6 +232,7 @@ func TestNotary(t *testing.T) {
 				script, err = smartcontract.CreateMultiSigRedeemScript(requesters[i].m, pubs)
 				require.NoError(t, err)
 				nKeys += uint8(len(requesters[i].accounts))
+			default:
 			}
 			signers[i] = transaction.Signer{
 				Account: hash.Hash160(script),
@@ -294,6 +295,7 @@ func TestNotary(t *testing.T) {
 				nSigs++
 			case notary.MultiSignature:
 				nSigs += r.m
+			default:
 			}
 		}
 		nSigners := len(requesters) + 1