jupyterlab_claude_code_extension 1.2.5 → 1.2.9

package/src/__tests__/jupyterlab_claude_code_extension.spec.ts

@@ -138,10 +138,15 @@ describe('launch spinner dismiss contract', () => {
       /private async _switchBranch[\s\S]*?\n  \}/
     ) ?? [''])[0];
 
-    it('shows the conversation count only when the project has branches', () => {
-      expect(widgetSrc).toMatch(
-        /session\.extra_sessions > 0\s*\?\s*`\$\{this\._lookupName\(session\)\} \(\$\{session\.extra_sessions \+ 1\}\)`/
+    it('shows a branch icon + count badge only when the project has branches', () => {
+      const renderRow = (widgetSrc.match(
+        /private _renderRow[\s\S]*?\n  \}/
+      ) ?? [''])[0];
+      expect(renderRow).toMatch(
+        /session\.extra_sessions > 0[\s\S]*?jp-ClaudeSessionsPanel-branchBadge/
       );
+      expect(renderRow).toMatch(/branchIcon\.element/);
+      expect(renderRow).toMatch(/String\(session\.extra_sessions \+ 1\)/);
     });
 
     it('rebuilds submenu items from a fresh branches fetch on open', () => {
@@ -150,10 +155,21 @@ describe('launch spinner dismiss contract', () => {
       expect(openMenu).toMatch(/_rebuildContextMenu\(hasBranches\)/);
     });
 
-    it('caps the submenu at 5 and adds More... beyond that', () => {
+    it('caps the inline submenu at 5 most recent', () => {
       expect(openMenu).toMatch(/\.slice\(0, 5\)/);
+    });
+
+    it('always adds the Manage Sessions entry, no >5 gate', () => {
+      // The popup is the management hub - it must be reachable even for
+      // projects with 2-5 conversations, so the entry is unconditional.
+      expect(openMenu).toMatch(/switch-branch-more/);
+      expect(openMenu).not.toMatch(/branches\.length > 5/);
+      expect(widgetSrc).toMatch(/`Manage Sessions\.\.\. \(\$\{/);
+    });
+
+    it('titles the submenu Switch and Manage Sessions with the count', () => {
       expect(openMenu).toMatch(
-        /branches\.length > 5[\s\S]*?switch-branch-more/
+        /title\.label = `Switch and Manage Sessions \(\$\{data\.branches\.length\}\)`/
       );
     });
 
@@ -183,6 +199,70 @@ describe('launch spinner dismiss contract', () => {
       );
     });
 
+    it('popup leads with the current row, badged and without checkbox', () => {
+      const popup = (widgetSrc.match(
+        /private _showBranchPopup[\s\S]*?\n  \}/
+      ) ?? [''])[0];
+      expect(popup).toMatch(/jp-mod-current/);
+      expect(popup).toMatch(/branchCurrentBadge/);
+      // The current row is appended before any checkbox is created.
+      const currentIdx = popup.indexOf('jp-mod-current');
+      const checkboxIdx = popup.indexOf("check.type = 'checkbox'");
+      expect(currentIdx).toBeGreaterThan(-1);
+      expect(checkboxIdx).toBeGreaterThan(currentIdx);
+    });
+
+    it('checkbox is its own click zone and selection gates row switch', () => {
+      const popup = (widgetSrc.match(
+        /private _showBranchPopup[\s\S]*?\n  \}/
+      ) ?? [''])[0];
+      expect(popup).toMatch(/stopPropagation\(\)/);
+      // Selection mode: row click toggles while anything is selected.
+      expect(popup).toMatch(
+        /if \(selected\.size > 0\) \{[\s\S]*?return;[\s\S]*?dialog\.dispose\(\)/
+      );
+    });
+
+    it('select-all toggles the visible (filtered) rows only', () => {
+      const popup = (widgetSrc.match(
+        /private _showBranchPopup[\s\S]*?\n  \}/
+      ) ?? [''])[0];
+      expect(popup).toMatch(
+        /selectAll\.addEventListener\('change'[\s\S]*?visibleMatches\(\)/
+      );
+    });
+
+    it('delete is two-step and any selection change disarms it', () => {
+      const popup = (widgetSrc.match(
+        /private _showBranchPopup[\s\S]*?\n  \}/
+      ) ?? [''])[0];
+      expect(popup).toMatch(/`Confirm delete \(\$\{selected\.size\}\)`/);
+      // updateControls (run on every selection change) resets the armed
+      // state and the button label.
+      expect(popup).toMatch(
+        /const updateControls = \(\) => \{\s*confirmArmed = false/
+      );
+    });
+
+    it('delete posts to delete-branches and resyncs the panel', () => {
+      const del = (widgetSrc.match(
+        /private async _deleteBranches[\s\S]*?\n  \}/
+      ) ?? [''])[0];
+      expect(del).toMatch(/sessions\/delete-branches/);
+      expect(del).toMatch(/finally[\s\S]*?await this\._fetch\(\)/);
+    });
+
+    it('rows carry age emphasis classes at the 60s and 7d thresholds', () => {
+      expect(widgetSrc).toMatch(/age < 60_000[\s\S]*?jp-mod-recentlyActive/);
+      expect(widgetSrc).toMatch(/age > 7 \* 86_400_000[\s\S]*?jp-mod-stale/);
+      const css: string = fs.readFileSync(
+        path.join(__dirname, '..', '..', 'style', 'base.css'),
+        'utf-8'
+      );
+      expect(css).toMatch(/jp-mod-recentlyActive[\s\S]*?--jp-brand-color1/);
+      expect(css).toMatch(/jp-mod-stale \{\s*opacity/);
+    });
+
     it('opens the menu without the submenu when the fetch fails', () => {
       expect(openMenu).toMatch(/catch[\s\S]*?hasBranches = false/);
     });
@@ -218,6 +298,87 @@ describe('launch spinner dismiss contract', () => {
         /result\.current !== result\.requested[\s\S]*?Notification\.warning/
       );
     });
+
+    it('renders the favourite star before the time column', () => {
+      const renderRow = (widgetSrc.match(
+        /private _renderRow[\s\S]*?\n  \}/
+      ) ?? [''])[0];
+      const starAt = renderRow.indexOf('jp-ClaudeSessionsPanel-favStar');
+      const timeAt = renderRow.indexOf('jp-ClaudeSessionsPanel-rowTime');
+      expect(starAt).toBeGreaterThan(-1);
+      expect(timeAt).toBeGreaterThan(-1);
+      expect(starAt).toBeLessThan(timeAt);
+    });
+
+    it('time labels form fixed-width right-aligned columns', () => {
+      const css: string = fs.readFileSync(
+        path.join(__dirname, '..', '..', 'style', 'base.css'),
+        'utf-8'
+      );
+      const rowTime = (css.match(
+        /\.jp-ClaudeSessionsPanel-rowTime \{[\s\S]*?\}/
+      ) ?? [''])[0];
+      expect(rowTime).toMatch(/width: 52px/);
+      expect(rowTime).toMatch(/text-align: right/);
+      const branchTime = (css.match(
+        /\.jp-ClaudeSessionsPanel-branchTime \{[\s\S]*?\}/
+      ) ?? [''])[0];
+      expect(branchTime).toMatch(/width: 52px/);
+      expect(branchTime).toMatch(/text-align: right/);
+    });
+
+    it('now label shares the recently-active emphasis colour', () => {
+      const css: string = fs.readFileSync(
+        path.join(__dirname, '..', '..', 'style', 'base.css'),
+        'utf-8'
+      );
+      expect(css).toMatch(
+        /jp-mod-recentlyActive[\s\S]{0,200}?jp-ClaudeSessionsPanel-rowTime[\s\S]{0,80}?--jp-brand-color1/
+      );
+    });
+
+    it('branch session commands exist in normal and skip-permissions modes', () => {
+      expect(widgetSrc).toMatch(/claude-code-sessions:branch-session'/);
+      expect(widgetSrc).toMatch(
+        /claude-code-sessions:branch-session-dangerous/
+      );
+      expect(widgetSrc).toMatch(
+        /Branch Session \(Skip Permissions\)\.\.\.[\s\S]{0,80}?icon: shieldIcon/
+      );
+    });
+
+    it('branch session asks for a name and launches a known fork id', () => {
+      const branch = (widgetSrc.match(
+        /private async _branchSession[\s\S]*?\n  \}/
+      ) ?? [''])[0];
+      expect(branch).toMatch(/InputDialog\.getText/);
+      expect(branch).toMatch(/UUID\.uuid4\(\)/);
+      expect(branch).toMatch(/fork_session_id: forkId/);
+      expect(branch).toMatch(/session_id: session\.session_id/);
+      expect(branch).toMatch(/_stampForkTitle/);
+    });
+
+    it('fork title stamping retries on 404 until the JSONL appears', () => {
+      const stamp = (widgetSrc.match(
+        /private async _stampForkTitle[\s\S]*?\n  \}/
+      ) ?? [''])[0];
+      expect(stamp).toMatch(/sessions\/set-title/);
+      expect(stamp).toMatch(/status === 404/);
+      expect(stamp).toMatch(/await this\._fetch\(\)/);
+      expect(stamp).toMatch(/Notification\.warning/);
+    });
+
+    it('live dot is softened with reduced opacity', () => {
+      const css: string = fs.readFileSync(
+        path.join(__dirname, '..', '..', 'style', 'base.css'),
+        'utf-8'
+      );
+      const dot = (css.match(/\.jp-ClaudeSessionsPanel-dot \{[\s\S]*?\}/) ?? [
+        ''
+      ])[0];
+      expect(dot).toMatch(/--jp-success-color1/);
+      expect(dot).toMatch(/opacity: 0\.75/);
+    });
   });
 
   it('_doResumeInTerminal dismisses spinner via dispose(), not resolve()', () => {

package/src/icons.ts

@@ -71,6 +71,17 @@ export const addIcon = new LabIcon({
   svgstr: addSvgStr
 });
 
+// Git-branch glyph (Octicons git-branch-16, MIT) - marks rows that carry
+// parallel conversations and the Branch Session menu entries.
+const branchSvgStr = `<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 16 16" width="16" height="16">
+  <path class="jp-icon3" fill="#616161" d="M9.5 3.25a2.25 2.25 0 1 1 3 2.122V6A2.5 2.5 0 0 1 10 8.5H6a1 1 0 0 0-1 1v1.128a2.251 2.251 0 1 1-1.5 0V5.372a2.25 2.25 0 1 1 1.5 0v1.836A2.493 2.493 0 0 1 6 7h4a1 1 0 0 0 1-1v-.628a2.25 2.25 0 0 1-1.5-2.122Zm-6 0a.75.75 0 1 0 1.5 0 .75.75 0 0 0-1.5 0Zm8.25-.75a.75.75 0 1 0 0 1.5.75.75 0 0 0 0-1.5ZM4.25 12a.75.75 0 1 0 0 1.5.75.75 0 0 0 0-1.5Z"/>
+</svg>`;
+
+export const branchIcon = new LabIcon({
+  name: 'jupyterlab_claude_code_extension:branch',
+  svgstr: branchSvgStr
+});
+
 // Funnel copied verbatim from @jupyterlab/ui-components'
 // `search/filter.svg` - the same image the file browser's filter
 // toggle uses. The `class="jp-icon3"` lets JupyterLab's theme drive

package/src/types.ts

@@ -73,6 +73,15 @@ export interface ISwitchResponse {
   current: string | null;
 }
 
+export interface IDeleteBranchesRequest {
+  encoded_path: string;
+  session_ids: string[];
+}
+
+export interface IDeleteBranchesResponse {
+  removed_count: number;
+}
+
 export interface ILaunchTerminalRequest {
   project_path: string;
   /** Omit to start a brand-new claude session instead of resuming one. */