Skip to content

Commit cb95c8e

Browse files
committed
add defensive handling of vscode api proposal usage
1 parent dab37cb commit cb95c8e

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

src/notebooks/controllers/vscodeNotebookController.ts

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -173,10 +173,14 @@ export class VSCodeNotebookController implements Disposable, IVSCodeNotebookCont
173173
displayDataProvider
174174
);
175175

176-
try {
177-
controller.controller.variableProvider = jupyterVairablesProvider;
178-
} catch (ex) {
179-
logger.warn('Failed to attach variable provider', ex);
176+
// Only attach variable provider if the API is available
177+
// Some VSCode forks (like Cursor) don't support the notebookVariableProvider API proposal yet
178+
if ('variableProvider' in controller.controller) {
179+
try {
180+
controller.controller.variableProvider = jupyterVairablesProvider;
181+
} catch (ex) {
182+
logger.warn('Failed to attach variable provider', ex);
183+
}
180184
}
181185

182186
return controller;

0 commit comments

Comments
 (0)