jupyterlab_ai_code_assistants_extension 1.0.17 → 1.0.19

package/lib/core/icons.d.ts

@@ -3,7 +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 warningIcon: 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,23 @@ 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.
+// 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"/>
+// 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).
+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"/>
 </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, 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)
             });
@@ -1840,7 +1840,7 @@ export class AssistantSessionsPanel extends Widget {
             // 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,
+            icon: shieldIcon,
             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 +1909,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.19",
     "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, 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', () => {
@@ -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,
@@ -365,11 +365,11 @@ 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
+    // `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.
-    const path = svg(warningIcon.svgstr).querySelector('path')!;
+    const path = svg(shieldIcon.svgstr).querySelector('path')!;
     expect(path.getAttribute('class')).toEqual('jp-icon-warn0');
-    expect(warningIcon.svgstr).not.toContain('jp-icon3');
+    expect(shieldIcon.svgstr).not.toContain('jp-icon3');
   });
 });
 
@@ -380,7 +380,7 @@ describe('DEF-45 - both destructive menu entries are marked alike', () => {
     const commands = (panel as any)._commands;
     expect(
       commands.icon(commandId(DESCRIPTOR.id, 'cleanup-parallel'), {})
-    ).toBe(warningIcon);
+    ).toBe(shieldIcon);
     expect(commands.icon(commandId(DESCRIPTOR.id, 'remove'), {})).toBeDefined();
   });
 });

package/src/core/icons.ts

@@ -48,22 +48,24 @@ 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.
+// 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"/>
+// 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).
+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"/>
 </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

@@ -30,9 +30,9 @@ import {
   providerIcon,
   refreshIcon,
   removeIcon,
+  shieldIcon,
   starFilledIcon,
-  switchIcon,
-  warningIcon
+  switchIcon
 } from './icons';
 import {
   DEFAULT_RECENT_LIMIT,
@@ -1996,7 +1996,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 +2024,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 +2036,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 +2053,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)
       });
@@ -2171,7 +2171,7 @@ export class AssistantSessionsPanel extends Widget {
       // 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,
+      icon: shieldIcon,
       isVisible: () => (active()?.extra_sessions ?? 0) > 0,
       execute: () => {
         const s = active();
@@ -2244,7 +2244,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),