supabase / cli / 24348105329

64%

Build Type

push

github

Committed by web-flow

Commit Message

feat(plan-gate): wire billing links into remaining plan-gated commands (#5066)

* refactor(plan-gate): check entitlements on any non-2xx, only show link when confirmed gated

Previously SuggestUpgradeOnError only triggered on 402. Many plan-gated
endpoints return 400 or 404 instead. Now checks entitlements on any
non-2xx and only sets CmdSuggestion when hasAccess is confirmed false,
preventing false positives.

* refactor(branches): rename was402 to isGated in upgrade check

Aligns variable name with the new semantics: SuggestUpgradeOnError
now checks entitlements on any non-2xx, not just 402.

* feat(sso): wire billing link into sso commands when plan-gated

Check org entitlements for auth.saml_2 on any non-2xx response from
SSO provider endpoints. Shows billing upgrade link when the feature
is confirmed gated, before falling through to existing error handling.

* feat(vanity-subdomains): wire billing link into vanity-subdomain commands when plan-gated

Check org entitlements for vanity_subdomain on any non-2xx response.
The platform API returns 400 when vanity subdomains are not available
on the org's plan; the entitlements check confirms the gate before
showing a billing upgrade link.

* docs(telemetry): clean up EventUpgradeSuggested comment

* refactor(plan-gate): use named returns for readability

* fix(plan-gate): use correct projectRef in branches/update, restrict check to 4xx

Two fixes:
- branches/update passed flags.ProjectRef instead of the local
  projectRef resolved from the branch ID, causing entitlements to
  look up the wrong org
- Restrict SuggestUpgradeOnError to 4xx client errors only (skip 2xx
  success and 5xx server errors) to avoid unnecessary API calls on
  server outages

* refactor(telemetry): extract shared TrackUpgradeSuggested helper

Replace 8 identical per-package trackUpgradeSuggested functions with a
single exported telemetry.TrackUpgradeSuggested. Avoids utils->telemetry
import cycle by keeping the helper in the tele... (continued)

Coverage Stats

20 of 41 new or added lines in 10 files covered. (48.78%)

4 existing lines in 4 files now uncovered.

9796 of 15364 relevant lines covered (63.76%)

7.03 hits per line