feat(model-prices): edit special pricing rules - #791
Conversation
|
Thanks for the PR. The overall direction looks correct and still matches #703. However, #793 has now been merged into When resolving the conflicts, please keep the latest #793 behavior as the baseline and re-apply the special pricing rule editor on top of it. In particular:
Please also add a regression test covering the combined behavior: editing/saving an existing model with context/service-tier rules should preserve those rules and still trigger the model-price attention refresh after a successful save. There is no need to expand this PR into route-prefix automatic pricing-rule matching. If that remains reproducible after #792/#793, we can track it separately. |
66f6290 to
2680e6e
Compare
2680e6e to
f4e66ef
Compare
|
Updated this PR onto the latest
Validation now passes locally (3,531 frontend tests, type check, production build, and lint with 0 errors), and all required GitHub checks are green on the updated commit. |
Summary
Editing a synced model price currently removes all long-context and service-tier rules. This adds rule editors that preserve existing rules and let users add, update, or remove their overrides.
Scope
Changes
User Impact
Users can edit base prices without losing synchronized pricing rules, and can manually configure long-context or service-tier overrides from the model prices page.
Compatibility / Runtime Notes
Data / Security Notes
N/A. The change only edits the existing model-price payload and does not touch credentials or usage events.
Risk / Rollback
Risk level: Low
Rollback notes: Revert this commit to restore the previous base-price-only editor.
Verification
Commands / evidence:
Local demo checks covered the empty editor, both rule types, existing rule values, desktop layout, and narrow-layout horizontal scrolling. The review follow-up also adds a combined component regression test that preserves existing rules and refreshes model-price attention after save.
Screenshots / Recordings
N/A — verified in the local demo; no binary artifact was added to the repository.
Docs
Docs decision: The model prices page contains inline localized descriptions for both rule types, and the existing Manager Server payload contract is unchanged.
Related
Fixes #703