fixed err return type for json marshalling

mattdholloway · mattdholloway · commit ec521d58096d · 2025-07-28T16:27:07.000+01:00
diff --git a/pkg/github/pullrequests.go b/pkg/github/pullrequests.go
@@ -414,7 +414,7 @@ func UpdatePullRequest(getClient GetClientFn, getGQLClient GetGQLClientFn, t tra
 
 			r, err := json.Marshal(finalPR)
 			if err != nil {
-				return ghErrors.NewGitHubGraphQLErrorResponse(ctx, "Failed to marshal response: %v", err), nil
+				return mcp.NewToolResultError(fmt.Sprintf("Failed to marshal response: %v", err)), nil
 			}
 
 			return mcp.NewToolResultText(string(r)), nil

Original file line number	Diff line number	Diff line change
`@@ -414,7 +414,7 @@ func UpdatePullRequest(getClient GetClientFn, getGQLClient GetGQLClientFn, t tra`
`414`	`414`
`415`	`415`	`r, err := json.Marshal(finalPR)`
`416`	`416`	`if err != nil {`
`417`		`- return ghErrors.NewGitHubGraphQLErrorResponse(ctx, "Failed to marshal response: %v", err), nil`
	`417`	`+ return mcp.NewToolResultError(fmt.Sprintf("Failed to marshal response: %v", err)), nil`
`418`	`418`	`}`
`419`	`419`
`420`	`420`	`return mcp.NewToolResultText(string(r)), nil`