Skip to content

Pin expert review threat detector model - #10684

Merged
Amaury Levé (Evangelink) merged 1 commit into
mainfrom
dev/amauryleve/fix-detection-runs
Aug 24, 2026
Merged

Pin expert review threat detector model#10684
Amaury Levé (Evangelink) merged 1 commit into
mainfrom
dev/amauryleve/fix-detection-runs

Conversation

@Evangelink

Copy link
Copy Markdown
Member

The latest Expert Code Review detection run judged the workflow output safe but emitted malformed result JSON through the default detection alias, creating a parse_error entry in the detection tracker.

Pin threat detection to gpt-5-mini in the shared expert-review configuration and regenerate all three consuming workflows so they no longer rely on the flaky default alias.

Fixes #10438

Use gpt-5-mini for expert-review threat detection to avoid malformed JSON from the default detection alias.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 6965c1cd-983f-48be-8e77-214c442e458c
Copilot AI balanced review requested due to automatic review settings August 24, 2026 10:08

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

Pins expert-review threat detection to gpt-5-mini to prevent malformed result JSON from the default model alias.

Changes:

  • Configures the shared threat-detection engine explicitly.
  • Regenerates all three consuming workflows with the pinned model.
Show a summary per file
File Description
.github/workflows/shared/review-shared.md Pins the shared threat-detection model.
.github/workflows/review.agent.lock.yml Regenerates command-triggered review workflow.
.github/workflows/review-on-open.agent.lock.yml Regenerates automatic PR review workflow.
.github/workflows/review-after-autofix.agent.lock.yml Regenerates post-autofix review workflow.

Review details

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

  • Files reviewed: 4/4 changed files
  • Comments generated: 0
  • Review effort level: Balanced

@Evangelink Amaury Levé (Evangelink) added the state/needs-review Awaiting review from the team. label Aug 24, 2026
@Evangelink
Amaury Levé (Evangelink) enabled auto-merge (squash) August 24, 2026 10:16
@Evangelink
Amaury Levé (Evangelink) merged commit 9c6cc0e into main Aug 24, 2026
16 of 17 checks passed
@Evangelink
Amaury Levé (Evangelink) deleted the dev/amauryleve/fix-detection-runs branch August 24, 2026 10:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state/needs-review Awaiting review from the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[aw] Detection Runs

3 participants