perf: ♻️ Optimize terminal performance

electron/exposes/adbkit/index.js

@@ -1,5 +1,5 @@
 import util from 'node:util'
-import child_process from 'node:child_process'
+import { exec as _exec, spawn } from 'node:child_process'
 import path from 'node:path'
 import fs from 'node:fs'
 import dayjs from 'dayjs'
@@ -8,7 +8,7 @@ import appStore from '@electron/helpers/store.js'
 import { adbPath } from '@electron/configs/index.js'
 import { uniq } from 'lodash-es'
 
-const exec = util.promisify(child_process.exec)
+const exec = util.promisify(_exec)
 
 let client = null
 
@@ -37,7 +37,66 @@ appStore.onDidChange('common.adbPath', async (value, oldValue) => {
   client = Adb.createClient({ bin: value || adbPath })
 })
 
-const shell = async command => exec(`${adbPath} ${command}`)
+const shell = async (command) => {
+  const execPath = appStore.get('common.adbPath') || adbPath
+  return exec(`${execPath} ${command}`, {
+    env: { ...process.env },
+    shell: true,
+  })
+}
+
+const spawnShell = async (command, { stdout, stderr } = {}) => {
+  const spawnPath = appStore.get('common.adbPath') || adbPath
+  const args = command.split(' ')
+
+  console.log('adb.spawnShell.spawnPath', spawnPath)
+  console.log('adb.spawnShell.args', args)
+
+  const spawnProcess = spawn(`"${spawnPath}"`, args, {
+    env: { ...process.env },
+    shell: true,
+    encoding: 'utf8',
+  })
+
+  spawnProcess.stdout.on('data', (data) => {
+    const stringData = data.toString()
+
+    console.log('spawnProcess.stdout.data:', stringData)
+
+    if (stdout) {
+      stdout(stringData, spawnProcess)
+    }
+  })
+
+  let lastStderr = ''
+  spawnProcess.stderr.on('data', (data) => {
+    const stringData = data.toString()
+
+    lastStderr = stringData
+
+    console.error('spawnProcess.stderr.data:', stringData)
+
+    if (stderr) {
+      stderr(stringData, spawnProcess)
+    }
+  })
+
+  return new Promise((resolve, reject) => {
+    spawnProcess.on('close', (code) => {
+      if (code === 0) {
+        resolve()
+      }
+      else {
+        reject(new Error(lastStderr || `Command failed with code ${code}`))
+      }
+    })
+
+    spawnProcess.on('error', (err) => {
+      reject(err)
+    })
+  })
+}
+
 const getDevices = async () => client.listDevicesWithPaths()
 const deviceShell = async (id, command) => {
   const res = await client.getDevice(id).shell(command).then(Adb.util.readAll)
@@ -193,6 +252,7 @@ export default () => {
 
   return {
     shell,
+    spawnShell,
     getDevices,
     deviceShell,
     kill,

electron/exposes/gnirehtet/index.js

@@ -46,9 +46,12 @@ const shell = async (command, { debug = false, stdout, stderr } = {}) => {
     }
   })
 
+  let lastStderr = ''
   gnirehtetProcess.stderr.on('data', (data) => {
     const stringData = data.toString()
 
+    lastStderr = stringData
+
     if (debug) {
       console.error(`${command}.gnirehtet.process.stderr.data:`, stringData)
     }

electron/exposes/scrcpy/index.js

@@ -33,9 +33,12 @@ const shell = async (command, { stdout, stderr } = {}) => {
     }
   })
 
+  let lastStderr = ''
   scrcpyProcess.stderr.on('data', (data) => {
     const stringData = data.toString()
 
+    lastStderr = stringData
+
     console.error('scrcpyProcess.stderr.data:', stringData)
 
     if (stderr) {
@@ -49,7 +52,7 @@ const shell = async (command, { stdout, stderr } = {}) => {
         resolve()
       }
       else {
-        reject(new Error(`Command failed with code ${code}`))
+        reject(new Error(lastStderr || `Command failed with code ${code}`))
       }
     })
 

src/components/Device/components/Terminal/components/TerminalDialog/composables/adb-spawn.js

@@ -0,0 +1,44 @@
+import { debounce } from 'lodash-es'
+import { createStderr, createStdout, textFormatter } from 'vue-command'
+import { useFixCursor } from './helper.js'
+
+const $adb = window.adbkit
+
+export function useAdb({ vShell, history, loading } = {}) {
+  const adb = async (args) => {
+    loading.value = true
+
+    const command = args.slice(1).join(' ')
+
+    const appendToHistory = debounce(vShell.value.appendToHistory, 500)
+
+    let stdoutText = ''
+    let stderrText = ''
+    $adb.spawnShell(command, {
+      stdout(text) {
+        loading.value = false
+
+        stdoutText += text
+
+        useFixCursor(history)
+
+        appendToHistory(createStdout(stdoutText))
+      },
+      stderr(text) {
+        loading.value = false
+
+        stderrText += text
+
+        useFixCursor(history)
+
+        appendToHistory(createStderr(stderrText))
+      },
+    })
+
+    return textFormatter('Waiting...')
+  }
+
+  return {
+    adb,
+  }
+}

src/components/Device/components/Terminal/components/TerminalDialog/composables/gnirehtet.js

@@ -1,15 +1,9 @@
 import { debounce } from 'lodash-es'
 import { createStderr, createStdout, textFormatter } from 'vue-command'
+import { useFixCursor } from './helper.js'
 
 const $gnirehtet = window.gnirehtet
 
-const fixCursor = (history) => {
-  const length = history.value.length
-  if (history.value[length - 1]?.__name === 'VueCommandQuery') {
-    history.value.splice(length - 1, 1, textFormatter('Waiting...'))
-  }
-}
-
 export function useGnirehtet({ vShell, history, loading } = {}) {
   const gnirehtet = async (args) => {
     loading.value = true
@@ -26,7 +20,7 @@ export function useGnirehtet({ vShell, history, loading } = {}) {
 
         stdoutText += text
 
-        fixCursor(history)
+        useFixCursor(history)
 
         appendToHistory(createStdout(stdoutText))
       },
@@ -35,13 +29,13 @@ export function useGnirehtet({ vShell, history, loading } = {}) {
 
         stderrText += text
 
-        fixCursor(history)
+        useFixCursor(history)
 
         appendToHistory(createStderr(stderrText))
       },
     })
 
-    return textFormatter('Loading...')
+    return textFormatter('Waiting...')
   }
 
   return {

src/components/Device/components/Terminal/components/TerminalDialog/composables/helper.js

@@ -0,0 +1,8 @@
+import { textFormatter } from 'vue-command'
+
+export function useFixCursor(history) {
+  const length = history.value.length
+  if (history.value[length - 1]?.__name === 'VueCommandQuery') {
+    history.value.splice(length - 1, 1, textFormatter('Waiting...'))
+  }
+}

src/components/Device/components/Terminal/components/TerminalDialog/composables/scrcpy.js

@@ -1,15 +1,9 @@
 import { debounce } from 'lodash-es'
 import { createStderr, createStdout, textFormatter } from 'vue-command'
+import { useFixCursor } from './helper.js'
 
 const $scrcpy = window.scrcpy
 
-const fixCursor = (history) => {
-  const length = history.value.length
-  if (history.value[length - 1]?.__name === 'VueCommandQuery') {
-    history.value.splice(length - 1, 1, textFormatter('Waiting...'))
-  }
-}
-
 export function useScrcpy({ vShell, history, loading } = {}) {
   const scrcpy = async (args) => {
     loading.value = true
@@ -22,11 +16,12 @@ export function useScrcpy({ vShell, history, loading } = {}) {
     let stderrText = ''
     $scrcpy.shell(command, {
       stdout(text) {
+        console.log('history', history)
         loading.value = false
 
         stdoutText += text
 
-        fixCursor(history)
+        useFixCursor(history)
 
         appendToHistory(createStdout(stdoutText))
       },
@@ -35,13 +30,13 @@ export function useScrcpy({ vShell, history, loading } = {}) {
 
         stderrText += text
 
-        fixCursor(history)
+        useFixCursor(history)
 
         appendToHistory(createStderr(stderrText))
       },
     })
 
-    return textFormatter('Loading...')
+    return textFormatter('Waiting...')
   }
 
   return {