fix(ng-dev): properly pass input to stdin of child processes

This is a small regression introduced via:
933dcd4

Author: devversion

ng-dev/utils/child-process.ts

@@ -14,10 +14,15 @@ import {
   SpawnSyncOptions as _SpawnSyncOptions,
   ExecOptions as _ExecOptions,
   exec as _exec,
+  ChildProcess as _ChildProcess,
 } from 'child_process';
 import {Log} from './logging.js';
+import assert from 'assert';
 
 export interface CommonCmdOpts {
+  // Stdin text to provide to the process. The raw text will be written to `stdin` and then
+  // the stream is closed. This is equivalent to the `input` option from `SpawnSyncOption`.
+  input?: string;
   /** Console output mode. Defaults to "enabled". */
   mode?: 'enabled' | 'silent' | 'on-error';
   /** Whether to prevent exit codes being treated as failures. */
@@ -27,14 +32,10 @@ export interface CommonCmdOpts {
 /** Interface describing the options for spawning a process synchronously. */
 export interface SpawnSyncOptions
   extends CommonCmdOpts,
-    Omit<_SpawnSyncOptions, 'shell' | 'stdio'> {}
+    Omit<_SpawnSyncOptions, 'shell' | 'stdio' | 'input'> {}
 
 /** Interface describing the options for spawning a process. */
-export interface SpawnOptions extends CommonCmdOpts, Omit<_SpawnOptions, 'shell' | 'stdio'> {
-  // Stdin text to provide to the process. The raw text will be written to `stdin` and then
-  // the stream is closed. This is equivalent to the `input` option from `SpawnSyncOption`.
-  input?: string;
-}
+export interface SpawnOptions extends CommonCmdOpts, Omit<_SpawnOptions, 'shell' | 'stdio'> {}
 
 /** Interface describing the options for exec-ing a process. */
 export interface ExecOptions extends CommonCmdOpts, Omit<_ExecOptions, 'shell' | 'stdio'> {}
@@ -172,28 +173,28 @@ function getEnvironmentForNonInteractiveCommand(
 /**
  * Process the ChildProcess object created by an async command.
  */
-function processAsyncCmd(
-  cmd: string,
-  opts: CommonCmdOpts,
-  childProcess: ReturnType<typeof _exec>,
-): Promise<ExecResult>;
-function processAsyncCmd(
-  cmd: string,
-  opts: CommonCmdOpts,
-  childProcess: ReturnType<typeof _spawn>,
-): Promise<SpawnResult>;
 function processAsyncCmd(
   command: string,
   options: CommonCmdOpts,
-  childProcess: ReturnType<typeof _exec | typeof _spawn>,
-) {
+  childProcess: _ChildProcess,
+): Promise<SpawnResult> {
   return new Promise((resolve, reject) => {
     let logOutput = '';
     let stdout = '';
     let stderr = '';
 
     Log.debug(`Executing command: ${command}`);
 
+    // If provided, write `input` text to the process `stdin`.
+    if (options.input !== undefined) {
+      assert(
+        childProcess.stdin,
+        'Cannot write process `input` if there is no pipe `stdin` channel.',
+      );
+      childProcess.stdin.write(options.input);
+      childProcess.stdin.end();
+    }
+
     // Capture the stdout separately so that it can be passed as resolve value.
     // This is useful if commands return parsable stdout.
     childProcess.stderr?.on('data', (message) => {