style(sdk): fix ESLint issues

Most of these were autofixed with `eslint --fix src/`

Author: aloisklink

cSpell.json

@@ -2,6 +2,7 @@
   "version": "0.2",
   "language": "en",
   "words": [
+    "Alois",
     "Cataa",
     "colour",
     "Cookiebot",

packages/sdk/src/index.e2e.test.ts

@@ -6,7 +6,7 @@ import { afterAll, beforeAll, describe, expect, it } from 'vitest';
 
 import process from 'node:process';
 import { AxiosError } from 'axios';
-import { MCDocument } from './types.js';
+import type { MCDocument } from './types.js';
 
 let testProjectId = '316557b3-cb6f-47ed-acf7-fcfb7ce188d5';
 let baseURL = new URL('https://test.mermaidchart.com');
@@ -44,14 +44,14 @@ beforeAll(async() => {
   // confirm that testProjectId is valid
   if (process.env.TEST_MERMAIDCHART_PROJECT_ID) {
     testProjectId = process.env.TEST_MERMAIDCHART_PROJECT_ID;
-    if (!projects.find((project) => project.id === testProjectId)) {
+    if (!projects.some((project) => project.id === testProjectId)) {
       throw new Error(
         `Invalid environment variable TEST_MERMAIDCHART_PROJECT_ID. `
         + `Please go to ${new URL('/app/projects', baseURL)} and create one that you have access to.`
       );
     }
   } else {
-    if (!projects.find((project) => project.id === testProjectId)) {
+    if (!projects.some((project) => project.id === testProjectId)) {
       throw new Error(
         `Missing environment variable TEST_MERMAIDCHART_PROJECT_ID. `
         + `Please go to ${new URL('/app/projects', baseURL)} and create one.`
@@ -80,7 +80,7 @@ const documentMatcher = expect.objectContaining({
  */
 const documentsToDelete = new Set<MCDocument["documentID"]>();
 afterAll(async() => {
-  await Promise.all(Array.from(documentsToDelete).map(async(document) => {
+  await Promise.all([...documentsToDelete].map(async(document) => {
     if (documentsToDelete.delete(document)) {
       await client.deleteDocument(document);
     }

packages/sdk/src/index.test.ts

@@ -1,6 +1,6 @@
 import { beforeEach, describe, expect, it, vi } from 'vitest';
 import { MermaidChart } from './index.js';
-import { AuthorizationData } from './types.js';
+import type { AuthorizationData } from './types.js';
 
 import { OAuth2Client } from '@badgateway/oauth2-client';
 

packages/sdk/src/index.ts

@@ -57,7 +57,8 @@ export class MermaidChart {
     this.axios.interceptors.response.use((res: AxiosResponse) => {
       // Reset token if a 401 is thrown
       if (res.status === 401) {
-        this.resetAccessToken();
+        // don't care if this function rejects/resolves
+        void this.resetAccessToken();
       }
       return res;
     });
@@ -143,7 +144,7 @@ export class MermaidChart {
   /**
    * This method is used after authentication to save the access token.
    * It should be called by the plugins if any update to access token is made outside this lib.
-   * @param accessToken access token to use for requests
+   * @param accessToken - access token to use for requests
    */
   public async setAccessToken(accessToken: string): Promise<void> {
     this.axios.defaults.headers.common['Authorization'] = `Bearer ${accessToken}`;
@@ -211,12 +212,12 @@ export class MermaidChart {
   /**
    * Update the given document.
    *
-   * @param document The document to update.
+   * @param document - The document to update.
    */
   public async setDocument(
     document: Pick<MCDocument, 'documentID' | 'projectID'> & Partial<MCDocument>,
   ) {
-    const {data} = await this.axios.put<{result: "ok"} | {result: "failed", error: any}>(
+    const {data} = await this.axios.put<{result: "ok"} | {result: "failed", error: unknown}>(
       URLS.rest.documents.pick(document).self,
       document,
     );
@@ -230,7 +231,7 @@ export class MermaidChart {
 
   /**
    * Delete the given document.
-   * @param documentID The ID of the document to delete.
+   * @param documentID - The ID of the document to delete.
    * @returns Metadata about the deleted document.
    */
   public async deleteDocument(documentID: MCDocument['documentID']) {