Skip to content

Pin Matt Pocock reviewer to supported Copilot model#34148

Merged
pelikhan merged 2 commits into
copilot/fix-safe-outputs-job-failurefrom
copilot/review-misconfigured-model
May 23, 2026
Merged

Pin Matt Pocock reviewer to supported Copilot model#34148
pelikhan merged 2 commits into
copilot/fix-safe-outputs-job-failurefrom
copilot/review-misconfigured-model

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 23, 2026

The Matt Pocock Skills Reviewer workflow was failing in the agent step due to model-level request misconfiguration (400 on anthropic-beta header). This change makes model selection explicit and consistent so the review job no longer depends on fallback resolution.

  • Model configuration

    • Set an explicit workflow model in mattpocock-skills-reviewer.md:
      • engine.id: copilot
      • engine.model: claude-sonnet-4.6
  • Compiled workflow alignment

    • Regenerated mattpocock-skills-reviewer.lock.yml so all runtime surfaces use the same pinned model (activation metadata, COPILOT_MODEL in agent/detection, and conclusion metadata).
  • Behavioral effect

    • Removes reliance on ${{ vars.GH_AW_MODEL_*_COPILOT || 'claude-sonnet-4.5' }} fallback paths for this workflow.
    • Ensures deterministic model routing for the reviewer pipeline.
engine:
  id: copilot
  model: claude-sonnet-4.6
  max-continuations: 6

Copilot AI and others added 2 commits May 23, 2026 01:20
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
@pelikhan pelikhan marked this pull request as ready for review May 23, 2026 01:27
Copilot AI review requested due to automatic review settings May 23, 2026 01:27
@pelikhan pelikhan merged commit a2f57e5 into copilot/fix-safe-outputs-job-failure May 23, 2026
@pelikhan pelikhan deleted the copilot/review-misconfigured-model branch May 23, 2026 01:27
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

Pins the Matt Pocock Skills Reviewer workflow to an explicit GitHub Copilot model to avoid relying on variable-based fallback model resolution and to prevent agent-step failures caused by misconfigured model-level requests.

Changes:

  • Added an explicit engine.model: claude-sonnet-4.6 to the source workflow markdown.
  • Regenerated the compiled lock workflow so the pinned model is reflected across activation metadata and both agent/detection COPILOT_MODEL envs.
Show a summary per file
File Description
.github/workflows/mattpocock-skills-reviewer.md Adds explicit engine.model to make model selection deterministic for this workflow.
.github/workflows/mattpocock-skills-reviewer.lock.yml Updates generated workflow artifact to consistently use the pinned model across runtime surfaces and metadata.

Copilot's findings

Tip

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

  • Files reviewed: 2/2 changed files
  • Comments generated: 0

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