Remove unused fronend dependencies #92
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
thlehmann-ionos
commented
Oct 7, 2025
thlehmann-ionos
commented
- Remove pyodide (frontend code execution is disabled)
- Remove cypress (e2e tests) (they're broken and we modified the UI drastically) (also removed one critical npm vulnerability)
thlehmann-ionos
changed the base branch from
ionos-dev
to
feature/remove-heavy-unneeded-dependencies
There was a problem hiding this comment.
Pull Request Overview
Removes frontend Python execution (Pyodide) and Cypress E2E testing infrastructure to eliminate unused dependencies and simplify the build.
- Removed all Pyodide-related workers, execution logic, and configuration options except for retaining only the Jupyter engine.
- Removed Cypress test suite, related scripts, and dependencies.
- Cleaned package scripts to drop Pyodide fetch step and eliminated unused dependencies (pyodide, cypress, eslint-plugin-cypress, undici).
Reviewed Changes
Copilot reviewed 14 out of 17 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| src/routes/+layout.svelte | Removed Pyodide worker import and Python execution handler branch. |
| src/lib/workers/pyodide.worker.ts | Deleted Pyodide worker implementation. |
| src/lib/pyodide/pyodideKernel.worker.ts | Deleted alternate Pyodide kernel worker. |
| src/lib/pyodide/pyodideKernel.ts | Deleted Pyodide kernel wrapper class. |
| src/lib/components/chat/Messages/CodeBlock.svelte | Stripped Python execution UI and worker logic. |
| src/lib/components/admin/Settings/CodeExecution.svelte | Restricted engines list to Jupyter only. |
| package.json | Removed Pyodide/Cypress scripts and dependencies. |
| cypress/** (multiple) | Removed Cypress config, support, tests, and fixtures. |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
thlehmann-ionos
force-pushed
the
feature/remove-heavy-unneeded-dependencies
branch
from
d38d5db to
ab970b1
Compare
thlehmann-ionos
force-pushed
the
feature/remove-pyodide
branch
2 times, most recently
from
9e0de8f to
a01d928
Compare
thlehmann-ionos
force-pushed
the
feature/remove-heavy-unneeded-dependencies
branch
from
ab970b1 to
46d09ca
Compare
|
looks good to me |
thlehmann-ionos
force-pushed
the
feature/remove-heavy-unneeded-dependencies
branch
from
46d09ca to
aaac397
Compare
thlehmann-ionos
changed the base branch from
feature/remove-heavy-unneeded-dependencies
to
ionos-dev
* They're broken anyway * We changed the application in a way that they're no longer compatible Refs: PRODAI-198
* Code execution is not enabled * Building pyodide heavily contributes to the build time and image size Refs: PRODAI-198
thlehmann-ionos
force-pushed
the
feature/remove-pyodide
branch
from
a01d928 to
66a70a4
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.