3.2.1 dont try to resize if we invalidated the fd (#43)

`ioctl TIOCSWINSZ failed: Bad file descriptor (os error 9)` makes me sad

- js owns the fd so its safe to mark the fd as dead after the read
stream derived from the pty fd ends
- after the fd is dead, dont try to resize

npm/darwin-arm64/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@replit/ruspty-darwin-arm64",
-  "version": "3.0.5",
+  "version": "3.2.1",
   "os": [
     "darwin"
   ],

npm/darwin-x64/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@replit/ruspty-darwin-x64",
-  "version": "3.0.5",
+  "version": "3.2.1",
   "os": [
     "darwin"
   ],

npm/linux-x64-gnu/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@replit/ruspty-linux-x64-gnu",
-  "version": "3.0.5",
+  "version": "3.2.1",
   "os": [
     "linux"
   ],

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@replit/ruspty",
-  "version": "3.2.0",
+  "version": "3.2.1",
   "main": "dist/wrapper.js",
   "types": "dist/wrapper.d.ts",
   "author": "Szymon Kaliski <[email protected]>",

tests/index.test.ts

@@ -254,6 +254,25 @@ describe(
         });
       }));
 
+    test('resize after exit shouldn\'t throw', () => new Promise<void>((done, reject) => {
+      const pty = new Pty({
+        command: '/bin/echo',
+        args: ['hello'],
+        onExit: (err, exitCode) => {
+          try {
+            expect(err).toBeNull();
+            expect(exitCode).toBe(0);
+            expect(() => {
+              pty.resize({ rows: 60, cols: 100 });
+            }).not.toThrow();
+            done();
+          } catch (e) {
+            reject(e)
+          }
+        },
+      });
+    }));
+
     test(
       'ordering is correct',
       () =>

wrapper.ts

@@ -45,6 +45,7 @@ type ExitResult = {
 export class Pty {
   #pty: RawPty;
   #fd: number;
+  #fdEnded: boolean = false;
   #socket: ReadStream;
 
   read: Readable;
@@ -78,13 +79,12 @@ export class Pty {
     this.write = userFacingWrite;
 
     // catch end events
-    let handleCloseCalled = false;
     const handleClose = () => {
-      if (handleCloseCalled) {
+      if (this.#fdEnded) {
         return;
       }
 
-      handleCloseCalled = true;
+      this.#fdEnded = true;
       exitResult.then((result) => realExit(result.error, result.code));
       userFacingRead.end();
     };
@@ -122,6 +122,10 @@ export class Pty {
   }
 
   resize(size: Size) {
+    if (this.#fdEnded) {
+      return;
+    }
+
     ptyResize(this.#fd, size);
   }