Skip to content

Route PR code-quality reviews through the Copilot gateway - #52377

Merged
pelikhan merged 2 commits into
mainfrom
copilot/deep-report-fix-api-endpoint
Aug 13, 2026
Merged

Route PR code-quality reviews through the Copilot gateway#52377
pelikhan merged 2 commits into
mainfrom
copilot/deep-report-fix-api-endpoint

Conversation

Copilot AI commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

The Pi-based PR Code Quality Reviewer fell back to its native Copilot provider, attempting the blocked api.individual.githubcopilot.com endpoint. Other Copilot workflows use the allowlisted gateway path.

  • Explicit model routing

    • Set the reviewer’s main model to copilot/gpt-5.4.
    • Regenerated workflow output now configures Pi’s aw-gateway provider through api-proxy.
    engine:
      id: pi
    model: copilot/gpt-5.4
  • Regression coverage

    • Assert the workflow retains an explicit main-agent Copilot model, preventing fallback to Pi’s native provider.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix misdirected api.individual.githubcopilot.com hostname in PR Code Quality Reviewer Route PR code-quality reviews through the Copilot gateway Aug 12, 2026
Copilot AI requested a review from pelikhan August 12, 2026 22:13
@github-actions

Copy link
Copy Markdown
Contributor

PR Triage Summary

Total score: 70/100
Breakdown: Impact 32 (fixes broken reviewer routing) + Urgency 20 (blocks quality-review workflow) + Quality 18 (small diff, CI unstable but draft) -> FAST TRACK

Automated triage — see labels applied for classification. Full report in the triage issue.

Generated by 🔧 PR Triage Agent · auto · 43.1 AIC · ⌖ 2.43 AIC · ⊞ 7.8K ·

@pelikhan
pelikhan marked this pull request as ready for review August 13, 2026 02:54
Copilot AI balanced review requested due to automatic review settings August 13, 2026 02:54
@pelikhan
pelikhan merged commit b2ef1f3 into main Aug 13, 2026
@pelikhan
pelikhan deleted the copilot/deep-report-fix-api-endpoint branch August 13, 2026 02:54

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

Routes the PR quality reviewer through the allowlisted Copilot gateway instead of Pi’s native provider.

Changes:

  • Pins the main agent to copilot/gpt-5.4.
  • Regenerates gateway-backed workflow configuration.
  • Adds regression coverage for model selection.
Show a summary per file
File Description
.github/workflows/pr-code-quality-reviewer.md Declares the explicit Copilot model.
.github/workflows/pr-code-quality-reviewer.lock.yml Configures generated gateway routing.
pkg/cli/pr_code_quality_reviewer_workflow_contract_test.go Adds a model contract assertion, but does not scope it to frontmatter.

Review details

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 3/3 changed files
  • Comments generated: 1
  • Review effort level: Balanced

require.NoError(t, err, "Should read pr-code-quality-reviewer workflow")

text := string(content)
assert.Contains(t, text, "model: copilot/gpt-5.4", "Main agent should use an explicit Copilot model")
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[deep-report] Fix misdirected api.individual.githubcopilot.com hostname in PR Code Quality Reviewer

3 participants