ci: restore previous linter rules (#13322)

.golangci.yaml

@@ -1,76 +1,51 @@
 version: "2"
 linters:
-  default: none
   enable:
     - asasalint
     - bidichk
     - bodyclose
     - containedctx
-    - copyloopvar
-    - errcheck
-    - errorlint
-    - exptostd
     - gocheckcompilerdirectives
-    - govet
-    - ineffassign
     - intrange
     - makezero
     - misspell
-    - modernize
     - nilerr
-    - nilnil
     - nolintlint
     - nosprintfhostport
-    - perfsprint
-    - prealloc
-    - sloglint
-    - staticcheck
     - unconvert
-    - unused
-    - usestdlibvars
     - usetesting
     - wastedassign
     - whitespace
+  disable:
+    - errcheck
+    - usestdlibvars
   settings:
-    errcheck:
+    govet:
-      exclude-functions:
+      disable:
-        - fmt.Fprintf
+        - unusedresult
-    perfsprint:
-      strconcat: false
-      concat-loop: false
     staticcheck:
       checks:
         - all
-        # Using a deprecated function, variable, constant or field.
+        - -QF* # disable quick fix suggestions
-        # https://staticcheck.dev/docs/checks/#SA1019
         - -SA1019
-        # Incorrect or missing package comment.
+        - -ST1000 # package comment format
-        # https://staticcheck.dev/docs/checks/#ST1000
+        - -ST1003 # underscores in package names
-        - -ST1000
+        - -ST1005 # error strings should not be capitalized
-        # Poorly chosen identifier.
+        - -ST1012 # error var naming (ErrFoo)
-        # https://staticcheck.dev/docs/checks/#ST1003
+        - -ST1016 # receiver name consistency
-        - -ST1003
+        - -ST1020 # comment on exported function format
-        # The documentation of an exported function should start with the function's name.
+        - -ST1021 # comment on exported type format
-        # https://staticcheck.dev/docs/checks/#ST1020
+        - -ST1022 # comment on exported var format
-        - -ST1020
+        - -ST1023 # omit type from declaration
-        # The documentation of an exported type should start with type's name.
+severity:
-        # https://staticcheck.dev/docs/checks/#ST1021
+  default: error
-        - -ST1021
+  rules:
-        # The documentation of an exported variable or constant should start with variable's name.
+    - linters:
-        # https://staticcheck.dev/docs/checks/#ST1022
+        - gofmt
-        - -ST1022
+        - goimports
-    usestdlibvars:
+        - intrange
-      http-method: false
+      severity: info
-      http-status-code: false
-
 formatters:
   enable:
-    - gci
     - gofmt
     - gofumpt
-  settings:
-    gci:
-      sections:
-        - standard
-        - default
-        - localmodule

api/examples/chat/main.go

@@ -15,19 +15,19 @@ func main() {
 	}
 
 	messages := []api.Message{
-		api.Message{
+		{
 			Role:    "system",
 			Content: "Provide very brief, concise responses",
 		},
-		api.Message{
+		{
 			Role:    "user",
 			Content: "Name some unusual animals",
 		},
-		api.Message{
+		{
 			Role:    "assistant",
 			Content: "Monotreme, platypus, echidna",
 		},
-		api.Message{
+		{
 			Role:    "user",
 			Content: "which of these is the most dangerous?",
 		},