refactor: bump prettier to version 2.8.1

This major bump creates a lot of noise, but it is necessary for
prettier to be able to parse new syntax such as:

- typed imports: `import { type xxx } from ...`
- private attributes: `class A { #b; #c() {} }`

Author: darrachequesne

lib/socket.ts

@@ -332,10 +332,10 @@ export class Socket extends Emitter<{}, {}, SocketReservedEvents> {
         addTrailingSlash: true,
         rejectUnauthorized: true,
         perMessageDeflate: {
-          threshold: 1024
+          threshold: 1024,
         },
         transportOptions: {},
-        closeOnBeforeunload: true
+        closeOnBeforeunload: true,
       },
       opts
     );
@@ -374,7 +374,7 @@ export class Socket extends Emitter<{}, {}, SocketReservedEvents> {
       if (this.hostname !== "localhost") {
         this.offlineEventListener = () => {
           this.onClose("transport close", {
-            description: "network connection lost"
+            description: "network connection lost",
           });
         };
         addEventListener("offline", this.offlineEventListener, false);
@@ -413,7 +413,7 @@ export class Socket extends Emitter<{}, {}, SocketReservedEvents> {
         socket: this,
         hostname: this.hostname,
         secure: this.secure,
-        port: this.port
+        port: this.port,
       }
     );
 
@@ -481,7 +481,7 @@ export class Socket extends Emitter<{}, {}, SocketReservedEvents> {
       .on("drain", this.onDrain.bind(this))
       .on("packet", this.onPacket.bind(this))
       .on("error", this.onError.bind(this))
-      .on("close", reason => this.onClose("transport close", reason));
+      .on("close", (reason) => this.onClose("transport close", reason));
   }
 
   /**
@@ -502,7 +502,7 @@ export class Socket extends Emitter<{}, {}, SocketReservedEvents> {
 
       debug('probe transport "%s" opened', name);
       transport.send([{ type: "ping", data: "probe" }]);
-      transport.once("packet", msg => {
+      transport.once("packet", (msg) => {
         if (failed) return;
         if ("pong" === msg.type && "probe" === msg.data) {
           debug('probe transport "%s" pong', name);
@@ -549,7 +549,7 @@ export class Socket extends Emitter<{}, {}, SocketReservedEvents> {
     }
 
     // Handle any error that happens while probing
-    const onerror = err => {
+    const onerror = (err) => {
       const error = new Error("probe error: " + err);
       // @ts-ignore
       error.transport = transport.name;
@@ -828,7 +828,7 @@ export class Socket extends Emitter<{}, {}, SocketReservedEvents> {
     const packet = {
       type: type,
       data: data,
-      options: options
+      options: options,
     };
     this.emitReserved("packetCreate", packet);
     this.writeBuffer.push(packet);

lib/transports/index.ts

@@ -3,5 +3,5 @@ import { WS } from "./websocket.js";
 
 export const transports = {
   websocket: WS,
-  polling: Polling
+  polling: Polling,
 };

lib/transports/polling.ts

@@ -13,9 +13,9 @@ const debug = debugModule("engine.io-client:polling"); // debug()
 
 function empty() {}
 
-const hasXHR2 = (function() {
+const hasXHR2 = (function () {
   const xhr = new XMLHttpRequest({
-    xdomain: false
+    xdomain: false,
   });
   return null != xhr.responseType;
 })();
@@ -96,7 +96,7 @@ export class Polling extends Transport {
       if (this.polling) {
         debug("we are currently polling - waiting to pause");
         total++;
-        this.once("pollComplete", function() {
+        this.once("pollComplete", function () {
           debug("pre-pause polling complete");
           --total || pause();
         });
@@ -105,7 +105,7 @@ export class Polling extends Transport {
       if (!this.writable) {
         debug("we are currently writing - waiting to pause");
         total++;
-        this.once("drain", function() {
+        this.once("drain", function () {
           debug("pre-pause writing complete");
           --total || pause();
         });
@@ -134,7 +134,7 @@ export class Polling extends Transport {
    */
   onData(data) {
     debug("polling got data %s", data);
-    const callback = packet => {
+    const callback = (packet) => {
       // if its the first message we consider the transport open
       if ("opening" === this.readyState && packet.type === "open") {
         this.onOpen();
@@ -199,7 +199,7 @@ export class Polling extends Transport {
   write(packets) {
     this.writable = false;
 
-    encodePayload(packets, data => {
+    encodePayload(packets, (data) => {
       this.doWrite(data, () => {
         this.writable = true;
         this.emitReserved("drain");
@@ -269,7 +269,7 @@ export class Polling extends Transport {
   doWrite(data, fn) {
     const req = this.request({
       method: "POST",
-      data: data
+      data: data,
     });
     req.on("success", fn);
     req.on("error", (xhrStatus, context) => {

lib/transports/websocket-constructor.browser.ts

@@ -4,7 +4,7 @@ export const nextTick = (() => {
   const isPromiseAvailable =
     typeof Promise === "function" && typeof Promise.resolve === "function";
   if (isPromiseAvailable) {
-    return cb => Promise.resolve().then(cb);
+    return (cb) => Promise.resolve().then(cb);
   } else {
     return (cb, setTimeoutFn) => setTimeoutFn(cb, 0);
   }

lib/transports/websocket.ts

@@ -6,7 +6,7 @@ import {
   defaultBinaryType,
   nextTick,
   usingBrowserWebSocket,
-  WebSocket
+  WebSocket,
 } from "./websocket-constructor.js";
 import debugModule from "debug"; // debug()
 import { encodePacket } from "engine.io-parser";
@@ -111,13 +111,13 @@ export class WS extends Transport {
       }
       this.onOpen();
     };
-    this.ws.onclose = closeEvent =>
+    this.ws.onclose = (closeEvent) =>
       this.onClose({
         description: "websocket connection closed",
-        context: closeEvent
+        context: closeEvent,
       });
-    this.ws.onmessage = ev => this.onData(ev.data);
-    this.ws.onerror = e => this.onError("websocket error", e);
+    this.ws.onmessage = (ev) => this.onData(ev.data);
+    this.ws.onerror = (e) => this.onError("websocket error", e);
   }
 
   /**
@@ -135,7 +135,7 @@ export class WS extends Transport {
       const packet = packets[i];
       const lastPacket = i === packets.length - 1;
 
-      encodePacket(packet, this.supportsBinary, data => {
+      encodePacket(packet, this.supportsBinary, (data) => {
         // always create a new object (GH-437)
         const opts: { compress?: boolean } = {};
         if (!usingBrowserWebSocket) {

package-lock.json

@@ -62,7 +62,7 @@
     "expect.js": "^0.3.1",
     "express": "^4.17.1",
     "mocha": "^3.2.0",
-    "prettier": "^1.19.1",
+    "prettier": "^2.8.1",
     "rollup": "^2.58.0",
     "rollup-plugin-terser": "^7.0.2",
     "socket.io-browsers": "~1.0.4",

package.json

@@ -4,16 +4,16 @@ module.exports = {
   ...config,
   output: {
     ...config.output,
-    filename: "engine.io.min.js"
+    filename: "engine.io.min.js",
   },
   mode: "production",
   module: {
     rules: [
       ...config.module.rules,
       {
         test: /\.js$/,
-        loader: "webpack-remove-debug"
-      }
-    ]
-  }
+        loader: "webpack-remove-debug",
+      },
+    ],
+  },
 };

support/prod.config.js

@@ -16,12 +16,12 @@ module.exports = {
     format: "esm",
     sourcemap: true,
     plugins: [terser()],
-    banner
+    banner,
   },
   plugins: [
     nodeResolve({
-      browser: true
+      browser: true,
     }),
-    commonjs()
-  ]
+    commonjs(),
+  ],
 };

support/rollup.config.esm.js

@@ -18,26 +18,26 @@ module.exports = {
       format: "umd",
       name: "eio",
       sourcemap: true,
-      banner
+      banner,
     },
     {
       file: "./dist/engine.io.min.js",
       format: "umd",
       name: "eio",
       sourcemap: true,
       plugins: [terser()],
-      banner
-    }
+      banner,
+    },
   ],
   plugins: [
     nodeResolve({
-      browser: true
+      browser: true,
     }),
     commonjs(),
     babel({
       babelHelpers: "bundled",
       presets: [["@babel/env"]],
-      plugins: ["@babel/plugin-transform-object-assign"]
-    })
-  ]
+      plugins: ["@babel/plugin-transform-object-assign"],
+    }),
+  ],
 };