all: fix lint suggestions

Author: zchee

basic.go

@@ -593,10 +593,10 @@ type DocumentSelector []*DocumentFilter
 type MarkupKind string
 
 const (
-	// PlainText is supported as a content format
+	// PlainText is supported as a content format.
 	PlainText MarkupKind = "plaintext"
 
-	// Markdown is supported as a content format
+	// Markdown is supported as a content format.
 	Markdown MarkupKind = "markdown"
 )
 

client_gojay.go

@@ -18,6 +18,7 @@ import (
 )
 
 // Deliver implements jsonrpc2.Handler.
+//nolint:funlen,gocognit
 func (h clientHandler) Deliver(ctx context.Context, r *jsonrpc2.Request, delivered bool) bool {
 	if delivered {
 		return false

client_json.go

@@ -17,6 +17,7 @@ import (
 )
 
 // Deliver implements jsonrpc2.Handler.
+//nolint:funlen,gocognit
 func (h clientHandler) Deliver(ctx context.Context, r *jsonrpc2.Request, delivered bool) bool {
 	if delivered {
 		return false

errors.go

@@ -6,6 +6,7 @@ package protocol
 
 import (
 	"context"
+	"errors"
 
 	"go.uber.org/zap"
 
@@ -14,7 +15,8 @@ import (
 
 // ReplyError replies error message.
 func ReplyError(ctx context.Context, err error, req *jsonrpc2.Request) {
-	if _, ok := err.(*jsonrpc2.Error); !ok {
+	var jrpcErr *jsonrpc2.Error
+	if !errors.As(err, &jrpcErr) {
 		err = jsonrpc2.Errorf(jsonrpc2.ParseError, "%v", err)
 	}
 

general.go

@@ -67,7 +67,7 @@ type InitializeParams struct {
 	WorkspaceFolders []WorkspaceFolder `json:"workspaceFolders,omitempty"`
 }
 
-// WorkspaceClientCapabilitiesWorkspaceEdit capabilities specific to `WorkspaceEdit`s
+// WorkspaceClientCapabilitiesWorkspaceEdit capabilities specific to `WorkspaceEdit`s.
 type WorkspaceClientCapabilitiesWorkspaceEdit struct {
 	// DocumentChanges is the client supports versioned document changes in `WorkspaceEdit`s
 	DocumentChanges bool `json:"documentChanges,omitempty"`
@@ -170,7 +170,7 @@ type TextDocumentClientCapabilitiesSynchronization struct {
 	WillSaveWaitUntil bool `json:"willSaveWaitUntil,omitempty"`
 }
 
-// TextDocumentClientCapabilitiesCompletion Capabilities specific to the `textDocument/completion`
+// TextDocumentClientCapabilitiesCompletion Capabilities specific to the `textDocument/completion`.
 type TextDocumentClientCapabilitiesCompletion struct {
 	// Whether completion supports dynamic registration.
 	DynamicRegistration bool `json:"dynamicRegistration,omitempty"`
@@ -226,7 +226,7 @@ type TextDocumentClientCapabilitiesCompletionItem struct {
 	PreselectSupport bool `json:"preselectSupport,omitempty"`
 }
 
-// TextDocumentClientCapabilitiesHover capabilities specific to the `textDocument/hover`
+// TextDocumentClientCapabilitiesHover capabilities specific to the `textDocument/hover`.
 type TextDocumentClientCapabilitiesHover struct {
 	// DynamicRegistration whether hover supports dynamic registration.
 	DynamicRegistration bool `json:"dynamicRegistration,omitempty"`
@@ -236,7 +236,7 @@ type TextDocumentClientCapabilitiesHover struct {
 	ContentFormat []MarkupKind `json:"contentFormat,omitempty"`
 }
 
-// TextDocumentClientCapabilitiesSignatureHelp capabilities specific to the `textDocument/signatureHelp`
+// TextDocumentClientCapabilitiesSignatureHelp capabilities specific to the `textDocument/signatureHelp`.
 type TextDocumentClientCapabilitiesSignatureHelp struct {
 	// DynamicRegistration whether signature help supports dynamic registration.
 	DynamicRegistration bool `json:"dynamicRegistration,omitempty"`
@@ -266,19 +266,19 @@ type TextDocumentClientCapabilitiesParameterInformation struct {
 	LabelOffsetSupport bool `json:"labelOffsetSupport,omitempty"`
 }
 
-// TextDocumentClientCapabilitiesReferences capabilities specific to the `textDocument/references`
+// TextDocumentClientCapabilitiesReferences capabilities specific to the `textDocument/references`.
 type TextDocumentClientCapabilitiesReferences struct {
 	// DynamicRegistration whether references supports dynamic registration.
 	DynamicRegistration bool `json:"dynamicRegistration,omitempty"`
 }
 
-// TextDocumentClientCapabilitiesDocumentHighlight capabilities specific to the `textDocument/documentHighlight`
+// TextDocumentClientCapabilitiesDocumentHighlight capabilities specific to the `textDocument/documentHighlight`.
 type TextDocumentClientCapabilitiesDocumentHighlight struct {
 	// DynamicRegistration Whether document highlight supports dynamic registration.
 	DynamicRegistration bool `json:"dynamicRegistration,omitempty"`
 }
 
-// TextDocumentClientCapabilitiesDocumentSymbol capabilities specific to the `textDocument/documentSymbol`
+// TextDocumentClientCapabilitiesDocumentSymbol capabilities specific to the `textDocument/documentSymbol`.
 type TextDocumentClientCapabilitiesDocumentSymbol struct {
 	// DynamicRegistration whether document symbol supports dynamic registration.
 	DynamicRegistration bool `json:"dynamicRegistration,omitempty"`
@@ -290,25 +290,25 @@ type TextDocumentClientCapabilitiesDocumentSymbol struct {
 	HierarchicalDocumentSymbolSupport bool `json:"hierarchicalDocumentSymbolSupport,omitempty"`
 }
 
-// TextDocumentClientCapabilitiesFormatting capabilities specific to the `textDocument/formatting`
+// TextDocumentClientCapabilitiesFormatting capabilities specific to the `textDocument/formatting`.
 type TextDocumentClientCapabilitiesFormatting struct {
 	// DynamicRegistration whether formatting supports dynamic registration.
 	DynamicRegistration bool `json:"dynamicRegistration,omitempty"`
 }
 
-// TextDocumentClientCapabilitiesRangeFormatting capabilities specific to the `textDocument/rangeFormatting`
+// TextDocumentClientCapabilitiesRangeFormatting capabilities specific to the `textDocument/rangeFormatting`.
 type TextDocumentClientCapabilitiesRangeFormatting struct {
 	// DynamicRegistration whether range formatting supports dynamic registration.
 	DynamicRegistration bool `json:"dynamicRegistration,omitempty"`
 }
 
-// TextDocumentClientCapabilitiesOnTypeFormatting Capabilities specific to the `textDocument/onTypeFormatting`
+// TextDocumentClientCapabilitiesOnTypeFormatting Capabilities specific to the `textDocument/onTypeFormatting`.
 type TextDocumentClientCapabilitiesOnTypeFormatting struct {
 	// DynamicRegistration whether on type formatting supports dynamic registration.
 	DynamicRegistration bool `json:"dynamicRegistration,omitempty"`
 }
 
-// TextDocumentClientCapabilitiesDeclaration capabilities specific to the `textDocument/declaration`
+// TextDocumentClientCapabilitiesDeclaration capabilities specific to the `textDocument/declaration`.
 type TextDocumentClientCapabilitiesDeclaration struct {
 	// DynamicRegistration whether declaration supports dynamic registration. If this is set to `true`
 	// the client supports the new `(TextDocumentRegistrationOptions & StaticRegistrationOptions)`
@@ -323,7 +323,7 @@ type TextDocumentClientCapabilitiesDeclaration struct {
 
 // TextDocumentClientCapabilitiesDefinition capabilities specific to the `textDocument/definition`.
 //
-// Since 3.14.0
+// Since 3.14.0.
 type TextDocumentClientCapabilitiesDefinition struct {
 	// DynamicRegistration whether definition supports dynamic registration.
 	DynamicRegistration bool `json:"dynamicRegistration,omitempty"`
@@ -334,7 +334,7 @@ type TextDocumentClientCapabilitiesDefinition struct {
 
 // TextDocumentClientCapabilitiesTypeDefinition capabilities specific to the `textDocument/typeDefinition`
 //
-// Since 3.6.0
+// Since 3.6.0.
 type TextDocumentClientCapabilitiesTypeDefinition struct {
 	// DynamicRegistration whether typeDefinition supports dynamic registration. If this is set to `true`
 	// the client supports the new `(TextDocumentRegistrationOptions & StaticRegistrationOptions)`
@@ -349,7 +349,7 @@ type TextDocumentClientCapabilitiesTypeDefinition struct {
 
 // TextDocumentClientCapabilitiesImplementation capabilities specific to the `textDocument/implementation`.
 //
-// Since 3.6.0
+// Since 3.6.0.
 type TextDocumentClientCapabilitiesImplementation struct {
 	// DynamicRegistration whether implementation supports dynamic registration. If this is set to `true`
 	// the client supports the new `(TextDocumentRegistrationOptions & StaticRegistrationOptions)`
@@ -362,7 +362,7 @@ type TextDocumentClientCapabilitiesImplementation struct {
 	LinkSupport bool `json:"linkSupport,omitempty"`
 }
 
-// TextDocumentClientCapabilitiesCodeAction capabilities specific to the `textDocument/codeAction`
+// TextDocumentClientCapabilitiesCodeAction capabilities specific to the `textDocument/codeAction`.
 type TextDocumentClientCapabilitiesCodeAction struct {
 	// DynamicRegistration whether code action supports dynamic registration.
 	DynamicRegistration bool `json:"dynamicRegistration,omitempty"`
@@ -389,13 +389,13 @@ type TextDocumentClientCapabilitiesCodeActionKind struct {
 	ValueSet []CodeActionKind `json:"valueSet"`
 }
 
-// TextDocumentClientCapabilitiesCodeLens capabilities specific to the `textDocument/codeLens`
+// TextDocumentClientCapabilitiesCodeLens capabilities specific to the `textDocument/codeLens`.
 type TextDocumentClientCapabilitiesCodeLens struct {
 	// DynamicRegistration Whether code lens supports dynamic registration.
 	DynamicRegistration bool `json:"dynamicRegistration,omitempty"`
 }
 
-// TextDocumentClientCapabilitiesDocumentLink capabilities specific to the `textDocument/documentLink`
+// TextDocumentClientCapabilitiesDocumentLink capabilities specific to the `textDocument/documentLink`.
 type TextDocumentClientCapabilitiesDocumentLink struct {
 	// DynamicRegistration whether document link supports dynamic registration.
 	DynamicRegistration bool `json:"dynamicRegistration,omitempty"`
@@ -404,15 +404,15 @@ type TextDocumentClientCapabilitiesDocumentLink struct {
 // TextDocumentClientCapabilitiesColorProvider capabilities specific to the `textDocument/documentColor` and the
 // `textDocument/colorPresentation` request.
 //
-// Since 3.6.0
+// Since 3.6.0.
 type TextDocumentClientCapabilitiesColorProvider struct {
 	// DynamicRegistration whether colorProvider supports dynamic registration. If this is set to `true`
 	// the client supports the new `(ColorProviderOptions & TextDocumentRegistrationOptions & StaticRegistrationOptions)`
 	// return value for the corresponding server capability as well.
 	DynamicRegistration bool `json:"dynamicRegistration,omitempty"`
 }
 
-// TextDocumentClientCapabilitiesRename capabilities specific to the `textDocument/rename`
+// TextDocumentClientCapabilitiesRename capabilities specific to the `textDocument/rename`.
 type TextDocumentClientCapabilitiesRename struct {
 	// DynamicRegistration whether rename supports dynamic registration.
 	DynamicRegistration bool `json:"dynamicRegistration,omitempty"`
@@ -433,7 +433,7 @@ type TextDocumentClientCapabilitiesPublishDiagnostics struct {
 
 // TextDocumentClientCapabilitiesFoldingRange capabilities specific to `textDocument/foldingRange` requests.
 //
-// Since 3.10.0
+// Since 3.10.0.
 type TextDocumentClientCapabilitiesFoldingRange struct {
 	// DynamicRegistration whether implementation supports dynamic registration for folding range providers. If this is set to `true`
 	// the client supports the new `(FoldingRangeProviderOptions & TextDocumentRegistrationOptions & StaticRegistrationOptions)`
@@ -695,14 +695,14 @@ type StaticRegistrationOptions struct {
 	ID string `json:"id,omitempty"`
 }
 
-// ServerCapabilitiesWorkspace specific server capabilities
+// ServerCapabilitiesWorkspace specific server capabilities.
 type ServerCapabilitiesWorkspace struct {
 	WorkspaceFolders *ServerCapabilitiesWorkspaceFolders `json:"workspaceFolders,omitempty"`
 }
 
 // ServerCapabilitiesWorkspaceFolders is the server supports workspace folder.
 //
-// Since 3.6.0
+// Since 3.6.0.
 type ServerCapabilitiesWorkspaceFolders struct {
 	// Supported is the server has support for workspace folders
 	Supported bool `json:"supported,omitempty"`

general_gojay.go

@@ -221,7 +221,7 @@ func (v SymbolKinds) MarshalJSONArray(enc *gojay.Encoder) {
 // IsNil implements gojay.MarshalerJSONArray.
 func (v SymbolKinds) IsNil() bool { return len(v) == 0 }
 
-// UnmarshalJSONArray decodes JSON array elements into slice
+// UnmarshalJSONArray decodes JSON array elements into slice.
 func (v *SymbolKinds) UnmarshalJSONArray(dec *gojay.Decoder) error {
 	var value SymbolKind
 	if err := dec.Float64((*float64)(&value)); err != nil {
@@ -492,7 +492,7 @@ func (v MarkupKinds) MarshalJSONArray(enc *gojay.Encoder) {
 // IsNil implements gojay.MarshalerJSONArray.
 func (v MarkupKinds) IsNil() bool { return len(v) == 0 }
 
-// UnmarshalJSONArray decodes JSON array elements into slice
+// UnmarshalJSONArray decodes JSON array elements into slice.
 func (v *MarkupKinds) UnmarshalJSONArray(dec *gojay.Decoder) error {
 	var value MarkupKind
 	if err := dec.String((*string)(&value)); err != nil {
@@ -1236,6 +1236,7 @@ func (v *TextDocumentClientCapabilities) MarshalJSONObject(enc *gojay.Encoder) {
 func (v *TextDocumentClientCapabilities) IsNil() bool { return v == nil }
 
 // UnmarshalJSONObject implements gojay's UnmarshalerJSONObject.
+//nolint:funlen,gocognit
 func (v *TextDocumentClientCapabilities) UnmarshalJSONObject(dec *gojay.Decoder, k string) error {
 	switch k {
 	case keySynchronization:
@@ -1887,6 +1888,7 @@ func (v *ServerCapabilities) MarshalJSONObject(enc *gojay.Encoder) {
 func (v *ServerCapabilities) IsNil() bool { return v == nil }
 
 // UnmarshalJSONObject implements gojay's UnmarshalerJSONObject.
+//nolint:funlen
 func (v *ServerCapabilities) UnmarshalJSONObject(dec *gojay.Decoder, k string) error {
 	switch k {
 	case keyTextDocumentSync:

language.go

@@ -395,7 +395,7 @@ type SignatureHelpRegistrationOptions struct {
 	TriggerCharacters []string `json:"triggerCharacters,omitempty"`
 }
 
-// ReferenceParams params of Find References Request
+// ReferenceParams params of Find References Request.
 type ReferenceParams struct {
 	TextDocumentPositionParams
 
@@ -657,12 +657,12 @@ type CodeActionParams struct {
 // to a default value when unknown.
 type CodeActionKind string
 
-// A set of predefined code action kinds
+// A set of predefined code action kinds.
 const (
-	// QuickFix base kind for quickfix actions: 'quickfix'
+	// QuickFix base kind for quickfix actions: 'quickfix'.
 	QuickFix CodeActionKind = "quickfix"
 
-	// Refactor base kind for refactoring actions: 'refactor'
+	// Refactor base kind for refactoring actions: 'refactor'.
 	Refactor CodeActionKind = "refactor"
 
 	// RefactorExtract base kind for refactoring extraction actions: 'refactor.extract'
@@ -703,7 +703,7 @@ const (
 	// Source code actions apply to the entire file.
 	Source CodeActionKind = "source"
 
-	// SourceOrganizeImports base kind for an organize imports source action: `source.organizeImports`
+	// SourceOrganizeImports base kind for an organize imports source action: `source.organizeImports`.
 	SourceOrganizeImports CodeActionKind = "source.organizeImports"
 )
 
@@ -957,7 +957,7 @@ const (
 
 // FoldingRange capabilities specific to `textDocument/foldingRange` requests.
 //
-// Since 3.10.0
+// Since 3.10.0.
 type FoldingRange struct {
 	// StartLine is the zero-based line number from where the folded range starts.
 	StartLine float64 `json:"startLine"`

server.go

@@ -323,7 +323,7 @@ func (s *server) CodeLensResolve(ctx context.Context, params *CodeLens) (result
 // Clients can use the result to
 //
 // - modify a color reference.
-// - show in a color picker and let users pick one of the presentations
+// - show in a color picker and let users pick one of the presentations.
 func (s *server) ColorPresentation(ctx context.Context, params *ColorPresentationParams) (result []ColorPresentation, err error) {
 	err = s.Conn.Call(ctx, MethodTextDocumentColorPresentation, params, &result)
 
@@ -418,7 +418,7 @@ func (s *server) DidChangeWatchedFiles(ctx context.Context, params *DidChangeWat
 // or if the server has registered itself to receive this notification.
 // To register for the workspace/didChangeWorkspaceFolders send a client/registerCapability request from the server to the client.
 //
-// The registration parameter must have a registrations item of the following form, where id is a unique id used to unregister the capability (the example uses a UUID)
+// The registration parameter must have a registrations item of the following form, where id is a unique id used to unregister the capability (the example uses a UUID).
 func (s *server) DidChangeWorkspaceFolders(ctx context.Context, params *DidChangeWorkspaceFoldersParams) (err error) {
 	err = s.Conn.Notify(ctx, MethodWorkspaceDidChangeWorkspaceFolders, params)
 	return
@@ -467,7 +467,7 @@ func (s *server) DidSave(ctx context.Context, params *DidSaveTextDocumentParams)
 // For example:
 //
 // - Color boxes showing the actual color next to the reference
-// - Show a color picker when a color reference is edited
+// - Show a color picker when a color reference is edited.
 func (s *server) DocumentColor(ctx context.Context, params *DocumentColorParams) (result []ColorInformation, err error) {
 	err = s.Conn.Call(ctx, MethodTextDocumentDocumentColor, params, &result)
 

server_gojay.go

@@ -18,6 +18,7 @@ import (
 )
 
 // Deliver implements jsonrpc2.Handler.
+//nolint:funlen,gocognit
 func (h serverHandler) Deliver(ctx context.Context, r *jsonrpc2.Request, delivered bool) bool {
 	if delivered {
 		return false

server_json.go

@@ -17,6 +17,7 @@ import (
 )
 
 // Deliver implements jsonrpc2.Handler.
+//nolint:funlen,gocognit
 func (h serverHandler) Deliver(ctx context.Context, r *jsonrpc2.Request, delivered bool) bool {
 	if delivered {
 		return false