pass clean env to pty process (#49)

apps/app/package.json

@@ -44,6 +44,7 @@
     "node-loader": "^2.0.0",
     "node-pty": "^1.0.0",
     "os-locale": "^6.0.2",
+    "shell-env": "^4.0.1",
     "vm": "^0.1.0",
     "webpack": "^5.72.0",
     "winston": "^3.8.2"

apps/app/src/nativeBridge/modules/terminalModule.ts

@@ -36,14 +36,14 @@ class PTYInstance {
       cols: cols,
       rows: rows,
       cwd: startupDirectory || os.homedir() || process.cwd(),
-      env: {
-        ...process.env,
+      // eslint-disable-next-line @typescript-eslint/no-var-requires
+      env: Object.assign({}, require('shell-env').shellEnvSync(shell), {
         LANG: `${osLocaleSync().replace(/-/, '_')}.UTF-8`,
         TERM: 'xterm-256color',
         COLORTERM: 'truecolor',
         TERM_PROGRAM: app.name,
         TERM_PROGRAM_VERSION: app.getVersion(),
-      },
+      }),
     });
 
     Logger.getInstance().log(

apps/app/webpack.preload.config.js

@@ -12,6 +12,8 @@ module.exports = {
   },
   externals: {
     electron: 'commonjs electron',
+    'node:os': 'commonjs os',
+    'node:process': 'commonjs process',
   },
   plugins: [
     new webpack.NormalModuleReplacementPlugin(
@@ -51,6 +53,7 @@ module.exports = {
     fallback: {
       assert: false,
       buffer: false,
+      child_process: false,
       console: false,
       constants: false,
       crypto: false,

package.json

@@ -1,7 +1,7 @@
 {
   "name": "@terminalone/monorepo",
   "productName": "Terminal One",
-  "version": "1.5.0",
+  "version": "1.5.1",
   "description": "A fast, elegant and intelligent cross-platform terminal.",
   "author": "atinylittleshell <[email protected]>",
   "license": "MIT",