jbrowse-plugin-msaview 2.6.8 → 2.7.1

package/dist/version.d.ts

@@ -1 +1 @@
-export declare const version = "2.6.8";
+export declare const version = "2.7.1";

package/dist/version.js

@@ -1 +1 @@
-export const version = '2.6.8';
+export const version = '2.7.1';

package/package.json

@@ -1,5 +1,5 @@
 {
-  "version": "2.6.8",
+  "version": "2.7.1",
   "license": "MIT",
   "name": "jbrowse-plugin-msaview",
   "repository": {
@@ -20,7 +20,6 @@
     "@gmod/bgzf-filehandle": "^6.2.0",
     "g2p_mapper": "^2.1.5",
     "idb": "^8.0.3",
-    "react-msaview": "^5.5.0",
     "swr": "^2.4.2"
   },
   "devDependencies": {
@@ -28,39 +27,38 @@
     "@eslint/js": "^10.0.1",
     "@fal-works/esbuild-plugin-global-externals": "^2.1.2",
     "@jbrowse/core": "^4.3.0",
-    "@jbrowse/mobx-state-tree": "^5.11.1",
+    "@jbrowse/mobx-state-tree": "^5.13.0",
     "@jbrowse/plugin-linear-genome-view": "^4.3.0",
-    "@mui/icons-material": "^7.3.11",
-    "@mui/material": "^7.3.11",
-    "@mui/system": "^7.3.11",
-    "@mui/x-data-grid": "^8.29.1",
-    "@types/node": "^25.9.4",
+    "@mui/icons-material": "^9.2.0",
+    "@mui/material": "^9.2.0",
+    "@mui/system": "^9.2.0",
+    "@mui/x-data-grid": "^9.10.0",
+    "@types/node": "^26.1.1",
     "@types/react": "^19.2.17",
-    "@typescript-eslint/eslint-plugin": "^8.62.0",
-    "@typescript-eslint/parser": "^8.62.0",
     "esbuild": "^0.28.1",
-    "eslint": "^10.5.0",
-    "eslint-plugin-import-x": "^4.17.0",
+    "eslint": "^10.7.0",
+    "eslint-plugin-import-x": "^4.17.1",
     "eslint-plugin-react": "^7.37.5",
     "eslint-plugin-react-hooks": "^7.1.1",
-    "eslint-plugin-unicorn": "^64.0.0",
+    "eslint-plugin-unicorn": "^72.0.0",
     "mobx": "^6.16.1",
     "mobx-react": "^9.2.2",
-    "msa-parsers": "^5.5.0",
+    "msa-parsers": "^5.6.1",
     "pixelmatch": "^7.2.0",
     "pngjs": "^7.0.0",
-    "prettier": "^3.8.4",
-    "pretty-bytes": "^7.1.0",
-    "puppeteer": "^25.2.1",
-    "react": "^19.2.7",
-    "react-dom": "^19.2.7",
+    "prettier": "^3.9.6",
+    "pretty-bytes": "^7.1.1",
+    "puppeteer": "^25.3.0",
+    "react": "^19.2.8",
+    "react-dom": "^19.2.8",
+    "react-msaview": "^5.6.3",
     "rimraf": "^6.1.3",
     "rxjs": "^7.8.2",
     "serve": "^14.2.6",
     "tss-react": "^4.9.21",
-    "typescript": "^6.0.3",
-    "typescript-eslint": "^8.62.0",
-    "vitest": "^4.1.9"
+    "typescript": "^6.0.2",
+    "typescript-eslint": "^8.65.0",
+    "vitest": "^4.1.10"
   },
   "scripts": {
     "clean": "rimraf dist",

package/src/LaunchMsaView/index.ts

@@ -14,45 +14,70 @@ function isDisplay(elt: { name: string }): elt is DisplayType {
   return elt.name === 'LinearBasicDisplay'
 }
 
+// The canvas LinearBasicDisplay (JBrowse >=4.3) exposes the right-clicked
+// feature via contextMenuInfo + async fetchFullFeature rather than a synchronous
+// feature object.
+interface ContextMenuInfo {
+  item: { featureId: string; type?: string }
+  displayedRegionIndex: number
+}
+
+interface DisplayModel {
+  contextMenuItems: () => MenuItem[]
+  contextMenuInfo?: ContextMenuInfo
+  isGeneLike: boolean
+  fetchFullFeature: (
+    featureId: string,
+    displayedRegionIndex: number,
+  ) => Promise<Feature | undefined>
+}
+
 function extendStateModel(stateModel: IAnyModelType) {
-  return stateModel.views(
-    (self: {
-      contextMenuItems: () => MenuItem[]
-      contextMenuFeature?: Feature
-    }) => {
-      const superContextMenuItems = self.contextMenuItems
-      return {
-        contextMenuItems() {
-          const feature = self.contextMenuFeature
-          const track = getContainingTrack(self)
-          const featureType = feature?.get('type')
-          const showMsaMenuItem =
-            feature && ['gene', 'mRNA', 'transcript'].includes(featureType)
-          return [
-            ...superContextMenuItems(),
-            ...(showMsaMenuItem
-              ? [
-                  {
-                    label: 'Launch MSA view',
-                    icon: AddIcon,
-                    onClick: () => {
-                      getSession(track).queueDialog(handleClose => [
-                        LaunchMsaViewDialog,
-                        {
-                          model: track,
-                          handleClose,
-                          feature,
-                        },
-                      ])
-                    },
+  return stateModel.views((self: DisplayModel) => {
+    const superContextMenuItems = self.contextMenuItems
+    return {
+      contextMenuItems() {
+        const track = getContainingTrack(self)
+        const session = getSession(track)
+        const info = self.contextMenuInfo
+        const showMsaMenuItem = info && self.isGeneLike
+        return [
+          ...superContextMenuItems(),
+          ...(showMsaMenuItem
+            ? [
+                {
+                  label: 'Launch MSA view',
+                  icon: AddIcon,
+                  onClick: () => {
+                    self
+                      .fetchFullFeature(
+                        info.item.featureId,
+                        info.displayedRegionIndex,
+                      )
+                      .then(feature => {
+                        if (feature) {
+                          session.queueDialog(handleClose => [
+                            LaunchMsaViewDialog,
+                            { model: track, handleClose, feature },
+                          ])
+                        } else {
+                          session.notify(
+                            'Could not load feature for MSA view',
+                            'warning',
+                          )
+                        }
+                      })
+                      .catch((e: unknown) => {
+                        session.notifyError(`${e}`, e)
+                      })
                   },
-                ]
-              : []),
-          ]
-        },
-      }
-    },
-  )
+                },
+              ]
+            : []),
+        ]
+      },
+    }
+  })
 }
 
 export default function LaunchMsaViewF(pluginManager: PluginManager) {

package/src/MsaViewPanel/model.ts

@@ -77,6 +77,11 @@ export default function stateModelFactory() {
         /**
          * #property
          */
+        // Plain defaults, not types.stripDefault: that postdates the
+        // mobx-state-tree every released core exposes (present only on main), and
+        // the missing function throws while this model is being built, which
+        // error-pages the whole app rather than just this view. Restore
+        // stripDefault once a release ships it.
         querySeqName: 'QUERY',
 
         /**
@@ -87,6 +92,7 @@ export default function stateModelFactory() {
         /**
          * #property
          */
+        // see querySeqName above re: types.stripDefault
         zoomToBaseLevel: false,
 
         /**
@@ -207,9 +213,10 @@ export default function stateModelFactory() {
        * #getter
        */
       get connectedHighlights(): IRegion[] {
-        return [this.connectedHoverHighlight, this.connectedClickHighlight].filter(
-          (r): r is IRegion => r !== undefined,
-        )
+        return [
+          this.connectedHoverHighlight,
+          this.connectedClickHighlight,
+        ].filter((r): r is IRegion => r !== undefined)
       },
     }))
 
@@ -312,7 +319,6 @@ export default function stateModelFactory() {
           connectedView.centerAt(r2.start, r2.refName)
         }
       },
-
     }))
     .actions(self => {
       const superSetMouseClickPos = self.setMouseClickPos.bind(self)

package/src/version.ts

@@ -1 +1 @@
-export const version = '2.6.8'
+export const version = '2.7.1'