chore(ui): drop the antd dependency and its leftovers

[ryan-crabbe-berri]

TLDR

Problem this solves:

• Nothing renders antd, yet it still ships to users
• A global antd StyleProvider wraps every page
• A z-index hack lifts popups over antd modals
• 15 test files mock a module nothing imports

How it solves it:

• Drops antd and @ant-design/cssinjs from package.json
• Deletes the StyleProvider, cascade layer and z-index hack
• Removes the inert vi.mock("antd") factories
• Renames the shims that outlived the library

User Flow

Before: an admin loading the dashboard downloads and parses a UI library that no longer draws a single control

1. They open https://litellm-domain/ui/?page=api-keys with a cold cache
2. The page pulls the antd style engine alongside the shadcn bundle, though no component uses it since refactor(ui): migrate the last antd components off antd onto shadcn #37569
3. Every dialog they open runs through a global style provider that has nothing left to style
4. Nothing on screen comes from that library, so the download bought them nothing

After: the same page ships only the kit it actually renders

1. They open https://litellm-domain/ui/?page=api-keys with a cold cache
2. Only the shadcn bundle loads
3. Every page, dialog and dropdown looks and behaves exactly as before
4. The stacking hack that lifted dropdowns over antd modals is gone, and dropdowns still open above their dialogs

Relevant issues

Linear ticket

Pre-Submission checklist

Please complete all items before asking a LiteLLM maintainer to review your PR

• I have added meaningful tests (dependency removal only; the existing dashboard unit tests cover the touched components)
• My PR passes all CI/CD checks (e.g., lint, format, unit tests)
• My PR's scope is as isolated as possible; it only solves 1 specific problem
• I have received a Greptile Confidence Score of at least 4/5 before requesting a maintainer review (Greptile reviews automatically once the PR is opened; only comment @greptileai to re-request a review after pushing changes)

Delays in PR merge?

If you're seeing a delay in your PR being merged, ping the LiteLLM Team on Slack (#pr-review).

Screenshots / Proof of Fix

Shared setup: one worktree per side, cd ui/litellm-dashboard && npm ci && npm run build, the build copied into litellm/proxy/_experimental/out, and a live proxy per side (python litellm/proxy/proxy_cli.py --config litellm/proxy/dev_config.yaml --port <port> --use_v2_migration_resolver, fresh Postgres DB each) so the browser loads the UI exactly as an admin would at http://localhost:PORT/ui/. Browser steps executed in a real Chromium driven by Playwright against those live proxies, logged in as admin

Before (0b37454)

antd presence in node_modules and the served bundle

1. npm ls antd @ant-design/cssinjs

├── @ant-design/cssinjs@1.24.0
└─┬ antd@5.29.3
  ├─┬ @ant-design/cssinjs-utils@1.1.3
  │ └── @ant-design/cssinjs@1.24.0 deduped
  └── @ant-design/cssinjs@1.24.0 deduped

2. npm run build completes, then find out/_next/static -name '*.js' | xargs cat | wc -c gives 10522409 bytes and grep -rl cssinjs out/_next/static hits one served chunk, _next/static/chunks/0i9exd9dj3-75.js: the style engine still ships to every visitor

Dashboard pages and the create-key dialog still render

1. GET http://localhost:53817/ui/ returns 200; after logging in, ?page=api-keys, ?page=teams, ?page=new_model, ?page=mcp-servers, and ?page=guardrails all render the full dashboard with zero browser console errors and zero page errors
2. On ?page=api-keys, Create New Key opens, and clicking the Models field pops a 10-option dropdown; document.elementFromPoint at the first option's center hits that option (hitInsideOption: true, hitInsideDialogOverlay: false), so the dropdown paints above the dialog, and clicking it selects the model

After (552884d)

antd presence in node_modules and the served bundle

1. npm ls antd @ant-design/cssinjs

litellm-dashboard@0.1.0
└── (empty)

2. npm run build completes, then the same byte count gives 10510152 bytes and grep -rl cssinjs out/_next/static (plus ant-design, data-ant, and bare antd) hits zero files: nothing antd-flavored ships anymore

Dashboard pages and the create-key dialog still render

1. GET http://localhost:54293/ui/ returns 200; the same five pages render identically with zero browser console errors and zero page errors
2. The same Create New Key flow gives the same 10-option Models dropdown with the identical stacking probe result (hitInsideOption: true, hitInsideDialogOverlay: false) and the same successful selection, with the z-index override gone

QA observations:

• antd component code already left the bundle in refactor(ui): migrate the last antd components off antd onto shadcn #37569
• Only the cssinjs style engine still shipped; this PR removes it
• node_modules sheds antd's 58M of install weight
• Fresh empty DBs needed prisma db push despite --use_v2_migration_resolver; both legs, PR leaves it alone

Type

🧹 Refactoring

Caveats (if any)

• Comments still name antd where it explains a contract
• tests/proxy_admin_ui_tests/ui_unit_tests still declares antd
• That directory is dead: no CI job runs it

Final Attestation

• The tests check the right things, including the edge cases, and regressions in the respective real-world customer use-cases are not possible after this PR

• 552884d passes /live-pr-risk

[greptile-apps]

Greptile Summary

This PR removes the dashboard’s unused Ant Design runtime, global style provider, compatibility CSS, lint rule, and obsolete test mocks. It also renames surviving form and URL-validation helpers without changing their behavior

• Removes antd, @ant-design/cssinjs, and their lockfile dependency graph
• Removes the global Ant Design provider and obsolete cascade and popup-layering CSS
• Renames legacy helper symbols and updates their production callers
• Cleans inert Ant Design mocks from dashboard tests

Confidence Score: 4/5

The PR appears safe to merge, with only the non-blocking test-classification cleanup remaining

The removed packages and provider have no active dashboard importers, helper behavior is preserved, and deleted mocks do not remove behavioral assertions

Files Needing Attention: ui/litellm-dashboard/src/**/*.test.tsx and ui/litellm-dashboard/src/lib/forms/urlValidation.test.ts

Important Files Changed

Filename	Overview
ui/litellm-dashboard/package.json	Removes the two unused Ant Design dependencies; no active dashboard importer remains
ui/litellm-dashboard/src/app/layout.tsx	Removes the inert global Ant Design style wrapper while preserving the existing provider hierarchy
ui/litellm-dashboard/src/app/globals.css	Removes the Ant Design cascade layer and modal popup override while retaining Base UI nested-dialog styling
ui/litellm-dashboard/src/components/common_components/formRules.ts	Renames the legacy form-rule shim while preserving required and asynchronous validator behavior
ui/litellm-dashboard/src/lib/forms/urlValidation.ts	Renames the URL-validation shim and constants without changing the compatibility regex or length limit
ui/litellm-dashboard/src/app/(dashboard)/policies/_components/impact_popover.test.tsx	Removes an inert Ant Design mock, but leaves a touched legacy test unclassified under the dashboard test naming convention

Comments Outside Diff (1)

1. ui/litellm-dashboard/src/app/(dashboard)/policies/_components/impact_popover.test.tsx, line 1 (link)

   Touched tests remain unclassified

   This PR modifies fourteen legacy tests without classifying them as unit or integration tests, leaving their ownership and execution scope ambiguous

   Context Used: CLAUDE.md (source)

   Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

_{Reviews (1): Last reviewed commit: "chore(ui): drop the antd dependency and ..." | Re-trigger Greptile}

[tin-berri]

Reviewed the antd removal — this is a large diff (105 additions / 1,410 deletions across 50 files) so I verified it against the full post-removal tree, not just the hunks. Completeness: grep across ui/litellm-dashboard/src/ for any remaining from "antd"/from '@ant-design returns zero hits; the ~40 remaining string matches for "antd" are comments/test-description strings documenting legacy contracts, matching the PR's own caveat. package.json/package-lock.json drop antd and @ant-design/cssinjs from real dependencies (not just dev), and the lockfile prunes the full transitive tree (~993 lines) — a real npm install, not a hand-edit. No behavior loss from the StyleProvider/z-index deletion: the deleted CSS was scoped to .ant-modal-wrap, a class that never renders once antd is gone, so it was already inert — and critically, the underlying nested-dialog stacking mechanism itself (NESTED_DIALOG_LAYER, renamed from ABOVE_ANTD_MODAL) is preserved and still used by KeywordModal/PatternModal/CustomPatternModal, unrelated to antd. The 15 deleted vi.mock("antd") factories were confirmed dead — no file in src/ imports antd, so nothing depended on those mocks. Scope is clean (all 50 files under ui/litellm-dashboard/**, no backend/auth/billing/security touched). CI green — build-ui, ui-unit-tests, frontend-lint and the full backend suite all pass; only non-blocking image-scan and the external PR-review bot remain pending.

[mateo-berri]

LGTM. Thanks Ryan, this rocks!!