Skip to content

Conversation

@mrubens
Copy link
Collaborator

@mrubens mrubens commented Oct 27, 2025

Make sure not to show usage cost for free models


Important

Add isFree flag to model schema and update logic to set usage cost to zero for free models.

  • Behavior:
    • Add isFree flag to modelInfoSchema in model.ts to indicate free models.
    • Update getRooModels in fetchers/roo.ts to set isFree based on model tags.
    • Modify RooHandler in roo.ts to set totalCost to 0 for free models.
  • Misc:
    • Add comments in roo.ts to explain the handling of free models.

This description was created by Ellipsis for 963de9d. You can customize this summary. It will automatically update as commits are pushed.

@mrubens mrubens requested review from cte and jr as code owners October 27, 2025 18:51
@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. bug Something isn't working labels Oct 27, 2025
@roomote
Copy link
Contributor

roomote bot commented Oct 27, 2025

Review Summary

I've reviewed the changes to ensure free models don't show usage costs. The implementation correctly sets totalCost to 0 for free models at runtime, but there's one issue to address:

Issues to Resolve

  • Free models may still display pricing information in the model settings UI. The pricing fields are populated from the API response without checking the isFree flag, which could confuse users if non-zero prices are shown for free models.

Follow Along on Roo Code Cloud

@mrubens mrubens merged commit 84cf332 into main Oct 27, 2025
17 checks passed
@mrubens mrubens deleted the handle_free_models branch October 27, 2025 18:58
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Oct 27, 2025
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap Oct 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants