jupyterlab_ai_code_assistants_extension 1.0.19 → 1.0.21

package/lib/core/icons.d.ts

@@ -3,6 +3,7 @@ export declare const assistantsIcon: LabIcon;
 export declare const starFilledIcon: LabIcon;
 export declare const refreshIcon: LabIcon;
 export declare const removeIcon: LabIcon;
+export declare const cleanupIcon: LabIcon;
 export declare const shieldIcon: LabIcon;
 export declare const addIcon: LabIcon;
 export declare const branchIcon: LabIcon;

package/lib/core/icons.js

@@ -37,19 +37,28 @@ export const removeIcon = new LabIcon({
     name: `${ICON_PREFIX}:remove`,
     svgstr: removeSvgStr
 });
+// `Clean Up Parallel Sessions` - deletes several at once, so it wears the
+// sweep variant of the trash glyph (Material "delete_sweep"), same family as
+// `Remove from ...`'s plain trash but visibly the bulk one (user direction
+// 2026-08-11, replacing the shared danger marker it wore under DEF-45).
+const cleanupSvgStr = `<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24" width="16" height="16">
+  <path class="jp-icon3" fill="#616161" d="M15 16h4v2h-4zm0-8h7v2h-7zm0 4h6v2h-6zM3 18c0 1.1.9 2 2 2h6c1.1 0 2-.9 2-2V8H3v10zM14 5h-3l-1-1H6L5 5H2v2h12z"/>
+</svg>`;
+export const cleanupIcon = new LabIcon({
+    name: `${ICON_PREFIX}:cleanup`,
+    svgstr: cleanupSvgStr
+});
 // Marks the unsafe launch variants. The shield is the established marker for
 // an action that touches the permission system (the UAC pattern), and it is
 // what the reference implementation used; Material "shield" half-fill, kept
 // per user direction 2026-08-11 over the earlier caution triangle.
 //
-// 16x16 and `jp-icon-warn0`, like every other icon in these menus and unlike
-// them in colour: authored at 13x13 in `jp-icon3` it rendered smaller than
-// its neutral siblings AND in the identical grey, measured rgb(97,97,97) in
-// the light theme and rgb(189,189,189) in the dark one - the one glyph whose
-// whole job is to say "this differs" was the least conspicuous in the menu
-// (DEF-40, and the reason the reference's 13x13 grey is not copied verbatim).
+// 16x16 like its siblings (DEF-40's size half - authored at 13x13 it
+// rendered smaller than every icon beside it), and in the standard `jp-icon3`
+// like them too: the user ruled 2026-08-11 that the shield shape alone marks
+// the difference, reversing DEF-40's colour half (the orange `jp-icon-warn0`).
 const shieldSvgStr = `<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24" width="16" height="16">
-  <path class="jp-icon-warn0" fill="#f57c00" d="M12 1L3 5v6c0 5.55 3.84 10.74 9 12 5.16-1.26 9-6.45 9-12V5l-9-4zm0 10.99h7c-.53 4.12-3.28 7.79-7 8.94V12H5V6.3l7-3.11v8.8z"/>
+  <path class="jp-icon3" fill="#616161" d="M12 1L3 5v6c0 5.55 3.84 10.74 9 12 5.16-1.26 9-6.45 9-12V5l-9-4zm0 10.99h7c-.53 4.12-3.28 7.79-7 8.94V12H5V6.3l7-3.11v8.8z"/>
 </svg>`;
 export const shieldIcon = new LabIcon({
     name: `${ICON_PREFIX}:shield`,

package/lib/core/panel.js

@@ -4,7 +4,7 @@ import { CommandRegistry } from '@lumino/commands';
 import { UUID } from '@lumino/coreutils';
 import { Widget } from '@lumino/widgets';
 import { ColourStore } from './colour';
-import { addIcon, branchIcon, filterIcon, providerIcon, refreshIcon, removeIcon, shieldIcon, starFilledIcon, switchIcon } from './icons';
+import { addIcon, branchIcon, cleanupIcon, filterIcon, providerIcon, refreshIcon, removeIcon, shieldIcon, starFilledIcon, switchIcon } from './icons';
 import { DEFAULT_RECENT_LIMIT, MAX_RECENT_LIMIT, MIN_RECENT_LIMIT } from './limits';
 import { resolveLaunchMode, resolvedLaunchModeEntry } from './modes';
 import { showManageSessionsPopup } from './popup';
@@ -1839,8 +1839,9 @@ export class AssistantSessionsPanel extends Widget {
             // Marked, like `Remove from ...` below it. The two share the group the
             // separator was added to protect, and both delete history that cannot be
             // recovered; leaving this one bare made the pair read as one dangerous
-            // item beside an ordinary one (DEF-45).
-            icon: shieldIcon,
+            // item beside an ordinary one (DEF-45). The sweep variant of the trash
+            // glyph, because this one deletes several at once.
+            icon: cleanupIcon,
             isVisible: () => { var _a, _b; return ((_b = (_a = active()) === null || _a === void 0 ? void 0 : _a.extra_sessions) !== null && _b !== void 0 ? _b : 0) > 0; },
             execute: () => {
                 const s = active();

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "jupyterlab_ai_code_assistants_extension",
-    "version": "1.0.19",
+    "version": "1.0.21",
     "description": "Jupyterlab extension to replace individual jupyterlab_*_code_extension and integrate into one, that offers joint capability, joint settings page and individual right toolbar panels per code assistant, with the ability in settings to decide which code assistant support is on (default - all are enabled)",
     "keywords": [
         "jupyter",

package/src/__tests__/panel.spec.ts

@@ -42,7 +42,7 @@ jest.mock('../core/request', () => ({
 }));
 
 import { TAB_COLOUR_IDS, fnv1aColour } from '../core/colour';
-import { addIcon, branchIcon, shieldIcon } from '../core/icons';
+import { addIcon, branchIcon, cleanupIcon, shieldIcon } from '../core/icons';
 import { AssistantSessionsPanel, commandId } from '../core/panel';
 import { requestProvider } from '../core/request';
 import { IProviderDescriptor, ISession } from '../core/types';
@@ -346,7 +346,7 @@ describe('DEF-35 / DEF-39 - the branch entry with no visible variants', () => {
 
 // --------------------------------------------------------------- DEF-40
 
-describe('DEF-40 - the warning glyph is sized and coloured to stand out', () => {
+describe('DEF-40 - the shield glyph matches its siblings in size and colour', () => {
   const svg = (svgstr: string): SVGElement =>
     new DOMParser().parseFromString(svgstr, 'image/svg+xml')
       .documentElement as unknown as SVGElement;
@@ -364,23 +364,27 @@ describe('DEF-40 - the warning glyph is sized and coloured to stand out', () =>
     }
   });
 
-  it('paints in jp-icon-warn0 and not the neutral grey', () => {
-    // `jp-icon3` is the theme's neutral: in it the glyph measured the same
-    // rgb(97,97,97) as the icons it was supposed to differ from.
+  it('paints in the standard jp-icon3, like its siblings', () => {
+    // DEF-40's orange (`jp-icon-warn0`) was reversed by user ruling
+    // 2026-08-11: the shield SHAPE alone marks the difference, the colour
+    // stays the theme's neutral like every other icon in the menu.
     const path = svg(shieldIcon.svgstr).querySelector('path')!;
-    expect(path.getAttribute('class')).toEqual('jp-icon-warn0');
-    expect(shieldIcon.svgstr).not.toContain('jp-icon3');
+    expect(path.getAttribute('class')).toEqual('jp-icon3');
+    expect(shieldIcon.svgstr).not.toContain('jp-icon-warn0');
   });
 });
 
 // ------------------------------------------------- DEF-45 / DEF-46 (lane A)
 
 describe('DEF-45 - both destructive menu entries are marked alike', () => {
-  it('gives Clean Up Parallel Sessions the same glyph Remove carries', () => {
+  it('gives Clean Up Parallel Sessions the sweep variant of the trash glyph', () => {
+    // Same family as Remove's plain trash - marked alike, per DEF-45 - but
+    // the bulk variant, because this one deletes several at once (user
+    // direction 2026-08-11; it no longer shares the unsafe-launch shield).
     const commands = (panel as any)._commands;
     expect(
       commands.icon(commandId(DESCRIPTOR.id, 'cleanup-parallel'), {})
-    ).toBe(shieldIcon);
+    ).toBe(cleanupIcon);
     expect(commands.icon(commandId(DESCRIPTOR.id, 'remove'), {})).toBeDefined();
   });
 });

package/src/core/icons.ts

@@ -48,19 +48,30 @@ export const removeIcon = new LabIcon({
   svgstr: removeSvgStr
 });
 
+// `Clean Up Parallel Sessions` - deletes several at once, so it wears the
+// sweep variant of the trash glyph (Material "delete_sweep"), same family as
+// `Remove from ...`'s plain trash but visibly the bulk one (user direction
+// 2026-08-11, replacing the shared danger marker it wore under DEF-45).
+const cleanupSvgStr = `<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24" width="16" height="16">
+  <path class="jp-icon3" fill="#616161" d="M15 16h4v2h-4zm0-8h7v2h-7zm0 4h6v2h-6zM3 18c0 1.1.9 2 2 2h6c1.1 0 2-.9 2-2V8H3v10zM14 5h-3l-1-1H6L5 5H2v2h12z"/>
+</svg>`;
+
+export const cleanupIcon = new LabIcon({
+  name: `${ICON_PREFIX}:cleanup`,
+  svgstr: cleanupSvgStr
+});
+
 // Marks the unsafe launch variants. The shield is the established marker for
 // an action that touches the permission system (the UAC pattern), and it is
 // what the reference implementation used; Material "shield" half-fill, kept
 // per user direction 2026-08-11 over the earlier caution triangle.
 //
-// 16x16 and `jp-icon-warn0`, like every other icon in these menus and unlike
-// them in colour: authored at 13x13 in `jp-icon3` it rendered smaller than
-// its neutral siblings AND in the identical grey, measured rgb(97,97,97) in
-// the light theme and rgb(189,189,189) in the dark one - the one glyph whose
-// whole job is to say "this differs" was the least conspicuous in the menu
-// (DEF-40, and the reason the reference's 13x13 grey is not copied verbatim).
+// 16x16 like its siblings (DEF-40's size half - authored at 13x13 it
+// rendered smaller than every icon beside it), and in the standard `jp-icon3`
+// like them too: the user ruled 2026-08-11 that the shield shape alone marks
+// the difference, reversing DEF-40's colour half (the orange `jp-icon-warn0`).
 const shieldSvgStr = `<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24" width="16" height="16">
-  <path class="jp-icon-warn0" fill="#f57c00" d="M12 1L3 5v6c0 5.55 3.84 10.74 9 12 5.16-1.26 9-6.45 9-12V5l-9-4zm0 10.99h7c-.53 4.12-3.28 7.79-7 8.94V12H5V6.3l7-3.11v8.8z"/>
+  <path class="jp-icon3" fill="#616161" d="M12 1L3 5v6c0 5.55 3.84 10.74 9 12 5.16-1.26 9-6.45 9-12V5l-9-4zm0 10.99h7c-.53 4.12-3.28 7.79-7 8.94V12H5V6.3l7-3.11v8.8z"/>
 </svg>`;
 
 export const shieldIcon = new LabIcon({

package/src/core/panel.ts

@@ -26,6 +26,7 @@ import { ColourStore } from './colour';
 import {
   addIcon,
   branchIcon,
+  cleanupIcon,
   filterIcon,
   providerIcon,
   refreshIcon,
@@ -2170,8 +2171,9 @@ export class AssistantSessionsPanel extends Widget {
       // Marked, like `Remove from ...` below it. The two share the group the
       // separator was added to protect, and both delete history that cannot be
       // recovered; leaving this one bare made the pair read as one dangerous
-      // item beside an ordinary one (DEF-45).
-      icon: shieldIcon,
+      // item beside an ordinary one (DEF-45). The sweep variant of the trash
+      // glyph, because this one deletes several at once.
+      icon: cleanupIcon,
       isVisible: () => (active()?.extra_sessions ?? 0) > 0,
       execute: () => {
         const s = active();