feat: introduce internal_model UNKNOWN, add support for it in the Suite

packages/components/src/components/Image/images.ts

@@ -5,6 +5,7 @@ export const SVG_IMAGES = {
     UNI_SUCCESS: 'uni-success.svg',
     UNI_WARNING: 'uni-warning.svg',
     BRIDGE_CHECK_TREZOR_T2T1: 'bridge-check-trezor-t2t1.svg',
+    DEVICE_CONFIRM_TREZOR_UNKNOWN: 'device_confirm_trezor_unknown.svg',
     DEVICE_CONFIRM_TREZOR_T1B1: 'device-confirm-trezor-t1b1.svg',
     DEVICE_CONFIRM_TREZOR_T2T1: 'device-confirm-trezor-t2t1.svg',
     DEVICE_CONFIRM_TREZOR_T2B1: 'device-confirm-trezor-t3b1.svg',
@@ -81,6 +82,8 @@ export const PNG_IMAGES = {
     UNDERSTAND_2x: '[email protected]',
     WALLET: 'wallet.png',
     WALLET_2x: '[email protected]',
+    DONT_DISCONNECT_TREZOR_UNKNOWN: 'trezor-unknown.png',
+    DONT_DISCONNECT_TREZOR_UNKNOWN_2x: '[email protected]',
     DONT_DISCONNECT_TREZOR_T2T1: 'dont-disconnect-trezor-t2t1.png',
     DONT_DISCONNECT_TREZOR_T2T1_2x: '[email protected]',
     DONT_DISCONNECT_TREZOR_T1B1: 'dont-disconnect-trezor-t1b1.png',
@@ -93,6 +96,8 @@ export const PNG_IMAGES = {
     DONT_DISCONNECT_TREZOR_T3T1_2x: '[email protected]',
     DONT_DISCONNECT_TREZOR_T3W1: 'dont-disconnect-trezor-t3w1.png',
     DONT_DISCONNECT_TREZOR_T3W1_2x: '[email protected]',
+    TREZOR_UNKNOWN: 'trezor-unknown.png',
+    TREZOR_UNKNOWN_2x: '[email protected]',
     TREZOR_T1B1: 'trezor-t1b1.png',
     TREZOR_T1B1_2x: '[email protected]',
     TREZOR_T2T1: 'trezor-t2t1.png',
@@ -105,6 +110,8 @@ export const PNG_IMAGES = {
     TREZOR_T3T1_2x: '[email protected]',
     TREZOR_T3W1: 'trezor-t3w1.png',
     TREZOR_T3W1_2x: '[email protected]',
+    TREZOR_UNKNOWN_LARGE: 'trezor-unknown.png',
+    TREZOR_UNKNOWN_LARGE_2x: '[email protected]',
     TREZOR_T1B1_LARGE: 'trezor-t1b1-large.png',
     TREZOR_T1B1_LARGE_2x: '[email protected]',
     TREZOR_T2T1_LARGE: 'trezor-t2t1-large.png',
@@ -117,6 +124,8 @@ export const PNG_IMAGES = {
     TREZOR_T3T1_LARGE_2x: '[email protected]',
     TREZOR_T3W1_LARGE: 'trezor-t3w1-large.png',
     TREZOR_T3W1_LARGE_2x: '[email protected]',
+    TREZOR_UNKNOWN_GHOST: 'trezor-unknown.png',
+    TREZOR_UNKNOWN_GHOST_2x: '[email protected]',
     TREZOR_T1B1_GHOST: 'trezor-t1b1-ghost.png',
     TREZOR_T1B1_GHOST_2x: '[email protected]',
     TREZOR_T2T1_GHOST: 'trezor-t2t1-ghost.png',

packages/connect/src/api/common/__fixtures__/paramsValidator.ts

@@ -170,7 +170,8 @@ export const validateParams = [
     },
 ];
 
-const DEFAULT_RANGE = {
+const DEFAULT_RANGE: FirmwareRange = {
+    UNKNOWN: { min: '1.0.0', max: '0' },
     T1B1: { min: '1.0.0', max: '0' },
     T2T1: { min: '2.0.0', max: '0' },
     T2B1: { min: '2.0.0', max: '0' },
@@ -179,8 +180,13 @@ const DEFAULT_RANGE = {
     T3W1: { min: '2.0.0', max: '0' },
 };
 
-const DEFAULT_COIN_INFO = {
+const DEFAULT_COIN_INFO: {
+    support: Record<DeviceModelInternal, string>;
+    shortcut: string;
+    type: string;
+} = {
     support: {
+        UNKNOWN: '1.0.0',
         T1B1: '1.6.2',
         T2T1: '2.1.0',
         T2B1: '2.0.0',
@@ -268,6 +274,7 @@ export const getFirmwareRange = [
                     T3B1: false,
                     T3T1: false,
                     T3W1: false,
+                    UNKNOWN: false,
                 },
                 shortcut: 'btc',
                 type: 'bitcoin',

packages/connect/src/api/ethereum/ethereumDefinitions.ts

@@ -156,6 +156,7 @@ export const ethereumNetworkInfoFromDefinition = (
         T3B1: '2.0.0',
         T3T1: '2.0.0',
         T3W1: '2.0.0',
+        UNKNOWN: '0.0.0',
     },
     blockchainLink: undefined,
 });

packages/connect/src/core/AbstractMethod.ts

@@ -35,6 +35,7 @@ export interface MethodInfo {
 }
 
 export const DEFAULT_FIRMWARE_RANGE: FirmwareRange = {
+    UNKNOWN: { min: '1.0.0', max: '0' },
     T1B1: { min: '1.0.0', max: '0' },
     T2T1: { min: '2.0.0', max: '0' },
     T2B1: { min: '2.6.1', max: '0' },

packages/connect/src/data/deviceAuthenticityConfigTypes.ts

@@ -12,12 +12,12 @@ type ModelPubKeys = Static<typeof ModelPubKeys>;
 
 const ModelsWithKeys = Type.Exclude(
     Type.KeyOfEnum(PROTO.DeviceModelInternal),
-    Type.Union([Type.Literal('T1B1'), Type.Literal('T2T1')]),
+    Type.Union([Type.Literal('T1B1'), Type.Literal('T2T1'), Type.Literal('UNKNOWN')]),
 );
 
 const ModelsWithoutKeys = Type.Extract(
     Type.KeyOfEnum(PROTO.DeviceModelInternal),
-    Type.Union([Type.Literal('T1B1'), Type.Literal('T2T1')]),
+    Type.Union([Type.Literal('T1B1'), Type.Literal('T2T1'), Type.Literal('UNKNOWN')]),
 );
 
 const ModelPubKeys = Type.Intersect([

packages/connect/src/data/models.ts

@@ -17,6 +17,10 @@ const safe3Model: ModelConfig = {
 };
 
 export const models: Record<DeviceModelInternal, ModelConfig> = {
+    UNKNOWN: {
+        name: 'Unknown',
+        colors: safe3Model.colors, // just in case
+    },
     T1B1: {
         name: 'Trezor Model One',
         colors: {},

packages/connect/src/device/Device.ts

@@ -48,6 +48,7 @@ import {
     FirmwareHashCheckResult,
     FirmwareHashCheckError,
     DeviceUniquePath,
+    DeviceModelInternal,
 } from '../types';
 import { models } from '../data/models';
 import { getLanguage } from '../data/getLanguage';
@@ -966,7 +967,8 @@ export class Device extends TypedEmitter<DeviceEvents> {
             feat.model = '1';
         }
         // 2. - old fw does not include internal_model. T1B1 does not report it yet, T2T1 starts in 2.6.0
-        if (!feat.internal_model) {
+        //    - or reported internal_model is not known to connect
+        if (!feat.internal_model || !DeviceModelInternal[feat.internal_model]) {
             feat.internal_model = ensureInternalModelFeature(feat.model);
         }
 

packages/connect/src/utils/assetUtils.ts

@@ -6,6 +6,7 @@ const isDeviceModel = (model: string): model is DeviceModelInternal =>
     isArrayMember(model, Object.values(DeviceModelInternal));
 
 export const firmwareAssets: Record<DeviceModelInternal, NodeRequire> = {
+    [DeviceModelInternal.UNKNOWN]: {} as NodeRequire,
     [DeviceModelInternal.T1B1]: require('@trezor/connect-common/files/firmware/t1b1/releases.json'),
     [DeviceModelInternal.T2T1]: require('@trezor/connect-common/files/firmware/t2t1/releases.json'),
     [DeviceModelInternal.T2B1]: require('@trezor/connect-common/files/firmware/t2b1/releases.json'),

packages/connect/src/utils/deviceFeaturesUtils.ts

@@ -180,7 +180,8 @@ export const ensureInternalModelFeature = (model: Features['model']): DeviceMode
         case 'T':
             return DeviceModelInternal.T2T1;
         case '1':
-        default:
             return DeviceModelInternal.T1B1;
+        default:
+            return DeviceModelInternal.UNKNOWN;
     }
 };

packages/product-components/src/index.ts

@@ -7,7 +7,7 @@ export {
 export { SearchAsset } from './components/SelectAssetModal/SearchAsset';
 export { PassphraseTypeCard } from './components/PassphraseTypeCard/PassphraseTypeCard';
 export { ConfirmOnDevice } from './components/ConfirmOnDevice/ConfirmOnDevice';
-export { mapTrezorModelToIcon } from './utils/mapTrezorModelToIcon';
+export { mapTrezorModelToIconDeprecated, mapTrezorModelToIcon } from './utils/mapTrezorModelToIcon';
 export { RotateDeviceImage } from './components/RotateDeviceImage/RotateDeviceImage';
 export { TrezorLogo } from './components/TrezorLogo/TrezorLogo';
 export { PasswordStrengthIndicator } from './components/PasswordStrengthIndicator/PasswordStrengthIndicator';

packages/product-components/src/utils/mapTrezorModelToIcon.ts

@@ -2,10 +2,21 @@ import { IconName } from '@trezor/components';
 import { DeviceModelInternal } from '@trezor/connect';
 
 export const mapTrezorModelToIcon: Record<DeviceModelInternal, IconName> = {
+    [DeviceModelInternal.UNKNOWN]: 'trezorModelOneFilled', // Just to provide something that wont break UI
     [DeviceModelInternal.T1B1]: 'trezorModelOneFilled',
     [DeviceModelInternal.T2T1]: 'trezorModelTFilled',
     [DeviceModelInternal.T2B1]: 'trezorSafe3Filled',
     [DeviceModelInternal.T3B1]: 'trezorSafe3Filled',
     [DeviceModelInternal.T3T1]: 'trezorSafe5Filled',
     [DeviceModelInternal.T3W1]: 'trezorSafe7Filled',
 };
+
+export const mapTrezorModelToIconDeprecated: Record<DeviceModelInternal, IconName> = {
+    [DeviceModelInternal.UNKNOWN]: 'trezorT1B1', // Just to provide something that won't break UI
+    [DeviceModelInternal.T1B1]: 'trezorT1B1',
+    [DeviceModelInternal.T2T1]: 'trezorT2T1',
+    [DeviceModelInternal.T2B1]: 'trezorT2B1',
+    [DeviceModelInternal.T3B1]: 'trezorT3B1',
+    [DeviceModelInternal.T3T1]: 'trezorT3T1',
+    [DeviceModelInternal.T3W1]: 'trezorT3W1',
+};

packages/protobuf/scripts/protobuf-patches/customTypes.ts

@@ -5,4 +5,5 @@ export enum DeviceModelInternal {
     T3B1 = 'T3B1',
     T3T1 = 'T3T1',
     T3W1 = 'T3W1',
+    UNKNOWN = 'UNKNOWN',
 }

packages/protobuf/src/messages-schema.ts

@@ -7,6 +7,7 @@ export enum DeviceModelInternal {
     T3B1 = 'T3B1',
     T3T1 = 'T3T1',
     T3W1 = 'T3W1',
+    UNKNOWN = 'UNKNOWN',
 }
 
 export type EnumDeviceModelInternal = Static<typeof EnumDeviceModelInternal>;

packages/protobuf/src/messages.ts

@@ -13,6 +13,7 @@ export enum DeviceModelInternal {
     T3B1 = 'T3B1',
     T3T1 = 'T3T1',
     T3W1 = 'T3W1',
+    UNKNOWN = 'UNKNOWN',
 }
 
 export type BinanceGetAddress = {

packages/suite/src/components/firmware/ReconnectDevicePrompt.tsx

@@ -130,6 +130,9 @@ export const ReconnectDevicePrompt = ({ onClose, onSuccess }: ReconnectDevicePro
 
         const deviceFwVersion = getFirmwareVersion(uiEvent?.payload.device);
         const switchToBootloaderMap: Record<DeviceModelInternal, TranslationKey> = {
+            // just to have something, I assume new models will have touch screen
+            [DeviceModelInternal.UNKNOWN]: 'TR_SWITCH_TO_BOOTLOADER_SWIPE_YOUR_FINGERS',
+
             [DeviceModelInternal.T1B1]:
                 semver.valid(deviceFwVersion) && semver.satisfies(deviceFwVersion, '<1.8.0')
                     ? 'TR_SWITCH_TO_BOOTLOADER_HOLD_BOTH_BUTTONS'

packages/suite/src/components/suite/modals/ReduxModal/UserContextModal/CoinmarketTermsModal.tsx

@@ -11,6 +11,7 @@ import {
 } from '@trezor/components';
 import type { Deferred } from '@trezor/utils';
 import { spacings } from '@trezor/theme';
+import { mapTrezorModelToIconDeprecated } from '@trezor/product-components';
 
 import { Translation } from 'src/components/suite';
 import { useDevice } from 'src/hooks/suite';
@@ -90,7 +91,7 @@ export const CoinmarketTermsModal = ({
                 <List.Item
                     bulletComponent={
                         <IconCircle
-                            name={`trezor${device.features.internal_model}`}
+                            name={mapTrezorModelToIconDeprecated[device.features.internal_model]}
                             {...iconProps}
                         />
                     }

packages/suite/src/constants/suite/device.ts

@@ -5,6 +5,7 @@ export const DEFAULT_PASSPHRASE_PROTECTION = true;
 export const DEFAULT_SKIP_BACKUP = true;
 
 export const DEFAULT_STRENGTH: Record<DeviceModelInternal, number> = {
+    [DeviceModelInternal.UNKNOWN]: 128, // just to have something
     [DeviceModelInternal.T1B1]: 256,
     [DeviceModelInternal.T2T1]: 128,
     [DeviceModelInternal.T2B1]: 128,
@@ -14,6 +15,7 @@ export const DEFAULT_STRENGTH: Record<DeviceModelInternal, number> = {
 };
 
 export const MAX_ROWS_PER_PAGE: Record<DeviceModelInternal, number> = {
+    [DeviceModelInternal.UNKNOWN]: 7, // just to have something
     [DeviceModelInternal.T1B1]: 4,
     [DeviceModelInternal.T2T1]: 5,
     [DeviceModelInternal.T2B1]: 4,
@@ -23,6 +25,7 @@ export const MAX_ROWS_PER_PAGE: Record<DeviceModelInternal, number> = {
 };
 
 export const MAX_CHARACTERS_ON_ROW: Record<DeviceModelInternal, number> = {
+    [DeviceModelInternal.UNKNOWN]: 18, // just to have something
     [DeviceModelInternal.T1B1]: 21,
     [DeviceModelInternal.T2T1]: 17, // -1 for the space for the scrollbar (Trezor T only)
     [DeviceModelInternal.T2B1]: 18,
@@ -32,6 +35,7 @@ export const MAX_CHARACTERS_ON_ROW: Record<DeviceModelInternal, number> = {
 };
 
 export const CHARACTER_OFFSET_FOR_CONTINUES_ARROW: Record<DeviceModelInternal, number> = {
+    [DeviceModelInternal.UNKNOWN]: 4, // just to have something
     [DeviceModelInternal.T1B1]: 3,
     [DeviceModelInternal.T2T1]: 4,
     [DeviceModelInternal.T2B1]: 2,
@@ -41,6 +45,7 @@ export const CHARACTER_OFFSET_FOR_CONTINUES_ARROW: Record<DeviceModelInternal, n
 };
 
 export const CHARACTER_OFFSET_FOR_NEXT_ARROW: Record<DeviceModelInternal, number> = {
+    [DeviceModelInternal.UNKNOWN]: 4, // just to have something
     [DeviceModelInternal.T1B1]: 0,
     [DeviceModelInternal.T2T1]: 4,
     [DeviceModelInternal.T2B1]: 2,
@@ -50,6 +55,7 @@ export const CHARACTER_OFFSET_FOR_NEXT_ARROW: Record<DeviceModelInternal, number
 };
 
 export const HAS_MONOCHROME_SCREEN: Record<DeviceModelInternal, boolean> = {
+    [DeviceModelInternal.UNKNOWN]: false, // just to have something
     [DeviceModelInternal.T1B1]: true,
     [DeviceModelInternal.T2T1]: false,
     [DeviceModelInternal.T2B1]: true,

packages/suite/src/constants/suite/homescreens.ts

@@ -12,6 +12,10 @@ const safe3Homescreens = [
 
 // original trezor images has to be first as they are send to device as '0' and the default image from fw is used
 export const getHomescreens = (isBitcoinOnlyFirmware: boolean) => ({
+    [DeviceModelInternal.UNKNOWN]: [
+        isBitcoinOnlyFirmware ? 'orange' : 'green', // just to have something
+        isBitcoinOnlyFirmware ? 'green' : 'orange',
+    ],
     [DeviceModelInternal.T1B1]: [
         'original_t1b1', // note - has to be first
         'blank',

packages/suite/src/utils/suite/homescreen.ts

@@ -18,6 +18,7 @@ export const deviceModelInformation: Record<
     DeviceModelInternal,
     { width: number; height: number; supports: Array<'png' | 'jpeg'> }
 > = {
+    [DeviceModelInternal.UNKNOWN]: { width: 280, height: 520, supports: ['jpeg'] }, // just to have something
     [DeviceModelInternal.T1B1]: { width: 128, height: 64, supports: ['png', 'jpeg'] },
     [DeviceModelInternal.T2T1]: { width: 240, height: 240, supports: ['jpeg'] },
     [DeviceModelInternal.T2B1]: safe3Information,

packages/suite/src/views/onboarding/steps/SelectBackupType/SelectBackupType.tsx

@@ -41,6 +41,7 @@ const SHAMIR_TYPES: BackupType[] = ['shamir-single', 'shamir-advanced'];
 export const isShamirBackupType = (type: BackupType) => SHAMIR_TYPES.includes(type);
 
 export const defaultBackupTypeMap: Record<DeviceModelInternal, BackupType> = {
+    [DeviceModelInternal.UNKNOWN]: '12-words', // just to have something
     [DeviceModelInternal.T1B1]: '24-words',
     [DeviceModelInternal.T2T1]: '12-words',
     [DeviceModelInternal.T2B1]: 'shamir-single',

packages/suite/src/views/wallet/staking/components/DeviceButton.tsx

@@ -2,6 +2,7 @@ import { ReactNode } from 'react';
 
 import { Button, Tooltip } from '@trezor/components';
 import { DeviceModelInternal } from '@trezor/connect';
+import { mapTrezorModelToIconDeprecated } from '@trezor/product-components';
 
 interface DeviceButtonProps {
     isDisabled?: boolean;
@@ -25,7 +26,7 @@ export const DeviceButton = ({
             isDisabled={isDisabled}
             isLoading={isLoading}
             onClick={onClick}
-            icon={`trezor${deviceModelInternal}`}
+            icon={mapTrezorModelToIconDeprecated[deviceModelInternal]}
         >
             {children}
         </Button>

suite-common/suite-constants/src/device.ts

@@ -3,6 +3,7 @@ import { DeviceModelInternal } from '@trezor/connect';
 export const DEFAULT_FLAGSHIP_MODEL = DeviceModelInternal.T3T1;
 
 export const SUPPORTS_DEVICE_AUTHENTICITY_CHECK: Record<DeviceModelInternal, boolean> = {
+    [DeviceModelInternal.UNKNOWN]: true, // We must require device authenticity check so it cannot be used as and exploit to bypass it
     [DeviceModelInternal.T1B1]: false,
     [DeviceModelInternal.T2T1]: false,
     [DeviceModelInternal.T2B1]: true,

suite-common/suite-utils/src/device.ts

@@ -251,9 +251,9 @@ export const getChangelogUrl = (device: TrezorDevice, revision?: string | null)
 };
 
 /**
- Returns DeviceModelInternal, but returns T3B1 if T2B1 is provided.
- This is desirable because they both represent the same model (differing internally in the chip) and should behave the same in most situations,
- so we don't need to provide separate assets, translations keys, URLs etc. for both.
+ * Returns DeviceModelInternal, but returns T3B1 if T2B1 is provided.
+ * This is desirable because they both represent the same model (differing internally in the chip) and should behave the same in most situations,
+ * so we don't need to provide separate assets, translations keys, URLs etc. for both.
  */
 export const getNarrowedDeviceModelInternal = <T extends DeviceModelInternal>(
     model: T,
@@ -263,7 +263,7 @@ export const getNarrowedDeviceModelInternal = <T extends DeviceModelInternal>(
 export const getCheckBackupUrl = (device?: TrezorDevice) => {
     const deviceModelInternal = device?.features?.internal_model;
 
-    if (!deviceModelInternal) {
+    if (!deviceModelInternal || deviceModelInternal === DeviceModelInternal.UNKNOWN) {
         return undefined;
     }
 
@@ -273,7 +273,7 @@ export const getCheckBackupUrl = (device?: TrezorDevice) => {
 export const getPackagingUrl = (device?: TrezorDevice) => {
     const deviceModelInternal = device?.features?.internal_model;
 
-    if (!deviceModelInternal) {
+    if (!deviceModelInternal || deviceModelInternal === DeviceModelInternal.UNKNOWN) {
         return '';
     }
 
@@ -283,7 +283,7 @@ export const getPackagingUrl = (device?: TrezorDevice) => {
 export const getFirmwareDowngradeUrl = (device?: TrezorDevice) => {
     const deviceModelInternal = device?.features?.internal_model;
 
-    if (!deviceModelInternal) {
+    if (!deviceModelInternal || deviceModelInternal === DeviceModelInternal.UNKNOWN) {
         return undefined;
     }
 

suite-native/device/src/components/DeviceImage.tsx

@@ -3,6 +3,7 @@ import { DeviceModelInternal } from '@trezor/connect';
 import { prepareNativeStyle, useNativeStyles } from '@trezor/styles';
 
 const deviceImageMap: Record<DeviceModelInternal, string> = {
+    [DeviceModelInternal.UNKNOWN]: require('../assets/unknown.png'),
     [DeviceModelInternal.T1B1]: require('../assets/t1b1.png'),
     [DeviceModelInternal.T2T1]: require('../assets/t2t1.png'),
     [DeviceModelInternal.T2B1]: require('../assets/t3b1.png'),