Skip to content

feat(toolkit): Simplify deepnote toolkit kernel management, to not use deepnote environments - #376

Draft
tkislan wants to merge 14 commits into
mainfrom
tk/simplify-deepnote-toolkit-kernel
Draft

feat(toolkit): Simplify deepnote toolkit kernel management, to not use deepnote environments#376
tkislan wants to merge 14 commits into
mainfrom
tk/simplify-deepnote-toolkit-kernel

refactor!: remove Deepnote environments

d5fcf94
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project failed Aug 27, 2026 in 1s

36% (-1%) compared to 3da46a0

View this Pull Request on Codecov

36% (-1%) compared to 3da46a0

Details

Codecov Report

❌ Patch coverage is 53.93258% with 82 lines in your changes missing coverage. Please review.
✅ Project coverage is 36%. Comparing base (3da46a0) to head (d5fcf94).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...ebooks/deepnote/deepnoteKernelAutoSelector.node.ts 29% 52 Missing and 5 partials ⚠️
src/kernels/deepnote/deepnoteServerStarter.node.ts 58% 4 Missing and 3 partials ⚠️
src/platform/api/pythonApi.ts 12% 7 Missing ⚠️
src/kernels/helpers.ts 42% 3 Missing and 1 partial ⚠️
.../deepnote/deepnoteToolkitDependencyService.node.ts 90% 1 Missing and 2 partials ⚠️
...ooks/deepnote/snapshots/environmentCapture.node.ts 75% 2 Missing ⚠️
src/notebooks/serviceRegistry.node.ts 0% 1 Missing ⚠️
...latform/interpreter/installer/pipInstaller.node.ts 0% 0 Missing and 1 partial ⚠️
Additional details and impacted files 
@@          Coverage Diff           @@
##            main    #376    +/-   ##
======================================
- Coverage     37%     36%    -1%     
======================================
  Files        828     821     -7     
  Lines      41679   40931   -748     
  Branches    9136    9003   -133     
======================================
- Hits       15449   15017   -432     
+ Misses     24116   23859   -257     
+ Partials    2114    2055    -59
Files with missing lines Coverage Δ
...ls/deepnote/deepnoteSharedToolkitInstaller.node.ts 0% <ø> (ø)
.../kernels/deepnote/deepnoteToolkitInstaller.node.ts 48% <ø> (ø)
src/kernels/deepnote/types.ts 92% <100%> (+42%) ⬆️
...rc/notebooks/deepnote/deepnoteActivationService.ts 88% <100%> (-1%) ⬇️
...otebooks/deepnote/deepnoteMultiNotebookSplitter.ts 89% <ø> (-1%) ⬇️
src/platform/common/constants.ts 100% <100%> (ø)
src/platform/common/utils/localize.ts 92% <100%> (+<1%) ⬆️
...orm/interpreter/installer/productInstaller.node.ts 28% <100%> (+2%) ⬆️
src/platform/interpreter/installer/productNames.ts 100% <100%> (ø)
...tform/interpreter/installer/productService.node.ts 100% <100%> (ø)
... and 10 more

... and 4 files with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
View more details on Codecov