chore: use prettier v3 everywhere (#5169)

socketio · Sep 16, 2024 · a1ccba3 · a1ccba3
1 parent e347a3c
commit a1ccba3
Show file tree

Hide file tree

Showing 68 changed files with 513 additions and 508 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,4 +1,5 @@
 {
+  "name": "socket.io",
   "private": true,
   "workspaces": [
     "packages/socket.io-component-emitter",
@@ -52,7 +53,7 @@
     "mocha": "^10.6.0",
     "node-forge": "^1.3.1",
     "nyc": "^17.0.0",
-    "prettier": "^2.8.8",
+    "prettier": "^3.3.2",
     "redis": "^4.6.15",
     "rimraf": "^6.0.0",
     "rollup": "^2.79.1",

diff --git a/packages/engine.io-client/lib/globals.node.ts b/packages/engine.io-client/lib/globals.node.ts
@@ -55,7 +55,7 @@ export function parse(setCookieString: string): Cookie {
       case "Max-Age":
         const expiration = new Date();
         expiration.setUTCSeconds(
-          expiration.getUTCSeconds() + parseInt(value, 10)
+          expiration.getUTCSeconds() + parseInt(value, 10),
         );
         cookie.expires = expiration;
         break;

diff --git a/packages/engine.io-client/lib/socket.ts b/packages/engine.io-client/lib/socket.ts
@@ -381,8 +381,8 @@ export class SocketWithoutUpgrade extends Emitter<
       (typeof location !== "undefined" && location.port
         ? location.port
         : this.secure
-        ? "443"
-        : "80");
+          ? "443"
+          : "80");
 
     this.transports = [];
     this._transportsByName = {};
@@ -408,7 +408,7 @@ export class SocketWithoutUpgrade extends Emitter<
         transportOptions: {},
         closeOnBeforeunload: false,
       },
-      opts
+      opts,
     );
 
     this.opts.path =
@@ -434,7 +434,7 @@ export class SocketWithoutUpgrade extends Emitter<
         addEventListener(
           "beforeunload",
           this._beforeunloadEventListener,
-          false
+          false,
         );
       }
       if (this.hostname !== "localhost") {
@@ -484,7 +484,7 @@ export class SocketWithoutUpgrade extends Emitter<
         secure: this.secure,
         port: this.port,
       },
-      this.opts.transportOptions[name]
+      this.opts.transportOptions[name],
     );
 
     debug("options: %j", opts);
@@ -749,7 +749,7 @@ export class SocketWithoutUpgrade extends Emitter<
     type: PacketType,
     data?: RawData,
     options?: WriteOptions,
-    fn?: () => void
+    fn?: () => void,
   ) {
     if ("function" === typeof data) {
       fn = data;
@@ -874,7 +874,7 @@ export class SocketWithoutUpgrade extends Emitter<
         removeEventListener(
           "beforeunload",
           this._beforeunloadEventListener,
-          false
+          false,
         );
         removeEventListener("offline", this._offlineEventListener, false);
       }

diff --git a/packages/engine.io-client/lib/transport.ts b/packages/engine.io-client/lib/transport.ts
@@ -14,7 +14,7 @@ export class TransportError extends Error {
   constructor(
     reason: string,
     readonly description: any,
-    readonly context: any
+    readonly context: any,
   ) {
     super(reason);
   }
@@ -79,7 +79,7 @@ export abstract class Transport extends Emitter<
   protected onError(reason: string, description: any, context?: any) {
     super.emitReserved(
       "error",
-      new TransportError(reason, description, context)
+      new TransportError(reason, description, context),
     );
     return this;
   }

diff --git a/packages/engine.io-client/lib/transports/polling-xhr.node.ts b/packages/engine.io-client/lib/transports/polling-xhr.node.ts
@@ -15,12 +15,12 @@ export class XHR extends BaseXHR {
     Object.assign(
       opts,
       { xd: this.xd, cookieJar: this.socket?._cookieJar },
-      this.opts
+      this.opts,
     );
     return new Request(
       (opts) => new XMLHttpRequest(opts),
       this.uri(),
-      opts as RequestOptions
+      opts as RequestOptions,
     );
   }
 }
diff --git a/packages/engine.io-client/lib/transports/polling-xhr.ts b/packages/engine.io-client/lib/transports/polling-xhr.ts
@@ -122,7 +122,7 @@ export class Request extends Emitter<
   constructor(
     private readonly createRequest: (opts: RequestOptions) => XMLHttpRequest,
     uri: string,
-    opts: RequestOptions
+    opts: RequestOptions,
   ) {
     super();
     installTimerFunctions(this, opts);
@@ -151,7 +151,7 @@ export class Request extends Emitter<
       "ca",
       "ciphers",
       "rejectUnauthorized",
-      "autoUnref"
+      "autoUnref",
     );
     opts.xdomain = !!this._opts.xd;
 
@@ -197,7 +197,7 @@ export class Request extends Emitter<
         if (xhr.readyState === 3) {
           this._opts.cookieJar?.parseCookies(
             // @ts-ignore
-            xhr.getResponseHeader("set-cookie")
+            xhr.getResponseHeader("set-cookie"),
           );
         }
 
@@ -354,7 +354,7 @@ function newRequest(opts) {
   if (!xdomain) {
     try {
       return new globalThis[["Active"].concat("Object").join("X")](
-        "Microsoft.XMLHTTP"
+        "Microsoft.XMLHTTP",
       );
     } catch (e) {}
   }

diff --git a/packages/engine.io-client/lib/transports/websocket.node.ts b/packages/engine.io-client/lib/transports/websocket.node.ts
@@ -14,7 +14,7 @@ export class WS extends BaseWS {
   createSocket(
     uri: string,
     protocols: string | string[] | undefined,
-    opts: Record<string, any>
+    opts: Record<string, any>,
   ) {
     if (this.socket?._cookieJar) {
       opts.headers = opts.headers || {};

diff --git a/packages/engine.io-client/lib/transports/websocket.ts b/packages/engine.io-client/lib/transports/websocket.ts
@@ -43,7 +43,7 @@ export abstract class BaseWS extends Transport {
           "origin",
           "maxPayload",
           "family",
-          "checkServerIdentity"
+          "checkServerIdentity",
         );
 
     if (this.opts.extraHeaders) {
@@ -64,7 +64,7 @@ export abstract class BaseWS extends Transport {
   abstract createSocket(
     uri: string,
     protocols: string | string[] | undefined,
-    opts: Record<string, any>
+    opts: Record<string, any>,
   );
 
   /**
@@ -166,7 +166,7 @@ export class WS extends BaseWS {
   createSocket(
     uri: string,
     protocols: string | string[] | undefined,
-    opts: Record<string, any>
+    opts: Record<string, any>,
   ) {
     return !isReactNative
       ? protocols

diff --git a/packages/engine.io-client/lib/transports/webtransport.ts b/packages/engine.io-client/lib/transports/webtransport.ts
@@ -30,7 +30,7 @@ export class WT extends Transport {
       // @ts-ignore
       this._transport = new WebTransport(
         this.createUri("https"),
-        this.opts.transportOptions[this.name]
+        this.opts.transportOptions[this.name],
       );
     } catch (err) {
       return this.emitReserved("error", err);
@@ -51,7 +51,7 @@ export class WT extends Transport {
       this._transport.createBidirectionalStream().then((stream) => {
         const decoderStream = createPacketDecoderStream(
           Number.MAX_SAFE_INTEGER,
-          this.socket.binaryType
+          this.socket.binaryType,
         );
         const reader = stream.readable.pipeThrough(decoderStream).getReader();
 

diff --git a/packages/engine.io-client/test/connection.js b/packages/engine.io-client/test/connection.js
@@ -34,12 +34,12 @@ describe("connection", function () {
     const socket = new Socket();
     socket.on("open", () => {
       socket.send(
-        "\uD800\uDC00-\uDB7F\uDFFF\uDB80\uDC00-\uDBFF\uDFFF\uE000-\uF8FF"
+        "\uD800\uDC00-\uDB7F\uDFFF\uDB80\uDC00-\uDBFF\uDFFF\uE000-\uF8FF",
       );
       socket.on("message", (data) => {
         if ("hi" === data) return;
         expect(data).to.be(
-          "\uD800\uDC00-\uDB7F\uDFFF\uDB80\uDC00-\uDBFF\uDFFF\uE000-\uF8FF"
+          "\uD800\uDC00-\uDB7F\uDFFF\uDB80\uDC00-\uDBFF\uDFFF\uE000-\uF8FF",
         );
         socket.close();
         done();

diff --git a/packages/engine.io-client/test/socket.js b/packages/engine.io-client/test/socket.js
@@ -90,7 +90,7 @@ describe("Socket", function () {
 
     socket.on("error", (err) => {
       expect(err.message).to.eql(
-        useFetch ? "fetch read error" : "xhr poll error"
+        useFetch ? "fetch read error" : "xhr poll error",
       );
       done();
     });
@@ -237,7 +237,7 @@ describe("Socket", function () {
           // err.context is a XMLHttpRequest object
           expect(err.context.readyState).to.eql(4);
           expect(err.context.responseText).to.eql(
-            '{"code":1,"message":"Session ID unknown"}'
+            '{"code":1,"message":"Session ID unknown"}',
           );
         }
       });

diff --git a/packages/engine.io-client/test/support/hooks.js b/packages/engine.io-client/test/support/hooks.js
@@ -21,7 +21,7 @@ exports.mochaHooks = {
       maxHttpBufferSize: 100,
       allowRequest: (req, fn) => {
         const denyRequest = new URL(`http://${req.url}`).searchParams.has(
-          "deny"
+          "deny",
         );
         fn(null, !denyRequest);
       },

diff --git a/packages/engine.io-client/test/transport.js b/packages/engine.io-client/test/transport.js
@@ -100,7 +100,7 @@ describe("Transport", () => {
         timestampRequests: false,
       });
       expect(polling.uri()).to.contain(
-        "http://localhost:3000/engine.io?sid=test"
+        "http://localhost:3000/engine.io?sid=test",
       );
     });
 
@@ -124,7 +124,7 @@ describe("Transport", () => {
         timestampRequests: true,
       });
       expect(polling.uri()).to.match(
-        /http:\/\/localhost\/engine\.io\?(j=[0-9]+&)?(t=[0-9A-Za-z-_]+)/
+        /http:\/\/localhost\/engine\.io\?(j=[0-9]+&)?(t=[0-9A-Za-z-_]+)/,
       );
     });
 
@@ -180,7 +180,7 @@ describe("Transport", () => {
         timestampRequests: true,
       });
       expect(ws.uri()).to.match(
-        /ws:\/\/localhost\/engine\.io\?woot=[0-9A-Za-z-_]+/
+        /ws:\/\/localhost\/engine\.io\?woot=[0-9A-Za-z-_]+/,
       );
     });
 

diff --git a/packages/engine.io-client/test/webtransport.mjs b/packages/engine.io-client/test/webtransport.mjs
@@ -17,7 +17,7 @@ async function setup(opts, cb) {
     [{ shortName: "CN", value: "localhost" }],
     {
       days: 14, // the total length of the validity period MUST NOT exceed two weeks (https://w3c.github.io/webtransport/#custom-certificate-requirements)
-    }
+    },
   );
 
   const engine = new Server(opts);
@@ -73,8 +73,8 @@ function createSocket(port, certificate, opts) {
           },
         },
       },
-      opts
-    )
+      opts,
+    ),
   );
 }
 
@@ -113,7 +113,7 @@ describe("WebTransport", () => {
           httpServer.close();
           success(engine, h3Server, done);
         });
-      }
+      },
     );
   });
 
@@ -137,7 +137,7 @@ describe("WebTransport", () => {
           httpServer.close();
           success(engine, h3Server, done);
         });
-      }
+      },
     );
   });
 
@@ -160,7 +160,7 @@ describe("WebTransport", () => {
             success(engine, h3Server, done);
           }
         });
-      }
+      },
     );
   });
 

diff --git a/packages/engine.io-parser/package.json b/packages/engine.io-parser/package.json
@@ -10,9 +10,6 @@
     "require": "./build/cjs/index.js"
   },
   "types": "build/esm/index.d.ts",
-  "devDependencies": {
-    "prettier": "^3.3.2"
-  },
   "scripts": {
     "compile": "rimraf ./build && tsc && tsc -p tsconfig.esm.json && ./postcompile.sh",
     "test": "npm run format:check && npm run compile && if test \"$BROWSERS\" = \"1\" ; then npm run test:browser; else npm run test:node; fi",