jupyterlab_claude_code_extension 1.1.15 → 1.1.17

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "jupyterlab_claude_code_extension",
-    "version": "1.1.15",
+    "version": "1.1.17",
     "description": "Browse, resume, and manage your Claude Code CLI sessions from a JupyterLab side panel. One click reactivates the right terminal - no duplicate tabs, live remote-control indicator, and favourites for the projects you keep coming back to.",
     "keywords": [
         "jupyter",

package/src/__tests__/jupyterlab_claude_code_extension.spec.ts

@@ -1,3 +1,10 @@
+declare const __dirname: string;
+declare function require(name: string): any;
+// eslint-disable-next-line @typescript-eslint/no-var-requires
+const fs: { readFileSync: (p: string, enc: string) => string } = require('fs');
+// eslint-disable-next-line @typescript-eslint/no-var-requires
+const path: { join: (...args: string[]) => string } = require('path');
+
 import type { ISession } from '../types';
 
 const session = (over: Partial<ISession> = {}): ISession => ({
@@ -61,3 +68,29 @@ describe('session sorting', () => {
     expect(recent[9].project_path).toBe('p15');
   });
 });
+
+/**
+ * Regression guard for 1.1.13 -> 1.1.14: the launch-spinner Dialog is
+ * constructed with ``buttons: []`` so ``Dialog.resolve()`` has no button
+ * to "click" and silently no-ops, leaving the modal stuck over the
+ * panel. The dismiss call in ``_doResumeInTerminal``'s ``finally`` MUST
+ * be ``spinner.dispose()``. These are source-level invariants - cheap,
+ * deterministic, and exactly the contract that broke in 1.1.13.
+ */
+describe('launch spinner dismiss contract', () => {
+  const widgetSrc: string = fs.readFileSync(
+    path.join(__dirname, '..', 'widget.ts'),
+    'utf-8'
+  );
+
+  it('_doResumeInTerminal dismisses spinner via dispose(), not resolve()', () => {
+    expect(widgetSrc).toMatch(/spinner\.dispose\(\)/);
+    expect(widgetSrc).not.toMatch(/spinner\.resolve\(\)/);
+  });
+
+  it('_showLaunchSpinner constructs Dialog with buttons:[]', () => {
+    // The empty buttons array is the reason resolve() no-ops; if this
+    // ever becomes non-empty, the dismiss call can revisit resolve().
+    expect(widgetSrc).toMatch(/_showLaunchSpinner[\s\S]*?buttons:\s*\[\]/);
+  });
+});