@jbrowse/product-core 5.0.0-beta.1 → 5.0.0-beta.11

package/esm/Session/BaseSession.d.ts

@@ -4,6 +4,7 @@ import type PluginManager from '@jbrowse/core/PluginManager';
 import type { BaseAssemblyConfigModel } from '@jbrowse/core/assemblyManager';
 import type { AnyConfigurationSchemaType } from '@jbrowse/core/configuration';
 import type { AnimationMode, DialogComponentType, TrackConfigChange } from '@jbrowse/core/util';
+import type { HighlightType } from '@jbrowse/core/util/highlights';
 import type { IAnyStateTreeNode, Instance } from '@jbrowse/mobx-state-tree';
 type DoneCallback = (doneCallback: () => void) => [DialogComponentType, Record<string, unknown>];
 /**
@@ -28,22 +29,31 @@ export declare function BaseSessionModel<ROOT_MODEL_TYPE extends BaseRootModelTy
     focusedViewId: import("@jbrowse/mobx-state-tree").IMaybe<import("@jbrowse/mobx-state-tree").ISimpleType<string>>;
     /**
      * #property
-     * one session-wide toggle for all region highlight bands (URL/view
-     * highlights and bookmark overlays)
+     * translucent bands over genomic regions; every view draws the ones on
+     * its own assemblies
+     */
+    highlights: import("@jbrowse/mobx-state-tree").IOptionalIType<import("@jbrowse/mobx-state-tree").IArrayType<import("@jbrowse/mobx-state-tree").IType<HighlightType, HighlightType, HighlightType>>, [undefined]>;
+    /**
+     * #property
      */
     highlightsVisible: import("@jbrowse/mobx-state-tree").IOptionalIType<import("@jbrowse/mobx-state-tree").ISimpleType<boolean>, [undefined]>;
+    /**
+     * #property
+     * whether a band's label is written on it
+     */
+    highlightLabelsVisible: import("@jbrowse/mobx-state-tree").IOptionalIType<import("@jbrowse/mobx-state-tree").ISimpleType<boolean>, [undefined]>;
     /**
      * #property
      * views, tracks, displays and widgets `pruneUnbuildableNodes` took out of
      * the tree because this build has no plugin for their type, each with the
-     * anchor that puts it back. Opaque here on purpose — nothing in the
-     * session reads it, and the shape belongs to that module.
+     * anchor that puts it back. Nothing in the session reads its contents,
+     * and `pruneUnbuildableNodes` defines their structure.
      *
-     * It is a **declared property** rather than data riding along in the
-     * snapshot because MST silently drops an undeclared key: without this
-     * line the held nodes survive the prune and then vanish the moment
-     * `setSession` builds the tree, and every unit test of the prune still
-     * passes. `heldNodesSurviveTheSession.test.ts` is the canary.
+     * It is a **declared property** because MST drops undeclared snapshot
+     * keys without error. Without this declaration the held nodes survive
+     * the prune and are then discarded when `setSession` builds the tree,
+     * while every unit test of the prune still passes.
+     * `heldNodesSurviveTheSession.test.ts` tests that they survive.
      */
     heldForMissingPlugins: import("@jbrowse/mobx-state-tree").IOptionalIType<import("@jbrowse/mobx-state-tree").IType<HeldNode[] | undefined, HeldNode[] | undefined, HeldNode[] | undefined>, [undefined]>;
 }, never>, {
@@ -75,23 +85,15 @@ export declare function BaseSessionModel<ROOT_MODEL_TYPE extends BaseRootModelTy
      *
      * An `observable.map` (not a plain object reassigned wholesale) so each
      * preference is its own tracked key: writing one (`setScrollZoom`) can't
-     * invalidate a reader of another (`getDisplayTypeDefault` in a track's
-     * `rpcProps`). A single spread-replaced object made every setter wake
-     * every reader, so toggling scroll-to-zoom re-fetched every track. For the
-     * same reason each promoted per-display-type default is a flat composite
-     * key (see `displayTypeDefaultKey`), not a single nested `displayTypeDefaults`
-     * object — promoting one default can't wake readers of a different one.
+     * invalidate a reader of another. A single spread-replaced object made
+     * every setter wake every reader, so toggling scroll-to-zoom re-fetched
+     * every track.
      *
-     * `deep: false` is load-bearing, not a micro-optimization. The default
-     * enhancer wraps an object/array value in a MobX Proxy on `set`, and a
-     * promoted default is handed straight back out by `getConf` — so an
-     * object-valued promotable slot (alignments `colorBy`) put a Proxy into
-     * `rpcProps()`, and V8's structured-clone serializer rejects a Proxy:
-     * `worker.postMessage` threw `DataCloneError` on the next fetch of any
-     * track following that default (electron IPC and `structuredClone` in the
-     * share bake likewise). The map still notifies per key on `set`, so shallow
-     * values lose no reactivity — and nothing can mutate a preference in place,
-     * because `setPreferenceOverride` freezes what it stores.
+     * `deep: false` keeps an object-valued preference a plain object rather
+     * than a MobX Proxy, which V8's structured-clone serializer rejects. The
+     * map still notifies per key on `set`, so shallow values lose no
+     * reactivity — and nothing can mutate a preference in place, because
+     * `setPreferenceOverride` freezes what it stores.
      */
     preferencesOverrides: import("mobx").ObservableMap<string, unknown>;
 } & {
@@ -152,37 +154,10 @@ export declare function BaseSessionModel<ROOT_MODEL_TYPE extends BaseRootModelTy
     getPreference(key: string): unknown;
     /**
      * #method
-     * resolved value of a per-display-type slot default the user promoted
-     * (see `setDisplayTypeDefault`); undefined when nothing was promoted.
-     */
-    getDisplayTypeDefault(displayType: string, slot: string): unknown;
-    /**
-     * #method
-     * every per-display-type default the user has promoted, as
-     * `{ displayType, slot, value }` — the inventory the Preferences dialog
-     * lists and clears one at a time (`setDisplayTypeDefault(…, undefined)`).
-     *
-     * Here rather than filtered out of `getPreferenceChanges` by the dialog,
-     * because the composite-key layout is this file's: a consumer that
-     * recognized these rows by matching the path head is exactly the coupling
-     * `DISPLAY_TYPE_DEFAULTS_PATH_HEAD` stopped being exported over, where a
-     * rename on one side alone silently no-ops the other.
-     */
-    getDisplayTypeDefaults(): {
-        displayType: string;
-        slot: string;
-        value: unknown;
-    }[];
-    /**
-     * #method
-     * every runtime preference-override that currently differs from its
-     * config/admin default, as `{ path, from, to }` rows — the exact set
-     * `clearPreferenceOverrides` reverts. Backs the confirmation diff shown
-     * before "Reset to defaults" (mirrors the per-track changes dialog). A
-     * scalar pref (animationMode, scrollZoom) whose override equals the
-     * default is omitted (reverting it is a no-op); each promoted
-     * per-display-type default is always a difference from the un-promoted
-     * state, so `from` reads "(default)".
+     * every scalar preference override that currently differs from its
+     * config/admin default, as `{ path, from, to }` rows whose path is the
+     * override's own key. A scalar pref (animationMode, scrollZoom) whose
+     * override equals the default is omitted, since reverting it is a no-op.
      */
     getPreferenceChanges(): TrackConfigChange[];
 } & {
@@ -229,15 +204,31 @@ export declare function BaseSessionModel<ROOT_MODEL_TYPE extends BaseRootModelTy
     setHovered(thing: unknown): void;
     /**
      * #action
-     * toggle all region highlight bands across every view
      */
     setHighlightsVisible(arg: boolean): void;
     /**
      * #action
-     * turn highlight bands back on, so a newly made highlight or bookmark is
-     * never silently swallowed by an earlier "highlights off"
      */
-    revealHighlights(): void;
+    setHighlightLabelsVisible(arg: boolean): void;
+    /**
+     * #action
+     * an identical entry is not added twice, so a spec launched again does not
+     * stack its bands. Adding shows the bands again, since a highlight made
+     * while they are off would otherwise read as nothing happening
+     */
+    addHighlight(highlight: HighlightType): void;
+    /**
+     * #action
+     */
+    removeHighlight(highlight: HighlightType): void;
+    /**
+     * #action
+     */
+    updateHighlight(old: HighlightType, updates: Partial<HighlightType>): void;
+    /**
+     * #action
+     */
+    setHighlights(highlights: HighlightType[]): void;
     /**
      * #action
      * set a runtime user-preference override (see `getPreference`). Mutates
@@ -248,10 +239,9 @@ export declare function BaseSessionModel<ROOT_MODEL_TYPE extends BaseRootModelTy
     setPreferenceOverride(key: string, value: unknown): void;
     /**
      * #action
-     * clear every runtime preference override at once — scrollZoom,
-     * animationMode, and every promoted per-display-type default (see
-     * `setDisplayTypeDefault`) — so each falls back to its config/admin
-     * default. Backs the Preferences dialog "Reset to defaults" button.
+     * clear every runtime preference override at once, so each falls back to
+     * its config/admin default. Backs the Preferences dialog "Reset to
+     * defaults" button.
      */
     clearPreferenceOverrides(): void;
     /**
@@ -261,32 +251,11 @@ export declare function BaseSessionModel<ROOT_MODEL_TYPE extends BaseRootModelTy
      * Preferences dialog "Reset to defaults" confirmation.
      */
     clearPreferenceOverride(key: string): void;
-    /**
-     * #action
-     * revert one row emitted by `getPreferenceChanges`, addressed by its
-     * display `path`. Backs the per-entry reset in the Preferences dialog's
-     * "Reset to defaults" confirmation.
-     *
-     * Lives here rather than in that dialog because this model owns both path
-     * shapes it has to undo: a promoted per-display-type default, whose row
-     * path is a readable label over a flat composite storage key, and every
-     * other override, whose path *is* its key. The dialog used to re-derive
-     * the first case from an exported path-head constant.
-     */
-    resetPreferenceChange(path: string[]): void;
     /**
      * #action
      * set the global scroll-to-zoom preference (see the `scrollZoom` getter)
      */
     setScrollZoom(flag: boolean): void;
-    /**
-     * #action
-     * promote (or, with `value` undefined, clear) a per-display-type slot
-     * default. Just a preference override under one flat composite key (see
-     * `displayTypeDefaultKey`), so it persists and independently tracks like
-     * any other pref, and clearing deletes only that key.
-     */
-    setDisplayTypeDefault(displayType: string, slot: string, value: unknown): void;
     /**
      * #action
      */
@@ -310,6 +279,10 @@ export declare function BaseSessionModel<ROOT_MODEL_TYPE extends BaseRootModelTy
      * #action
      */
     queueDialog(doneCallback: DoneCallback): void;
+    /**
+     * #action
+     */
+    removeDialog(entry: [DialogComponentType, Record<string, unknown>]): void;
 } & {
     snackbarMessages: import("mobx").IObservableArray<import("@jbrowse/core/ui/SnackbarModel").SnackbarMessage>;
     errorDialog: import("@jbrowse/core/ui/SnackbarModel").ErrorDialogState | undefined;
@@ -322,6 +295,8 @@ export declare function BaseSessionModel<ROOT_MODEL_TYPE extends BaseRootModelTy
     pushSnackbarMessage(message: string, level?: import("@jbrowse/core/util").NotificationLevel, actions?: import("@jbrowse/core/util").SnackAction[]): void;
     popSnackbarMessage(): import("@jbrowse/core/ui/SnackbarModel").SnackbarMessage | undefined;
     removeSnackbarMessage(message: string): void;
+} & {
+    afterAttach(): void;
 }, import("@jbrowse/mobx-state-tree")._NotCustomized, import("@jbrowse/mobx-state-tree")._NotCustomized>;
 /** Session mixin MST type for the most basic session */
 export type BaseSessionType = ReturnType<typeof BaseSessionModel>;

package/esm/Session/BaseSession.js

@@ -1,55 +1,16 @@
-import { getConf } from '@jbrowse/core/configuration';
+import { readConfObject } from '@jbrowse/core/configuration';
 import SnackbarModel from '@jbrowse/core/ui/SnackbarModel';
 import { setNumberGrouping } from '@jbrowse/core/util';
 import { freezeDeep } from '@jbrowse/core/util/freezeDeep';
+import { isSameHighlight } from '@jbrowse/core/util/highlights';
 import { isFeature, unwrapFeature } from '@jbrowse/core/util/simpleFeature';
 import { ElementId } from '@jbrowse/core/util/types/mst';
-import { getParent, isStateTreeNode, types } from '@jbrowse/mobx-state-tree';
+import { cast, getParent, isStateTreeNode, types, } from '@jbrowse/mobx-state-tree';
 import { observable } from 'mobx';
+import { applyDeveloperMode } from './developerMode.js';
 function isAnimationMode(val) {
     return val === 'system' || val === 'enabled' || val === 'disabled';
 }
-// Promoted per-display-type slot defaults live flat in `preferencesOverrides`
-// under one composite key each (`displayTypeDefault\0<type>\0<slot>`), not under
-// a single nested `displayTypeDefaults` object. Flat keys make each promoted
-// default its own tracked observable-map entry — promoting one can't invalidate
-// a reader of another (every promotable display's `rpcProps` reads one via
-// `getDisplayTypeDefault`) — and collapse the get/set/diff logic to one lookup.
-// The `\0` delimiter can't appear in a display type or slot name.
-// The literal is a **persisted localStorage key prefix**, so treat it as data
-// rather than as an identifier a rename should sweep — a blanket
-// `displayTypeDefault` → `pin` pass did sweep it, and orphans every promoted
-// default already in a browser's storage. Nothing released stores these yet, so
-// the reason it keeps this spelling is naming, not compatibility: the key names
-// *what is stored* (a display-type default, reached through
-// `get/setDisplayTypeDefault`), while `Pin` names the menu control the write
-// is offered from.
-const DISPLAY_TYPE_DEFAULT_PREFIX = 'displayTypeDefault\0';
-// Module-private on purpose: the composite-key spelling is a storage detail of
-// this file's get/set/diff methods, and nothing outside it should be able to
-// depend on the layout.
-function displayTypeDefaultKey(displayType, slot) {
-    return `${DISPLAY_TYPE_DEFAULT_PREFIX}${displayType}\0${slot}`;
-}
-function parseDisplayTypeDefaultKey(key) {
-    const rest = key.startsWith(DISPLAY_TYPE_DEFAULT_PREFIX)
-        ? key.slice(DISPLAY_TYPE_DEFAULT_PREFIX.length)
-        : '';
-    const [displayType, slot] = rest.split('\0');
-    return displayType && slot ? { displayType, slot } : undefined;
-}
-// Head of the display path a promoted default takes in a `getPreferenceChanges`
-// row: `[DISPLAY_TYPE_DEFAULTS_PATH_HEAD, displayType, slot]`. Purely a
-// readable row label, deliberately distinct from the flat
-// `DISPLAY_TYPE_DEFAULT_PREFIX` storage key above.
-//
-// Module-private, like the storage key: both the producer
-// (`getPreferenceChanges`) and the consumer (`resetPreferenceChange`) are
-// methods on this model, so the two path shapes this file emits are the only
-// ones it has to undo. It used to be exported for the Preferences dialog to
-// match on and re-route back through `setDisplayTypeDefault`, which meant a
-// rename on one side alone would silently no-op that dialog's per-row reset.
-const DISPLAY_TYPE_DEFAULTS_PATH_HEAD = 'displayTypeDefaults';
 /**
  * #stateModel BaseSessionModel
  *
@@ -74,22 +35,31 @@ export function BaseSessionModel(_pluginManager) {
         focusedViewId: types.maybe(types.string),
         /**
          * #property
-         * one session-wide toggle for all region highlight bands (URL/view
-         * highlights and bookmark overlays)
+         * translucent bands over genomic regions; every view draws the ones on
+         * its own assemblies
+         */
+        highlights: types.stripDefault(types.array(types.frozen()), []),
+        /**
+         * #property
          */
         highlightsVisible: types.stripDefault(types.boolean, true),
+        /**
+         * #property
+         * whether a band's label is written on it
+         */
+        highlightLabelsVisible: types.stripDefault(types.boolean, true),
         /**
          * #property
          * views, tracks, displays and widgets `pruneUnbuildableNodes` took out of
          * the tree because this build has no plugin for their type, each with the
-         * anchor that puts it back. Opaque here on purpose — nothing in the
-         * session reads it, and the shape belongs to that module.
+         * anchor that puts it back. Nothing in the session reads its contents,
+         * and `pruneUnbuildableNodes` defines their structure.
          *
-         * It is a **declared property** rather than data riding along in the
-         * snapshot because MST silently drops an undeclared key: without this
-         * line the held nodes survive the prune and then vanish the moment
-         * `setSession` builds the tree, and every unit test of the prune still
-         * passes. `heldNodesSurviveTheSession.test.ts` is the canary.
+         * It is a **declared property** because MST drops undeclared snapshot
+         * keys without error. Without this declaration the held nodes survive
+         * the prune and are then discarded when `setSession` builds the tree,
+         * while every unit test of the prune still passes.
+         * `heldNodesSurviveTheSession.test.ts` tests that they survive.
          */
         heldForMissingPlugins: types.stripDefault(types.frozen(), undefined),
     })
@@ -124,23 +94,15 @@ export function BaseSessionModel(_pluginManager) {
          *
          * An `observable.map` (not a plain object reassigned wholesale) so each
          * preference is its own tracked key: writing one (`setScrollZoom`) can't
-         * invalidate a reader of another (`getDisplayTypeDefault` in a track's
-         * `rpcProps`). A single spread-replaced object made every setter wake
-         * every reader, so toggling scroll-to-zoom re-fetched every track. For the
-         * same reason each promoted per-display-type default is a flat composite
-         * key (see `displayTypeDefaultKey`), not a single nested `displayTypeDefaults`
-         * object — promoting one default can't wake readers of a different one.
+         * invalidate a reader of another. A single spread-replaced object made
+         * every setter wake every reader, so toggling scroll-to-zoom re-fetched
+         * every track.
          *
-         * `deep: false` is load-bearing, not a micro-optimization. The default
-         * enhancer wraps an object/array value in a MobX Proxy on `set`, and a
-         * promoted default is handed straight back out by `getConf` — so an
-         * object-valued promotable slot (alignments `colorBy`) put a Proxy into
-         * `rpcProps()`, and V8's structured-clone serializer rejects a Proxy:
-         * `worker.postMessage` threw `DataCloneError` on the next fetch of any
-         * track following that default (electron IPC and `structuredClone` in the
-         * share bake likewise). The map still notifies per key on `set`, so shallow
-         * values lose no reactivity — and nothing can mutate a preference in place,
-         * because `setPreferenceOverride` freezes what it stores.
+         * `deep: false` keeps an object-valued preference a plain object rather
+         * than a MobX Proxy, which V8's structured-clone serializer rejects. The
+         * map still notifies per key on `set`, so shallow values lose no
+         * reactivity — and nothing can mutate a preference in place, because
+         * `setPreferenceOverride` freezes what it stores.
          */
         preferencesOverrides: observable.map(undefined, {
             deep: false,
@@ -216,7 +178,8 @@ export function BaseSessionModel(_pluginManager) {
          * `defaultUseWorkspaces`).
          */
         getPreferenceDefault(key) {
-            return getConf(self, ['preferences', key]);
+            const { preferences } = self.configuration;
+            return preferences ? readConfObject(preferences, key) : undefined;
         },
         /**
          * #method
@@ -232,70 +195,22 @@ export function BaseSessionModel(_pluginManager) {
         },
         /**
          * #method
-         * resolved value of a per-display-type slot default the user promoted
-         * (see `setDisplayTypeDefault`); undefined when nothing was promoted.
-         */
-        getDisplayTypeDefault(displayType, slot) {
-            return self.preferencesOverrides.get(displayTypeDefaultKey(displayType, slot));
-        },
-        /**
-         * #method
-         * every per-display-type default the user has promoted, as
-         * `{ displayType, slot, value }` — the inventory the Preferences dialog
-         * lists and clears one at a time (`setDisplayTypeDefault(…, undefined)`).
-         *
-         * Here rather than filtered out of `getPreferenceChanges` by the dialog,
-         * because the composite-key layout is this file's: a consumer that
-         * recognized these rows by matching the path head is exactly the coupling
-         * `DISPLAY_TYPE_DEFAULTS_PATH_HEAD` stopped being exported over, where a
-         * rename on one side alone silently no-ops the other.
-         */
-        getDisplayTypeDefaults() {
-            const found = [];
-            for (const [key, value] of self.preferencesOverrides.entries()) {
-                const parsed = parseDisplayTypeDefaultKey(key);
-                if (parsed) {
-                    found.push({ ...parsed, value });
-                }
-            }
-            return found;
-        },
-        /**
-         * #method
-         * every runtime preference-override that currently differs from its
-         * config/admin default, as `{ path, from, to }` rows — the exact set
-         * `clearPreferenceOverrides` reverts. Backs the confirmation diff shown
-         * before "Reset to defaults" (mirrors the per-track changes dialog). A
-         * scalar pref (animationMode, scrollZoom) whose override equals the
-         * default is omitted (reverting it is a no-op); each promoted
-         * per-display-type default is always a difference from the un-promoted
-         * state, so `from` reads "(default)".
+         * every scalar preference override that currently differs from its
+         * config/admin default, as `{ path, from, to }` rows whose path is the
+         * override's own key. A scalar pref (animationMode, scrollZoom) whose
+         * override equals the default is omitted, since reverting it is a no-op.
          */
         getPreferenceChanges() {
             const changes = [];
             for (const [key, value] of self.preferencesOverrides.entries()) {
-                const dtd = parseDisplayTypeDefaultKey(key);
-                if (dtd) {
+                const dflt = this.getPreferenceDefault(key);
+                if (value !== dflt) {
                     changes.push({
-                        path: [
-                            DISPLAY_TYPE_DEFAULTS_PATH_HEAD,
-                            dtd.displayType,
-                            dtd.slot,
-                        ],
-                        from: undefined,
+                        path: [key],
+                        from: dflt,
                         to: value,
                     });
                 }
-                else {
-                    const dflt = this.getPreferenceDefault(key);
-                    if (value !== dflt) {
-                        changes.push({
-                            path: [key],
-                            from: dflt,
-                            to: value,
-                        });
-                    }
-                }
             }
             return changes;
         },
@@ -366,19 +281,53 @@ export function BaseSessionModel(_pluginManager) {
         },
         /**
          * #action
-         * toggle all region highlight bands across every view
          */
         setHighlightsVisible(arg) {
             self.highlightsVisible = arg;
         },
         /**
          * #action
-         * turn highlight bands back on, so a newly made highlight or bookmark is
-         * never silently swallowed by an earlier "highlights off"
          */
-        revealHighlights() {
+        setHighlightLabelsVisible(arg) {
+            self.highlightLabelsVisible = arg;
+        },
+        /**
+         * #action
+         * an identical entry is not added twice, so a spec launched again does not
+         * stack its bands. Adding shows the bands again, since a highlight made
+         * while they are off would otherwise read as nothing happening
+         */
+        addHighlight(highlight) {
+            if (!self.highlights.some(h => isSameHighlight(h, highlight))) {
+                self.highlights.push(highlight);
+            }
             self.highlightsVisible = true;
         },
+        /**
+         * #action
+         */
+        removeHighlight(highlight) {
+            self.highlights.remove(highlight);
+        },
+        /**
+         * #action
+         */
+        updateHighlight(old, updates) {
+            const idx = self.highlights.indexOf(old);
+            if (idx !== -1) {
+                self.highlights[idx] = { ...old, ...updates };
+                self.highlightsVisible = true;
+            }
+        },
+        /**
+         * #action
+         */
+        setHighlights(highlights) {
+            self.highlights = cast(highlights);
+            if (highlights.length) {
+                self.highlightsVisible = true;
+            }
+        },
         /**
          * #action
          * set a runtime user-preference override (see `getPreference`). Mutates
@@ -391,19 +340,18 @@ export function BaseSessionModel(_pluginManager) {
                 self.preferencesOverrides.delete(key);
             }
             else {
-                // frozen because `deep: false` hands an object-valued preference (a
-                // promoted `colorBy`) straight back out by reference to every display
-                // following it, so the "nothing mutates a preference in place" this
-                // store relies on is enforced rather than assumed
+                // frozen because `deep: false` hands an object-valued preference
+                // straight back out by reference, so the "nothing mutates a
+                // preference in place" this store relies on is enforced rather than
+                // assumed
                 self.preferencesOverrides.set(key, freezeDeep(value));
             }
         },
         /**
          * #action
-         * clear every runtime preference override at once — scrollZoom,
-         * animationMode, and every promoted per-display-type default (see
-         * `setDisplayTypeDefault`) — so each falls back to its config/admin
-         * default. Backs the Preferences dialog "Reset to defaults" button.
+         * clear every runtime preference override at once, so each falls back to
+         * its config/admin default. Backs the Preferences dialog "Reset to
+         * defaults" button.
          */
         clearPreferenceOverrides() {
             self.preferencesOverrides.clear();
@@ -417,27 +365,6 @@ export function BaseSessionModel(_pluginManager) {
         clearPreferenceOverride(key) {
             self.preferencesOverrides.delete(key);
         },
-        /**
-         * #action
-         * revert one row emitted by `getPreferenceChanges`, addressed by its
-         * display `path`. Backs the per-entry reset in the Preferences dialog's
-         * "Reset to defaults" confirmation.
-         *
-         * Lives here rather than in that dialog because this model owns both path
-         * shapes it has to undo: a promoted per-display-type default, whose row
-         * path is a readable label over a flat composite storage key, and every
-         * other override, whose path *is* its key. The dialog used to re-derive
-         * the first case from an exported path-head constant.
-         */
-        resetPreferenceChange(path) {
-            const [head, displayType, slot] = path;
-            if (head === DISPLAY_TYPE_DEFAULTS_PATH_HEAD && displayType && slot) {
-                this.setDisplayTypeDefault(displayType, slot, undefined);
-            }
-            else if (head) {
-                this.clearPreferenceOverride(head);
-            }
-        },
         /**
          * #action
          * set the global scroll-to-zoom preference (see the `scrollZoom` getter)
@@ -445,16 +372,6 @@ export function BaseSessionModel(_pluginManager) {
         setScrollZoom(flag) {
             this.setPreferenceOverride('scrollZoom', flag);
         },
-        /**
-         * #action
-         * promote (or, with `value` undefined, clear) a per-display-type slot
-         * default. Just a preference override under one flat composite key (see
-         * `displayTypeDefaultKey`), so it persists and independently tracks like
-         * any other pref, and clearing deletes only that key.
-         */
-        setDisplayTypeDefault(displayType, slot, value) {
-            this.setPreferenceOverride(displayTypeDefaultKey(displayType, slot), value);
-        },
         /**
          * #action
          */
@@ -484,13 +401,30 @@ export function BaseSessionModel(_pluginManager) {
          * #action
          */
         queueDialog(doneCallback) {
-            const [component, props] = doneCallback(() => {
-                this.removeActiveDialog();
+            // its own entry, not the head: a late second call must not close the
+            // dialog queued behind it
+            const entry = doneCallback(() => {
+                this.removeDialog(entry);
             });
-            self.queueOfDialogs = [...self.queueOfDialogs, [component, props]];
+            self.queueOfDialogs = [...self.queueOfDialogs, entry];
+        },
+        /**
+         * #action
+         */
+        removeDialog(entry) {
+            self.queueOfDialogs = self.queueOfDialogs.filter(d => d !== entry);
+        },
+    }));
+    return types.compose(baseModel, SnackbarModel()).actions(self => ({
+        // after the compose rather than in the afterAttach above, because the
+        // notices land in the snackbar this line is the first place to hold both
+        // halves of. Web and desktop reach it a second time once the user's stored
+        // overrides are loaded (`PreferencesSessionMixin`), and our MST fork
+        // auto-chains both hooks.
+        afterAttach() {
+            applyDeveloperMode(self);
         },
     }));
-    return types.compose(baseModel, SnackbarModel());
 }
 /** Type guard for BaseSession */
 export function isBaseSession(thing) {

package/esm/Session/Connections.d.ts

@@ -111,9 +111,10 @@ export declare function ConnectionManagementSessionMixin(pluginManager: PluginMa
     pruneConnectionTrackConfig(trackId: string): void;
     /**
      * #action
-     * Lazily establish a single connection by id if it isn't already live —
-     * used when its category is expanded in the track selector. Fetches
-     * silently (no view launch / success snackbar); already-open tracks keep
+     * Lazily establish a single connection by id if it isn't already live;
+     * the track selector calls it when the connection's category is
+     * expanded. Fetches without launching a view or showing a success
+     * snackbar; already-open tracks keep
      * rendering from `connectionTrackConfigs` meanwhile. Idempotent.
      */
     hydrateConnection(connectionId: string): void;

package/esm/Session/Connections.js

@@ -189,9 +189,10 @@ export function ConnectionManagementSessionMixin(pluginManager) {
             },
             /**
              * #action
-             * Lazily establish a single connection by id if it isn't already live —
-             * used when its category is expanded in the track selector. Fetches
-             * silently (no view launch / success snackbar); already-open tracks keep
+             * Lazily establish a single connection by id if it isn't already live;
+             * the track selector calls it when the connection's category is
+             * expanded. Fetches without launching a view or showing a success
+             * snackbar; already-open tracks keep
              * rendering from `connectionTrackConfigs` meanwhile. Idempotent.
              */
             hydrateConnection(connectionId) {