Skip to content

fix: clarify issue format guard label message - #2971

Merged
stranske merged 1 commit into
mainfrom
automation/manager-format-message
Aug 7, 2026
Merged

fix: clarify issue format guard label message#2971
stranske merged 1 commit into
mainfrom
automation/manager-format-message

Conversation

@stranske

@stranske stranske commented Aug 7, 2026

Copy link
Copy Markdown
Owner

Clarifies that an invalid issue clears the stale agents:formatted marker regardless of whether the issue is eligible for optimizer routing. Updates the canonical workflow and synced consumer template together.\n\nValidation: python -m pytest -q tests/scripts/test_issue_format.py; python scripts/sync_tool_versions.py --check; git diff --check.

Summary by CodeRabbit

  • Documentation
    • Clarified workflow messages to indicate when the stale agents:formatted label is cleared.
    • Removed references to issue rerouting from invalid-issue cleanup messages.

@cursor

cursor Bot commented Aug 7, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@coderabbitai

coderabbitai Bot commented Aug 7, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: f3773604-1f50-4f01-8a9a-99c226f722c6

📥 Commits

Reviewing files that changed from the base of the PR and between 1e9ad1e and f426077.

📒 Files selected for processing (2)
  • .github/workflows/agents-issue-format-guard.yml
  • templates/consumer-repo/.github/workflows/agents-issue-format-guard.yml

📝 Walkthrough

Walkthrough

The pull request updates cleanup log messages in the main and consumer-repository issue format guard workflows. Both messages now state that the stale agents:formatted label was cleared.

Changes

Issue format guard logging

Layer / File(s) Summary
Update cleanup log messages
.github/workflows/agents-issue-format-guard.yml, templates/consumer-repo/.github/workflows/agents-issue-format-guard.yml
The invalid-issue cleanup messages now state that the stale agents:formatted label was cleared and no longer mention rerouting.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested labels: autofix:patch

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the change to the issue format guard label message.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch automation/manager-format-message

Comment @coderabbitai help to get the list of available commands.

@stranske
stranske marked this pull request as ready for review August 7, 2026 12:30
Copilot AI lite review requested due to automatic review settings August 7, 2026 12:30
@agents-workflows-bot

Copy link
Copy Markdown
Contributor

Workflow source needed

PR #2971 needs either a linked GitHub issue or one valid non-issue Workflow Source before PR metadata automation can manage it safely.

Please do one of:

  • Add <!-- meta:issue:123 --> or a normal Closes #123 / Related to #123 line.
  • Check one Workflow Source option in the PR body.
  • Add a hidden marker such as <!-- workflow-source:local_request -->, <!-- workflow-source:manual_remote -->, <!-- workflow-source:review_followup -->, <!-- workflow-source:sync_campaign -->, or <!-- workflow-source:dependabot -->.
  • Add a workflow source label such as workflow:source-direct-pr, workflow:source-local-request, workflow:source-review-followup, workflow:source-sync, or workflow:no-automation.

Once a valid source is present, this warning will not be reposted.

@stranske
stranske had a problem deploying to agent-high-privilege August 7, 2026 12:30 — with GitHub Actions Error
@stranske-keepalive

Copy link
Copy Markdown
Contributor

Automated Status Summary

Head SHA: ce28bba
Latest Runs: ⏳ pending — Gate
Required contexts: summary
Required: core tests (3.12): ⏳ pending, core tests (3.13): ⏳ pending, docker smoke: ⏳ pending, gate: ⏳ pending

Workflow / Job Result Logs
(no jobs reported) ⏳ pending

Coverage Overview

  • Coverage history entries: 0

Updated automatically; will refresh on subsequent CI/Docker completions.


Keepalive checklist

Scope

No scope information available

Tasks

  • No tasks defined

Acceptance criteria

  • No acceptance criteria defined

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Clarifies the log output emitted when the workflow removes a stale agents:formatted label after detecting an invalid issue body, making it explicit that the label removal is independent of any subsequent rerouting/dispatch behavior. This update is applied to both the canonical workflow and the synced consumer template to keep them aligned.

Changes:

  • Update the success-path echo message after removing agents:formatted to say it cleared a stale marker (instead of implying rerouting).
  • Apply the same message update to templates/consumer-repo/ to avoid template drift.

Reviewed changes

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

File Description
.github/workflows/agents-issue-format-guard.yml Updates the label-removal log message to accurately describe clearing a stale agents:formatted marker.
templates/consumer-repo/.github/workflows/agents-issue-format-guard.yml Mirrors the same log message update in the consumer template to keep it in sync with the canonical workflow.

@stranske
stranske merged commit 9180ebd into main Aug 7, 2026
62 of 69 checks passed
@stranske
stranske deleted the automation/manager-format-message branch August 7, 2026 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants