Soft-skip submit_pull_request_review when no PR review context is available (#33435)

Author: Copilot

actions/setup/js/pr_review_buffer.cjs

@@ -235,8 +235,12 @@ function createReviewBuffer() {
     }
 
     if (!reviewContext) {
-      core.warning("No review context set - cannot submit review");
-      return { success: false, error: "No review context available" };
+      core.info("No review context set - skipping PR review submission");
+      return {
+        success: true,
+        skipped: true,
+        reason: "No review context available",
+      };
     }
 
     const { repo, repoParts, pullRequestNumber, pullRequest } = reviewContext;

actions/setup/js/pr_review_buffer.test.cjs

@@ -179,13 +179,14 @@ describe("pr_review_buffer (factory pattern)", () => {
       expect(mockGithub.rest.pulls.createReview).not.toHaveBeenCalled();
     });
 
-    it("should fail when no review context is set", async () => {
+    it("should skip when no review context is set", async () => {
       buffer.addComment({ path: "test.js", line: 1, body: "comment" });
 
       const result = await buffer.submitReview();
 
-      expect(result.success).toBe(false);
-      expect(result.error).toContain("No review context available");
+      expect(result.success).toBe(true);
+      expect(result.skipped).toBe(true);
+      expect(result.reason).toContain("No review context available");
     });
 
     it("should fail when PR head SHA is missing", async () => {

actions/setup/js/submit_pr_review.test.cjs

@@ -729,8 +729,8 @@ describe("submit_pr_review (Handler Factory Architecture)", () => {
     expect(result.success).toBe(true);
     expect(localBuffer.hasReviewMetadata()).toBe(true);
 
-    // Review context should NOT be set because the disallowed repo was rejected
-    // submitReview() will subsequently fail with "No review context available"
+    // Review context should NOT be set because the disallowed repo was rejected.
+    // submitReview() will subsequently be skipped due to missing review context.
     expect(localBuffer.getReviewContext()).toBeNull();
 
     delete global.context;