lint: fix if to case

Author: mattdholloway

pkg/github/actions_resource.go

@@ -45,12 +45,12 @@ func WorkflowRunLogsResourceHandler(getClient GetClientFn) func(ctx context.Cont
 			return nil, errors.New("repo is required")
 		}
 
-		runIdStr, ok := request.Params.Arguments["runId"].([]string)
-		if !ok || len(runIdStr) == 0 {
+		runIDStr, ok := request.Params.Arguments["runId"].([]string)
+		if !ok || len(runIDStr) == 0 {
 			return nil, errors.New("runId is required")
 		}
 
-		runId, err := strconv.ParseInt(runIdStr[0], 10, 64)
+		runID, err := strconv.ParseInt(runIDStr[0], 10, 64)
 		if err != nil {
 			return nil, fmt.Errorf("invalid runId: %w", err)
 		}
@@ -61,7 +61,7 @@ func WorkflowRunLogsResourceHandler(getClient GetClientFn) func(ctx context.Cont
 		}
 
 		// Get the JIT URL for workflow run logs
-		url, resp, err := client.Actions.GetWorkflowRunLogs(ctx, owner[0], repo[0], runId, 1)
+		url, resp, err := client.Actions.GetWorkflowRunLogs(ctx, owner[0], repo[0], runID, 1)
 		if err != nil {
 			return nil, fmt.Errorf("failed to get workflow run logs URL: %w", err)
 		}
@@ -77,7 +77,7 @@ func WorkflowRunLogsResourceHandler(getClient GetClientFn) func(ctx context.Cont
 			mcp.TextResourceContents{
 				URI:      request.Params.URI,
 				MIMEType: "application/zip",
-				Text:     fmt.Sprintf("Workflow run logs for run %d (ZIP archive)\n\nNote: This is a ZIP archive containing all job logs. Download URL was: %s\n\nContent length: %d bytes", runId, url.String(), len(content)),
+				Text:     fmt.Sprintf("Workflow run logs for run %d (ZIP archive)\n\nNote: This is a ZIP archive containing all job logs. Download URL was: %s\n\nContent length: %d bytes", runID, url.String(), len(content)),
 			},
 		}, nil
 	}
@@ -97,12 +97,12 @@ func JobLogsResourceHandler(getClient GetClientFn) func(ctx context.Context, req
 			return nil, errors.New("repo is required")
 		}
 
-		jobIdStr, ok := request.Params.Arguments["jobId"].([]string)
-		if !ok || len(jobIdStr) == 0 {
+		jobIDStr, ok := request.Params.Arguments["jobId"].([]string)
+		if !ok || len(jobIDStr) == 0 {
 			return nil, errors.New("jobId is required")
 		}
 
-		jobId, err := strconv.ParseInt(jobIdStr[0], 10, 64)
+		jobID, err := strconv.ParseInt(jobIDStr[0], 10, 64)
 		if err != nil {
 			return nil, fmt.Errorf("invalid jobId: %w", err)
 		}
@@ -113,7 +113,7 @@ func JobLogsResourceHandler(getClient GetClientFn) func(ctx context.Context, req
 		}
 
 		// Get the JIT URL for job logs
-		url, resp, err := client.Actions.GetWorkflowJobLogs(ctx, owner[0], repo[0], jobId, 1)
+		url, resp, err := client.Actions.GetWorkflowJobLogs(ctx, owner[0], repo[0], jobID, 1)
 		if err != nil {
 			return nil, fmt.Errorf("failed to get job logs URL: %w", err)
 		}

pkg/github/repositories.go

@@ -1671,20 +1671,21 @@ func handleFileContentsResourceLink(owner, repo, path, ref, sha string) (*mcp.Ca
 		description = fmt.Sprintf("File content for %s at commit %s in %s/%s", path, sha[:8], owner, repo)
 	case ref != "":
 		// Handle different ref types
-		if strings.HasPrefix(ref, "refs/heads/") {
+		switch {
+		case strings.HasPrefix(ref, "refs/heads/"):
 			branch := strings.TrimPrefix(ref, "refs/heads/")
 			resourceURI = fmt.Sprintf("repo://%s/%s/refs/heads/%s/contents%s", owner, repo, branch, path)
 			description = fmt.Sprintf("File content for %s on branch %s in %s/%s", path, branch, owner, repo)
-		} else if strings.HasPrefix(ref, "refs/tags/") {
+		case strings.HasPrefix(ref, "refs/tags/"):
 			tag := strings.TrimPrefix(ref, "refs/tags/")
 			resourceURI = fmt.Sprintf("repo://%s/%s/refs/tags/%s/contents%s", owner, repo, tag, path)
 			description = fmt.Sprintf("File content for %s at tag %s in %s/%s", path, tag, owner, repo)
-		} else if strings.HasPrefix(ref, "refs/pull/") && strings.HasSuffix(ref, "/head") {
+		case strings.HasPrefix(ref, "refs/pull/") && strings.HasSuffix(ref, "/head"):
 			// Extract PR number from refs/pull/{number}/head
 			prNumber := strings.TrimSuffix(strings.TrimPrefix(ref, "refs/pull/"), "/head")
 			resourceURI = fmt.Sprintf("repo://%s/%s/pulls/%s/contents%s", owner, repo, prNumber, path)
 			description = fmt.Sprintf("File content for %s in PR #%s in %s/%s", path, prNumber, owner, repo)
-		} else {
+		default:
 			// Generic ref handling - try to clean it up
 			cleanRef := strings.TrimPrefix(ref, "refs/")
 			resourceURI = fmt.Sprintf("repo://%s/%s/refs/%s/contents%s", owner, repo, cleanRef, path)