Skip to content

ref(ui): Remove unused frontend overrides - #121549

Merged
scttcper merged 2 commits into
masterfrom
scttcper/remove-unused-overrides
Aug 7, 2026
Merged

ref(ui): Remove unused frontend overrides#121549
scttcper merged 2 commits into
masterfrom
scttcper/remove-unused-overrides

Conversation

@scttcper

@scttcper scttcper commented Aug 7, 2026

Copy link
Copy Markdown
Member

Drop override types and SaaS registrations with no production consumers, then remove the implementations left unreachable by the cleanup.

Keep the dynamically constructed SSO overrides and document why exact string searches do not find their call sites.

Drop override types and SaaS registrations with no production consumers, then remove the implementations left unreachable by the cleanup.

Keep the dynamically constructed SSO overrides and document why exact string searches do not find their call sites.

Co-Authored-By: Codex <noreply@openai.com>
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Aug 7, 2026
Production Knip found two implementations that were only kept alive by their own tests after the override registry cleanup. Remove those files and keep the remaining billing helper private.

Co-Authored-By: Codex <noreply@openai.com>
@scttcper
scttcper marked this pull request as ready for review August 7, 2026 18:42
@scttcper
scttcper requested a review from a team as a code owner August 7, 2026 18:42
@scttcper
scttcper requested a review from a team August 7, 2026 18:43
@scttcper
scttcper merged commit cd5eab7 into master Aug 7, 2026
71 checks passed
@scttcper
scttcper deleted the scttcper/remove-unused-overrides branch August 7, 2026 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants