Skip to content

Conversation

jamesmurdza
Copy link
Collaborator

@jamesmurdza jamesmurdza commented Sep 30, 2025

This PR removes the pylume library and all related files, configurations, and references from the codebase.

Deleted Files

File/Directory Description
⛔️ .github/workflows/pypi-publish-pylume.yml GitHub Actions workflow for publishing the pylume package.
⛔️ examples/pylume_examples.py Example script showing how to use pylume.
⛔️ libs/python/pylume/ Entire pylume Python library (source code, README, pyproject, etc.).
⛔️ notebooks/pylume_nb.ipynb Jupyter Notebook demonstrating pylume usage.

Modified Files

File Description of Change
📝 .github/workflows/pypi-reusable-publish.yml Removed mentions of pylume in reusable publishing workflows.
📝 .vscode/launch.json Dropped pylume launch config and PYTHONPATH reference.
📝 .vscode/py.code-workspace Removed pylume from workspace folders and analysis paths.
📝 Development.md Removed libs/pylume/ from main package list.
📝 Dockerfile Removed pylume from PYTHONPATH.
📝 libs/python/mcp-server/scripts/start_mcp_server.sh Dropped pylume from PYTHONPATH.
📝 pdm.lock Removed pylume dependency entry.
📝 pyproject.toml Removed pylume from development dependencies.
📝 notebooks/README.md Removed entry for pylume notebook.
📝 scripts/build-uv.sh, scripts/build.ps1, scripts/build.sh, scripts/run-docker-dev.sh Removed install steps and environment references for pylume.

Still Remaining Mentions

File Type of Mention
⚠️ libs/python/computer/computer/providers/factory.py Comments noting pylume requirement for LumeProvider.
⚠️ libs/python/computer/computer/utils.py Docstring referencing “pylume response.”
⚠️ libs/python/computer/computer/providers/lume/provider.py Comments about removing pylume dependency.
⚠️ libs/python/computer/README.md Mentions pylume in relation to cua-computer package.
⚠️ scripts/run-docker-dev.sh Stray pylume reference.
⚠️ libs/python/agent/agent/ui/gradio/app.py Still mentions pylume.
⚠️ .github/workflows/pypi-reusable-publish.yml Some pylume mentions remain.
⚠️ .devcontainer/devcontainer.json Includes pylume reference.

@jamesmurdza jamesmurdza marked this pull request as ready for review October 1, 2025 00:09
@jamesmurdza jamesmurdza requested a review from ddupont808 October 1, 2025 00:09
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.

1 participant