@jbrowse/core 2.13.0 → 2.14.0

package/data_adapters/dataAdapterCache.d.ts

@@ -9,13 +9,16 @@ interface AdapterCacheEntry {
     sessionIds: Set<string>;
 }
 /**
- * instantiate a data adapter, or return an already-instantiated one if we have one with the same
- * configuration
+ * instantiate a data adapter, or return an already-instantiated one if we have
+ * one with the same configuration
  *
- * @param pluginManager -
- * @param sessionId - session ID of the associated worker session.
- *   used for reference counting
- * @param adapterConfigSnapshot - plain-JS configuration snapshot for the adapter
+ * @param pluginManager
+ *
+ * @param sessionId - session ID of the associated worker session. used for
+ * reference counting
+ *
+ * @param adapterConfigSnapshot - plain-JS configuration snapshot for the
+ * adapter
  */
 export declare function getAdapter(pluginManager: PluginManager, sessionId: string, adapterConfigSnapshot: SnapshotIn<AnyConfigurationSchemaType>): Promise<AdapterCacheEntry>;
 /**

package/data_adapters/dataAdapterCache.js

@@ -13,13 +13,16 @@ function adapterConfigCacheKey(conf = {}) {
 }
 let adapterCache = {};
 /**
- * instantiate a data adapter, or return an already-instantiated one if we have one with the same
- * configuration
+ * instantiate a data adapter, or return an already-instantiated one if we have
+ * one with the same configuration
  *
- * @param pluginManager -
- * @param sessionId - session ID of the associated worker session.
- *   used for reference counting
- * @param adapterConfigSnapshot - plain-JS configuration snapshot for the adapter
+ * @param pluginManager
+ *
+ * @param sessionId - session ID of the associated worker session. used for
+ * reference counting
+ *
+ * @param adapterConfigSnapshot - plain-JS configuration snapshot for the
+ * adapter
  */
 async function getAdapter(pluginManager, sessionId, adapterConfigSnapshot) {
     // cache the adapter object
@@ -43,9 +46,6 @@ async function getAdapter(pluginManager, sessionId, adapterConfigSnapshot) {
         //
         const { AdapterClass, getAdapterClass } = dataAdapterType;
         const CLASS = AdapterClass || (await getAdapterClass());
-        if (!CLASS) {
-            throw new Error('Failed to get adapter');
-        }
         const dataAdapter = new CLASS(adapterConfig, getSubAdapter, pluginManager);
         // store it in our cache
         adapterCache[cacheKey] = {
@@ -57,12 +57,11 @@ async function getAdapter(pluginManager, sessionId, adapterConfigSnapshot) {
     cacheEntry.sessionIds.add(sessionId);
     return cacheEntry;
 }
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
 function freeAdapterResources(specification) {
     let deleteCount = 0;
     const specKeys = Object.keys(specification);
-    // if we don't specify a range, delete any adapters that are
-    // only associated with that session
+    // if we don't specify a range, delete any adapters that are only associated
+    // with that session
     if (specKeys.length === 1 && specKeys[0] === 'sessionId') {
         const { sessionId } = specification;
         Object.entries(adapterCache).forEach(([cacheKey, cacheEntry]) => {
@@ -76,16 +75,11 @@ function freeAdapterResources(specification) {
     else {
         // otherwise call freeResources on all the cached data adapters
         Object.values(adapterCache).forEach(cacheEntry => {
-            if (!cacheEntry.dataAdapter.freeResources) {
-                console.warn(cacheEntry.dataAdapter, 'does not implement freeResources');
-            }
-            else {
-                const regions = specification.regions ||
-                    (specification.region ? [specification.region] : []);
-                for (const region of regions) {
-                    if (region.refName !== undefined) {
-                        cacheEntry.dataAdapter.freeResources(region);
-                    }
+            const regions = specification.regions ||
+                (specification.region ? [specification.region] : []);
+            for (const region of regions) {
+                if (region.refName !== undefined) {
+                    cacheEntry.dataAdapter.freeResources(region);
                 }
             }
         });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jbrowse/core",
-  "version": "2.13.0",
+  "version": "2.14.0",
   "description": "JBrowse 2 core libraries used by plugins",
   "keywords": [
     "jbrowse",
@@ -20,7 +20,7 @@
   "author": "JBrowse Team",
   "main": "index.js",
   "scripts": {
-    "test": "cd ../..; jest packages/core",
+    "test": "cd ../..; jest --passWithNoTests packages/core",
     "storybook": "storybook dev --port 6006",
     "storybook:build": "storybook build",
     "coverage": "yarn test --coverage",
@@ -73,5 +73,5 @@
     "access": "public",
     "directory": "dist"
   },
-  "gitHead": "efe02fb9616e20e929cceeed46eb96420c1b6923"
+  "gitHead": "9fb8231d932db40adf0a283081765431756c66ff"
 }

package/pluggableElementTypes/AddTrackWorkflowType.js

@@ -9,12 +9,6 @@ class AddTrackWorkflow extends PluggableElementBase_1.default {
         super(stuff);
         this.ReactComponent = stuff.ReactComponent;
         this.stateModel = stuff.stateModel;
-        if (!this.ReactComponent) {
-            throw new Error(`no ReactComponent defined for view ${this.name}`);
-        }
-        if (!this.stateModel) {
-            throw new Error(`no stateModel defined for view ${this.name}`);
-        }
     }
 }
 exports.default = AddTrackWorkflow;

package/pluggableElementTypes/ConnectionType.js

@@ -12,12 +12,6 @@ class ConnectionType extends PluggableElementBase_1.default {
         this.description = stuff.description;
         this.url = stuff.url;
         this.configEditorComponent = stuff.configEditorComponent;
-        if (!this.stateModel) {
-            throw new Error(`no stateModel defined for connection ${this.name}`);
-        }
-        if (!this.configSchema) {
-            throw new Error(`no configSchema defined for connection ${this.name}`);
-        }
     }
 }
 exports.default = ConnectionType;

package/pluggableElementTypes/DisplayType.js

@@ -4,7 +4,6 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 const PluggableElementBase_1 = __importDefault(require("./PluggableElementBase"));
-const mst_reflection_1 = require("../util/mst-reflection");
 class DisplayType extends PluggableElementBase_1.default {
     constructor(stuff) {
         super(stuff);
@@ -14,25 +13,6 @@ class DisplayType extends PluggableElementBase_1.default {
         this.ReactComponent = stuff.ReactComponent;
         this.trackType = stuff.trackType;
         this.viewType = stuff.viewType;
-        if (!this.stateModel) {
-            throw new Error(`no stateModel defined for display ${this.name}`);
-        }
-        if (!this.configSchema) {
-            throw new Error(`no configSchema provided for display ${this.name}`);
-        }
-        if (!this.ReactComponent) {
-            throw new Error(`no ReactComponent provided for display ${this.name}`);
-        }
-        if (!this.trackType) {
-            throw new Error(`no trackType provided for display ${this.name}`);
-        }
-        if (!this.viewType) {
-            throw new Error(`no viewType provided for display ${this.name}`);
-        }
-        if (!(0, mst_reflection_1.getDefaultValue)(this.configSchema).type) {
-            const name = this.configSchema ? this.configSchema.name : 'UNKNOWN';
-            throw new Error(`${name} is not explicitlyTyped`);
-        }
     }
 }
 exports.default = DisplayType;

package/pluggableElementTypes/InternetAccountType.js

@@ -4,22 +4,11 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 const PluggableElementBase_1 = __importDefault(require("./PluggableElementBase"));
-const mst_reflection_1 = require("../util/mst-reflection");
 class InternetAccountType extends PluggableElementBase_1.default {
     constructor(stuff) {
         super(stuff);
         this.stateModel = stuff.stateModel;
         this.configSchema = stuff.configSchema;
-        if (!this.stateModel) {
-            throw new Error(`no stateModel defined for internet account type ${this.name}`);
-        }
-        if (!this.configSchema) {
-            throw new Error(`no configSchema provided for internet account type ${this.name}`);
-        }
-        if (!(0, mst_reflection_1.getDefaultValue)(this.configSchema).type) {
-            const name = this.configSchema ? this.configSchema.name : 'UNKNOWN';
-            throw new Error(`${name} is not explicitlyTyped`);
-        }
     }
 }
 exports.default = InternetAccountType;

package/pluggableElementTypes/RpcMethodType.d.ts

@@ -10,7 +10,7 @@ export type RpcMethodConstructor = new (pm: PluginManager) => RpcMethodType;
 export default abstract class RpcMethodType extends PluggableElementBase {
     pluginManager: PluginManager;
     constructor(pluginManager: PluginManager);
-    serializeArguments(args: {}, rpcDriverClassName: string): Promise<{}>;
+    serializeArguments(args: Record<string, unknown>, rpcDriverClassName: string): Promise<Record<string, unknown>>;
     serializeNewAuthArguments(loc: UriLocation, rpcDriverClassName: string): Promise<UriLocation>;
     deserializeArguments<T extends SerializedArgs>(serializedArgs: T, _rpcDriverClassName: string): Promise<T & {
         signal: AbortSignal | undefined;

package/pluggableElementTypes/TextSearchAdapterType.js

@@ -10,9 +10,6 @@ class TextSearchAdapterType extends PluggableElementBase_1.default {
         this.description = stuff.description;
         this.configSchema = stuff.configSchema;
         this.AdapterClass = stuff.AdapterClass;
-        if (!this.AdapterClass) {
-            throw new Error(`no AdapterClass defined for adapter type ${this.name}`);
-        }
     }
 }
 exports.default = TextSearchAdapterType;

package/pluggableElementTypes/TrackType.js

@@ -4,23 +4,12 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 const PluggableElementBase_1 = __importDefault(require("./PluggableElementBase"));
-const mst_reflection_1 = require("../util/mst-reflection");
 class TrackType extends PluggableElementBase_1.default {
     constructor(stuff) {
         super(stuff);
         this.displayTypes = [];
         this.stateModel = stuff.stateModel;
         this.configSchema = stuff.configSchema;
-        if (!this.stateModel) {
-            throw new Error(`no stateModel defined for track ${this.name}`);
-        }
-        if (!this.configSchema) {
-            throw new Error(`no configSchema provided for track ${this.name}`);
-        }
-        if (!(0, mst_reflection_1.getDefaultValue)(this.configSchema).type) {
-            const name = this.configSchema ? this.configSchema.name : 'UNKNOWN';
-            throw new Error(`${name} is not explicitlyTyped`);
-        }
     }
     addDisplayType(display) {
         this.displayTypes.push(display);

package/pluggableElementTypes/ViewType.js

@@ -11,12 +11,6 @@ class ViewType extends PluggableElementBase_1.default {
         this.ReactComponent = stuff.ReactComponent;
         this.stateModel = stuff.stateModel;
         this.extendedName = stuff.extendedName;
-        if (!this.ReactComponent) {
-            throw new Error(`no ReactComponent defined for view ${this.name}`);
-        }
-        if (!this.stateModel) {
-            throw new Error(`no stateModel defined for view ${this.name}`);
-        }
     }
     addDisplayType(display) {
         this.displayTypes.push(display);

package/pluggableElementTypes/WidgetType.js

@@ -12,12 +12,6 @@ class WidgetType extends PluggableElementBase_1.default {
         this.configSchema = stuff.configSchema;
         this.stateModel = stuff.stateModel;
         this.ReactComponent = stuff.ReactComponent;
-        if (!this.ReactComponent) {
-            throw new Error(`no ReactComponent defined for widget ${this.name}`);
-        }
-        if (!this.stateModel) {
-            throw new Error(`no stateModel defined for widget ${this.name}`);
-        }
     }
 }
 exports.default = WidgetType;

package/pluggableElementTypes/models/BaseConnectionModelFactory.d.ts

@@ -1,5 +1,6 @@
 import { AnyConfigurationModel } from '../../configuration';
 import PluginManager from '../../PluginManager';
+type TrackConf = AnyConfigurationModel | Record<string, unknown>;
 /**
  * #stateModel BaseConnectionModel
  */
@@ -37,15 +38,15 @@ declare function stateModelFactory(pluginManager: PluginManager): import("mobx-s
     /**
      * #action
      */
-    addTrackConf(trackConf: AnyConfigurationModel): any;
+    addTrackConf(trackConf: TrackConf): any;
     /**
      * #action
      */
-    addTrackConfs(trackConfs: AnyConfigurationModel[]): any[];
+    addTrackConfs(trackConfs: TrackConf[]): void;
     /**
      * #action
      */
-    setTrackConfs(trackConfs: AnyConfigurationModel[]): import("mobx-state-tree").IMSTArray<import("mobx-state-tree").IAnyModelType> & import("mobx-state-tree").IStateTreeNode<import("mobx-state-tree").IArrayType<import("mobx-state-tree").IAnyModelType>>;
+    setTrackConfs(trackConfs: AnyConfigurationModel[]): void;
     /**
      * #action
      */

package/pluggableElementTypes/models/BaseConnectionModelFactory.js

@@ -48,15 +48,13 @@ function stateModelFactory(pluginManager) {
          * #action
          */
         addTrackConfs(trackConfs) {
-            const length = self.tracks.push(...trackConfs);
-            return self.tracks.slice(length - 1 - trackConfs.length, length - 1);
+            self.tracks.push(...trackConfs);
         },
         /**
          * #action
          */
         setTrackConfs(trackConfs) {
             self.tracks = (0, mobx_state_tree_1.cast)(trackConfs);
-            return self.tracks;
         },
         /**
          * #action

package/pluggableElementTypes/models/BaseDisplayModel.d.ts

@@ -49,7 +49,7 @@ export declare const BaseDisplay: import("mobx-state-tree").IModelType<{
             error: unknown;
             message: string | undefined;
         }, import("mobx-state-tree")._NotCustomized, import("mobx-state-tree")._NotCustomized>>;
-        onHorizontalScroll?: Function;
+        onHorizontalScroll?: () => void;
         blockState?: Record<string, any>;
     }>;
     /**
@@ -99,14 +99,13 @@ export declare const BaseDisplay: import("mobx-state-tree").IModelType<{
     renderProps(): any;
     /**
      * #getter
-     * the pluggable element type object for this display's
-     * renderer
+     * the pluggable element type object for this display's renderer
      */
     readonly rendererType: import("..").RendererType;
     /**
      * #getter
-     * if a display-level message should be displayed instead,
-     * make this return a react component
+     * if a display-level message should be displayed instead, make this
+     * return a react component
      */
     readonly DisplayMessageComponent: undefined | React.FC<any>;
     /**

package/pluggableElementTypes/models/BaseDisplayModel.js

@@ -38,8 +38,8 @@ function stateModelFactory() {
          */
         get RenderingComponent() {
             const { pluginManager } = (0, util_1.getEnv)(self);
-            const displayType = pluginManager.getDisplayType(self.type);
-            return displayType.ReactComponent;
+            return pluginManager.getDisplayType(self.type)
+                .ReactComponent;
         },
         /**
          * #getter
@@ -81,24 +81,16 @@ function stateModelFactory() {
         },
         /**
          * #getter
-         * the pluggable element type object for this display's
-         * renderer
+         * the pluggable element type object for this display's renderer
          */
         get rendererType() {
             const { pluginManager } = (0, util_1.getEnv)(self);
-            const RendererType = pluginManager.getRendererType(self.rendererTypeName);
-            if (!RendererType) {
-                throw new Error(`renderer "${self.rendererTypeName}" not found`);
-            }
-            if (!RendererType.ReactComponent) {
-                throw new Error(`renderer ${self.rendererTypeName} has no ReactComponent, it may not be completely implemented yet`);
-            }
-            return RendererType;
+            return pluginManager.getRendererType(self.rendererTypeName);
         },
         /**
          * #getter
-         * if a display-level message should be displayed instead,
-         * make this return a react component
+         * if a display-level message should be displayed instead, make this
+         * return a react component
          */
         get DisplayMessageComponent() {
             return undefined;

package/pluggableElementTypes/models/BaseTrackModel.js

@@ -55,8 +55,7 @@ function createBaseTrackModel(pm, trackType, baseTrackConfig) {
          * determines which webworker to send the track to, currently based on trackId
          */
         get rpcSessionId() {
-            var _a;
-            return (_a = self.configuration) === null || _a === void 0 ? void 0 : _a.trackId;
+            return self.configuration.trackId;
         },
         /**
          * #getter
@@ -98,7 +97,7 @@ function createBaseTrackModel(pm, trackType, baseTrackConfig) {
             const { sessionTracks, adminMode } = session;
             return ((0, types_1.isSessionModelWithConfigEditing)(session) &&
                 (adminMode ||
-                    (sessionTracks === null || sessionTracks === void 0 ? void 0 : sessionTracks.find(t => { var _a; return t.trackId === ((_a = self.configuration) === null || _a === void 0 ? void 0 : _a.trackId); }))));
+                    (sessionTracks === null || sessionTracks === void 0 ? void 0 : sessionTracks.find(t => t.trackId === self.configuration.trackId))));
         },
     }))
         .actions(self => ({
@@ -132,7 +131,9 @@ function createBaseTrackModel(pm, trackType, baseTrackConfig) {
             const schema = pm.pluggableConfigSchemaType('display');
             const conf = (0, mobx_state_tree_1.resolveIdentifier)(schema, (0, mobx_state_tree_1.getRoot)(self), displayId);
             const t = self.displays.filter(d => d.configuration === conf);
-            (0, mobx_1.transaction)(() => t.forEach(d => self.displays.remove(d)));
+            (0, mobx_1.transaction)(() => {
+                t.forEach(d => self.displays.remove(d));
+            });
             return t.length;
         },
         /**
@@ -175,7 +176,9 @@ function createBaseTrackModel(pm, trackType, baseTrackConfig) {
                                 type: 'radio',
                                 label: pm.getDisplayType(d.type).displayName,
                                 checked: d.displayId === shownId,
-                                onClick: () => self.replaceDisplay(shownId, d.displayId),
+                                onClick: () => {
+                                    self.replaceDisplay(shownId, d.displayId);
+                                },
                             })),
                         },
                     ]

package/pluggableElementTypes/models/InternetAccountModel.d.ts

@@ -147,7 +147,7 @@ export declare const InternetAccount: import("mobx-state-tree").IModelType<{
     /**
      * #action
      */
-    addAuthHeaderToInit(init: RequestInit | undefined, token: string): {
+    addAuthHeaderToInit(init?: RequestInit, token?: string): {
         headers: Headers;
         body?: BodyInit | null;
         cache?: RequestCache;

package/pluggableElementTypes/models/InternetAccountModel.js

@@ -96,7 +96,7 @@ exports.InternetAccount = mobx_state_tree_1.types
      * @returns true or false
      */
     handlesLocation(location) {
-        return self.domains.some(domain => location === null || location === void 0 ? void 0 : location.uri.includes(domain));
+        return self.domains.some(domain => location.uri.includes(domain));
     },
     /**
      * #getter
@@ -183,13 +183,15 @@ exports.InternetAccount = mobx_state_tree_1.types
                 tokenPromise = Promise.resolve(token);
                 return tokenPromise;
             }
-            tokenPromise = new Promise((resolve, reject) => self.getTokenFromUser(token => {
-                self.storeToken(token);
-                resolve(token);
-            }, error => {
-                self.removeToken();
-                reject(error);
-            }));
+            tokenPromise = new Promise((resolve, reject) => {
+                self.getTokenFromUser(token => {
+                    self.storeToken(token);
+                    resolve(token);
+                }, error => {
+                    self.removeToken();
+                    reject(error);
+                });
+            });
             return tokenPromise;
         },
     };
@@ -198,14 +200,18 @@ exports.InternetAccount = mobx_state_tree_1.types
     /**
      * #action
      */
-    addAuthHeaderToInit(init = {}, token) {
+    addAuthHeaderToInit(init, token) {
         return {
             ...init,
             headers: new Headers({
-                ...init.headers,
-                [self.authHeader]: self.tokenType
-                    ? `${self.tokenType} ${token}`
-                    : token,
+                ...init === null || init === void 0 ? void 0 : init.headers,
+                ...(token
+                    ? {
+                        [self.authHeader]: self.tokenType
+                            ? `${self.tokenType} ${token}`
+                            : token,
+                    }
+                    : {}),
             }),
         };
     },

package/pluggableElementTypes/models/baseTrackConfig.js

@@ -140,7 +140,7 @@ function createBaseTrackConfig(pluginManager) {
                 // Gets the displays on the track snapshot and the possible displays
                 // from the track type and adds any missing possible displays to the
                 // snapshot
-                const configDisplayTypes = new Set(displays.filter(d => !!d).map(d => d.type));
+                const configDisplayTypes = new Set(displays.map(d => d.type));
                 pluginManager.getTrackType(snap.type).displayTypes.forEach(d => {
                     if (!configDisplayTypes.has(d.name)) {
                         displays.push({
@@ -157,16 +157,13 @@ function createBaseTrackConfig(pluginManager) {
          */
         explicitIdentifier: 'trackId',
         explicitlyTyped: true,
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
         actions: (self) => ({
             addDisplayConf(conf) {
                 const { type } = conf;
                 if (!type) {
                     throw new Error(`unknown display type ${type}`);
                 }
-                const display = self.displays.find(
-                // eslint-disable-next-line @typescript-eslint/no-explicit-any
-                (d) => (d === null || d === void 0 ? void 0 : d.displayId) === conf.displayId);
+                const display = self.displays.find((d) => (d === null || d === void 0 ? void 0 : d.displayId) === conf.displayId);
                 if (display) {
                     return display;
                 }

package/pluggableElementTypes/renderers/BoxRendererType.d.ts

@@ -10,18 +10,18 @@ import RpcManager from '../../rpc/RpcManager';
 export interface LayoutSessionProps {
     config: AnyConfigurationModel;
     bpPerPx: number;
-    filters: SerializableFilterChain;
+    filters?: SerializableFilterChain;
 }
 export type MyMultiLayout = MultiLayout<GranularRectLayout<unknown>, unknown>;
 export interface CachedLayout {
     layout: MyMultiLayout;
     config: AnyConfigurationModel;
-    filters: SerializableFilterChain;
+    filters?: SerializableFilterChain;
 }
 export declare class LayoutSession implements LayoutSessionProps {
     config: AnyConfigurationModel;
     bpPerPx: number;
-    filters: SerializableFilterChain;
+    filters?: SerializableFilterChain;
     constructor(args: LayoutSessionProps);
     update(props: LayoutSessionProps): void;
     makeLayout(): MultiLayout<GranularRectLayout<unknown>, unknown>;
@@ -61,7 +61,13 @@ export default class BoxRendererType extends FeatureRendererType {
         sessionId: string;
         layoutId: string;
     }): LayoutSession;
-    getExpandedRegion(region: Region, renderArgs: RenderArgsDeserialized): Region;
+    getExpandedRegion(region: Region, renderArgs: RenderArgsDeserialized): {
+        start: number;
+        end: number;
+        refName: string;
+        reversed?: boolean | undefined;
+        assemblyName: string;
+    };
     createSession(props: LayoutSessionProps): LayoutSession;
     freeResourcesInClient(rpcManager: RpcManager, args: RenderArgs): Promise<number>;
     deserializeLayoutInClient(json: SerializedLayout): PrecomputedLayout<unknown>;

package/pluggableElementTypes/renderers/BoxRendererType.js

@@ -36,8 +36,7 @@ class LayoutSession {
      * @returns true if the given layout is a valid one to use for this session
      */
     cachedLayoutIsValid(cachedLayout) {
-        return (cachedLayout &&
-            cachedLayout.layout.subLayoutConstructorArgs.pitchX === this.bpPerPx &&
+        return (cachedLayout.layout.subLayoutConstructorArgs.pitchX === this.bpPerPx &&
             (0, fast_deep_equal_1.default)((0, configuration_1.readConfObject)(this.config), cachedLayout.config) &&
             (0, fast_deep_equal_1.default)(this.filters, cachedLayout.filters));
     }
@@ -67,18 +66,9 @@ class BoxRendererType extends FeatureRendererType_1.default {
         session.update(props);
         return session;
     }
-    // expands region for glyphs to use
     getExpandedRegion(region, renderArgs) {
-        if (!region) {
-            return region;
-        }
         const { bpPerPx, config } = renderArgs;
-        const maxFeatureGlyphExpansion = config === undefined
-            ? 0
-            : (0, configuration_1.readConfObject)(config, 'maxFeatureGlyphExpansion');
-        if (!maxFeatureGlyphExpansion) {
-            return region;
-        }
+        const maxFeatureGlyphExpansion = (0, configuration_1.readConfObject)(config, 'maxFeatureGlyphExpansion') || 0;
         const bpExpansion = Math.round(maxFeatureGlyphExpansion * bpPerPx);
         return {
             ...region,
@@ -92,16 +82,12 @@ class BoxRendererType extends FeatureRendererType_1.default {
     async freeResourcesInClient(rpcManager, args) {
         const { regions } = args;
         const key = (0, util_1.getLayoutId)(args);
-        let freed = 0;
         const session = this.sessions[key];
-        if (!regions && session) {
-            delete this.sessions[key];
-            freed = 1;
-        }
-        if (session && regions) {
-            session.layout.discardRange(regions[0].refName, regions[0].start, regions[0].end);
+        if (session) {
+            const region = regions[0];
+            session.layout.discardRange(region.refName, region.start, region.end);
         }
-        return freed + (await super.freeResourcesInClient(rpcManager, args));
+        return await super.freeResourcesInClient(rpcManager, args);
     }
     deserializeLayoutInClient(json) {
         return new PrecomputedLayout_1.default(json);
@@ -117,13 +103,11 @@ class BoxRendererType extends FeatureRendererType_1.default {
     }
     serializeResultsInWorker(results, args) {
         const serialized = super.serializeResultsInWorker(results, args);
-        const [region] = args.regions;
+        const region = args.regions[0];
         serialized.layout = results.layout.serializeRegion(this.getExpandedRegion(region, args));
-        if (serialized.layout.rectangles) {
-            serialized.features = serialized.features.filter(f => {
-                return Boolean(serialized.layout.rectangles[f.uniqueId]);
-            });
-        }
+        serialized.features = serialized.features.filter(f => {
+            return Boolean(serialized.layout.rectangles[f.uniqueId]);
+        });
         serialized.maxHeightReached = serialized.layout.maxHeightReached;
         return serialized;
     }

package/pluggableElementTypes/renderers/CircularChordRendererType.d.ts

@@ -8,7 +8,7 @@ export default class CircularChordRendererType extends FeatureRenderer {
         features: SimpleFeatureSerialized[];
         html: string;
     }, args: {
-        displayModel: any;
+        displayModel?: any;
         theme: ThemeOptions;
         exportSVG?: {
             rasterizeLayers?: boolean;