@electerm/electerm-react 2.3.198 → 2.4.16

package/client/components/terminal/terminal.jsx

@@ -8,13 +8,13 @@ import {
   ReloadOutlined
 } from '@ant-design/icons'
 import {
-  notification,
   Spin,
   Button,
-  Dropdown,
-  message,
-  Modal
+  Dropdown
 } from 'antd'
+import { notification } from '../common/notification'
+import message from '../common/message'
+import Modal from '../common/modal'
 import classnames from 'classnames'
 import './terminal.styl'
 import {
@@ -24,7 +24,6 @@ import {
   isWin,
   transferTypeMap,
   rendererTypes,
-  cwdId,
   isMac,
   zmodemTransferPackSize
 } from '../../common/constants.js'
@@ -51,6 +50,10 @@ import { getFilePath, isUnsafeFilename } from '../../common/file-drop-utils.js'
 import { CommandTrackerAddon } from './command-tracker-addon.js'
 import AIIcon from '../icons/ai-icon.jsx'
 import { formatBytes } from '../../common/byte-format.js'
+import {
+  getShellIntegrationCommand,
+  detectShellType
+} from './shell.js'
 import * as fs from './fs.js'
 import iconsMap from '../sys-menu/icons-map.jsx'
 import { refs, refsStatic } from '../common/ref.js'
@@ -59,19 +62,6 @@ import SearchResultBar from './terminal-search-bar'
 
 const e = window.translate
 
-const PS1_SETUP_CMD = `\recho $0|grep csh >/dev/null && set prompt_bak="$prompt" && set prompt="$prompt${cwdId}%/${cwdId}"\r
-echo $0|grep zsh >/dev/null && PS1_bak=$PS1&&PS1=$PS1'${cwdId}%d${cwdId}'\r
-echo $0|grep ash >/dev/null && PS1_bak=$PS1&&PS1=$PS1'\`echo ${cwdId}$PWD${cwdId}\`'\r
-echo $0|grep ksh >/dev/null && PS1_bak=$PS1&&PS1=$PS1'\`echo ${cwdId}$PWD${cwdId}\`'\r
-echo $0|grep '^sh' >/dev/null && PS1_bak=$PS1&&PS1=$PS1'\`echo ${cwdId}$PWD${cwdId}\`'\r
-clear\r`
-
-const PS1_RESTORE_CMD = `\recho $0|grep csh >/dev/null && set prompt="$prompt_bak"\r
-echo $0|grep zsh >/dev/null && PS1="$PS1_bak"\r
-echo $0|grep ash >/dev/null && PS1="$PS1_bak"\r
-echo $0|grep ksh >/dev/null && PS1="$PS1_bak"\r
-echo $0|grep '^sh' >/dev/null && PS1="$PS1_bak"\r
-clear\r`
 class Term extends Component {
   constructor (props) {
     super(props)
@@ -88,6 +78,7 @@ class Term extends Component {
     }
     this.id = `term-${this.props.tab.id}`
     refs.add(this.id, this)
+    this.currentInput = ''
   }
 
   domRef = createRef()
@@ -140,29 +131,6 @@ class Term extends Component {
         background: 'rgba(0,0,0,0)'
       }
     }
-
-    const sftpPathFollowSshChanged = !isEqual(
-      this.props.sftpPathFollowSsh,
-      prevProps.sftpPathFollowSsh
-    )
-
-    if (sftpPathFollowSshChanged) {
-      if (this.props.sftpPathFollowSsh) {
-        if (this.attachAddon && this.term) {
-          this.attachAddon._sendData(PS1_SETUP_CMD)
-          this.term.cwdId = cwdId
-        } else {
-          console.warn('Term or attachAddon not ready for PS1_SETUP_CMD in componentDidUpdate')
-        }
-      } else {
-        if (this.attachAddon) {
-          this.attachAddon._sendData(PS1_RESTORE_CMD)
-        }
-        if (this.term) {
-          delete this.term.cwdId
-        }
-      }
-    }
   }
 
   componentWillUnmount () {
@@ -321,8 +289,7 @@ class Term extends Component {
       ),
       okText: e('ok'),
       cancelText: e('cancel'),
-      onOk: () => this.onPaste(true),
-      onCancel: Modal.destroyAll
+      onOk: () => this.onPaste(true)
     })
   }
 
@@ -382,7 +349,7 @@ class Term extends Component {
         this.attachAddon._sendData(`"${filePath}" `)
         return
       } catch (e) {
-        log.error('Failed to parse fromFile data:', e)
+        console.error('Failed to parse fromFile data:', e)
       }
     }
 
@@ -441,12 +408,15 @@ class Term extends Component {
   }
 
   onzmodemRetract = () => {
-    log.debug('zmodemRetract')
+    console.debug('zmodemRetract')
   }
 
   writeBanner = (type) => {
-    this.term.write('\r\nRecommmend use trzsz instead: https://github.com/trzsz/trzsz\r\n')
-    this.term.write(`\x1b[32mZMODEM::${type}::START\x1b[0m\r\n\r\n`)
+    const border = '='.repeat(50)
+    this.term.write(`\r\n${border}\r\n`)
+    this.term.write('\x1b[33m\x1b[1mRecommend use trzsz instead: https://github.com/trzsz/trzsz\x1b[0m\r\n')
+    this.term.write(`${border}\r\n\r\n`)
+    this.term.write(`\x1b[32m\x1b[1mZMODEM::${type}::START\x1b[0m\r\n`)
   }
 
   onReceiveZmodemSession = async () => {
@@ -885,20 +855,16 @@ class Term extends Component {
   }
 
   getCwd = () => {
-    if (
-      this.props.sftpPathFollowSsh &&
-      this.term &&
-      this.term.buffer.active.type !== 'alternate' && !this.term.cwdId
-    ) {
-      // This block should ideally not be hit for initial setup if runInitScript works.
-      // It acts as a fallback.
-      this.term.cwdId = cwdId
-      if (this.attachAddon) {
-        this.attachAddon._sendData(PS1_SETUP_CMD)
-      } else {
-        console.warn('attachAddon not ready for PS1_SETUP_CMD in getCwd fallback')
+    // Use shell integration CWD if available
+    if (this.cmdAddon && this.cmdAddon.hasShellIntegration()) {
+      const cwd = this.cmdAddon.getCwd()
+      if (cwd) {
+        this.setCwd(cwd)
+        return cwd
       }
     }
+    // Fallback: no longer needed with shell integration
+    return ''
   }
 
   setCwd = (cwd) => {
@@ -942,35 +908,66 @@ class Term extends Component {
       ?.closeSuggestions()
   }
 
-  onData = (d) => {
-    if (this.cmdAddon) {
-      this.cmdAddon.handleData(d)
+  openSuggestions = (cursorPos, data) => {
+    refsStatic
+      .get('terminal-suggestions')
+      ?.openSuggestions(cursorPos, data)
+  }
+
+  getCurrentInput = () => {
+    return this.currentInput
+  }
+
+  setCurrentInput = (value) => {
+    this.currentInput = value
+  }
+
+  updateCurrentInput = (d) => {
+    // Handle backspace (both \x7f and \b)
+    if (d === '\x7f' || d === '\b') {
+      this.currentInput = this.currentInput.slice(0, -1)
+      return
+    }
+    // Handle Ctrl+U (clear line)
+    if (d === '\x15') {
+      this.currentInput = ''
+      return
     }
-    const data = this.getCmd().trim()
-    if (!d.includes('\r')) {
-      delete this.userTypeExit
+    // Handle Ctrl+W (delete word)
+    if (d === '\x17') {
+      this.currentInput = this.currentInput.replace(/\S*\s*$/, '')
+      return
+    }
+    // Handle Ctrl+C (cancel)
+    if (d === '\x03') {
+      this.currentInput = ''
+      return
+    }
+    // Handle Enter
+    if (d === '\r' || d === '\n') {
+      this.currentInput = ''
+      return
+    }
+    // Handle Escape and other control characters
+    if (d.charCodeAt(0) < 32 && d !== '\t') {
+      return
+    }
+    // Handle arrow keys and other escape sequences
+    if (d.startsWith('\x1b')) {
+      return
+    }
+    // Regular character input - append to buffer
+    this.currentInput += d
+  }
+
+  onData = (d) => {
+    this.updateCurrentInput(d)
+    const data = this.getCurrentInput()
+    if (this.props.config.showCmdSuggestions && data) {
       const cursorPos = this.getCursorPosition()
-      if (this.props.config.showCmdSuggestions && data.length > 1) {
-        refsStatic
-          .get('terminal-suggestions')
-          ?.openSuggestions(cursorPos, data)
-      }
+      this.openSuggestions(cursorPos, data)
     } else {
       this.closeSuggestions()
-      if (this.term.buffer.active.type !== 'alternate') {
-        this.timers.getCwd = setTimeout(this.getCwd, 200)
-      }
-      const exitCmds = [
-        'exit',
-        'logout'
-      ]
-      window.store.addCmdHistory(data)
-      if (exitCmds.includes(data)) {
-        this.userTypeExit = true
-        this.timers.userTypeExit = setTimeout(() => {
-          delete this.userTypeExit
-        }, 2000)
-      }
     }
   }
 
@@ -981,8 +978,8 @@ class Term extends Component {
       try {
         term.loadAddon(new WebglAddon())
       } catch (e) {
-        log.error('render with webgl failed, fallback to canvas')
-        log.error(e)
+        console.error('render with webgl failed, fallback to canvas')
+        console.error(e)
         term.loadAddon(new CanvasAddon())
       }
     }
@@ -1021,6 +1018,16 @@ class Term extends Component {
     // term.on('keydown', this.handleEvent)
     this.fitAddon = new FitAddon()
     this.cmdAddon = new CommandTrackerAddon()
+    // Register callback for shell integration command tracking
+    this.cmdAddon.onCommandExecuted((cmd) => {
+      if (cmd && cmd.trim()) {
+        window.store.addCmdHistory(cmd.trim())
+      }
+    })
+    // Register callback for shell integration CWD tracking
+    this.cmdAddon.onCwdChanged((cwd) => {
+      this.setCwd(cwd)
+    })
     this.searchAddon = new SearchAddon()
     const ligtureAddon = new LigaturesAddon()
     this.searchAddon.onDidChangeResults(this.onSearchResultsChange)
@@ -1035,7 +1042,6 @@ class Term extends Component {
     term.onData(this.onData)
     this.term = term
     term.onSelectionChange(this.onSelectionChange)
-    term.attachCustomKeyEventHandler(this.handleKeyboardEvent.bind(this))
     await this.remoteInit(term)
   }
 
@@ -1060,29 +1066,85 @@ class Term extends Component {
       startDirectory,
       runScripts
     } = this.props.tab
+
+    const scripts = runScripts ? [...runScripts] : []
     const startFolder = startDirectory || window.initFolder
     if (startFolder) {
-      if (this.attachAddon) {
-        const cmd = `cd "${startFolder}"\r`
-        this.attachAddon._sendData(cmd)
-      } else {
-        console.warn('attachAddon not ready for cd command in runInitScript')
-      }
+      scripts.unshift({ script: `cd "${startFolder}"`, delay: 0 })
     }
+    this.pendingRunScripts = scripts
 
-    if (this.props.sftpPathFollowSsh) {
-      if (this.term && this.attachAddon) {
-        this.attachAddon._sendData(PS1_SETUP_CMD)
-        this.term.cwdId = cwdId
-      } else {
-        console.warn('Term or attachAddon not ready for PS1_SETUP_CMD in runInitScript')
-      }
+    // Inject shell integration from client-side (works for both local and remote)
+    // Skip on Windows as shell integration is not supported there
+    if (this.canInjectShellIntegration()) {
+      this.injectShellIntegration()
+    } else {
+      // No shell integration, run scripts immediately
+      this.startDelayedScripts()
     }
+  }
+
+  canInjectShellIntegration = () => {
+    return this.isSsh() || this.isLocal()
+  }
+
+  isSsh = () => {
+    const { host, type } = this.props.tab
+    return host && (type === 'ssh' || type === undefined)
+  }
 
+  isLocal = () => {
+    const { host, type } = this.props.tab
+    return !host && (type === 'local' || type === undefined)
+  }
+
+  /**
+   * Start running delayed scripts
+   * Called after shell integration injection completes (or immediately if disabled)
+   */
+  startDelayedScripts = () => {
+    const runScripts = this.pendingRunScripts
     if (runScripts && runScripts.length) {
       this.delayedScripts = deepCopy(runScripts)
       this.timers.timerDelay = setTimeout(this.runDelayedScripts, this.delayedScripts[0].delay || 0)
     }
+    this.pendingRunScripts = null
+  }
+
+  /**
+   * Inject shell integration commands from client-side
+   * This replaces the server-side source xxx.xxx approach
+   * Uses output suppression to hide the injection command
+   */
+  injectShellIntegration = () => {
+    // Detect shell type from login script or local shell config
+    let shellType = 'bash'
+    if (this.isLocal()) {
+      const { config } = this.props
+      const localShell = isMac ? config.execMac : config.execLinux
+      shellType = detectShellType(localShell)
+    }
+
+    const isRemote = this.isSsh()
+
+    // Remote sessions might need longer timeout for shell integration detection
+    const integrationCmd = getShellIntegrationCommand(shellType)
+
+    if (integrationCmd && this.attachAddon) {
+      // Wait for initial data (prompt/banner) to arrive before injecting
+      this.attachAddon.onInitialData(() => {
+        if (this.attachAddon) {
+          // Start suppressing output before sending the integration command
+          // This hides the command and its output until OSC 633 is detected
+          const suppressionTimeout = isRemote ? 5000 : 3000
+          // Pass callback to run delayed scripts after suppression ends
+          this.attachAddon.startOutputSuppression(suppressionTimeout, () => {
+            this.startDelayedScripts()
+          })
+          this.attachAddon._sendData(integrationCmd)
+        }
+      })
+    }
   }
 
   runDelayedScripts = () => {
@@ -1292,13 +1354,13 @@ class Term extends Component {
       try {
         this.fitAddon.fit()
       } catch (e) {
-        log.info('resize failed')
+        console.info('resize failed')
       }
     }
   }, 200)
 
   onerrorSocket = err => {
-    log.error('onerrorSocket', err)
+    console.error('onerrorSocket', err)
   }
 
   oncloseSocket = () => {
@@ -1320,7 +1382,7 @@ class Term extends Component {
     }
     this.socketCloseWarning = notification.warning({
       key,
-      title: e('socketCloseTip'),
+      message: e('socketCloseTip'),
       duration: 30,
       description: (
         <div className='pd2y'>

package/client/components/theme/theme-form.jsx

@@ -1,5 +1,6 @@
 import { useRef, useState } from 'react'
-import { Button, Input, message, Upload, Form, Space } from 'antd'
+import { Button, Input, Upload, Form, Space } from 'antd'
+import message from '../common/message'
 import {
   convertTheme,
   convertThemeToText,

package/client/components/tree-list/bookmark-transport.jsx

@@ -8,7 +8,7 @@ import {
   ImportOutlined,
   EditOutlined
 } from '@ant-design/icons'
-import { Upload, Button } from 'antd'
+import { Upload, Button, Space } from 'antd'
 
 const e = window.translate
 
@@ -32,16 +32,20 @@ export default class BookmarkTransport extends PureComponent {
     )
   }
 
-  render () {
-    return [
-      this.renderEdit(),
+  renderExport () {
+    return (
       <Button
         icon={<ExportOutlined />}
         onClick={this.handleDownload}
         title={e('export')}
         className='download-bookmark-icon'
         key='export'
-      />,
+      />
+    )
+  }
+
+  renderImport () {
+    return (
       <Upload
         beforeUpload={this.beforeUpload}
         fileList={[]}
@@ -53,6 +57,24 @@ export default class BookmarkTransport extends PureComponent {
           title={e('importFromFile')}
         />
       </Upload>
+    )
+  }
+
+  render () {
+    const edit = this.renderEdit()
+    console.log('edit', edit)
+    if (edit === null) {
+      return (
+        <Space.Compact>
+          {this.renderExport()}
+          {this.renderImport()}
+        </Space.Compact>
+      )
+    }
+    return [
+      edit,
+      this.renderExport(),
+      this.renderImport()
     ]
   }
 }

package/client/components/vnc/vnc-session.jsx

@@ -9,9 +9,9 @@ import {
 } from '../../common/constants'
 import {
   Spin,
-  message,
   Tag
 } from 'antd'
+import message from '../common/message'
 import Modal from '../common/modal'
 import * as ls from '../../common/safe-local-storage'
 import { copy } from '../../common/clipboard'

package/client/components/widgets/widget-notification-with-details.jsx

@@ -1,4 +1,4 @@
-import { notification } from 'antd'
+import { notification } from '../common/notification'
 import { CopyOutlined } from '@ant-design/icons'
 import { copy } from '../../common/clipboard'
 

package/client/store/common.js

@@ -3,7 +3,7 @@
  */
 
 import handleError from '../common/error-handler'
-import { Modal } from 'antd'
+import Modal from '../components/common/modal'
 import { debounce, some, get, pickBy } from 'lodash-es'
 import {
   modals,
@@ -117,7 +117,7 @@ export default Store => {
       setTimeout(queue, 3000)
       setTimeout(run, 4000)
     } catch (e) {
-      log.error(e)
+      console.error(e)
     }
   }
 
@@ -341,6 +341,9 @@ export default Store => {
   }
 
   Store.prototype.addCmdHistory = action(function (cmd) {
+    if (!cmd || !cmd.trim()) {
+      return
+    }
     const { terminalCommandHistory } = window.store
     terminalCommandHistory.add(cmd)
     if (terminalCommandHistory.size > 100) {

package/client/store/db-upgrade.js

@@ -2,7 +2,7 @@
  * db upgrade
  */
 
-import { Modal } from 'antd'
+import Modal from '../components/common/modal'
 import delay from '../common/wait'
 
 export default (Store) => {

package/client/store/init-state.js

@@ -9,6 +9,7 @@ import {
   infoTabs,
   openedSidebarKey,
   sidebarPinnedKey,
+  pinnedQuickCommandBarKey,
   sftpDefaultSortSettingKey,
   batchInputLsKey,
   expandedKeysLsKey,
@@ -150,7 +151,7 @@ export default () => {
     quickCommands: [],
     quickCommandId: '',
     openQuickCommandBar: false,
-    pinnedQuickCommandBar: false,
+    pinnedQuickCommandBar: ls.getItem(pinnedQuickCommandBarKey) === 'y',
     qmSortByFrequency: ls.getItem(qmSortByFrequencyKey) === 'yes',
 
     // sidebar

package/client/store/load-data.js

@@ -35,7 +35,7 @@ function getHost (argv, opts) {
 }
 
 export async function addTabFromCommandLine (store, opts) {
-  log.debug('command line params', opts)
+  console.debug('command line params', opts)
   if (!opts) {
     return false
   }
@@ -94,7 +94,7 @@ export async function addTabFromCommandLine (store, opts) {
   if (options.privateKeyPath) {
     conf.privateKey = await fs.readFile(options.privateKeyPath)
   }
-  log.debug('command line opts', conf)
+  console.debug('command line opts', conf)
   if (
     (conf.username && conf.host) ||
     conf.fromCmdLine

package/client/store/mcp-handler.js

@@ -5,6 +5,8 @@
 
 import uid from '../common/uid'
 import { settingMap } from '../common/constants'
+import { refs } from '../components/common/ref'
+import deepCopy from 'json-deep-copy'
 
 export default Store => {
   // Initialize MCP handler - called when MCP widget is started
@@ -142,27 +144,8 @@ export default Store => {
 
   // ==================== Bookmark APIs ====================
 
-  Store.prototype.mcpListBookmarks = function (args = {}) {
-    const { store } = window
-    let bookmarks = store.bookmarks
-
-    if (args.groupId) {
-      const group = store.bookmarkGroups.find(g => g.id === args.groupId)
-      if (group && group.bookmarkIds) {
-        const idSet = new Set(group.bookmarkIds)
-        bookmarks = bookmarks.filter(b => idSet.has(b.id))
-      }
-    }
-
-    return bookmarks.map(b => ({
-      id: b.id,
-      title: b.title,
-      host: b.host,
-      port: b.port,
-      username: b.username,
-      type: b.type || 'ssh',
-      color: b.color
-    }))
+  Store.prototype.mcpListBookmarks = function () {
+    return deepCopy(window.store.bookmarks)
   }
 
   Store.prototype.mcpGetBookmark = function (args) {
@@ -171,9 +154,7 @@ export default Store => {
     if (!bookmark) {
       throw new Error(`Bookmark not found: ${args.id}`)
     }
-    // Return bookmark without sensitive data
-    const { password, passphrase, privateKey, ...safeBookmark } = bookmark
-    return safeBookmark
+    return deepCopy(bookmark)
   }
 
   Store.prototype.mcpAddBookmark = async function (args) {
@@ -218,16 +199,11 @@ export default Store => {
 
   Store.prototype.mcpDeleteBookmark = function (args) {
     const { store } = window
-    const bookmark = store.bookmarks.find(b => b.id === args.id)
-    if (!bookmark) {
-      throw new Error(`Bookmark not found: ${args.id}`)
-    }
-
     store.delItem({ id: args.id }, settingMap.bookmarks)
 
     return {
       success: true,
-      message: `Bookmark "${bookmark.title}" deleted`
+      message: `Bookmark "${args.id}" deleted`
     }
   }
 
@@ -249,14 +225,7 @@ export default Store => {
   // ==================== Bookmark Group APIs ====================
 
   Store.prototype.mcpListBookmarkGroups = function () {
-    const { store } = window
-    return store.bookmarkGroups.map(g => ({
-      id: g.id,
-      title: g.title,
-      level: g.level,
-      bookmarkCount: (g.bookmarkIds || []).length,
-      subgroupCount: (g.bookmarkGroupIds || []).length
-    }))
+    return deepCopy(window.store.bookmarkGroups)
   }
 
   Store.prototype.mcpAddBookmarkGroup = async function (args) {
@@ -281,15 +250,7 @@ export default Store => {
   // ==================== Quick Command APIs ====================
 
   Store.prototype.mcpListQuickCommands = function () {
-    const { store } = window
-    return store.quickCommands.map(q => ({
-      id: q.id,
-      name: q.name,
-      command: q.command,
-      commands: q.commands,
-      inputOnly: q.inputOnly,
-      labels: q.labels
-    }))
+    return deepCopy(window.store.quickCommands)
   }
 
   Store.prototype.mcpAddQuickCommand = function (args) {
@@ -297,7 +258,7 @@ export default Store => {
     const qm = {
       id: uid(),
       name: args.name,
-      command: args.command,
+      commands: args.commands,
       inputOnly: args.inputOnly || false,
       labels: args.labels || []
     }
@@ -475,7 +436,6 @@ export default Store => {
 
   Store.prototype.mcpGetTerminalSelection = function (args) {
     const { store } = window
-    const { refs } = require('../components/common/ref')
     const tabId = args.tabId || store.activeTabId
 
     if (!tabId) {
@@ -497,7 +457,6 @@ export default Store => {
 
   Store.prototype.mcpGetTerminalOutput = function (args) {
     const { store } = window
-    const { refs } = require('../components/common/ref')
     const tabId = args.tabId || store.activeTabId
     const lineCount = args.lines || 50