-
Notifications
You must be signed in to change notification settings - Fork 111
Separate no-op run comments from failed runs issue #14589
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…flows] No-Op Runs" Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR updates the no-op workflow run reporting so that no-op comments are posted to a dedicated tracking issue ([agentic-workflows] No-Op Runs) instead of being mixed into the failures tracking issue, improving operational triage and separation of signal.
Changes:
- Updated the parent tracking issue title used by
handle_noop_message.cjsto"[agentic-workflows] No-Op Runs". - Updated log/variable/test expectations to align with the new “No-Op Runs” parent issue naming.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| actions/setup/js/handle_noop_message.cjs | Switches the parent issue title and related messaging to target the dedicated no-op runs issue. |
| actions/setup/js/handle_noop_message.test.cjs | Updates test expectations to assert the new issue title and updated warning text. |
Comments suppressed due to low confidence (1)
actions/setup/js/handle_noop_message.cjs:148
- The PR description says terminology was renamed throughout
handle_noop_message.cjs, but there are still references to an "agent runs issue" in comments (e.g., the function header comment aroundmain()and the inline comment above the agentConclusion check). Please update those remaining references for consistency with the new "No-Op Runs" issue.
core.info("Agent succeeded with only noop outputs - posting to no-op runs issue");
const { owner, repo } = context.repo;
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -12,12 +12,12 @@ const { renderTemplate } = require("./messages_core.cjs"); | |||
| */ | |||
| async function ensureAgentRunsIssue() { | |||
Copilot
AI
Feb 9, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The helper name ensureAgentRunsIssue no longer matches its behavior (it now searches/creates the "[agentic-workflows] No-Op Runs" issue). Please rename the function/export (and any callers) to reflect no-op runs, and update any remaining in-file comments/log text that still refer to an "agent runs" issue to avoid confusion during debugging.
This issue also appears on line 146 of the same file.
| async function ensureAgentRunsIssue() { | |
| async function ensureNoOpRunsIssue() { |
No-op workflow run comments were being posted to the "[agentic-workflows] Failed runs" issue. They need a separate tracking issue to distinguish successful no-action runs from actual failures.
Changes
"[agentic-workflows] Agent runs"→"[agentic-workflows] No-Op Runs"handle_noop_message.cjsResult
Two distinct tracking issues:
"[agentic-workflows] Failed runs"- workflow failures (viahandle_agent_failure.cjs)"[agentic-workflows] No-Op Runs"- successful no-action runs (viahandle_noop_message.cjs)Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.