Skip to content

revert: "chore: remove _experimental/out" (#31546) - #31904

Merged
yuneng-berri merged 1 commit into
litellm_internal_stagingfrom
litellm_/competent-mestorf-165751
Jul 1, 2026
Merged

revert: "chore: remove _experimental/out" (#31546)#31904
yuneng-berri merged 1 commit into
litellm_internal_stagingfrom
litellm_/competent-mestorf-165751

Conversation

@yuneng-berri

Copy link
Copy Markdown
Collaborator

Relevant issues

Reverts #31546

Linear ticket

Pre-Submission checklist

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

  • I have added meaningful tests
  • 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 requested a Greptile review by commenting @greptileai and received a Confidence Score of at least 4/5 before requesting a maintainer review

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

With the export restored, a proxy started from a plain git checkout serves the bundled admin UI again without running the dashboard build first:

$ curl -sS -o /dev/null -w 'GET /ui/ -> %{http_code} %{content_type} %{size_download} bytes\n' http://localhost:4001/ui/
GET /ui/ -> 200 text/html; charset=utf-8 22340 bytes

$ curl -sS -o /dev/null -w 'GET /get_image -> %{http_code} %{content_type}\n' http://localhost:4001/get_image
GET /get_image -> 200 image/jpeg

tests/test_litellm/proxy/test_proxy_server.py (which this revert also restores to its pre-#31546 state) passes: 214 passed

Type

🧹 Refactoring

Changes

Clean git revert of the #31546 squash commit (72bcb74), no conflicts. It restores the committed admin UI export under litellm/proxy/_experimental/out/ and rolls back the changes that made the codebase tolerate its absence: the get_favicon swagger fallback, the check_dir=False static mounts, the module-level _restructure_ui_html_files, the mkdir -p in the UI build scripts, the .gitignore entry, and the synthetic-export rewrite of test_admin_ui_export_serves_nested_extensionless_routes

Four commits landed on litellm_internal_staging after #31546 that touch proxy_server.py (#31775, #31754, #31753, #31578); they are in unrelated areas (MCP registry hydration, audit logging, spend counters) and the revert applied without touching their hunks

@greptile-apps

greptile-apps Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Too many files changed for review. (709 files found, 100 file limit)

@codecov

codecov Bot commented Jul 1, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@yuneng-berri
yuneng-berri merged commit 34039df into litellm_internal_staging Jul 1, 2026
123 checks passed
@yuneng-berri
yuneng-berri deleted the litellm_/competent-mestorf-165751 branch July 1, 2026 20:38
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