jupyterlab_ai_code_assistants_extension 1.2.118 → 1.2.123

package/README.md

@@ -33,7 +33,7 @@ Chat-panel extensions re-implement the agent loop and trail the real tool. This
 - **File browser follows the opened session** - opening a session moves the JupyterLab file browser to that session's project folder (a setting, on by default); a right-click on a panel header toggles this and moves the panels between the left and right sidebars
 - **Launcher tiles** - every enabled assistant has a tile in an "AI Assistants" section of the JupyterLab Launcher (after Other, under the extension's own robot-head icon); a click opens the assistant in the file browser's current folder, resuming the folder's conversation when one exists and starting a new one otherwise, through the companion `jupyterlab_basic_terminal_extension` (installed automatically). Disable an assistant and its tile goes; disable them all and the section goes with them
 - **Your terminal's `claude -c` follows the panel** (Claude) - switching to a conversation makes plain `claude -c` in that project resume that same conversation, including one that has been compacted, which Claude Code otherwise refuses to continue. Launching repairs a compacted conversation the same way, but only switching moves what `claude -c` picks
-- **Conversation switcher** - a right-click "Switch and Manage Sessions" submenu lists a project's other conversations by name and short id with last-activity time; "Manage Sessions..." opens a searchable popup over the full list with multi-select delete and per-row open and copy-id buttons
+- **Conversation switcher** - a right-click "Switch and Manage Sessions" submenu lists a project's other conversations by name and short id with last-activity time; "Manage Sessions..." opens a searchable popup over the full list with multi-select delete, a per-row action that switches or opens according to the submenu it was opened from, and a copy-id button
 - **Branch session** - fork the current conversation into a new named session via the right-click menu; each assistant forks its own way (Claude's native `--fork-session`, Codex's app-server `thread/fork`, server-side copies for Kimi, Gemini and DeepSeek) behind the same menu item
 - **Rename session** - name a conversation from the right-click menu, and the name is written where the assistant itself reads it back: Claude gets the record its own `/rename` writes, Kimi a custom title, Gemini the chat's summary, DeepSeek its title event. The panel shows the name the assistant actually kept. Codex's name is set through Codex's own app-server, so its CLI lists the same name the panel shows
 - **Launch modes under each assistant's own name** - Claude's skip-permissions, Codex's approval bypass, Kimi's `--yolo`, Gemini's YOLO; unsafe variants carry a shield glyph in the launch menus

package/lib/core/panel.js

@@ -2083,7 +2083,13 @@ export class AssistantSessionsPanel extends Widget {
         });
         this._commands.addCommand(this._cmd('manage-sessions'), {
             label: () => `Manage Sessions... (${this._lastBranches.length})`,
-            execute: () => this._openManagePopup()
+            execute: () => this._openManagePopup('switch')
+        });
+        // The open submenu's copy of the item: same popup, but a row action
+        // launches the conversation rather than making it current.
+        this._commands.addCommand(this._cmd('manage-sessions-open'), {
+            label: () => `Manage Sessions... (${this._lastBranches.length})`,
+            execute: () => this._openManagePopup('open')
         });
         this._commands.addCommand(this._cmd('open-branch'), {
             // Names the mode the launch carries, like Resume, `+` and Branch Session
@@ -2417,10 +2423,10 @@ export class AssistantSessionsPanel extends Widget {
         }
         this._openBranchSubmenu.addItem({ type: 'separator' });
         this._openBranchSubmenu.addItem({
-            command: this._cmd('manage-sessions')
+            command: this._cmd('manage-sessions-open')
         });
     }
-    _openManagePopup() {
+    _openManagePopup(mode) {
         const session = this._activeSession;
         if (!session) {
             return;
@@ -2428,6 +2434,7 @@ export class AssistantSessionsPanel extends Widget {
         const current = this._lastBranchesCurrent || session.session_id;
         showManageSessionsPopup({
             trans: this._trans,
+            mode,
             branches: this._lastBranches,
             current,
             currentName: `${this._lookupName(session)} (${shortSessionId(current, this._descriptor.sessionIdPrefix)})`,

package/lib/core/popup.d.ts

@@ -29,6 +29,10 @@ export declare namespace ManageSessionsPopup {
         formatTime: (epochMs: number) => string;
         /** Provider chip for a branch row, or null. */
         branchBadge?: (branch: IBranch) => HTMLElement | null;
+        /** What a row's action does, inherited from the submenu that opened the
+         * popup: `switch` makes the conversation current, `open` launches it in
+         * its own terminal. */
+        mode: 'switch' | 'open';
         /** Tooltip for one row's Open button. The panel owns it because the launch
          * mode in force is panel state, and every other surface that opens a
          * conversation names it. */

package/lib/core/popup.js

@@ -115,19 +115,32 @@ export function showManageSessionsPopup(options) {
     });
     // Scopes the action-row layout in base.css to this one dialog.
     dialog.addClass('jp-AiAssistantsPanel-branchDialog');
-    // Per-row "Open" launches that conversation in its own terminal and closes
-    // the popup. stopPropagation keeps the click from toggling selection or
-    // switching the row.
-    const openButton = (sessionId) => {
+    // The row action follows the mode the invoking submenu passed: switch makes
+    // the conversation current, open launches it in its own terminal. Both close
+    // the popup first - the action is complete once it runs.
+    const runAction = (sessionId) => {
+        dialog.dispose();
+        if (options.mode === 'switch') {
+            options.onSwitch(sessionId);
+        }
+        else {
+            options.onOpen(sessionId);
+        }
+    };
+    // Per-row action button. stopPropagation keeps the click from toggling
+    // selection or repeating the action through the row's own click handler.
+    const actionButton = (sessionId) => {
         const btn = document.createElement('button');
         btn.type = 'button';
         btn.className = 'jp-AiAssistantsPanel-branchOpen';
-        btn.textContent = 'Open';
-        btn.title = options.openTitle();
+        btn.textContent = options.mode === 'switch' ? 'Switch' : 'Open';
+        btn.title =
+            options.mode === 'switch'
+                ? "Make this the project's current conversation"
+                : options.openTitle();
         btn.addEventListener('click', e => {
             e.stopPropagation();
-            dialog.dispose();
-            options.onOpen(sessionId);
+            runAction(sessionId);
         });
         return btn;
     };
@@ -192,7 +205,11 @@ export function showManageSessionsPopup(options) {
         badge.className = 'jp-AiAssistantsPanel-branchCurrentBadge';
         badge.textContent = 'current';
         currentRow.appendChild(badge);
-        currentRow.appendChild(openButton(options.current));
+        // Switch mode has nothing to switch to on the row that is already current;
+        // open mode can still launch it in a terminal.
+        if (options.mode === 'open') {
+            currentRow.appendChild(actionButton(options.current));
+        }
         currentRow.appendChild(copyButton(options.current, trans));
         list.appendChild(currentRow);
         const matches = visibleMatches();
@@ -270,20 +287,19 @@ export function showManageSessionsPopup(options) {
             time.className = 'jp-AiAssistantsPanel-branchTime';
             time.textContent = options.formatTime(b.file_mtime);
             row.appendChild(time);
-            row.appendChild(openButton(b.session_id));
+            row.appendChild(actionButton(b.session_id));
             row.appendChild(copyButton(b.session_id, trans));
-            // A row click always switches, whatever is selected: the checkbox cell
-            // is the one select target, so a click has one meaning (DEF-PANE-209).
-            const activate = () => {
-                dialog.dispose();
-                options.onSwitch(b.session_id);
-            };
+            // A row click follows the popup's mode, whatever is selected: the
+            // checkbox cell is the one select target, so a click has one meaning
+            // (DEF-PANE-209).
+            const activate = () => runAction(b.session_id);
             row.addEventListener('click', activate);
-            // Switch has no button of its own, so it needs a role and a key route or
-            // it is reachable by mouse only. The affordance sits on the label span,
-            // not the row: the row also hosts a checkbox and the Open/Copy buttons,
-            // and a role="button" wrapping them both swallows their own Enter/Space
-            // and makes them presentational children (ARIA nested-interactive).
+            // The row action also has a button; the label keeps a role and key route
+            // so the row itself stays keyboard-reachable. The affordance sits on the
+            // label span, not the row: the row also hosts a checkbox and the
+            // action/Copy buttons, and a role="button" wrapping them both swallows
+            // their own Enter/Space and makes them presentational children (ARIA
+            // nested-interactive).
             label.setAttribute('role', 'button');
             label.tabIndex = 0;
             label.addEventListener('keydown', e => {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "jupyterlab_ai_code_assistants_extension",
-    "version": "1.2.118",
+    "version": "1.2.123",
     "description": "A full launcher and manager for every AI code assistant in JupyterLab - Claude Code, Codex, Kimi, Gemini and DeepSeek. Start, resume, fork, switch, and clean up CLI sessions from a side panel per assistant - one click lands you in the right terminal with the assistant already running, no duplicate tabs, no UUID hunting.",
     "keywords": [
         "jupyter",

package/src/__tests__/panel.spec.ts

@@ -986,17 +986,43 @@ describe('the Manage Sessions popup is handed the current row name', () => {
         name: 'proj',
         session_id: 'session_9f8e7d6c-1111-2222-3333-444455556666'
       });
-      (p as any)._openManagePopup();
+      (p as any)._openManagePopup('switch');
       const calls = (showManageSessionsPopup as jest.Mock).mock.calls;
       const [options] = calls[calls.length - 1];
       // A front-slice would read `proj (session_)` on every conversation.
       expect(options.currentName).toEqual('proj (9f8e7d6c)');
+      expect(options.mode).toEqual('switch');
     } finally {
       p.dispose();
     }
   });
 });
 
+describe('Manage Sessions popup mode follows the invoking submenu (DEF-PANE-326)', () => {
+  const lastOptions = (): any => {
+    const calls = (showManageSessionsPopup as jest.Mock).mock.calls;
+    return calls[calls.length - 1][0];
+  };
+
+  it('the switch submenu opens the popup in switch mode', () => {
+    (panel as any)._activeSession = session({ name: 'proj' });
+    (panel as any)._fillBranchSubmenus([]);
+    const item = (panel as any)._switchSubmenu.items.at(-1);
+    (panel as any)._commands.execute(item.command, item.args);
+
+    expect(lastOptions().mode).toBe('switch');
+  });
+
+  it('the open submenu opens the popup in open mode', () => {
+    (panel as any)._activeSession = session({ name: 'proj' });
+    (panel as any)._fillBranchSubmenus([]);
+    const item = (panel as any)._openBranchSubmenu.items.at(-1);
+    (panel as any)._commands.execute(item.command, item.args);
+
+    expect(lastOptions().mode).toBe('open');
+  });
+});
+
 describe('per-conversation open surfaces', () => {
   function submenuLabels(p: AssistantSessionsPanel): string[] {
     (p as any)._rebuildContextMenu(true);
@@ -1827,7 +1853,7 @@ describe('ACC-SESS-190..192 - opening a session moves the file browser', () => {
       'the Manage Sessions Open button',
       (p, row) => {
         (p as any)._activeSession = row;
-        (p as any)._openManagePopup();
+        (p as any)._openManagePopup('open');
         const calls = (showManageSessionsPopup as jest.Mock).mock.calls;
         calls[calls.length - 1][0].onOpen('sid-branch');
       }

package/src/__tests__/popup.spec.ts

@@ -26,13 +26,15 @@ function branch(sessionId: string, label: string): IBranch {
 }
 
 let onSwitch: jest.Mock;
+let onOpen: jest.Mock;
 let onDelete: jest.Mock;
 // Each test opens a fresh popup in beforeEach; a describe that needs the
 // permanent mode flips this in its beforeAll, which runs ahead of it.
 let deleteToTrash = true;
 
-function open(): void {
+function open(mode: 'switch' | 'open' = 'switch'): void {
   showManageSessionsPopup({
+    mode,
     branches: [branch(FIRST, 'first'), branch(SECOND, 'second')],
     current: 'session-00000000-0000-4000-8000-000000000000',
     currentName: 'proj (00000000)',
@@ -41,12 +43,20 @@ function open(): void {
     formatTime: () => 'now',
     openTitle: () => 'Open',
     onSwitch,
-    onOpen: jest.fn(),
+    onOpen,
     onDelete,
     onRefetch: jest.fn(async () => [])
   });
 }
 
+/** Replace the default switch-mode popup with one opened in `mode`, and let
+ * the launch queue attach it before the test queries rows. */
+async function reopen(mode: 'switch' | 'open'): Promise<void> {
+  Dialog.flush();
+  open(mode);
+  await new Promise(resolve => setTimeout(resolve, 0));
+}
+
 function rows(): HTMLElement[] {
   return Array.from(
     document.querySelectorAll<HTMLElement>('.jp-AiAssistantsPanel-branchRow')
@@ -63,8 +73,16 @@ function deleteButton(): HTMLButtonElement {
   )!;
 }
 
+/** The per-row action button - Switch or Open, by popup mode. */
+function actionButtons(row: HTMLElement): HTMLButtonElement[] {
+  return Array.from(
+    row.querySelectorAll<HTMLButtonElement>('.jp-AiAssistantsPanel-branchOpen')
+  );
+}
+
 beforeEach(() => {
   onSwitch = jest.fn();
+  onOpen = jest.fn();
   onDelete = jest.fn(async () => 0);
   open();
 });
@@ -100,6 +118,61 @@ describe('Manage Sessions popup click zones', () => {
   });
 });
 
+describe('Manage Sessions popup action mode (DEF-PANE-326)', () => {
+  it('switch mode labels non-current rows Switch and calls onSwitch', () => {
+    const [, first, second] = rows();
+    const buttons = [...actionButtons(first), ...actionButtons(second)];
+    expect(buttons.map(b => b.textContent)).toEqual(['Switch', 'Switch']);
+    expect(buttons[0].title).toBe(
+      "Make this the project's current conversation"
+    );
+
+    actionButtons(second)[0].click();
+
+    expect(onSwitch).toHaveBeenCalledWith(SECOND);
+    expect(onOpen).not.toHaveBeenCalled();
+  });
+
+  it('open mode labels non-current rows Open and calls onOpen', async () => {
+    await reopen('open');
+    const [, first, second] = rows();
+    const buttons = [...actionButtons(first), ...actionButtons(second)];
+    expect(buttons.map(b => b.textContent)).toEqual(['Open', 'Open']);
+    expect(buttons[0].title).toBe('Open');
+
+    actionButtons(second)[0].click();
+
+    expect(onOpen).toHaveBeenCalledWith(SECOND);
+    expect(onSwitch).not.toHaveBeenCalled();
+  });
+
+  it('a row click switches in switch mode and opens in open mode', async () => {
+    const [, first] = rows();
+    first.click();
+
+    expect(onSwitch).toHaveBeenCalledWith(FIRST);
+    expect(onOpen).not.toHaveBeenCalled();
+
+    await reopen('open');
+    const [, openFirst] = rows();
+    openFirst.click();
+
+    expect(onOpen).toHaveBeenCalledWith(FIRST);
+    expect(onSwitch).toHaveBeenCalledTimes(1);
+  });
+
+  it('the current row has no action in switch mode and Open in open mode', async () => {
+    const [current] = rows();
+    expect(actionButtons(current)).toHaveLength(0);
+
+    await reopen('open');
+    const [openCurrent] = rows();
+    expect(actionButtons(openCurrent).map(b => b.textContent)).toEqual([
+      'Open'
+    ]);
+  });
+});
+
 describe('Manage Sessions popup focus (DEF-PANE-314)', () => {
   // The popup moves the focus to the filter once the dialog is ready.
   const ready = (): Promise<void> => Dialog.tracker.find(() => true)!.ready;

package/src/core/panel.ts

@@ -2424,7 +2424,14 @@ export class AssistantSessionsPanel extends Widget {
 
     this._commands.addCommand(this._cmd('manage-sessions'), {
       label: () => `Manage Sessions... (${this._lastBranches.length})`,
-      execute: () => this._openManagePopup()
+      execute: () => this._openManagePopup('switch')
+    });
+
+    // The open submenu's copy of the item: same popup, but a row action
+    // launches the conversation rather than making it current.
+    this._commands.addCommand(this._cmd('manage-sessions-open'), {
+      label: () => `Manage Sessions... (${this._lastBranches.length})`,
+      execute: () => this._openManagePopup('open')
     });
 
     this._commands.addCommand(this._cmd('open-branch'), {
@@ -2791,11 +2798,11 @@ export class AssistantSessionsPanel extends Widget {
     }
     this._openBranchSubmenu.addItem({ type: 'separator' });
     this._openBranchSubmenu.addItem({
-      command: this._cmd('manage-sessions')
+      command: this._cmd('manage-sessions-open')
     });
   }
 
-  private _openManagePopup(): void {
+  private _openManagePopup(mode: 'switch' | 'open'): void {
     const session = this._activeSession;
     if (!session) {
       return;
@@ -2803,6 +2810,7 @@ export class AssistantSessionsPanel extends Widget {
     const current = this._lastBranchesCurrent || session.session_id;
     showManageSessionsPopup({
       trans: this._trans,
+      mode,
       branches: this._lastBranches,
       current,
       currentName: `${this._lookupName(session)} (${shortSessionId(

package/src/core/popup.ts

@@ -35,6 +35,10 @@ export namespace ManageSessionsPopup {
     formatTime: (epochMs: number) => string;
     /** Provider chip for a branch row, or null. */
     branchBadge?: (branch: IBranch) => HTMLElement | null;
+    /** What a row's action does, inherited from the submenu that opened the
+     * popup: `switch` makes the conversation current, `open` launches it in
+     * its own terminal. */
+    mode: 'switch' | 'open';
     /** Tooltip for one row's Open button. The panel owns it because the launch
      * mode in force is panel state, and every other surface that opens a
      * conversation names it. */
@@ -185,19 +189,32 @@ export function showManageSessionsPopup(
   // Scopes the action-row layout in base.css to this one dialog.
   dialog.addClass('jp-AiAssistantsPanel-branchDialog');
 
-  // Per-row "Open" launches that conversation in its own terminal and closes
-  // the popup. stopPropagation keeps the click from toggling selection or
-  // switching the row.
-  const openButton = (sessionId: string): HTMLButtonElement => {
+  // The row action follows the mode the invoking submenu passed: switch makes
+  // the conversation current, open launches it in its own terminal. Both close
+  // the popup first - the action is complete once it runs.
+  const runAction = (sessionId: string): void => {
+    dialog.dispose();
+    if (options.mode === 'switch') {
+      options.onSwitch(sessionId);
+    } else {
+      options.onOpen(sessionId);
+    }
+  };
+
+  // Per-row action button. stopPropagation keeps the click from toggling
+  // selection or repeating the action through the row's own click handler.
+  const actionButton = (sessionId: string): HTMLButtonElement => {
     const btn = document.createElement('button');
     btn.type = 'button';
     btn.className = 'jp-AiAssistantsPanel-branchOpen';
-    btn.textContent = 'Open';
-    btn.title = options.openTitle();
+    btn.textContent = options.mode === 'switch' ? 'Switch' : 'Open';
+    btn.title =
+      options.mode === 'switch'
+        ? "Make this the project's current conversation"
+        : options.openTitle();
     btn.addEventListener('click', e => {
       e.stopPropagation();
-      dialog.dispose();
-      options.onOpen(sessionId);
+      runAction(sessionId);
     });
     return btn;
   };
@@ -271,7 +288,11 @@ export function showManageSessionsPopup(
     badge.className = 'jp-AiAssistantsPanel-branchCurrentBadge';
     badge.textContent = 'current';
     currentRow.appendChild(badge);
-    currentRow.appendChild(openButton(options.current));
+    // Switch mode has nothing to switch to on the row that is already current;
+    // open mode can still launch it in a terminal.
+    if (options.mode === 'open') {
+      currentRow.appendChild(actionButton(options.current));
+    }
     currentRow.appendChild(copyButton(options.current, trans));
     list.appendChild(currentRow);
 
@@ -353,21 +374,20 @@ export function showManageSessionsPopup(
       time.textContent = options.formatTime(b.file_mtime);
       row.appendChild(time);
 
-      row.appendChild(openButton(b.session_id));
+      row.appendChild(actionButton(b.session_id));
       row.appendChild(copyButton(b.session_id, trans));
 
-      // A row click always switches, whatever is selected: the checkbox cell
-      // is the one select target, so a click has one meaning (DEF-PANE-209).
-      const activate = (): void => {
-        dialog.dispose();
-        options.onSwitch(b.session_id);
-      };
+      // A row click follows the popup's mode, whatever is selected: the
+      // checkbox cell is the one select target, so a click has one meaning
+      // (DEF-PANE-209).
+      const activate = (): void => runAction(b.session_id);
       row.addEventListener('click', activate);
-      // Switch has no button of its own, so it needs a role and a key route or
-      // it is reachable by mouse only. The affordance sits on the label span,
-      // not the row: the row also hosts a checkbox and the Open/Copy buttons,
-      // and a role="button" wrapping them both swallows their own Enter/Space
-      // and makes them presentational children (ARIA nested-interactive).
+      // The row action also has a button; the label keeps a role and key route
+      // so the row itself stays keyboard-reachable. The affordance sits on the
+      // label span, not the row: the row also hosts a checkbox and the
+      // action/Copy buttons, and a role="button" wrapping them both swallows
+      // their own Enter/Space and makes them presentational children (ARIA
+      // nested-interactive).
       label.setAttribute('role', 'button');
       label.tabIndex = 0;
       label.addEventListener('keydown', e => {

package/style/base.css

@@ -550,26 +550,6 @@ body[data-jp-theme-light='true'] .jp-AiAssistantsPanel-dotHalo {
   order: -1;
 }
 
-/* The spin carries "busy", not information, so under a reduced-motion
-   preference it degrades to the same ring held still and dimmed. */
-@media (prefers-reduced-motion: reduce) {
-  .jp-AiAssistantsPanel-iconButton.jp-mod-spinning svg,
-  .jp-aiAssistants-spinner,
-  .jp-AiAssistantsPanel-spinner {
-    animation: none;
-    opacity: 0.5;
-  }
-
-  /* The click-time busy spinner is the only thing on screen while a click is
-     being handled, so it keeps full opacity: dimmed, its brand arc measures
-     1.74:1 on the light panel, below the 3:1 minimum for a non-text cue. Held
-     still it would read as idle, so the overlay names the action in a visible
-     label beside it; the label, not an animation, carries the busy state. */
-  .jp-AiAssistants-busySpinner {
-    opacity: 1;
-  }
-}
-
 /* Full-panel veil shown only during an explicit refresh (refresh button or
    panel show) - the background poll never raises it. The rule is gated on
    :not([hidden]) because this author `display` would otherwise beat the UA