fix golangci-lint config

Author: chendelin1982

Makefile

@@ -134,6 +134,34 @@ lint:
 		echo "💡 Then add $$(go env GOPATH)/bin to your PATH"; \
 		exit 1; \
 	}
+	@echo "🔧 Verifying golangci-lint configuration..."
+	@cd core && ( \
+		for i in 1 2 3; do \
+			output=$$(golangci-lint config verify --config=.golangci.yml 2>&1); \
+			exit_code=$$?; \
+			if [ $$exit_code -eq 0 ]; then \
+				echo "✅ Configuration verified"; \
+				break; \
+			else \
+				if echo "$$output" | grep -q -i "timeout\|deadline\|network"; then \
+					if [ $$i -lt 3 ]; then \
+						echo "⚠️  Network timeout on attempt $$i/3, retrying in 2 seconds..."; \
+						sleep 2; \
+					else \
+						echo "❌ Config verification failed after 3 attempts due to network timeout"; \
+						echo "💡 Please check your internet connection or try again later"; \
+						exit 1; \
+					fi \
+				else \
+					echo "❌ Config verification failed:"; \
+					echo "$$output"; \
+					exit 1; \
+				fi \
+			fi \
+		done \
+	)
+	@echo ""
+	@echo "🔍 Running lint checks..."
 	@cd core && golangci-lint run --timeout=5m --config=.golangci.yml
 	@echo "✅ Linting completed"
 	@echo ""

core/.golangci.yml

@@ -1,6 +1,6 @@
 # golangci-lint configuration for apprun project
-# Compatible with golangci-lint v2.7.2
-# Documentation: https://golangci-lint.run/product/config
+# Schema Version: v2.7.2
+# Documentation: Based on https://golangci-lint.run/jsonschema/golangci.v2.7.jsonschema.json
 
 version: "2"
 
@@ -11,101 +11,113 @@ run:
     - integration
 
 linters:
+  default: "none"
   enable:
+    - bodyclose
+    - dupl
     - errcheck
+    - errorlint
+    - gocritic
+    - gocyclo
+    - gosec
     - govet
     - ineffassign
-    - staticcheck
-    - unused
     - misspell
-    - dupl
-    - gosec
-    - unconvert
     - prealloc
-    - gocyclo
-    - gocritic
-    - errorlint
-    - bodyclose
-
-settings:
-  errcheck:
-    check-type-assertions: true
-    check-blank: true
-  
-  govet:
-    enable:
-      - all
-    disable:
-      - shadow
-      - fieldalignment
-  
-  misspell:
-    locale: US
-  
-  dupl:
-    threshold: 150
-  
-  gosec:
-    config:
-      global:
-        nosec: false
-        audit: true
-  
-  staticcheck:
-    checks:
-      - all
-  
-  gocyclo:
-    min-complexity: 20
-  
-  gocritic:
-    enabled-tags:
-      - diagnostic
-      - style
-      - performance
-    disabled-checks:
-      - unnamedResult
-      - commentFormatting
-      - httpNoBody
-      - octalLiteral
-      - paramTypeCombine
-      - equalFold
-      - exitAfterDefer
+    - staticcheck
+    - unconvert
+    - unused
 
-issues:
-  exclude-dirs:
-    - ent
-    - docs
-  
-  exclude-generated-strict: true
-  
-  exclude-rules:
-    - path: cmd/
-      linters:
-        - errcheck
-    
-    - linters:
-        - errcheck
-      text: "Error return value of.*WithContext"
-    
-    - linters:
-        - errcheck
-      text: "Error return value of.*logger\\.Sync"
-    
-    - linters:
-        - gosec
-      text: "G302"
+  settings:
+    govet:
+      enable:
+        - nilness
+        - shadow
+      disable:
+        - fieldalignment
 
-    - linters:
-        - gocritic
-      text: "exitAfterDefer"
+    gosec:
+      excludes:
+        - G104
+        - G304
+
+    gocyclo:
+      min-complexity: 20
+      
+    dupl:
+      threshold: 150
+      
+    misspell:
+      locale: US
+      
+    errcheck:
+      check-type-assertions: true
+      check-blank: true
+
+    staticcheck:
+      checks: ["all"]
+
+    gocritic:
+      enabled-tags:
+        - diagnostic
+        - style
+        - performance
+      disabled-checks:
+        - unnamedResult
+        - commentFormatting
+        - httpNoBody
+        - octalLiteral
+        - paramTypeCombine
+        - equalFold
+        - exitAfterDefer
+
+  exclusions: 
+    generated: strict
 
-    - path: _test\.go
-      linters:
-        - dupl
-        - gosec
-        - errcheck
-        - gocyclo
-  
+    paths:
+      - "^ent/"
+      - "^docs/"
+
+    rules:
+      # Exclude cmd/ from errcheck
+      - path: "cmd/"
+        linters:
+          - errcheck
+      
+      # Specific error suppressions
+      - linters:
+          - errcheck
+        text: "Error return value of.*WithContext"
+        
+      - linters:
+          - errcheck
+        text: "Error return value of.*logger\\.Sync"
+        
+      - linters:
+          - errcheck
+        text: "Error return value of.*DeleteConfig"
+        
+      - linters:
+          - errcheck
+        text: "Error return value of.*Setenv"
+        
+      - linters:
+          - gosec
+        text: "G302"
+        
+      - linters:
+          - gocritic
+        text: "exitAfterDefer"
+        
+      # Test file exclusions
+      - path: "_test\\.go"
+        linters:
+          - dupl
+          - gosec
+          - errcheck
+          - gocyclo
+
+issues:
   max-issues-per-linter: 50
   max-same-issues: 3
+  uniq-by-line: true

core/cmd/server/main.go

@@ -98,8 +98,8 @@ func run() error {
 		return err
 	}
 	defer func() {
-		if err := dbClient.Close(); err != nil {
-			log.Printf("⚠️  Warning: Failed to close database connection: %v", err)
+		if closeErr := dbClient.Close(); closeErr != nil {
+			log.Printf("⚠️  Warning: Failed to close database connection: %v", closeErr)
 		}
 	}()
 	log.Println("✅ Database connected")

core/handlers/demo_handler.go

@@ -1,3 +1,5 @@
+// Package handlers provides HTTP request handlers for the application.
+// It includes demo handlers for testing and reference implementations.
 package handlers
 
 import (

core/handlers/handlers.go

@@ -1,3 +1,4 @@
+// Package handlers provides HTTP request handlers for the application.
 package handlers
 
 // TODO: Implement handlers using net/http

core/internal/config/types.go

@@ -1,3 +1,4 @@
+// Package config defines the configuration types and structures for the application.
 package config
 
 // Config is the root define source of all configuration items

core/internal/middleware/language.go

@@ -1,3 +1,5 @@
+// Package middleware provides HTTP middleware components for request processing,
+// including language detection and internationalization support.
 package middleware
 
 import (

core/modules/config/bootstrap.go

@@ -1,3 +1,6 @@
+// Package config provides the configuration management module,
+// including service initialization, dynamic configuration loading,
+// and integration with the configuration center database.
 package config
 
 import (

core/modules/config/handler.go

@@ -1,3 +1,4 @@
+// Package config provides HTTP handlers for configuration management endpoints.
 package config
 
 import (

core/modules/config/handler_test.go

@@ -462,11 +462,11 @@ poc:
 		// 删除后仍可能从配置文件或默认值返回，所以不检查 404
 		// 只验证不再是我们设置的值
 		if w.Code == http.StatusOK {
-			var response GetConfigResponse
-			err := json.NewDecoder(w.Body).Decode(&response)
+			var resp GetConfigResponse
+			err := json.NewDecoder(w.Body).Decode(&resp)
 			require.NoError(t, err)
 			// 验证返回的是默认值，而不是我们设置的值
-			assert.NotEqual(t, testValue, response.Value)
+			assert.NotEqual(t, testValue, resp.Value)
 		}
 	})
 }