node-pty 1.2.0-beta.12 → 1.2.0-beta.14

package/lib/conpty_console_list_agent.js

@@ -8,7 +8,7 @@
  */
 Object.defineProperty(exports, "__esModule", { value: true });
 var utils_1 = require("./utils");
-var getConsoleProcessList = utils_1.loadNativeModule('conpty_console_list').module.getConsoleProcessList;
+var getConsoleProcessList = (0, utils_1.loadNativeModule)('conpty_console_list').module.getConsoleProcessList;
 var shellPid = parseInt(process.argv[2], 10);
 var consoleProcessList = [];
 if (shellPid > 0) {

package/lib/index.js

@@ -5,7 +5,11 @@
  * Copyright (c) 2018, Microsoft Corporation (MIT License).
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.native = exports.open = exports.createTerminal = exports.fork = exports.spawn = void 0;
+exports.native = void 0;
+exports.spawn = spawn;
+exports.fork = fork;
+exports.createTerminal = createTerminal;
+exports.open = open;
 var utils_1 = require("./utils");
 var terminalCtor;
 if (process.platform === 'win32') {
@@ -29,24 +33,20 @@ else {
 function spawn(file, args, opt) {
     return new terminalCtor(file, args, opt);
 }
-exports.spawn = spawn;
 /** @deprecated */
 function fork(file, args, opt) {
     return new terminalCtor(file, args, opt);
 }
-exports.fork = fork;
 /** @deprecated */
 function createTerminal(file, args, opt) {
     return new terminalCtor(file, args, opt);
 }
-exports.createTerminal = createTerminal;
 function open(options) {
     return terminalCtor.open(options);
 }
-exports.open = open;
 /**
  * Expose the native API when not Windows, note that this is not public API and
  * could be removed at any time.
  */
-exports.native = (process.platform !== 'win32' ? utils_1.loadNativeModule('pty').module : null);
+exports.native = (process.platform !== 'win32' ? (0, utils_1.loadNativeModule)('pty').module : null);
 //# sourceMappingURL=index.js.map

package/lib/shared/conout.js

@@ -3,9 +3,8 @@
  * Copyright (c) 2020, Microsoft Corporation (MIT License).
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getWorkerPipeName = void 0;
+exports.getWorkerPipeName = getWorkerPipeName;
 function getWorkerPipeName(conoutPipeName) {
-    return conoutPipeName + "-worker";
+    return "".concat(conoutPipeName, "-worker");
 }
-exports.getWorkerPipeName = getWorkerPipeName;
 //# sourceMappingURL=conout.js.map

package/lib/terminal.js

@@ -101,14 +101,14 @@ var Terminal = /** @class */ (function () {
             if (Array.isArray(value)) {
                 value.forEach(function (v, i) {
                     if (typeof v !== type) {
-                        throw new Error(name + "[" + i + "] must be a " + type + " (not a " + typeof v[i] + ")");
+                        throw new Error("".concat(name, "[").concat(i, "] must be a ").concat(type, " (not a ").concat(typeof v[i], ")"));
                     }
                 });
                 return;
             }
         }
         if (typeof value !== type) {
-            throw new Error(name + " must be a " + type + " (not a " + typeof value + ")");
+            throw new Error("".concat(name, " must be a ").concat(type, " (not a ").concat(typeof value, ")"));
         }
     };
     /** See net.Socket.end */

package/lib/unixTerminal.js

@@ -3,10 +3,12 @@ var __extends = (this && this.__extends) || (function () {
     var extendStatics = function (d, b) {
         extendStatics = Object.setPrototypeOf ||
             ({ __proto__: [] } instanceof Array && function (d, b) { d.__proto__ = b; }) ||
-            function (d, b) { for (var p in b) if (b.hasOwnProperty(p)) d[p] = b[p]; };
+            function (d, b) { for (var p in b) if (Object.prototype.hasOwnProperty.call(b, p)) d[p] = b[p]; };
         return extendStatics(d, b);
     };
     return function (d, b) {
+        if (typeof b !== "function" && b !== null)
+            throw new TypeError("Class extends value " + String(b) + " is not a constructor or null");
         extendStatics(d, b);
         function __() { this.constructor = d; }
         d.prototype = b === null ? Object.create(b) : (__.prototype = b.prototype, new __());
@@ -24,7 +26,7 @@ var path = require("path");
 var tty = require("tty");
 var terminal_1 = require("./terminal");
 var utils_1 = require("./utils");
-var native = utils_1.loadNativeModule('pty');
+var native = (0, utils_1.loadNativeModule)('pty');
 var pty = native.module;
 var helperPath = native.dir + '/spawn-helper';
 helperPath = path.resolve(__dirname, helperPath);
@@ -52,7 +54,7 @@ var UnixTerminal = /** @class */ (function (_super) {
         _this._rows = opt.rows || terminal_1.DEFAULT_ROWS;
         var uid = (_a = opt.uid) !== null && _a !== void 0 ? _a : -1;
         var gid = (_b = opt.gid) !== null && _b !== void 0 ? _b : -1;
-        var env = utils_1.assign({}, opt.env);
+        var env = (0, utils_1.assign)({}, opt.env);
         if (opt.env === process.env) {
             _this._sanitizeEnv(env);
         }

package/lib/utils.js

@@ -4,7 +4,8 @@
  * Copyright (c) 2018, Microsoft Corporation (MIT License).
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.loadNativeModule = exports.assign = void 0;
+exports.assign = assign;
+exports.loadNativeModule = loadNativeModule;
 function assign(target) {
     var sources = [];
     for (var _i = 1; _i < arguments.length; _i++) {
@@ -13,10 +14,9 @@ function assign(target) {
     sources.forEach(function (source) { return Object.keys(source).forEach(function (key) { return target[key] = source[key]; }); });
     return target;
 }
-exports.assign = assign;
 function loadNativeModule(name) {
     // Check build, debug, and then prebuilds.
-    var dirs = ['build/Release', 'build/Debug', "prebuilds/" + process.platform + "-" + process.arch];
+    var dirs = ['build/Release', 'build/Debug', "prebuilds/".concat(process.platform, "-").concat(process.arch)];
     // Check relative to the parent dir for unbundled and then the current dir for bundled
     var relative = ['..', '.'];
     var lastError;
@@ -24,16 +24,15 @@ function loadNativeModule(name) {
         var d = dirs_1[_i];
         for (var _a = 0, relative_1 = relative; _a < relative_1.length; _a++) {
             var r = relative_1[_a];
-            var dir = r + "/" + d;
+            var dir = "".concat(r, "/").concat(d);
             try {
-                return { dir: dir, module: require(dir + "/" + name + ".node") };
+                return { dir: dir, module: require("".concat(dir, "/").concat(name, ".node")) };
             }
             catch (e) {
                 lastError = e;
             }
         }
     }
-    throw new Error("Failed to load native module: " + name + ".node, checked: " + dirs.join(', ') + ": " + lastError);
+    throw new Error("Failed to load native module: ".concat(name, ".node, checked: ").concat(dirs.join(', '), ": ").concat(lastError));
 }
-exports.loadNativeModule = loadNativeModule;
 //# sourceMappingURL=utils.js.map

package/lib/windowsConoutConnection.js

@@ -12,12 +12,12 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
     });
 };
 var __generator = (this && this.__generator) || function (thisArg, body) {
-    var _ = { label: 0, sent: function() { if (t[0] & 1) throw t[1]; return t[1]; }, trys: [], ops: [] }, f, y, t, g;
-    return g = { next: verb(0), "throw": verb(1), "return": verb(2) }, typeof Symbol === "function" && (g[Symbol.iterator] = function() { return this; }), g;
+    var _ = { label: 0, sent: function() { if (t[0] & 1) throw t[1]; return t[1]; }, trys: [], ops: [] }, f, y, t, g = Object.create((typeof Iterator === "function" ? Iterator : Object).prototype);
+    return g.next = verb(0), g["throw"] = verb(1), g["return"] = verb(2), typeof Symbol === "function" && (g[Symbol.iterator] = function() { return this; }), g;
     function verb(n) { return function (v) { return step([n, v]); }; }
     function step(op) {
         if (f) throw new TypeError("Generator is already executing.");
-        while (_) try {
+        while (g && (g = 0, op[0] && (_ = 0)), _) try {
             if (f = 1, y && (t = op[0] & 2 ? y["return"] : op[0] ? y["throw"] || ((t = y["return"]) && t.call(y), 0) : y.next) && !(t = t.call(y, op[1])).done) return t;
             if (y = 0, t) op = [op[0] & 2, t.value];
             switch (op[0]) {
@@ -73,10 +73,10 @@ var ConoutConnection = /** @class */ (function () {
             conoutPipeName: _conoutPipeName
         };
         var scriptPath = __dirname.replace('node_modules.asar', 'node_modules.asar.unpacked');
-        this._worker = new worker_threads_1.Worker(path_1.join(scriptPath, 'worker/conoutSocketWorker.js'), { workerData: workerData });
+        this._worker = new worker_threads_1.Worker((0, path_1.join)(scriptPath, 'worker/conoutSocketWorker.js'), { workerData: workerData });
         this._worker.on('message', function (message) {
             switch (message) {
-                case 1 /* READY */:
+                case 1 /* ConoutWorkerMessage.READY */:
                     _this._onReady.fire();
                     return;
                 default:
@@ -98,7 +98,7 @@ var ConoutConnection = /** @class */ (function () {
         this._drainDataAndClose();
     };
     ConoutConnection.prototype.connectSocket = function (socket) {
-        socket.connect(conout_1.getWorkerPipeName(this._conoutPipeName));
+        socket.connect((0, conout_1.getWorkerPipeName)(this._conoutPipeName));
     };
     ConoutConnection.prototype._drainDataAndClose = function () {
         var _this = this;

package/lib/windowsPtyAgent.js

@@ -5,12 +5,14 @@
  * Copyright (c) 2018, Microsoft Corporation (MIT License).
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.argsToCommandLine = exports.WindowsPtyAgent = void 0;
+exports.WindowsPtyAgent = void 0;
+exports.argsToCommandLine = argsToCommandLine;
 var fs = require("fs");
 var path = require("path");
 var child_process_1 = require("child_process");
 var net_1 = require("net");
 var windowsConoutConnection_1 = require("./windowsConoutConnection");
+var eventEmitter2_1 = require("./eventEmitter2");
 var utils_1 = require("./utils");
 var conptyNative;
 /**
@@ -24,13 +26,14 @@ var FLUSH_DATA_INTERVAL = 1000;
  */
 var WindowsPtyAgent = /** @class */ (function () {
     function WindowsPtyAgent(file, args, env, cwd, cols, rows, debug, _useConptyDll, conptyInheritCursor) {
-        var _this = this;
         if (_useConptyDll === void 0) { _useConptyDll = false; }
         if (conptyInheritCursor === void 0) { conptyInheritCursor = false; }
+        var _this = this;
         this._useConptyDll = _useConptyDll;
         this._innerPid = 0;
+        this._onError = new eventEmitter2_1.EventEmitter2();
         if (!conptyNative) {
-            conptyNative = utils_1.loadNativeModule('conpty').module;
+            conptyNative = (0, utils_1.loadNativeModule)('conpty').module;
         }
         this._ptyNative = conptyNative;
         // Sanitize input variable.
@@ -80,6 +83,11 @@ var WindowsPtyAgent = /** @class */ (function () {
         });
         this._inSocket.setEncoding('utf8');
     }
+    Object.defineProperty(WindowsPtyAgent.prototype, "onError", {
+        get: function () { return this._onError.event; },
+        enumerable: false,
+        configurable: true
+    });
     Object.defineProperty(WindowsPtyAgent.prototype, "inSocket", {
         get: function () { return this._inSocket; },
         enumerable: false,
@@ -107,13 +115,31 @@ var WindowsPtyAgent = /** @class */ (function () {
     });
     WindowsPtyAgent.prototype._completePtyConnection = function () {
         var _this = this;
+        var _a;
         if (!this._pendingPtyInfo) {
             return;
         }
-        var _a = this._pendingPtyInfo, pty = _a.pty, commandLine = _a.commandLine, cwd = _a.cwd, env = _a.env;
+        var _b = this._pendingPtyInfo, pty = _b.pty, commandLine = _b.commandLine, cwd = _b.cwd, env = _b.env;
         this._pendingPtyInfo = undefined;
-        var connect = conptyNative.connect(pty, commandLine, cwd, env, this._useConptyDll, function (c) { return _this._$onProcessExit(c); });
-        this._innerPid = connect.pid;
+        try {
+            var connect = conptyNative.connect(pty, commandLine, cwd, env, this._useConptyDll, function (c) { return _this._$onProcessExit(c); });
+            this._innerPid = connect.pid;
+        }
+        catch (err) {
+            // connect() runs from the conout worker's onReady callback (or its
+            // timeout fallback), so a throw here would otherwise surface as an
+            // uncaughtException with no way for the consumer to observe it.
+            var code = (_a = /error code: (\d+)/.exec(err.message)) === null || _a === void 0 ? void 0 : _a[1];
+            this._exitCode = code ? parseInt(code, 10) : -1;
+            try {
+                this._ptyNative.kill(this._pty, this._useConptyDll);
+            }
+            catch ( /* already gone */_c) { /* already gone */ }
+            this._conoutSocketWorker.dispose();
+            this._inSocket.destroy();
+            this._outSocket.destroy();
+            this._onError.fire(err);
+        }
     };
     WindowsPtyAgent.prototype.resize = function (cols, rows) {
         if (this._exitCode !== undefined) {
@@ -160,7 +186,7 @@ var WindowsPtyAgent = /** @class */ (function () {
             return Promise.resolve([]);
         }
         return new Promise(function (resolve) {
-            var agent = child_process_1.fork(path.join(__dirname, 'conpty_console_list_agent'), [_this._innerPid.toString()]);
+            var agent = (0, child_process_1.fork)(path.join(__dirname, 'conpty_console_list_agent'), [_this._innerPid.toString()]);
             agent.on('message', function (message) {
                 clearTimeout(timeout);
                 resolve(message.consoleProcessList);
@@ -180,7 +206,7 @@ var WindowsPtyAgent = /** @class */ (function () {
         configurable: true
     });
     WindowsPtyAgent.prototype._generatePipeName = function () {
-        return "conpty-" + Math.random() * 10000000;
+        return "conpty-".concat(Math.random() * 10000000);
     };
     /**
      * Triggered from the native side when a contpy process exits.
@@ -222,7 +248,7 @@ function argsToCommandLine(file, args) {
         if (args.length === 0) {
             return file;
         }
-        return argsToCommandLine(file, []) + " " + args;
+        return "".concat(argsToCommandLine(file, []), " ").concat(args);
     }
     var argv = [file];
     Array.prototype.push.apply(argv, args);
@@ -270,7 +296,6 @@ function argsToCommandLine(file, args) {
     }
     return result;
 }
-exports.argsToCommandLine = argsToCommandLine;
 function isCommandLine(args) {
     return typeof args === 'string';
 }

package/lib/windowsTerminal.js

@@ -8,10 +8,12 @@ var __extends = (this && this.__extends) || (function () {
     var extendStatics = function (d, b) {
         extendStatics = Object.setPrototypeOf ||
             ({ __proto__: [] } instanceof Array && function (d, b) { d.__proto__ = b; }) ||
-            function (d, b) { for (var p in b) if (b.hasOwnProperty(p)) d[p] = b[p]; };
+            function (d, b) { for (var p in b) if (Object.prototype.hasOwnProperty.call(b, p)) d[p] = b[p]; };
         return extendStatics(d, b);
     };
     return function (d, b) {
+        if (typeof b !== "function" && b !== null)
+            throw new TypeError("Class extends value " + String(b) + " is not a constructor or null");
         extendStatics(d, b);
         function __() { this.constructor = d; }
         d.prototype = b === null ? Object.create(b) : (__.prototype = b.prototype, new __());
@@ -37,7 +39,7 @@ var WindowsTerminal = /** @class */ (function (_super) {
         if (opt.encoding) {
             console.warn('Setting encoding on Windows is not supported');
         }
-        var env = utils_1.assign({}, opt.env);
+        var env = (0, utils_1.assign)({}, opt.env);
         _this._cols = opt.cols || terminal_1.DEFAULT_COLS;
         _this._rows = opt.rows || terminal_1.DEFAULT_ROWS;
         var cwd = opt.cwd || process.cwd();
@@ -50,6 +52,12 @@ var WindowsTerminal = /** @class */ (function (_super) {
         // Create new termal.
         _this._agent = new windowsPtyAgent_1.WindowsPtyAgent(file, args, parsedEnv, cwd, _this._cols, _this._rows, false, opt.useConptyDll, opt.conptyInheritCursor);
         _this._socket = _this._agent.outSocket;
+        // The socket 'close' handler that drives 'exit' is only attached after
+        // ready_datapipe, which never fires when connect() fails.
+        _this._agent.onError(function () {
+            _this._close();
+            _this.emit('exit', _this._agent.exitCode);
+        });
         // Not available until `ready` event emitted.
         _this._pid = _this._agent.innerPid;
         _this._fd = _this._agent.fd;

package/lib/worker/conoutSocketWorker.js

@@ -10,13 +10,13 @@ var conoutPipeName = worker_threads_1.workerData.conoutPipeName;
 var conoutSocket = new net_1.Socket();
 conoutSocket.setEncoding('utf8');
 conoutSocket.connect(conoutPipeName, function () {
-    var server = net_1.createServer(function (workerSocket) {
+    var server = (0, net_1.createServer)(function (workerSocket) {
         conoutSocket.pipe(workerSocket);
     });
-    server.listen(conout_1.getWorkerPipeName(conoutPipeName));
+    server.listen((0, conout_1.getWorkerPipeName)(conoutPipeName));
     if (!worker_threads_1.parentPort) {
         throw new Error('worker_threads parentPort is null');
     }
-    worker_threads_1.parentPort.postMessage(1 /* READY */);
+    worker_threads_1.parentPort.postMessage(1 /* ConoutWorkerMessage.READY */);
 });
 //# sourceMappingURL=conoutSocketWorker.js.map

package/package.json

@@ -4,7 +4,7 @@
   "author": {
     "name": "Microsoft Corporation"
   },
-  "version": "1.2.0-beta.12",
+  "version": "1.2.0-beta.14",
   "license": "MIT",
   "main": "./lib/index.js",
   "types": "./typings/node-pty.d.ts",
@@ -38,7 +38,7 @@
   "scripts": {
     "build": "tsc -b ./src/tsconfig.json",
     "watch": "tsc -b -w ./src/tsconfig.json",
-    "lint": "eslint -c .eslintrc.js --ext .ts src/",
+    "lint": "eslint src/",
     "install": "node scripts/prebuild.js || node-gyp rebuild",
     "postinstall": "node scripts/post-install.js",
     "compileCommands": "node scripts/gen-compile-commands.js",
@@ -53,13 +53,14 @@
   "devDependencies": {
     "@types/mocha": "^7.0.2",
     "@types/node": "12",
-    "@typescript-eslint/eslint-plugin": "^2.27.0",
-    "@typescript-eslint/parser": "^2.27.0",
+    "@typescript-eslint/eslint-plugin": "^8.0.0",
+    "@typescript-eslint/parser": "^8.0.0",
     "cross-env": "^5.1.4",
-    "eslint": "^6.8.0",
+    "eslint": "^10.4.0",
+    "globals": "^14.0.0",
     "mocha": "10",
     "node-gyp": "^11.4.2",
     "ps-list": "^6.0.0",
-    "typescript": "^3.8.3"
+    "typescript": "^5.0.0"
   }
 }

package/src/unix/pty.cc

@@ -203,6 +203,7 @@ void SetupExitCallback(Napi::Env env, Napi::Function cb, pid_t pid) {
         }
       }
     }
+    close(kq);
 #else
     while (true) {
       errno = 0;

package/src/win/conpty.cc

@@ -14,6 +14,8 @@
 #include <node_api.h>
 #include <assert.h>
 #include <Shlwapi.h> // PathCombine, PathIsRelative
+#include <atomic>
+#include <mutex>
 #include <sstream>
 #include <iostream>
 #include <string>
@@ -44,15 +46,18 @@ struct pty_baton {
   HANDLE hOut;
   HPCON hpc;
 
-  HANDLE hShell;
+  HANDLE hShell = nullptr;
 
   pty_baton(int _id, HANDLE _hIn, HANDLE _hOut, HPCON _hpc) : id(_id), hIn(_hIn), hOut(_hOut), hpc(_hpc) {};
 };
 
 static std::vector<std::unique_ptr<pty_baton>> ptyHandles;
-static volatile LONG ptyCounter;
+static std::mutex g_ptyHandlesMutex;
+static std::atomic<int> ptyCounter{0};
 
-static pty_baton* get_pty_baton(int id) {
+// The leading scoped-lock parameter encodes the precondition that the caller
+// holds g_ptyHandlesMutex.
+static pty_baton* get_pty_baton(const std::lock_guard<std::mutex>&, int id) {
   auto it = std::find_if(ptyHandles.begin(), ptyHandles.end(), [id](const auto& ptyHandle) {
     return ptyHandle->id == id;
   });
@@ -62,17 +67,6 @@ static pty_baton* get_pty_baton(int id) {
   return nullptr;
 }
 
-static bool remove_pty_baton(int id) {
-  auto it = std::remove_if(ptyHandles.begin(), ptyHandles.end(), [id](const auto& ptyHandle) {
-    return ptyHandle->id == id;
-  });
-  if (it != ptyHandles.end()) {
-    ptyHandles.erase(it);
-    return true;
-  }
-  return false;
-}
-
 struct ExitEvent {
   int exit_code = 0;
 };
@@ -99,11 +93,15 @@ void SetupExitCallback(Napi::Env env, Napi::Function cb, pty_baton* baton) {
     ExitEvent *exit_event = new ExitEvent;
     // Wait for process to complete.
     WaitForSingleObject(baton->hShell, INFINITE);
-    // Get process exit code.
-    GetExitCodeProcess(baton->hShell, (LPDWORD)(&exit_event->exit_code));
-    // Clean up handles
-    CloseHandle(baton->hShell);
-    assert(remove_pty_baton(baton->id));
+    {
+      std::lock_guard<std::mutex> lock(g_ptyHandlesMutex);
+      GetExitCodeProcess(baton->hShell, (LPDWORD)(&exit_event->exit_code));
+      CloseHandle(baton->hShell);
+      const int id = baton->id;
+      std::erase_if(ptyHandles, [id](const auto& ptyHandle) {
+        return ptyHandle->id == id;
+      });
+    }
 
     auto status = tsfn.BlockingCall(exit_event, callback); // In main thread
     switch (status) {
@@ -298,10 +296,13 @@ static Napi::Value PtyStartProcess(const Napi::CallbackInfo& info) {
 
   if (SUCCEEDED(hr)) {
     // We were able to instantiate a conpty
-    const int ptyId = InterlockedIncrement(&ptyCounter);
+    const int ptyId = ++ptyCounter;
     marshal.Set("pty", Napi::Number::New(env, ptyId));
-    ptyHandles.emplace_back(
-        std::make_unique<pty_baton>(ptyId, hIn, hOut, hpc));
+    {
+      std::lock_guard<std::mutex> lock(g_ptyHandlesMutex);
+      ptyHandles.emplace_back(
+          std::make_unique<pty_baton>(ptyId, hIn, hOut, hpc));
+    }
   } else {
     throw Napi::Error::New(env, "Cannot launch conpty");
   }
@@ -349,10 +350,13 @@ static Napi::Value PtyConnect(const Napi::CallbackInfo& info) {
   const bool useConptyDll = info[4].As<Napi::Boolean>().Value();
   Napi::Function exitCallback = info[5].As<Napi::Function>();
 
-  // Fetch pty handle from ID and start process
-  pty_baton* handle = get_pty_baton(id);
-  if (!handle) {
-    throw Napi::Error::New(env, "Invalid pty handle");
+  pty_baton* handle;
+  {
+    std::lock_guard<std::mutex> lock(g_ptyHandlesMutex);
+    handle = get_pty_baton(lock, id);
+    if (!handle) {
+      throw Napi::Error::New(env, "Invalid pty handle");
+    }
   }
 
   // Prepare command line
@@ -380,6 +384,14 @@ static Napi::Value PtyConnect(const Napi::CallbackInfo& info) {
   ConnectNamedPipe(handle->hIn, nullptr);
   ConnectNamedPipe(handle->hOut, nullptr);
 
+  // Close hIn/hOut on every exit so a throw doesn't leak the named-pipe server handles.
+  auto closePipeHandles = [handle]() {
+    CloseHandle(handle->hIn);
+    CloseHandle(handle->hOut);
+    handle->hIn = nullptr;
+    handle->hOut = nullptr;
+  };
+
   // Attach the pseudoconsole to the client application we're creating
   STARTUPINFOEXW siEx{0};
   siEx.StartupInfo.cb = sizeof(STARTUPINFOEXW);
@@ -390,12 +402,14 @@ static Napi::Value PtyConnect(const Napi::CallbackInfo& info) {
 
   SIZE_T size = 0;
   InitializeProcThreadAttributeList(NULL, 1, 0, &size);
-  BYTE *attrList = new BYTE[size];
-  siEx.lpAttributeList = reinterpret_cast<PPROC_THREAD_ATTRIBUTE_LIST>(attrList);
+  std::unique_ptr<BYTE[]> attrList = std::make_unique<BYTE[]>(size);
+  siEx.lpAttributeList = reinterpret_cast<PPROC_THREAD_ATTRIBUTE_LIST>(attrList.get());
 
   fSuccess = InitializeProcThreadAttributeList(siEx.lpAttributeList, 1, 0, &size);
   if (!fSuccess) {
-    throw errorWithCode(info, "InitializeProcThreadAttributeList failed");
+    Napi::Error error = errorWithCode(info, "InitializeProcThreadAttributeList failed");
+    closePipeHandles();
+    throw error;
   }
   fSuccess = UpdateProcThreadAttribute(siEx.lpAttributeList,
                                        0,
@@ -405,7 +419,10 @@ static Napi::Value PtyConnect(const Napi::CallbackInfo& info) {
                                        NULL,
                                        NULL);
   if (!fSuccess) {
-    throw errorWithCode(info, "UpdateProcThreadAttribute failed");
+    Napi::Error error = errorWithCode(info, "UpdateProcThreadAttribute failed");
+    DeleteProcThreadAttributeList(siEx.lpAttributeList);
+    closePipeHandles();
+    throw error;
   }
 
   PROCESS_INFORMATION piClient{};
@@ -422,9 +439,14 @@ static Napi::Value PtyConnect(const Napi::CallbackInfo& info) {
       &piClient                     // lpProcessInformation
   );
   if (!fSuccess) {
-    throw errorWithCode(info, "Cannot create process");
+    Napi::Error error = errorWithCode(info, "Cannot create process");
+    DeleteProcThreadAttributeList(siEx.lpAttributeList);
+    closePipeHandles();
+    throw error;
   }
 
+  DeleteProcThreadAttributeList(siEx.lpAttributeList);
+
   HANDLE hLibrary = LoadConptyDll(info, useConptyDll);
   bool fLoadedDll = hLibrary != nullptr;
   if (useConptyDll && fLoadedDll)
@@ -443,8 +465,7 @@ static Napi::Value PtyConnect(const Napi::CallbackInfo& info) {
   // Close the thread handle to avoid resource leak
   CloseHandle(piClient.hThread);
   // Close the input read and output write handle of the pseudoconsole
-  CloseHandle(handle->hIn);
-  CloseHandle(handle->hOut);
+  closePipeHandles();
 
   SetupExitCallback(env, exitCallback, handle);
 
@@ -471,7 +492,8 @@ static Napi::Value PtyResize(const Napi::CallbackInfo& info) {
   SHORT rows = static_cast<SHORT>(info[2].As<Napi::Number>().Uint32Value());
   const bool useConptyDll = info[3].As<Napi::Boolean>().Value();
 
-  const pty_baton* handle = get_pty_baton(id);
+  std::lock_guard<std::mutex> lock(g_ptyHandlesMutex);
+  const pty_baton* handle = get_pty_baton(lock, id);
 
   if (handle != nullptr) {
     HANDLE hLibrary = LoadConptyDll(info, useConptyDll);
@@ -512,7 +534,8 @@ static Napi::Value PtyClear(const Napi::CallbackInfo& info) {
     return env.Undefined();
   }
 
-  const pty_baton* handle = get_pty_baton(id);
+  std::lock_guard<std::mutex> lock(g_ptyHandlesMutex);
+  const pty_baton* handle = get_pty_baton(lock, id);
 
   if (handle != nullptr) {
     HANDLE hLibrary = LoadConptyDll(info, useConptyDll);
@@ -543,7 +566,8 @@ static Napi::Value PtyKill(const Napi::CallbackInfo& info) {
   int id = info[0].As<Napi::Number>().Int32Value();
   const bool useConptyDll = info[1].As<Napi::Boolean>().Value();
 
-  const pty_baton* handle = get_pty_baton(id);
+  std::lock_guard<std::mutex> lock(g_ptyHandlesMutex);
+  pty_baton* handle = get_pty_baton(lock, id);
 
   if (handle != nullptr) {
     HANDLE hLibrary = LoadConptyDll(info, useConptyDll);
@@ -558,7 +582,17 @@ static Napi::Value PtyKill(const Napi::CallbackInfo& info) {
         pfnClosePseudoConsole(handle->hpc);
       }
     }
-    if (useConptyDll) {
+    // Defensive: if PtyConnect was never called (or failed before closing the
+    // pipe handles), release the named pipe server handles here.
+    if (handle->hIn != nullptr) {
+      CloseHandle(handle->hIn);
+      handle->hIn = nullptr;
+    }
+    if (handle->hOut != nullptr) {
+      CloseHandle(handle->hOut);
+      handle->hOut = nullptr;
+    }
+    if (useConptyDll && handle->hShell != nullptr) {
       TerminateProcess(handle->hShell, 1);
     }
   }