INT-3364: Update dependencies (#472)

* Update dev dependencies

* Fix no-floating-promises typescript-eslint error

Added change to await nextTick promises.

typescript-eslint rule: https://typescript-eslint.io/rules/no-floating-promises/

* Disable `@typescript-eslint/no-floating-promises` rule on `nextTick` calls

`nextTick` cannot be awaited because it is called inside non-async functions.

* Update tinymce/eslint

* Update other dependencies

---------

Co-authored-by: Ben Tran <[email protected]>

Author: soritaheng

.eslintrc.json

@@ -0,0 +1,76 @@
+// eslint.config.js
+import { defineConfig } from "eslint/config";
+import tinymceEslintPlugin from "@tinymce/eslint-plugin";
+import js from '@eslint/js';
+
+export default defineConfig([
+	{
+    plugins: {
+        "@tinymce": tinymceEslintPlugin
+    },
+    extends: [ "@tinymce/standard" ],
+    files: [
+      "src/**/*.ts",
+      "src/**/*.tsx"
+    ],
+    ignores: [
+        "src/demo/demo.ts",
+        "src/**/*.stories.*"
+    ],
+    languageOptions: {
+      parserOptions: {
+          sourceType: "module",
+          project: [
+              "./tsconfig.json"
+          ]
+      },
+    },
+    rules: {
+      "@tinymce/prefer-fun": "off"
+    }
+  },
+  {
+    files: [ "src/**/*.stories.*" ],
+    languageOptions: {
+      parserOptions: {
+        sourceType: "module",
+        project: [
+            "./tsconfig.storybook.json"
+        ]
+      },
+    },
+    rules: {
+      "@tinymce/prefer-fun": "off",
+      "no-console": "off",
+      "@typescript-eslint/no-implied-eval": "off",
+      "@typescript-eslint/no-unsafe-argument": "off"
+    }
+  },
+  {
+    files: [
+      "**/*.js"
+    ],
+    plugins: { js },
+    env: {
+      "es6": true,
+      "node": true,
+      "browser": true
+    },
+    extends: [ "js/recommended" ],
+    parser: "espree",
+    languageOptions: {
+      parserOptions: {
+        ecmaVersion: 2020,
+        sourceType: "module"
+      }
+    },
+    rules: {
+      indent: [ "error", 2, { "SwitchCase": 1 } ],
+      "no-shadow": "error",
+      "no-unused-vars": [ "error", { "argsIgnorePattern": "^_" } ],
+      "object-curly-spacing": [ "error", "always", { "arraysInObjects": false, "objectsInObjects": false } ],
+      quotes: [ "error", "single" ],
+      semi: "error"
+    }
+  }
+]);

eslint.config.mjs

@@ -44,8 +44,8 @@
   "devDependencies": {
     "@babel/core": "^7.20.2",
     "@ephox/agar": "^8.0.1",
-    "@ephox/bedrock-client": "^14.1.1",
-    "@ephox/bedrock-server": "^14.1.4",
+    "@ephox/bedrock-client": "^14.0.0",
+    "@ephox/bedrock-server": "^14.0.0",
     "@ephox/katamari": "^9.1.6",
     "@ephox/sugar": "^9.3.1",
     "@storybook/addon-actions": "^6.5.13",
@@ -56,7 +56,7 @@
     "@storybook/manager-webpack5": "^6.5.16",
     "@storybook/vue3": "^6.5.13",
     "@tinymce/beehive-flow": "^0.19.0",
-    "@tinymce/eslint-plugin": "^2.3.1",
+    "@tinymce/eslint-plugin": "^3.0.0",
     "@tinymce/miniature": "^6.0.0",
     "@types/node": "^20.14.0",
     "@vitejs/plugin-vue": "^5.0.5",

package.json

@@ -107,6 +107,8 @@ export const Editor = defineComponent({
           cache = vueEditor.getContent();
         }
         getTinymce()?.remove(vueEditor);
+        // The Vue docs state you can either use the callback form or await it. Ref: https://vuejs.org/api/general.html#nexttick
+        // eslint-disable-next-line @typescript-eslint/no-floating-promises
         nextTick(() => initWrapper());
       }
     });
@@ -151,7 +153,11 @@ export const Editor = defineComponent({
         cache = vueEditor.getContent();
         getTinymce()?.remove(vueEditor);
         conf = { ...conf, ...init, ...defaultInitValues };
+
+        // The Vue docs state you can either use the callback form or await it. Ref: https://vuejs.org/api/general.html#nexttick
+        // eslint-disable-next-line @typescript-eslint/no-floating-promises
         nextTick(() => initWrapper());
+
       }
     };
     ctx.expose({

src/main/ts/components/Editor.ts

@@ -7,7 +7,6 @@ import { cleanupGlobalTinymce, VALID_API_KEY } from '../alien/TestHelper';
 import { Arr } from '@ephox/katamari';
 
 describe('Editor Component Initialization Tests', () => {
-  // eslint-disable-next-line @typescript-eslint/require-await
   const pFakeType = async (str: string, vmContext: any) => {
     vmContext.editor.getBody().innerHTML = '<p>' + str + '</p>';
     Keyboard.keystroke(Keys.space(), {}, SugarElement.fromDom(vmContext.editor.getBody()) as SugarElement<Node>);

src/test/ts/browser/InitTest.ts

@@ -33,6 +33,7 @@ describe('LoadTinyTest', () => {
       await pRender({}, `
         <editor
           :init="init"
+          license-key="gpl"
           tinymce-script-src="/project/node_modules/tinymce-7/tinymce.min.js"
         ></editor>
       `);
@@ -79,6 +80,7 @@ describe('LoadTinyTest', () => {
           :init="init"
           api-key="${VALID_API_KEY}"
           cloud-channel="8"
+          license-key="gpl"
         ></editor>
       `);