Merge branch 'tweak_for_eslint'

eslint.config.mjs

@@ -2,7 +2,6 @@ import eslint from "@eslint/js";
 import tseslint from 'typescript-eslint';
 
 const commonRules = {
-    "@typescript-eslint/no-explicit-any": "off",
     "@typescript-eslint/consistent-type-assertions": ["error", {
         assertionStyle: "as",
         objectLiteralTypeAssertions: "never",
@@ -34,7 +33,6 @@ const commonRules = {
             "ignoreRestSiblings": true
         }
     ],
-    "@typescript-eslint/no-unsafe-enum-comparison": "off",
     "@typescript-eslint/prefer-includes": "error",
     "@typescript-eslint/prefer-readonly": "error",
     "no-return-await": "off",
@@ -175,7 +173,6 @@ export default tseslint.config(
                     format: null,
                 }
             ],
-            "@typescript-eslint/ban-ts-comment": "off",
             ...commonRules
         },
     }

src/components/manager.ts

@@ -865,6 +865,7 @@ export class Manager {
 
     private buildOnFileChanged(fileUri: vscode.Uri, bibChanged = false) {
         const configuration = vscode.workspace.getConfiguration('latex-toybox', fileUri)
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
         if (configuration.get('latex.autoBuild.run') as string !== BuildEvents.onFileChange) {
             return
         }
@@ -873,6 +874,7 @@ export class Manager {
 
     private buildOnSave(fileUri: vscode.Uri) {
         const configuration = vscode.workspace.getConfiguration('latex-toybox', fileUri)
+        // eslint-disable-next-line @typescript-eslint/no-unsafe-enum-comparison
         if (configuration.get('latex.autoBuild.run') as string !== BuildEvents.onSave) {
             return
         }

src/utils/externalpromise.ts

@@ -1,6 +1,7 @@
 function promiseTriplet<T>() {
     let resolve: ((value: T | PromiseLike<T>) => void) = () => undefined
-    let reject: ((reason?: any) => void) = () => undefined
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    let reject: ((reason: any) => void) = () => undefined
     const promise = new Promise<T>((r, rej) => {
         resolve = r
         reject = rej
@@ -27,7 +28,8 @@ export class ExternalPromise<T> {
         this.promiseTriplet.resolve(value)
     }
 
-    reject(reason?: any) {
+    // eslint-disable-next-line @typescript-eslint/no-explicit-any
+    reject(reason: any) {
         if (this.#isResolved) {
             return
         }

src/utils/utils.ts

@@ -30,6 +30,7 @@ export function escapeRegExp(str: string) {
  * @author Joseph JDBar Barron
  * @link https://dev.to/jdbar/the-problem-with-handling-node-js-errors-in-typescript-and-the-workaround-m64
  */
+// eslint-disable-next-line @typescript-eslint/no-explicit-any
 export function instanceOfNodeError<T extends new(...args: any) => Error>(
     value: unknown,
     errorType: T

test/unittests/components/linter.test.ts

@@ -1,4 +1,4 @@
-/* eslint-disable @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-argument, @typescript-eslint/no-unsafe-member-access */
+/* eslint-disable @typescript-eslint/no-explicit-any, @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-argument, @typescript-eslint/no-unsafe-member-access */
 
 import * as vscode from 'vscode'
 import * as fs from 'node:fs'

test/unittests/providers/structure.test.ts

@@ -1,4 +1,4 @@
-/* eslint-disable @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-argument, @typescript-eslint/no-unsafe-member-access */
+/* eslint-disable @typescript-eslint/no-explicit-any, @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-argument, @typescript-eslint/no-unsafe-member-access */
 
 import * as vscode from 'vscode'
 import * as path from 'node:path'

viewer/components/pagetrimmer.ts

@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-unsafe-enum-comparison */
 import { ScaleMode, trimSelectElement, viewerDom } from './constants.js'
 import type { IPDFViewerApplication } from './interface.js'
 

viewer/latextoybox.ts

@@ -1,3 +1,4 @@
+// eslint-disable-next-line @typescript-eslint/ban-ts-comment
 // @ts-expect-error
 const importPdfMjsPromise = import('/node_modules/pdfjs-dist/build/pdf.mjs')
 // Edit viewer.html
@@ -168,6 +169,7 @@ const extension = new LateXToyboxPdfViewer()
 await extension.waitSetupAppOptionsReady()
 
 // Defines PDFViewerApplication, PDFViewerApplicationOptions, and PDFViewerApplicationConstants globally.
+// eslint-disable-next-line @typescript-eslint/ban-ts-comment
 // @ts-expect-error
 await import('/viewer.mjs')