chore: sync actions from gh-aw@v0.74.9 (#111)

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

Author: github-actions[bot]

setup/js/add_comment.cjs

@@ -377,6 +377,8 @@ async function main(config = {}) {
   const maxCount = config.max || 20;
   const { defaultTargetRepo, allowedRepos } = resolveTargetRepoConfig(config);
   const includeFooter = parseBoolTemplatable(config.footer, true);
+  const requiredLabels = Array.isArray(config.required_labels) ? config.required_labels : [];
+  const requiredTitlePrefix = config.required_title_prefix || "";
   const mentionsDisabled = config.mentions === false || config.mentions?.enabled === false;
   const configuredMentionAliases =
     !mentionsDisabled && Array.isArray(config.mentions?.allowed) ? config.mentions.allowed.map(alias => (typeof alias === "string" ? alias.trim().replace(/^@+/, "") : "")).filter(alias => alias.length > 0) : [];
@@ -397,6 +399,8 @@ async function main(config = {}) {
   if (allowedRepos.size > 0) {
     core.info(`Allowed repos: ${Array.from(allowedRepos).join(", ")}`);
   }
+  if (requiredLabels.length > 0) core.info(`Required labels (all): ${requiredLabels.join(", ")}`);
+  if (requiredTitlePrefix) core.info(`Required title prefix: ${requiredTitlePrefix}`);
   if (hideOlderCommentsEnabled) {
     core.info("Hide-older-comments is enabled");
   }
@@ -521,6 +525,31 @@ async function main(config = {}) {
       }
     }
 
+    // Apply required-labels and required-title-prefix filters (issues/PRs only, not discussions)
+    if (!isDiscussion && (requiredLabels.length > 0 || requiredTitlePrefix)) {
+      try {
+        const { data: filterItem } = await githubClient.rest.issues.get({
+          owner: repoParts.owner,
+          repo: repoParts.repo,
+          issue_number: itemNumber,
+        });
+        if (requiredLabels.length > 0) {
+          const itemLabels = (filterItem.labels || []).map(/** @param {any} l */ l => (typeof l === "string" ? l : l.name || ""));
+          if (!requiredLabels.every(r => itemLabels.includes(r))) {
+            core.info(`Skipping add_comment for #${itemNumber}: does not match required-labels filter (${requiredLabels.join(", ")})`);
+            return { success: false, skipped: true, error: `Item does not match required-labels filter` };
+          }
+        }
+        if (requiredTitlePrefix && !filterItem.title?.startsWith(requiredTitlePrefix)) {
+          core.info(`Skipping add_comment for #${itemNumber}: title does not start with required prefix "${requiredTitlePrefix}"`);
+          return { success: false, skipped: true, error: `Item title does not start with required prefix` };
+        }
+      } catch (err) {
+        core.warning(`Could not fetch item #${itemNumber} to check filters: ${getErrorMessage(err)}`);
+        return { success: false, error: `Failed to check required-labels/required-title-prefix filter: ${getErrorMessage(err)}` };
+      }
+    }
+
     // Collect parent issue/PR/discussion authors to allow in @mentions.
     // The body was already sanitized in collect_ndjson_output with allowed mentions from the
     // event payload (which includes the issue author). Re-sanitizing here without the same

setup/js/add_labels.cjs

@@ -41,12 +41,16 @@ const main = createCountGatedHandler({
   handlerType: HANDLER_TYPE,
   setup: async (config, maxCount, isStaged) => {
     const { allowed: allowedLabels = [], blocked: blockedPatterns = [] } = config;
+    const requiredLabels = Array.isArray(config.required_labels) ? config.required_labels : [];
+    const requiredTitlePrefix = config.required_title_prefix || "";
     const { defaultTargetRepo, allowedRepos } = resolveTargetRepoConfig(config);
     const githubClient = await createAuthenticatedGitHubClient(config);
 
     core.info(`Add labels configuration: max=${maxCount}`);
     if (allowedLabels.length > 0) core.info(`Allowed labels: ${allowedLabels.join(", ")}`);
     if (blockedPatterns.length > 0) core.info(`Blocked patterns: ${blockedPatterns.join(", ")}`);
+    if (requiredLabels.length > 0) core.info(`Required labels (all): ${requiredLabels.join(", ")}`);
+    if (requiredTitlePrefix) core.info(`Required title prefix: ${requiredTitlePrefix}`);
     core.info(`Default target repo: ${defaultTargetRepo}`);
     if (allowedRepos.size > 0) core.info(`Allowed repos: ${[...allowedRepos].join(", ")}`);
 
@@ -85,6 +89,26 @@ const main = createCountGatedHandler({
       const requestedLabels = message.labels ?? [];
       core.info(`Requested labels: ${JSON.stringify(requestedLabels)}`);
 
+      // Apply required-labels and required-title-prefix filters
+      if (requiredLabels.length > 0 || requiredTitlePrefix) {
+        const { data: item } = await githubClient.rest.issues.get({
+          owner: repoParts.owner,
+          repo: repoParts.repo,
+          issue_number: itemNumber,
+        });
+        if (requiredLabels.length > 0) {
+          const itemLabels = (item.labels || []).map(/** @param {any} l */ l => (typeof l === "string" ? l : l.name || ""));
+          if (!requiredLabels.every(r => itemLabels.includes(r))) {
+            core.info(`Skipping add_labels for ${contextType} #${itemNumber}: does not match required-labels filter (${requiredLabels.join(", ")})`);
+            return { success: false, skipped: true, error: `Item does not match required-labels filter` };
+          }
+        }
+        if (requiredTitlePrefix && !item.title?.startsWith(requiredTitlePrefix)) {
+          core.info(`Skipping add_labels for ${contextType} #${itemNumber}: title does not start with required prefix "${requiredTitlePrefix}"`);
+          return { success: false, skipped: true, error: `Item title does not start with required prefix` };
+        }
+      }
+
       // If no labels provided, return a helpful message with allowed labels if configured
       if (requestedLabels.length === 0) {
         const labelSource = allowedLabels.length > 0 ? `the allowed list: ${JSON.stringify(allowedLabels)}` : "the repository's available labels";

setup/js/add_reaction_and_edit_comment.cjs

@@ -139,7 +139,8 @@ async function resolveEventEndpoints(eventName, owner, repo, payload) {
 
 async function main() {
   const reaction = process.env.GH_AW_REACTION || "eyes";
-  const command = process.env.GH_AW_COMMAND; // Only present for command workflows
+  const commandsJSON = process.env.GH_AW_COMMANDS;
+  const command = commandsJSON ? (JSON.parse(commandsJSON)[0] ?? null) : null; // Only present for command workflows
   const invocationContext = resolveInvocationContext(context);
   const runUrl = buildWorkflowRunUrl(context, invocationContext.workflowRepo);
 

setup/js/add_reviewer.cjs

@@ -19,7 +19,7 @@ const { processItems } = require("./safe_output_processor.cjs");
 const { getErrorMessage } = require("./error_helpers.cjs");
 const { getPullRequestNumber } = require("./pr_helpers.cjs");
 const { logStagedPreviewInfo } = require("./staged_preview.cjs");
-const { isStagedMode } = require("./safe_output_helpers.cjs");
+const { isStagedMode, checkRequiredFilter } = require("./safe_output_helpers.cjs");
 const { createAuthenticatedGitHubClient } = require("./handler_auth.cjs");
 const { COPILOT_REVIEWER_BOT } = require("./constants.cjs");
 
@@ -35,6 +35,11 @@ async function main(config = {}) {
   const githubClient = await createAuthenticatedGitHubClient(config);
   const isStaged = isStagedMode(config);
 
+  const requiredLabels = Array.isArray(config.required_labels) ? config.required_labels : [];
+  const requiredTitlePrefix = config.required_title_prefix || "";
+  if (requiredLabels.length > 0) core.info(`Required labels (all): ${requiredLabels.join(", ")}`);
+  if (requiredTitlePrefix) core.info(`Required title prefix: ${requiredTitlePrefix}`);
+
   core.info(`Add reviewer configuration: max=${maxCount}`);
   if (allowedReviewers.length > 0) {
     core.info(`Allowed reviewers: ${allowedReviewers.join(", ")}`);
@@ -77,6 +82,10 @@ async function main(config = {}) {
       };
     }
 
+    const repoParts = { owner: context.repo.owner, repo: context.repo.repo };
+    const filterResult = await checkRequiredFilter(githubClient, repoParts, prNumber, requiredLabels, requiredTitlePrefix, HANDLER_TYPE);
+    if (filterResult) return filterResult;
+
     const requestedReviewers = message.reviewers ?? [];
     const requestedTeamReviewers = message.team_reviewers ?? [];
     core.info(`Requested reviewers: ${JSON.stringify(requestedReviewers)}`);

setup/js/assign_milestone.cjs

@@ -7,7 +7,7 @@
 
 const { getErrorMessage } = require("./error_helpers.cjs");
 const { logStagedPreviewInfo } = require("./staged_preview.cjs");
-const { isStagedMode } = require("./safe_output_helpers.cjs");
+const { isStagedMode, checkRequiredFilter } = require("./safe_output_helpers.cjs");
 const { createAuthenticatedGitHubClient } = require("./handler_auth.cjs");
 const { loadTemporaryIdMapFromResolved, resolveRepoIssueTarget } = require("./temporary_id.cjs");
 
@@ -49,6 +49,11 @@ async function main(config = {}) {
   // Check if we're in staged mode
   const isStaged = isStagedMode(config);
 
+  const requiredLabels = Array.isArray(config.required_labels) ? config.required_labels : [];
+  const requiredTitlePrefix = config.required_title_prefix || "";
+  if (requiredLabels.length > 0) core.info(`Required labels (all): ${requiredLabels.join(", ")}`);
+  if (requiredTitlePrefix) core.info(`Required title prefix: ${requiredTitlePrefix}`);
+
   core.info(`Assign milestone configuration: max=${maxCount}, auto_create=${autoCreate}`);
   if (allowedMilestones.length > 0) {
     core.info(`Allowed milestones: ${allowedMilestones.join(", ")}`);
@@ -143,6 +148,11 @@ async function main(config = {}) {
     }
 
     const issueNumber = resolvedIssueTarget.resolved.number;
+
+    const repoParts = { owner: context.repo.owner, repo: context.repo.repo };
+    const filterResult = await checkRequiredFilter(githubClient, repoParts, issueNumber, requiredLabels, requiredTitlePrefix, "assign_milestone");
+    if (filterResult) return filterResult;
+
     if (resolvedIssueTarget.wasTemporaryId) {
       core.info(`Resolved temporary ID '${item.issue_number}' to issue #${issueNumber}`);
     }

setup/js/assign_to_user.cjs

@@ -8,7 +8,7 @@
 const { processItems } = require("./safe_output_processor.cjs");
 const { getErrorMessage } = require("./error_helpers.cjs");
 const { resolveTargetRepoConfig, resolveAndValidateRepo } = require("./repo_helpers.cjs");
-const { resolveIssueNumber, extractAssignees } = require("./safe_output_helpers.cjs");
+const { resolveIssueNumber, extractAssignees, checkRequiredFilter } = require("./safe_output_helpers.cjs");
 const { logStagedPreviewInfo } = require("./staged_preview.cjs");
 const { parseBoolTemplatable } = require("./templatable.cjs");
 const { createAuthenticatedGitHubClient } = require("./handler_auth.cjs");
@@ -31,6 +31,10 @@ const main = createCountGatedHandler({
     const unassignFirst = parseBoolTemplatable(config.unassign_first, false);
     const { defaultTargetRepo, allowedRepos } = resolveTargetRepoConfig(config);
     const githubClient = await createAuthenticatedGitHubClient(config);
+    const requiredLabels = Array.isArray(config.required_labels) ? config.required_labels : [];
+    const requiredTitlePrefix = config.required_title_prefix || "";
+    if (requiredLabels.length > 0) core.info(`Required labels (all): ${requiredLabels.join(", ")}`);
+    if (requiredTitlePrefix) core.info(`Required title prefix: ${requiredTitlePrefix}`);
 
     core.info(`Assign to user configuration: max=${maxCount}, unassign_first=${unassignFirst}`);
     if (allowedAssignees.length > 0) {
@@ -76,6 +80,9 @@ const main = createCountGatedHandler({
       }
       const issueNumber = issueResult.issueNumber;
 
+      const filterResult = await checkRequiredFilter(githubClient, repoParts, issueNumber, requiredLabels, requiredTitlePrefix, HANDLER_TYPE);
+      if (filterResult) return filterResult;
+
       // Extract assignees using shared helper
       const requestedAssignees = extractAssignees(assignItem);
 

setup/js/close_discussion.cjs

@@ -164,6 +164,7 @@ async function main(config = {}) {
   const requiredTitlePrefix = config.required_title_prefix || "";
   const maxCount = config.max || 10;
   const githubClient = await createAuthenticatedGitHubClient(config);
+  const allowBody = config.allow_body !== false; // default true; false only when explicitly set to false
   let allowedMentionAliases = [];
   if (Array.isArray(config.allowedMentionAliases)) {
     allowedMentionAliases = config.allowedMentionAliases;
@@ -174,7 +175,7 @@ async function main(config = {}) {
   // Check if we're in staged mode
   const isStaged = isStagedMode(config);
 
-  core.info(`Close discussion configuration: max=${maxCount}`);
+  core.info(`Close discussion configuration: max=${maxCount}, allow_body=${allowBody}`);
   if (requiredLabels.length > 0) {
     core.info(`Required labels: ${requiredLabels.join(", ")}`);
   }
@@ -278,9 +279,18 @@ async function main(config = {}) {
         };
       }
 
-      // Add comment if body is provided
+      // Add comment if body is provided and allow-body is not false
       let commentUrl;
-      if (item.body) {
+      if (!allowBody) {
+        // allow-body: false — drop any body the agent provided and close without a comment
+        if (item.body) {
+          core.warning(
+            `close_discussion: allow-body is false — dropping non-empty body (length=${item.body.length}) and closing without a comment`
+          );
+        } else {
+          core.info("close_discussion: allow-body is false — closing without a comment");
+        }
+      } else if (item.body) {
         const sanitizedBody = sanitizeContent(item.body, { allowedAliases: allowedMentionAliases });
         const comment = await addDiscussionComment(githubClient, discussion.id, sanitizedBody);
         core.info(`Added comment to discussion #${discussionNumber}: ${comment.url}`);

setup/js/close_entity_helpers.cjs

@@ -269,6 +269,7 @@ function createCloseEntityHandler(config, entityConfig, callbacks, githubClient)
   const comment = config.comment || "";
   const isStaged = isStagedMode(config);
   const { defaultTargetRepo, allowedRepos } = resolveTargetRepoConfig(config);
+  const allowBody = config.allow_body !== false; // default true; false only when explicitly set to false
 
   let processedCount = 0;
 
@@ -293,12 +294,22 @@ function createCloseEntityHandler(config, entityConfig, callbacks, githubClient)
     core.info(`Processing ${entityConfig.itemType} message: ${JSON.stringify(logFields)}`);
 
     // 2. Comment body resolution
-    /** @type {string} */
+    /** @type {string|undefined} */
     let commentToPost;
     /** @type {string} */
     let commentSource = "unknown";
 
-    if (typeof item.body === "string" && item.body.trim() !== "") {
+    if (!allowBody) {
+      // allow-body: false — drop any body the agent provided and skip the comment
+      if (typeof item.body === "string" && item.body.trim() !== "") {
+        core.warning(
+          `${entityConfig.itemType}: allow-body is false — dropping non-empty body (length=${item.body.length}) and closing without a comment`
+        );
+      } else {
+        core.info(`${entityConfig.itemType}: allow-body is false — closing without a comment`);
+      }
+      commentToPost = undefined;
+    } else if (typeof item.body === "string" && item.body.trim() !== "") {
       commentToPost = item.body;
       commentSource = "item.body";
     } else if (typeof comment === "string" && comment.trim() !== "") {
@@ -309,10 +320,12 @@ function createCloseEntityHandler(config, entityConfig, callbacks, githubClient)
       return { success: false, error: "No comment body provided" };
     }
 
-    core.info(`Comment body determined: length=${commentToPost.length}, source=${commentSource}`);
+    if (commentToPost !== undefined) {
+      core.info(`Comment body determined: length=${commentToPost.length}, source=${commentSource}`);
 
-    // 3. Content sanitization
-    commentToPost = sanitizeContent(commentToPost);
+      // 3. Content sanitization
+      commentToPost = sanitizeContent(commentToPost);
+    }
 
     // 4. Target repository / entity number resolution
     const targetResult = callbacks.resolveTarget(item, config, resolvedTemporaryIds);
@@ -379,34 +392,38 @@ function createCloseEntityHandler(config, entityConfig, callbacks, githubClient)
         };
       }
 
-      // 9. Comment posting
-      const commentBody = callbacks.buildCommentBody(commentToPost, item);
-      core.info(`Adding comment to ${entityConfig.displayName} #${entityNumber}: length=${commentBody.length}`);
-
+      // 9. Comment posting (skipped when allow-body: false or no body available)
       /** @type {{id: number, html_url: string}|null} */
       let commentResult = null;
       let commentPosted = false;
-      try {
-        commentResult = await callbacks.addComment(githubClient, owner, repoName, entityNumber, commentBody);
-        commentPosted = true;
-        core.info(`✓ Comment posted to ${entityConfig.displayName} #${entityNumber}: ${commentResult.html_url}`);
-        core.info(`Comment details: id=${commentResult.id}, body_length=${commentBody.length}`);
-      } catch (commentError) {
-        const errorMsg = getErrorMessage(commentError);
-        if (callbacks.continueOnCommentError) {
-          core.error(`Failed to add comment to ${entityConfig.displayName} #${entityNumber}: ${errorMsg}`);
-          core.error(
-            `Error details: ${JSON.stringify({
-              entityNumber,
-              hasBody: !!item.body,
-              bodyLength: item.body ? item.body.length : 0,
-              errorMessage: errorMsg,
-            })}`
-          );
-          // commentPosted stays false; close operation continues
-        } else {
-          throw new Error(`${ERR_API}: Failed to add comment to ${entityConfig.displayName} #${entityNumber}: ${errorMsg}`, { cause: commentError });
+      if (commentToPost !== undefined) {
+        const commentBody = callbacks.buildCommentBody(commentToPost, item);
+        core.info(`Adding comment to ${entityConfig.displayName} #${entityNumber}: length=${commentBody.length}`);
+
+        try {
+          commentResult = await callbacks.addComment(githubClient, owner, repoName, entityNumber, commentBody);
+          commentPosted = true;
+          core.info(`✓ Comment posted to ${entityConfig.displayName} #${entityNumber}: ${commentResult.html_url}`);
+          core.info(`Comment details: id=${commentResult.id}, body_length=${commentBody.length}`);
+        } catch (commentError) {
+          const errorMsg = getErrorMessage(commentError);
+          if (callbacks.continueOnCommentError) {
+            core.error(`Failed to add comment to ${entityConfig.displayName} #${entityNumber}: ${errorMsg}`);
+            core.error(
+              `Error details: ${JSON.stringify({
+                entityNumber,
+                hasBody: !!item.body,
+                bodyLength: item.body ? item.body.length : 0,
+                errorMessage: errorMsg,
+              })}`
+            );
+            // commentPosted stays false; close operation continues
+          } else {
+            throw new Error(`${ERR_API}: Failed to add comment to ${entityConfig.displayName} #${entityNumber}: ${errorMsg}`, { cause: commentError });
+          }
         }
+      } else {
+        core.info(`Skipping comment for ${entityConfig.displayName} #${entityNumber}: no comment body`);
       }
 
       // 10. Entity close (skipped when already closed)