Skip to content

Remove emojis from agent failure messages#23445

Merged
pelikhan merged 2 commits intomainfrom
copilot/remove-emojis-from-failure-messages
Mar 29, 2026
Merged

Remove emojis from agent failure messages#23445
pelikhan merged 2 commits intomainfrom
copilot/remove-emojis-from-failure-messages

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 29, 2026

Removes emojis from all agent failure message templates in actions/setup/md/.

Changes

The following files had emojis removed from headings, <summary> tags, and inline bold text:

  • agent_failure_issue.md — removed 🤖 and 🛠️
  • agent_timeout.md — removed ⏱️
  • app_token_minting_failed.md — removed 🔑
  • assign_copilot_to_created_issues_failure.md — removed 🤖
  • inference_access_error.md — removed 🔑
  • lockdown_check_failed.md — removed 🔒
  • manifest_protection_create_pr_fallback.md — removed 🛡️ and 🔒
  • manifest_protection_push_failed_fallback.md — removed 🛡️, 🔒, and 📋
  • manifest_protection_push_to_pr_fallback.md — removed 🛡️, 🔒, and 📋
  • noop_runs_issue.md — removed 📘, 🎯, and 📚
  • pr_checkout_failure.md — removed ❌

Copilot AI requested a review from pelikhan March 29, 2026 12:02
@pelikhan pelikhan marked this pull request as ready for review March 29, 2026 12:06
Copilot AI review requested due to automatic review settings March 29, 2026 12:06
@pelikhan pelikhan merged commit cc3e969 into main Mar 29, 2026
49 of 51 checks passed
@pelikhan pelikhan deleted the copilot/remove-emojis-from-failure-messages branch March 29, 2026 12:06
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Removes emoji glyphs from agent failure (and related) Markdown message templates under actions/setup/md/ to keep headings and summaries plain-text.

Changes:

  • Removed emojis from template headings, <summary> tags, and bold inline headings in multiple actions/setup/md/*.md templates.
  • Regenerated/updated multiple .github/workflows/*.lock.yml files, resulting in re-ordered import entries in the “Resolved workflow manifest” comment blocks.

Reviewed changes

Copilot reviewed 32 out of 32 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
actions/setup/md/agent_failure_issue.md Removes emojis from <summary> headings in failure issue template.
actions/setup/md/agent_timeout.md Removes emoji from bold failure heading.
actions/setup/md/app_token_minting_failed.md Removes emoji from bold failure heading.
actions/setup/md/assign_copilot_to_created_issues_failure.md Removes emoji from bold failure heading.
actions/setup/md/inference_access_error.md Removes emoji from bold failure heading.
actions/setup/md/lockdown_check_failed.md Removes emoji from bold failure heading.
actions/setup/md/manifest_protection_create_pr_fallback.md Removes emojis from warning header and <summary> text.
actions/setup/md/manifest_protection_push_failed_fallback.md Removes emojis from warning header and <summary> text.
actions/setup/md/manifest_protection_push_to_pr_fallback.md Removes emojis from warning header and <summary> text.
actions/setup/md/noop_runs_issue.md Removes emojis from <summary> section titles.
actions/setup/md/pr_checkout_failure.md Removes emoji from Markdown heading.
.github/workflows/archie.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/cloclo.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/daily-compiler-quality.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/daily-file-diet.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/daily-function-namer.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/daily-testify-uber-super-expert.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/developer-docs-consolidator.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/duplicate-code-detector.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/glossary-maintainer.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/go-fan.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/mcp-inspector.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/q.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/repository-quality-improver.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/semantic-function-refactor.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/sergo.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/smoke-claude.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/smoke-codex.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/smoke-copilot-arm.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/smoke-copilot.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/terminal-stylist.lock.yml Reorders generated “Imports” comment entries.
.github/workflows/typist.lock.yml Reorders generated “Imports” comment entries.

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

Comment on lines 24 to 30
#
# Resolved workflow manifest:
# Imports:
# - shared/mcp/serena-go.md
# - shared/mcp/serena.md
# - shared/mcp/serena-go.md
# - shared/reporting.md
#
Copy link

Copilot AI Mar 29, 2026

Choose a reason for hiding this comment

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

The PR description says changes are limited to removing emojis from templates in actions/setup/md/, but this PR also changes multiple generated workflow lock files (e.g., import ordering in this lock file). Please either (a) update the PR description to explicitly include these lockfile diffs and why they changed, or (b) split/revert the lockfile churn if it's unrelated to the emoji removal.

Copilot uses AI. Check for mistakes.
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