renderer changes for widgets renderer + controller function change

[IanMatthewHuff]

For #6118

• Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
• Title summarizes what is changing.
• Has a news entry file (remember to thank yourself!).
• Appropriate comments and documentation strings in the code.
• Has sufficient logging.
• Has telemetry for enhancements.
• Unit tests & system/integration tests are added/updated.
• Test plan is updated as appropriate.
• package-lock.json has been regenerated by running npm install (if dependencies have changed).

[IanMatthewHuff]

Jupyter side of renderer changes. Again. I need to test with VSCode main after notebook/dev merges.

[IanMatthewHuff]

@DavidKutu I reverted your review on this, just since I added in the notebook controller changes as well. Was easier to test these together in one PR.

[codecov-commenter]

Codecov Report

Merging #6120 (49ff82f) into main (e3ea1de) will decrease coverage by 3%.
The diff coverage is 50%.

@@          Coverage Diff           @@
##            main   #6120    +/-   ##
======================================
- Coverage     71%     68%    -4%     
======================================
  Files        398     398            
  Lines      26850   26846     -4     
  Branches    3950    3950            
======================================
- Hits       19277   18404   -873     
- Misses      5943    6917   +974     
+ Partials    1630    1525   -105     

Impacted Files	Coverage Δ
...t/datascience/notebook/vscodeNotebookController.ts	28% <50%> (-50%)	⬇️
.../datascience/jupyter/kernels/cellExecutionQueue.ts	10% <0%> (-79%)	⬇️
src/client/datascience/jupyter/kernels/kernel.ts	9% <0%> (-67%)	⬇️
...lient/datascience/jupyter/kernels/cellExecution.ts	6% <0%> (-65%)	⬇️
...ent/datascience/jupyter/kernels/kernelExecution.ts	12% <0%> (-55%)	⬇️
...ient/datascience/jupyter/kernels/kernelProvider.ts	50% <0%> (-46%)	⬇️
...t/datascience/notebook/helpers/executionHelpers.ts	35% <0%> (-45%)	⬇️
src/client/datascience/kernelSocketWrapper.ts	45% <0%> (-42%)	⬇️
...ience/notebook/emptyNotebookCellLanguageService.ts	35% <0%> (-42%)	⬇️
... and 49 more

[IanMatthewHuff]

Manual testing results testing against main branch (ipywidget renderer):

Basic ipywidgets load and work + we can see the renderer loading:

Manual testing against main branch (client_renderer copied from vscode-notebook-renderer build into jupyter client_renderer):


Logging for renderer activated:

The notebookController change was just a rename, basic tests by just swapping kernels and making sure that sys.executable updated after change. Also check that I hit a BP in the new handler function name.

[IanMatthewHuff]

Manual run of our native notebook tests.

[IanMatthewHuff]

Test failures expected due to breaking API change. Pushing.