@electerm/electerm-react 5.1.25 → 5.2.0

package/client/store/common.js

@@ -193,7 +193,13 @@ export default Store => {
       type,
       authType
     } = tab
-    if (!profile || authType !== 'profiles') {
+    // SSH picks auth type explicitly (password/privateKey/profiles radio),
+    // other types only have the profile dropdown, so a selected profile
+    // always means profile auth for them
+    const useProfile = type === connectionMap.ssh
+      ? authType === 'profiles'
+      : true
+    if (!profile || !useProfile) {
       return tab
     }
     let p = window.store.profiles.find(x => x.id === profile)
@@ -224,10 +230,17 @@ export default Store => {
         ...tab,
         ...filtered
       }
+    } else if (type === connectionMap.ftp) {
+      const filtered = pickBy(p.ftp, (value) => value !== undefined && value !== '')
+      return {
+        ...tab,
+        ...filtered
+      }
     }
     delete p.rdp
     delete p.vnc
     delete p.telnet
+    delete p.ftp
     const filtered = pickBy(p, (value) => value !== undefined && value !== '')
     return {
       ...tab,
@@ -455,6 +468,11 @@ export default Store => {
     let i = len - 1
     for (;i >= 0; i--) {
       const f = profiles[i]
+      // migrate old rdp profile key userName -> username
+      if (f.rdp?.userName !== undefined) {
+        f.rdp.username = f.rdp.userName
+        delete f.rdp.userName
+      }
       if (f.name) {
         continue
       }

package/client/store/load-data.js

@@ -3,6 +3,7 @@
  */
 
 import { dbNames, getData, fetchInitData } from '../common/db'
+import deepCopy from 'json-deep-copy'
 import parseInt10 from '../common/parse-int10'
 import { infoTabs, statusMap, defaultEnvLang } from '../common/constants'
 import generate from '../common/id-with-stamp'
@@ -237,7 +238,11 @@ export default (Store) => {
         .then(arr => {
           for (const { name, data } of arr) {
             const dt = JSON.parse(data || '[]')
-            refsStatic.add('oldState-' + name, dt)
+            // seed the watcher snapshot with a *copy* - manate wraps the exact
+            // array assigned into the store, so seeding with `dt` itself would
+            // alias snapshot and store to one backing array; the watcher's
+            // no-change early-return would then skip DB writes forever
+            refsStatic.add('oldState-' + name, deepCopy(dt))
             if (name === 'bookmarks') {
               ext.bookmarksMap = new Map(
                 dt.map(d => [d.id, d])

package/client/store/store.js

@@ -42,6 +42,7 @@ import {
 } from '../common/constants'
 import getInitItem from '../common/init-setting-item'
 import createTitle from '../common/create-title'
+import buildMap from '../common/build-map'
 import {
   theme
 } from 'antd'
@@ -278,12 +279,7 @@ class Store {
     const {
       bookmarks
     } = window.store
-    return bookmarks.reduce((p, v) => {
-      return {
-        ...p,
-        [v.id]: v
-      }
-    }, {})
+    return buildMap(bookmarks)
   }
 
   hasSshConfig () {
@@ -296,12 +292,7 @@ class Store {
     const {
       bookmarkGroups
     } = window.store
-    return bookmarkGroups.reduce((p, v) => {
-      return {
-        ...p,
-        [v.id]: v
-      }
-    }, {})
+    return buildMap(bookmarkGroups)
   }
 
   get hasOldConnectionHoppingBookmark () {

package/client/store/sync.js

@@ -4,6 +4,7 @@
 
 import { get, pick, debounce } from 'lodash-es'
 import copy from 'json-deep-copy'
+import { action } from 'manate'
 import {
   settingMap, packInfo, syncTypes, syncDataMaps
 } from '../common/constants'
@@ -13,6 +14,9 @@ import download from '../common/download'
 import { fixBookmarks } from '../common/db-fix'
 import dayjs from 'dayjs'
 import parseJsonSafe from '../common/parse-json-safe'
+import { runImportTask } from '../common/import-task'
+
+const e = window.translate
 
 const {
   version: packVer
@@ -694,15 +698,40 @@ export default (Store) => {
     const { store } = window
     const objs = JSON.parse(txt)
     const { names } = store.getDataSyncNames(true)
+    const fixed = {}
     for (const n of names) {
-      let arr = objs[n]
+      let arr = objs[n] || []
       if (n === settingMap.terminalThemes) {
         arr = store.fixThemes(arr)
       } else if (n === settingMap.bookmarks) {
         arr = fixBookmarks(arr)
       }
-      store.setItems(n, objs[n])
+      fixed[n] = arr
     }
+    // clear all targets first - importAll replaces data sets, not appends.
+    // watchers are stopped inside runImportTask, so this is silent until restart
+    action(() => {
+      for (const n of names) {
+        store.setItems(n, [])
+      }
+    })()
+    await runImportTask({
+      title: e('import'),
+      batch: 200,
+      stopWatchers: names,
+      // one chunked step per data set (bookmarks, groups, themes, ...),
+      // so the progress bar advances per batch instead of per data set
+      steps: names.map((n) => {
+        const arr = fixed[n]
+        return {
+          label: n,
+          items: arr,
+          process: (chunk) => {
+            store[n].push(...chunk)
+          }
+        }
+      })
+    })
     store.updateConfig(stripServerManagedKeys(objs.config))
     if (objs.config?.theme) {
       store.setTheme(objs.config.theme)

package/client/store/watch.js

@@ -31,6 +31,16 @@ export default store => {
         old,
         n
       )
+      const newOrder = (n || []).map(d => d.id)
+      // with large collections (thousands of bookmarks) this autoRun fires
+      // often; when neither content nor order changed, skip the deep-copy
+      // snapshot and the DB writes entirely
+      const orderChanged = !old ||
+        old.length !== (n || []).length ||
+        old.some((d, i) => d.id !== newOrder[i])
+      if (!updated.length && !added.length && !removed.length && !orderChanged) {
+        return store[name]
+      }
       // Update snapshot immediately before async DB writes to prevent
       // race conditions: a second autoRun firing before the first
       // completes would see stale oldState and re-insert the same items,
@@ -41,7 +51,6 @@ export default store => {
         ...updated.map(item => update(item.id, item, name, false)),
         added.length ? insert(name, added) : Promise.resolve()
       ])
-      const newOrder = (n || []).map(d => d.id)
       await update(
         `${name}:order`,
         newOrder

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@electerm/electerm-react",
-  "version": "5.1.25",
+  "version": "5.2.0",
   "description": "react components src for electerm",
   "main": "./client/components/main/main.jsx",
   "license": "MIT",