Clear variables when restarting regardless if visible or not (#10117)

Author: rchiodo

news/2 Fixes/9991.md

@@ -0,0 +1 @@
+Clear variables in notebooks and interactive-window when restarting.

src/client/datascience/interactive-ipynb/nativeEditor.ts

@@ -327,7 +327,12 @@ export class NativeEditor extends InteractiveBase implements INotebookEditor {
         return this.setDirty();
     }
 
-    protected addSysInfo(_reason: SysInfoReason): Promise<void> {
+    protected addSysInfo(reason: SysInfoReason): Promise<void> {
+        // We need to send a message when restarting
+        if (reason === SysInfoReason.Restart || reason === SysInfoReason.New) {
+            this.postMessage(InteractiveWindowMessages.RestartKernel).ignoreErrors();
+        }
+
         // These are not supported.
         return Promise.resolve();
     }

src/datascience-ui/interactive-common/redux/reducers/variables.ts

@@ -106,16 +106,21 @@ function handleResponse(arg: VariableReducerArg<IJupyterVariablesResponse>): IVa
 }
 
 function handleRestarted(arg: VariableReducerArg): IVariableState {
-    // If the variables are visible, refresh them
-    if (arg.prevState.visible) {
-        const result = handleRequest({ ...arg, payload: { executionCount: 0, sortColumn: 'name', sortAscending: true, startIndex: 0, pageSize: arg.prevState.pageSize } });
-        return {
-            ...result,
-            currentExecutionCount: 0,
-            variables: []
-        };
-    }
-    return arg.prevState;
+    const result = handleRequest({
+        ...arg,
+        payload: {
+            executionCount: 0,
+            sortColumn: 'name',
+            sortAscending: true,
+            startIndex: 0,
+            pageSize: arg.prevState.pageSize
+        }
+    });
+    return {
+        ...result,
+        currentExecutionCount: 0,
+        variables: []
+    };
 }
 
 function handleFinishCell(arg: VariableReducerArg<ICell>): IVariableState {