Skip to content

feat(usage): prioritize auto_model from metadata in usage queries#2214

Merged
chrarnoldus merged 1 commit intomainfrom
session/agent_a1011cb7-a603-4771-88c3-2a386b9d3da3
Apr 9, 2026
Merged

feat(usage): prioritize auto_model from metadata in usage queries#2214
chrarnoldus merged 1 commit intomainfrom
session/agent_a1011cb7-a603-4771-88c3-2a386b9d3da3

Conversation

@kilo-code-bot
Copy link
Copy Markdown
Contributor

@kilo-code-bot kilo-code-bot Bot commented Apr 9, 2026

Summary

Update usage queries to prioritize auto_model from microdollar_usage_metadata when selecting the model name, falling back to requested_model and then model from microdollar_usage.

Changes

  • Modified apps/web/src/app/api/profile/usage/route.ts:

    • Added joins to microdollar_usage_metadata and auto_model tables
    • Updated COALESCE to prioritize auto_model.auto_model before requested_model and model
  • Modified apps/web/src/routers/organizations/organization-usage-details-router.ts:

    • Added joins to microdollar_usage_metadata and auto_model tables
    • Updated COALESCE in getTimeSeries query to prioritize auto_model
    • Updated COALESCE in get query to prioritize auto_model

Verification

  • pnpm typecheck passes
  • pnpm lint passes
  • pnpm format applied

Visual Changes

N/A - Backend query changes only

Reviewer Notes

The model selection priority is now:

  1. auto_model.auto_model (from metadata)
  2. microdollar_usage.requested_model
  3. microdollar_usage.model

This ensures that when auto-model selection is used, the actual model that was selected is displayed in usage reports rather than the requested model.


Generated by: moonshotai/kimi-k2.5

Update usage queries to select auto_model from microdollar_usage_metadata
before falling back to requested_model and model. This affects:
- src/app/api/profile/usage/route.ts
- src/routers/organizations/organization-usage-details-router.ts
@kilo-code-bot
Copy link
Copy Markdown
Contributor Author

kilo-code-bot Bot commented Apr 9, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • apps/web/src/app/api/profile/usage/route.ts
  • apps/web/src/routers/organizations/organization-usage-details-router.ts

Reviewed by gpt-5.4-20260305 · 205,834 tokens

@chrarnoldus
Copy link
Copy Markdown
Contributor

if there's a perf issue we'll just revert it

@chrarnoldus chrarnoldus requested a review from RSO April 9, 2026 09:50
@chrarnoldus chrarnoldus merged commit 78c1fad into main Apr 9, 2026
19 checks passed
@chrarnoldus chrarnoldus deleted the session/agent_a1011cb7-a603-4771-88c3-2a386b9d3da3 branch April 9, 2026 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants