fix: lint

Author: nusr

.eslintignore

@@ -0,0 +1,12 @@
+
+node_modules/
+lib/
+dist/
+build/
+tests/
+coverage/
+demo/
+es/
+.rax/
+.ice/
+scripts/*.js

.eslintrc.js

@@ -0,0 +1,14 @@
+/* eslint-env node,browser */
+module.exports = {
+  extends: ['eslint:recommended', 'plugin:@typescript-eslint/recommended'],
+  parser: '@typescript-eslint/parser',
+  plugins: ['@typescript-eslint'],
+  root: true,
+  rules: {
+    '@typescript-eslint/no-explicit-any': 'warn',
+  },
+  env: {
+    browser: true,
+    node: true,
+  }
+};

.github/workflows/main.yml

@@ -18,6 +18,7 @@ jobs:
           npm install
           npm run build
           npm run type-check
+          npm run lint
           npm run coverage
       - name: Upload coverage
         uses: codecov/codecov-action@v3

declarations.d.ts

@@ -1 +1 @@
-declare module '*.css'
+declare module '*.css'

e2e/toolbar.test.ts

@@ -6,16 +6,14 @@ describe('toolbar.test', () => {
   }, 20 * 1000);
 
   const clickToolbar = async function (selector: string) {
-    await page.click(selector);
+    await __browserPage.click(selector);
     await sleep(1000);
-    const result = await page.$eval(selector, (element) => element.className);
+    const result = await __browserPage.$eval(selector, (element) => element.className);
     return result.split(' ').length > 1;
   };
   for (const item of ['bold', 'italic', 'wrap-text']) {
     test(`test toolbar ${item}`, async () => {
-      expect(
-        await clickToolbar(getTestIdSelector(`toolbar-${item}`)),
-      ).toBeTruthy();
+      expect(await clickToolbar(getTestIdSelector(`toolbar-${item}`))).toBeTruthy();
     });
   }
 });

e2e/util.ts

@@ -1,7 +1,8 @@
 import * as puppeteer from 'puppeteer';
 
 declare global {
-  var page: puppeteer.Page;
+  // eslint-disable-next-line no-var
+  var __browserPage: puppeteer.Page;
 }
 
 async function setupPuppeteer() {
@@ -13,20 +14,22 @@ async function setupPuppeteer() {
       height: 800,
     },
   });
-  global.page = await browser.newPage();
+  __browserPage = await browser.newPage();
 }
 
-export function sleep(milliseconds: number) {
+export function sleep(milliseconds: number): Promise<unknown> {
   return new Promise((resolve) => setTimeout(resolve, milliseconds));
 }
 
-export async function openPage() {
+export async function openPage(): Promise<void> {
   await setupPuppeteer();
-  const filePath = `http://localhost:8000`;
-  await page.goto(filePath);
+  const port = 8000;
+  const filePath = `https://localhost:${port}`;
+  console.log(filePath);
+  await __browserPage.goto(filePath);
   await sleep(1000);
 }
 
-export function getTestIdSelector(testId: string) {
+export function getTestIdSelector(testId: string): string {
   return `[data-testid="${testId}"]`;
 }

package.json

@@ -9,11 +9,12 @@
     "build": "node ./scripts/esbuild.js",
     "test": "jest --config ./scripts/unit-test-jest.config.js --detectOpenHandles",
     "coverage": "npm run test -- --coverage",
-    "ci": "npm run test && npm run type-check && npm run build && npm run e2e",
+    "ci": "npm run test && npm run type-check && npm run build && npm run e2e && npm run lint",
     "type-check": "npx tsc",
     "postinstall": "node ./post-install.js",
     "prebuild": "node ./post-install.js",
-    "e2e": "jest --config ./scripts/e2e-jest.config.js --forceExit --detectOpenHandles"
+    "e2e": "jest --config ./scripts/e2e-jest.config.js --forceExit --detectOpenHandles",
+    "lint": "eslint --fix --quiet --ext  .ts,.tsx ./"
   },
   "repository": {
     "type": "git",
@@ -29,8 +30,11 @@
     "@types/jest": "29.5.8",
     "@types/react": "18.2.37",
     "@types/react-dom": "18.2.15",
+    "@typescript-eslint/eslint-plugin": "^6.11.0",
+    "@typescript-eslint/parser": "^6.11.0",
     "canvas": "2.11.2",
     "esbuild": "0.19.5",
+    "eslint": "^8.54.0",
     "jest": "29.7.0",
     "jest-environment-jsdom": "29.7.0",
     "puppeteer": "21.5.2",
@@ -45,4 +49,4 @@
     "react": "18.2.0",
     "react-dom": "18.2.0"
   }
-}
+}