Update noop comment footer and move issue template to markdown file

[Copilot]

Plan: Update noop comment footer to use quote markdown syntax

• Understand the issue and existing code
• Update handle_noop_message.cjs footer to use quote markdown syntax with url link
• Remove timestamp from noop message (no longer needed)
• Test the changes (all 9 tests passing)
• Run linters and formatters (all passing)
• Move noop runs issue template to actions/setup/md
• Update handle_noop_message.cjs to render template from file
• Update tests to mock fs.readFileSync
• Remove Safe Outputs Reference link from template
• Add no-action instruction and XML marker
• Final validation (complete)

Summary

Updated the noop comment footer and moved the noop runs issue template to a separate markdown file for better maintainability.

Changes Made

1. Removed timestamp field - No longer displayed in noop messages
2. Updated footer format - Changed from plain text with horizontal rule to quote markdown syntax: > Generated from [workflow_name](run_url)
3. Created template file - Moved noop runs issue body content to actions/setup/md/noop_runs_issue.md
4. Updated code to render template - Modified handle_noop_message.cjs to read and render the template file
5. Updated tests - Added fs.readFileSync mocking to handle template loading in tests
6. Removed Safe Outputs link - Removed the Safe Outputs Reference link from the noop runs issue template
7. Added no-action instruction - Added clear instruction in template: "No action to take - Do not assign to an agent"
8. Added XML marker - Added <!-- gh-aw-noop-runs --> marker for searchability and programmatic identification

Original prompt

Update the noop comment footer to use quote markdown syntax with url link - similar to other footers, no need for timestamp.

See #14601 (comment)

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[pelikhan]

@copilot move template to actions/setup/md and render

[Copilot]

@copilot move template to actions/setup/md and render

Done. Moved the noop runs issue template to actions/setup/md/noop_runs_issue.md and updated handle_noop_message.cjs to read and render it from the file. Commit: 1f0217c

[pelikhan]

@copilot Remove line

[Copilot]

Done. Removed the Safe Outputs Reference link from the template. Commit: 31a4bc2

[pelikhan]

@copilot

• add one more comment: no action to take, do not assign to an agent.
• add xml marker

[Copilot]

Done. Added "No action to take - Do not assign to an agent" instruction and <!-- gh-aw-noop-runs --> XML marker. Commit: f5a5635

[Copilot]

Pull request overview

Updates how “no-op runs” are tracked and displayed by switching the no-op issue body to a dedicated markdown template file and adjusting the posted comment footer format to match other workflow-generated footers.

Changes:

• Add actions/setup/md/noop_runs_issue.md as the source template for the “[agentic-workflows] No-Op Runs” parent issue body.
• Update handle_noop_message.cjs to load the template from /opt/gh-aw/prompts/noop_runs_issue.md and change the no-op comment footer to quote-markdown format (and remove timestamp).
• Update tests to mock fs.readFileSync for template loading.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
actions/setup/md/noop_runs_issue.md	Introduces a dedicated markdown template for the no-op runs parent issue body.
actions/setup/js/handle_noop_message.cjs	Loads issue body template from prompts dir and updates the posted comment footer formatting.
actions/setup/js/handle_noop_message.test.cjs	Mocks template loading via fs.readFileSync to support the new runtime template dependency.

Comments suppressed due to low confidence (2)

actions/setup/js/handle_noop_message.cjs:55

• generateFooterWithExpiration appends the > - [x] expires ... line to the very end of parentBodyContent. Because the template ends with an XML marker (<!-- gh-aw-noop-runs -->), this will place the expiration line after the marker and break the “marker-at-end” layout (and the helper comment says expiration should be inside the quoted section). Use addExpirationToFooter(parentBodyContent, expiresHours, ...) (or otherwise insert the expiration line before the XML marker) so the XML marker stays last and the expiration stays in the intended quoted block.

  // Load template from file
  const templatePath = "/opt/gh-aw/prompts/noop_runs_issue.md";
  const parentBodyContent = fs.readFileSync(templatePath, "utf8");

  // Add expiration marker (30 days from now) inside the quoted section using helper
  const footer = generateFooterWithExpiration({
    footerText: parentBodyContent,
    expiresHours: 24 * 30, // 30 days
  });
  const parentBody = footer;

actions/setup/js/handle_noop_message.cjs:8

• renderTemplate is required but never used in this module. Please remove the unused import to avoid dead code and keep the dependency surface minimal.

const { generateFooterWithExpiration } = require("./ephemerals.cjs");
const { renderTemplate } = require("./messages_core.cjs");

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.