@lvce-editor/shared-process 0.11.4 → 0.11.6

package/config/defaultKeyBindings.json

@@ -742,5 +742,9 @@
     "key": "Enter",
     "command": "SimpleBrowser.go",
     "when": "focus.SimpleBrowserInput"
+  },
+  {
+    "key": "ctrl+r",
+    "command": "Window.reload"
   }
 ]

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lvce-editor/shared-process",
-  "version": "0.11.4",
+  "version": "0.11.6",
   "description": "Utility package for @lvce-editor/server",
   "main": "index.js",
   "type": "module",
@@ -17,9 +17,9 @@
   },
   "dependencies": {
     "@babel/code-frame": "^7.18.6",
-    "@lvce-editor/extension-host": "0.11.4",
-    "@lvce-editor/extension-host-helper-process": "0.11.4",
-    "@lvce-editor/pty-host": "0.11.4",
+    "@lvce-editor/extension-host": "0.11.6",
+    "@lvce-editor/extension-host-helper-process": "0.11.6",
+    "@lvce-editor/pty-host": "0.11.6",
     "debug": "^4.3.4",
     "execa": "^6.1.0",
     "exit-hook": "^3.1.2",

package/src/parts/Cli/Cli.js

@@ -1,3 +1,6 @@
+import * as Logger from '../Logger/Logger.js'
+import * as Process from '../Process/Process.js'
+
 const getModule = (argv0) => {
   switch (argv0) {
     case 'install':
@@ -20,7 +23,7 @@ export const handleCliArgs = async (argv, console, process) => {
   try {
     await module.handleCliArgs(argv, console, process)
   } catch (error) {
-    console.error(error)
-    process.exitCode = 1
+    Logger.error(error)
+    Process.setExitCode(1)
   }
 }

package/src/parts/CliInstall/CliInstall.js

@@ -1,9 +1,10 @@
 import * as ExtensionInstall from '../ExtensionInstall/ExtensionInstall.js'
+import * as Logger from '../Logger/Logger.js'
 
 export const handleCliArgs = async (argv, console) => {
   const extension = argv[1]
   if (!extension) {
-    console.error('extension argument is required')
+    Logger.error('extension argument is required')
     return
   }
   await ExtensionInstall.install(extension)

package/src/parts/CliLink/CliLink.js

@@ -1,5 +1,7 @@
 import * as ExtensionLink from '../ExtensionLink/ExtensionLink.js'
 import * as ErrorCodes from '../ErrorCodes/ErrorCodes.js'
+import * as Logger from '../Logger/Logger.js'
+import * as Process from '../Process/Process.js'
 
 export const handleCliArgs = async (argv) => {
   try {
@@ -7,8 +9,8 @@ export const handleCliArgs = async (argv) => {
     await ExtensionLink.link(path)
   } catch (error) {
     if (error && error.code === ErrorCodes.E_MANIFEST_NOT_FOUND) {
-      console.error(error.message)
-      process.exitCode = 128
+      Logger.error(error.message)
+      Process.setExitCode(128)
       return
     }
     throw error

package/src/parts/CliUnlink/CliUnlink.js

@@ -1,6 +1,7 @@
 import * as ExtensionUnlink from '../ExtensionUnlink/ExtensionUnlink.js'
+import * as Process from '../Process/Process.js'
 
 export const handleCliArgs = async (argv, console, process) => {
-  const path = argv.length > 1 ? argv[1] : process.cwd()
+  const path = argv.length > 1 ? argv[1] : Process.cwd()
   await ExtensionUnlink.unlink(path)
 }

package/src/parts/ClipBoard/ClipBoardGnome.js

@@ -4,6 +4,7 @@
 
 import VError from 'verror'
 import * as Exec from '../Exec/Exec.js'
+import * as SplitLines from '../SplitLines/SplitLines.js'
 
 const removePrefix = (file) => {
   if (file.startsWith('file://')) {
@@ -35,7 +36,7 @@ export const readFiles = async () => {
     }
     throw error
   }
-  const [type, ...files] = result.stdout.split('\n')
+  const [type, ...files] = SplitLines.splitLines(result.stdout)
   const actualFiles = files.map(removePrefix)
   return {
     source: 'gnomeCopiedFiles',

package/src/parts/Developer/Developer.js

@@ -1,6 +1,8 @@
 import { createWriteStream, writeFileSync } from 'node:fs'
 import { performance } from 'node:perf_hooks'
 import * as ExtensionHost from '../ExtensionHost/ExtensionHost.js'
+import * as Process from '../Process/Process.js'
+import * as Timeout from '../Timeout/Timeout.js'
 
 export const measureLatencyBetweenExtensionHostAndSharedProcess = async (
   socket,
@@ -63,7 +65,7 @@ export const getNodeStartupTiming = () => {
 
 export const sharedProcessMemoryUsage = () => {
   // TODO also print out number of watched files
-  return process.memoryUsage()
+  return Process.memoryUsage()
 }
 
 // TODO not sure if this is actually useful
@@ -76,7 +78,7 @@ export const allocateMemory = () => {
 
 /* istanbul ignore next */
 export const crashSharedProcess = () => {
-  setTimeout(() => {
+  Timeout.setTimeout(() => {
     throw new Error('oops')
   }, 0)
 }
@@ -112,7 +114,7 @@ export const createProfile = async () => {
       session.post('Profiler.start', () => {
         // Invoke business logic under measurement here...
 
-        setTimeout(() => {
+        Timeout.setTimeout(() => {
           session.post('Profiler.stop', (error, { profile }) => {
             // Write profile to disk, upload, etc.
             if (!error) {

package/src/parts/ErrorCodes/ErrorCodes.js

@@ -7,3 +7,15 @@ export const EEXIST = 'EEXIST'
 export const ENOTDIR = 'ENOTDIR'
 
 export const E_MANIFEST_NOT_FOUND = 'E_MANIFEST_NOT_FOUND'
+
+export const EPIPE = 'EPIPE'
+
+export const ERR_IPC_CHANNEL_CLOSED = 'ERR_IPC_CHANNEL_CLOSED'
+
+export const E_COLOR_THEME_NOT_FOUND = 'E_COLOR_THEME_NOT_FOUND'
+
+export const E_ICON_THEME_NOT_FOUND = 'E_ICON_THEME_NOT_FOUND'
+
+export const ECONNRESET = 'ECONNRESET'
+
+export const EPERM = 'EPERM'

package/src/parts/ErrorHandling/ErrorHandling.js

@@ -1,4 +1,7 @@
+import * as ErrorCodes from '../ErrorCodes/ErrorCodes.js'
+import * as PrettyError from '../PrettyError/PrettyError.js'
 import * as Socket from '../Socket/Socket.js'
+import * as Process from '../Process/Process.js'
 
 export const state = {
   seenErrors: [],
@@ -38,3 +41,39 @@ export const handleError = (error) => {
   })
   printPrettyError(prettyError)
 }
+
+// TODO handle structure: one shared process multiple extension hosts
+
+// TODO use named functions here
+
+const firstErrorLine = (error) => {
+  if (error.stack) {
+    return error.stack.slice(0, error.stack.indexOf('\n'))
+  }
+  if (error.message) {
+    return error.message
+  }
+  return `${error}`
+}
+
+export const handleUncaughtExceptionMonitor = (error, origin) => {
+  console.info(`[shared process] uncaught exception: ${firstErrorLine(error)}`)
+  if (error && error.code === ErrorCodes.EPIPE && !process.connected) {
+    // parent process is disposed, ignore
+    return
+  }
+  if (
+    error &&
+    error.code === ErrorCodes.ERR_IPC_CHANNEL_CLOSED &&
+    !Process.isConnected()
+  ) {
+    // parent process is disposed, ignore
+    return
+  }
+  // console.log(error)
+  const prettyError = PrettyError.prepare(error)
+  // console.error(prettyError.message)
+  console.error(prettyError.codeFrame)
+  console.error(prettyError.stack)
+  Process.exit(1)
+}

package/src/parts/ExtensionInstallFromFile/ExtensionInstallFromFile.js

@@ -1,7 +1,7 @@
 import VError from 'verror'
-import * as Assert from '../Assert/Assert.js'
 import * as Extract from '../Extract/Extract.js'
 import * as FileSystem from '../FileSystem/FileSystem.js'
+import * as JsonFile from '../JsonFile/JsonFile.js'
 import * as Path from '../Path/Path.js'
 import * as Platform from '../Platform/Platform.js'
 
@@ -24,8 +24,7 @@ export const install = async ({ path }) => {
     await Extract.extractTarBr(path, cachedExtensionPath)
     const extensionsPath = Platform.getExtensionsPath()
     const manifestPath = Path.join(cachedExtensionPath, 'extension.json')
-    const manifestContent = await FileSystem.readFile(manifestPath)
-    const manifestJson = JSON.parse(manifestContent)
+    const manifestJson = await JsonFile.readJson(manifestPath)
     const id = manifestJson.id
     if (!id) {
       throw new Error('missing id in extension manifest')

package/src/parts/ExtensionInstallFromGitHub/ExtensionInstallFromGitHub.js

@@ -1,6 +1,7 @@
 import VError from 'verror'
 import * as DownloadAndExtract from '../DownloadAndExtract/DownloadAndExtract.js'
 import * as FileSystem from '../FileSystem/FileSystem.js'
+import * as JsonFile from '../JsonFile/JsonFile.js'
 import * as Path from '../Path/Path.js'
 import * as Platform from '../Platform/Platform.js'
 
@@ -19,8 +20,7 @@ export const install = async ({ user, repo, branch }) => {
     })
     const extensionsPath = Platform.getExtensionsPath()
     const manifestPath = Path.join(cachedExtensionPath, 'extension.json')
-    const manifestContent = await FileSystem.readFile(manifestPath)
-    const manifestJson = JSON.parse(manifestContent)
+    const manifestJson = await JsonFile.readJson(manifestPath)
     const id = manifestJson.id
     if (!id) {
       throw new Error('missing id in extension manifest')

package/src/parts/ExtensionInstallFromUrl/ExtensionInstallFromUrl.js

@@ -1,11 +1,11 @@
-import { readFile, rename, rm } from 'fs/promises'
+import { rename, rm } from 'fs/promises'
 import { join } from 'path'
 import VError from 'verror'
 import * as Download from '../Download/Download.js'
 import * as Extract from '../Extract/Extract.js'
+import * as JsonFile from '../JsonFile/JsonFile.js'
 import * as Platform from '../Platform/Platform.js'
 import * as TmpFile from '../TmpFile/TmpFile.js'
-import * as EncodingType from '../EncodingType/EncodingType.js'
 
 export const install = async ({ url }) => {
   try {
@@ -18,8 +18,7 @@ export const install = async ({ url }) => {
     const tmpDir = await TmpFile.getTmpDir()
     await Extract.extractTarBr(tmpFile, tmpDir)
     const manifestPath = join(tmpDir, 'extension.json')
-    const manifestContent = await readFile(manifestPath, EncodingType.Utf8)
-    const manifestJson = JSON.parse(manifestContent)
+    const manifestJson = await JsonFile.readJson(manifestPath)
     const id = manifestJson.id
     if (!id) {
       throw new Error('missing id in extension manifest')

package/src/parts/ExtensionManagement/ExtensionManagementColorTheme.js

@@ -1,9 +1,10 @@
 import VError from 'verror'
 import * as Error from '../Error/Error.js'
+import * as ErrorCodes from '../ErrorCodes/ErrorCodes.js'
+import * as FileSystemWatch from '../FileSystemWatch/FileSystemWatch.js'
 import * as ReadJson from '../JsonFile/JsonFile.js'
 import * as Path from '../Path/Path.js'
 import * as ExtensionManagement from './ExtensionManagement.js'
-import * as FileSystemWatch from '../FileSystemWatch/FileSystemWatch.js'
 
 // TODO test this function
 // TODO very similar with getIconTheme
@@ -29,7 +30,7 @@ export const getColorThemeJson = async (colorThemeId) => {
   const colorThemePath = await getColorThemePath(extensions, colorThemeId)
   if (!colorThemePath) {
     throw new Error.OperationalError({
-      code: 'E_COLOR_THEME_NOT_FOUND',
+      code: ErrorCodes.E_COLOR_THEME_NOT_FOUND,
       message: `Color theme "${colorThemeId}" not found in extensions folder`,
     })
   }

package/src/parts/ExtensionManagement/ExtensionManagementIconTheme.js

@@ -1,5 +1,6 @@
 import VError from 'verror'
 import * as Error from '../Error/Error.js'
+import * as ErrorCodes from '../ErrorCodes/ErrorCodes.js'
 import * as ReadJson from '../JsonFile/JsonFile.js'
 import * as Path from '../Path/Path.js'
 import * as ExtensionManagement from './ExtensionManagement.js'
@@ -27,7 +28,7 @@ export const getIconTheme = async (iconThemeId) => {
     }
   }
   throw new Error.OperationalError({
-    code: 'E_ICON_THEME_NOT_FOUND',
+    code: ErrorCodes.E_ICON_THEME_NOT_FOUND,
     message: `Icon theme "${iconThemeId}" not found in extensions folder`,
   })
 }

package/src/parts/GetResponse/GetResponse.js

@@ -32,6 +32,7 @@ export const getResponse = async (message, handle) => {
         },
       }
     }
+    // @ts-ignore
     if (error && error instanceof Error && error.code === ErrorCodes.ENOENT) {
       return {
         jsonrpc: JsonRpc.Version,
@@ -43,7 +44,7 @@ export const getResponse = async (message, handle) => {
       }
     }
     const prettyError = PrettyError.prepare(error)
-    PrettyError.print(prettyError)
+    PrettyError.print(prettyError, `[shared-process] `)
     return {
       jsonrpc: JsonRpc.Version,
       id: message.id,

package/src/parts/Logger/Logger.js

@@ -0,0 +1,50 @@
+import { Console } from 'node:console'
+import { createWriteStream } from 'node:fs'
+import { tmpdir } from 'node:os'
+
+// TODO mock this module when used in unit tests
+
+const state = {
+  /**
+   * @type {Console|undefined}
+   */
+  console: undefined,
+}
+
+const createConsole = () => {
+  const logFile = `${tmpdir()}/log-shared-process.txt`
+  const writeStream = createWriteStream(logFile)
+  const logger = new Console(writeStream)
+  return logger
+}
+
+const getOrCreateLogger = () => {
+  if (!state.console) {
+    state.console = createConsole()
+  }
+  return state.console
+}
+
+export const log = (...args) => {
+  const logger = getOrCreateLogger()
+  logger.log(...args)
+  console.log(...args)
+}
+
+export const info = (...args) => {
+  const logger = getOrCreateLogger()
+  logger.info(...args)
+  console.info(...args)
+}
+
+export const warn = (...args) => {
+  const logger = getOrCreateLogger()
+  logger.warn(...args)
+  console.warn(...args)
+}
+
+export const error = (...args) => {
+  const logger = getOrCreateLogger()
+  logger.error(...args)
+  console.error(...args)
+}

package/src/parts/ParentIpc/ParentIpc.js

@@ -3,6 +3,7 @@ import VError from 'verror'
 import * as Callback from '../Callback/Callback.js'
 import * as Command from '../Command/Command.js'
 import * as Debug from '../Debug/Debug.js'
+import * as ErrorCodes from '../ErrorCodes/ErrorCodes.js'
 import * as ExtensionHostHelperProcessIpc from '../ExtensionHostHelperProcessIpc/ExtensionHostHelperProcessIpc.js'
 import * as ExtensionHostIpc from '../ExtensionHostIpc/ExtensionHostIpc.js'
 import * as ExtensionHostRpc from '../ExtensionHostRpc/ExtensionHostRpc.js'
@@ -36,7 +37,7 @@ export const electronSend = (message) => {
 const handleWebSocketSharedProcess = (message, handle) => {
   // TODO when it is an extension host websocket, spawn extension host
   handle.on('error', (error) => {
-    if (error && error.code === 'ECONNRESET') {
+    if (error && error.code === ErrorCodes.ECONNRESET) {
       return
     }
     console.info('[info shared process: handle error]', error)

package/src/parts/PrettyError/PrettyError.js

@@ -1,8 +1,9 @@
-import { readFileSync } from 'node:fs'
-import { fileURLToPath } from 'node:url'
 import { codeFrameColumns } from '@babel/code-frame'
 import cleanStack from 'clean-stack'
 import { LinesAndColumns } from 'lines-and-columns'
+import { readFileSync } from 'node:fs'
+import { fileURLToPath } from 'node:url'
+import * as SplitLines from '../SplitLines/SplitLines.js'
 
 const getActualPath = (fileUri) => {
   if (fileUri.startsWith('file://')) {
@@ -20,7 +21,7 @@ export const prepare = (error) => {
     }
   }
   const cleanedStack = cleanStack(error.stack)
-  const lines = cleanedStack.split('\n')
+  const lines = SplitLines.splitLines(cleanedStack)
   const file = lines[1]
   let codeFrame = ''
   if (error.codeFrame) {
@@ -82,8 +83,8 @@ export const prepareJsonError = (json, property, message) => {
   return jsonError
 }
 
-export const print = (prettyError) => {
+export const print = (prettyError, prefix = '') => {
   console.error(
-    `Error: ${prettyError.message}\n\n${prettyError.codeFrame}\n\n${prettyError.stack}`
+    `${prefix}Error: ${prettyError.message}\n\n${prettyError.codeFrame}\n\n${prettyError.stack}`
   )
 }

package/src/parts/Process/Process.js

@@ -5,3 +5,27 @@ export const crash = (message) => {
 export const crashAsync = async (message) => {
   throw new Error(message)
 }
+
+export const exit = (code) => {
+  process.exit(code)
+}
+
+export const setExitCode = (exitCode) => {
+  process.exitCode = exitCode
+}
+
+export const argv = process.argv
+
+export const cwd = () => {
+  return process.cwd()
+}
+
+export const memoryUsage = () => {
+  return process.memoryUsage()
+}
+
+export const isConnected = () => {
+  return process.connected
+}
+
+export const platform = process.platform

package/src/parts/RecentlyOpened/RecentlyOpened.js

@@ -4,6 +4,7 @@ import * as Assert from '../Assert/Assert.js'
 import { FileNotFoundError } from '../Error/FileNotFoundError.js'
 import * as FileSystem from '../FileSystem/FileSystem.js'
 import * as Json from '../Json/Json.js'
+import * as JsonFile from '../JsonFile/JsonFile.js'
 import * as Platform from '../Platform/Platform.js'
 
 const isValid = (recentlyOpened) => {
@@ -24,8 +25,7 @@ const addToArrayUnique = (recentlyOpened, path) => {
 
 const getRecentlyOpened = async (recentlyOpenedPath) => {
   try {
-    const content = await FileSystem.readFile(recentlyOpenedPath)
-    const parsed = await Json.parse(content, recentlyOpenedPath)
+    const parsed = await JsonFile.readJson(recentlyOpenedPath)
     return parsed
   } catch (error) {
     // TODO should check for error.code

package/src/parts/RipGrepParsedLineType/RipGrepParsedLineType.js

@@ -0,0 +1,4 @@
+export const Begin = 'begin'
+export const Match = 'match'
+export const Summary = 'summary'
+export const Context = 'context'

package/src/parts/Search/Search.js

@@ -1,6 +1,8 @@
 import { spawn } from 'node:child_process'
-import * as RgPath from '../RgPath/RgPath.js'
 import * as Platform from '../Platform/Platform.js'
+import * as RgPath from '../RgPath/RgPath.js'
+import * as RipGrepParsedLineType from '../RipGrepParsedLineType/RipGrepParsedLineType.js'
+import * as SplitLines from '../SplitLines/SplitLines.js'
 
 const MAX_SEARCH_RESULTS = 300
 
@@ -25,64 +27,51 @@ const useNice = !Platform.isWindows
 // TODO update client
 // TODO not always run nice, maybe configure nice via flag/options
 
-const ParsedLineType = {
-  Begin: 'begin',
-  Match: 'match',
-  Summary: 'summary',
-}
-
 export const search = async (searchDir, searchString) => {
   // TODO reject promise when ripgrep search fails
   return new Promise((resolve, reject) => {
+    const ripGrepArgs = [
+      '--smart-case',
+      '--stats',
+      '--json',
+      '--fixed-strings',
+      searchString,
+      '.',
+    ]
     const childProcess = useNice
-      ? spawn(
-          'nice',
-          [
-            '-20',
-            RgPath.rgPath,
-            '--smart-case',
-            '--stats',
-            '--json',
-            searchString,
-            '.',
-          ],
-          {
-            cwd: searchDir,
-          }
-        )
-      : spawn(
-          RgPath.rgPath,
-          ['--smart-case', '--stats', '--json', searchString, '.'],
-          {
-            cwd: searchDir,
-          }
-        )
+      ? spawn('nice', ['-20', RgPath.rgPath, ...ripGrepArgs], {
+          cwd: searchDir,
+        })
+      : spawn(RgPath.rgPath, ripGrepArgs, {
+          cwd: searchDir,
+        })
     const allSearchResults = Object.create(null)
     let buffer = ''
     let stats = {}
+    let limitHit = false
     let numberOfResults = 0
     // TODO use pipeline / transform stream maybe
 
     const handleData = (chunk) => {
       buffer += chunk
-      const lines = buffer.split('\n')
+      const lines = SplitLines.splitLines(buffer)
       // @ts-ignore
       buffer = lines.pop()
       for (const line of lines) {
         const parsedLine = JSON.parse(line)
+        console.log(parsedLine)
         switch (parsedLine.type) {
-          case ParsedLineType.Begin: {
+          case RipGrepParsedLineType.Begin: {
             allSearchResults[parsedLine.data.path.text] = []
             break
           }
-          case ParsedLineType.Match: {
+          case RipGrepParsedLineType.Match:
             numberOfResults++
             allSearchResults[parsedLine.data.path.text].push(
               toSearchResult(parsedLine)
             )
             break
-          }
-          case ParsedLineType.Summary:
+          case RipGrepParsedLineType.Summary:
             stats = parsedLine.data
             break
           default:
@@ -90,6 +79,7 @@ export const search = async (searchDir, searchString) => {
         }
       }
       if (numberOfResults > MAX_SEARCH_RESULTS) {
+        limitHit = true
         childProcess.kill()
       }
     }
@@ -98,6 +88,7 @@ export const search = async (searchDir, searchString) => {
       resolve({
         results: Object.entries(allSearchResults),
         stats,
+        limitHit,
       })
     }
     const handleError = (error) => {
@@ -106,6 +97,7 @@ export const search = async (searchDir, searchString) => {
       resolve({
         results: [],
         stats,
+        limitHit,
       })
     }
     childProcess.stdout.on('data', handleData)

package/src/parts/Socket/Socket.js

@@ -41,7 +41,6 @@ const handleMessage = async (event) => {
 }
 
 const handleError = (error) => {
-  console.log('EEERRR')
   // console.error(error)
   throw new VError(error, 'Shared Process Socket Error')
 

package/src/parts/SplitLines/SplitLines.js

@@ -0,0 +1,3 @@
+export const splitLines = (lines) => {
+  return lines.split('\n')
+}

package/src/parts/Stats/Stats.js

@@ -1,9 +1,9 @@
 // parse ps output based on vscode https://github.com/microsoft/vscode/blob/c0769274fa136b45799edeccc0d0a2f645b75caf/src/vs/base/node/ps.ts (License MIT)
 
 import * as Exec from '../Exec/Exec.js'
+import * as SplitLines from '../SplitLines/SplitLines.js'
 
-const PID_CMD =
-  /^\s*(\d+)\s+(\d+)\s+(\d+\.\d+)\s+(\d+\.\d+)\s+(.+)$/
+const PID_CMD = /^\s*(\d+)\s+(\d+)\s+(\d+\.\d+)\s+(\d+\.\d+)\s+(.+)$/
 
 const parsePsOutputLine = (line) => {
   const matches = PID_CMD.exec(line.trim())
@@ -18,7 +18,7 @@ const parsePsOutputLine = (line) => {
   }
 }
 const parsePsOutput = (stdout, rootPid) => {
-  const lines = stdout.split('\n')
+  const lines = SplitLines.splitLines(stdout)
   return lines.map(parsePsOutputLine)
 }
 

package/src/sharedProcessMain.js

@@ -1,40 +1,12 @@
 import * as Command from './parts/Command/Command.js'
+import * as ErrorHandling from './parts/ErrorHandling/ErrorHandling.js'
 import * as Module from './parts/Module/Module.js'
 import * as ParentIpc from './parts/ParentIpc/ParentIpc.js'
-import * as PrettyError from './parts/PrettyError/PrettyError.js'
-
+import * as Process from './parts/Process/Process.js'
 // TODO handle structure: one shared process multiple extension hosts
 
 // TODO use named functions here
 
-const firstErrorLine = (error) => {
-  if (error.stack) {
-    return error.stack.slice(0, error.stack.indexOf('\n'))
-  }
-  if (error.message) {
-    return error.message
-  }
-  return `${error}`
-}
-
-const handleUncaughtExceptionMonitor = (error, origin) => {
-  console.info(`[shared process] uncaught exception: ${firstErrorLine(error)}`)
-  if (error && error.code === 'EPIPE' && !process.connected) {
-    // parent process is disposed, ignore
-    return
-  }
-  if (error && error.code === 'ERR_IPC_CHANNEL_CLOSED' && !process.connected) {
-    // parent process is disposed, ignore
-    return
-  }
-  // console.log(error)
-  const prettyError = PrettyError.prepare(error)
-  // console.error(prettyError.message)
-  console.error(prettyError.codeFrame)
-  console.error(prettyError.stack)
-  process.exit(1)
-}
-
 const handleDisconnect = () => {
   console.info('[shared process] disconnected')
 }
@@ -51,7 +23,7 @@ const knownCliArgs = ['install', 'list', 'link', 'unlink']
 
 const main = async () => {
   Command.setLoad(Module.load)
-  const argv = process.argv.slice(2)
+  const argv = Process.argv.slice(2)
   const argv0 = argv[0]
   if (knownCliArgs.includes(argv0)) {
     const module = await import('./parts/Cli/Cli.js')
@@ -64,7 +36,10 @@ const main = async () => {
   process.on('disconnect', handleDisconnect)
   process.on('SIGTERM', handleSigTerm)
 
-  process.on('uncaughtExceptionMonitor', handleUncaughtExceptionMonitor)
+  process.on(
+    'uncaughtExceptionMonitor',
+    ErrorHandling.handleUncaughtExceptionMonitor
+  )
   ParentIpc.listen()
 
   // ExtensionHost.start() // TODO start on demand, e.g. not when extensions should be disabled