Skip to content

Centralize slash_command routing for perf-improver, pr-fix, and test-improver#353

Merged
pelikhan merged 3 commits into
mainfrom
copilot/add-strategy-centralized-to-slash-command
Jun 26, 2026
Merged

Centralize slash_command routing for perf-improver, pr-fix, and test-improver#353
pelikhan merged 3 commits into
mainfrom
copilot/add-strategy-centralized-to-slash-command

Conversation

Copilot AI commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

gh aw compile --strict flagged these three workflows as outliers: they declared slash_command without centralized routing, causing duplicate per-workflow issue_comment triggers. This aligns them with the repository’s existing centralized slash-command pattern.

  • What changed

    • Added on.slash_command.strategy: centralized in:
      • workflows/perf-improver.md (/perf-assist)
      • workflows/pr-fix.md (/pr-fix)
      • workflows/test-improver.md (/test-assist)
  • Resulting behavior

    • Slash commands for these workflows are routed through centralized command handling (agentic_commands.yml) instead of generating duplicate local trigger wiring.
  • Config pattern applied

    on:
      slash_command:
        name: pr-fix
        strategy: centralized

Co-authored-by: dsyme <7204669+dsyme@users.noreply.github.com>
Copilot AI changed the title [WIP] Add 'strategy: centralized' to slash_command in workflows Centralize slash_command routing for perf-improver, pr-fix, and test-improver Jun 24, 2026
Copilot AI requested a review from dsyme June 24, 2026 18:46
Copilot finished work on behalf of dsyme June 24, 2026 18:46
@pelikhan pelikhan marked this pull request as ready for review June 26, 2026 05:30
@pelikhan pelikhan merged commit 4b12a3d into main Jun 26, 2026
1 check passed
@pelikhan pelikhan deleted the copilot/add-strategy-centralized-to-slash-command branch June 26, 2026 05:31
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