Skip to content

Sweep the 'subscription' settings-panel leftovers and orphaned SubscriptionPanel components #14741

Description

@christian-byrne

Residual from the review of #14613. Non-blocking, cleanup.

#14613 correctly deleted subscriptionPanel, shouldShowPlanCreditsPanel and shouldShowLegacyPlanCreditsPanel from useSettingUI.ts once they became unreachable. Three leftovers were not swept:

  1. src/platform/settings/types.ts:87 still lists | 'subscription' in SettingPanelType, and src/platform/settings/composables/useSettingUI.ts:36 still carries subscription: 'icon-[lucide--credit-card]' in CATEGORY_ICONS. No panel can now produce that key.
  2. SubscriptionPanel.vue and SubscriptionPanelContentLegacy.vue are now reachable only from SubscriptionPanel.test.ts and SubscriptionPanelContentLegacy.test.ts. The test imports keep knip quiet while the components have no production entry point, so knip will not flag them as dead.
  3. src/platform/settings/composables/useSettingUI.test.ts:303-312 still parameterises over billingType with it.for(['legacy', 'workspace']), which the composable no longer reads. Both cases are now byte-identical and the not.toContain('subscription') assertion passes vacuously.

Thread: #14613 (comment)

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions