chore: Bump dependencies (#287)

* bump node

* fix flaky tests

* bump test packages

* bump typescript

* bump vite to fix types issue

* allow bypassing onUnhandledRequest to fix socketio tests

* remove cross-fetch, no longer needed

* bump patch versions

* bump more patch versions

* bump eslint

* remove unnecessary tsconfig changes

* better eslint rule

* Object.hasOwn

Author: jimmycallin

.eslintignore

@@ -1,4 +1 @@
-#!/bin/sh
-. "$(dirname "$0")/_/husky.sh"
-
 yarn lint-staged

.eslintrc

@@ -1 +1 @@
-20
+22

.husky/pre-commit

@@ -0,0 +1,45 @@
+import eslint from "@eslint/js";
+import tseslint from "typescript-eslint";
+import vitest from "@vitest/eslint-plugin";
+import globals from "globals";
+
+// @ts-check
+
+export default tseslint.config(
+  { ignores: ["dist"] },
+  eslint.configs.recommended,
+  tseslint.configs.recommended,
+  {
+    files: ["test/**/*.{ts,js}", "vitest.setup.js", "vitest.globalsetup.js"],
+    ...vitest.configs.recommended,
+    languageOptions: {
+      globals: {
+        ...globals.browser,
+        ...globals.node,
+        ...globals.es2025,
+        ...vitest.environments.env.globals,
+      },
+    },
+  },
+  {
+    files: ["source/**/*.{ts,js}"],
+    languageOptions: {
+      globals: { ...globals.browser, ...globals.node, ...globals.es2025 },
+    },
+  },
+  {
+    files: ["**/*.ts"],
+    rules: {
+      "@typescript-eslint/no-explicit-any": "off",
+      "@typescript-eslint/no-empty-object-type": "off",
+      "@typescript-eslint/ban-ts-comment": "off",
+      "@typescript-eslint/no-unused-vars": [
+        "error",
+        {
+          caughtErrors: "none",
+          ignoreRestSiblings: true,
+        },
+      ],
+    },
+  },
+);

.nvmrc

@@ -25,22 +25,24 @@
     "dist"
   ],
   "devDependencies": {
-    "@types/uuid": "^9.0.8",
-    "cross-fetch": "^4.0.0",
-    "dayjs": "^1.11.10",
-    "eslint": "^8.56.0",
-    "eslint-config-react-app": "^7.0.1",
-    "husky": "^9.0.10",
-    "jsdom": "^24.0.0",
-    "lint-staged": "^15.2.2",
-    "msw": "^2.2.0",
+    "@types/node": "^22.13.9",
+    "@types/uuid": "^10.0.0",
+    "@vitest/eslint-plugin": "^1.1.36",
+    "dayjs": "^1.11.13",
+    "eslint": "^9.21.0",
+    "globals": "^16.0.0",
+    "husky": "^9.1.7",
+    "jsdom": "^26.0.0",
+    "lint-staged": "^15.4.3",
+    "msw": "^2.7.3",
     "pinst": "^3.0.0",
-    "prettier": "^3.2.5",
-    "typescript": "^5.3.3",
-    "vite": "^5.4.7",
-    "vite-plugin-dts": "^3.7.2",
-    "vitest": "^1.2.2",
-    "ws": "^8.16.0"
+    "prettier": "^3.5.3",
+    "typescript": "^5.8.2",
+    "typescript-eslint": "^8.26.0",
+    "vite": "^6.2.0",
+    "vite-plugin-dts": "^4.5.3",
+    "vitest": "^3.0.7",
+    "ws": "^8.18.1"
   },
   "repository": {
     "type": "git",
@@ -59,9 +61,9 @@
   "homepage": "http://ftrack.com",
   "dependencies": {
     "isomorphic-ws": "^5.0.0",
-    "loglevel": "^1.9.1",
+    "loglevel": "^1.9.2",
     "moment": "^2.30.1",
-    "uuid": "^9.0.1"
+    "uuid": "^11.1.0"
   },
   "peerDependencies": {
     "ws": "^8.13.0"

eslint.config.mjs

@@ -306,7 +306,7 @@ export class Session<
     if (data && data.constructor === Object) {
       const out: Data = {};
       for (const key in data) {
-        if (data.hasOwnProperty(key)) {
+        if (Object.hasOwn(data, key)) {
           out[key] = this.encode(data[key]);
         }
       }
@@ -545,7 +545,7 @@ export class Session<
     const mergedEntity = identityMap[identifier];
 
     for (const key in entity) {
-      if (entity.hasOwnProperty(key)) {
+      if (Object.hasOwn(entity, key)) {
         mergedEntity[key] = this.decode(entity[key], identityMap, {
           decodeDatesAsIso,
           ensureSerializableResponse,

package.json

@@ -322,17 +322,9 @@ export class Uploader<TEntityTypeMap extends Record<string, any>> {
     part: MultiPartUploadPart,
     onUploadChunkStart: () => void,
   ): Promise<void> {
-    try {
-      const status = await this.uploadFileChunk(
-        chunk,
-        part,
-        onUploadChunkStart,
-      );
-      if (status !== 200) {
-        throw new CreateComponentError(`Failed to upload file part: ${status}`);
-      }
-    } catch (error) {
-      throw error;
+    const status = await this.uploadFileChunk(chunk, part, onUploadChunkStart);
+    if (status !== 200) {
+      throw new CreateComponentError(`Failed to upload file part: ${status}`);
     }
   }
 
@@ -516,7 +508,7 @@ export class Uploader<TEntityTypeMap extends Record<string, any>> {
       };
 
       for (const key in headers) {
-        if (headers.hasOwnProperty(key) && key !== "Content-Length") {
+        if (Object.hasOwn(headers, key) && key !== "Content-Length") {
           this.xhr.setRequestHeader(key, headers[key]);
         }
       }

source/session.ts

@@ -123,7 +123,7 @@ describe("EventHub", () => {
     expect(callback).not.toHaveBeenCalled();
   });
 
-  test("should handle events with unexpected topics", () => {
+  test("should handle events with more unexpected topics", () => {
     const callback = vi.fn();
     eventHub.subscribe("topic=*", callback);
     const testEvent = { topic: null, data: {} };

source/uploader.ts

@@ -1,11 +1,11 @@
 // :copyright: Copyright (c) 2022 ftrack
 import { HttpResponse, type PathParams, http, HttpHandler } from "msw";
 import fs from "fs/promises";
-import querySchemas from "./fixtures/query_schemas.json";
-import queryServerInformation from "./fixtures/query_server_information.json";
-import getUploadMetadata from "./fixtures/get_upload_metadata.json";
-import completeMultipartUpload from "./fixtures/complete_multipart_upload.json";
-import exampleQuery from "./fixtures/query_select_name_from_task_limit_1.json";
+import querySchemas from "./fixtures/query_schemas.json" with { type: "json" };
+import queryServerInformation from "./fixtures/query_server_information.json" with { type: "json" };
+import getUploadMetadata from "./fixtures/get_upload_metadata.json" with { type: "json" };
+import completeMultipartUpload from "./fixtures/complete_multipart_upload.json" with { type: "json" };
+import exampleQuery from "./fixtures/query_select_name_from_task_limit_1.json" with { type: "json" };
 import { setupServer } from "msw/node";
 const InvalidCredentialsError = {
   content:
@@ -25,7 +25,7 @@ function authenticate(info: Parameters<Parameters<typeof http.post>[1]>[0]) {
 
 function pick<T>(object: T, keys: (keyof T)[]) {
   return keys.reduce((obj, key) => {
-    if (object && object.hasOwnProperty(key)) {
+    if (object && Object.hasOwn(object, key)) {
       obj[key] = object[key];
     }
     return obj;
@@ -66,7 +66,7 @@ export const handlers: HttpHandler[] = [
               return queryServerInformation;
             case "query_schemas":
               return querySchemas;
-            case "create":
+            case "create": {
               // create are fetched from test/fixtures where the file name matches the full expression
               const createFixture = await fs.readFile(
                 `${__dirname}/fixtures/create_${entityType.toLowerCase()}.json`,
@@ -82,6 +82,7 @@ export const handlers: HttpHandler[] = [
                   ...pick(entityData, ["id"]),
                 },
               };
+            }
             case "delete":
               return {
                 action: "delete",
@@ -128,7 +129,7 @@ export const handlers: HttpHandler[] = [
       },
     });
   }),
-  http.put("http://ftrack.test/file-url", async (info) => {
+  http.put("http://ftrack.test/file-url", async () => {
     return new Response(null, {
       status: 200,
       headers: { "Access-Control-Allow-Origin": "*" },
@@ -137,11 +138,11 @@ export const handlers: HttpHandler[] = [
   // Get socket io session id
   http.get("https://ftrack.test/socket.io/1/", handleSocketIORequest),
   http.get("http://ftrack.test/socket.io/1/", handleSocketIORequest),
-  http.get("https://ftrack.test/*", (info) => {
+  http.get("https://ftrack.test/*", () => {
     return new Response(null, { status: 200 });
   }),
 
-  http.get("http://ftrack.test:8080/*", (info) => {
+  http.get("http://ftrack.test:8080/*", () => {
     return new Response(null, { status: 200 });
   }),
 ];