Exclude and satisfy linters

Author: williammartin

.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:

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 {

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)
 		}

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 {

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 {