Update GPT pricing catalog and default detection/evals model routing via aliases - #49586
Conversation
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
PR Triage
Draft; pricing catalog + model routing alias changes, moderate behavioral impact, still draft.
|
There was a problem hiding this comment.
Pull request overview
Updates model pricing and routes default detection/evaluation workloads through dedicated small-model aliases.
Changes:
- Refreshes GPT-5.6 pricing in both catalogs.
- Adds
detectionandevalsaliases backed bysmall. - Updates compiler defaults, tests, and generated fixtures.
Show a summary per file
| File | Description |
|---|---|
pkg/workflow/threat_detection_test.go |
Updates inline detection expectations. |
pkg/workflow/threat_detection_inline_engine.go |
Adds the detection fallback. |
pkg/workflow/threat_detection_external.go |
Adds the external-detector fallback. |
pkg/workflow/evals_steps.go |
Defaults evaluations to evals. |
pkg/workflow/evals_config.go |
Documents the new default. |
pkg/workflow/data/model_aliases.json |
Defines detection and evals aliases. |
pkg/workflow/model_aliases_test.go |
Verifies the aliases. |
pkg/cli/data/models.json |
Updates the CLI pricing catalog. |
actions/setup/js/models.json |
Updates the setup-action pricing catalog. |
pkg/workflow/testdata/TestWasmGolden_CompileFixtures/with-imports.golden |
Refreshes generated version metadata. |
pkg/workflow/testdata/TestWasmGolden_CompileFixtures/smoke-copilot.golden |
Refreshes generated version metadata. |
pkg/workflow/testdata/TestWasmGolden_CompileFixtures/playwright-cli-mode.golden |
Refreshes generated version metadata. |
pkg/workflow/testdata/TestWasmGolden_CompileFixtures/basic-copilot.golden |
Refreshes generated version metadata. |
pkg/workflow/testdata/TestWasmGolden_AllEngines/copilot.golden |
Refreshes generated version metadata. |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 14/14 changed files
- Comments generated: 3
- Review effort level: Balanced
| // Model is the default LLM model to use for evaluations. Use a model alias such as | ||
| // "small" or a full model ID. Per-question Model fields override this value. | ||
| // When empty, the compiler default ("small") is used. | ||
| // When empty, the compiler default ("evals") is used. |
| if model == "" { | ||
| model = "small" | ||
| model = "evals" | ||
| } |
| if resolvedDetectionModel == "" { | ||
| resolvedDetectionModel = "detection" | ||
| } |
|
@copilot merge main and recompile |
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
…pricing-table Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
|
✅ Smoke Copilot Auto completed successfully! |
|
🚀 Smoke Gemini MISSION COMPLETE! Gemini has spoken. ✨ Warning threat detection engine error |
|
🚀 Smoke Pi MISSION COMPLETE! Pi delivered. 🥧 No PR comment/label action taken: workflow lacks pull_request triggering context; issue created with smoke test results. |
|
✨ The prophecy is fulfilled... Smoke Codex has completed its mystical journey. The stars align. 🌟 Warning threat detection engine error |
|
🎬 THE END — Smoke Claude MISSION: ACCOMPLISHED! The hero saves the day! ✨ |
|
✅ All tools validated successfully! Agent Container Smoke Test confirms agent container is ready. |
|
📰 BREAKING: Smoke Copilot - AOAI (Entra) is now investigating this pull request. Sources say the story is developing... |
|
✅ Smoke Copilot Small completed successfully! |
|
✅ Smoke OTEL completed successfully! |
|
📰 BREAKING: Smoke Copilot - AOAI (apikey) is now investigating this pull request. Sources say the story is developing... |
|
✅ Smoke Copilot MAI completed successfully! |
|
📰 BREAKING: Smoke Copilot is now investigating this pull request. Sources say the story is developing... |
|
Small names, clear routes now— Inspired by the PR's new
|
|
Warning threat detection engine error Smoke Test Results
Overall Status: FAIL
|
Agent Container Tool Check
Result: 12/12 tools available ✅ Overall status: PASS
|
|
Smoke test summary:
|
|
Commit pushed:
|
|
Warning threat detection engine error Comment MemoryNote This comment is managed by comment memory.It stores persistent context for this thread in the code block at the top of this comment. Warning Firewall blocked 6 domainsThe following domains were blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "accounts.google.com"
- "android.clients.google.com"
- "clients2.google.com"
- "contentautofill.googleapis.com"
- "safebrowsingohttpgateway.googleapis.com"
- "www.google.com"See Network Configuration for more information.
|
Smoke Test Copilot - ResultsPR: Update GPT pricing catalog and default detection/evals model routing via aliases
Overall: FAIL cc Warning Firewall blocked 6 domainsThe following domains were blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "accounts.google.com"
- "android.clients.google.com"
- "clients2.google.com"
- "contentautofill.googleapis.com"
- "safebrowsingohttpgateway.googleapis.com"
- "www.google.com"See Network Configuration for more information.
|
Comment MemoryNote This comment is managed by comment memory.It stores persistent context for this thread in the code block at the top of this comment. Warning Firewall blocked 6 domainsThe following domains were blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "accounts.google.com"
- "android.clients.google.com"
- "clients2.google.com"
- "contentautofill.googleapis.com"
- "safebrowsingohttpgateway.googleapis.com"
- "www.google.com"See Network Configuration for more information.
|
There was a problem hiding this comment.
Smoke test automated review (test).
Warning
Firewall blocked 6 domains
The following domains were blocked by the firewall during workflow execution:
accounts.google.comandroid.clients.google.comclients2.google.comcontentautofill.googleapis.comsafebrowsingohttpgateway.googleapis.comwww.google.com
To allow these domains, add them to the network.allowed list in your workflow frontmatter:
network:
allowed:
- defaults
- "accounts.google.com"
- "android.clients.google.com"
- "clients2.google.com"
- "contentautofill.googleapis.com"
- "safebrowsingohttpgateway.googleapis.com"
- "www.google.com"See Network Configuration for more information.
📰 BREAKING: Report filed by Smoke Copilot · auto · 42.8 AIC · ⌖ 3.66 AIC · ⊞ 8.7K
Comment /smoke-copilot to run again
Add label smoke to run again
There was a problem hiding this comment.
💥 Automated smoke test review - all systems nominal! Run 30715626945
Warning
Firewall blocked 6 domains
The following domains were blocked by the firewall during workflow execution:
accounts.google.comandroid.clients.google.comclients2.google.comcontentautofill.googleapis.comsafebrowsingohttpgateway.googleapis.comwww.google.com
To allow these domains, add them to the network.allowed list in your workflow frontmatter:
network:
allowed:
- defaults
- "accounts.google.com"
- "android.clients.google.com"
- "clients2.google.com"
- "contentautofill.googleapis.com"
- "safebrowsingohttpgateway.googleapis.com"
- "www.google.com"See Network Configuration for more information.
💥 [THE END] — Illustrated by Smoke Claude · sonnet46 · 103.7 AIC · ⌖ 10 AIC · ⊞ 6.3K
Comment /smoke-claude to run again
Comments that could not be inline-anchored
.changeset/patch-update-model-pricing-and-default-routing.md:5
📝 Smoke Test Review Comment #1 — Run 30715626945: The changeset description clearly captures the intent of this PR. Consider also noting the compiler fixture updates in the summary for full traceability.
.github/workflows/ab-testing-advisor.lock.yml:840
🔍 Smoke Test Review Comment #2 — Run 30715626945: The model routing update here looks consistent with the new alias-based approach. Verify that all lock files have been recompiled consistently across all workflow variants.
|
Smoke Test 30715680366: FAIL Warning Firewall blocked 1 domainThe following domain was blocked by the firewall during workflow execution:
To allow these domains, add them to the network:
allowed:
- defaults
- "dc.services.visualstudio.com"See Network Configuration for more information.
|
|
🎉 This pull request is included in a new release. Release: |
This PR updates GPT pricing entries in the GitHub/OpenAI model catalog and introduces explicit
detection/evalsaliases for small-model routing. Detection and evals now default to these aliases when no model is configured, instead of relying on implicit fallback behavior.Pricing catalog updates (GitHub/OpenAI)
models.jsonfor both runtime copies:pkg/cli/data/models.jsonactions/setup/js/models.jsonNew built-in model aliases
pkg/workflow/data/model_aliases.json:detection: ["small"]evals: ["small"]Default model behavior changes
"detection"when no model is set (both inline and external detection paths)."evals"when no model is set (replacing the hardcoded"small"fallback).GH_AW_DEFAULT_DETECTION_MODEL,GH_AW_DEFAULT_EVALS_MODEL) still take precedence.Compiler fixture/test alignment
✨ PR Review Safe Output Test - Run 30715626945
Warning
Firewall blocked 6 domains
The following domains were blocked by the firewall during workflow execution:
accounts.google.comandroid.clients.google.comclients2.google.comcontentautofill.googleapis.comsafebrowsingohttpgateway.googleapis.comwww.google.comTo allow these domains, add them to the
network.allowedlist in your workflow frontmatter:See Network Configuration for more information.