[WIP] Fix failing GitHub Actions job JS Tests (shard 4/4)

actions/setup/js/safe_outputs_handlers.cjs

@@ -53,11 +53,7 @@ function buildIntentErrorResponse(error) {
  */
 function hasUpdatePullRequestFields(args) {
   const safeArgs = args || {};
-  return (
-    typeof safeArgs.title === "string" ||
-    typeof safeArgs.body === "string" ||
-    safeArgs.update_branch === true
-  );
+  return typeof safeArgs.title === "string" || typeof safeArgs.body === "string" || safeArgs.update_branch === true;
 }
 
 /**

actions/setup/js/safe_outputs_handlers.test.cjs

@@ -1776,76 +1776,56 @@ describe("safe_outputs_handlers", () => {
     });
 
     it("should throw MCP error when called with null/undefined args", () => {
-      expect(() => handlers.updatePullRequestHandler(null)).toThrow(
-        expect.objectContaining({ code: -32602 })
-      );
-      expect(() => handlers.updatePullRequestHandler(undefined)).toThrow(
-        expect.objectContaining({ code: -32602 })
-      );
+      expect(() => handlers.updatePullRequestHandler(null)).toThrow(expect.objectContaining({ code: -32602 }));
+      expect(() => handlers.updatePullRequestHandler(undefined)).toThrow(expect.objectContaining({ code: -32602 }));
     });
 
     it("should throw MCP error when update_branch is explicitly false and no other fields", () => {
-      expect(() => handlers.updatePullRequestHandler({ update_branch: false })).toThrow(
-        expect.objectContaining({ code: -32602 })
-      );
+      expect(() => handlers.updatePullRequestHandler({ update_branch: false })).toThrow(expect.objectContaining({ code: -32602 }));
     });
 
     it("should throw MCP error when title is null", () => {
-      expect(() => handlers.updatePullRequestHandler({ title: null })).toThrow(
-        expect.objectContaining({ code: -32602 })
-      );
+      expect(() => handlers.updatePullRequestHandler({ title: null })).toThrow(expect.objectContaining({ code: -32602 }));
     });
 
     it("should throw MCP error when body is null", () => {
-      expect(() => handlers.updatePullRequestHandler({ body: null })).toThrow(
-        expect.objectContaining({ code: -32602 })
-      );
+      expect(() => handlers.updatePullRequestHandler({ body: null })).toThrow(expect.objectContaining({ code: -32602 }));
     });
 
     it("should throw MCP error when update_branch is null", () => {
-      expect(() => handlers.updatePullRequestHandler({ update_branch: null })).toThrow(
-        expect.objectContaining({ code: -32602 })
-      );
+      expect(() => handlers.updatePullRequestHandler({ update_branch: null })).toThrow(expect.objectContaining({ code: -32602 }));
     });
 
     it("should write entry and return success when title is provided", () => {
       const result = handlers.updatePullRequestHandler({ title: "New Title" });
       expect(result).toHaveProperty("content");
       const data = JSON.parse(result.content[0].text);
       expect(data.result).toBe("success");
-      expect(mockAppendSafeOutput).toHaveBeenCalledWith(
-        expect.objectContaining({ type: "update_pull_request", title: "New Title" })
-      );
+      expect(mockAppendSafeOutput).toHaveBeenCalledWith(expect.objectContaining({ type: "update_pull_request", title: "New Title" }));
     });
 
     it("should write entry and return success when body is provided", () => {
       const result = handlers.updatePullRequestHandler({ body: "Updated body" });
       expect(result).toHaveProperty("content");
       const data = JSON.parse(result.content[0].text);
       expect(data.result).toBe("success");
-      expect(mockAppendSafeOutput).toHaveBeenCalledWith(
-        expect.objectContaining({ type: "update_pull_request", body: "Updated body" })
-      );
+      expect(mockAppendSafeOutput).toHaveBeenCalledWith(expect.objectContaining({ type: "update_pull_request", body: "Updated body" }));
     });
 
     it("should write entry and return success when update_branch is true", () => {
       const result = handlers.updatePullRequestHandler({ update_branch: true });
       expect(result).toHaveProperty("content");
       const data = JSON.parse(result.content[0].text);
       expect(data.result).toBe("success");
-      expect(mockAppendSafeOutput).toHaveBeenCalledWith(
-        expect.objectContaining({ type: "update_pull_request", update_branch: true })
-      );
+      expect(mockAppendSafeOutput).toHaveBeenCalledWith(expect.objectContaining({ type: "update_pull_request", update_branch: true }));
     });
 
     it("should write entry and return success when both title and body are provided", () => {
       const result = handlers.updatePullRequestHandler({ title: "New Title", body: "New body" });
       expect(result).toHaveProperty("content");
       const data = JSON.parse(result.content[0].text);
       expect(data.result).toBe("success");
-      expect(mockAppendSafeOutput).toHaveBeenCalledWith(
-        expect.objectContaining({ type: "update_pull_request", title: "New Title", body: "New body" })
-      );
+      expect(mockAppendSafeOutput).toHaveBeenCalledWith(expect.objectContaining({ type: "update_pull_request", title: "New Title", body: "New body" }));
     });
 
     it("error message should mention all required fields", () => {

actions/setup/js/safe_outputs_tools.json

@@ -799,7 +799,6 @@
   },
   {
     "name": "update_pull_request",
-
     "description": "Update an existing GitHub pull request's title or body. Supports replacing, appending to, or prepending content to the body. Title is always replaced. Only the fields you specify will be updated; other fields remain unchanged. REQUIRED: You must provide at least one of: 'title' (a non-empty string), 'body' (a string), or 'update_branch' set to true (not false). Omitting all three, or passing only 'update_branch: false', will return a -32602 error.",
     "inputSchema": {
       "type": "object",
@@ -846,14 +845,6 @@
           "description": "Trustworthiness level of the message source (e.g., \"low\", \"medium\", \"high\")."
         }
       },
-      "anyOf": [
-        { "required": ["title"] },
-        { "required": ["body"] },
-        {
-          "required": ["update_branch"],
-          "properties": { "update_branch": { "const": true } }
-        }
-      ],
       "additionalProperties": false
     }
   },