@jbrowse/plugin-alignments 2.16.1 → 2.18.0

package/esm/PileupRPC/methods/GetReducedFeatures.js

@@ -1,12 +1,10 @@
 import { getAdapter } from '@jbrowse/core/data_adapters/dataAdapterCache';
 import { dedupe, groupBy } from '@jbrowse/core/util';
-import { toArray } from 'rxjs/operators';
 import { firstValueFrom } from 'rxjs';
-// locals
-import { filterForPairs, getInsertSizeStats } from '../util';
+import { toArray } from 'rxjs/operators';
+import { getClip } from '../../MismatchParser';
 import PileupBaseRPC from '../base';
-import { getTag } from '../../util';
-// specialized get features to return limited data about alignments
+import { filterForPairs, getInsertSizeStats } from '../util';
 export default class PileupGetReducedFeatures extends PileupBaseRPC {
     constructor() {
         super(...arguments);
@@ -17,21 +15,24 @@ export default class PileupGetReducedFeatures extends PileupBaseRPC {
         const { adapterConfig, sessionId, regions } = des;
         const dataAdapter = (await getAdapter(this.pluginManager, sessionId, adapterConfig)).dataAdapter;
         const featuresArray = await firstValueFrom(dataAdapter.getFeaturesInMultipleRegions(regions, des).pipe(toArray()));
-        const reduced = dedupe(featuresArray.map(f => ({
-            id: f.id(),
-            refName: f.get('refName'),
-            name: f.get('name'),
-            start: f.get('start'),
-            strand: f.get('strand'),
-            end: f.get('end'),
-            flags: f.get('flags'),
-            tlen: f.get('template_length'),
-            pair_orientation: f.get('pair_orientation'),
-            next_ref: f.get('next_ref'),
-            next_pos: f.get('next_pos'),
-            clipPos: f.get('clipPos'),
-            SA: getTag(f, 'SA'),
-        })), f => f.id);
+        const reduced = dedupe(featuresArray.map(f => {
+            var _a;
+            return ({
+                id: f.id(),
+                refName: f.get('refName'),
+                name: f.get('name'),
+                start: f.get('start'),
+                strand: f.get('strand'),
+                end: f.get('end'),
+                flags: f.get('flags'),
+                tlen: f.get('template_length'),
+                pair_orientation: f.get('pair_orientation'),
+                next_ref: f.get('next_ref'),
+                next_pos: f.get('next_pos'),
+                clipPos: getClip(f.get('CIGAR'), f.get('strand')),
+                SA: (_a = f.get('tags')) === null || _a === void 0 ? void 0 : _a.SA,
+            });
+        }), f => f.id);
         const filtered = filterForPairs(reduced);
         const stats = filtered.length ? getInsertSizeStats(filtered) : undefined;
         const chains = groupBy(reduced, f => f.name);

package/esm/PileupRPC/methods/GetVisibleModifications.d.ts

@@ -1,14 +1,14 @@
-import { RemoteAbortSignal } from '@jbrowse/core/rpc/remoteAbortSignals';
-import { Region } from '@jbrowse/core/util';
 import PileupBaseRPC from '../base';
+import type { ModificationType } from '../../shared/types';
+import type { Region } from '@jbrowse/core/util';
 export default class PileupGetVisibleModifications extends PileupBaseRPC {
     name: string;
     execute(args: {
         adapterConfig: Record<string, unknown>;
-        signal?: RemoteAbortSignal;
+        stopToken?: string;
         headers?: Record<string, string>;
         regions: Region[];
         sessionId: string;
         tag: string;
-    }, rpcDriver: string): Promise<string[]>;
+    }, rpcDriver: string): Promise<ModificationType[]>;
 }

package/esm/PileupRPC/methods/GetVisibleModifications.js

@@ -1,10 +1,9 @@
 import { getAdapter } from '@jbrowse/core/data_adapters/dataAdapterCache';
-import { toArray } from 'rxjs/operators';
 import { firstValueFrom } from 'rxjs';
-// locals
-import { getModificationTypes } from '../../MismatchParser';
-import PileupBaseRPC from '../base';
+import { toArray } from 'rxjs/operators';
+import { getModTypes } from '../../ModificationParser';
 import { getTagAlt } from '../../util';
+import PileupBaseRPC from '../base';
 export default class PileupGetVisibleModifications extends PileupBaseRPC {
     constructor() {
         super(...arguments);
@@ -14,10 +13,14 @@ export default class PileupGetVisibleModifications extends PileupBaseRPC {
         const { adapterConfig, sessionId, regions } = await this.deserializeArguments(args, rpcDriver);
         const dataAdapter = (await getAdapter(this.pluginManager, sessionId, adapterConfig)).dataAdapter;
         const featuresArray = await firstValueFrom(dataAdapter.getFeaturesInMultipleRegions(regions).pipe(toArray()));
-        const uniqueValues = new Set();
+        const uniqueModifications = new Map();
         featuresArray.forEach(f => {
-            getModificationTypes(getTagAlt(f, 'MM', 'Mm') || '').forEach(t => uniqueValues.add(t));
+            for (const mod of getModTypes(getTagAlt(f, 'MM', 'Mm') || '')) {
+                if (!uniqueModifications.has(mod.type)) {
+                    uniqueModifications.set(mod.type, mod);
+                }
+            }
         });
-        return [...uniqueValues];
+        return [...uniqueModifications.values()];
     }
 }

package/esm/PileupRPC/util.d.ts

@@ -1,4 +1,4 @@
-import { ReducedFeature } from '../shared/fetchChains';
+import type { ReducedFeature } from '../shared/fetchChains';
 export declare function getInsertSizeStats(features: ReducedFeature[]): {
     upper: number;
     lower: number;

package/esm/PileupRenderer/PileupLayoutSession.d.ts

@@ -1,28 +1,29 @@
 import { LayoutSession } from '@jbrowse/core/pluggableElementTypes/renderers/BoxRendererType';
-import { AnyConfigurationModel } from '@jbrowse/core/configuration';
-import SerializableFilterChain from '@jbrowse/core/pluggableElementTypes/renderers/util/serializableFilterChain';
-import GranularRectLayout from '@jbrowse/core/util/layouts/GranularRectLayout';
-import MultiLayout from '@jbrowse/core/util/layouts/MultiLayout';
+import type { FilterBy, SortedBy } from '../shared/types';
+import type { AnyConfigurationModel } from '@jbrowse/core/configuration';
+import type SerializableFilterChain from '@jbrowse/core/pluggableElementTypes/renderers/util/serializableFilterChain';
+import type GranularRectLayout from '@jbrowse/core/util/layouts/GranularRectLayout';
+import type MultiLayout from '@jbrowse/core/util/layouts/MultiLayout';
 export interface PileupLayoutSessionProps {
     config: AnyConfigurationModel;
     bpPerPx: number;
     filters: SerializableFilterChain;
-    filterBy: unknown;
-    sortedBy: unknown;
-    showSoftClip: unknown;
+    filterBy: FilterBy;
+    sortedBy: SortedBy;
+    showSoftClip: boolean;
 }
 type MyMultiLayout = MultiLayout<GranularRectLayout<unknown>, unknown>;
 interface CachedPileupLayout {
     layout: MyMultiLayout;
     config: AnyConfigurationModel;
     filters?: SerializableFilterChain;
-    filterBy: unknown;
-    sortedBy: unknown;
+    filterBy?: FilterBy;
+    sortedBy?: SortedBy;
     showSoftClip: boolean;
 }
 export declare class PileupLayoutSession extends LayoutSession {
-    sortedBy: unknown;
-    filterBy: unknown;
+    sortedBy?: SortedBy;
+    filterBy?: FilterBy;
     showSoftClip: boolean;
     constructor(args: PileupLayoutSessionProps);
     cachedLayoutIsValid(cachedLayout: CachedPileupLayout): boolean;

package/esm/PileupRenderer/PileupLayoutSession.js

@@ -1,9 +1,6 @@
-import deepEqual from 'fast-deep-equal';
-import { LayoutSession } from '@jbrowse/core/pluggableElementTypes/renderers/BoxRendererType';
 import { readConfObject } from '@jbrowse/core/configuration';
-// The pileup layout session adds
-// - sorting and revealing soft clip changes the layout of pileup renderer
-// - extra conditions to see if cached layout is valid
+import { LayoutSession } from '@jbrowse/core/pluggableElementTypes/renderers/BoxRendererType';
+import deepEqual from 'fast-deep-equal';
 export class PileupLayoutSession extends LayoutSession {
     constructor(args) {
         super(args);

package/esm/PileupRenderer/PileupRenderer.d.ts

@@ -1,37 +1,16 @@
-import BoxRendererType, { RenderArgsDeserialized as BoxRenderArgsDeserialized } from '@jbrowse/core/pluggableElementTypes/renderers/BoxRendererType';
-import { Feature, Region } from '@jbrowse/core/util';
-import { BaseLayout } from '@jbrowse/core/util/layouts/BaseLayout';
-import { PileupLayoutSession, PileupLayoutSessionProps } from './PileupLayoutSession';
-export interface RenderArgsDeserialized extends BoxRenderArgsDeserialized {
-    colorBy?: {
-        type: string;
-        tag?: string;
-    };
-    colorTagMap?: Record<string, string>;
-    modificationTagMap?: Record<string, string>;
-    sortedBy?: {
-        type: string;
-        pos: number;
-        refName: string;
-        assemblyName: string;
-        tag?: string;
-    };
-    showSoftClip: boolean;
-    highResolutionScaling: number;
-}
-export interface RenderArgsDeserializedWithFeaturesAndLayout extends RenderArgsDeserialized {
-    features: Map<string, Feature>;
-    layout: BaseLayout<Feature>;
-    regionSequence?: string;
-}
+import BoxRendererType from '@jbrowse/core/pluggableElementTypes/renderers/BoxRendererType';
+import { PileupLayoutSession } from './PileupLayoutSession';
+import type { PileupLayoutSessionProps } from './PileupLayoutSession';
+import type { RenderArgsDeserialized } from './types';
+import type { Region } from '@jbrowse/core/util';
 export default class PileupRenderer extends BoxRendererType {
     supportsSVG: boolean;
-    fetchSequence(renderProps: RenderArgsDeserialized): Promise<any>;
+    fetchSequence(renderProps: RenderArgsDeserialized, region: Region): Promise<any>;
     getExpandedRegion(region: Region, renderArgs: RenderArgsDeserialized): {
         start: number;
         end: number;
-        refName: string;
         reversed?: boolean | undefined;
+        refName: string;
         assemblyName: string;
     };
     render(renderProps: RenderArgsDeserialized): Promise<{
@@ -40,6 +19,7 @@ export default class PileupRenderer extends BoxRendererType {
         height: number;
         width: number;
         maxHeightReached: boolean;
+        containsNoTransferables: boolean;
         canvasRecordedData: any;
         reactElement?: import("react").ReactElement;
         html?: string;
@@ -49,6 +29,7 @@ export default class PileupRenderer extends BoxRendererType {
         height: number;
         width: number;
         maxHeightReached: boolean;
+        containsNoTransferables: boolean;
         reactElement: React.JSX.Element;
         html?: string;
     } | {
@@ -57,10 +38,11 @@ export default class PileupRenderer extends BoxRendererType {
         height: number;
         width: number;
         maxHeightReached: boolean;
+        containsNoTransferables: boolean;
         imageData: any;
         reactElement?: import("react").ReactElement;
         html?: string;
     }>;
     createSession(args: PileupLayoutSessionProps): PileupLayoutSession;
 }
-export { type RenderArgs, type RenderResults, type RenderArgsSerialized, type ResultsSerialized, type ResultsDeserialized, } from '@jbrowse/core/pluggableElementTypes/renderers/BoxRendererType';
+export type { RenderArgs, RenderArgsSerialized, RenderResults, ResultsDeserialized, ResultsSerialized, } from '@jbrowse/core/pluggableElementTypes/renderers/BoxRendererType';

package/esm/PileupRenderer/PileupRenderer.js

@@ -1,26 +1,27 @@
-import BoxRendererType from '@jbrowse/core/pluggableElementTypes/renderers/BoxRendererType';
-import { notEmpty, renderToAbstractCanvas, } from '@jbrowse/core/util';
 import { readConfObject } from '@jbrowse/core/configuration';
 import { getAdapter } from '@jbrowse/core/data_adapters/dataAdapterCache';
-import { PileupLayoutSession, } from './PileupLayoutSession';
-// locals
-import { fetchSequence, shouldFetchReferenceSequence } from '../util';
+import BoxRendererType from '@jbrowse/core/pluggableElementTypes/renderers/BoxRendererType';
+import { renderToAbstractCanvas } from '@jbrowse/core/util';
+import { PileupLayoutSession } from './PileupLayoutSession';
+import { fetchSequence } from '../util';
 import { layoutFeats } from './layoutFeatures';
 export default class PileupRenderer extends BoxRendererType {
     constructor() {
         super(...arguments);
         this.supportsSVG = true;
     }
-    async fetchSequence(renderProps) {
-        const { sessionId, regions, adapterConfig } = renderProps;
+    async fetchSequence(renderProps, region) {
+        const { sessionId, adapterConfig } = renderProps;
         const { sequenceAdapter } = adapterConfig;
         if (!sequenceAdapter) {
             return undefined;
         }
-        const pm = this.pluginManager;
-        const { dataAdapter } = await getAdapter(pm, sessionId, sequenceAdapter);
-        const region = regions[0];
-        return fetchSequence(region, dataAdapter);
+        const { dataAdapter } = await getAdapter(this.pluginManager, sessionId, sequenceAdapter);
+        return fetchSequence({
+            ...region,
+            start: Math.max(0, region.start - 1),
+            end: region.end + 1,
+        }, dataAdapter);
     }
     getExpandedRegion(region, renderArgs) {
         const { config, showSoftClip } = renderArgs;
@@ -28,35 +29,30 @@ export default class PileupRenderer extends BoxRendererType {
         const maxClippingSize = readConfObject(config, 'maxClippingSize');
         const bpExpansion = showSoftClip ? Math.round(maxClippingSize) : 0;
         return {
-            // xref https://github.com/mobxjs/mobx-state-tree/issues/1524 for Omit
             ...region,
             start: Math.floor(Math.max(start - bpExpansion, 0)),
             end: Math.ceil(end + bpExpansion),
         };
     }
     async render(renderProps) {
-        var _a;
         const features = await this.getFeatures(renderProps);
         const layout = this.createLayoutInWorker(renderProps);
-        const { regions, bpPerPx } = renderProps;
+        const { colorBy, regions, bpPerPx } = renderProps;
         const region = regions[0];
-        const layoutRecords = layoutFeats({
+        const regionSequence = (colorBy === null || colorBy === void 0 ? void 0 : colorBy.type) === 'methylation' && features.size
+            ? await this.fetchSequence(renderProps, region)
+            : undefined;
+        const { layoutRecords, height } = layoutFeats({
             ...renderProps,
             features,
             layout,
         });
-        // only need reference sequence if there are features and only for some
-        // cases
-        const regionSequence = features.size && shouldFetchReferenceSequence((_a = renderProps.colorBy) === null || _a === void 0 ? void 0 : _a.type)
-            ? await this.fetchSequence(renderProps)
-            : undefined;
         const width = (region.end - region.start) / bpPerPx;
-        const height = Math.max(layout.getTotalHeight(), 1);
         const { makeImageData } = await import('./makeImageData');
         const res = await renderToAbstractCanvas(width, height, renderProps, ctx => {
             makeImageData({
                 ctx,
-                layoutRecords: layoutRecords.filter(notEmpty),
+                layoutRecords,
                 canvasWidth: width,
                 renderArgs: {
                     ...renderProps,
@@ -83,6 +79,7 @@ export default class PileupRenderer extends BoxRendererType {
             height,
             width,
             maxHeightReached: layout.maxHeightReached,
+            containsNoTransferables: true,
         };
     }
     createSession(args) {

package/esm/PileupRenderer/colorBy.d.ts

@@ -1,5 +1,5 @@
-import { AnyConfigurationModel } from '@jbrowse/core/configuration';
-import { Feature } from '@jbrowse/core/util';
+import type { AnyConfigurationModel } from '@jbrowse/core/configuration';
+import type { Feature } from '@jbrowse/core/util';
 export declare function colorByInsertSize(feature: Feature): string;
 export declare function colorByMappingQuality(feature: Feature): string;
 export declare function colorByStrand(feature: Feature): "#EC8B8B" | "#8F8FD8";

package/esm/PileupRenderer/colorBy.js

@@ -1,4 +1,4 @@
-import { readConfObject, } from '@jbrowse/core/configuration';
+import { readConfObject } from '@jbrowse/core/configuration';
 import { fillColor } from '../shared/color';
 import { orientationTypes } from '../util';
 export function colorByInsertSize(feature) {
@@ -30,27 +30,21 @@ export function colorByOrientation(feature, config) {
 function getStranded(feature) {
     const flags = feature.get('flags');
     const strand = feature.get('strand');
-    // is paired
     if (flags & 1) {
-        // first-of-pair?
         const flipper = flags & 64 ? -1 : 1;
-        // proper pairing
         if (flags & 2) {
             return strand * flipper === 1 ? 'color_rev_strand' : 'color_fwd_strand';
         }
-        // mate missing, separate color
         if (flags & 8) {
             return strand * flipper === 1
                 ? 'color_rev_missing_mate'
                 : 'color_fwd_missing_mate';
         }
-        // same chrom without proper pairing gets separate color
         if (feature.get('refName') === feature.get('next_ref')) {
             return strand * flipper === 1
                 ? 'color_rev_strand_not_proper'
                 : 'color_fwd_strand_not_proper';
         }
-        // abberant chrom
         return strand === 1 ? 'color_fwd_diff_chr' : 'color_rev_diff_chr';
     }
     return 'color_unknown';

package/esm/PileupRenderer/components/PileupRendering.d.ts

@@ -1,5 +1,5 @@
 import React from 'react';
-import { Region } from '@jbrowse/core/util/types';
+import type { Region } from '@jbrowse/core/util/types';
 import type { BaseLinearDisplayModel } from '@jbrowse/plugin-linear-genome-view';
 declare const PileupRendering: (props: {
     blockKey: string;

package/esm/PileupRenderer/components/PileupRendering.js

@@ -1,4 +1,4 @@
-import React, { useRef, useState, useEffect } from 'react';
+import React, { useEffect, useRef, useState } from 'react';
 import { PrerenderedCanvas } from '@jbrowse/core/ui';
 import { bpSpanPx } from '@jbrowse/core/util';
 import { observer } from 'mobx-react';
@@ -81,9 +81,7 @@ const PileupRendering = observer(function (props) {
         onMouseMove === null || onMouseMove === void 0 ? void 0 : onMouseMove(event, displayModel === null || displayModel === void 0 ? void 0 : displayModel.getFeatureOverlapping(blockKey, clientBp, offsetY));
     }
     function callMouseHandler(handlerName, event) {
-        // @ts-expect-error
         const featureHandler = props[`onFeature${handlerName}`];
-        // @ts-expect-error
         const canvasHandler = props[`on${handlerName}`];
         if (featureHandler && featureIdUnderMouse) {
             featureHandler(event, featureIdUnderMouse);
@@ -93,7 +91,6 @@ const PileupRendering = observer(function (props) {
         }
     }
     const canvasWidth = Math.ceil(width);
-    // need to call this in render so we get the right observer behavior
     return (React.createElement("div", { ref: ref, "data-testid": [
             'pileup-overlay',
             sortedBy === null || sortedBy === void 0 ? void 0 : sortedBy.type,

package/esm/PileupRenderer/configSchema.d.ts

@@ -1,84 +1,53 @@
 declare const PileupRenderer: import("@jbrowse/core/configuration/configurationSchema").ConfigurationSchemaType<{
-    /**
-     * #slot
-     * default magenta here is used to detect the user has not customized this
-     */
     color: {
         type: string;
         description: string;
         defaultValue: string;
         contextVariable: string[];
     };
-    /**
-     * #slot
-     */
     orientationType: {
         type: string;
         model: import("mobx-state-tree").ISimpleType<string>;
         defaultValue: string;
         description: string;
     };
-    /**
-     * #slot
-     */
     displayMode: {
         type: string;
         model: import("mobx-state-tree").ISimpleType<string>;
         description: string;
         defaultValue: string;
     };
-    /**
-     * #slot
-     */
     minSubfeatureWidth: {
         type: string;
         description: string;
         defaultValue: number;
     };
-    /**
-     * #slot
-     */
     maxHeight: {
         type: string;
         description: string;
         defaultValue: number;
     };
-    /**
-     * #slot
-     */
     maxClippingSize: {
         type: string;
         description: string;
         defaultValue: number;
     };
-    /**
-     * #slot
-     */
     height: {
         type: string;
         description: string;
         defaultValue: number;
         contextVariable: string[];
     };
-    /**
-     * #slot
-     */
     noSpacing: {
         type: string;
         description: string;
         defaultValue: boolean;
     };
-    /**
-     * #slot
-     */
     largeInsertionIndicatorScale: {
         type: string;
         description: string;
         defaultValue: number;
     };
-    /**
-     * #slot
-     */
     mismatchAlpha: {
         type: string;
         defaultValue: boolean;

package/esm/PileupRenderer/configSchema.js

@@ -1,32 +1,19 @@
 import { ConfigurationSchema } from '@jbrowse/core/configuration';
 import { types } from 'mobx-state-tree';
-/**
- * #config PileupRenderer
- */
-function x() { } // eslint-disable-line @typescript-eslint/no-unused-vars
+function x() { }
 const PileupRenderer = ConfigurationSchema('PileupRenderer', {
-    /**
-     * #slot
-     * default magenta here is used to detect the user has not customized this
-     */
     color: {
         type: 'color',
         description: 'the color of each feature in a pileup alignment',
         defaultValue: '#f0f',
         contextVariable: ['feature'],
     },
-    /**
-     * #slot
-     */
     orientationType: {
         type: 'stringEnum',
         model: types.enumeration('orientationType', ['fr', 'rf', 'ff']),
         defaultValue: 'fr',
         description: 'read sequencer orientation. fr is normal "reads pointing at each other ---> <--- while some other sequencers can use other options',
     },
-    /**
-     * #slot
-     */
     displayMode: {
         type: 'stringEnum',
         model: types.enumeration('displayMode', [
@@ -37,58 +24,37 @@ const PileupRenderer = ConfigurationSchema('PileupRenderer', {
         description: 'Alternative display modes',
         defaultValue: 'normal',
     },
-    /**
-     * #slot
-     */
     minSubfeatureWidth: {
         type: 'number',
         description: 'the minimum width in px for a pileup mismatch feature. use for increasing/decreasing mismatch marker widths when zoomed out, e.g. 0 or 1',
         defaultValue: 1,
     },
-    /**
-     * #slot
-     */
     maxHeight: {
         type: 'integer',
         description: 'the maximum height to be used in a pileup rendering',
         defaultValue: 1200,
     },
-    /**
-     * #slot
-     */
     maxClippingSize: {
         type: 'integer',
         description: 'the max clip size to be used in a pileup rendering',
         defaultValue: 10000,
     },
-    /**
-     * #slot
-     */
     height: {
         type: 'number',
         description: 'the height of each feature in a pileup alignment',
         defaultValue: 7,
         contextVariable: ['feature'],
     },
-    /**
-     * #slot
-     */
     noSpacing: {
         type: 'boolean',
         description: 'remove spacing between features',
         defaultValue: false,
     },
-    /**
-     * #slot
-     */
     largeInsertionIndicatorScale: {
         type: 'number',
         description: 'scale at which to draw the large insertion indicators (bp/pixel)',
         defaultValue: 10,
     },
-    /**
-     * #slot
-     */
     mismatchAlpha: {
         type: 'boolean',
         defaultValue: false,

package/esm/PileupRenderer/getAlignmentShapeColor.d.ts

@@ -1,5 +1,5 @@
-import { AnyConfigurationModel } from '@jbrowse/core/configuration';
-import { Feature } from '@jbrowse/core/util';
+import type { AnyConfigurationModel } from '@jbrowse/core/configuration';
+import type { Feature } from '@jbrowse/core/util';
 export declare function getAlignmentShapeColor({ colorType, tag, feature, config, defaultColor, colorTagMap, }: {
     colorType: string;
     tag: string;

package/esm/PileupRenderer/getAlignmentShapeColor.js

@@ -1,9 +1,7 @@
-import { readConfObject, } from '@jbrowse/core/configuration';
-import { fillColor } from '../shared/color';
+import { readConfObject } from '@jbrowse/core/configuration';
 import { colorByInsertSize, colorByMappingQuality, colorByOrientation, colorByStrand, colorByStrandedRnaSeq, } from './colorBy';
+import { fillColor } from '../shared/color';
 export function getAlignmentShapeColor({ colorType, tag, feature, config, defaultColor, colorTagMap, }) {
-    // first pass for simple color changes that change the color of the
-    // alignment
     switch (colorType) {
         case 'insertSize':
             return colorByInsertSize(feature);
@@ -51,9 +49,6 @@ export function getAlignmentShapeColor({ colorType, tag, feature, config, defaul
             break;
         case 'modifications':
         case 'methylation':
-            // this coloring is similar to igv.js, and is helpful to color negative
-            // strand reads differently because their c-g will be flipped (e.g. g-c
-            // read right to left)
             return feature.get('flags') & 16 ? '#c8dcc8' : '#c8c8c8';
         default:
             return defaultColor

package/esm/PileupRenderer/index.d.ts

@@ -1,2 +1,2 @@
-import PluginManager from '@jbrowse/core/PluginManager';
+import type PluginManager from '@jbrowse/core/PluginManager';
 export default function register(pluginManager: PluginManager): void;

package/esm/PileupRenderer/index.js

@@ -1,6 +1,6 @@
 import PileupRenderer from './PileupRenderer';
-import configSchema from './configSchema';
 import ReactComponent from './components/PileupRendering';
+import configSchema from './configSchema';
 export default function register(pluginManager) {
     pluginManager.addRendererType(() => {
         return new PileupRenderer({

package/esm/PileupRenderer/layoutFeature.d.ts

@@ -1,5 +1,5 @@
-import { Feature, Region } from '@jbrowse/core/util';
-import { BaseLayout } from '@jbrowse/core/util/layouts';
+import type { Feature, Region } from '@jbrowse/core/util';
+import type { BaseLayout } from '@jbrowse/core/util/layouts';
 export interface LayoutRecord {
     feature: Feature;
     leftPx: number;

package/esm/PileupRenderer/layoutFeature.js

@@ -2,7 +2,6 @@ import { bpSpanPx } from '@jbrowse/core/util';
 export function layoutFeature({ feature, layout, bpPerPx, region, showSoftClip, heightPx, displayMode, }) {
     let expansionBefore = 0;
     let expansionAfter = 0;
-    // Expand the start and end of feature when softclipping enabled
     if (showSoftClip) {
         const mismatches = feature.get('mismatches');
         const seq = feature.get('seq');