refactor: update src/node/wrapper.ts

jsjoeio · code-asher · web-flow · commit 861dc5a1f301 · 2021-03-30T09:53:45.000-07:00
Use `inherit` so we can get rid of the manual `pipe` calls

Co-authored-by: Asher &lt;ash@coder.com&gt;
diff --git a/src/node/wrapper.ts b/src/node/wrapper.ts
@@ -314,7 +314,7 @@ export class ParentProcess extends Process {
         CODE_SERVER_PARENT_PID: process.pid.toString(),
         NODE_OPTIONS: `--max-old-space-size=2048 ${process.env.NODE_OPTIONS || ""}`,
       },
-      stdio: ["inherit", "pipe", "pipe", "ipc"],
+      stdio: ["inherit", "inherit", "inherit", "ipc"],
     })
   }
 

Original file line number	Diff line number	Diff line change
`@@ -314,7 +314,7 @@ export class ParentProcess extends Process {`
`314`	`314`	`CODE_SERVER_PARENT_PID: process.pid.toString(),`
`315`	`315`	NODE_OPTIONS: `--max-old-space-size=2048 ${process.env.NODE_OPTIONS \|\| ""}`,
`316`	`316`	`},`
`317`		`- stdio: ["inherit", "pipe", "pipe", "ipc"],`
	`317`	`+ stdio: ["inherit", "inherit", "inherit", "ipc"],`
`318`	`318`	`})`
`319`	`319`	`}`
`320`	`320`