jupyterlab_ai_code_assistants_extension 1.2.99 → 1.2.115

package/README.md

@@ -76,6 +76,23 @@ pip install jupyterlab_ai_code_assistants_extension
 > [!WARNING]
 > `package.json` pins `webpack: 5.106.0` and `chalk: 4.1.2` in both `resolutions` and `overrides`. Do not remove these. webpack `>= 5.106.1` changed its module-federation share identifier format and crashes the unmaintained `license-webpack-plugin` (`split('=')[1].trim()`) that `@jupyterlab/builder` injects into every production build; the duplicate `chalk@2.4.2` pulled by `duplicate-package-checker-webpack-plugin` crashes on Node 24+ in the build-isolation install. Without the pins, `make publish` and CI fail on `python -m build`.
 
+## Agent skill
+
+The wheel ships a session-operations skill for coding agents at `share/jupyter/agents/skills/jupyterlab-ai-code-assistants-extension/SKILL.md`. It covers finding, moving, splitting, renaming and promoting sessions, copying a session into another project under a fresh id, the task-list environment variables, and running Claude Code against Kimi K3 or DeepSeek.
+
+After `pip install`, with the Python that runs the lab:
+
+```bash
+mkdir -p ~/.agents/skills
+ln -s "$(python -c 'import sys; print(sys.prefix)')/share/jupyter/agents/skills/jupyterlab-ai-code-assistants-extension" ~/.agents/skills/jupyterlab-ai-code-assistants-extension
+```
+
+From a clone:
+
+```bash
+ln -s "$PWD/.agents/skills/jupyterlab-ai-code-assistants-extension" ~/.claude/skills/jupyterlab-ai-code-assistants-extension
+```
+
 ## Migrating from the standalone extensions
 
 This package supersedes `jupyterlab_claude_code_extension`, `jupyterlab_codex_extension` and `jupyterlab_kimi_code_extension`. On first run it carries their settings and favourites over automatically, and it never replaces a setting you have already saved in this extension; uninstall the standalones once you are ready:

package/lib/core/panel.d.ts

@@ -45,10 +45,14 @@ export declare class AssistantSessionsPanel extends Widget {
     protected onBeforeHide(_msg: Message): void;
     protected onCloseRequest(msg: Message): void;
     private _buildShell;
-    /** Raise the click-time busy overlay. Counted, because an open and a branch
-     * can overlap; the overlay clears when the last one ends. */
+    /** Raise the click-time busy overlay. Each action gets a token, because an
+     * open and a branch can overlap; the overlay clears when the last one ends,
+     * and the label names the newest action still running. */
     private _beginBusy;
     private _endBusy;
+    /** Name the newest still-running action in the visible label and the live
+     * region, so neither keeps naming an action that ended. */
+    private _paintBusyMessage;
     /** Show / hide the filter input. Hiding also clears the active filter, so
      * the user never returns to an "invisible" filter narrowing the rows. */
     private _toggleFilterBar;
@@ -90,9 +94,9 @@ export declare class AssistantSessionsPanel extends Widget {
     private _logError;
     private _fetch;
     private _toggleFavourite;
-    /** The warning a permanent deletion earns and a trashed one does not - a
-     * trashed conversation is recoverable, and the popup one click away says
-     * so; the same sentence on both would be false half the time. */
+    /** The warning a deletion earns: with the trash setting on, a refused move
+     * may remove the conversations permanently or leave them in place, so
+     * neither the verb nor the note may promise one outcome. */
     private get _irreversibleNote();
     private get _disposalVerb();
     private _removeProject;
@@ -307,8 +311,10 @@ export declare class AssistantSessionsPanel extends Widget {
     private _bodyEl;
     private _loadingEl;
     private _busyEl;
+    private _busyLabelEl;
     private _busyTextEl;
-    private _busyCount;
+    private _busySeq;
+    private _busyMessages;
     private _errorEl;
     private _refreshBtn;
     private _filterBtn;

package/lib/core/panel.js

@@ -113,8 +113,10 @@ export class AssistantSessionsPanel extends Widget {
         this._removingPaths = new Set();
         this._loadingEl = null;
         this._busyEl = null;
+        this._busyLabelEl = null;
         this._busyTextEl = null;
-        this._busyCount = 0;
+        this._busySeq = 0;
+        this._busyMessages = new Map();
         this._errorEl = null;
         this._refreshBtn = null;
         this._filterBtn = null;
@@ -186,7 +188,13 @@ export class AssistantSessionsPanel extends Widget {
         return this._descriptor;
     }
     dispose() {
+        var _a;
         this._stopPolling();
+        // The busy overlay lives on the shell, outside this widget's tree, so
+        // removing the panel does not remove it. Left behind it would show a
+        // spinner for a panel that no longer exists.
+        (_a = this._busyEl) === null || _a === void 0 ? void 0 : _a.remove();
+        this._busyEl = null;
         // Disabling a provider must leave its running terminals alive and only
         // drop the tint this panel applied.
         void this._terminals.clearColours();
@@ -394,17 +402,28 @@ export class AssistantSessionsPanel extends Widget {
         loading.appendChild(loadingSpinner);
         // Click-time busy overlay: raised synchronously when a row is opened or
         // branched, before the terminal probe or the branch request, so the click
-        // never reads as ignored (DEF-PANE-318). Unlike the refresh veil it takes
-        // no pointer events and no focus: the context menu that launched a branch
-        // stays open until its command runs, and a focus change would close it.
+        // never reads as ignored (DEF-PANE-318). It sits on the LAB SHELL, not on
+        // the panel: the feedback belongs in the main working area, where the
+        // launch spinner appears, and a panel that is hidden or scrolled must not
+        // keep it out of sight (DEF-PANE-318-1). It takes no pointer events and no
+        // focus: the context menu that launched a branch stays open until its
+        // command runs, and a focus change would close it.
         const busy = document.createElement('div');
-        busy.className = 'jp-AiAssistantsPanel-busyOverlay';
+        busy.className = 'jp-AiAssistants-busyOverlay';
         busy.hidden = true;
         const busySpinner = document.createElement('div');
         busySpinner.className =
-            'jp-aiAssistants-spinner jp-AiAssistantsPanel-busySpinner';
+            'jp-aiAssistants-spinner jp-AiAssistants-busySpinner';
         busySpinner.setAttribute('aria-hidden', 'true');
         busy.appendChild(busySpinner);
+        // The action, named in sight: under a reduced-motion preference the ring
+        // is held still, so the label carries the busy state with no animation.
+        // Hidden from assistive technology because the live region below
+        // announces the same words once.
+        const busyLabel = document.createElement('div');
+        busyLabel.className = 'jp-AiAssistants-busyLabel';
+        busyLabel.setAttribute('aria-hidden', 'true');
+        busy.appendChild(busyLabel);
         // The spinner is decoration; a screen-reader user hears the action
         // instead. The region sits on the root, always rendered and outside the
         // overlay and the aria-busy body: a live region created inside a
@@ -428,34 +447,47 @@ export class AssistantSessionsPanel extends Widget {
         root.appendChild(error);
         root.appendChild(body);
         root.appendChild(loading);
-        root.appendChild(busy);
         root.appendChild(busyText);
+        // The shell carries the visible feedback; the panel carries the
+        // announcement.
+        this._app.shell.node.appendChild(busy);
         this._bodyEl = body;
         this._loadingEl = loading;
         this._busyEl = busy;
+        this._busyLabelEl = busyLabel;
         this._busyTextEl = busyText;
         this._errorEl = error;
     }
-    /** Raise the click-time busy overlay. Counted, because an open and a branch
-     * can overlap; the overlay clears when the last one ends. */
+    /** Raise the click-time busy overlay. Each action gets a token, because an
+     * open and a branch can overlap; the overlay clears when the last one ends,
+     * and the label names the newest action still running. */
     _beginBusy(message) {
-        this._busyCount += 1;
+        const token = ++this._busySeq;
+        this._busyMessages.set(token, message);
         if (this._busyEl) {
             this._busyEl.hidden = false;
-            if (this._busyTextEl) {
-                this._busyTextEl.textContent = message;
-            }
+            this._paintBusyMessage();
             // The body is what the action changes; the root stays outside the
             // busy subtree so the live region above is not withheld.
             if (this._bodyEl) {
                 this._bodyEl.setAttribute('aria-busy', 'true');
             }
         }
+        return token;
     }
-    _endBusy() {
-        this._busyCount = Math.max(0, this._busyCount - 1);
-        if (this._busyCount === 0 && this._busyEl) {
+    _endBusy(token) {
+        this._busyMessages.delete(token);
+        if (this._busyMessages.size > 0) {
+            // Another action is still running; neither the visible label nor the
+            // live region may keep naming the one that just ended.
+            this._paintBusyMessage();
+            return;
+        }
+        if (this._busyEl) {
             this._busyEl.hidden = true;
+            if (this._busyLabelEl) {
+                this._busyLabelEl.textContent = '';
+            }
             if (this._busyTextEl) {
                 this._busyTextEl.textContent = '';
             }
@@ -464,6 +496,20 @@ export class AssistantSessionsPanel extends Widget {
             }
         }
     }
+    /** Name the newest still-running action in the visible label and the live
+     * region, so neither keeps naming an action that ended. */
+    _paintBusyMessage() {
+        let message = '';
+        for (const active of this._busyMessages.values()) {
+            message = active;
+        }
+        if (this._busyLabelEl) {
+            this._busyLabelEl.textContent = message;
+        }
+        if (this._busyTextEl) {
+            this._busyTextEl.textContent = message;
+        }
+    }
     /** Show / hide the filter input. Hiding also clears the active filter, so
      * the user never returns to an "invisible" filter narrowing the rows. */
     _toggleFilterBar() {
@@ -663,14 +709,18 @@ export class AssistantSessionsPanel extends Widget {
             this._showActionError('Could not toggle favorite - try again.', err);
         }
     }
-    /** The warning a permanent deletion earns and a trashed one does not - a
-     * trashed conversation is recoverable, and the popup one click away says
-     * so; the same sentence on both would be false half the time. */
+    /** The warning a deletion earns: with the trash setting on, a refused move
+     * may remove the conversations permanently or leave them in place, so
+     * neither the verb nor the note may promise one outcome. */
     get _irreversibleNote() {
-        return this._deleteToTrash ? '' : ' This cannot be undone.';
+        return this._deleteToTrash
+            ? ' A permanent removal cannot be undone.'
+            : ' This cannot be undone.';
     }
     get _disposalVerb() {
-        return this._deleteToTrash ? 'moved to trash' : 'deleted permanently';
+        return this._deleteToTrash
+            ? 'moved to the trash when the system accepts them; a refused move may remove them permanently or leave them in place'
+            : 'deleted permanently';
     }
     async _removeProject(session) {
         var _a;
@@ -805,7 +855,7 @@ export class AssistantSessionsPanel extends Widget {
     async _openSession(session, force, sessionId) {
         // The click's own feedback, raised before the terminal walk and the
         // launch - both can take seconds (DEF-PANE-318).
-        this._beginBusy(this._trans.__('Opening conversation'));
+        const busy = this._beginBusy(this._trans.__('Opening conversation'));
         try {
             await this._terminals.openSession(session, this._launchMode(force), sessionId);
             // Not awaited: the open has succeeded, and the sync is not part of it.
@@ -820,7 +870,7 @@ export class AssistantSessionsPanel extends Widget {
             }
         }
         finally {
-            this._endBusy();
+            this._endBusy(busy);
             // Reuse or fresh launch, either way the picture changed. Pull fresh
             // state; a poll tick retries if this one fails.
             void this._fetch().catch(() => undefined);
@@ -955,7 +1005,10 @@ export class AssistantSessionsPanel extends Widget {
             if (row) {
                 // A terminal already holding that conversation IS the answer: a second
                 // process on one history is never what the click meant (ACC-LNCH-152).
-                const found = await this._terminals.findForSession(row.session_id);
+                // The folder is required as well: a transcript re-homed into another
+                // project keeps its id, and that terminal is not this folder's
+                // (DEF-PANE-319).
+                const found = await this._terminals.findForSession(row.session_id, folder);
                 if (found) {
                     this._terminals.focus(found.widget);
                     return found.widget;
@@ -1049,12 +1102,12 @@ export class AssistantSessionsPanel extends Widget {
         // Raised on the click: the branch POST mints the fork server-side (for
         // Claude and Codex by running their own CLI) and can take seconds
         // (DEF-PANE-318).
-        this._beginBusy(this._trans.__('Branching conversation'));
+        const busy = this._beginBusy(this._trans.__('Branching conversation'));
         try {
             await this._branchByServerCopy(session, undefined, force);
         }
         finally {
-            this._endBusy();
+            this._endBusy(busy);
         }
     }
     /** The server mints the new id and answers with it before the terminal
@@ -1217,7 +1270,6 @@ export class AssistantSessionsPanel extends Widget {
     }
     /** Make another conversation the row's current one. */
     async _switchBranch(session, sessionId) {
-        var _a;
         try {
             const result = await requestProvider(this._descriptor.id, 'switch', this._serverSettings, {
                 method: 'POST',
@@ -1229,15 +1281,6 @@ export class AssistantSessionsPanel extends Widget {
             if (result.current !== result.requested) {
                 Notification.warning('That conversation could not be made the current one.', { autoClose: 4000 });
             }
-            else {
-                // The re-render is the only other signal, and it can read as the
-                // opposite of success: the row now reports the conversation's true
-                // last activity, so switching to an old one dims the row and drops
-                // it out of Recent (DEF-PANE-181).
-                const target = this._lastBranches.find(b => b.session_id === sessionId);
-                const label = (_a = target === null || target === void 0 ? void 0 : target.label) !== null && _a !== void 0 ? _a : shortSessionId(sessionId, this._descriptor.sessionIdPrefix);
-                Notification.success(`${this._lookupName(session)}: switched to ${label}`, { autoClose: 3000 });
-            }
         }
         catch (err) {
             Notification.error(isResponseStatus(err, 404)

package/lib/core/popup.js

@@ -44,9 +44,9 @@ export function showManageSessionsPopup(options) {
     let items = [...options.branches];
     const selected = new Set();
     let deleting = false; // guards the async delete against double invocation
-    // Permanent deletion arms first and executes on the second click; a trashed
-    // deletion is recoverable and goes straight through.
-    const armsFirst = !options.deleteToTrash;
+    // Every deletion arms first and executes on the second click: the trash
+    // setting does not promise the system will accept the move, and a fallback
+    // removal is permanent.
     let armed = false;
     let armTimer = 0;
     const body = document.createElement('div');
@@ -87,7 +87,7 @@ export function showManageSessionsPopup(options) {
     deleteBtn.type = 'button';
     deleteBtn.className = 'jp-AiAssistantsPanel-branchDelete';
     deleteBtn.title = options.deleteToTrash
-        ? 'Deleted conversations move to the trash (recoverable)'
+        ? 'Deleted conversations move to the trash when the system accepts them; a refused move may remove them permanently or leave them in place'
         : 'Deleted conversations are removed permanently';
     footer.appendChild(deleteBtn);
     // Plain visual counter (selection, or the last action), beside the button
@@ -336,12 +336,12 @@ export function showManageSessionsPopup(options) {
             return;
         }
         // No confirmation DIALOG here: a second Lumino dialog stacked on this popup
-        // renders detached. A permanent deletion still confirms, in the footer
-        // itself - the first click arms the button, the second executes, and any
-        // selection change or four seconds of hesitation disarms it. A trashed
-        // deletion is recoverable, so it keeps going through on one click, with
-        // the live "N deleted" status and a toast from the caller on failure.
-        if (armsFirst && !armed) {
+        // renders detached. The deletion confirms in the footer itself - the first
+        // click arms the button, the second executes, and any selection change or
+        // four seconds of hesitation disarms it. Every deletion arms, because the
+        // trash setting does not promise the system will accept the move, and a
+        // refused move may remove the conversation permanently or leave it.
+        if (!armed) {
             armed = true;
             window.clearTimeout(armTimer);
             armTimer = window.setTimeout(() => {
@@ -384,9 +384,9 @@ export function showManageSessionsPopup(options) {
             (_a = options.onSync) === null || _a === void 0 ? void 0 : _a.call(options, items);
             render();
             updateControls();
-            const moved = options.deleteToTrash
-                ? `${deleted} moved to trash`
-                : `${deleted} deleted`;
+            // One wording for both outcomes: the client cannot know whether the
+            // trash accepted the move, so it does not claim it did.
+            const moved = `${deleted} deleted`;
             selCount.textContent = moved;
             selCount.classList.add('jp-mod-deleted');
             // Clear then re-set on the next tick so an identical count (two single

package/lib/core/terminals.d.ts

@@ -56,18 +56,26 @@ export declare class TerminalManager {
      * widget is attached and visible.
      */
     focus(widget: any): void;
-    /** Find an open terminal running exactly this conversation, or null.
+    /** Find an open terminal running exactly this conversation in this project,
+     * or null.
      *
-     * Matches on the conversation id ALONE. The server reads the true id from
-     * the running process, so a session the user started by hand is recognised
-     * too, not only launches that carry an argv id. An id is globally unique to
-     * one conversation, so a terminal holding it IS the one to focus, whatever
-     * cwd it reports - a process that changed directory, or whose project dir
-     * was recreated, must still be reused rather than duplicated. */
-    findForSession(wantedSessionId: string | undefined): Promise<{
+     * The server reads the true id from the running process, so a session the
+     * user started by hand is recognised too, not only launches that carry an
+     * argv id. The terminal must ALSO work in the requested project. An id is
+     * not globally unique in the store: Claude re-homes a transcript into a
+     * second project under the same id, so an id match alone can name a process
+     * running it from another tree, and reusing that one opens nothing in the
+     * folder the user clicked (DEF-PANE-319). A process that changed directory
+     * WITHIN the project still matches - the probe lists every cwd in its
+     * tree. */
+    findForSession(wantedSessionId: string | undefined, projectPath: string): Promise<{
         widget: any;
         runningId: string | null;
     } | null>;
+    /** Whether any process in the terminal's tree works in `projectPath` or a
+     * directory under it. No cwds means nothing to prove the terminal belongs
+     * to this project, so it is not reused. */
+    private _runsInProject;
     /** Ask the server what a terminal is running. Null when the widget has no
      * session name yet, or the probe fails - a terminal can vanish between
      * enumeration and probe, which is a retry, not an error. */

package/lib/core/terminals.js

@@ -184,8 +184,9 @@ export class TerminalManager {
             this.focus(cached.widget);
             return;
         }
-        // 2. Walk every live terminal, asking the server what each is running.
-        const found = await this.findForSession(wanted);
+        // 2. Walk every live terminal, asking the server what each is running in
+        // this project.
+        const found = await this.findForSession(wanted, session.project_path);
         if (found) {
             // Tag with the OBSERVED conversation, so a later reuse trusts a
             // confirmed identity rather than a wish.
@@ -255,26 +256,43 @@ export class TerminalManager {
             }
         });
     }
-    /** Find an open terminal running exactly this conversation, or null.
+    /** Find an open terminal running exactly this conversation in this project,
+     * or null.
      *
-     * Matches on the conversation id ALONE. The server reads the true id from
-     * the running process, so a session the user started by hand is recognised
-     * too, not only launches that carry an argv id. An id is globally unique to
-     * one conversation, so a terminal holding it IS the one to focus, whatever
-     * cwd it reports - a process that changed directory, or whose project dir
-     * was recreated, must still be reused rather than duplicated. */
-    async findForSession(wantedSessionId) {
+     * The server reads the true id from the running process, so a session the
+     * user started by hand is recognised too, not only launches that carry an
+     * argv id. The terminal must ALSO work in the requested project. An id is
+     * not globally unique in the store: Claude re-homes a transcript into a
+     * second project under the same id, so an id match alone can name a process
+     * running it from another tree, and reusing that one opens nothing in the
+     * folder the user clicked (DEF-PANE-319). A process that changed directory
+     * WITHIN the project still matches - the probe lists every cwd in its
+     * tree. */
+    async findForSession(wantedSessionId, projectPath) {
         if (!this._tracker || !wantedSessionId) {
             return null;
         }
         for (const widget of this._liveTerminals()) {
             const info = await this.probe(widget);
-            if (info && info.session_id && info.session_id === wantedSessionId) {
+            if (info &&
+                info.session_id &&
+                info.session_id === wantedSessionId &&
+                this._runsInProject(info.cwds, projectPath)) {
                 return { widget, runningId: info.session_id };
             }
         }
         return null;
     }
+    /** Whether any process in the terminal's tree works in `projectPath` or a
+     * directory under it. No cwds means nothing to prove the terminal belongs
+     * to this project, so it is not reused. */
+    _runsInProject(cwds, projectPath) {
+        const root = projectPath.replace(/\/+$/, '');
+        return (cwds !== null && cwds !== void 0 ? cwds : []).some(cwd => {
+            const path = cwd.replace(/\/+$/, '');
+            return path === root || path.startsWith(`${root}/`);
+        });
+    }
     /** Ask the server what a terminal is running. Null when the widget has no
      * session name yet, or the probe fails - a terminal can vanish between
      * enumeration and probe, which is a retry, not an error. */

package/lib/index.js

@@ -174,9 +174,9 @@ const plugin = {
                     descriptor: module.descriptor,
                     hooks: module.hooks,
                     rootDir: (_a = status === null || status === void 0 ? void 0 : status.root_dir) !== null && _a !== void 0 ? _a : '',
-                    // Unknown reads as "not trash": the popup then arms its Delete
-                    // button before it executes, which is the only confirmation a
-                    // permanent bulk delete has (DEF-135).
+                    // Unknown reads as "not trash", so the dialogs and the popup
+                    // present the permanent-delete wording rather than promising a
+                    // recoverable move the server never confirmed (DEF-135).
                     deleteToTrash: status ? status.delete_to_trash !== false : false,
                     terminalTracker,
                     fileBrowser,

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "jupyterlab_ai_code_assistants_extension",
-    "version": "1.2.99",
+    "version": "1.2.115",
     "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__/index.spec.ts

@@ -110,7 +110,8 @@ describe('DEF-GUARD-139 - reconcile hands a late roster to a docked panel', () =
     const app = {
       serviceManager: { serverSettings: {} },
       commands: new CommandRegistry(),
-      hasPlugin: () => false
+      hasPlugin: () => false,
+      shell: { node: document.createElement('div') }
     } as any;
     const settings = {
       raw: '{}',
@@ -213,7 +214,8 @@ describe('ACC-LNCH-143..146 - the Launcher tile has the panel lifecycle', () =>
     const app = {
       serviceManager: { serverSettings: {} },
       commands: new CommandRegistry(),
-      hasPlugin: () => false
+      hasPlugin: () => false,
+      shell: { node: document.createElement('div') }
     } as any;
     const labShell = {
       add: (widget: AssistantSessionsPanel) => docked.push(widget)
@@ -288,7 +290,8 @@ describe('ACC-LNCH-143..146 - the Launcher tile has the panel lifecycle', () =>
       {
         serviceManager: { serverSettings: {} },
         commands: new CommandRegistry(),
-        hasPlugin: () => false
+        hasPlugin: () => false,
+        shell: { node: document.createElement('div') }
       } as any,
       { add: (widget: AssistantSessionsPanel) => docked.push(widget) } as any,
       null,
@@ -416,7 +419,8 @@ describe('ACC-PROV - the activation contract', () => {
     const app = {
       serviceManager: { serverSettings },
       commands: new CommandRegistry(),
-      hasPlugin: () => false
+      hasPlugin: () => false,
+      shell: { node: document.createElement('div') }
     } as any;
 
     const composite = options.composite;
@@ -878,6 +882,8 @@ describe('ACC-SETT - settings drive the panels', () => {
     const shellMoves: [string, string][] = [];
     const sequence: string[] = [];
     const labShell = {
+      // The panel mounts its click-time overlay on the shell node.
+      node: document.createElement('div'),
       // As the real sidebar does: a widget is hidden when it is added, and a
       // placement the user made with JupyterLab's own command overrides the
       // area the caller asks for.
@@ -934,7 +940,8 @@ describe('ACC-SETT - settings drive the panels', () => {
     const app = {
       serviceManager: { serverSettings },
       commands,
-      hasPlugin: () => false
+      hasPlugin: () => false,
+      shell: { node: document.createElement('div') }
     } as any;
 
     const changed = new Signal<unknown, void>({});
@@ -1758,7 +1765,8 @@ describe('ACC-RETI-82..84 - a retired standalone extension still installed', ()
     const app = {
       serviceManager: { serverSettings },
       commands: new CommandRegistry(),
-      hasPlugin: (id: string) => installed.includes(id)
+      hasPlugin: (id: string) => installed.includes(id),
+      shell: { node: document.createElement('div') }
     } as any;
     await (plugin.activate as any)(
       app,

package/src/__tests__/launcher.spec.ts

@@ -192,7 +192,10 @@ function harness(
       }
     },
     commands,
-    shell: { activateById: (id: string) => activated.push(id) }
+    shell: {
+      node: document.createElement('div'),
+      activateById: (id: string) => activated.push(id)
+    }
   } as any;
   const panel = new AssistantSessionsPanel({
     app,
@@ -279,7 +282,8 @@ describe('the tile click path resumes a folder that has a conversation', () => {
       'terminal/term-1': {
         terminal_name: 'term-1',
         running: true,
-        session_id: 'sid-proj'
+        session_id: 'sid-proj',
+        cwds: ['/srv/lab/proj']
       }
     });
     const h = harness({ tracker: true });
@@ -291,6 +295,32 @@ describe('the tile click path resumes a folder that has a conversation', () => {
     expect(callsTo('launch-argv')).toHaveLength(0);
   });
 
+  it('launches here when the terminal holds the conversation in another folder', async () => {
+    // DEF-PANE-319: a transcript re-homed into another project keeps its id,
+    // so the id alone cannot prove the terminal is this folder's.
+    routes({
+      sessions: { sessions: [session()] },
+      'terminal/term-1': {
+        terminal_name: 'term-1',
+        running: true,
+        session_id: 'sid-proj',
+        cwds: ['/srv/other']
+      },
+      'launch-argv': { argv: ['/bin/testbed', '--resume', 'sid-proj'] }
+    });
+    const h = harness({ tracker: true });
+    await flush();
+
+    expect(await h.panel.launchHere('proj')).toBe(LAUNCHED_WIDGET);
+    expect(h.activated).not.toContain('terminal-open-1');
+    expect(h.launches).toEqual([
+      {
+        argv: ['/bin/testbed', '--resume', 'sid-proj'],
+        cwd: '/srv/lab/proj'
+      }
+    ]);
+  });
+
   it('asks for resume argv when no terminal holds it, and launches that', async () => {
     // ACC-LNCH-151 / ACC-LNCH-154: the row's own conversation, resumed through
     // the sibling's terminal.
@@ -489,7 +519,8 @@ describe('the tiles the plugin adds to the Launcher', () => {
     const app = {
       serviceManager: { serverSettings: {} },
       commands: new CommandRegistry(),
-      hasPlugin: () => false
+      hasPlugin: () => false,
+      shell: { node: document.createElement('div') }
     } as any;
     const labShell = {
       add: (widget: AssistantSessionsPanel) => docked.push(widget)