bare-tty 1.0.7 → 2.0.1

package/README.md

@@ -9,9 +9,9 @@ npm install bare-tty
 ## Usage
 
 ```js
-const TTY = require('bare-tty')
+const tty = require('bare-tty')
 
-const stdout = new TTY(1)
+const stdout = new tty.WriteStream(1)
 
 stdout.write('Hello world!\n')
 ```

package/binding.c

@@ -369,8 +369,6 @@ bare_tty_resume (js_env_t *env, js_callback_info_t *info) {
   err = js_get_arraybuffer_info(env, argv[0], (void **) &tty, NULL);
   assert(err == 0);
 
-  if (!uv_is_readable((uv_stream_t *) &tty->handle)) return NULL;
-
   err = uv_read_start((uv_stream_t *) &tty->handle, on_alloc, on_read);
 
   if (err < 0) {
@@ -397,8 +395,6 @@ bare_tty_pause (js_env_t *env, js_callback_info_t *info) {
   err = js_get_arraybuffer_info(env, argv[0], (void **) &tty, NULL);
   assert(err == 0);
 
-  if (!uv_is_readable((uv_stream_t *) &tty->handle)) return NULL;
-
   err = uv_read_stop((uv_stream_t *) &tty->handle);
 
   if (err < 0) {
@@ -425,17 +421,14 @@ bare_tty_close (js_env_t *env, js_callback_info_t *info) {
   err = js_get_arraybuffer_info(env, argv[0], (void **) &tty, NULL);
   assert(err == 0);
 
-  uv_close((uv_handle_t *) &tty->handle, on_close);
+  err = uv_tty_set_mode(&tty->handle, UV_TTY_MODE_NORMAL);
 
-  return NULL;
-}
-
-static js_value_t *
-bare_tty_reset (js_env_t *env, js_callback_info_t *info) {
-  int err;
+  if (err < 0) {
+    js_throw_error(env, uv_err_name(err), uv_strerror(err));
+    return NULL;
+  }
 
-  err = uv_tty_reset_mode();
-  assert(err == 0);
+  uv_close((uv_handle_t *) &tty->handle, on_close);
 
   return NULL;
 }
@@ -502,11 +495,6 @@ init (js_env_t *env, js_value_t *exports) {
     js_create_function(env, "close", -1, bare_tty_close, NULL, &fn);
     js_set_named_property(env, exports, "close", fn);
   }
-  {
-    js_value_t *fn;
-    js_create_function(env, "reset", -1, bare_tty_reset, NULL, &fn);
-    js_set_named_property(env, exports, "reset", fn);
-  }
   {
     js_value_t *fn;
     js_create_function(env, "setMode", -1, bare_tty_set_mode, NULL, &fn);

package/index.js

@@ -1,40 +1,107 @@
-const { Duplex } = require('streamx')
+const { Readable, Writable } = require('streamx')
 const binding = require('./binding')
 
 const DEFAULT_READ_BUFFER = 65536
 
-process.on('exit', () => binding.reset())
-
-module.exports = exports = class TTY extends Duplex {
+exports.ReadStream = class TTYReadStream extends Readable {
   constructor (fd, opts = {}) {
-    super({ mapWritable })
+    super()
 
     const {
       readBufferSize = DEFAULT_READ_BUFFER,
       allowHalfOpen = true
     } = opts
 
-    this._pendingWrite = null
-    this._pendingFinal = null
     this._pendingDestroy = null
 
+    this._reading = false
     this._allowHalfOpen = allowHalfOpen
 
     this._buffer = Buffer.alloc(readBufferSize)
 
     this._handle = binding.init(fd, this._buffer, this,
-      this._onwrite,
-      this._onfinal,
+      noop,
+      noop,
       this._onread,
       this._onclose
     )
   }
 
-  _open (cb) {
-    binding.resume(this._handle)
+  get isTTY () {
+    return true
+  }
+
+  setMode (mode) {
+    binding.setMode(this._handle, mode)
+  }
+
+  _read (cb) {
+    if (!this._reading) {
+      this._reading = true
+      binding.resume(this._handle)
+    }
+    cb(null)
+  }
+
+  _destroy (cb) {
+    this._pendingDestroy = cb
+    binding.close(this._handle)
+  }
+
+  _continueDestroy () {
+    if (this._pendingDestroy === null) return
+    const cb = this._pendingDestroy
+    this._pendingDestroy = null
     cb(null)
   }
 
+  _onread (err, read) {
+    if (err) {
+      this.destroy(err)
+      return
+    }
+
+    if (read === 0) {
+      this.push(null)
+      if (this._allowHalfOpen === false) this.end()
+      return
+    }
+
+    const copy = Buffer.allocUnsafe(read)
+    copy.set(this._buffer.subarray(0, read))
+
+    if (this.push(copy) === false && this.destroying === false) {
+      this._reading = false
+      binding.pause(this._handle)
+    }
+  }
+
+  _onclose () {
+    this._handle = null
+    this._continueDestroy()
+  }
+}
+
+exports.WriteStream = class TTYWriteStream extends Writable {
+  constructor (fd, opts = {}) {
+    super({ mapWritable })
+
+    this._pendingWrite = null
+    this._pendingFinal = null
+    this._pendingDestroy = null
+
+    this._handle = binding.init(fd, empty, this,
+      this._onwrite,
+      this._onfinal,
+      noop,
+      this._onclose
+    )
+  }
+
+  get isTTY () {
+    return true
+  }
+
   _writev (datas, cb) {
     this._pendingWrite = cb
     binding.writev(this._handle, datas)
@@ -90,7 +157,10 @@ module.exports = exports = class TTY extends Duplex {
     const copy = Buffer.allocUnsafe(read)
     copy.set(this._buffer.subarray(0, read))
 
-    this.push(copy)
+    if (this.push(copy) === false && this.destroying === false) {
+      this._reading = false
+      binding.pause(this._handle)
+    }
   }
 
   _onfinal (err) {
@@ -101,10 +171,6 @@ module.exports = exports = class TTY extends Duplex {
     this._handle = null
     this._continueDestroy()
   }
-
-  setMode (mode) {
-    binding.setMode(this._handle, mode)
-  }
 }
 
 exports.constants = {
@@ -113,6 +179,10 @@ exports.constants = {
   MODE_IO: binding.MODE_IO || 0
 }
 
+const empty = Buffer.alloc(0)
+
+function noop () {}
+
 function mapWritable (buf) {
   return typeof buf === 'string' ? Buffer.from(buf) : buf
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "bare-tty",
-  "version": "1.0.7",
+  "version": "2.0.1",
   "description": "Native TTY streams for JavaScript",
   "main": "index.js",
   "files": [