Skip to content

fix(web): remove paid-credit non-expiration promises - #7320

Merged
emilieschario merged 1 commit into
mainfrom
fix/remove-paid-credit-expiry-promises
Oct 10, 2026
Merged

emilieschario merged 1 commit into
mainfrom
fix/remove-paid-credit-expiry-promises

Conversation

@lambertjosh

Copy link
Copy Markdown
Contributor

Summary

  • Remove seven customer-facing paid-credit non-expiration guarantees from credits, automatic top-up, Kilo Pass, and KiloClaw billing surfaces.
  • Remove the purchased-credit guarantee column and extra divider from the balance card, preserving promotional expiry information.
  • Remove the explicit paid-credit non-expiration test comment without changing fixtures or assertions.
  • Wording-only cleanup: billing logic, existing balances, and expiration behavior remain unchanged.

Validation

  • Formatted all seven changed files with pnpm format.
  • Scoped oxlint passed with zero warnings and errors on all seven files.
  • git diff --check passed; reviewed the complete one-commit diff against main.
  • Tests, standalone typecheck, and browser validation were not run.
  • Local tooling warned that Node 26 is active while the repository requires Node 24.

@kilo-code-bot

kilo-code-bot Bot commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Wording-only cleanup that removes customer-facing paid-credit non-expiration promises across six UI surfaces and one test comment; the changes are internally consistent and no bugs were found.

The CheckCircle2 import in credits/page.tsx is still used elsewhere (line 328), so removing the purchased-credits block does not leave an unused import. The grid change from lg:grid-cols-[1fr_1px_1fr_1px_1fr] to lg:grid-cols-[1fr_1px_1fr] correctly matches the two remaining columns plus one divider. A repository search confirms no remaining customer-facing "never expire" promises; the surviving matches are admin credit-grant labels ("Never expire" checkbox) and internal comments, which are accurate.

Note: tests, typecheck, and browser validation were not run per the PR description.

Files Reviewed (7 files)
  • apps/web/src/app/(app)/claw/components/billing/PlanSelectionDialog.tsx
  • apps/web/src/app/(app)/claw/components/billing/WelcomePage.tsx
  • apps/web/src/app/(app)/credits/page.tsx
  • apps/web/src/components/organizations/OrganizationAutoTopUpConfigureDialog.tsx
  • apps/web/src/components/payment/AutoTopUpToggle.tsx
  • apps/web/src/components/profile/kilo-pass/KiloPassLoadingCard.tsx
  • apps/web/src/scripts/d2026-03-18_expire-free-credits.test.ts

Reviewed by deepseek-v4.1-flash · Input: 37.1K · Output: 3.9K · Cached: 280.2K

Review guidance: REVIEW.md from base branch main

@lambertjosh

Copy link
Copy Markdown
Contributor Author
image

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