Skip to content

Fix Duplicate Code Detector Copilot model - #8626

Merged
lpcox merged 2 commits into
mainfrom
copilot/aw-fix-duplicate-code-detector-issue
Sep 16, 2026
Merged

lpcox merged 2 commits into
mainfrom
copilot/aw-fix-duplicate-code-detector-issue

Conversation

Copilot AI commented Sep 16, 2026 •

Copy link
Copy Markdown
Contributor

The scheduled Duplicate Code Detector workflow was failing before analysis with a Copilot engine HTTP 400 Bad Request, consistent with request validation rejecting the configured model.

  • Workflow model
    • Changed the detector from gpt-5.4-mini to the supported gpt-5-mini model.
model: gpt-5-mini
engine:
  id: copilot
  • Generated lockfile
    • Regenerated the corresponding .lock.yml so runtime metadata and Copilot environment values use the same model string.

Copilot AI changed the title [WIP] Fix HTTP 400 bad request for duplicate code detector Fix Duplicate Code Detector Copilot model Sep 16, 2026
Copilot AI requested a review from lpcox September 16, 2026 15:16
@lpcox
lpcox marked this pull request as ready for review September 16, 2026 16:13
Copilot AI balanced review requested due to automatic review settings September 16, 2026 16:13

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.

🟢 Approval recommended

No unresolved review comments remain, and both files consistently use the supported model.

Pull request overview

Updates the Duplicate Code Detector to use the supported gpt-5-mini Copilot model and synchronizes its lockfile.

Changes:

  • Updated the workflow model configuration.
  • Regenerated matching lockfile metadata and environment values.
File summaries
File Description
.github/workflows/duplicate-code-detector.md Sets the Copilot model to gpt-5-mini.
.github/workflows/duplicate-code-detector.lock.yml Propagates the updated model through runtime configuration.
Review details
  • Files reviewed: 2/2 changed files
  • Comments generated: 0
  • Review effort level: Lite (auto)

Note

Copilot is running an experiment and ran this review at Lite.


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

@github-actions

Copy link
Copy Markdown
Contributor

✅ Copilot review passed with no inline comments.

@copilot Add the ready-for-aw label to this PR to trigger agentic CI smoke tests.

@github-actions

Copy link
Copy Markdown
Contributor

✅ Coverage Check Passed

Overall Coverage

Metric Base PR Delta
Lines 94.12% 94.13% ➡️ +0.01%
Statements 92.80% 92.81% 📈 +0.01%
Functions 93.44% 93.44% ➡️ +0.00%
Branches 86.63% 86.64% 📈 +0.01%
📁 Per-file Coverage Changes (1 files)
File Lines (Before → After) Statements (Before → After)
src/log-directory-setup.ts 96.2% → 100.0% (+3.78%) 96.3% → 100.0% (+3.71%)

Coverage comparison generated by scripts/ci/compare-coverage.ts

@lpcox
lpcox merged commit 54e0e55 into main Sep 16, 2026
26 of 30 checks passed
@lpcox
lpcox deleted the copilot/aw-fix-duplicate-code-detector-issue branch September 16, 2026 16:37
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.

[aw] Duplicate Code Detector hit HTTP 400 bad request

3 participants