Skip to content

chore: remove legacy deployment artifacts and litellm-js packages - #27541

Merged
yassin-berriai merged 1 commit into
litellm_internal_stagingfrom
litellm_fix/remove-dockerfiles-js-bloat
May 9, 2026
Merged

chore: remove legacy deployment artifacts and litellm-js packages#27541
yassin-berriai merged 1 commit into
litellm_internal_stagingfrom
litellm_fix/remove-dockerfiles-js-bloat

Conversation

@yassin-berriai

@yassin-berriai yassin-berriai commented May 9, 2026

Copy link
Copy Markdown
Contributor
  • Remove litellm-js/proxy and litellm-js/spend-logs TypeScript packages that provided Cloudflare Worker proxy and Node.js spend logging services, as these are no longer maintained
  • Remove deprecated Docker variants (Dockerfile.alpine, Dockerfile.dev, Dockerfile.custom_ui, Dockerfile.health_check, Dockerfile.ghcr_base) that have been superseded by the primary Dockerfile
  • Remove legacy Kubernetes manifests (kub.yaml, service.yaml) from deploy/kubernetes in favor of the Helm chart
  • Remove stale index.yaml Helm chart index pinned to an old version (v1.43.18)
  • Remove dev_config.yaml development configuration file that contained hardcoded credentials and example endpoints
  • Clean up ~3,500 lines of unused code and configuration to reduce repository maintenance burden

Relevant issues

Linear ticket

Resolves LIT-2963

Pre-Submission checklist

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

  • I have Added testing in the tests/test_litellm/ directory, Adding at least 1 test is a hard requirement - see details
  • My PR passes all unit tests on make test-unit
  • 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).

CI (LiteLLM team)

CI status guideline:

  • 50-55 passing tests: main is stable with minor issues.
  • 45-49 passing tests: acceptable but needs attention
  • <= 40 passing tests: unstable; be careful with your merges and assess the risk.
  • Branch creation CI run
    Link:

  • CI run for the last commit
    Link:

  • Merge / cherry-pick CI run
    Links:

Screenshots / Proof of Fix

Type

🆕 New Feature
🐛 Bug Fix
🧹 Refactoring
📖 Documentation
🚄 Infrastructure
✅ Test

Changes

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@greptile-apps

greptile-apps Bot commented May 9, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR removes ~3,500 lines of legacy, unmaintained artifacts from the repository and fixes two documentation files that were left pointing at now-deleted resources.

  • Deletes the litellm-js/proxy (Cloudflare Worker) and litellm-js/spend-logs (Node.js batching service) TypeScript packages, along with deprecated Docker variants (Dockerfile.alpine, Dockerfile.dev, Dockerfile.custom_ui, Dockerfile.health_check, Dockerfile.ghcr_base).
  • Removes legacy Kubernetes manifests (kub.yaml, service.yaml) and a stale Helm chart index (index.yaml pinned to v1.43.18).
  • Fixes AGENTS.md to embed an inline config.yaml example instead of referencing the deleted dev_config.yaml, and updates CLAUDE.md to drop the litellm-js/spend-logs/ entry from the Prisma schema-sync list.

Confidence Score: 5/5

Pure deletion of unmaintained packages and deprecated infrastructure files with accurate follow-up documentation fixes — safe to merge.

Every deleted file is either an explicitly deprecated artifact or a legacy package with no active callers in the main codebase. The two documentation files that referenced deleted paths (AGENTS.md, CLAUDE.md) have both been corrected in the same commit, leaving no dangling references.

No files require special attention.

Important Files Changed

Filename Overview
AGENTS.md Updated proxy startup instructions to embed an inline config.yaml example instead of referencing the deleted dev_config.yaml, fixing the broken onboarding command.
CLAUDE.md Removed the stale litellm-js/spend-logs/ entry from the Prisma schema-sync list, keeping the list accurate after that directory's deletion.
dev_config.yaml Deleted development config containing only placeholder/fake credentials; content is now inlined directly in AGENTS.md.
deploy/kubernetes/kub.yaml Deleted legacy Kubernetes Deployment manifest superseded by Helm chart; contained only redacted/placeholder credential values.
index.yaml Deleted stale Helm chart index pinned to v1.43.18, no longer kept in sync with current chart releases.
litellm-js/spend-logs/src/index.ts Deleted unmaintained Node.js spend-log batching service (Hono + Prisma); no active users expected.
litellm-js/proxy/src/index.ts Deleted unmaintained Cloudflare Worker proxy (Hono + OpenAI SDK); no active users expected.

Reviews (2): Last reviewed commit: "chore: remove legacy deployment artifact..." | Re-trigger Greptile

@codecov

codecov Bot commented May 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@yassin-berriai
yassin-berriai force-pushed the litellm_fix/remove-dockerfiles-js-bloat branch from 9ea9fd5 to 929f25c Compare May 9, 2026 20:06
@yassin-berriai

Copy link
Copy Markdown
Contributor Author

@greptileai

- Remove litellm-js/proxy and litellm-js/spend-logs TypeScript packages that provided Cloudflare Worker proxy and Node.js spend logging services, as these are no longer maintained
- Remove deprecated Docker variants (Dockerfile.alpine, Dockerfile.dev, Dockerfile.custom_ui, Dockerfile.health_check, Dockerfile.ghcr_base) that have been superseded by the primary Dockerfile
- Remove legacy Kubernetes manifests (kub.yaml, service.yaml) from deploy/kubernetes in favor of the Helm chart
- Remove stale index.yaml Helm chart index pinned to an old version (v1.43.18)
- Remove dev_config.yaml development configuration file that contained hardcoded credentials and example endpoints
- Clean up ~3,500 lines of unused code and configuration to reduce repository maintenance burden
@yassin-berriai
yassin-berriai force-pushed the litellm_fix/remove-dockerfiles-js-bloat branch from 929f25c to 7d41da8 Compare May 9, 2026 20:45
@yassin-berriai
yassin-berriai enabled auto-merge (squash) May 9, 2026 20:47
@yassin-berriai
yassin-berriai merged commit fa5eae8 into litellm_internal_staging May 9, 2026
96 of 103 checks passed
@yassin-berriai
yassin-berriai deleted the litellm_fix/remove-dockerfiles-js-bloat branch May 9, 2026 20:51
fzowl pushed a commit to fzowl/litellm that referenced this pull request Jun 24, 2026
…rriAI#27541)

- Remove litellm-js/proxy and litellm-js/spend-logs TypeScript packages that provided Cloudflare Worker proxy and Node.js spend logging services, as these are no longer maintained
- Remove deprecated Docker variants (Dockerfile.alpine, Dockerfile.dev, Dockerfile.custom_ui, Dockerfile.health_check, Dockerfile.ghcr_base) that have been superseded by the primary Dockerfile
- Remove legacy Kubernetes manifests (kub.yaml, service.yaml) from deploy/kubernetes in favor of the Helm chart
- Remove stale index.yaml Helm chart index pinned to an old version (v1.43.18)
- Remove dev_config.yaml development configuration file that contained hardcoded credentials and example endpoints
- Clean up ~3,500 lines of unused code and configuration to reduce repository maintenance burden

Co-authored-by: Yassin Kortam <yassinkortam@g.ucla.edu>
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.

5 participants