Exclude and satisfy linters

williammartin · williammartin · commit f6d0b4a1f46b · 2025-05-12T11:48:02.000+02:00
diff --git a/.golangci.yml b/.golangci.yml
@@ -20,13 +20,25 @@ linters:
     - gocritic
     - makezero
     - gosec
+  settings:
+    staticcheck:
+      checks:
+        - all
+        - '-QF1008' # Allow embedded structs to be referenced by field
+        - '-ST1000' # Do not require package comments
+    revive:
+      rules:
+        - name: exported
+          disabled: true
+        - name: exported
+          disabled: true
+        - name: package-comments
+          disabled: true
 
 formatters:
   enable:
-    - gci
     - gofmt
     - goimports
-    - golines
 
 output:
   formats:
diff --git a/cmd/mcpcurl/main.go b/cmd/mcpcurl/main.go
@@ -77,7 +77,7 @@ type (
 		Arguments map[string]interface{} `json:"arguments"`
 	}
 
-	// Define structure to match the response format
+	// Content matches the response format of a text content response
 	Content struct {
 		Type string `json:"type"`
 		Text string `json:"text"`
@@ -284,10 +284,10 @@ func addCommandFromTool(toolsCmd *cobra.Command, tool *Tool, prettyPrint bool) {
 			cmd.Flags().Bool(name, false, description)
 		case "array":
 			if prop.Items != nil {
-				if prop.Items.Type == "string" {
+				switch prop.Items.Type {
+				case "string":
 					cmd.Flags().StringSlice(name, []string{}, description)
-				} else if prop.Items.Type == "object" {
-					// For complex objects in arrays, we'll use a JSON string that users can provide
+				case "object":
 					cmd.Flags().String(name+"-json", "", description+" (provide as JSON array)")
 				}
 			}
@@ -327,11 +327,12 @@ func buildArgumentsMap(cmd *cobra.Command, tool *Tool) (map[string]interface{},
 			}
 		case "array":
 			if prop.Items != nil {
-				if prop.Items.Type == "string" {
+				switch prop.Items.Type {
+				case "string":
 					if values, _ := cmd.Flags().GetStringSlice(name); len(values) > 0 {
 						arguments[name] = values
 					}
-				} else if prop.Items.Type == "object" {
+				case "object":
 					if jsonStr, _ := cmd.Flags().GetString(name + "-json"); jsonStr != "" {
 						var jsonArray []interface{}
 						if err := json.Unmarshal([]byte(jsonStr), &jsonArray); err != nil {
diff --git a/internal/ghmcp/server.go b/internal/ghmcp/server.go
@@ -14,7 +14,6 @@ import (
 	"github.com/github/github-mcp-server/pkg/translations"
 	gogithub "github.com/google/go-github/v69/github"
 	"github.com/mark3labs/mcp-go/mcp"
-
 	"github.com/mark3labs/mcp-go/server"
 	"github.com/sirupsen/logrus"
 )
@@ -170,7 +169,7 @@ func RunStdioServer(cfg StdioServerConfig) error {
 
 	logrusLogger := logrus.New()
 	if cfg.LogFilePath != "" {
-		file, err := os.OpenFile(cfg.LogFilePath, os.O_CREATE|os.O_WRONLY|os.O_APPEND, 0666)
+		file, err := os.OpenFile(cfg.LogFilePath, os.O_CREATE|os.O_WRONLY|os.O_APPEND, 0600)
 		if err != nil {
 			return fmt.Errorf("failed to open log file: %w", err)
 		}
diff --git a/pkg/github/repositories.go b/pkg/github/repositories.go
@@ -612,7 +612,7 @@ func CreateBranch(getClient GetClientFn, t translations.TranslationHelperFunc) (
 				if err != nil {
 					return nil, fmt.Errorf("failed to get repository: %w", err)
 				}
-				defer resp.Body.Close()
+				defer func() { _ = resp.Body.Close() }()
 
 				fromBranch = *repository.DefaultBranch
 			}
@@ -622,7 +622,7 @@ func CreateBranch(getClient GetClientFn, t translations.TranslationHelperFunc) (
 			if err != nil {
 				return nil, fmt.Errorf("failed to get reference: %w", err)
 			}
-			defer resp.Body.Close()
+			defer func() { _ = resp.Body.Close() }()
 
 			// Create new branch
 			newRef := &github.Reference{
@@ -634,7 +634,7 @@ func CreateBranch(getClient GetClientFn, t translations.TranslationHelperFunc) (
 			if err != nil {
 				return nil, fmt.Errorf("failed to create branch: %w", err)
 			}
-			defer resp.Body.Close()
+			defer func() { _ = resp.Body.Close() }()
 
 			r, err := json.Marshal(createdRef)
 			if err != nil {
diff --git a/pkg/translations/translations.go b/pkg/translations/translations.go
@@ -56,7 +56,7 @@ func TranslationHelper() (TranslationHelperFunc, func()) {
 		}
 }
 
-// dump translationKeyMap to a json file called github-mcp-server-config.json
+// DumpTranslationKeyMap writes the translation map to a json file called github-mcp-server-config.json
 func DumpTranslationKeyMap(translationKeyMap map[string]string) error {
 	file, err := os.Create("github-mcp-server-config.json")
 	if err != nil {

Original file line number	Diff line number	Diff line change
`@@ -612,7 +612,7 @@ func CreateBranch(getClient GetClientFn, t translations.TranslationHelperFunc) (`
`612`	`612`	`if err != nil {`
`613`	`613`	`return nil, fmt.Errorf("failed to get repository: %w", err)`
`614`	`614`	`}`
`615`		`- defer resp.Body.Close()`
	`615`	`+ defer func() { _ = resp.Body.Close() }()`
`616`	`616`
`617`	`617`	`fromBranch = *repository.DefaultBranch`
`618`	`618`	`}`
`@@ -622,7 +622,7 @@ func CreateBranch(getClient GetClientFn, t translations.TranslationHelperFunc) (`
`622`	`622`	`if err != nil {`
`623`	`623`	`return nil, fmt.Errorf("failed to get reference: %w", err)`
`624`	`624`	`}`
`625`		`- defer resp.Body.Close()`
	`625`	`+ defer func() { _ = resp.Body.Close() }()`
`626`	`626`
`627`	`627`	`// Create new branch`
`628`	`628`	`newRef := &github.Reference{`
`@@ -634,7 +634,7 @@ func CreateBranch(getClient GetClientFn, t translations.TranslationHelperFunc) (`
`634`	`634`	`if err != nil {`
`635`	`635`	`return nil, fmt.Errorf("failed to create branch: %w", err)`
`636`	`636`	`}`
`637`		`- defer resp.Body.Close()`
	`637`	`+ defer func() { _ = resp.Body.Close() }()`
`638`	`638`
`639`	`639`	`r, err := json.Marshal(createdRef)`
`640`	`640`	`if err != nil {`
Original file line number	Diff line number	Diff line change
`@@ -56,7 +56,7 @@ func TranslationHelper() (TranslationHelperFunc, func()) {`
`56`	`56`	`}`
`57`	`57`	`}`
`58`	`58`
`59`		`-// dump translationKeyMap to a json file called github-mcp-server-config.json`
	`59`	`+// DumpTranslationKeyMap writes the translation map to a json file called github-mcp-server-config.json`
`60`	`60`	`func DumpTranslationKeyMap(translationKeyMap map[string]string) error {`
`61`	`61`	`file, err := os.Create("github-mcp-server-config.json")`
`62`	`62`	`if err != nil {`