jupyterlab_ai_code_assistants_extension 1.0.17 → 1.0.21

package/lib/core/icons.d.ts

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

package/lib/core/icons.js

@@ -37,21 +37,32 @@ export const removeIcon = new LabIcon({
     name: `${ICON_PREFIX}:remove`,
     svgstr: removeSvgStr
 });
-// Marks the unsafe launch variants. A shield reads as "protected", which is
-// the inverse of what these entries do, so the caution triangle carries them
-// instead. Material "warning" outline.
+// `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.
-const warningSvgStr = `<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="M1 21h22L12 2 1 21zm12-3h-2v-2h2v2zm0-4h-2v-4h2v4z"/>
+// 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-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 warningIcon = new LabIcon({
-    name: `${ICON_PREFIX}:warning`,
-    svgstr: warningSvgStr
+export const shieldIcon = new LabIcon({
+    name: `${ICON_PREFIX}:shield`,
+    svgstr: shieldSvgStr
 });
 const addSvgStr = `<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24" width="16" height="16">
   <path class="jp-icon3" fill="#616161" d="M19 13h-6v6h-2v-6H5v-2h6V5h2v6h6v2z"/>

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, starFilledIcon, switchIcon, warningIcon } 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';
@@ -1691,7 +1691,7 @@ export class AssistantSessionsPanel extends Widget {
     /** The glyph, only where the mode WIDENS what runs without asking. */
     _variantIcon() {
         var _a;
-        return ((_a = this._resolvedVariant()) === null || _a === void 0 ? void 0 : _a.unsafe) ? warningIcon : undefined;
+        return ((_a = this._resolvedVariant()) === null || _a === void 0 ? void 0 : _a.unsafe) ? shieldIcon : undefined;
     }
     _setupCommands() {
         this._commands = new CommandRegistry();
@@ -1716,7 +1716,7 @@ export class AssistantSessionsPanel extends Widget {
                 const verb = (_c = (_b = (_a = this._hooks).resumeLabel) === null || _b === void 0 ? void 0 : _b.call(_a, active())) !== null && _c !== void 0 ? _c : 'Resume';
                 return variant ? `${verb} (${variant.label})` : verb;
             },
-            icon: () => { var _a; return (((_a = this._resumeVariant()) === null || _a === void 0 ? void 0 : _a.unsafe) ? warningIcon : undefined); },
+            icon: () => { var _a; return (((_a = this._resumeVariant()) === null || _a === void 0 ? void 0 : _a.unsafe) ? shieldIcon : undefined); },
             execute: () => {
                 const s = active();
                 if (s) {
@@ -1727,7 +1727,7 @@ export class AssistantSessionsPanel extends Widget {
         for (const mode of this._variantModes) {
             this._commands.addCommand(this._cmd(`resume-${mode.id}`), {
                 label: `Resume (${mode.menuLabel})`,
-                icon: warningIcon,
+                icon: shieldIcon,
                 // Absent while the plain item already resolves to this same mode - it
                 // would be a second entry building an identical launch.
                 isVisible: () => !this._buildsSameLaunch(mode),
@@ -1744,13 +1744,13 @@ export class AssistantSessionsPanel extends Widget {
             });
             this._commands.addCommand(this._cmd(`new-session-${mode.id}`), {
                 label: `New ${this._descriptor.label} Session (${mode.menuLabel})`,
-                icon: warningIcon,
+                icon: shieldIcon,
                 isVisible: () => !this._buildsSameLaunch(mode),
                 execute: () => void this._newSession(mode.id)
             });
             this._commands.addCommand(this._cmd(`branch-session-${mode.id}`), {
                 label: mode.menuLabel,
-                icon: warningIcon,
+                icon: shieldIcon,
                 isVisible: () => !this._buildsSameLaunch(mode),
                 execute: () => void this._branchSession(mode.id)
             });
@@ -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: warningIcon,
+            // 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();
@@ -1909,7 +1910,7 @@ export class AssistantSessionsPanel extends Widget {
             // danger everywhere else in this menu - spending it on a branch glyph
             // left `Branch Session (Skip Permissions)` looking exactly as safe as
             // Show in File Browser while `Resume (Skip Permissions)` beside it wore
-            // the triangle. The branch glyph is the fallback, not the winner.
+            // the shield. The branch glyph is the fallback, not the winner.
             icon: () => {
                 var _a;
                 return (_a = this._variantIcon()) !== null && _a !== void 0 ? _a : (this._visibleVariantCount() === 0 ? branchIcon : undefined);

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "jupyterlab_ai_code_assistants_extension",
-    "version": "1.0.17",
+    "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, warningIcon } 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';
@@ -284,7 +284,7 @@ describe('the + button names and marks the launch it performs', () => {
     expect(glyphs()).toEqual([addIcon.name]);
     panel.setModes({ skip: true });
     expect((panel as any)._visibleVariantCount()).toEqual(0);
-    expect(glyphs()).toEqual([warningIcon.name]);
+    expect(glyphs()).toEqual([shieldIcon.name]);
     // `LabIcon.element` appends, so a button that is not cleared collects one
     // svg per settings change.
     panel.setModes({ skip: false });
@@ -314,7 +314,7 @@ describe('DEF-35 / DEF-39 - the branch entry with no visible variants', () => {
     panel.setModes({ skip: true });
     // The suffix names the mode in words, but the icon slot is what marks
     // danger everywhere else in this menu.
-    expect(cmds(panel).icon(id(panel, 'branch-session'), {})).toBe(warningIcon);
+    expect(cmds(panel).icon(id(panel, 'branch-session'), {})).toBe(shieldIcon);
   });
 
   it('goes back to naming the launch alone while the variants render', () => {
@@ -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;
@@ -354,7 +354,7 @@ describe('DEF-40 - the warning glyph is sized and coloured to stand out', () =>
   it('renders at 16x16, like its neutral siblings', () => {
     // Authored at 13x13 it rendered SMALLER than every icon beside it - the one
     // glyph whose job is to say "this differs" was also the least visible.
-    for (const icon of [warningIcon, addIcon, branchIcon]) {
+    for (const icon of [shieldIcon, addIcon, branchIcon]) {
       const el = svg(icon.svgstr);
       expect([
         icon.name,
@@ -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 triangle measured the same
-    // rgb(97,97,97) as the icons it was supposed to differ from.
-    const path = svg(warningIcon.svgstr).querySelector('path')!;
-    expect(path.getAttribute('class')).toEqual('jp-icon-warn0');
-    expect(warningIcon.svgstr).not.toContain('jp-icon3');
+  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-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(warningIcon);
+    ).toBe(cleanupIcon);
     expect(commands.icon(commandId(DESCRIPTOR.id, 'remove'), {})).toBeDefined();
   });
 });

package/src/core/icons.ts

@@ -48,22 +48,35 @@ export const removeIcon = new LabIcon({
   svgstr: removeSvgStr
 });
 
-// Marks the unsafe launch variants. A shield reads as "protected", which is
-// the inverse of what these entries do, so the caution triangle carries them
-// instead. Material "warning" outline.
+// `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.
-const warningSvgStr = `<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="M1 21h22L12 2 1 21zm12-3h-2v-2h2v2zm0-4h-2v-4h2v4z"/>
+// 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-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 warningIcon = new LabIcon({
-  name: `${ICON_PREFIX}:warning`,
-  svgstr: warningSvgStr
+export const shieldIcon = new LabIcon({
+  name: `${ICON_PREFIX}:shield`,
+  svgstr: shieldSvgStr
 });
 
 const addSvgStr = `<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 24 24" width="16" height="16">

package/src/core/panel.ts

@@ -26,13 +26,14 @@ import { ColourStore } from './colour';
 import {
   addIcon,
   branchIcon,
+  cleanupIcon,
   filterIcon,
   providerIcon,
   refreshIcon,
   removeIcon,
+  shieldIcon,
   starFilledIcon,
-  switchIcon,
-  warningIcon
+  switchIcon
 } from './icons';
 import {
   DEFAULT_RECENT_LIMIT,
@@ -1996,7 +1997,7 @@ export class AssistantSessionsPanel extends Widget {
 
   /** The glyph, only where the mode WIDENS what runs without asking. */
   private _variantIcon(): LabIcon | undefined {
-    return this._resolvedVariant()?.unsafe ? warningIcon : undefined;
+    return this._resolvedVariant()?.unsafe ? shieldIcon : undefined;
   }
 
   private _setupCommands(): void {
@@ -2024,7 +2025,7 @@ export class AssistantSessionsPanel extends Widget {
         const verb = this._hooks.resumeLabel?.(active()) ?? 'Resume';
         return variant ? `${verb} (${variant.label})` : verb;
       },
-      icon: () => (this._resumeVariant()?.unsafe ? warningIcon : undefined),
+      icon: () => (this._resumeVariant()?.unsafe ? shieldIcon : undefined),
       execute: () => {
         const s = active();
         if (s) {
@@ -2036,7 +2037,7 @@ export class AssistantSessionsPanel extends Widget {
     for (const mode of this._variantModes) {
       this._commands.addCommand(this._cmd(`resume-${mode.id}`), {
         label: `Resume (${mode.menuLabel})`,
-        icon: warningIcon,
+        icon: shieldIcon,
         // Absent while the plain item already resolves to this same mode - it
         // would be a second entry building an identical launch.
         isVisible: () => !this._buildsSameLaunch(mode),
@@ -2053,13 +2054,13 @@ export class AssistantSessionsPanel extends Widget {
       });
       this._commands.addCommand(this._cmd(`new-session-${mode.id}`), {
         label: `New ${this._descriptor.label} Session (${mode.menuLabel})`,
-        icon: warningIcon,
+        icon: shieldIcon,
         isVisible: () => !this._buildsSameLaunch(mode),
         execute: () => void this._newSession(mode.id)
       });
       this._commands.addCommand(this._cmd(`branch-session-${mode.id}`), {
         label: mode.menuLabel as string,
-        icon: warningIcon,
+        icon: shieldIcon,
         isVisible: () => !this._buildsSameLaunch(mode),
         execute: () => void this._branchSession(mode.id)
       });
@@ -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: warningIcon,
+      // 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();
@@ -2244,7 +2246,7 @@ export class AssistantSessionsPanel extends Widget {
       // danger everywhere else in this menu - spending it on a branch glyph
       // left `Branch Session (Skip Permissions)` looking exactly as safe as
       // Show in File Browser while `Resume (Skip Permissions)` beside it wore
-      // the triangle. The branch glyph is the fallback, not the winner.
+      // the shield. The branch glyph is the fallback, not the winner.
       icon: () =>
         this._variantIcon() ??
         (this._visibleVariantCount() === 0 ? branchIcon : undefined),