From 461991ebaf1f5f84bd75512c7df69d34b80ae66e Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 9 Mar 2026 10:38:38 +0000 Subject: [PATCH 1/2] Initial plan From 27d28b3bac8a226916bbf7bb7794835b241612d2 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 9 Mar 2026 10:50:44 +0000 Subject: [PATCH 2/2] fix: include pull_request_target in PR context detection for safe-outputs resolveTarget Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --- actions/setup/js/safe_output_helpers.cjs | 2 +- actions/setup/js/safe_output_helpers.test.cjs | 26 +++++++++++++++++++ 2 files changed, 27 insertions(+), 1 deletion(-) diff --git a/actions/setup/js/safe_output_helpers.cjs b/actions/setup/js/safe_output_helpers.cjs index 0d97fa86b55..3dc9897fb14 100644 --- a/actions/setup/js/safe_output_helpers.cjs +++ b/actions/setup/js/safe_output_helpers.cjs @@ -69,7 +69,7 @@ function resolveTarget(params) { // Check context type const isIssueContext = context.eventName === "issues" || context.eventName === "issue_comment"; - const isPRContext = context.eventName === "pull_request" || context.eventName === "pull_request_review" || context.eventName === "pull_request_review_comment"; + const isPRContext = context.eventName === "pull_request" || context.eventName === "pull_request_target" || context.eventName === "pull_request_review" || context.eventName === "pull_request_review_comment"; // Default target is "triggering" const target = targetConfig || "triggering"; diff --git a/actions/setup/js/safe_output_helpers.test.cjs b/actions/setup/js/safe_output_helpers.test.cjs index b7296a9ef2d..6cc63b22912 100644 --- a/actions/setup/js/safe_output_helpers.test.cjs +++ b/actions/setup/js/safe_output_helpers.test.cjs @@ -200,6 +200,19 @@ describe("safe_output_helpers", () => { expect(result.contextType).toBe("pull request"); }); + it("should handle pull_request_target event", () => { + const result = helpers.resolveTarget({ + ...baseParams, + context: { + eventName: "pull_request_target", + payload: { pull_request: { number: 654 } }, + }, + }); + expect(result.success).toBe(true); + expect(result.number).toBe(654); + expect(result.contextType).toBe("pull request"); + }); + it("should fail when issue context but no issue in payload", () => { const result = helpers.resolveTarget({ ...baseParams, @@ -246,6 +259,19 @@ describe("safe_output_helpers", () => { expect(result.contextType).toBe("pull request"); }); + it("should resolve triggering pull_request_target context", () => { + const result = helpers.resolveTarget({ + ...baseParams, + context: { + eventName: "pull_request_target", + payload: { pull_request: { number: 987 } }, + }, + }); + expect(result.success).toBe(true); + expect(result.number).toBe(987); + expect(result.contextType).toBe("pull request"); + }); + it("should fail when triggering and not in PR context", () => { const result = helpers.resolveTarget({ ...baseParams,