chore: improve types in component utilities

Author: targos

src/component/panels/MatrixGenerationPanel/MatrixGenerationPanelHeader.tsx

@@ -3,6 +3,7 @@ import { IoAnalytics } from 'react-icons/io5';
 import { TbBrandGoogleAnalytics } from 'react-icons/tb';
 import { TooltipHelpContent } from 'react-science/ui';
 
+import { isSpectrum1D } from '../../../data/data1d/Spectrum1D/index.ts';
 import { useChartData } from '../../context/ChartContext.js';
 import { usePreferences } from '../../context/PreferencesContext.js';
 import { usePanelPreferences } from '../../hooks/usePanelPreferences.js';
@@ -41,7 +42,11 @@ export function MatrixGenerationPanelHeader(
   const signalProcessingFilterData = useHasSignalProcessingFilter();
 
   function handleExportAsMatrix() {
-    exportAsMatrix(data, spectraPreferences?.columns || [], 'Spectra Matrix');
+    exportAsMatrix(
+      data.filter(isSpectrum1D),
+      spectraPreferences?.columns || [],
+      'Spectra Matrix',
+    );
   }
   function handleToggleStocsy() {
     dispatchPreferences({

src/component/utility/LocalStorage.ts

@@ -2,7 +2,10 @@ import dlv from 'dlv';
 import lodashSet from 'lodash/set.js';
 import { useCallback, useEffect, useMemo, useState } from 'react';
 
-export function useStateWithLocalStorage(localStorageKey: any, key?: string) {
+export function useStateWithLocalStorage(
+  localStorageKey: string,
+  key?: string,
+) {
   const [value, setValue] = useState(
     localStorage.getItem(localStorageKey) || '{}',
   );
@@ -28,12 +31,12 @@ export function useStateWithLocalStorage(localStorageKey: any, key?: string) {
   }, [key, setData, value]);
 }
 
-export function getLocalStorage(localStorageKey: any, isJson = true) {
+export function getLocalStorage(localStorageKey: string, isJson = true) {
   const settings = localStorage.getItem(localStorageKey);
   return settings && isJson ? JSON.parse(settings) : settings;
 }
 
-export function storeData(localStorageKey: any, value: any) {
+export function storeData(localStorageKey: string, value: any) {
   localStorage.setItem(localStorageKey, value);
 }
 

src/component/utility/convertPathArrayToString.ts

@@ -1,3 +1,3 @@
-export function convertPathArrayToString(value: any) {
+export function convertPathArrayToString(value: string | string[]) {
   return Array.isArray(value) ? value.join('.') : value;
 }

src/component/utility/export.ts

@@ -2,7 +2,9 @@ import type { ToastProps } from '@blueprintjs/core';
 import type { SerializedStyles } from '@emotion/react';
 import type {
   JpathTableColumn,
+  SerializedNmriumState,
   SpectraTableColumn,
+  Spectrum1D,
 } from '@zakodium/nmrium-core';
 import { BlobWriter, TextReader, ZipWriter } from '@zip.js/zip.js';
 import dlv from 'dlv';
@@ -23,14 +25,14 @@ export const browserNotSupportedErrorToast: ToastProps = {
  * @param isCompressed
  */
 async function exportAsJsonBlob(
-  data: any,
+  data: SerializedNmriumState,
   fileName = 'experiment',
   spaceIndent = 0,
   isCompressed = false,
 ) {
   const fileData = JSON.stringify(
     data,
-    (key, value: any) =>
+    (key, value: unknown) =>
       ArrayBuffer.isView(value) ? Array.from(value as any) : value,
     spaceIndent,
   );
@@ -44,11 +46,10 @@ async function exportAsJsonBlob(
 }
 
 function exportAsMatrix(
-  data: any,
+  data: Spectrum1D[],
   spectraColumns: SpectraTableColumn[],
   name: string,
 ) {
-  //columns labels
   const columnsLabels: string[] = [];
   // listed the spectra panel columns
   for (const col of spectraColumns) {
@@ -58,7 +59,7 @@ function exportAsMatrix(
   }
 
   for (const value of data[0].data.x) {
-    columnsLabels.push(value);
+    columnsLabels.push(String(value));
   }
   let matrix = `${columnsLabels.join('\t')}\n`;
 
@@ -76,7 +77,7 @@ function exportAsMatrix(
       }
     }
     for (const value of re) {
-      cellsValues.push(value);
+      cellsValues.push(String(value));
     }
     matrix += `${cellsValues.join('\t')}\n`;
   }
@@ -289,7 +290,7 @@ function transferToCanvas(offscreenCanvas: OffscreenCanvas) {
 // hack way to copy the image to the clipboard
 // TODO: remove when Firefox widely supports ClipboardItem
 // https://caniuse.com/mdn-api_clipboarditem
-function copyDataURLClipboardFireFox(image: any) {
+function copyDataURLClipboardFireFox(image: string) {
   const img = document.createElement('img');
   img.src = image;
 
@@ -397,13 +398,13 @@ function parseDimension(value: string | null) {
 
 function getBlob(targetElementID: string, options: GetBlobOptions): BlobObject {
   const { rootElement, css } = options;
-  const _svg: any = (rootElement.getRootNode() as Document)
+  const _svg = (rootElement.getRootNode() as Document)
     .querySelector(`#${targetElementID}`)
-    ?.cloneNode(true);
+    ?.cloneNode(true) as SVGElement;
 
-  const width = parseDimension(_svg?.getAttribute('width'));
-  const height = parseDimension(_svg?.getAttribute('height'));
-  const viewBox = _svg?.getAttribute('viewBox') || `0 0 ${width} ${height}`;
+  const width = parseDimension(_svg.getAttribute('width'));
+  const height = parseDimension(_svg.getAttribute('height'));
+  const viewBox = _svg.getAttribute('viewBox') || `0 0 ${width} ${height}`;
 
   for (const element of _svg.querySelectorAll('[data-no-export="true"]')) {
     element.remove();

src/component/utility/nucleusToString.ts

@@ -2,6 +2,6 @@ export function is2DNucleus(nucleus: string) {
   return nucleus.includes(',');
 }
 
-export default function nucleusToString(nucleus: any) {
+export default function nucleusToString(nucleus: string | string[]) {
   return typeof nucleus === 'string' ? nucleus : nucleus.join(',');
 }

src/component/utility/stackOverlappingLabels.ts

@@ -26,7 +26,7 @@ export function stackOverlappingLabelsArray<T extends Record<string, any>>(
 
   return groups.flatMap((group) => {
     let i = 0;
-    return group.map((item: any) => {
+    return group.map((item) => {
       const stackIndex = i;
       if (item.assignment) {
         i++;
@@ -86,8 +86,8 @@ function stackOverlappingLabels<T extends Record<string, any>>(
   options: StackOverlappingLabelsOptions<T>,
 ) {
   const { startPositionKey, labelWidthKey, padding = 0 } = options;
-  const groups: any[][] = [];
-  let currentGroup: any[] = [];
+  const groups: T[][] = [];
+  let currentGroup: T[] = [];
   let lastInPixel = 0;
 
   for (const item of items) {