From b295616c5985a453be62f4ab608d3219608af6af Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 24 Mar 2026 06:19:14 +0000 Subject: [PATCH 1/3] Initial plan From 0592bfaf5eb1470750cd4af0159e6cc21b0cdaa1 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 24 Mar 2026 06:29:32 +0000 Subject: [PATCH 2/3] =?UTF-8?q?fix(safe-outputs):=20add=20=F0=9F=8E=AD=20e?= =?UTF-8?q?moji=20to=20staged=20mode=20in=20PR=20review=20handlers=20(USE-?= =?UTF-8?q?003)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> Agent-Logs-Url: https://github.com/github/gh-aw/sessions/eaa97989-67eb-4c84-8a62-004770e579fe --- actions/setup/js/create_pr_review_comment.cjs | 2 ++ actions/setup/js/safe_output_helpers.cjs | 2 ++ actions/setup/js/submit_pr_review.cjs | 2 ++ 3 files changed, 6 insertions(+) diff --git a/actions/setup/js/create_pr_review_comment.cjs b/actions/setup/js/create_pr_review_comment.cjs index 8384ec1d7e5..d06bb5b8eef 100644 --- a/actions/setup/js/create_pr_review_comment.cjs +++ b/actions/setup/js/create_pr_review_comment.cjs @@ -10,6 +10,7 @@ const { resolveTargetRepoConfig, resolveAndValidateRepo } = require("./repo_help const { sanitizeContent } = require("./sanitize_content.cjs"); const { createAuthenticatedGitHubClient } = require("./handler_auth.cjs"); const { buildWorkflowRunUrl } = require("./workflow_metadata_helpers.cjs"); +const { logStagedPreviewInfo } = require("./staged_preview.cjs"); /** @type {string} Safe output type handled by this module */ const HANDLER_TYPE = "create_pull_request_review_comment"; @@ -49,6 +50,7 @@ async function main(config = {}) { // Propagate per-handler staged flag to the shared PR review buffer if (config.staged === true) { buffer.setStaged(true); + logStagedPreviewInfo("PR review comments will be previewed without being submitted"); } // Track how many items we've processed for max limit diff --git a/actions/setup/js/safe_output_helpers.cjs b/actions/setup/js/safe_output_helpers.cjs index 42dbbb1fb94..c123bef1210 100644 --- a/actions/setup/js/safe_output_helpers.cjs +++ b/actions/setup/js/safe_output_helpers.cjs @@ -8,6 +8,7 @@ const { getErrorMessage } = require("./error_helpers.cjs"); const { matchesSimpleGlob } = require("./glob_pattern_helpers.cjs"); +const { logStagedPreviewInfo } = require("./staged_preview.cjs"); /** * Parse a comma-separated list of allowed items from environment variable @@ -417,4 +418,5 @@ module.exports = { matchesBlockedPattern, isUsernameBlocked, isStagedMode, + logStagedPreviewInfo, }; diff --git a/actions/setup/js/submit_pr_review.cjs b/actions/setup/js/submit_pr_review.cjs index 547e632b7f9..0798127a18c 100644 --- a/actions/setup/js/submit_pr_review.cjs +++ b/actions/setup/js/submit_pr_review.cjs @@ -9,6 +9,7 @@ const { resolveTarget } = require("./safe_output_helpers.cjs"); const { resolveTargetRepoConfig, resolveAndValidateRepo } = require("./repo_helpers.cjs"); const { createAuthenticatedGitHubClient } = require("./handler_auth.cjs"); const { getErrorMessage } = require("./error_helpers.cjs"); +const { logStagedPreviewInfo } = require("./staged_preview.cjs"); /** @type {string} Safe output type handled by this module */ const HANDLER_TYPE = "submit_pull_request_review"; @@ -49,6 +50,7 @@ async function main(config = {}) { // Propagate per-handler staged flag to the shared PR review buffer if (config.staged === true) { buffer.setStaged(true); + logStagedPreviewInfo("PR review will be previewed without being submitted"); } let processedCount = 0; From d819ce49172f3429983092fa88123f046aa77d34 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 24 Mar 2026 11:31:13 +0000 Subject: [PATCH 3/3] fix: use isStagedMode(config) for logStagedPreviewInfo to cover global env var Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> Agent-Logs-Url: https://github.com/github/gh-aw/sessions/cc74daba-05ff-43a1-a371-f1af5f87dc80 --- actions/setup/js/create_pr_review_comment.cjs | 4 +++- actions/setup/js/submit_pr_review.cjs | 5 +++-- 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/actions/setup/js/create_pr_review_comment.cjs b/actions/setup/js/create_pr_review_comment.cjs index d06bb5b8eef..bda92911c3c 100644 --- a/actions/setup/js/create_pr_review_comment.cjs +++ b/actions/setup/js/create_pr_review_comment.cjs @@ -10,7 +10,7 @@ const { resolveTargetRepoConfig, resolveAndValidateRepo } = require("./repo_help const { sanitizeContent } = require("./sanitize_content.cjs"); const { createAuthenticatedGitHubClient } = require("./handler_auth.cjs"); const { buildWorkflowRunUrl } = require("./workflow_metadata_helpers.cjs"); -const { logStagedPreviewInfo } = require("./staged_preview.cjs"); +const { isStagedMode, logStagedPreviewInfo } = require("./safe_output_helpers.cjs"); /** @type {string} Safe output type handled by this module */ const HANDLER_TYPE = "create_pull_request_review_comment"; @@ -50,6 +50,8 @@ async function main(config = {}) { // Propagate per-handler staged flag to the shared PR review buffer if (config.staged === true) { buffer.setStaged(true); + } + if (isStagedMode(config)) { logStagedPreviewInfo("PR review comments will be previewed without being submitted"); } diff --git a/actions/setup/js/submit_pr_review.cjs b/actions/setup/js/submit_pr_review.cjs index 0798127a18c..2858c695d51 100644 --- a/actions/setup/js/submit_pr_review.cjs +++ b/actions/setup/js/submit_pr_review.cjs @@ -5,11 +5,10 @@ * @typedef {import('./types/handler-factory').HandlerFactoryFunction} HandlerFactoryFunction */ -const { resolveTarget } = require("./safe_output_helpers.cjs"); +const { resolveTarget, isStagedMode, logStagedPreviewInfo } = require("./safe_output_helpers.cjs"); const { resolveTargetRepoConfig, resolveAndValidateRepo } = require("./repo_helpers.cjs"); const { createAuthenticatedGitHubClient } = require("./handler_auth.cjs"); const { getErrorMessage } = require("./error_helpers.cjs"); -const { logStagedPreviewInfo } = require("./staged_preview.cjs"); /** @type {string} Safe output type handled by this module */ const HANDLER_TYPE = "submit_pull_request_review"; @@ -50,6 +49,8 @@ async function main(config = {}) { // Propagate per-handler staged flag to the shared PR review buffer if (config.staged === true) { buffer.setStaged(true); + } + if (isStagedMode(config)) { logStagedPreviewInfo("PR review will be previewed without being submitted"); }