refactor: modernize code with Go 1.18 idioms

Replace interface{} with the any type alias throughout the codebase,
as recommended since Go 1.18. Also improve logging by replacing
the Debugf method with more structured DebugJSON calls and removing
redundant newlines in format strings.

Author: nametake

golangci-lint.go

@@ -3,11 +3,11 @@ package main
 import "strings"
 
 type Issue struct {
-	FromLinter  string      `json:"FromLinter"`
-	Text        string      `json:"Text"`
-	Severity    string      `json:"Severity"`
-	SourceLines []string    `json:"SourceLines"`
-	Replacement interface{} `json:"Replacement"`
+	FromLinter  string   `json:"FromLinter"`
+	Text        string   `json:"Text"`
+	Severity    string   `json:"Severity"`
+	SourceLines []string `json:"SourceLines"`
+	Replacement any      `json:"Replacement"`
 	Pos         struct {
 		Filename string `json:"Filename"`
 		Offset   int    `json:"Offset"`

handler.go

@@ -64,7 +64,8 @@ func (h *langHandler) lint(uri DocumentURI) ([]Diagnostic, error) {
 	} else {
 		cmd.Dir = dir
 	}
-	h.logger.Debugf("golangci-lint-langserver: golingci-lint cmd: %v\n", cmd)
+
+	h.logger.DebugJSON("golangci-lint-langserver: golingci-lint cmd:", cmd.Args)
 
 	b, err := cmd.Output()
 	if err == nil {
@@ -151,7 +152,7 @@ func (h *langHandler) linter() {
 	}
 }
 
-func (h *langHandler) handle(ctx context.Context, conn *jsonrpc2.Conn, req *jsonrpc2.Request) (result interface{}, err error) {
+func (h *langHandler) handle(ctx context.Context, conn *jsonrpc2.Conn, req *jsonrpc2.Request) (result any, err error) {
 	h.logger.DebugJSON("golangci-lint-langserver: request:", req)
 
 	switch req.Method {
@@ -176,7 +177,7 @@ func (h *langHandler) handle(ctx context.Context, conn *jsonrpc2.Conn, req *json
 	return nil, &jsonrpc2.Error{Code: jsonrpc2.CodeMethodNotFound, Message: fmt.Sprintf("method not supported: %s", req.Method)}
 }
 
-func (h *langHandler) handleInitialize(_ context.Context, conn *jsonrpc2.Conn, req *jsonrpc2.Request) (result interface{}, err error) {
+func (h *langHandler) handleInitialize(_ context.Context, conn *jsonrpc2.Conn, req *jsonrpc2.Request) (result any, err error) {
 	var params InitializeParams
 	if err := json.Unmarshal(*req.Params, &params); err != nil {
 		return nil, err
@@ -198,13 +199,13 @@ func (h *langHandler) handleInitialize(_ context.Context, conn *jsonrpc2.Conn, r
 	}, nil
 }
 
-func (h *langHandler) handleShutdown(_ context.Context, _ *jsonrpc2.Conn, _ *jsonrpc2.Request) (result interface{}, err error) {
+func (h *langHandler) handleShutdown(_ context.Context, _ *jsonrpc2.Conn, _ *jsonrpc2.Request) (result any, err error) {
 	close(h.request)
 
 	return nil, nil
 }
 
-func (h *langHandler) handleTextDocumentDidOpen(_ context.Context, _ *jsonrpc2.Conn, req *jsonrpc2.Request) (result interface{}, err error) {
+func (h *langHandler) handleTextDocumentDidOpen(_ context.Context, _ *jsonrpc2.Conn, req *jsonrpc2.Request) (result any, err error) {
 	var params DidOpenTextDocumentParams
 	if err := json.Unmarshal(*req.Params, &params); err != nil {
 		return nil, err
@@ -215,15 +216,15 @@ func (h *langHandler) handleTextDocumentDidOpen(_ context.Context, _ *jsonrpc2.C
 	return nil, nil
 }
 
-func (h *langHandler) handleTextDocumentDidClose(_ context.Context, _ *jsonrpc2.Conn, _ *jsonrpc2.Request) (result interface{}, err error) {
+func (h *langHandler) handleTextDocumentDidClose(_ context.Context, _ *jsonrpc2.Conn, _ *jsonrpc2.Request) (result any, err error) {
 	return nil, nil
 }
 
-func (h *langHandler) handleTextDocumentDidChange(_ context.Context, _ *jsonrpc2.Conn, _ *jsonrpc2.Request) (result interface{}, err error) {
+func (h *langHandler) handleTextDocumentDidChange(_ context.Context, _ *jsonrpc2.Conn, _ *jsonrpc2.Request) (result any, err error) {
 	return nil, nil
 }
 
-func (h *langHandler) handleTextDocumentDidSave(_ context.Context, _ *jsonrpc2.Conn, req *jsonrpc2.Request) (result interface{}, err error) {
+func (h *langHandler) handleTextDocumentDidSave(_ context.Context, _ *jsonrpc2.Conn, req *jsonrpc2.Request) (result any, err error) {
 	var params DidSaveTextDocumentParams
 	if err := json.Unmarshal(*req.Params, &params); err != nil {
 		return nil, err
@@ -234,6 +235,6 @@ func (h *langHandler) handleTextDocumentDidSave(_ context.Context, _ *jsonrpc2.C
 	return nil, nil
 }
 
-func (h *langHandler) handlerWorkspaceDidChangeConfiguration(_ context.Context, _ *jsonrpc2.Conn, req *jsonrpc2.Request) (result interface{}, err error) {
+func (h *langHandler) handlerWorkspaceDidChangeConfiguration(_ context.Context, _ *jsonrpc2.Conn, req *jsonrpc2.Request) (result any, err error) {
 	return nil, nil
 }

logger.go

@@ -9,9 +9,8 @@ import (
 var _ logger = (*stdLogger)(nil)
 
 type logger interface {
-	Printf(format string, args ...interface{})
-	Debugf(format string, args ...interface{})
-	DebugJSON(label string, arg interface{})
+	Printf(format string, args ...any)
+	DebugJSON(label string, arg any)
 }
 
 type stdLogger struct {
@@ -26,15 +25,11 @@ func newStdLogger(debug bool) *stdLogger {
 	}
 }
 
-func (l *stdLogger) Printf(format string, args ...interface{}) {
+func (l *stdLogger) Printf(format string, args ...any) {
 	l.stderr.Printf(format, args...)
 }
 
-func (l *stdLogger) Debugf(format string, args ...interface{}) {
-	l.stderr.Printf(format, args...)
-}
-
-func (l *stdLogger) DebugJSON(label string, arg interface{}) {
+func (l *stdLogger) DebugJSON(label string, arg any) {
 	if !l.debug {
 		return
 	}