Skip to content

refactor(billing): update terminology from "5-hour" to "10-hour" .#431

Merged
graycyrus merged 1 commit intotinyhumansai:mainfrom
YellowSnnowmann:fix/update-context-window
Apr 8, 2026
Merged

refactor(billing): update terminology from "5-hour" to "10-hour" .#431
graycyrus merged 1 commit intotinyhumansai:mainfrom
YellowSnnowmann:fix/update-context-window

Conversation

@YellowSnnowmann
Copy link
Copy Markdown
Contributor

@YellowSnnowmann YellowSnnowmann commented Apr 8, 2026

Summary by CodeRabbit

  • Updates
    • Updated billing and rate limit window display from 5-hour to 10-hour across billing panels and conversation pages.
    • Revised documentation to reflect 10-hour rolling inference window for spending caps.

…consistency across billing components

- Updated comments and variable names in the billingHelpers, InferenceBudget, SubscriptionPlans, and Conversations components to reflect the new 10-hour rolling inference window.
- Adjusted UI labels and descriptions to ensure clarity regarding the 10-hour cap and its implications for users.
- Enhanced the TeamUsage interface documentation to accurately describe the rolling inference window and its associated limits.
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 8, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a1d01c43-3c1d-480f-9b20-fa5983237c66

📥 Commits

Reviewing files that changed from the base of the PR and between a8b9304 and dab6dde.

📒 Files selected for processing (7)
  • app/src/components/settings/panels/BillingPanel.tsx
  • app/src/components/settings/panels/billing/InferenceBudget.tsx
  • app/src/components/settings/panels/billing/SubscriptionPlans.tsx
  • app/src/components/settings/panels/billingHelpers.ts
  • app/src/pages/Conversations.tsx
  • app/src/services/api/creditsApi.ts
  • app/src/types/api.ts

📝 Walkthrough

Walkthrough

The pull request updates billing and rate-limit UI text across multiple components to reflect a "10-hour" rolling inference window instead of "5-hour," while preserving underlying field names for backward compatibility. Two new type fields (fiveHourCapUsd) are added to billing interfaces for type safety, and API documentation is updated to describe the 10-hour semantics.

Changes

Cohort / File(s) Summary
Billing UI Text Updates
app/src/components/settings/panels/BillingPanel.tsx, app/src/components/settings/panels/billing/InferenceBudget.tsx, app/src/components/settings/panels/billing/SubscriptionPlans.tsx, app/src/pages/Conversations.tsx
Updated displayed "cap" and "limit" labels from "5-hour" to "10-hour" in billing panels and rate-limit messaging, including tooltip text and LimitPill labels, while keeping underlying data fields unchanged.
Billing Type Definitions
app/src/components/settings/panels/billingHelpers.ts, app/src/types/api.ts
Added new fiveHourCapUsd: number field to PlanMeta and CurrentPlanData interfaces with documentation describing the USD cap per 10-hour rolling inference window by plan tier.
API Documentation
app/src/services/api/creditsApi.ts
Updated inline comments on TeamUsage fields (fiveHourSpendUsd, fiveHourCapUsd, fiveHourResetsAt) to clarify they represent a server-enforced 10-hour rolling window with historical field naming preserved.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Poem

🐰 From five to ten, our hours now ring,
A cap expanded, more room to swing!
Though field names stay their historical way,
The UI shows ten—hooray, hooray! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and accurately describes the main change: updating UI terminology from '5-hour' to '10-hour' across billing-related components.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

@graycyrus graycyrus merged commit 8c52236 into tinyhumansai:main Apr 8, 2026
8 of 9 checks passed
@coderabbitai coderabbitai Bot mentioned this pull request Apr 27, 2026
5 tasks
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