Skip to content

fix(boot): resolve built-in provider pricing safely#4040

Draft
taibai233 wants to merge 1 commit into
esengine:main-v2from
taibai233:fix/pricing-fallback
Draft

fix(boot): resolve built-in provider pricing safely#4040
taibai233 wants to merge 1 commit into
esengine:main-v2from
taibai233:fix/pricing-fallback

Conversation

@taibai233

Copy link
Copy Markdown
Contributor

Summary

Splits the provider pricing fallback out of the Guardian Subagent work into a standalone PR.

  • Adds resolvePricing for boot-time agent/task/skill/planner pricing wiring.
  • Preserves explicitly configured provider prices.
  • Falls back only when the provider identity matches a built-in provider (name, kind, base_url) and the built-in provider serves the model.
  • Avoids assigning built-in pricing to third-party providers that happen to expose the same model name.

Why

The Guardian PR introduced pricing fallback while wiring Guardian usage display, but this affects executor, planner, task, and skill pricing globally. Keeping it separate makes the behavior easier to review and avoids mixing it with Guardian approval semantics.

Validation

  • go test ./internal/boot -run TestResolvePricing

@github-actions github-actions Bot added v2 Go rewrite (1.x) — main-v2 branch, active development config Configuration & setup (internal/config) labels Jun 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

config Configuration & setup (internal/config) v2 Go rewrite (1.x) — main-v2 branch, active development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant