chore: refactoring and separation of suggestUdevInstaller into @trezor/utils

Author: peter-sanderson

packages/connect/src/data/udevInfo.ts

@@ -13,22 +13,22 @@ const info: UdevInfo = {
             name: 'DEB package',
             platform: ['deb32', 'deb64'],
             url: '/udev/trezor-udev_2_all.deb',
+            preferred: true, // DEB package is the most common
         },
     ],
 };
 
-export const suggestUdevInstaller = (platform?: string) => {
-    const info2 = info;
-    // check if preferred field was already added
-    if (!info2.packages.find(p => p.preferred)) {
-        if (platform) {
-            // override UdevInfo packages, add preferred field
-            info2.packages = info2.packages.map(p => ({
-                ...p,
-                preferred: p.platform.indexOf(platform) >= 0,
-            }));
-        }
+export const suggestUdevInstaller = (platform?: string): UdevInfo => {
+    // `platform` is not available on the desktop, in that case we assume the preferred as DEB
+
+    if (platform !== undefined) {
+        return {
+            packages: info.packages.map(it => ({
+                ...it,
+                preferred: it.platform.indexOf(platform) >= 0,
+            })),
+        };
     }
 
-    return info2;
+    return info;
 };

packages/connect/src/events/transport.ts

@@ -10,6 +10,7 @@ import type { MessageFactoryFn } from '../types/utils';
 export { TRANSPORT } from '@trezor/transport/src/constants';
 
 export const TRANSPORT_EVENT = 'TRANSPORT_EVENT';
+
 export interface BridgeInfo {
     version: number[];
     packages: {

packages/suite/src/components/suite/UdevDownload.tsx

@@ -50,14 +50,11 @@ interface Installer {
 export const UdevDownload = () => {
     const udev = useSelector(selectUdevInstaller);
 
-    const hasPreferred = udev !== undefined && udev.packages.some(p => p.preferred);
-
     const installers: Installer[] = udev
         ? udev.packages.map(p => ({
               label: p.name,
               value: DATA_URL + p.url.substring(1),
-              // preferred is not available on the desktop, so we fallback on the DEB package as it is the most common
-              preferred: hasPreferred ? p.preferred : p.name === 'DEB package',
+              preferred: p.preferred,
           }))
         : [];