Native REPL wrongly caches state from previous session

[cbrnr]

I'm not sure if I'm missing something, but I cannot find an option to restart the native REPL. If I just close the pane and then send another Python statement (Shift-Enter), it gets executed in the previously started native REPL (i.e., all created object in that session will still be available).

[anthonykim1]

Interesting, I can't seem to repro this.
If you close the REPL tab via clicking on the x icon, It should launch new REPL for you when you try to send command again.
Could you try "Python Clear cache and reload" and see if that helps?
Also make sure it says "Python REPL" on the top right corner.

Here is what happens for me on the latest version of vscode insider and python extension.

Screen.Recording.2024-10-31.at.2.13.40.PM.mov

[cbrnr]

No, this does not work for me. When I close the Native REPL tab and run a new command, it opens the previous REPL (in this example, x should not exist in a new REPL, but apparently this is still the same session):

Screen.Recording.2024-11-04.at.15.41.22.mov

When I run "Python: Clear Cache and Reload Window", the native REPL session is indeed closed and a new one is started. However, I don't think this is intended, as the REPL should either close when closing the tab or when selecting a dedicated "Trash" action (which does not exist, but this is how the integrated terminal works, so I think the native REPL should behave similarly).

[anthonykim1]

Hey @cbrnr Thanks for this, you are 100% correct.
Will make sure this eventually gets resolved, I think this is very important.

[hutch3232]

I would love for this issue to be solved. I tried looking into it but haven't yet figured it out. I don't know typescript so it is tricky for me.

I feel like in here:

vscode-python/src/client/repl/nativeRepl.ts

Lines 70 to 80 in 42962ce

	private watchNotebookClosed(): void {
	this.disposables.push(
	workspace.onDidCloseNotebookDocument(async (nb) => {
	if (this.notebookDocument && nb.uri.toString() === this.notebookDocument.uri.toString()) {
	this.notebookDocument = undefined;
	this.newReplSession = true;
	await updateWorkspaceStateValue<string | undefined>(NATIVE_REPL_URI_MEMENTO, undefined);
	}
	}),
	);
	}

we need to run:

nativeRepl = undefined;

That way, in getNativeRepl it will create a new REPL.

But that on its own is not sufficient. After closing the REPL, sending new code throws: notebook controller with id 'pythonREPL' ALREADY exist

There is a setReplController method maybe we need an unsetReplController or something that gets called as well.

With a bit of guidance I could look more into this. Thanks!

[anthonykim1]

Sorry, just saw this @hutch3232
Will take a look at the referenced PR and leave feedback there.

Thanks very much for looking into this!