Gateway/config: add models.pricing.enabled to skip pricing bootstrap#68651
Gateway/config: add models.pricing.enabled to skip pricing bootstrap#68651palewire wants to merge 7 commits intoopenclaw:mainfrom
Conversation
Greptile SummaryAdds Confidence Score: 5/5Safe to merge; the only finding is a P2 changelog placement style issue. All changed surfaces (type, schema, generated baseline, gateway logic, tests) are consistent and correct. The === false gate is intentionally strict so absent/true config falls through to the existing bootstrap path. The single finding is a changelog ordering convention violation, which is non-blocking. CHANGELOG.md — new entry placed at the top of ### Changes instead of appended at the end per project convention. Prompt To Fix All With AIThis is a comment left during a code review.
Path: CHANGELOG.md
Line: 9
Comment:
**Changelog entry inserted at top of section**
Per the project's changelog placement rule, new entries should be appended to the **end** of the target section (`### Changes` or `### Fixes`), not inserted at the top. This entry appears before the three pre-existing lines in the `### Changes` block.
```suggestion
- Anthropic/models: add Claude Opus 4.7 `xhigh` reasoning effort support and keep it separate from adaptive thinking.
- Control UI/settings: overhaul the settings and slash-command experience with faster presets, quick-create flows, and refreshed command discovery. (#67819) Thanks @BunsDev.
- macOS/gateway: add `screen.snapshot` support for macOS app nodes, including runtime plumbing, default macOS allowlisting, and docs for monitor preview flows. (#67954) Thanks @BunsDev.
- Gateway/config: add `models.pricing.enabled` config option to disable the model-pricing bootstrap fetch, avoiding a ~30 s timeout on networks that cannot reach the pricing endpoint. Fixes #53639.
```
**Context Used:** CLAUDE.md ([source](https://app.greptile.com/review/custom-context?memory=fd949e91-5c3a-4ab5-90a1-cbe184fd6ce8))
How can I resolve this? If you propose a fix, please make it concise.Reviews (1): Last reviewed commit: "Update config baseline hash for models.p..." | Re-trigger Greptile |
Agent-Logs-Url: https://github.com/palewire/openclaw/sessions/21f9223e-9ced-41fd-a1bf-f679f3d99fc6 Co-authored-by: palewire <9993+palewire@users.noreply.github.com>
…option-disable-model-pricing-bootstrap # Conflicts: # docs/.generated/config-baseline.sha256
This reverts commit ad2accc.
|
I believe I've resolved the conflicts with the main branch and this is again ready to merge. |
Summary
models.pricing.enabledconfig boolean (defaulttrue) that, whenfalse, skips the bootstrap entirely with a single log line.Closes #53639
Files changed
src/config/types.models.ts— AddModelPricingConfigtype andpricingfield toModelsConfigsrc/config/zod-schema.core.ts— AddModelPricingConfigSchemato the zod validationsrc/config/schema.help.ts— Add help text formodels.pricingandmodels.pricing.enabledsrc/config/schema.labels.ts— Add labels for the new config keyssrc/gateway/model-pricing-cache.ts— Checkconfig.models?.pricing?.enabledand skip bootstrap whenfalsesrc/gateway/model-pricing-cache.test.ts— Add 3 tests for default, enabled, and disabled behaviordocs/gateway/configuration-reference.md— Document the new fieldCHANGELOG.md— Add entrysrc/config/schema.base.generated.ts— Regenerateddocs/.generated/config-baseline.sha256— Updated hash