@morscherlab/mint-sdk 1.0.0-beta.7 → 1.0.0-rc.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@morscherlab/mint-sdk",
-  "version": "1.0.0-beta.7",
+  "version": "1.0.0-rc.1",
   "description": "MINT Platform SDK — Vue 3 components, composables, and types for plugin development. MINT = Mass-spec INtegrated Toolkit.",
   "type": "module",
   "main": "./dist/index.js",

package/src/__tests__/components/GroupAssigner.test.ts

@@ -27,4 +27,22 @@ describe('GroupAssigner', () => {
     expect(wrapper.emitted('update:group1')?.[0]).toEqual([[]])
     expect(wrapper.emitted('update:group2')).toBeUndefined()
   })
+
+  it('does not throw when drag events omit dataTransfer', async () => {
+    const wrapper = mount(GroupAssigner, {
+      props: {
+        groups,
+        group1: [],
+        group2: [],
+      },
+    })
+
+    await expect(
+      wrapper.find('.mint-group-assigner__pill--unassigned').trigger('dragstart', { dataTransfer: null }),
+    ).resolves.toBeUndefined()
+
+    await expect(
+      wrapper.find('.mint-group-assigner__zone').trigger('dragover', { dataTransfer: null }),
+    ).resolves.toBeUndefined()
+  })
 })

package/src/__tests__/composables/useApi.test.ts

@@ -1,7 +1,16 @@
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest'
 import { createPinia, setActivePinia } from 'pinia'
+import type { InternalAxiosRequestConfig } from 'axios'
 
 import { useApi } from '../../composables/useApi'
+import { useAuthStore } from '../../stores/auth'
+
+function readAuthorizationHeader(headers: InternalAxiosRequestConfig['headers']): unknown {
+  const bag = headers as Record<string, unknown> & { get?: (header: string) => unknown }
+  return (typeof bag.get === 'function' ? bag.get('Authorization') : undefined)
+    ?? bag.Authorization
+    ?? bag.authorization
+}
 
 describe('useApi URL helpers', () => {
   beforeEach(() => {
@@ -9,6 +18,7 @@ describe('useApi URL helpers', () => {
   })
 
   afterEach(() => {
+    vi.restoreAllMocks()
     vi.unstubAllEnvs()
   })
 
@@ -27,4 +37,39 @@ describe('useApi URL helpers', () => {
     expect(api.buildWsUrl('/events')).toMatch(/\/api\/events$/)
     expect(api.buildWsUrl('/events')).not.toContain('/api//events')
   })
+
+  it('normalizes paths that already include the configured API prefix', async () => {
+    const api = useApi({ baseUrl: '/api' })
+    const getSpy = vi.spyOn(api.client, 'get').mockResolvedValue({ data: { ok: true } })
+
+    expect(api.buildUrl('/api/projects')).toBe('/api/projects')
+    expect(api.buildUrl('/api')).toBe('/api')
+    expect(api.buildUrl('/api?active=true')).toBe('/api?active=true')
+
+    await expect(api.get<{ ok: boolean }>('/api/projects')).resolves.toEqual({ ok: true })
+    expect(getSpy).toHaveBeenCalledWith('/projects', expect.objectContaining({ baseURL: '/api' }))
+  })
+
+  it('honors withAuth=false even when the SDK auth store has a token', async () => {
+    const authStore = useAuthStore()
+    authStore.setToken('secret-token', 3600)
+    const api = useApi({ baseUrl: '/api', withAuth: false })
+
+    let authorization: unknown
+    const result = await api.get<{ ok: boolean }>('/public', {
+      adapter: async (config) => {
+        authorization = readAuthorizationHeader(config.headers)
+        return {
+          data: { ok: true },
+          status: 200,
+          statusText: 'OK',
+          headers: {},
+          config,
+        }
+      },
+    })
+
+    expect(result).toEqual({ ok: true })
+    expect(authorization).toBeUndefined()
+  })
 })

package/src/__tests__/composables/useAuth.test.ts

@@ -94,6 +94,26 @@ describe('useAuth', () => {
     })
   })
 
+  describe('storage fallback', () => {
+    it('keeps auth usable when localStorage operations throw', () => {
+      vi.stubGlobal('localStorage', {
+        getItem: vi.fn(() => { throw new Error('blocked') }),
+        setItem: vi.fn(() => { throw new Error('blocked') }),
+        removeItem: vi.fn(() => { throw new Error('blocked') }),
+      })
+
+      const authStore = useAuthStore()
+
+      expect(() => authStore.initialize()).not.toThrow()
+
+      authStore.setToken('memory-token', 3600)
+      expect(authStore.token).toBe('memory-token')
+
+      authStore.clearToken()
+      expect(authStore.token).toBeNull()
+    })
+  })
+
   describe('refreshToken - race condition', () => {
     it('should deduplicate concurrent refresh calls', async () => {
       const authStore = useAuthStore()

package/src/__tests__/composables/useProtocolTemplates.test.ts

@@ -0,0 +1,64 @@
+import { beforeEach, describe, expect, it, vi } from 'vitest'
+import { useProtocolTemplates } from '../../composables/useProtocolTemplates'
+
+const STORAGE_KEY = 'mint-custom-protocol-templates'
+
+function installLocalStorage() {
+  const storage: Record<string, string> = {}
+  vi.stubGlobal('localStorage', {
+    get length() {
+      return Object.keys(storage).length
+    },
+    key: vi.fn((index: number) => Object.keys(storage)[index] ?? null),
+    getItem: vi.fn((key: string) => storage[key] ?? null),
+    setItem: vi.fn((key: string, value: string) => {
+      storage[key] = value
+    }),
+    removeItem: vi.fn((key: string) => {
+      delete storage[key]
+    }),
+    clear: vi.fn(() => {
+      for (const key of Object.keys(storage)) delete storage[key]
+    }),
+  } as unknown as Storage)
+}
+
+describe('useProtocolTemplates', () => {
+  beforeEach(() => {
+    vi.restoreAllMocks()
+    vi.unstubAllGlobals()
+    installLocalStorage()
+  })
+
+  it('ignores corrupted custom template storage', () => {
+    localStorage.setItem(STORAGE_KEY, JSON.stringify({ invalid: true }))
+
+    const templates = useProtocolTemplates()
+
+    expect(templates.customTemplates.value).toEqual([])
+    expect(templates.allTemplates.value.length).toBeGreaterThan(0)
+  })
+
+  it('filters invalid custom template entries', () => {
+    localStorage.setItem(STORAGE_KEY, JSON.stringify([
+      { id: 'bad', name: 'Missing parameters', type: 'custom' },
+      {
+        id: 'custom-valid',
+        name: 'Valid custom step',
+        type: 'custom',
+        parameters: [],
+      },
+    ]))
+
+    const templates = useProtocolTemplates()
+
+    expect(templates.customTemplates.value).toEqual([
+      {
+        id: 'custom-valid',
+        name: 'Valid custom step',
+        type: 'custom',
+        parameters: [],
+      },
+    ])
+  })
+})

package/src/__tests__/stores/settings.test.ts

@@ -0,0 +1,78 @@
+import { beforeEach, describe, expect, it, vi } from 'vitest'
+import { createPinia, setActivePinia } from 'pinia'
+import { useSettingsStore } from '../../stores/settings'
+
+function mockMatchMedia(matches = false) {
+  Object.defineProperty(window, 'matchMedia', {
+    configurable: true,
+    writable: true,
+    value: vi.fn(() => ({
+      matches,
+      media: '(prefers-color-scheme: dark)',
+      onchange: null,
+      addEventListener: vi.fn(),
+      removeEventListener: vi.fn(),
+      addListener: vi.fn(),
+      removeListener: vi.fn(),
+      dispatchEvent: vi.fn(),
+    })),
+  })
+}
+
+function installLocalStorage() {
+  const storage: Record<string, string> = {}
+  vi.stubGlobal('localStorage', {
+    get length() {
+      return Object.keys(storage).length
+    },
+    key: vi.fn((index: number) => Object.keys(storage)[index] ?? null),
+    getItem: vi.fn((key: string) => storage[key] ?? null),
+    setItem: vi.fn((key: string, value: string) => {
+      storage[key] = value
+    }),
+    removeItem: vi.fn((key: string) => {
+      delete storage[key]
+    }),
+    clear: vi.fn(() => {
+      for (const key of Object.keys(storage)) delete storage[key]
+    }),
+  } as unknown as Storage)
+}
+
+describe('useSettingsStore', () => {
+  beforeEach(() => {
+    vi.restoreAllMocks()
+    vi.unstubAllGlobals()
+    setActivePinia(createPinia())
+    installLocalStorage()
+    document.documentElement.classList.remove('dark')
+    mockMatchMedia(false)
+  })
+
+  it('migrates legacy MLD settings into the MINT settings key', () => {
+    localStorage.setItem('mld-settings', JSON.stringify({
+      serverHost: '127.0.0.1',
+      serverPort: 8001,
+      theme: 'dark',
+      colorPalette: 'viridis',
+      tableDensity: 'compact',
+    }))
+
+    const settings = useSettingsStore()
+    settings.initialize()
+
+    expect(settings.serverHost).toBe('127.0.0.1')
+    expect(settings.serverPort).toBe(8001)
+    expect(settings.theme).toBe('dark')
+    expect(settings.colorPalette).toBe('viridis')
+    expect(settings.tableDensity).toBe('compact')
+    expect(localStorage.getItem('mld-settings')).toBeNull()
+    expect(JSON.parse(localStorage.getItem('mint-settings') || '{}')).toEqual(expect.objectContaining({
+      serverHost: '127.0.0.1',
+      serverPort: 8001,
+      theme: 'dark',
+      colorPalette: 'viridis',
+      tableDensity: 'compact',
+    }))
+  })
+})

package/src/components/AppAvatarMenu.vue

@@ -35,9 +35,12 @@ const { isOpen, rootRef, close, toggle } = useDropdownState({
 const initials = computed(() => {
   if (props.userInitial) return props.userInitial.slice(0, 2).toUpperCase()
   if (props.userName) {
-    const parts = props.userName.trim().split(/\s+/)
-    if (parts.length >= 2) return (parts[0]!.charAt(0) + parts[1]!.charAt(0)).toUpperCase()
-    return parts[0]!.slice(0, 2).toUpperCase()
+    const parts = props.userName.trim().split(/\s+/).filter(Boolean)
+    const first = parts[0]
+    if (!first) return 'U'
+    const second = parts[1]
+    if (second) return (first.charAt(0) + second.charAt(0)).toUpperCase()
+    return first.slice(0, 2).toUpperCase()
   }
   return 'U'
 })

package/src/components/AppTopBar.vue

@@ -117,7 +117,12 @@ const { isIntegrated, plugin } = usePlatformContext()
 const isStandalone = computed(() => !isIntegrated.value)
 const appExperiment = inject(APP_EXPERIMENT_KEY, null)
 
-const hasPluginIcon = computed(() => !!plugin.value?.icon)
+const pluginIcon = computed(() => {
+  const currentPlugin = plugin.value
+  return currentPlugin?.icon
+    ? { icon: currentPlugin.icon, color: currentPlugin.color }
+    : null
+})
 
 const profileInitial = computed(() => {
   if (props.userInitial) return props.userInitial
@@ -225,10 +230,10 @@ function currentItemIdFromLocation(pages: Array<Pick<PageSelectorItem, 'id' | 't
         >
           <slot name="icon">
             <PluginIcon
-              v-if="hasPluginIcon"
+              v-if="pluginIcon"
               class="mint-topbar__plugin-icon"
-              :icon="plugin!.icon"
-              :tone="plugin!.color"
+              :icon="pluginIcon.icon"
+              :tone="pluginIcon.color"
               size="md"
               variant="solid"
             />
@@ -248,10 +253,10 @@ function currentItemIdFromLocation(pages: Array<Pick<PageSelectorItem, 'id' | 't
         >
           <slot name="icon">
             <PluginIcon
-              v-if="hasPluginIcon"
+              v-if="pluginIcon"
               class="mint-topbar__plugin-icon"
-              :icon="plugin!.icon"
-              :tone="plugin!.color"
+              :icon="pluginIcon.icon"
+              :tone="pluginIcon.color"
               size="md"
               variant="solid"
             />
@@ -267,10 +272,10 @@ function currentItemIdFromLocation(pages: Array<Pick<PageSelectorItem, 'id' | 't
         <template v-else>
           <slot name="icon">
             <PluginIcon
-              v-if="hasPluginIcon"
+              v-if="pluginIcon"
               class="mint-topbar__plugin-icon"
-              :icon="plugin!.icon"
-              :tone="plugin!.color"
+              :icon="pluginIcon.icon"
+              :tone="pluginIcon.color"
               size="md"
               variant="solid"
             />

package/src/components/AuditTrail.vue

@@ -32,7 +32,7 @@ const uniqueTypes = computed(() => {
 })
 
 const uniqueUsers = computed(() => {
-  const users = new Set(props.entries.filter(e => e.user).map(e => e.user!))
+  const users = new Set(props.entries.flatMap(e => e.user ? [e.user] : []))
   return Array.from(users)
 })
 

package/src/components/Calendar.vue

@@ -134,8 +134,12 @@ const markerMap = computed(() => {
   for (const marker of props.markers) {
     const date = parseDate(marker.date)
     const key = toDateKey(date)
-    if (!map.has(key)) map.set(key, [])
-    map.get(key)!.push(marker)
+    const markers = map.get(key)
+    if (markers) {
+      markers.push(marker)
+    } else {
+      map.set(key, [marker])
+    }
   }
   return map
 })

package/src/components/ConcentrationInput.vue

@@ -37,13 +37,14 @@ const { unitCategories, getConversionHint } = useConcentrationUnits()
 
 // Filter categories based on allowedUnits
 const filteredCategories = computed(() => {
-  if (!props.allowedUnits || props.allowedUnits.length === 0) {
+  const allowedUnits = props.allowedUnits
+  if (!allowedUnits || allowedUnits.length === 0) {
     return unitCategories.value
   }
   return unitCategories.value
     .map(cat => ({
       label: cat.label,
-      units: cat.units.filter(u => props.allowedUnits!.includes(u)),
+      units: cat.units.filter(u => allowedUnits.includes(u)),
     }))
     .filter(cat => cat.units.length > 0)
 })

package/src/components/GroupAssigner.vue

@@ -42,9 +42,12 @@ const assignment = useGroupAssignment({
 const { unassignedGroups, zone1Groups, zone2Groups, zone1Count, zone2Count, validationMessage } = assignment
 
 function handleDragStart(event: DragEvent, groupName: string) {
+  const transfer = event.dataTransfer
+  if (!transfer) return
+
   draggingGroup.value = groupName
-  event.dataTransfer?.setData('groupName', groupName)
-  event.dataTransfer!.effectAllowed = 'move'
+  transfer.setData('groupName', groupName)
+  transfer.effectAllowed = 'move'
 }
 
 function handleDragEnd() {
@@ -54,7 +57,9 @@ function handleDragEnd() {
 
 function handleDragOver(event: DragEvent, zone: GroupAssignmentZone) {
   event.preventDefault()
-  event.dataTransfer!.dropEffect = 'move'
+  if (event.dataTransfer) {
+    event.dataTransfer.dropEffect = 'move'
+  }
   dragOverZone.value = zone
 }
 

package/src/components/NumberInput.vue

@@ -28,10 +28,12 @@ const emit = defineEmits<{
 const isSliderMode = computed(() => props.min !== undefined && props.max !== undefined)
 
 const sliderPercent = computed(() => {
-  if (!isSliderMode.value || props.modelValue === undefined) return 0
-  const range = (props.max! - props.min!)
+  const min = props.min
+  const max = props.max
+  if (min === undefined || max === undefined || props.modelValue === undefined) return 0
+  const range = max - min
   if (range === 0) return 0
-  return ((props.modelValue - props.min!) / range) * 100
+  return ((props.modelValue - min) / range) * 100
 })
 
 const canDecrement = computed(() => {

package/src/components/SampleHierarchyTree.vue

@@ -149,6 +149,7 @@ function canShowChildren(node: TreeNode, depth: number): boolean {
 
 // Render tree node recursively
 function renderNode(node: TreeNode, depth: number): VNode {
+  const children = node.children
   const expanded = isExpanded(node.id)
   const canExpand = hasChildren(node)
   const showChildNodes = canShowChildren(node, depth)
@@ -197,11 +198,11 @@ function renderNode(node: TreeNode, depth: number): VNode {
   )
 
   const childNodes =
-    showChildNodes && node.children
+    showChildNodes && children
       ? h(
           Transition,
           { enterActiveClass: 'mint-sample-tree__children--entering', leaveActiveClass: 'mint-sample-tree__children--leaving' },
-          () => h('div', { class: 'mint-sample-tree__children' }, node.children!.map((child) => renderNode(child, depth + 1)))
+          () => h('div', { class: 'mint-sample-tree__children' }, children.map((child) => renderNode(child, depth + 1)))
         )
       : null
 

package/src/components/SettingsModal.vue

@@ -416,6 +416,13 @@ function isControlModelBinding(model: ControlModel | ControlModelBinding): model
         </div>
       </component>
     </div>
+    <div v-if="$slots.footer" class="mint-settings-modal__footer">
+      <slot
+        name="footer"
+        :values="builder.form.data"
+        :close="handleClose"
+      />
+    </div>
   </BaseModal>
 </template>
 

package/src/components/UnitInput.vue

@@ -47,8 +47,12 @@ const groupedUnits = computed(() => {
   const ungrouped: UnitOption[] = []
   for (const u of props.units) {
     if (u.group) {
-      if (!groups.has(u.group)) groups.set(u.group, [])
-      groups.get(u.group)!.push(u)
+      const group = groups.get(u.group)
+      if (group) {
+        group.push(u)
+      } else {
+        groups.set(u.group, [u])
+      }
     } else {
       ungrouped.push(u)
     }

package/src/components/WellPlate.vue

@@ -759,10 +759,10 @@ const tableStyle = computed(() => ({
                 <span
                   v-if="getWellBadge(well)"
                   class="mint-well-plate__badge"
-                  :style="{ backgroundColor: getWellBadge(well)!.color }"
-                  :title="getWellBadge(well)!.text === '+' ? 'Custom method' : `${getWellBadge(well)!.text}x injections`"
+                  :style="{ backgroundColor: getWellBadge(well)?.color }"
+                  :title="getWellBadge(well)?.text === '+' ? 'Custom method' : `${getWellBadge(well)?.text ?? ''}x injections`"
                 >
-                  {{ getWellBadge(well)!.text }}
+                  {{ getWellBadge(well)?.text }}
                 </span>
               </div>
             </td>