@alexgyver/serial 1.0.5 → 1.0.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@alexgyver/serial",
-  "version": "1.0.5",
+  "version": "1.0.7",
   "description": "Web Serial API wrapper",
   "main": "./serial.js",
   "module": "./serial.js",
@@ -14,14 +14,14 @@
     "url": "git+https://github.com/GyverLibs/Serial.js.git"
   },
   "devDependencies": {
-    "webpack": "^5.105.1",
-    "webpack-cli": "^6.0.1",
-    "webpack-dev-server": "^5.2.3",
+    "webpack": "^5.106.2",
+    "webpack-cli": "^7.0.2",
+    "webpack-dev-server": "^5.2.4",
     "style-loader": "^4.0.0",
-    "css-loader": "^7.1.3"
+    "css-loader": "^7.1.4"
   },
   "dependencies": {
-    "@alexgyver/utils": "^1.2.4"
+    "@alexgyver/utils": "^1.2.6"
   },
   "author": "AlexGyver <alex@alexgyver.ru>",
   "license": "MIT"

package/serial.js

@@ -1,4 +1,4 @@
-import { sleep, ShiftBuffer, StreamSplitter } from "@alexgyver/utils";
+import { sleep, StreamSplitter, SerialExecutor } from "@alexgyver/utils";
 
 export default class SerialJS {
     static State = {
@@ -8,17 +8,14 @@ export default class SerialJS {
         Closing: 'closing',
     };
 
-    //#region handlers
-    onbin = null;
     ontext = null;
-
+    onbin(b) { }
     onopen() { }
     onclose() { }
     onchange(s) { }
     onselect(name) { }
     onerror(e) { }
 
-    //#region constructor
     constructor(params = {}) {
         const def = {
             eol: /\r?\n/,
@@ -26,14 +23,21 @@ export default class SerialJS {
             auto_open: false,
             reconnect: 1000,
         };
+
         this.cfg = { ...def, ...params };
 
-        this._setLastPort().then(() => this.onselect(this.getName()));
         this.splitter = new StreamSplitter(this.cfg.eol);
-        this.splitter.ontext = (t) => this.ontext(t);
+        this.splitter.ontext = (t) => {
+            try {
+                if (this.ontext) this.ontext(t);
+            } catch (e) {
+                this._error(e);
+            }
+        };
+
+        this._setLastPort().then(() => this.onselect(this.getName()));
     }
 
-    //#region methods
     config(params = {}) {
         this.cfg = { ...this.cfg, ...params };
         this.splitter.eol = this.cfg.eol;
@@ -44,7 +48,7 @@ export default class SerialJS {
     }
 
     opened() {
-        return this._state == SerialJS.State.Open;
+        return this._state === SerialJS.State.Open;
     }
 
     selected() {
@@ -62,136 +66,211 @@ export default class SerialJS {
             case 0x0402: case 0x0403: case 0x0404: case 0x0405: case 0x6001: case 0x0602: case 0x6010: return 'FT232';
             case 0x9500: case 0x0102: case 0x0501: case 0x80a9: case 0xea60: case 0xea61: case 0xea63: return 'CP210x';
         }
+
         return 'Unknown';
     }
 
     async select() {
         try {
             await this.close();
+
             this._port = null;
-            let ports = await navigator.serial.getPorts();
-            for (let p of ports) await p.forget();
+
+            const ports = await navigator.serial.getPorts();
+            for (const p of ports) await p.forget();
+
             await sleep(50);
             await navigator.serial.requestPort();
             await this._setLastPort();
         } catch (e) {
             this._error(e);
         }
+
         this.onselect(this.getName());
+
         if (this.cfg.auto_open) this.open();
+
         return this.selected();
     }
 
     async open() {
-        const ports = await navigator.serial.getPorts();
-        if (!ports.length) return;
-        
-        if (this.cfg.reconnect) this.retry = true;
-        await this._open();
+        return this._lifecycle.runNothrow(async () => {
+            const ports = await navigator.serial.getPorts();
+            if (!ports.length) return false;
+
+            if (this.cfg.reconnect) this.retry = true;
+
+            await this._open();
+
+            return this.opened();
+        });
     }
+
     async _open() {
-        if (this.opened()) return;
+        if (this.opened() || this._state === SerialJS.State.Opening) return;
+
         this._change(SerialJS.State.Opening);
 
         try {
             await this._close();
             await this._setLastPort();
+
+            if (!this._port) {
+                this._change(SerialJS.State.Closed);
+                return;
+            }
+
             await this._port.open({ baudRate: this.cfg.baud });
+
             this.writer = this._port.writable.getWriter();
             this.reader = this._port.readable.getReader();
-            this._buffer.clear();
+
+            this._sender.reset();
             this._change(SerialJS.State.Open);
 
-            while (this._state == SerialJS.State.Open) {
+            while (this._state === SerialJS.State.Open) {
                 const { value, done } = await this.reader.read();
+
                 if (done) break;
+
                 if (value) {
-                    if (this.onbin) this.onbin(value);
-                    if (this.ontext) this.splitter.write(this._decoder.decode(value, { stream: true }));
+                    try {
+                        this.onbin(value);
+                    } catch (e) {
+                        this._error(e);
+                    }
+
+                    if (this.ontext) {
+                        this.splitter.write(
+                            this._decoder.decode(value, { stream: true })
+                        );
+                    }
                 }
             }
         } catch (e) {
             this._error(e);
-            this._change(SerialJS.State.Closed);
-            if (this.retry) setTimeout(() => this._open(), this.cfg.reconnect);
         }
 
-        if (this.reader) this.reader.releaseLock();
-        if (this.writer) this.writer.releaseLock();
+        this._sender.reset();
+
+        if (this._state !== SerialJS.State.Closing) {
+            this._change(SerialJS.State.Closing);
+        }
+
+        try {
+            if (this.reader) this.reader.releaseLock();
+        } catch (e) { }
+
+        try {
+            if (this.writer) this.writer.releaseLock();
+        } catch (e) { }
+
         this.reader = null;
         this.writer = null;
 
         try {
-            await this._port.close();
-            this._change(SerialJS.State.Closed);
+            if (this._port) await this._port.close();
         } catch (e) { }
+
+        this._change(SerialJS.State.Closed);
+
+        if (this.retry) {
+            setTimeout(() => this._lifecycle.runNothrow(() => this._open()), this.cfg.reconnect);
+        }
     }
 
     async close() {
-        this.retry = false;
-        await this._close();
+        return this._lifecycle.runNothrow(async () => {
+            this.retry = false;
+            this._sender.reset();
+            await this._close();
+            return true;
+        });
     }
+
     async _close() {
         switch (this._state) {
-            case SerialJS.State.Closed: return;
+            case SerialJS.State.Closed:
+                return;
+
+            case SerialJS.State.Opening:
             case SerialJS.State.Open:
-                if (this.reader) await this.reader.cancel();
                 this._change(SerialJS.State.Closing);
+
+                if (this.reader) {
+                    try {
+                        await this.reader.cancel();
+                    } catch (e) { }
+                }
+
+                break;
+
+            case SerialJS.State.Closing:
                 break;
         }
 
         let i = 0;
-        while (this._state == SerialJS.State.Closing) {
+
+        while (this._state === SerialJS.State.Closing) {
             await sleep(10);
+
             if (++i > 200) {
                 this._error('Close timeout');
                 this._change(SerialJS.State.Closed);
                 break;
             }
         }
-
     }
 
     async sendText(text) {
-        await this.sendBin((new TextEncoder()).encode(text));
+        return this.sendBin(new TextEncoder().encode(text));
     }
 
     async sendBin(data) {
-        this._buffer.push(data);
-        this._send();
+        if (!this.opened() || !this.writer) return false;
+
+        return this._sender.runNothrow(async () => {
+            if (!this.opened() || !this.writer) return false;
+
+            await this.writer.write(data);
+            return true;
+        });
     }
 
-    //#region private
     _port = null;
     _state = SerialJS.State.Closed;
-    _buffer = new ShiftBuffer();
+
+    reader = null;
+    writer = null;
+    retry = false;
+
     _decoder = new TextDecoder();
+    _sender = new SerialExecutor();
+    _lifecycle = new SerialExecutor();
 
-    async _send() {
-        if (this._busy) return;
-        this._busy = true;
-        while (this._buffer.length && this.writer) {
-            let d = this._buffer.shiftAll();
-            try {
-                await this.writer.write(d);
-            } catch (e) { }
-        }
-        this._busy = false;
-    }
     async _setLastPort() {
-        let ports = await navigator.serial.getPorts();
+        const ports = await navigator.serial.getPorts();
         this._port = ports.length ? ports[0] : null;
     }
 
     _error(e) {
         this.onerror('[SerialJS] ' + e);
     }
+
     _change(s) {
+        if (this._state === s) return;
+
         this._state = s;
         this.onchange(s);
+
         switch (s) {
-            case SerialJS.State.Open: this.onopen(); break;
-            case SerialJS.State.Closed: this.onclose(); break;
+            case SerialJS.State.Open:
+                this.onopen();
+                break;
+
+            case SerialJS.State.Closed:
+                this.onclose();
+                break;
         }
     }
 }