Skip to content

fix: add default-ai-credits-pricing fallback to daily-safeoutputs-git-simulator - #48292

Merged
pelikhan merged 2 commits into
mainfrom
copilot/aw-fix-daily-safe-outputs-simulator
Jul 27, 2026
Merged

fix: add default-ai-credits-pricing fallback to daily-safeoutputs-git-simulator#48292
pelikhan merged 2 commits into
mainfrom
copilot/aw-fix-daily-safe-outputs-simulator

Conversation

CopilotAI commented Jul 27, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor

When the AWF bootstrap fetch fails (401), the proxy loses its remote pricing table. If claude-opus-5 is selected as the model at that point, the proxy has no pricing entry for it and immediately rejects the request with HTTP 400 — terminating the workflow after one turn.

Change

Added models.default-ai-credits-pricing to daily-safeoutputs-git-simulator.md frontmatter, matching claude-opus-5's actual rates from pkg/cli/data/models.json:

models:
default-ai-credits-pricing:
input: 5.0output: 25.0

This provides the AWF proxy a static fallback rate so it can compute AI credits cost regardless of whether bootstrap data is available.

CopilotAI linked an issue Jul 27, 2026 that may be closed by this pull request
[aw] Daily Safe Outputs Git Simulator failed #48284
Closed
@github-actionsgithub-actionsBot mentioned this pull request Jul 27, 2026
Closed
@github-actions

github-actionsBot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Hey @copilot-swe-agent 👋 — thanks for starting work on the daily safe outputs git simulator failure! This PR is properly structured as a work-in-progress following the core team's agentic development process.

A few things to keep in mind as you complete this:

  • Add tests — when you implement the fix for the claude-opus-5 pricing configuration issue, please include tests that verify the error handling and recovery path.
  • Update the workflow — ensure the daily-safeoutputs-git-simulator.md workflow is also updated with the pricing configuration fix if needed.
  • Remove [WIP] — once the implementation is complete and tested, update the PR title to remove the [WIP] marker.

This looks ready for continued development. Feel free to push commits as you make progress!

Generated by ✅ Contribution Check · sonnet46 · 53.3 AIC · ⌖ 7.74 AIC · ⊞ 6.2K · ◷

@pelikhan
fix: add default-ai-credits-pricing fallback to daily-safeoutputs-git…
1d5a48e
…-simulator workflow
The claude-opus-5 model lacks pricing in the AWF proxy when bootstrap
fails with 401. Adding models.default-ai-credits-pricing with the
correct rate ($5/1M input, $25/1M output) prevents the HTTP 400
rejection when the proxy cannot load remote pricing data.
Closes#48284
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
CopilotAI changed the title [WIP] Fix daily safe outputs git simulator failurefix: add default-ai-credits-pricing fallback to daily-safeoutputs-git-simulatorJul 27, 2026
CopilotAI requested a review from pelikhanJuly 27, 2026 06:16
@pelikhan
pelikhan marked this pull request as ready for review July 27, 2026 06:19
CopilotAI review requested due to automatic review settings July 27, 2026 06:19
@pelikhan
pelikhan merged commit cfd2849 into mainJul 27, 2026
1 check passed
@pelikhan
pelikhan deleted the copilot/aw-fix-daily-safe-outputs-simulator branch July 27, 2026 06:19
CopilotAI reviewed Jul 27, 2026
View reviewed changes

CopilotAI 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

Adds fallback AI-credit pricing so the workflow can continue when remote pricing bootstrap fails.

Changes:

  • Configures static Claude Opus 5 fallback pricing.
  • Regenerates the compiled workflow configuration.
Show a summary per file
FileDescription
.github/workflows/daily-safeoutputs-git-simulator.mdAdds fallback pricing frontmatter.
.github/workflows/daily-safeoutputs-git-simulator.lock.ymlEmits fallback pricing in AWF configuration.

Review details

Tip

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

  • Files reviewed: 2/2 changed files
  • Comments generated: 1
  • Review effort level: Medium

Comment thread.github/workflows/daily-safeoutputs-git-simulator.md
Comment on lines +64 to +65
input: 5.0
output: 25.0
@github-actionsgithub-actionsBot mentioned this pull request Jul 27, 2026
Closed
@github-actions

github-actionsBot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.83.4

This was referenced Jul 28, 2026
Closed
Closed
CopilotAI mentioned this pull request Jul 28, 2026
Merged
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code reviewCopilot
Copilot review effort
Applies to this pull request for everyone.Learn more about Copilot code review.
Copilot left review comments
+1 more reviewer
@pelikhanpelikhanpelikhan approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@pelikhanpelikhan

Copilot code reviewCopilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[aw] Daily Safe Outputs Git Simulator failed

3 participants

@pelikhan