cli-jaw 2.17.41 → 2.17.42

package/public/manager/src/components/Workbench.tsx

@@ -9,9 +9,6 @@ type WorkbenchProps = {
     overview: ReactNode;
     preview: ReactNode;
     logs: ReactNode;
-    settings: ReactNode;
-    settingsOpen: boolean;
-    onSettingsClose: () => void;
     active: boolean;
 };
 
@@ -23,8 +20,8 @@ function modeLabel(mode: DashboardDetailTab): string {
 
 export function Workbench(props: WorkbenchProps) {
     return (
-        <section className={`workbench workbench-${props.mode}`} data-instance-settings-open={props.settingsOpen} aria-label="Selected instance workbench">
-            <div className="workbench-header" hidden={props.settingsOpen}>
+        <section className={`workbench workbench-${props.mode}`} aria-label="Selected instance workbench">
+            <div className="workbench-header">
                 {props.header}
                 <div className="workbench-mode-bar">
                     <div className="workbench-mode-tabs" role="tablist" aria-label="Workbench modes"
@@ -56,7 +53,7 @@ export function Workbench(props: WorkbenchProps) {
                 </div>
             </div>
             <div className="workbench-body">
-                {!props.settingsOpen && props.mode === 'overview' && (
+                {props.mode === 'overview' && (
                     <div key="overview" id="workbench-panel-overview" role="tabpanel" aria-labelledby="workbench-tab-overview" className="workbench-panel workbench-panel-overview">{props.overview}</div>
                 )}
                 <div
@@ -65,16 +62,15 @@ export function Workbench(props: WorkbenchProps) {
                     role="tabpanel"
                     aria-labelledby="workbench-tab-preview"
                     className="workbench-panel workbench-panel-preview"
-                    hidden={props.settingsOpen || props.mode !== 'preview'}
-                    aria-hidden={props.settingsOpen || props.mode !== 'preview'}
+                    hidden={props.mode !== 'preview'}
+                    aria-hidden={props.mode !== 'preview'}
                     data-preview-host="persistent"
                 >
                     {props.preview}
                 </div>
-                {!props.settingsOpen && props.mode === 'logs' && (
+                {props.mode === 'logs' && (
                     <div key="logs" id="workbench-panel-logs" role="tabpanel" aria-labelledby="workbench-tab-logs" className="workbench-panel workbench-panel-logs">{props.logs}</div>
                 )}
-                {props.settingsOpen && <div id="workbench-instance-settings" className="workbench-settings-page">{props.settings}</div>}
             </div>
         </section>
     );

package/public/manager/src/components/WorkbenchHeader.tsx

@@ -107,14 +107,3 @@ export function WorkbenchHeader(props: WorkbenchHeaderProps) {
         </div>
     );
 }
-
-export function WorkbenchSettingsToggle(props: { open: boolean; onToggle: () => void }) {
-    const [slot, setSlot] = useState<HTMLElement | null>(null);
-    useEffect(() => { setSlot(document.getElementById('command-settings-slot')); }, []);
-    if (!slot) return null;
-    return createPortal(<button type="button" className="workbench-settings-toggle" aria-label="Instance settings"
-        title="Instance settings" aria-pressed={props.open} aria-controls="workbench-instance-settings"
-        onClick={props.onToggle}>
-        <span aria-hidden="true" dangerouslySetInnerHTML={{ __html: icon('settings', 16) }} />
-    </button>, slot);
-}

package/public/manager/src/dashboard-settings/dashboard-settings-ui.ts

@@ -11,7 +11,9 @@ export function dashboardSettingsUiFromView(
     return {
         selectedPort: view.selectedPort,
         selectedTab: view.activeDetailTab,
-        instanceSettingsOpen: view.instanceSettingsOpen,
+        // Retained in the schema for one version so older clients and PATCH callers do not
+        // 400, but the rail owns settings now and nothing opens a Workbench panel.
+        instanceSettingsOpen: false,
         sidebarCollapsed: view.sidebarCollapsed,
         activityDockCollapsed: view.activityDockCollapsed,
         activityDockHeight: view.activityDockHeight,

package/public/manager/src/hooks/useDashboardView.ts

@@ -6,7 +6,6 @@ import type { DashboardDetailTab, DashboardDiffMode, DashboardDiffRootPolicy, Da
 export function useDashboardView() {
     const [selectedPort, setSelectedPort] = useState<number | null>(null);
     const [activeDetailTab, setActiveDetailTab] = useState<DashboardDetailTab>('overview');
-    const [instanceSettingsOpen, setInstanceSettingsOpen] = useState(false);
     const [drawerOpen, setDrawerOpen] = useState(false);
     const [sidebarCollapsed, setSidebarCollapsed] = useState(false);
     const [activityDockCollapsed, setActivityDockCollapsed] = useState(false);
@@ -42,7 +41,6 @@ export function useDashboardView() {
     return {
         selectedPort,
         setSelectedPort,
-        instanceSettingsOpen, setInstanceSettingsOpen,
         activeDetailTab,
         setActiveDetailTab,
         drawerOpen,
@@ -102,19 +100,26 @@ export function useDashboardView() {
     };
 }
 
-export function hydrateInstanceSettings(ui: Pick<DashboardRegistryUi, 'selectedTab'> & { instanceSettingsOpen?: unknown }) {
+/**
+ * The in-Workbench instance settings panel is gone; the rail owns settings now. A registry
+ * written by an older client can still carry `instanceSettingsOpen: true` or the retired
+ * `selectedTab: 'settings'`, so map either onto the rail's sidebar mode rather than dropping
+ * the user on a workspace whose settings tab no longer exists. The field stays accepted by
+ * the server normalizer for one version; nothing writes it any more.
+ */
+export function hydrateInstanceSettings(ui: Pick<DashboardRegistryUi, 'selectedTab' | 'sidebarMode'> & { instanceSettingsOpen?: unknown }) {
+    const legacySettingsRequest = ui.selectedTab === 'settings' || ui.instanceSettingsOpen === true;
     return { selectedTab: ui.selectedTab === 'settings' ? 'overview' as const : ui.selectedTab,
-        instanceSettingsOpen: ui.selectedTab === 'settings' || ui.instanceSettingsOpen === true };
+        sidebarMode: legacySettingsRequest ? 'settings' as const : ui.sidebarMode };
 }
 
 type SettingsNavigation = {
-    view: Pick<ReturnType<typeof useDashboardView>, 'instanceSettingsOpen' | 'activeDetailTab' | 'sidebarMode' |
-        'setSelectedPort' | 'setInstanceSettingsOpen' | 'setSidebarMode' | 'setViewMode' | 'setDrawerOpen'>;
+    view: Pick<ReturnType<typeof useDashboardView>, 'activeDetailTab' | 'sidebarMode' |
+        'setSelectedPort' | 'setSidebarMode' | 'setViewMode' | 'setDrawerOpen'>;
     selectedPort: number | null;
     settingsDirty: boolean;
-    panelSettingsDirty: boolean;
     dashboardSettingsDirty: boolean;
-    clearDirty: (entry: 'panel' | 'dashboard') => void;
+    clearDirty: (entry: 'dashboard') => void;
     saveUi: (patch: Partial<DashboardRegistryUi>) => Promise<void>;
     confirmDiscard?: () => boolean;
 };
@@ -125,49 +130,44 @@ export function createInstanceSettingsNavigation(args: SettingsNavigation) {
             (args.confirmDiscard ?? (() => window.confirm('Discard unsaved Settings changes?')))();
     }
     function guardSettingsTransition(target: Partial<{
-        sidebarMode: DashboardSidebarMode; selectedPort: number | null; instanceSettingsOpen: boolean;
+        sidebarMode: DashboardSidebarMode; selectedPort: number | null;
     }>): boolean {
         const portChanged = target.selectedPort !== undefined && target.selectedPort !== selectedPort;
         const leavingDashboard = view.sidebarMode === 'settings' && (target.sidebarMode ?? view.sidebarMode) !== 'settings';
-        const closingPanel = view.instanceSettingsOpen && !(target.instanceSettingsOpen ?? view.instanceSettingsOpen);
-        const departing = (['panel', 'dashboard'] as const).filter(entry =>
-            portChanged || (entry === 'panel' ? closingPanel : leavingDashboard));
-        const dirty = departing.some(entry => entry === 'panel' ? args.panelSettingsDirty : args.dashboardSettingsDirty);
-        if (dirty && !(args.confirmDiscard ?? (() => window.confirm('Discard unsaved Settings changes?')))()) return false;
-        for (const entry of departing) args.clearDirty(entry);
+        // The rail workspace is the only settings surface, so there is one draft to guard.
+        const departing = portChanged || leavingDashboard;
+        if (!departing) return true;
+        if (args.dashboardSettingsDirty
+            && !(args.confirmDiscard ?? (() => window.confirm('Discard unsaved Settings changes?')))()) return false;
+        args.clearDirty('dashboard');
         return true;
     }
-    function setInstanceSettingsOpen(open: boolean, port = selectedPort): void {
-        if (!guardSettingsTransition({ selectedPort: port, instanceSettingsOpen: open,
-            sidebarMode: open ? 'instances' : view.sidebarMode })) return;
-        if (open) {
-            view.setSelectedPort(port); view.setSidebarMode('instances');
-            view.setViewMode('jaw'); view.setDrawerOpen(false);
-        }
-        view.setInstanceSettingsOpen(open);
-        void args.saveUi({ instanceSettingsOpen: open,
-            selectedTab: view.activeDetailTab === 'settings' ? 'overview' : view.activeDetailTab,
-            ...(open ? { selectedPort: port, sidebarMode: 'instances' as const } : {}) });
+    /** Opens or closes the rail's manager settings workspace, honouring the dirty guard. */
+    function setDashboardSettingsOpen(open: boolean): void {
+        const sidebarMode = open ? 'settings' as const : 'instances' as const;
+        if (view.sidebarMode === sidebarMode) return;
+        if (!guardSettingsTransition({ sidebarMode })) return;
+        if (open) { view.setViewMode('jaw'); view.setDrawerOpen(false); }
+        view.setSidebarMode(sidebarMode);
+        void args.saveUi({ sidebarMode,
+            selectedTab: view.activeDetailTab === 'settings' ? 'overview' : view.activeDetailTab });
     }
-    return { canLeaveDirtySettings, guardSettingsTransition, setInstanceSettingsOpen };
+    return { canLeaveDirtySettings, guardSettingsTransition, setDashboardSettingsOpen };
 }
 
-// Both entry points may remain mounted: a clean hidden Shell cannot clear its peer.
-export function settingsDirtyAfter(current: { panel: boolean; dashboard: boolean },
-    entry: 'panel' | 'dashboard', dirty: boolean) {
+// One settings surface remains, so there is one draft. The entry name is kept so the
+// dirty-store call sites read the same as before.
+export function settingsDirtyAfter(current: { dashboard: boolean },
+    entry: 'dashboard', dirty: boolean) {
     return { ...current, [entry]: dirty };
 }
 export function useSettingsDirtyState() {
-    const [entries, setEntries] = useState({ panel: false, dashboard: false });
-    const onSettingsDirtyChange = useCallback((entry: 'panel' | 'dashboard', dirty: boolean) => {
+    const [entries, setEntries] = useState({ dashboard: false });
+    const onSettingsDirtyChange = useCallback((entry: 'dashboard', dirty: boolean) => {
         setEntries(current => settingsDirtyAfter(current, entry, dirty));
     }, []);
-    // SettingsShell keys its dirty notification and unmount cleanup to this callback.
-    const onPanelSettingsDirtyChange = useCallback((dirty: boolean) => {
-        onSettingsDirtyChange('panel', dirty);
-    }, [onSettingsDirtyChange]);
     const setSettingsDirty = useCallback((dirty: boolean) => {
-        setEntries({ panel: dirty, dashboard: dirty });
+        setEntries({ dashboard: dirty });
     }, []);
-    return { settingsDirty: entries.panel || entries.dashboard, panelSettingsDirty: entries.panel, dashboardSettingsDirty: entries.dashboard, setSettingsDirty, onSettingsDirtyChange, onPanelSettingsDirtyChange };
+    return { settingsDirty: entries.dashboard, dashboardSettingsDirty: entries.dashboard, setSettingsDirty, onSettingsDirtyChange };
 }

package/public/manager/src/manager-components.css

@@ -1297,15 +1297,7 @@ button.instance-group-header[aria-expanded="false"] .instance-group-chevron {
 .drawer-body { min-height: 0; overflow: auto; }
 
 /* Settings replaces the workspace while retaining the preview host. */
-.workbench[data-instance-settings-open="true"] { grid-template-rows: minmax(0, 1fr); }
 .workbench .workbench-header[hidden], #manager-sidebar-list[hidden] { display: none; }
-.workbench[data-instance-settings-open="true"] .workbench-body { display: grid; min-height: 0; padding: 0; }
-.workbench-settings-page { min-width: 0; min-height: 0; height: 100%; }
-.workbench-settings-toggle { min-width: 32px; min-height: 32px; display: inline-flex; align-items: center; justify-content: center; }
-.workbench-settings-toggle svg { width: 16px; height: 16px; stroke-width: 1.8; }
-.workbench-settings-toggle[aria-pressed="true"] { color: var(--accent); background: var(--accent-soft); }
-.workbench-settings-toggle:focus-visible { outline: var(--focus-ring); outline-offset: 2px; }
-@media (pointer: coarse) { .workbench-settings-toggle { min-width: 44px; min-height: 44px; } }
 
 @media (max-width: 639px) {
     .command-center .command-title { display: none; }

package/public/manager/src/manager-shortcut-runner.ts

@@ -76,7 +76,6 @@ export function runManagerShortcut(action: DashboardShortcutAction, deps: Manage
     if (action === 'switchTab1') { deps.handleTabChange('overview'); return; }
     if (action === 'switchTab2') { deps.handleTabChange('preview'); return; }
     if (action === 'switchTab3') { deps.handleTabChange('logs'); return; }
-    if (action === 'switchTab4') { deps.handleTabChange('settings'); return; }
     if (action === 'previousTab' || action === 'nextTab') {
         const tabs: DashboardDetailTab[] = ['overview', 'preview', 'logs'];
         const idx = tabs.indexOf(deps.activeDetailTab);

package/public/manager/src/manager-shortcuts.ts

@@ -17,7 +17,6 @@ export const MANAGER_SHORTCUT_ACTIONS: DashboardShortcutAction[] = [
     'switchTab1',
     'switchTab2',
     'switchTab3',
-    'switchTab4',
     'previousTab',
     'nextTab',
     'browserReload',
@@ -57,7 +56,6 @@ export const DEFAULT_MANAGER_SHORTCUT_KEYMAP: DashboardShortcutKeymap = {
     switchTab1: 'Meta+1',
     switchTab2: 'Meta+2',
     switchTab3: 'Meta+3',
-    switchTab4: 'Meta+4',
     previousTab: 'Meta+Shift+[',
     nextTab: 'Meta+Shift+]',
     browserReload: 'Meta+R',

package/public/manager/src/settings/pages/Agent.tsx

@@ -124,7 +124,7 @@ type RuntimeDraft = {
     model: string;
     effort: string;
     workingDir: string;
-    permissions: 'auto' | string[];
+    permissions: 'auto' | 'safe' | string[];
 };
 
 export default function Agent({ port, client, dirty, registerSave }: SettingsPageProps) {
@@ -284,7 +284,10 @@ export default function Agent({ port, client, dirty, registerSave }: SettingsPag
             model: runtimeModelFor(cli, state.data.perCli, state.data.activeOverrides),
             effort: runtimeEffortFor(cli, state.data.perCli, state.data.activeOverrides),
             workingDir: state.data.workingDir || '',
-            permissions: permissions.mode === 'custom' ? permissions.tokens : 'auto',
+            // Preserve 'safe' rather than collapsing it: an untouched Safe instance must not
+            // become Auto (YOLO) as a side effect of saving something unrelated on this page.
+            permissions: permissions.mode === 'custom' ? permissions.tokens
+                : permissions.mode === 'safe' ? 'safe' : 'auto',
         });
     }, [cliMeta, state]);
 

package/public/manager/src/settings/pages/DashboardMeta.tsx

@@ -95,7 +95,26 @@ export type DashboardMetaProps = SettingsPageProps & {
     managerClient?: ManagerClient;
 };
 
-export default function DashboardMeta({
+/**
+ * No selected instance. SettingsShell passes `port={port ?? 0}`, and this page writes to the
+ * MANAGER registry keyed by port, so rendering the form here would let a save PATCH
+ * /api/dashboard/registry under instance key "0". This is a separate component rather than an
+ * early return because the form below calls hooks before its first return: an early return
+ * above them breaks the rules of hooks, and one below them has already issued the fetch.
+ */
+function DashboardMetaEmpty() {
+    return (
+        <p className="settings-empty" role="status">
+            Select an instance to edit its dashboard metadata.
+        </p>
+    );
+}
+
+export default function DashboardMeta(props: DashboardMetaProps) {
+    return props.port ? <DashboardMetaForm {...props} /> : <DashboardMetaEmpty />;
+}
+
+function DashboardMetaForm({
     port,
     client,
     dirty,

package/public/manager/src/settings/pages/Permissions.tsx

@@ -16,7 +16,7 @@ import {
 } from './page-shell';
 import { InlineWarn } from './components/InlineWarn';
 
-export type PermissionMode = 'auto' | 'custom';
+export type PermissionMode = 'auto' | 'safe' | 'custom';
 
 type PermissionsSnapshot = {
     permissions?: 'auto' | string[] | unknown;
@@ -58,9 +58,13 @@ export function parsePermissionsValue(
     value: unknown,
 ):
     | { mode: 'auto' }
+    | { mode: 'safe' }
     | { mode: 'custom'; tokens: string[] }
     | { mode: 'unknown' } {
     if (isPermissionsAuto(value)) return { mode: 'auto' };
+    // 'safe' is a real stored policy — the Classic sidebar is its only writer today. Reporting
+    // it as 'unknown' is what let callers collapse it to 'auto', silently widening permissions.
+    if (value === 'safe') return { mode: 'safe' };
     if (Array.isArray(value)) {
         const tokens = value
             .filter((t): t is string => typeof t === 'string')
@@ -102,6 +106,7 @@ export function isAllowlistValid(tokens: ReadonlyArray<string>): boolean {
 
 const MODE_OPTIONS = [
     { value: 'auto', label: 'Auto (YOLO)' },
+    { value: 'safe', label: 'Safe' },
     { value: 'custom', label: 'Custom — explicit allowlist below' },
 ];
 
@@ -125,7 +130,10 @@ export default function Permissions({ port, client, dirty, registerSave }: Setti
             setMode('custom');
             setTokens(parsed.tokens);
         } else {
-            setMode('auto');
+            // A Safe instance must read as Safe. Showing it as Auto (YOLO) hides the policy the
+            // user is about to change and reports 'auto' as the original, so a round trip
+            // through Custom and back writes a widened policy the user never chose.
+            setMode(parsed.mode === 'safe' ? 'safe' : 'auto');
             setTokens([]);
         }
     }, [state]);
@@ -141,9 +149,10 @@ export default function Permissions({ port, client, dirty, registerSave }: Setti
         [dirty],
     );
 
-    const originalSerialized = useMemo<'auto' | string[]>(() => {
+    const originalSerialized = useMemo<'auto' | 'safe' | string[]>(() => {
         if (!original) return 'auto';
         if (original.mode === 'custom') return original.tokens;
+        if (original.mode === 'safe') return 'safe';
         return 'auto';
     }, [original]);
 
@@ -159,6 +168,16 @@ export default function Permissions({ port, client, dirty, registerSave }: Setti
                 });
                 return;
             }
+            if (next === 'safe') {
+                setMode('safe');
+                setTokens([]);
+                writeEntry({
+                    value: 'safe',
+                    original: originalSerialized,
+                    valid: true,
+                });
+                return;
+            }
             if (next === 'custom') {
                 setMode('custom');
                 const seed =
@@ -204,7 +223,7 @@ export default function Permissions({ port, client, dirty, registerSave }: Setti
             setMode('custom');
             setTokens(parsed.tokens);
         } else {
-            setMode('auto');
+            setMode(parsed.mode === 'safe' ? 'safe' : 'auto');
             setTokens([]);
         }
         setData(fresh);

package/public/manager/src/settings/pages/components/agent/PermissionQuickSection.tsx

@@ -10,17 +10,20 @@ import {
 type PermissionQuickSectionProps = {
     value: unknown;
     configuredValue: unknown;
-    onChange(next: 'auto' | string[]): void;
+    onChange(next: 'auto' | 'safe' | string[]): void;
 };
 
 const MODE_OPTIONS = [
     { value: 'auto', label: 'Auto (YOLO)' },
+    { value: 'safe', label: 'Safe' },
     { value: 'custom', label: 'Custom allowlist' },
 ];
 
 export function PermissionQuickSection({ value, configuredValue, onChange }: PermissionQuickSectionProps) {
     const parsed = parsePermissionsValue(value);
-    const mode = parsed.mode === 'custom' ? 'custom' : 'auto';
+    // Never fall through to 'auto': a Safe instance shown as Auto (YOLO) means the user cannot
+    // see the permission they are about to lose when they touch this control.
+    const mode = parsed.mode === 'custom' ? 'custom' : parsed.mode === 'safe' ? 'safe' : 'auto';
     const tokens = parsed.mode === 'custom' ? parsed.tokens : [];
     const summary = parsed.mode === 'custom'
         ? `Editor draft: ${tokens.length} explicit token${tokens.length === 1 ? '' : 's'}`
@@ -38,6 +41,7 @@ export function PermissionQuickSection({ value, configuredValue, onChange }: Per
                 options={MODE_OPTIONS}
                 onChange={(next) => {
                     if (next === 'auto') onChange('auto');
+                    else if (next === 'safe') onChange('safe');
                     else onChange(tokens.length > 0 && isAllowlistValid(tokens) ? tokens : seedAutoAllowlist(null));
                 }}
             />

package/public/manager/src/settings/pages/components/agent/agent-save.ts

@@ -3,7 +3,9 @@ import { expandPatch } from '../../path-utils';
 import {
     applyRuntimeEmployeesDiff,
     runtimeEmployeesHaveErrors,
+    unwrapRuntimeEmployees,
     type RuntimeEmployeeRecord,
+    type RuntimeEmployeesResponse,
 } from './runtime-employees-helpers';
 
 export type AgentSettingsSnapshot = {
@@ -62,7 +64,28 @@ export async function saveAgentRuntime(options: {
     }
     try {
         if (runtimeEmployeesNext) {
-            await applyRuntimeEmployeesDiff(client, employeeOriginal, runtimeEmployeesNext);
+            // The Classic sidebar writes employees immediately and independently, so by save
+            // time the server can hold rows this page never saw. Two things follow, and only
+            // doing the first is worse than doing neither.
+            //
+            // Re-read so the diff is computed against reality rather than a stale snapshot.
+            // Then restrict deletion to rows this page actually knew about: a row that appeared
+            // on the server while the page was open is absent from the draft, which the plain
+            // diff reads as "the user removed it" and deletes. It was never the user's to
+            // remove. Additions and updates still come from the draft as the user expressed
+            // them.
+            //
+            // A failed re-read falls back to the snapshot rather than blocking the save: the
+            // stale diff is still the caller's explicit intent, and refusing to save would
+            // discard it.
+            let baseline = employeeOriginal;
+            try {
+                const server = unwrapRuntimeEmployees(await client.get<RuntimeEmployeesResponse>('/api/employees'));
+                const knownToThisPage = new Set(employeeOriginal.map(row => row.id));
+                const draftIds = new Set(runtimeEmployeesNext.map(row => row.id));
+                baseline = server.filter(row => knownToThisPage.has(row.id) || draftIds.has(row.id));
+            } catch { /* keep the snapshot baseline */ }
+            await applyRuntimeEmployeesDiff(client, baseline, runtimeEmployeesNext);
         }
     } catch (err: unknown) {
         throw new Error(`runtime employees save failed: ${err instanceof Error ? err.message : String(err)}`);

package/public/manager/src/settings/settings-icons.ts

@@ -10,6 +10,8 @@ const SETTINGS_ICONS: Readonly<Record<string, IconName>> = {
     'advanced-export': 'download', employees: 'people',
     'manager-display': 'monitor', 'manager-activity': 'heartPulse',
     'manager-developer': 'tool', 'manager-embedding': 'search',
-    'telegram-hub': 'radio', 'dashboard-meta': 'unknown',
+    // 'unknown' resolves through this table rather than the 'settings' fallback, so it rendered
+    // a literal question mark. The page edits an instance's label, group, favorite and notes.
+    'telegram-hub': 'radio', 'dashboard-meta': 'pencil',
 };
 export function settingsIcon(id: string): IconName { return SETTINGS_ICONS[id] ?? 'settings'; }

package/public/manager/src/settings/settings-registry.ts

@@ -3,7 +3,7 @@ import type { ComponentType } from 'react';
 import type { DashboardLocale } from '../types';
 import { DASHBOARD_SETTINGS_COPY } from '../dashboard-settings/DashboardSettingsSidebar';
 type Entry = { id: SettingsCategoryId; scope: SettingsScope; group: SettingsCategoryGroup;
-    label: string | ((locale: DashboardLocale) => string); requiresInstance?: boolean; hidden?: boolean; load: () => Promise<{ default: ComponentType<SettingsPageProps> }> };
+    label: string | ((locale: DashboardLocale) => string); hidden?: boolean; load: () => Promise<{ default: ComponentType<SettingsPageProps> }> };
 export const SETTINGS_REGISTRY: readonly Entry[] = [
     { id:'agent', scope:'instance', group:'runtime', label:'Agent', load:()=>import('./pages/Agent') },
     { id:'model', scope:'instance', group:'runtime', label:'Model defaults', load:()=>import('./pages/ModelProvider') },
@@ -27,9 +27,9 @@ export const SETTINGS_REGISTRY: readonly Entry[] = [
     { id:'manager-developer', scope:'manager', group:'advanced', label:locale=>DASHBOARD_SETTINGS_COPY[locale].sections.developer.label, load:()=>import('./pages/manager/Developer') },
     { id:'manager-embedding', scope:'manager', group:'automation', label:locale=>DASHBOARD_SETTINGS_COPY[locale].sections.embedding.label, load:()=>import('./pages/manager/Embedding') },
     { id:'telegram-hub', scope:'manager', group:'channels', label:locale=>DASHBOARD_SETTINGS_COPY[locale].sections.telegramHub.label, load:()=>import('./pages/manager/TelegramHub') },
-    { id:'dashboard-meta', scope:'manager', requiresInstance:true, group:'advanced', label:'Dashboard meta', load:()=>import('./pages/DashboardMeta') },
+    { id:'dashboard-meta', scope:'manager', group:'advanced', label:'Dashboard meta', load:()=>import('./pages/DashboardMeta') },
 ];
 export const entriesForScopes = (scopes: readonly SettingsScope[], hasInstance: boolean, locale: DashboardLocale) =>
     SETTINGS_REGISTRY.filter(entry => !entry.hidden && scopes.includes(entry.scope)
-        && (hasInstance || (entry.scope !== 'instance' && !entry.requiresInstance)))
+        && (hasInstance || entry.scope !== 'instance'))
         .map(entry => ({...entry, label:typeof entry.label==='function' ? entry.label(locale) : entry.label}));

package/public/manager/src/settings-controls.css

@@ -224,7 +224,6 @@
 .settings-shell .settings-field-error { grid-column:1 / -1; }
 .settings-shell .settings-select-menu { left:auto; right:0; width:min(100%,560px); }
 .settings-shell .settings-field :is(input:not([type=checkbox]),textarea), .settings-shell .settings-select-trigger { background:var(--surface-raised); box-shadow:none; }
-.workbench-settings-toggle:focus-visible,
 .settings-shell :focus-visible { outline: var(--focus-ring); outline-offset: 2px; }
 @container (max-width:719px) {
     .settings-shell .settings-field, .settings-shell .dashboard-settings-row { grid-template-columns:minmax(0,1fr); row-gap:12px; }

package/public/manager/src/settings-standalone.tsx

@@ -13,7 +13,6 @@ import './settings-embedding.css';
 
 const proxyMatch = location.pathname.match(/^\/i\/(\d+)\//);
 const apiBase = proxyMatch ? `/i/${proxyMatch[1]}` : '';
-const bootstrapPort = (window as Window & { __JAW_PORT?: unknown }).__JAW_PORT;
 const host = document.getElementById('settings-root');
 if (!host) throw new Error('Invalid settings host');
 
@@ -53,13 +52,14 @@ const client = createDirectClient(0, { base: apiBase, getHeaders: async () => {
     return token ? { Authorization: `Bearer ${token}` } : {};
 } });
 let snapshot: { port?: unknown } | null = null;
-if (validPort(bootstrapPort) === null) {
-    try {
-        const result = await client.get<{ port?: unknown; data?: { port?: unknown } }>('/api/settings');
-        snapshot = result.data ?? result;
-    } catch { /* Location is the final fallback; pages report their own API failures. */ }
-}
-const port = validPort(bootstrapPort) ?? validPort(snapshot?.port) ?? validPort(location.port);
+// There used to be a `window.__JAW_PORT` bootstrap fast path here. Nothing in the tree ever
+// assigned it, so it always resolved to null and this fetch ran regardless; keeping it implied
+// a handshake that did not exist.
+try {
+    const result = await client.get<{ port?: unknown; data?: { port?: unknown } }>('/api/settings');
+    snapshot = result.data ?? result;
+} catch { /* Location is the final fallback; pages report their own API failures. */ }
+const port = validPort(snapshot?.port) ?? validPort(location.port);
 if (port === null) {
     host.setAttribute('role', 'alert');
     host.textContent = 'Cannot resolve instance settings port';

package/public/manager/src/types.ts

@@ -30,7 +30,6 @@ export type DashboardShortcutAction =
     | 'switchTab1'
     | 'switchTab2'
     | 'switchTab3'
-    | 'switchTab4'
     | 'previousTab'
     | 'nextTab'
     | 'browserReload'