Skip to content

Add skillet skill-routed PR reviewer workflow#40399

Merged
pelikhan merged 8 commits into
mainfrom
copilot/skilled-reviewer-workflow
Jun 20, 2026
Merged

Add skillet skill-routed PR reviewer workflow#40399
pelikhan merged 8 commits into
mainfrom
copilot/skilled-reviewer-workflow

Conversation

Copilot AI commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

This adds a new PR review workflow, skillet, that treats slash commands as skill selectors under .github/skills. It pre-filters unmatched commands before agent execution, then runs a Copilot-based review with PR-safe outputs.

  • New workflow: skillet

    • Adds .github/workflows/skillet.md as a centralized slash-command reviewer with the 🍳 identity.
    • Triggers on PR comment and PR review comment slash commands.
    • Interprets the command name as a repository skill name and passes the remaining comment text through as the review request.
  • Pre-activation skill gating

    • Adds a pre-activation matcher that:
      • checks PR context,
      • resolves /.github/skills/<command>/SKILL.md,
      • checks centralized command routing metadata from the generated command router workflow,
      • exposes the matched skill path and trailing request text,
      • cancels before agent execution when no skill matches.
    • When neither a repository skill nor a centralized command route matches, pre-activation posts a pull request comment listing the available commands.
    • This keeps non-skill commands out of the expensive review path while giving contributors an in-band command discovery fallback.
  • PR review execution model

    • Runs Copilot with copilot-requests: write.
    • Reuses the standard PR review safe-output pattern:
      • inline review comments,
      • final review submission,
      • optional check-run on approval,
      • explicit noop when nothing actionable is found.
  • Catch-all slash-command support

    • Extends slash-command validation/runtime support to allow bare * as a catch-all centralized route.
    • Keeps prefix wildcard behavior (for example smoke*) intact while making * an explicit non-empty catch-all.
  • Routing coverage

    • Adds test coverage for catch-all slash routing so the dispatched workflow receives the actual matched command name.

Example of the new routing shape:

on:
  slash_command:
    strategy: centralized
    name: "*"
    events: [pull_request_comment, pull_request_review_comment]

A comment like:

/developer review auth changes

now resolves to .github/skills/developer/SKILL.md, and review auth changes becomes the reviewer’s task-specific request.

Copilot AI and others added 3 commits June 19, 2026 21:26
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Comment thread .github/workflows/skillet.md Outdated
shell: bash
run: |
set -euo pipefail
python <<'PY'

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@copilot use actions/github-script instead of python

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed in fb2b987.

Copilot AI and others added 2 commits June 20, 2026 00:14
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI requested a review from pelikhan June 20, 2026 00:21
@pelikhan pelikhan marked this pull request as ready for review June 20, 2026 00:30
Copilot AI review requested due to automatic review settings June 20, 2026 00:30

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@pelikhan

Copy link
Copy Markdown
Collaborator

@copilot when skillet can't find a matching slash command (and also there isn't any matching commands in the agentic-maintenance.yml file), it should post a comment with the available commands. Must be done in pre-activation.

@github-actions

Copy link
Copy Markdown
Contributor

@copilot please reply with the available slash commands before activation.

Generated by 👨‍🍳 PR Sous Chef · 38.5 AIC · ⌖ 1.02 AIC · ⊞ 17.2K ·

Copilot AI and others added 2 commits June 20, 2026 01:13
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

Copilot AI commented Jun 20, 2026

Copy link
Copy Markdown
Contributor Author

@copilot when skillet can't find a matching slash command (and also there isn't any matching commands in the agentic-maintenance.yml file), it should post a comment with the available commands. Must b...

Fixed in 9c6f77e.

@github-actions

Copy link
Copy Markdown
Contributor

@copilot review all comments and address unresolved review feedback.

Generated by 👨‍🍳 PR Sous Chef · 40.4 AIC · ⌖ 1 AIC · ⊞ 17.2K ·

@github-actions

Copy link
Copy Markdown
Contributor

@copilot please refresh the branch and summarize any remaining blockers.

Generated by 👨‍🍳 PR Sous Chef · 40.4 AIC · ⌖ 1 AIC · ⊞ 17.2K ·

@pelikhan pelikhan merged commit 7e30cbf into main Jun 20, 2026
44 checks passed
@pelikhan pelikhan deleted the copilot/skilled-reviewer-workflow branch June 20, 2026 02:02
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