BuzzbeeSCD/openchamber
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(quota): show actual overusage percentage for GitHub Copilot (#805)

Browse Source 
Remove the Math.min(100, ...) clamp on usedPercent in the Copilot quota
provider so overusage flows through as the real value (e.g. 353%).
Update formatPercent to display values above 100% instead of clamping,
matching what the GitHub Copilot settings page reports.

The progress bar remains capped at 100% width via its own independent
clampPercent call, so only the label text is affected.

Co-authored-by: Ariel Sandor <39200214+arielsandor@users.noreply.github.com>
This commit is contained in:
Ariel Sandor
2026-04-01 09:39:16 +03:00
committed by GitHub
co-authored by Ariel Sandor
parent 598e28462a
commit 1f21ab0886
2 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/ui/src/lib/quota/utils.ts
+2 -3
View File
@@ -6,11 +6,10 @@ export const clampPercent = (value: number | null): number | null => {
}; };
export const formatPercent = (value: number | null): string => { export const formatPercent = (value: number | null): string => {
const clamped = clampPercent(value); if (typeof value !== 'number' || Number.isNaN(value)) {
if (clamped === null) {
return '-'; return '-';
} }
return `${clamped}%`; return `${Math.round(value)}%`;
}; };
export const resolveUsageTone = (percent: number | null): 'safe' | 'warn' | 'critical' => { export const resolveUsageTone = (percent: number | null): 'safe' | 'warn' | 'critical' => {
packages/web/server/lib/quota/providers/copilot.js
+1 -1
View File
@@ -18,7 +18,7 @@ const buildCopilotWindows = (payload) => {
const entitlement = toNumber(snapshot.entitlement); const entitlement = toNumber(snapshot.entitlement);
const remaining = toNumber(snapshot.remaining); const remaining = toNumber(snapshot.remaining);
const usedPercent = entitlement && remaining !== null const usedPercent = entitlement && remaining !== null
? Math.max(0, Math.min(100, 100 - (remaining / entitlement) * 100)) ? Math.max(0, 100 - (remaining / entitlement) * 100)
: null; : null;
const valueLabel = entitlement !== null && remaining !== null const valueLabel = entitlement !== null && remaining !== null
? `${remaining.toFixed(0)} / ${entitlement.toFixed(0)} left` ? `${remaining.toFixed(0)} / ${entitlement.toFixed(0)} left`