add language id, restrict to python (#15168)

microsoft · Feb 9, 2024 · 5fea034 · 5fea034
1 parent 27a498b
commit 5fea034
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/pythonFiles/vscode_datascience_helpers/getVariableInfo/vscodeGetVariableInfo.py b/pythonFiles/vscode_datascience_helpers/getVariableInfo/vscodeGetVariableInfo.py
@@ -69,6 +69,7 @@ def _VSCODE_getVariableDescriptions(varNames):
                 **getVariableDescription(globals()[varName]),
                 "root": varName,
                 "propertyChain": [],
+                "language": "python",
             }
             for varName in varNames
             if varName in globals()
@@ -103,6 +104,7 @@ def _VSCODE_getAllChildrenDescriptions(rootVarName, propertyChain, startIndex):
                         "name": str(i),
                         "root": rootVarName,
                         "propertyChain": propertyChain + [i],
+                        "language": "python",
                     }
                     for i in range
                 ]

diff --git a/src/notebooks/controllers/vscodeNotebookController.ts b/src/notebooks/controllers/vscodeNotebookController.ts
@@ -213,7 +213,9 @@ export class VSCodeNotebookController implements Disposable, IVSCodeNotebookCont
         this.controller.interruptHandler = this.handleInterrupt.bind(this);
         this.controller.supportsExecutionOrder = true;
         this.controller.supportedLanguages = this.languageService.getSupportedLanguages(kernelConnection);
-        this.controller.variableProvider = new JupyterVariablesProvider(jupyterVariables, this.kernelProvider);
+        if (this.controller.supportedLanguages.includes('python')) {
+            this.controller.variableProvider = new JupyterVariablesProvider(jupyterVariables, this.kernelProvider);
+        }
         // Hook up to see when this NotebookController is selected by the UI
         this.controller.onDidChangeSelectedNotebooks(this.onDidChangeSelectedNotebooks, this, this.disposables);
         workspace.onDidCloseNotebookDocument(