From cf8aab5dbb616d811ee09648cc3a39d3d4abc462 Mon Sep 17 00:00:00 2001
From: Jean Ribeiro <contact@jeanribeiro.dev>
Date: Mon, 13 May 2024 10:50:39 -0300
Subject: [PATCH 1/4] fix: dropdowns on FilterModal (#2481)

* fix: makes Filter header visible on scroll

* chore: updates @bloomwaletio/ui

* chore: updates @bloomwalletio/ui

* yarn lock

---------

Co-authored-by: Mark Nardi <nardimark7@gmail.com>
---
 .../components/modals/FilterModal.svelte      | 15 ++-----
 packages/shared/package.json                  |  2 +-
 yarn.lock                                     | 39 ++++---------------
 3 files changed, 12 insertions(+), 44 deletions(-)

diff --git a/packages/desktop/components/modals/FilterModal.svelte b/packages/desktop/components/modals/FilterModal.svelte
index 51ff1ed756..670b7782f6 100644
--- a/packages/desktop/components/modals/FilterModal.svelte
+++ b/packages/desktop/components/modals/FilterModal.svelte
@@ -29,12 +29,12 @@
     bind:this={modal}
     on:close={closeFilters}
     position={{ absolute: true, right: '0', top: '30px' }}
-    classes="overflow-visible"
+    classes="flex !overflow-hidden"
     autoMaxHeight
 >
-    <filter-modal>
+    <filter-modal class="flex flex-col w-64 rounded-[inherit]">
         <filter-modal-header
-            class="flex flex-row items-center justify-between bg-surface-1 dark:bg-transparent px-4 py-2 rounded-t-xl"
+            class="flex-none flex flex-row items-center justify-between bg-surface-1 dark:bg-transparent px-4 py-2 rounded-t-xl"
         >
             <Button text={localize('actions.clear')} on:click={onClearClick} size="xs" variant="outlined" />
             <Text align="center">
@@ -42,15 +42,8 @@
             </Text>
             <Button text={localize('actions.apply')} on:click={onConfirmClick} size="xs" disabled={!isChanged} />
         </filter-modal-header>
-        <filter-modal-slot class="block">
+        <filter-modal-slot class="flex-1 h-0 overflow-y-scroll">
             <slot />
         </filter-modal-slot>
     </filter-modal>
 </Modal>
-
-<style lang="scss">
-    filter-modal {
-        @apply block w-64;
-        border-radius: inherit;
-    }
-</style>
diff --git a/packages/shared/package.json b/packages/shared/package.json
index 17d9e16196..abd2e2972e 100644
--- a/packages/shared/package.json
+++ b/packages/shared/package.json
@@ -5,7 +5,7 @@
     "author": "Bloom Labs Ltd <contact@bloomwallet.io>",
     "license": "PolyForm Strict License 1.0.0",
     "dependencies": {
-        "@bloomwalletio/ui": "0.20.21",
+        "@bloomwalletio/ui": "0.20.22",
         "@ethereumjs/common": "4.3.0",
         "@ethereumjs/rlp": "5.0.2",
         "@ethereumjs/tx": "5.3.0",
diff --git a/yarn.lock b/yarn.lock
index 1f2d275576..b01ec72c53 100644
--- a/yarn.lock
+++ b/yarn.lock
@@ -343,10 +343,10 @@
   resolved "https://registry.yarnpkg.com/@bcoe/v8-coverage/-/v8-coverage-0.2.3.tgz#75a2e8b51cb758a7553d6804a5932d7aace75c39"
   integrity sha512-0hYQ8SB4Db5zvZB4axdMHGwEaQjkZzFjQiN9LVYvIFB2nSUHW9tYpxWriPrWDASIxiaXax83REcLxuSdnGPZtw==
 
-"@bloomwalletio/ui@0.20.21":
-  version "0.20.21"
-  resolved "https://npm.pkg.github.com/download/@bloomwalletio/ui/0.20.21/ddbf6507eaeffd2d44b8c2564686c0d3d69715a9#ddbf6507eaeffd2d44b8c2564686c0d3d69715a9"
-  integrity sha512-XZLAZ5a7UN8g91OPC/VP2lz8+nDl6UvtBkjWPyryDLw8oaaEi0anWrMecKsULc9SLPblMbKOMRj2FfjCkiEdJA==
+"@bloomwalletio/ui@0.20.22":
+  version "0.20.22"
+  resolved "https://npm.pkg.github.com/download/@bloomwalletio/ui/0.20.22/a8f22408fe57db48e02b36208261aa334c479971#a8f22408fe57db48e02b36208261aa334c479971"
+  integrity sha512-m/+zQGTKVkIxbKxcdZmskz95y+j5nrwyCSIxqzsPJy2GkubhRGTLMyzPQBrWOfxZjTis9ApqF2W/yRLLRKyxXQ==
   dependencies:
     "@floating-ui/dom" "1.4.3"
     "@popperjs/core" "2.11.8"
@@ -10645,16 +10645,7 @@ string-length@^4.0.1:
     char-regex "^1.0.2"
     strip-ansi "^6.0.0"
 
-"string-width-cjs@npm:string-width@^4.2.0":
-  version "4.2.3"
-  resolved "https://registry.yarnpkg.com/string-width/-/string-width-4.2.3.tgz#269c7117d27b05ad2e536830a8ec895ef9c6d010"
-  integrity sha512-wKyQRQpjJ0sIp62ErSZdGsjMJWsap5oRNihHhu6G7JVO/9jIB6UyevL+tXuOqrng8j/cxKTWyWUwvSTriiZz/g==
-  dependencies:
-    emoji-regex "^8.0.0"
-    is-fullwidth-code-point "^3.0.0"
-    strip-ansi "^6.0.1"
-
-string-width@^4.1.0, string-width@^4.2.0, string-width@^4.2.3:
+"string-width-cjs@npm:string-width@^4.2.0", string-width@^4.1.0, string-width@^4.2.0, string-width@^4.2.3:
   version "4.2.3"
   resolved "https://registry.yarnpkg.com/string-width/-/string-width-4.2.3.tgz#269c7117d27b05ad2e536830a8ec895ef9c6d010"
   integrity sha512-wKyQRQpjJ0sIp62ErSZdGsjMJWsap5oRNihHhu6G7JVO/9jIB6UyevL+tXuOqrng8j/cxKTWyWUwvSTriiZz/g==
@@ -10731,14 +10722,7 @@ string_decoder@~1.1.1:
   dependencies:
     safe-buffer "~5.1.0"
 
-"strip-ansi-cjs@npm:strip-ansi@^6.0.1":
-  version "6.0.1"
-  resolved "https://registry.yarnpkg.com/strip-ansi/-/strip-ansi-6.0.1.tgz#9e26c63d30f53443e9489495b2105d37b67a85d9"
-  integrity sha512-Y38VPSHcqkFrCpFnQ9vuSXmquuv5oXOKpGeT6aGrr3o3Gc9AlVa6JBfUSOCnbxGGZF+/0ooI7KrPuUSztUdU5A==
-  dependencies:
-    ansi-regex "^5.0.1"
-
-strip-ansi@^6.0.0, strip-ansi@^6.0.1:
+"strip-ansi-cjs@npm:strip-ansi@^6.0.1", strip-ansi@^6.0.0, strip-ansi@^6.0.1:
   version "6.0.1"
   resolved "https://registry.yarnpkg.com/strip-ansi/-/strip-ansi-6.0.1.tgz#9e26c63d30f53443e9489495b2105d37b67a85d9"
   integrity sha512-Y38VPSHcqkFrCpFnQ9vuSXmquuv5oXOKpGeT6aGrr3o3Gc9AlVa6JBfUSOCnbxGGZF+/0ooI7KrPuUSztUdU5A==
@@ -12243,16 +12227,7 @@ wildcard@^2.0.0:
   resolved "https://registry.yarnpkg.com/wildcard/-/wildcard-2.0.1.tgz#5ab10d02487198954836b6349f74fff961e10f67"
   integrity sha512-CC1bOL87PIWSBhDcTrdeLo6eGT7mCFtrg0uIJtqJUFyK+eJnzl8A1niH56uu7KMa5XFrtiV+AQuHO3n7DsHnLQ==
 
-"wrap-ansi-cjs@npm:wrap-ansi@^7.0.0":
-  version "7.0.0"
-  resolved "https://registry.yarnpkg.com/wrap-ansi/-/wrap-ansi-7.0.0.tgz#67e145cff510a6a6984bdf1152911d69d2eb9e43"
-  integrity sha512-YVGIj2kamLSTxw6NsZjoBxfSwsn0ycdesmc4p+Q21c5zPuZ1pl+NfxVdxPtdHvmNVOQ6XSYG4AUtyt/Fi7D16Q==
-  dependencies:
-    ansi-styles "^4.0.0"
-    string-width "^4.1.0"
-    strip-ansi "^6.0.0"
-
-wrap-ansi@^7.0.0:
+"wrap-ansi-cjs@npm:wrap-ansi@^7.0.0", wrap-ansi@^7.0.0:
   version "7.0.0"
   resolved "https://registry.yarnpkg.com/wrap-ansi/-/wrap-ansi-7.0.0.tgz#67e145cff510a6a6984bdf1152911d69d2eb9e43"
   integrity sha512-YVGIj2kamLSTxw6NsZjoBxfSwsn0ycdesmc4p+Q21c5zPuZ1pl+NfxVdxPtdHvmNVOQ6XSYG4AUtyt/Fi7D16Q==

From 9c044b78202cde3c2ba1ec41102207a74322e088 Mon Sep 17 00:00:00 2001
From: Jean Ribeiro <contact@jeanribeiro.dev>
Date: Mon, 13 May 2024 11:20:12 -0300
Subject: [PATCH 2/4] fix: popup height and scrolling (#2468)

---
 .../desktop/components/popup/Popup.svelte     |  1 +
 .../components/popup/PopupTemplate.svelte     | 37 ++++++++-----------
 .../send-flow/views/SelectTokenView.svelte    | 29 ++++++---------
 3 files changed, 28 insertions(+), 39 deletions(-)

diff --git a/packages/desktop/components/popup/Popup.svelte b/packages/desktop/components/popup/Popup.svelte
index d5b63026a9..edea5361db 100644
--- a/packages/desktop/components/popup/Popup.svelte
+++ b/packages/desktop/components/popup/Popup.svelte
@@ -226,6 +226,7 @@
         bind:this={popupContent}
         class:relative
         class="popup {size}"
+        class:-mt-7={IS_WINDOWS}
     >
         <svelte:component this={POPUP_MAP[id]} {...props} />
         {#if !hideClose}
diff --git a/packages/desktop/components/popup/PopupTemplate.svelte b/packages/desktop/components/popup/PopupTemplate.svelte
index a01813b589..097cd4a159 100644
--- a/packages/desktop/components/popup/PopupTemplate.svelte
+++ b/packages/desktop/components/popup/PopupTemplate.svelte
@@ -57,13 +57,13 @@
 </script>
 
 <popup-template>
-    <div class="popup-banner">
+    <div class="popup-banner overflow-hidden rounded-t-[32px]">
         <slot name="banner" />
     </div>
-    <popop-content class="flex flex-col gap-6 p-8 {$$slots.banner ? 'pt-4' : ''}">
+    <popup-content class="max-h-[90vh] flex flex-col gap-6 p-8" class:pt-4={$$slots.banner}>
         {#if title || $$slots.menu || $$slots.description || description}
-            <popup-header class="flex flex-col space-y-2.5">
-                <title-row class="flex flex-row space-x-2 justify-between">
+            <popup-header class="flex-none flex flex-col gap-2.5">
+                <title-row class="flex flex-row gap-2 mr-7 justify-between">
                     {#if title}<Text type="h6" truncate>{title}</Text>{/if}
                     {#if $$slots.menu}
                         <slot name="menu" class="flex-0" />
@@ -76,13 +76,15 @@
                 {/if}
             </popup-header>
         {/if}
-        {#if $$slots.default}
-            <slot />
-        {:else if $$slots.content}
-            <slot name="content" />
-        {/if}
+        <div class="flex-1 h-0 flex flex-col">
+            {#if $$slots.default}
+                <slot />
+            {:else if $$slots.content}
+                <slot name="content" />
+            {/if}
+        </div>
         {#if backButton || continueButton}
-            <popup-footer class="flex flex-row space-x-3">
+            <popup-footer class="flex-none flex flex-row gap-3">
                 {#if backButton}
                     <Button
                         type="button"
@@ -91,7 +93,7 @@
                         disabled={busy || _backButton.disabled}
                         width="full"
                         form={_backButton.type === 'submit' ? _backButton.form : undefined}
-                        on:click={_backButton.type === 'button' && _backButton.onClick}
+                        on:click={_backButton.type === 'button' ? _backButton.onClick : undefined}
                         {..._backButton.restProps}
                     />
                 {/if}
@@ -105,20 +107,11 @@
                         disabled={_continueButton.disabled}
                         {busy}
                         width="full"
-                        on:click={_continueButton.type === 'button' && _continueButton.onClick}
+                        on:click={_continueButton.type === 'button' ? _continueButton.onClick : undefined}
                         {..._continueButton.restProps}
                     />
                 {/if}
             </popup-footer>
         {/if}
-    </popop-content>
+    </popup-content>
 </popup-template>
-
-<style lang="postcss">
-    .popup-banner {
-        @apply overflow-hidden rounded-t-[32px];
-    }
-    title-row {
-        margin-right: 28px;
-    }
-</style>
diff --git a/packages/desktop/views/dashboard/send-flow/views/SelectTokenView.svelte b/packages/desktop/views/dashboard/send-flow/views/SelectTokenView.svelte
index cedfd7e0a9..71f46e6430 100644
--- a/packages/desktop/views/dashboard/send-flow/views/SelectTokenView.svelte
+++ b/packages/desktop/views/dashboard/send-flow/views/SelectTokenView.svelte
@@ -168,13 +168,15 @@
         disabled: !selectedToken || tokenError,
     }}
 >
-    <div class="space-y-4">
-        {#if $networks.length > 1}
-            <SelectInput bind:selected={selectedOption} {options} />
-        {/if}
-        <SearchInput bind:value={searchValue} />
-        <div class="-mr-3">
-            <token-list class="w-full flex flex-col">
+    <div class="flex-1 h-0 flex flex-col gap-4">
+        <div class="flex-none flex flex-col gap-4">
+            {#if $networks.length > 1}
+                <SelectInput bind:selected={selectedOption} {options} />
+            {/if}
+            <SearchInput bind:value={searchValue} />
+        </div>
+        <div class="-mr-3 overflow-y-scroll">
+            <token-list class="flex flex-col p-0.5 pr-1.5 gap-2">
                 {#each tokenList as token}
                     {@const selected = selectedToken?.id === token.id && selectedToken?.networkId === token?.networkId}
                     {@const error = selected ? Boolean(tokenError) : false}
@@ -189,16 +191,9 @@
             </token-list>
         </div>
         {#if tokenError}
-            <Alert variant="danger" text={tokenError} />
+            <div class="flex-none">
+                <Alert variant="danger" text={tokenError} />
+            </div>
         {/if}
     </div>
 </PopupTemplate>
-
-<style lang="postcss">
-    token-list {
-        max-height: 400px;
-        overflow-y: scroll;
-        @apply p-0.5 pr-1.5;
-        @apply gap-2;
-    }
-</style>

From 15c1801b1a447d77abf0c02de2bcfdc1cbe80e0a Mon Sep 17 00:00:00 2001
From: Nicole O'Brien <nicole@bloomwallet.io>
Date: Mon, 13 May 2024 16:11:59 +0100
Subject: [PATCH 3/4] fix: don't override network with node info (#2472)

* fix: don't override network with node info

Co-authored-by: Mark Nardi <nardimark7@gmail.com>

* fix if endpoint is not a node

* fix return type

* chore: remove Promise constructor

---------

Co-authored-by: Mark Nardi <nardimark7@gmail.com>
Co-authored-by: Tuditi <45079109+Tuditi@users.noreply.github.com>
Co-authored-by: Tuditi <daviddetroch@pm.me>
---
 .../popup/popups/NodeInfoPopup.svelte         |  2 +-
 .../desktop/lib/electron/apis/wallet.api.ts   | 21 ++++++++------
 .../views/CustomNetworkView.svelte            |  2 ++
 .../organisms/NodeConfigurationForm.svelte    | 10 +++----
 .../actions/getNodeInfoWhileLoggedOut.ts      | 28 +++++++++++++------
 .../core/profile-manager/api/getNodeInfo.ts   |  6 ++--
 .../interfaces/api.interface.ts               |  2 +-
 .../checkAndInitializeActiveProfileNetwork.ts | 25 +++++++++++++++++
 .../checkAndUpdateActiveProfileNetwork.ts     | 24 ----------------
 .../profile/actions/active-profile/index.ts   |  2 +-
 .../profile/actions/active-profile/login.ts   |  4 +--
 packages/shared/src/locales/en.json           |  1 +
 .../shared/test/mocks/profile-manager.mock.ts |  2 +-
 13 files changed, 73 insertions(+), 56 deletions(-)
 create mode 100644 packages/shared/src/lib/core/profile/actions/active-profile/checkAndInitializeActiveProfileNetwork.ts
 delete mode 100644 packages/shared/src/lib/core/profile/actions/active-profile/checkAndUpdateActiveProfileNetwork.ts

diff --git a/packages/desktop/components/popup/popups/NodeInfoPopup.svelte b/packages/desktop/components/popup/popups/NodeInfoPopup.svelte
index 5534584a31..b8a657d627 100644
--- a/packages/desktop/components/popup/popups/NodeInfoPopup.svelte
+++ b/packages/desktop/components/popup/popups/NodeInfoPopup.svelte
@@ -134,7 +134,7 @@
     onMount(() => {
         getNodeInfo(node?.url, node?.auth)
             .then((nodeInfoResponse) => {
-                nodeInfo = nodeInfoResponse.nodeInfo
+                nodeInfo = nodeInfoResponse?.nodeInfo
             })
             .catch((err) => {
                 closePopup()
diff --git a/packages/desktop/lib/electron/apis/wallet.api.ts b/packages/desktop/lib/electron/apis/wallet.api.ts
index ab20b39c8a..fdfe7cef55 100644
--- a/packages/desktop/lib/electron/apis/wallet.api.ts
+++ b/packages/desktop/lib/electron/apis/wallet.api.ts
@@ -27,16 +27,21 @@ const sdkUtilsMethods = bindSdkUtilsMethods()
 
 export default {
     ...sdkUtilsMethods,
-    async getNodeInfo(managerId: number, url: string, auth: IAuth): Promise<INodeInfoResponse> {
-        const manager = profileManagers[managerId]
-        const client = await manager.getClient()
+    async getNodeInfo(managerId: number, url: string, auth: IAuth): Promise<INodeInfoResponse | undefined> {
+        try {
+            const manager = profileManagers[managerId]
+            const client = await manager.getClient()
 
-        const nodeUrl = url ?? (await client.getNode()).url
-        const nodeInfo = await client.getNodeInfo(nodeUrl, auth)
+            const nodeUrl = url ?? (await client.getNode()).url
+            const nodeInfo = await client.getNodeInfo(nodeUrl, auth)
 
-        return {
-            url: nodeUrl,
-            nodeInfo,
+            return {
+                url: nodeUrl,
+                nodeInfo,
+            }
+        } catch (err) {
+            console.error(err)
+            return undefined
         }
     },
     createWallet(id: string, options: WalletOptions): Wallet {
diff --git a/packages/desktop/views/onboarding/views/network-setup/views/CustomNetworkView.svelte b/packages/desktop/views/onboarding/views/network-setup/views/CustomNetworkView.svelte
index c4d8fd339c..7ff20256b6 100644
--- a/packages/desktop/views/onboarding/views/network-setup/views/CustomNetworkView.svelte
+++ b/packages/desktop/views/onboarding/views/network-setup/views/CustomNetworkView.svelte
@@ -61,6 +61,8 @@
                 formError = localize('error.node.unabledToConnect')
             } else if (err?.message === 'error.node.differentNetwork') {
                 formError = localize('error.node.differentNetwork')
+            } else if (err?.message === 'error.node.invalidNode') {
+                formError = localize('error.node.invalidNode')
             } else if (err?.type !== 'validationError') {
                 showNotification({
                     variant: 'error',
diff --git a/packages/shared/src/components/organisms/NodeConfigurationForm.svelte b/packages/shared/src/components/organisms/NodeConfigurationForm.svelte
index 87de8e20f3..f3f80bba14 100644
--- a/packages/shared/src/components/organisms/NodeConfigurationForm.svelte
+++ b/packages/shared/src/components/organisms/NodeConfigurationForm.svelte
@@ -69,11 +69,7 @@
             return Promise.reject({ type: 'validationError', error: formError })
         }
 
-        const errorUrlValidity = checkNodeUrlValidity(
-            currentClientOptions?.nodes,
-            node.url,
-            $activeProfile.features.developer
-        )
+        const errorUrlValidity = checkNodeUrlValidity(currentClientOptions?.nodes, node.url, false)
         if (errorUrlValidity) {
             formError = localize(errorUrlValidity) ?? ''
             return Promise.reject({ type: 'validationError', error: formError })
@@ -84,6 +80,10 @@
         if (options.checkNodeInfo) {
             try {
                 nodeInfoResponse = await getNodeInfo(node.url)
+                if (!nodeInfoResponse?.nodeInfo) {
+                    formError = localize('error.node.invalidNode')
+                    return Promise.reject({ type: 'validationError', error: formError })
+                }
             } catch (err) {
                 formError = localize('error.node.unabledToConnect')
                 return Promise.reject({ type: 'validationError', error: formError })
diff --git a/packages/shared/src/lib/core/network/actions/getNodeInfoWhileLoggedOut.ts b/packages/shared/src/lib/core/network/actions/getNodeInfoWhileLoggedOut.ts
index e5ef298b09..cd76019038 100644
--- a/packages/shared/src/lib/core/network/actions/getNodeInfoWhileLoggedOut.ts
+++ b/packages/shared/src/lib/core/network/actions/getNodeInfoWhileLoggedOut.ts
@@ -18,16 +18,26 @@ export async function getNodeInfoWhileLoggedOut(url: string, auth: IAuth): Promi
             { stronghold: { snapshotPath: `${storagePath}/wallet.stronghold` } }
         )
         nodeInfoResponse = await api.getNodeInfo(manager?.id, url, auth)
+
+        if (!nodeInfoResponse) {
+            throw new Error('error.node.invalidNode')
+        }
+
         return nodeInfoResponse
-    } catch (error) {
-        return Promise.reject(error)
     } finally {
-        if (manager) {
-            api.deleteWallet(manager?.id)
-            await manager.destroy()
-        }
-        if (storagePath) {
-            await Platform.removeProfileFolder(storagePath)
-        }
+        await cleanupTemporaryProfile(manager, storagePath)
+    }
+}
+
+async function cleanupTemporaryProfile(
+    manager: IProfileManager | undefined,
+    storagePath: string | undefined
+): Promise<void> {
+    if (manager) {
+        api.deleteWallet(manager?.id)
+        await manager.destroy()
+    }
+    if (storagePath) {
+        await Platform.removeProfileFolder(storagePath)
     }
 }
diff --git a/packages/shared/src/lib/core/profile-manager/api/getNodeInfo.ts b/packages/shared/src/lib/core/profile-manager/api/getNodeInfo.ts
index 20068f5763..946a885587 100644
--- a/packages/shared/src/lib/core/profile-manager/api/getNodeInfo.ts
+++ b/packages/shared/src/lib/core/profile-manager/api/getNodeInfo.ts
@@ -3,9 +3,7 @@ import { INodeInfoResponse } from '@core/network/interfaces/node-info-response.i
 import { getProfileManager } from '../stores'
 import { api } from '../api'
 
-export function getNodeInfo(url?: string, auth?: IAuth): Promise<INodeInfoResponse> | undefined {
+export async function getNodeInfo(url?: string, auth?: IAuth): Promise<INodeInfoResponse | undefined> {
     const id = getProfileManager()?.id
-    if (id) {
-        return api.getNodeInfo(id, url, auth)
-    }
+    return id ? api.getNodeInfo(id, url, auth) : undefined
 }
diff --git a/packages/shared/src/lib/core/profile-manager/interfaces/api.interface.ts b/packages/shared/src/lib/core/profile-manager/interfaces/api.interface.ts
index d09e0c74fe..e3d9d8f087 100644
--- a/packages/shared/src/lib/core/profile-manager/interfaces/api.interface.ts
+++ b/packages/shared/src/lib/core/profile-manager/interfaces/api.interface.ts
@@ -27,7 +27,7 @@ export interface IApi {
     getAccount(profileManagerId: string, index: number): Promise<IAccount>
     getAccounts(profileManagerId: string): Promise<IAccount[]>
     getClient(profileManagerId: string): Promise<Client>
-    getNodeInfo(profileManagerId: string, url?: string, auth?: IAuth): Promise<INodeInfoResponse>
+    getNodeInfo(profileManagerId: string, url?: string, auth?: IAuth): Promise<INodeInfoResponse | undefined>
     getSecretManager(profileManagerId: string): Promise<ISecretManager>
     recoverAccounts(profileManagerId: string, payload: RecoverAccountsPayload): Promise<IAccount[]>
     migrateStrongholdSnapshotV2ToV3(
diff --git a/packages/shared/src/lib/core/profile/actions/active-profile/checkAndInitializeActiveProfileNetwork.ts b/packages/shared/src/lib/core/profile/actions/active-profile/checkAndInitializeActiveProfileNetwork.ts
new file mode 100644
index 0000000000..c0cd4872c8
--- /dev/null
+++ b/packages/shared/src/lib/core/profile/actions/active-profile/checkAndInitializeActiveProfileNetwork.ts
@@ -0,0 +1,25 @@
+import { get } from 'svelte/store'
+import { localize } from '@core/i18n'
+import { getAndUpdateNodeInfo } from '@core/network/actions'
+import { activeProfile } from '@core/profile/stores'
+import { NetworkNamespace, initializeNetworks } from '@core/network'
+
+export async function checkAndInitializeActiveProfileNetwork(): Promise<void> {
+    const $activeProfile = get(activeProfile)
+    const existingNetwork = $activeProfile?.network
+    const nodeInfoResponse = await getAndUpdateNodeInfo(true)
+    if (!nodeInfoResponse) {
+        throw new Error(localize('error.network.mismatch'))
+    }
+
+    const networkIdFromNode = `${NetworkNamespace.Stardust}:${nodeInfoResponse?.nodeInfo?.protocol.networkName}`
+    if (existingNetwork?.id !== networkIdFromNode) {
+        throw new Error(localize('error.network.mismatch', { networkId: networkIdFromNode }))
+    }
+
+    if (existingNetwork?.protocol.version !== nodeInfoResponse?.nodeInfo?.protocol.version) {
+        throw new Error(localize('error.network.version.mismatch', { networkId: networkIdFromNode }))
+    }
+
+    initializeNetworks()
+}
diff --git a/packages/shared/src/lib/core/profile/actions/active-profile/checkAndUpdateActiveProfileNetwork.ts b/packages/shared/src/lib/core/profile/actions/active-profile/checkAndUpdateActiveProfileNetwork.ts
deleted file mode 100644
index 77d359b917..0000000000
--- a/packages/shared/src/lib/core/profile/actions/active-profile/checkAndUpdateActiveProfileNetwork.ts
+++ /dev/null
@@ -1,24 +0,0 @@
-import { get } from 'svelte/store'
-import { localize } from '@core/i18n'
-import { getAndUpdateNodeInfo } from '@core/network/actions'
-import { buildPersistedNetworkFromNodeInfoResponse } from '@core/network/utils'
-import { activeProfile, updateActiveProfile } from '@core/profile/stores'
-import { initializeNetworks } from '@core/network'
-
-export async function checkAndUpdateActiveProfileNetwork(): Promise<void> {
-    const $activeProfile = get(activeProfile)
-    const existingNetwork = $activeProfile?.network
-    const nodeInfoResponse = await getAndUpdateNodeInfo(true)
-    if (!nodeInfoResponse) {
-        return
-    }
-
-    const network = buildPersistedNetworkFromNodeInfoResponse(nodeInfoResponse, existingNetwork.coinType)
-    if (existingNetwork?.id === network.id) {
-        network.chainConfigurations = $activeProfile.network?.chainConfigurations || []
-        updateActiveProfile({ network })
-        initializeNetworks()
-    } else {
-        throw new Error(localize('error.network.mismatch', { networkId: network.id }))
-    }
-}
diff --git a/packages/shared/src/lib/core/profile/actions/active-profile/index.ts b/packages/shared/src/lib/core/profile/actions/active-profile/index.ts
index 4180bb8264..9751385cc1 100644
--- a/packages/shared/src/lib/core/profile/actions/active-profile/index.ts
+++ b/packages/shared/src/lib/core/profile/actions/active-profile/index.ts
@@ -1,6 +1,6 @@
 export * from './checkActiveProfileAuth'
 export * from './checkAndRemoveProfilePicture'
-export * from './checkAndUpdateActiveProfileNetwork'
+export * from './checkAndInitializeActiveProfileNetwork'
 export * from './getBaseToken'
 export * from './getNetworkHrp'
 export * from './loadAccounts'
diff --git a/packages/shared/src/lib/core/profile/actions/active-profile/login.ts b/packages/shared/src/lib/core/profile/actions/active-profile/login.ts
index 2aae66ebb8..2b017354e0 100644
--- a/packages/shared/src/lib/core/profile/actions/active-profile/login.ts
+++ b/packages/shared/src/lib/core/profile/actions/active-profile/login.ts
@@ -34,7 +34,7 @@ import {
 } from '../../stores'
 import { isLedgerProfile, waitForPreviousManagerToBeDestroyed } from '../../utils'
 // import { checkAndRemoveProfilePicture } from './checkAndRemoveProfilePicture'
-import { checkAndUpdateActiveProfileNetwork } from './checkAndUpdateActiveProfileNetwork'
+import { checkAndInitializeActiveProfileNetwork } from './checkAndInitializeActiveProfileNetwork'
 import { loadAccounts } from './loadAccounts'
 import { logout } from './logout'
 import { subscribeToWalletApiEventsForActiveProfile } from './subscribeToWalletApiEventsForActiveProfile'
@@ -67,7 +67,7 @@ export async function login(loginOptions?: ILoginOptions): Promise<void> {
 
         // Step 2: get node info to check we have a synced node
         incrementLoginProgress()
-        await checkAndUpdateActiveProfileNetwork()
+        await checkAndInitializeActiveProfileNetwork()
 
         // Step 3: load accounts
         incrementLoginProgress()
diff --git a/packages/shared/src/locales/en.json b/packages/shared/src/locales/en.json
index e32e2ade0e..bcd0099f2d 100644
--- a/packages/shared/src/locales/en.json
+++ b/packages/shared/src/locales/en.json
@@ -2026,6 +2026,7 @@
             "timeNotSynced": "Device time is incorrect, unable to sync node.",
             "unabledToConnect": "Unable to connect to the node",
             "differentNetwork": "The node is located on a different network",
+            "invalidNode": "The URL is not a valid node",
             "duplicateNodes": "Unable to add duplicate node",
             "noCoinType": "No coin type specified"
         },
diff --git a/packages/shared/test/mocks/profile-manager.mock.ts b/packages/shared/test/mocks/profile-manager.mock.ts
index af28132f0f..c8e3b045fa 100644
--- a/packages/shared/test/mocks/profile-manager.mock.ts
+++ b/packages/shared/test/mocks/profile-manager.mock.ts
@@ -79,7 +79,7 @@ export class ProfileManagerMock implements IProfileManager {
         throw new Error('Method not implemented.')
     }
 
-    getNodeInfo(url: string, auth?: IAuth): Promise<INodeInfoResponse> {
+    getNodeInfo(url: string, auth?: IAuth): Promise<INodeInfoResponse | undefined> {
         return Promise.resolve({
             nodeInfo: {
                 name: 'HORNET',

From 653082edcda00f5087837b0e21b75d9ee0863697 Mon Sep 17 00:00:00 2001
From: Jean Ribeiro <contact@jeanribeiro.dev>
Date: Mon, 13 May 2024 14:20:15 -0300
Subject: [PATCH 4/4] chore: updates required node version in desktop README
 (#2486)

Co-authored-by: Nicole O'Brien <nicole@bloomwallet.io>
---
 packages/desktop/README.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/desktop/README.md b/packages/desktop/README.md
index 404573d50a..34dd4aa1e4 100644
--- a/packages/desktop/README.md
+++ b/packages/desktop/README.md
@@ -4,7 +4,7 @@
 
 The following **must** be installed on all platforms:
 
--   [Node.js](https://nodejs.org/en/) 18.15.0+
+-   [Node.js](https://nodejs.org/en/) 20.10.0+
 -   [Yarn](https://classic.yarnpkg.com/en/docs/install)
 -   [Rust](https://www.rust-lang.org/tools/install)