Don't skip quota bucket when hasQuota: false

[pwang347]

Related #314238

[Copilot]

Pull request overview

Adjusts chat quota snapshot parsing so that quota buckets are not dropped solely because has_quota is false (notably under token-based billing), while still filtering out buckets that represent “no allocated credits” (e.g. entitlement: 0 and not unlimited). This keeps quota UI/telemetry behavior aligned with the service’s snapshot semantics.

Changes:

• Update parseQuotas to no longer skip snapshots based on has_quota === false, and instead skip only non-unlimited snapshots with entitlement === 0.
• Add/adjust unit tests to cover keeping TBB snapshots with has_quota: false and skipping zero-entitlement non-unlimited snapshots.

Show a summary per file

File	Description
src/vs/workbench/services/chat/common/chatEntitlementService.ts	Changes quota snapshot filtering logic to avoid dropping buckets purely due to has_quota: false.
src/vs/workbench/contrib/chat/test/common/chatEntitlementService.test.ts	Adds coverage for the new filtering behavior and updates fixtures/assertions accordingly.

Copilot's findings

• Files reviewed: 2/2 changed files
• Comments generated: 1