react-msaview 5.6.3 → 5.7.1

package/src/treeMetadata.test.ts

@@ -0,0 +1,39 @@
+import { expect, test, vi } from 'vitest'
+
+import stateModelFactory from './model.ts'
+
+const MsaView = stateModelFactory()
+
+const msa = '>seq1\nACGT\n>seq2\nACCT\n'
+
+test('treeMetadata is parsed and keyed by row name', () => {
+  const model = MsaView.create({ type: 'MsaView' })
+  model.setData({
+    msa,
+    treeMetadata: JSON.stringify({ seq1: { genome: 'Human' } }),
+  })
+  expect(model.treeMetadata.seq1?.genome).toBe('Human')
+  expect(model.treeMetadata.seq2).toBeUndefined()
+  expect(model.getRowData('seq1').treeMetadata).toEqual({ genome: 'Human' })
+})
+
+test('malformed treeMetadata degrades to empty rather than throwing', () => {
+  const model = MsaView.create({ type: 'MsaView' })
+  const spy = vi.spyOn(console, 'error').mockImplementation(() => {})
+  model.setData({ msa, treeMetadata: '{not json' })
+
+  // treeMetadata comes from a user-supplied file, and consumers of this
+  // computed run during layout/render (labelWidthMap measures every label), so
+  // a parse failure must not throw out of them
+  expect(model.treeMetadata).toEqual({})
+  expect(() => model.getRowData('seq1')).not.toThrow()
+  expect(spy).toHaveBeenCalled()
+  spy.mockRestore()
+})
+
+test('non-object treeMetadata degrades to empty', () => {
+  const model = MsaView.create({ type: 'MsaView' })
+  model.setData({ msa, treeMetadata: 'null' })
+  expect(model.treeMetadata).toEqual({})
+  expect(() => model.getRowData('seq1')).not.toThrow()
+})

package/src/useCanvasAutorun.ts

@@ -1,8 +1,9 @@
 import { useEffect, useRef } from 'react'
-import type React from 'react'
 
 import { autorun } from 'mobx'
 
+import type React from 'react'
+
 // Wires a canvas ref to a mobx autorun: grabs the 2d context once the canvas
 // mounts and re-runs `draw` whenever any observable it reads changes. Collapses
 // the getContext/null-guard/autorun-dispose boilerplate repeated by every
@@ -22,7 +23,7 @@ export function useCanvasAutorun(
           draw(ctx)
         })
       : undefined
-    // eslint-disable-next-line @eslint-react/exhaustive-deps
+    // eslint-disable-next-line react-hooks/exhaustive-deps
   }, deps)
   return ref
 }

package/src/version.ts

@@ -1 +1 @@
-export const version = '5.6.3'
+export const version = '5.7.1'