@jbrowse/plugin-linear-genome-view 5.0.0-beta.2 → 5.0.0-beta.3

package/esm/displayKitTests/perRegionTestEnv.d.ts

@@ -33,6 +33,11 @@ export interface PerRegionTestControl {
     estimateCalls: number;
     /** the display's own density verdict, canvas's second too-large axis */
     densityTooLarge: boolean;
+    /**
+     * What the next fetch stores, so a test can tell the payload a refetch
+     * replaced from the one that replaced it.
+     */
+    payloadTag: string;
 }
 /**
  * The smallest possible display on `MultiRegionDisplayMixin`: it fetches, it
@@ -321,6 +326,7 @@ declare function makeStateModel(control: PerRegionTestControl, gate: GateOptIns,
     readonly canRender: boolean;
     readonly rendersCanvas: boolean;
     readonly paintInert: boolean;
+    readonly paintSuperseded: boolean;
 } & {
     readonly painted: boolean;
 } & {
@@ -354,6 +360,7 @@ declare function makeStateModel(control: PerRegionTestControl, gate: GateOptIns,
     readonly awaitingPrerequisite: boolean;
     readonly awaitingDependentData: boolean;
     readonly rpcPropsCacheKey: string;
+    readonly adapterConfigKey: string;
 } & {
     setError(error?: unknown): void;
     setStatusMessage(status?: import("@jbrowse/core/util").RpcStatus): void;
@@ -374,31 +381,45 @@ declare function makeStateModel(control: PerRegionTestControl, gate: GateOptIns,
 } & {
     readonly host: import("@jbrowse/display-kit/regionHost").RegionHost;
     readonly canvasWidthPx: number;
+    readonly settledSubPixelBinBp: number;
     readonly canRender: boolean;
     readonly rendersCanvas: boolean;
     readonly viewportWithinLoadedData: boolean;
     readonly viewportEmpty: boolean;
     readonly fetchSuspended: boolean;
     readonly layoutReady: boolean;
-    readonly regionFetchKey: string;
-    regionHasData(_displayedRegionIndex: number): boolean;
+    readonly zoomFetchKey: string;
+    regionHasData(displayedRegionIndex: number): boolean;
     readonly dataSuperseded: boolean;
     readonly renderBlocks: import("@jbrowse/render-core/renderBlock").RenderBlock[];
+} & {
+    readonly fetchInputs: import("@jbrowse/display-kit/fetchInputs").FetchInputs;
+    readonly settingsFetchInputs: unknown;
+} & {
+    readonly regionPayloads: ReadonlyMap<number, unknown>;
+    readonly hasRegionData: boolean;
 } & {
     isCacheValid(displayedRegionIndex: number): boolean;
 } & {
+    readonly staleSettingsDrawn: boolean;
     readonly dataCurrent: boolean;
     readonly loadedAssembly: import("@jbrowse/core/assemblyManager/assembly").Assembly | undefined;
+} & {
+    readonly phaseViewportCurrent: boolean;
 } & {
     readonly svgReady: boolean;
     readonly paintInert: boolean;
+    readonly paintSuperseded: boolean;
     readonly displayPhase: import("@jbrowse/render-core/displayPhase").DisplayPhase;
 } & {
-    setLoadedRegion(displayedRegionIndex: number, region: import("@jbrowse/core/util").Region, fetchKey?: string): void;
+    setLoadedRegion(displayedRegionIndex: number, region: import("@jbrowse/core/util").Region, payload: import("@jbrowse/display-kit/regionCommit").RegionPayload | undefined, fetchInputs?: import("@jbrowse/display-kit/fetchInputs").FetchInputs): void;
+    evictRegionStore(keep: ReadonlySet<number>): void;
     dropLoadedRegion(displayedRegionIndex: number): void;
     clearDisplaySpecificData(): void;
+    clearSettingsBakedData(): void;
 } & {
     clearAllRpcData(): void;
+    invalidateSettings(): void;
     reload(): void;
 } & {
     fetchNeeded(_needed: IndexedRegion[]): void;
@@ -409,23 +430,23 @@ declare function makeStateModel(control: PerRegionTestControl, gate: GateOptIns,
 } & {
     /** every `fetchNeeded` call, as the region list it was handed */
     fetchLog: IndexedRegion[][];
-    loadedData: Map<number, string>;
     /**
-     * What `regionFetchKey` answers. A volatile, not a `control` knob, and
+     * What `zoomFetchKey` answers. A volatile, not a `control` knob, and
      * that is the point: a real display's key reads view state, so the
      * autorun re-runs when it moves. A plain closure value would be memoized
      * by the computed and never invalidate.
      */
     fetchKey: string;
 } & {
+    regionHasData(displayedRegionIndex: number): boolean;
+} & {
+    /** the fetched payloads, off the foundation's store */
+    readonly loadedData: ReadonlyMap<number, string>;
     readonly gateEnabled: boolean;
     readonly densityTooLarge: boolean;
-    readonly regionFetchKey: string;
-    regionHasData(_displayedRegionIndex: number): boolean;
+    readonly zoomFetchKey: string;
 } & {
     setFetchKey(key: string): void;
-    setLoaded(displayedRegionIndex: number, value: string): void;
-    clearDisplaySpecificData(): void;
 } & {
     fetchNeeded(needed: IndexedRegion[]): Promise<void>;
 }, import("@jbrowse/mobx-state-tree")._NotCustomized, import("@jbrowse/mobx-state-tree")._NotCustomized>;
@@ -471,6 +492,8 @@ export declare function createPerRegionTestEnvironment({ measuresBytes, gate, es
                 level?: string;
                 actions: import("@jbrowse/core/util").SnackAction[];
             }[];
+            widgets: Map<string, unknown>;
+            openedWidgets: Record<string, unknown>[];
         } & {
             getTrackById: (id: string) => unknown;
             getDisplayTypeDefault(displayType: string, slot: string): unknown;
@@ -480,6 +503,11 @@ export declare function createPerRegionTestEnvironment({ measuresBytes, gate, es
             notifyError(message: string): void;
             queueDialog(cb: (handleClose: () => void) => import("@jbrowse/display-test-utils").QueuedDialog): void;
             setSelection(thing: unknown): void;
+            addWidget(type: string, id: string, initialState?: Record<string, unknown>): {
+                type: string;
+                id: string;
+            };
+            showWidget(_widget: unknown): void;
             setDisplayTypeDefault(displayType: string, slot: string, value: unknown): void;
         } & import("@jbrowse/mobx-state-tree").IStateTreeNode<import("@jbrowse/mobx-state-tree").IModelType<{
             name: import("@jbrowse/mobx-state-tree").IType<string | undefined, string, string>;
@@ -498,6 +526,8 @@ export declare function createPerRegionTestEnvironment({ measuresBytes, gate, es
                 level?: string;
                 actions: import("@jbrowse/core/util").SnackAction[];
             }[];
+            widgets: Map<string, unknown>;
+            openedWidgets: Record<string, unknown>[];
         } & {
             getTrackById: (id: string) => unknown;
             getDisplayTypeDefault(displayType: string, slot: string): unknown;
@@ -507,6 +537,11 @@ export declare function createPerRegionTestEnvironment({ measuresBytes, gate, es
             notifyError(message: string): void;
             queueDialog(cb: (handleClose: () => void) => import("@jbrowse/display-test-utils").QueuedDialog): void;
             setSelection(thing: unknown): void;
+            addWidget(type: string, id: string, initialState?: Record<string, unknown>): {
+                type: string;
+                id: string;
+            };
+            showWidget(_widget: unknown): void;
             setDisplayTypeDefault(displayType: string, slot: string, value: unknown): void;
         }, import("@jbrowse/mobx-state-tree")._NotCustomized, import("@jbrowse/mobx-state-tree")._NotCustomized>>;
         view: any;
@@ -786,6 +821,7 @@ export declare function createPerRegionTestEnvironment({ measuresBytes, gate, es
             readonly canRender: boolean;
             readonly rendersCanvas: boolean;
             readonly paintInert: boolean;
+            readonly paintSuperseded: boolean;
         } & {
             readonly painted: boolean;
         } & {
@@ -819,6 +855,7 @@ export declare function createPerRegionTestEnvironment({ measuresBytes, gate, es
             readonly awaitingPrerequisite: boolean;
             readonly awaitingDependentData: boolean;
             readonly rpcPropsCacheKey: string;
+            readonly adapterConfigKey: string;
         } & {
             setError(error?: unknown): void;
             setStatusMessage(status?: import("@jbrowse/core/util").RpcStatus): void;
@@ -839,31 +876,45 @@ export declare function createPerRegionTestEnvironment({ measuresBytes, gate, es
         } & {
             readonly host: import("@jbrowse/display-kit/regionHost").RegionHost;
             readonly canvasWidthPx: number;
+            readonly settledSubPixelBinBp: number;
             readonly canRender: boolean;
             readonly rendersCanvas: boolean;
             readonly viewportWithinLoadedData: boolean;
             readonly viewportEmpty: boolean;
             readonly fetchSuspended: boolean;
             readonly layoutReady: boolean;
-            readonly regionFetchKey: string;
-            regionHasData(_displayedRegionIndex: number): boolean;
+            readonly zoomFetchKey: string;
+            regionHasData(displayedRegionIndex: number): boolean;
             readonly dataSuperseded: boolean;
             readonly renderBlocks: import("@jbrowse/render-core/renderBlock").RenderBlock[];
+        } & {
+            readonly fetchInputs: import("@jbrowse/display-kit/fetchInputs").FetchInputs;
+            readonly settingsFetchInputs: unknown;
+        } & {
+            readonly regionPayloads: ReadonlyMap<number, unknown>;
+            readonly hasRegionData: boolean;
         } & {
             isCacheValid(displayedRegionIndex: number): boolean;
         } & {
+            readonly staleSettingsDrawn: boolean;
             readonly dataCurrent: boolean;
             readonly loadedAssembly: import("@jbrowse/core/assemblyManager/assembly").Assembly | undefined;
+        } & {
+            readonly phaseViewportCurrent: boolean;
         } & {
             readonly svgReady: boolean;
             readonly paintInert: boolean;
+            readonly paintSuperseded: boolean;
             readonly displayPhase: import("@jbrowse/render-core/displayPhase").DisplayPhase;
         } & {
-            setLoadedRegion(displayedRegionIndex: number, region: import("@jbrowse/core/util").Region, fetchKey?: string): void;
+            setLoadedRegion(displayedRegionIndex: number, region: import("@jbrowse/core/util").Region, payload: import("@jbrowse/display-kit/regionCommit").RegionPayload | undefined, fetchInputs?: import("@jbrowse/display-kit/fetchInputs").FetchInputs): void;
+            evictRegionStore(keep: ReadonlySet<number>): void;
             dropLoadedRegion(displayedRegionIndex: number): void;
             clearDisplaySpecificData(): void;
+            clearSettingsBakedData(): void;
         } & {
             clearAllRpcData(): void;
+            invalidateSettings(): void;
             reload(): void;
         } & {
             fetchNeeded(_needed: IndexedRegion[]): void;
@@ -874,23 +925,23 @@ export declare function createPerRegionTestEnvironment({ measuresBytes, gate, es
         } & {
             /** every `fetchNeeded` call, as the region list it was handed */
             fetchLog: IndexedRegion[][];
-            loadedData: Map<number, string>;
             /**
-             * What `regionFetchKey` answers. A volatile, not a `control` knob, and
+             * What `zoomFetchKey` answers. A volatile, not a `control` knob, and
              * that is the point: a real display's key reads view state, so the
              * autorun re-runs when it moves. A plain closure value would be memoized
              * by the computed and never invalidate.
              */
             fetchKey: string;
         } & {
+            regionHasData(displayedRegionIndex: number): boolean;
+        } & {
+            /** the fetched payloads, off the foundation's store */
+            readonly loadedData: ReadonlyMap<number, string>;
             readonly gateEnabled: boolean;
             readonly densityTooLarge: boolean;
-            readonly regionFetchKey: string;
-            regionHasData(_displayedRegionIndex: number): boolean;
+            readonly zoomFetchKey: string;
         } & {
             setFetchKey(key: string): void;
-            setLoaded(displayedRegionIndex: number, value: string): void;
-            clearDisplaySpecificData(): void;
         } & {
             fetchNeeded(needed: IndexedRegion[]): Promise<void>;
         } & import("@jbrowse/mobx-state-tree").IStateTreeNode<import("@jbrowse/mobx-state-tree").IModelType<Omit<Omit<Omit<{
@@ -1168,6 +1219,7 @@ export declare function createPerRegionTestEnvironment({ measuresBytes, gate, es
             readonly canRender: boolean;
             readonly rendersCanvas: boolean;
             readonly paintInert: boolean;
+            readonly paintSuperseded: boolean;
         } & {
             readonly painted: boolean;
         } & {
@@ -1201,6 +1253,7 @@ export declare function createPerRegionTestEnvironment({ measuresBytes, gate, es
             readonly awaitingPrerequisite: boolean;
             readonly awaitingDependentData: boolean;
             readonly rpcPropsCacheKey: string;
+            readonly adapterConfigKey: string;
         } & {
             setError(error?: unknown): void;
             setStatusMessage(status?: import("@jbrowse/core/util").RpcStatus): void;
@@ -1221,31 +1274,45 @@ export declare function createPerRegionTestEnvironment({ measuresBytes, gate, es
         } & {
             readonly host: import("@jbrowse/display-kit/regionHost").RegionHost;
             readonly canvasWidthPx: number;
+            readonly settledSubPixelBinBp: number;
             readonly canRender: boolean;
             readonly rendersCanvas: boolean;
             readonly viewportWithinLoadedData: boolean;
             readonly viewportEmpty: boolean;
             readonly fetchSuspended: boolean;
             readonly layoutReady: boolean;
-            readonly regionFetchKey: string;
-            regionHasData(_displayedRegionIndex: number): boolean;
+            readonly zoomFetchKey: string;
+            regionHasData(displayedRegionIndex: number): boolean;
             readonly dataSuperseded: boolean;
             readonly renderBlocks: import("@jbrowse/render-core/renderBlock").RenderBlock[];
+        } & {
+            readonly fetchInputs: import("@jbrowse/display-kit/fetchInputs").FetchInputs;
+            readonly settingsFetchInputs: unknown;
+        } & {
+            readonly regionPayloads: ReadonlyMap<number, unknown>;
+            readonly hasRegionData: boolean;
         } & {
             isCacheValid(displayedRegionIndex: number): boolean;
         } & {
+            readonly staleSettingsDrawn: boolean;
             readonly dataCurrent: boolean;
             readonly loadedAssembly: import("@jbrowse/core/assemblyManager/assembly").Assembly | undefined;
+        } & {
+            readonly phaseViewportCurrent: boolean;
         } & {
             readonly svgReady: boolean;
             readonly paintInert: boolean;
+            readonly paintSuperseded: boolean;
             readonly displayPhase: import("@jbrowse/render-core/displayPhase").DisplayPhase;
         } & {
-            setLoadedRegion(displayedRegionIndex: number, region: import("@jbrowse/core/util").Region, fetchKey?: string): void;
+            setLoadedRegion(displayedRegionIndex: number, region: import("@jbrowse/core/util").Region, payload: import("@jbrowse/display-kit/regionCommit").RegionPayload | undefined, fetchInputs?: import("@jbrowse/display-kit/fetchInputs").FetchInputs): void;
+            evictRegionStore(keep: ReadonlySet<number>): void;
             dropLoadedRegion(displayedRegionIndex: number): void;
             clearDisplaySpecificData(): void;
+            clearSettingsBakedData(): void;
         } & {
             clearAllRpcData(): void;
+            invalidateSettings(): void;
             reload(): void;
         } & {
             fetchNeeded(_needed: IndexedRegion[]): void;
@@ -1256,23 +1323,23 @@ export declare function createPerRegionTestEnvironment({ measuresBytes, gate, es
         } & {
             /** every `fetchNeeded` call, as the region list it was handed */
             fetchLog: IndexedRegion[][];
-            loadedData: Map<number, string>;
             /**
-             * What `regionFetchKey` answers. A volatile, not a `control` knob, and
+             * What `zoomFetchKey` answers. A volatile, not a `control` knob, and
              * that is the point: a real display's key reads view state, so the
              * autorun re-runs when it moves. A plain closure value would be memoized
              * by the computed and never invalidate.
              */
             fetchKey: string;
         } & {
+            regionHasData(displayedRegionIndex: number): boolean;
+        } & {
+            /** the fetched payloads, off the foundation's store */
+            readonly loadedData: ReadonlyMap<number, string>;
             readonly gateEnabled: boolean;
             readonly densityTooLarge: boolean;
-            readonly regionFetchKey: string;
-            regionHasData(_displayedRegionIndex: number): boolean;
+            readonly zoomFetchKey: string;
         } & {
             setFetchKey(key: string): void;
-            setLoaded(displayedRegionIndex: number, value: string): void;
-            clearDisplaySpecificData(): void;
         } & {
             fetchNeeded(needed: IndexedRegion[]): Promise<void>;
         }, import("@jbrowse/mobx-state-tree")._NotCustomized, import("@jbrowse/mobx-state-tree")._NotCustomized>>;

package/esm/displayKitTests/perRegionTestEnv.js

@@ -28,44 +28,49 @@ function makeStateModel(control, gate, configSchema) {
         .volatile(() => ({
         /** every `fetchNeeded` call, as the region list it was handed */
         fetchLog: [],
-        loadedData: new Map(),
         /**
-         * What `regionFetchKey` answers. A volatile, not a `control` knob, and
+         * What `zoomFetchKey` answers. A volatile, not a `control` knob, and
          * that is the point: a real display's key reads view state, so the
          * autorun re-runs when it moves. A plain closure value would be memoized
          * by the computed and never invalidate.
          */
         fetchKey: '',
     }))
+        .views(self => {
+        // chained, not replaced: `staleAnswers` forces a false answer for a
+        // bounded number of calls, and everything else is the foundation's own
+        // store-derived default
+        const superRegionHasData = self.regionHasData;
+        return {
+            regionHasData(displayedRegionIndex) {
+                control.cacheValidCalls += 1;
+                if (control.staleAnswers > 0) {
+                    control.staleAnswers -= 1;
+                    return false;
+                }
+                return superRegionHasData.call(self, displayedRegionIndex);
+            },
+        };
+    })
         .views(self => ({
+        /** the fetched payloads, off the foundation's store */
+        get loadedData() {
+            return self.regionPayloads;
+        },
         get gateEnabled() {
             return gate.gateEnabled ?? false;
         },
         get densityTooLarge() {
             return control.densityTooLarge;
         },
-        get regionFetchKey() {
+        get zoomFetchKey() {
             return self.fetchKey;
         },
-        regionHasData(_displayedRegionIndex) {
-            control.cacheValidCalls += 1;
-            if (control.staleAnswers > 0) {
-                control.staleAnswers -= 1;
-                return false;
-            }
-            return true;
-        },
     }))
         .actions(self => ({
         setFetchKey(key) {
             self.fetchKey = key;
         },
-        setLoaded(displayedRegionIndex, value) {
-            self.loadedData.set(displayedRegionIndex, value);
-        },
-        clearDisplaySpecificData() {
-            self.loadedData.clear();
-        },
     }))
         .actions(self => ({
         fetchNeeded(needed) {
@@ -90,17 +95,15 @@ function makeStateModel(control, gate, configSchema) {
                     // budget means no measurement at all, which is why `estimateCalls`
                     // stays at zero for an ungated display.
                     if (byteLimit === undefined) {
-                        return { value: 'data' };
+                        return { value: control.payloadTag };
                     }
                     control.estimateCalls += 1;
                     const bytes = control.estimateBytes;
                     return bytes > byteLimit
                         ? { regionTooLarge: true, bytes }
-                        : { value: 'data', bytes };
-                },
-                onResult: (idx, result) => {
-                    self.setLoaded(idx, result.value);
+                        : { value: control.payloadTag, bytes };
                 },
+                onResult: (_idx, result) => result.value,
             });
         },
     }));
@@ -118,6 +121,7 @@ export function createPerRegionTestEnvironment({ measuresBytes = false, gate, es
         estimateBytes,
         estimateCalls: 0,
         densityTooLarge: false,
+        payloadTag: 'data',
     };
     const optIns = gate ?? { gateEnabled: measuresBytes };
     const env = createDisplayTestEnvironment({

package/esm/index.d.ts

@@ -4,19 +4,6 @@ export default class LinearGenomeViewPlugin extends Plugin {
     name: string;
     exports: {
         baseLinearDisplayConfigSchema: import("@jbrowse/core/configuration/configurationSchema").ConfigurationSchemaType<{
-            readonly densityTier: {
-                readonly type: 'stringEnum';
-                readonly model: import("@jbrowse/mobx-state-tree").ISimpleType<"auto" | "density" | "features">;
-                readonly defaultValue: 'auto';
-                readonly description: 'when to draw the features-per-bin density band in place of features: "auto" swaps to it where the region is too large to fetch, "features" never does and keeps the banner, "density" always does. Needs a density source on the adapter (its densityAdapter slot)';
-                readonly advanced: true;
-            };
-            readonly densityTierBpPerPx: {
-                readonly type: 'number';
-                readonly defaultValue: 0;
-                readonly description: 'in "auto" mode, also draw the density band from this many bp per pixel outward, before the region is too large to fetch; 0 leaves the swap to the fetch-size gate alone';
-                readonly advanced: true;
-            };
             readonly fetchSizeLimit: {
                 readonly type: 'number';
                 readonly defaultValue: 1000000;
@@ -29,12 +16,6 @@ export default class LinearGenomeViewPlugin extends Plugin {
                 readonly description: 'Declarative equivalent of the "Force load" button on the "too much data" banner: when true the display always renders, however large the region or dense the features. Off by default (the gate guards against huge downloads). Set it on a view no one can interact with — an embedded / notebook view, or a screenshot — where the region is known and you want it drawn without a click.';
                 readonly advanced: true;
             };
-            readonly maxFeatureScreenDensity: {
-                readonly type: "number";
-                readonly description: "maximum features per pixel before showing a \"too many features\" message";
-                readonly defaultValue: 1;
-                readonly advanced: true;
-            };
             readonly height: {
                 readonly type: "number";
                 readonly defaultValue: 100;
@@ -105,7 +86,6 @@ export { default as HeaderSearchBoxRow } from './LinearGenomeView/components/Hea
 export { searchBoxMenuItems, useSearchBoxPrefs, } from './LinearGenomeView/components/useSearchBoxPrefs.ts';
 export type { SearchBoxPrefs } from './LinearGenomeView/components/useSearchBoxPrefs.ts';
 export { normalizeTrackInit } from '@jbrowse/core/util/tracks';
-export { linearGenomeViewPropKeys, partitionLaunchKeys, } from './LinearGenomeView/initKeys.ts';
 export { applyInitHighlights } from './LinearGenomeView/afterAttach.ts';
 export { MultiLevelRubberband } from './MultiLevelRubberband/index.ts';
 export { fetchResults, SearchResultsNotFoundError } from './searchUtils.ts';
@@ -115,6 +95,7 @@ export type { ScalebarRefNameLabel } from './LinearGenomeView/util.ts';
 export type { RegionsOrientation } from './LinearGenomeView/util.ts';
 export { makeTicks } from './LinearGenomeView/util.ts';
 export type { Tick } from './LinearGenomeView/util.ts';
+export { containingLgv } from './LinearGenomeView/containingLgv.ts';
 export { showRegionsWithUndo } from './LinearGenomeView/showRegionsWithUndo.ts';
 export type { SharedFit } from './LinearGenomeView/sharedScaleContainer.ts';
 export { fitAllRegionsWindow } from './LinearGenomeView/util.ts';

package/esm/index.js

@@ -85,7 +85,6 @@ export { default as HeaderSearchBoxes } from './LinearGenomeView/components/Head
 export { default as HeaderSearchBoxRow } from './LinearGenomeView/components/HeaderSearchBoxRow.js';
 export { searchBoxMenuItems, useSearchBoxPrefs, } from './LinearGenomeView/components/useSearchBoxPrefs.js';
 export { normalizeTrackInit } from '@jbrowse/core/util/tracks';
-export { linearGenomeViewPropKeys, partitionLaunchKeys, } from './LinearGenomeView/initKeys.js';
 export { applyInitHighlights } from './LinearGenomeView/afterAttach.js';
 export { MultiLevelRubberband } from './MultiLevelRubberband/index.js';
 export { fetchResults, SearchResultsNotFoundError } from './searchUtils.js';
@@ -99,6 +98,7 @@ export { makeTicks } from './LinearGenomeView/util.js';
 // The two halves of "show these regions instead": navigate the view you are
 // looking at, or seed a snapshot for a new one. A caller deriving either itself
 // lands on a viewport property the model no longer reads — see each for how.
+export { containingLgv } from './LinearGenomeView/containingLgv.js';
 export { showRegionsWithUndo } from './LinearGenomeView/showRegionsWithUndo.js';
 export { fitAllRegionsWindow } from './LinearGenomeView/util.js';
 export { renderToSvg } from './LinearGenomeView/svgcomponents/SVGLinearGenomeView.js';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jbrowse/plugin-linear-genome-view",
-  "version": "5.0.0-beta.2",
+  "version": "5.0.0-beta.3",
   "type": "module",
   "description": "JBrowse 2 linear genome view",
   "keywords": [
@@ -26,10 +26,10 @@
     "@mui/material": "^9.4.0",
     "mobx": "^7.0.3",
     "mobx-react": "^10.0.2",
-    "@jbrowse/core": "^5.0.0-beta.2",
-    "@jbrowse/render-core": "^5.0.0-beta.2",
-    "@jbrowse/display-ui": "^5.0.0-beta.2",
-    "@jbrowse/display-kit": "^5.0.0-beta.2"
+    "@jbrowse/core": "^5.0.0-beta.3",
+    "@jbrowse/display-kit": "^5.0.0-beta.3",
+    "@jbrowse/display-ui": "^5.0.0-beta.3",
+    "@jbrowse/render-core": "^5.0.0-beta.3"
   },
   "peerDependencies": {
     "react": ">=19.0.0",
@@ -40,7 +40,7 @@
   },
   "sideEffects": false,
   "devDependencies": {
-    "@jbrowse/display-test-utils": "^5.0.0-beta.2"
+    "@jbrowse/display-test-utils": "^5.0.0-beta.3"
   },
   "scripts": {
     "prebuild": "pnpm clean",

package/esm/LinearGenomeView/initKeys.d.ts

@@ -1,12 +0,0 @@
-export declare function partitionLaunchKeys(spec: object, viewPropKeys: ReadonlySet<string>): {
-    init: Record<string, unknown>;
-    viewProps: Record<string, unknown>;
-    unknown: Record<string, unknown>;
-};
-export declare function linearGenomeViewPropKeys(pluginManager: {
-    getViewType: (name: string) => {
-        stateModel: {
-            properties: object;
-        };
-    };
-}): ReadonlySet<string>;

package/esm/LinearGenomeView/initKeys.js

@@ -1,68 +0,0 @@
-import { lgvLaunchKeys } from './launchKeys.js';
-// The remaining v4 partition, still used by the synteny view to build its LGV
-// rows. `lgvLaunchKeys` is the one declaration of what a launch key is, so the
-// two cannot disagree while both exist.
-const knownInitKeys = new Set(Object.keys(lgvLaunchKeys.keys));
-// Never settable from a spec, whatever the model declares: `type` picks the
-// view, `id` is passed top-level so MST's optional identifier honors it, and
-// `launch` is the blob the partition fills rather than something anyone writes.
-const RESERVED = new Set(['id', 'type', 'launch']);
-const LEGACY_VIEWPORT_PROPS = new Set(lgvLaunchKeys.passThrough);
-// Partition launch keys three ways, once, for every caller that needs to know
-// which is which:
-//
-// - `init`: resolution keys (loc, tracks, highlight, …) with no direct MST
-//   representation, which afterAttach applies on attach and then discards
-// - `viewProps`: plain persisted props MST restores natively, so they belong on
-//   the view snapshot rather than in the blob
-// - `unknown`: neither, i.e. a typo. MST silently drops unknown snapshot keys and
-//   `init` is a frozen blob, so nothing else would notice
-//
-// The two callers differ only in what they do with each bucket: the launcher
-// spreads a flat spec across the new view's snapshot and warns about the typos,
-// while afterAttach — which only ever sees an already-built blob — warns about
-// both a typo and a view prop that ended up inside it.
-export function partitionLaunchKeys(spec, 
-// The view's declared property names. Callers with a live node read it off
-// `getPropertyMembers(self).properties`; callers building a snapshot read it
-// off the registered view type's `stateModel.properties` — see
-// `linearGenomeViewPropKeys`.
-viewPropKeys) {
-    const init = {};
-    const viewProps = {};
-    const unknown = {};
-    for (const [key, value] of Object.entries(spec)) {
-        const bucket = knownInitKeys.has(key)
-            ? init
-            : (viewPropKeys.has(key) || LEGACY_VIEWPORT_PROPS.has(key)) &&
-                !RESERVED.has(key)
-                ? viewProps
-                : unknown;
-        bucket[key] = value;
-    }
-    return { init, viewProps, unknown };
-}
-// The registered LinearGenomeView's declared property names, for a caller that
-// has a PluginManager but no view yet (a launcher, or a synteny row being built).
-// A caller holding a live node reads the same thing off
-// `getPropertyMembers(self).properties` instead.
-//
-// `getViewType` THROWS on an unregistered name, and that is left alone: this is
-// only ever called from code that is about to add a LinearGenomeView, so an
-// absent view type is a broken plugin manager rather than a case to degrade
-// through — and degrading would mean silently dropping every property the
-// caller asked for, which is the failure this whole mechanism exists to end.
-//
-// Cached: the set is fixed for the life of the manager, and a launch that opens
-// a stack of rows would otherwise rebuild it per row.
-const propKeyCache = new WeakMap();
-export function linearGenomeViewPropKeys(pluginManager) {
-    const cached = propKeyCache.get(pluginManager);
-    if (cached) {
-        return cached;
-    }
-    const { stateModel } = pluginManager.getViewType('LinearGenomeView');
-    const keys = new Set(Object.keys(stateModel.properties));
-    propKeyCache.set(pluginManager, keys);
-    return keys;
-}