Skip to content

LCORE-0000: Clean up stale litellm and fastuuid references - #3072

Open
fullsend-ai-coder[bot] wants to merge 1 commit into
mainfrom
agent/3066-cleanup-stale-litellm-fastuuid-references
Open

fullsend-ai-coder[bot] wants to merge 1 commit into
mainfrom
agent/3066-cleanup-stale-litellm-fastuuid-references

Conversation

@fullsend-ai-coder

Copy link
Copy Markdown
Contributor

Summary

Clean up stale litellm, fastuuid, and LITELLM_DROP_PARAMS references left behind after PR #3057 removed litellm from pyproject.toml.

Changes

Stale dependency references removed (6 files)

  • examples/pyproject.ogx.toml — removed litellm>=1.72.1
  • README.md — removed litellm>=1.72.1
  • docs/basic_info/getting_started.md — removed litellm from install instructions and expected output
  • docs/user_doc/deployment_guide.md — removed litellm>=1.72.1
  • docs/demos/lcore/lcore.md — removed litellm from pdm add command

Stale env var references removed (4 files)

  • Makefile — removed -e LITELLM_DROP_PARAMS=true
  • docker-compose.yaml — removed LITELLM_DROP_PARAMS=true
  • docker-compose-library.yaml — removed LITELLM_DROP_PARAMS=true
  • docs/devel_doc/container_orchestration.md — removed LITELLM_DROP_PARAMS documentation

Provider docs updated (1 file)

  • docs/devel_doc/providers.md — replaced litellm with — for 7 providers (anthropic, gemini, groq, llama-openai-compat, openai, sambanova, watsonx) since OGX 1.2.5 no longer depends on litellm

Build file updated (1 file)

  • deploy/lightspeed-stack/Containerfile — updated comment to remove stale fastuuid, maturin reference

Version constraint added (1 file)

  • pyproject.toml — added >=1.35.0 lower bound to boto3, consistent with project pinning convention

Validation

grep -rn 'litellm\|fastuuid\|LITELLM' --include='*.py' --include='*.md' --include='*.yaml' --include='*.yml' --include='*.toml' --include='*.txt' --include='Makefile' --include='Containerfile' src/ deploy/ examples/ README.md Makefile docker-compose*.yaml docs/
# Returns zero matches

grep 'boto3>=' pyproject.toml
# Returns: "boto3>=1.35.0",

Closes #3066

Post-script verification

  • Branch is not main/master (agent/3066-cleanup-stale-litellm-fastuuid-references)
  • Secret scan passed (gitleaks — 401a7b55cd7f1561ef69e67ecbf9279e06c8e017..HEAD)
  • PR body secret scan passed (gitleaks — no-git)
  • Final PR description secret scan passed (gitleaks — no-git)

Remove all stale references to litellm, fastuuid, and LITELLM_DROP_PARAMS
that were left behind after PR #3057 removed litellm from pyproject.toml.

Changes across 12 files:
- Remove litellm from dependency lists in examples, docs, and README
- Remove LITELLM_DROP_PARAMS env var from Makefile, docker-compose files,
  and container orchestration docs
- Update Containerfile comment to remove fastuuid/maturin mention
- Replace litellm pip dependency with — in providers.md table
- Add version lower-bound (>=1.35.0) to boto3 in pyproject.toml

Closes #3066
@fullsend-ai-coder fullsend-ai-coder Bot added the ready-for-review Triggers review agent dispatch label Oct 29, 2026
@coderabbitai

coderabbitai Bot commented Oct 29, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

Run configuration

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@fullsend-ai-review

fullsend-ai-review Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

🤖 Finished Review · ✅ Success · Started 4:36 PM UTC · Completed 4:46 PM UTC

Commit: ca88db3 · View workflow run →

Runtime: pi · Model: opus → claude-opus-4-6 · Effort: high · Cost: $2.87

@fullsend-ai-review fullsend-ai-review Bot added the risk/low PR risk: low label Oct 29, 2026
@fullsend-ai-review

Copy link
Copy Markdown

<!-- fullsend:risk-assessment -->
Risk Assessment: low (1/5)

Details

Pure cleanup PR removing stale litellm/fastuuid references from docs, config, and build files with 29 lines across 12 files. No source code, protected paths, or security-sensitive files modified. Tightly scoped to issue #3066.

@fullsend-ai-review

Copy link
Copy Markdown

Looks good to me


Labels: PR is a documentation and dependency cleanup touching docs and config files.

@fullsend-ai-review fullsend-ai-review Bot added ready-for-merge All reviewers approved — ready to merge documentation Improvements or additions to documentation dependencies Pull requests that update a dependency file Review effort 1/5 labels Oct 29, 2026
tisnik

This comment was marked as outdated.

@tisnik tisnik left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All end to end tests are failing, so this cleanup is not correct.

@anik120

anik120 commented Oct 30, 2026

Copy link
Copy Markdown
Contributor

Looks good to me

Labels: PR is a documentation and dependency cleanup touching docs and config files.

Does it really? The uv lock needs to be regenerated, please regenerate it.

(let's see if the coder bot is actually intelligent enough to act on review feedback from humans)

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation ready-for-merge All reviewers approved — ready to merge ready-for-review Triggers review agent dispatch Review effort 1/5 risk/low PR risk: low

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Clean up stale litellm and fastuuid references after dependency removal in PR #3057

2 participants