diff --git a/actions/setup/js/submit_pr_review.cjs b/actions/setup/js/submit_pr_review.cjs index a68bcaad687..c340d7a38fa 100644 --- a/actions/setup/js/submit_pr_review.cjs +++ b/actions/setup/js/submit_pr_review.cjs @@ -88,9 +88,6 @@ async function main(config = {}) { // If create_pull_request_review_comment already set context, this is a no-op. // Use target config as single source of truth (same as add_comment): resolveTarget first, then use payload PR only when it matches. if (!buffer.getReviewContext()) { - const repo = `${context.repo.owner}/${context.repo.repo}`; - const repoParts = { owner: context.repo.owner, repo: context.repo.repo }; - const targetResult = resolveTarget({ targetConfig, item: message, @@ -106,6 +103,8 @@ async function main(config = {}) { } } else if (targetResult.number) { const prNum = targetResult.number; + const repo = `${context.repo.owner}/${context.repo.repo}`; + const repoParts = { owner: context.repo.owner, repo: context.repo.repo }; const payloadPR = context.payload?.pull_request; const usePayloadPR = payloadPR && payloadPR.number === prNum && payloadPR.head?.sha; diff --git a/pkg/workflow/submit_pr_review.go b/pkg/workflow/submit_pr_review.go index 8c8116a6d40..1754924074d 100644 --- a/pkg/workflow/submit_pr_review.go +++ b/pkg/workflow/submit_pr_review.go @@ -34,9 +34,8 @@ func (c *Compiler) parseSubmitPullRequestReviewConfig(outputMap map[string]any) // Parse target (same semantics as add-comment / create-pull-request-review-comment) if target, exists := configMap["target"]; exists { - if targetStr, ok := target.(string); ok && targetStr != "" { + if targetStr, ok := target.(string); ok { config.Target = targetStr - submitPRReviewLog.Printf("Target: %s", config.Target) } }