fix: Fix deepnote notebook deserializer and file change watcher

build/mocha-esm-loader.js

@@ -335,6 +335,7 @@ export async function load(url, context, nextLoad) {
                 export const EndOfLine = createClassProxy('EndOfLine');
                 export const PortAutoForwardAction = createClassProxy('PortAutoForwardAction');
                 export const PortAttributes = createClassProxy('PortAttributes');
+                export const TabInputNotebook = createClassProxy('TabInputNotebook');
             `,
                 shortCircuit: true
             };

package.json

@@ -93,6 +93,12 @@
                 "category": "Deepnote",
                 "icon": "$(reveal)"
             },
+            {
+                "command": "deepnote.copyNotebookDetails",
+                "title": "%deepnote.commands.copyNotebookDetails.title%",
+                "category": "Deepnote",
+                "icon": "$(copy)"
+            },
             {
                 "command": "deepnote.enableSnapshots",
                 "title": "%deepnote.commands.enableSnapshots.title%",

package.nls.json

@@ -250,6 +250,7 @@
     "deepnote.commands.openNotebook.title": "Open Notebook",
     "deepnote.commands.openFile.title": "Open File",
     "deepnote.commands.revealInExplorer.title": "Reveal in Explorer",
+    "deepnote.commands.copyNotebookDetails.title": "Copy Active Deepnote Notebook Details",
     "deepnote.commands.enableSnapshots.title": "Enable Snapshots",
     "deepnote.commands.disableSnapshots.title": "Disable Snapshots",
     "deepnote.commands.manageIntegrations.title": "Manage Integrations",

src/kernels/deepnote/deepnoteServerStarter.node.ts

@@ -280,7 +280,7 @@ export class DeepnoteServerStarter implements IDeepnoteServerStarter, IExtension
 
             throw new DeepnoteServerStartupError(
                 interpreter.uri.fsPath,
-                serverInfo?.jupyterPort ?? 0,
+                0,
                 'unknown',
                 capturedOutput?.stdout || '',
                 capturedOutput?.stderr || '',
@@ -402,6 +402,16 @@ export class DeepnoteServerStarter implements IDeepnoteServerStarter, IExtension
      */
     private monitorServerOutput(fileKey: string, serverInfo: DeepnoteServerInfo): void {
         const proc = serverInfo.process;
+        const existing = this.disposablesByFile.get(fileKey);
+        if (existing) {
+            for (const d of existing) {
+                try {
+                    d.dispose();
+                } catch (ex) {
+                    logger.warn(`Error disposing listener for ${fileKey}`, ex);
+                }
+            }
+        }
         const disposables: IDisposable[] = [];
         this.disposablesByFile.set(fileKey, disposables);
 

src/kernels/deepnote/deepnoteTestHelpers.node.ts

@@ -5,11 +5,66 @@ import type { ChildProcess } from 'node:child_process';
  * Satisfies the ChildProcess interface with minimal stub values.
  */
 export function createMockChildProcess(overrides?: Partial<ChildProcess>): ChildProcess {
-    return {
+    const mockProcess: ChildProcess = {
         pid: undefined,
+        stdio: [null, null, null, null, null],
+        stdin: null,
         stdout: null,
         stderr: null,
         exitCode: null,
+        killed: false,
+        connected: false,
+        signalCode: null,
+        spawnargs: [],
+        spawnfile: '',
+        kill: () => true,
+        send: () => true,
+        disconnect: () => true,
+        unref: () => true,
+        ref: () => true,
+        addListener: function () {
+            return this;
+        },
+        emit: () => true,
+        on: function () {
+            return this;
+        },
+        once: function () {
+            return this;
+        },
+        removeListener: function () {
+            return this;
+        },
+        removeAllListeners: function () {
+            return this;
+        },
+        prependListener: function () {
+            return this;
+        },
+        prependOnceListener: function () {
+            return this;
+        },
+        [Symbol.dispose]: () => {
+            return undefined;
+        },
+        off: function () {
+            return this;
+        },
+        setMaxListeners: function () {
+            return this;
+        },
+        getMaxListeners: () => 10,
+        listeners: function () {
+            return [];
+        },
+        rawListeners: function () {
+            return [];
+        },
+        eventNames: function () {
+            return [];
+        },
+        listenerCount: () => 0,
         ...overrides
-    } as ChildProcess;
+    };
+    return mockProcess;
 }

src/notebooks/deepnote/deepnoteActivationService.ts

@@ -1,5 +1,5 @@
 import { inject, injectable, optional } from 'inversify';
-import { commands, l10n, workspace, window, type Disposable, type NotebookDocumentContentOptions } from 'vscode';
+import { commands, l10n, window, workspace, type Disposable, type NotebookDocumentContentOptions } from 'vscode';
 
 import { IExtensionSyncActivationService } from '../../platform/activation/types';
 import { IExtensionContext } from '../../platform/common/types';
@@ -51,6 +51,11 @@ export class DeepnoteActivationService implements IExtensionSyncActivationServic
 
         this.registerSerializer();
         this.extensionContext.subscriptions.push(this.editProtection);
+        this.extensionContext.subscriptions.push(
+            workspace.onDidOpenNotebookDocument((doc) => {
+                void this.serializer.verifyDeserializedNotebook(doc);
+            })
+        );
         this.extensionContext.subscriptions.push(
             workspace.onDidChangeConfiguration((event) => {
                 if (event.affectsConfiguration('deepnote.snapshots.enabled')) {

src/notebooks/deepnote/deepnoteExplorerView.ts

@@ -260,7 +260,7 @@ export class DeepnoteExplorerView {
             await this.treeDataProvider.refreshNotebook(treeItem.context.projectId);
 
             // Optionally open the duplicated notebook
-            this.manager.selectNotebookForProject(treeItem.context.projectId, newNotebook.id);
+            this.registerNotebookOpenIntent(treeItem.context.projectId, newNotebook.id);
             const notebookUri = fileUri.with({ query: `notebook=${newNotebook.id}` });
             const document = await workspace.openNotebookDocument(notebookUri);
             await window.showNotebookDocument(document, {
@@ -508,7 +508,7 @@ export class DeepnoteExplorerView {
         await this.treeDataProvider.refreshNotebook(projectData.project.id);
 
         // Open the new notebook
-        this.manager.selectNotebookForProject(projectData.project.id, notebookId);
+        this.registerNotebookOpenIntent(projectData.project.id, notebookId);
         const notebookUri = fileUri.with({ query: `notebook=${notebookId}` });
         const document = await workspace.openNotebookDocument(notebookUri);
         await window.showNotebookDocument(document, {
@@ -517,6 +517,10 @@ export class DeepnoteExplorerView {
         });
     }
 
+    private registerNotebookOpenIntent(projectId: string, notebookId: string): void {
+        this.manager.queueNotebookResolution(projectId, notebookId);
+    }
+
     private refreshExplorer(): void {
         this.treeDataProvider.refresh();
     }
@@ -537,7 +541,7 @@ export class DeepnoteExplorerView {
 
             console.log(`Selecting notebook in manager.`);
 
-            this.manager.selectNotebookForProject(context.projectId, context.notebookId);
+            this.registerNotebookOpenIntent(context.projectId, context.notebookId);
 
             console.log(`Opening notebook document.`, fileUri);
 
@@ -701,7 +705,7 @@ export class DeepnoteExplorerView {
 
             this.treeDataProvider.refresh();
 
-            this.manager.selectNotebookForProject(projectId, notebookId);
+            this.registerNotebookOpenIntent(projectId, notebookId);
 
             const notebookUri = fileUri.with({ query: `notebook=${notebookId}` });
             const document = await workspace.openNotebookDocument(notebookUri);