Skip to content

Commit 5cae6f0

Browse files
Copilotrajbos
andcommitted
Fix plan limit display to show constant values instead of calculated results
Co-authored-by: rajbos <6085745+rajbos@users.noreply.github.com>
1 parent 018139a commit 5cae6f0

3 files changed

Lines changed: 12 additions & 12 deletions

File tree

src/App.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -287,8 +287,8 @@ function App() {
287287
limit = item.businessPlanLimit;
288288
}
289289

290-
// Handle infinity (unlimited) for 0x multiplier models
291-
return limit === Infinity ? "Unlimited" : limit.toLocaleString();
290+
// For 0x multiplier models, show "Unlimited" despite having constant plan limits
291+
return item.multiplier === 0 ? "Unlimited" : limit.toLocaleString();
292292
}, [selectedPlan]);
293293

294294
return (

src/lib/utils.ts

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -184,9 +184,9 @@ export function getModelUsageSummary(data: CopilotUsageData[]): ModelUsageSummar
184184
compliantRequests: 0,
185185
exceedingRequests: 0,
186186
multiplier,
187-
individualPlanLimit: multiplier === 0 ? Infinity : Math.floor(PLAN_MONTHLY_LIMITS[COPILOT_PLANS.INDIVIDUAL] / multiplier),
188-
businessPlanLimit: multiplier === 0 ? Infinity : Math.floor(PLAN_MONTHLY_LIMITS[COPILOT_PLANS.BUSINESS] / multiplier),
189-
enterprisePlanLimit: multiplier === 0 ? Infinity : Math.floor(PLAN_MONTHLY_LIMITS[COPILOT_PLANS.ENTERPRISE] / multiplier),
187+
individualPlanLimit: PLAN_MONTHLY_LIMITS[COPILOT_PLANS.INDIVIDUAL],
188+
businessPlanLimit: PLAN_MONTHLY_LIMITS[COPILOT_PLANS.BUSINESS],
189+
enterprisePlanLimit: PLAN_MONTHLY_LIMITS[COPILOT_PLANS.ENTERPRISE],
190190
excessCost: 0
191191
};
192192
}
@@ -219,12 +219,12 @@ export function getModelUsageSummary(data: CopilotUsageData[]): ModelUsageSummar
219219
groupedSummary[key].exceedingRequests += item.exceedingRequests;
220220
}
221221

222-
// For grouped default models, ensure multiplier is 0 and limits are set correctly
222+
// For grouped default models, ensure multiplier is 0 and limits use constant values
223223
if (key === 'Default') {
224224
groupedSummary[key].multiplier = 0;
225-
groupedSummary[key].individualPlanLimit = Infinity;
226-
groupedSummary[key].businessPlanLimit = Infinity;
227-
groupedSummary[key].enterprisePlanLimit = Infinity;
225+
groupedSummary[key].individualPlanLimit = PLAN_MONTHLY_LIMITS[COPILOT_PLANS.INDIVIDUAL];
226+
groupedSummary[key].businessPlanLimit = PLAN_MONTHLY_LIMITS[COPILOT_PLANS.BUSINESS];
227+
groupedSummary[key].enterprisePlanLimit = PLAN_MONTHLY_LIMITS[COPILOT_PLANS.ENTERPRISE];
228228
}
229229

230230
// Calculate excess cost

src/test/model-info-limits.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,9 +53,9 @@ describe('Model Info and Limits Feature', () => {
5353

5454
if (defaultGroup) {
5555
expect(defaultGroup.multiplier).toBe(0);
56-
expect(defaultGroup.individualPlanLimit).toBe(Infinity); // 0x multiplier = unlimited
57-
expect(defaultGroup.businessPlanLimit).toBe(Infinity); // 0x multiplier = unlimited
58-
expect(defaultGroup.enterprisePlanLimit).toBe(Infinity); // 0x multiplier = unlimited
56+
expect(defaultGroup.individualPlanLimit).toBe(50); // Constant plan limit, not Infinity
57+
expect(defaultGroup.businessPlanLimit).toBe(300); // Constant plan limit, not Infinity
58+
expect(defaultGroup.enterprisePlanLimit).toBe(1000); // Constant plan limit, not Infinity
5959
}
6060
});
6161

0 commit comments

Comments
 (0)