chore: fix all golangci-lint staticcheck issues

align kubo with unified golang ci linter from IPDX and
rules used in boxo and other go packages

addressed lint rules:
- ST1000: added package comments
- ST1020, ST1021, ST1022: fixed function/method comments
- QF1001: applied De Morgan's law
- QF1003: converted if-else chains to tagged switches
- QF1004: replaced strings.Replace with strings.ReplaceAll
- QF1008: simplified embedded struct field selectors
- unconvert: removed unnecessary type conversions
- usestdlibvars: used stdlib constants instead of literals

disabled errcheck linter in .golangci.yml

Author: lidel

.golangci.yml

@@ -3,6 +3,10 @@ linters:
   default: standard
   enable:
     - staticcheck
+    - unconvert
+    - usestdlibvars
+  disable:
+    - errcheck
   settings:
     staticcheck:
       checks:

assets/assets.go

@@ -1,3 +1,4 @@
+// Package assets provides embedded asset files for kubo.
 package assets
 
 import (

client/rpc/api.go

@@ -1,3 +1,4 @@
+// Package rpc provides an RPC client implementation for the kubo HTTP API.
 package rpc
 
 import (

client/rpc/api_test.go

@@ -111,7 +111,7 @@ func Test_NewURLApiWithClient_With_Headers(t *testing.T) {
 		http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
 			val := r.Header.Get(headerToTest)
 			if val != expectedHeaderValue {
-				w.WriteHeader(400)
+				w.WriteHeader(http.StatusBadRequest)
 				return
 			}
 			http.ServeContent(w, r, "", time.Now(), strings.NewReader("test"))

client/rpc/auth/auth.go

@@ -1,3 +1,4 @@
+// Package auth provides HTTP authentication utilities for the RPC client.
 package auth
 
 import "net/http"

client/rpc/response.go

@@ -80,7 +80,7 @@ func (r *Response) decode(dec interface{}) error {
 
 func (r *Request) Send(c *http.Client) (*Response, error) {
 	url := r.getURL()
-	req, err := http.NewRequest("POST", url, r.Body)
+	req, err := http.NewRequest(http.MethodPost, url, r.Body)
 	if err != nil {
 		return nil, err
 	}

cmd/ipfs/kubo/daemon.go

@@ -323,9 +323,10 @@ func daemonFunc(req *cmds.Request, re cmds.ResponseEmitter, env cmds.Environment
 		}
 		defer fetcher.Close()
 
-		if migrationCfg.Keep == "cache" {
+		switch migrationCfg.Keep {
+		case "cache":
 			cacheMigrations = true
-		} else if migrationCfg.Keep == "pin" {
+		case "pin":
 			pinMigrations = true
 		}
 

cmd/ipfs/kubo/start.go

@@ -1,4 +1,4 @@
-// cmd/ipfs/kubo implements the primary CLI binary for kubo
+// Package kubo implements the primary CLI binary for kubo
 package kubo
 
 import (

cmd/ipfs/util/signal.go

@@ -1,6 +1,7 @@
 //go:build !wasm
 // +build !wasm
 
+// Package util provides utility functions for the IPFS CLI.
 package util
 
 import (

commands/context.go

@@ -1,3 +1,4 @@
+// Package commands implements IPFS commands.
 package commands
 
 import (