Expect ESLint/vue-tsc errors for litegraph src

2026-04-23 15:59:47 +00:00 · 2025-08-05 17:15:41 -04:00
parent c53f197de2
commit 500c9c6631
12 changed files with 133 additions and 54 deletions
--- a/eslint.config.js
+++ b/eslint.config.js
@@ -14,8 +14,7 @@ export default [
    ignores: [
      'src/scripts/*',
      'src/extensions/core/*',
-      'src/types/vue-shim.d.ts',
-      'src/lib/litegraph/**/*' // TODO: Make litegraph code compatible with frontend's ESLint
+      'src/types/vue-shim.d.ts'
    ]
  },
  {
--- a/src/lib/litegraph/src/ContextMenu.ts
+++ b/src/lib/litegraph/src/ContextMenu.ts
@@ -274,6 +274,7 @@ export class ContextMenu<TValue = unknown> {
    }

    // menu option clicked
+    // eslint-disable-next-line @typescript-eslint/no-this-alias
    const that = this
    function inner_onclick(this: ContextMenuDivElement<TValue>, e: MouseEvent) {
      const value = this.value
--- a/src/lib/litegraph/src/CurveEditor.ts
+++ b/src/lib/litegraph/src/CurveEditor.ts
@@ -43,7 +43,7 @@ export class CurveEditor {
  draw(
    ctx: CanvasRenderingContext2D,
    size: Rect,
-    // @ts-ignore TODO: Fix after migration to frontend tsconfig rules
+    // @ts-expect-error - LGraphCanvas parameter type needs fixing
    graphcanvas?: LGraphCanvas,
    background_color?: string,
    line_color?: string,
--- a/src/lib/litegraph/src/LGraph.ts
+++ b/src/lib/litegraph/src/LGraph.ts
@@ -261,7 +261,7 @@ export class LGraph
    node: LGraphNode
  ): void

-  // @ts-ignore TODO: Fix after migration to frontend tsconfig rules
+  // @ts-expect-error - Private property type needs fixing
  private _input_nodes?: LGraphNode[]

  /**
@@ -440,7 +440,7 @@ export class LGraph
      on_frame()
    } else {
      // execute every 'interval' ms
-      // @ts-ignore TODO: Fix after migration to frontend tsconfig rules
+      // @ts-expect-error - Timer ID type mismatch needs fixing
      this.execution_timer_id = setInterval(() => {
        // execute
        this.onBeforeStep?.()
@@ -780,7 +780,7 @@ export class LGraph
        node[eventname]()
      } else if (params && params.constructor === Array) {
        // @ts-expect-error deprecated
-        node[eventname].apply(node, params)
+        node[eventname](...params)
      } else {
        // @ts-expect-error deprecated
        node[eventname](params)
@@ -807,6 +807,7 @@ export class LGraph
    if (!list_of_graphcanvas) return

    for (const c of list_of_graphcanvas) {
+      // eslint-disable-next-line prefer-spread
      c[action]?.apply(c, params)
    }
  }
@@ -1182,7 +1183,7 @@ export class LGraph
  triggerInput(name: string, value: any): void {
    const nodes = this.findNodesByTitle(name)
    for (const node of nodes) {
-      // @ts-expect-error
+      // @ts-expect-error - onTrigger method may not exist on all node types
      node.onTrigger(value)
    }
  }
@@ -1191,7 +1192,7 @@ export class LGraph
  setCallback(name: string, func: any): void {
    const nodes = this.findNodesByTitle(name)
    for (const node of nodes) {
-      // @ts-expect-error
+      // @ts-expect-error - setTrigger method may not exist on all node types
      node.setTrigger(func)
    }
  }
@@ -2017,15 +2018,13 @@ export class LGraph
  }

  load(url: string | Blob | URL | File, callback: () => void) {
-    const that = this
-
    // from file
    if (url instanceof Blob || url instanceof File) {
      const reader = new FileReader()
-      reader.addEventListener('load', function (event) {
+      reader.addEventListener('load', (event) => {
        const result = stringOrEmpty(event.target?.result)
        const data = JSON.parse(result)
-        that.configure(data)
+        this.configure(data)
        callback?.()
      })

@@ -2037,13 +2036,13 @@ export class LGraph
    const req = new XMLHttpRequest()
    req.open('GET', url, true)
    req.send(null)
-    req.addEventListener('load', function () {
+    req.addEventListener('load', () => {
      if (req.status !== 200) {
        console.error('Error loading graph:', req.status, req.response)
        return
      }
      const data = JSON.parse(req.response)
-      that.configure(data)
+      this.configure(data)
      callback?.()
    })
    req.addEventListener('error', (err) => {
--- a/src/lib/litegraph/src/LGraphCanvas.ts
+++ b/src/lib/litegraph/src/LGraphCanvas.ts
@@ -1,4 +1,3 @@
-// @ts-nocheck TODO: Fix after migration to frontend tsconfig rules
 import { LinkConnector } from '@/lib/litegraph/src/canvas/LinkConnector'

 import { CanvasPointer } from './CanvasPointer'
@@ -886,7 +885,9 @@ export class LGraphCanvas
  }

  static onGroupAdd(
+    // @ts-expect-error - unused parameter
    info: unknown,
+    // @ts-expect-error - unused parameter
    entry: unknown,
    mouse_event: MouseEvent
  ): void {
@@ -934,7 +935,9 @@ export class LGraphCanvas
  }

  static onNodeAlign(
+    // @ts-expect-error - unused parameter
    value: IContextMenuValue,
+    // @ts-expect-error - unused parameter
    options: IContextMenuOptions,
    event: MouseEvent,
    prev_menu: ContextMenu<string>,
@@ -957,7 +960,9 @@ export class LGraphCanvas
  }

  static onGroupAlign(
+    // @ts-expect-error - unused parameter
    value: IContextMenuValue,
+    // @ts-expect-error - unused parameter
    options: IContextMenuOptions,
    event: MouseEvent,
    prev_menu: ContextMenu<string>
@@ -978,7 +983,9 @@ export class LGraphCanvas
  }

  static createDistributeMenu(
+    // @ts-expect-error - unused parameter
    value: IContextMenuValue,
+    // @ts-expect-error - unused parameter
    options: IContextMenuOptions,
    event: MouseEvent,
    prev_menu: ContextMenu<string>
@@ -1000,7 +1007,9 @@ export class LGraphCanvas
  }

  static onMenuAdd(
+    // @ts-expect-error - unused parameter
    value: unknown,
+    // @ts-expect-error - unused parameter
    options: unknown,
    e: MouseEvent,
    prev_menu?: ContextMenu<string>,
@@ -1058,7 +1067,14 @@ export class LGraphCanvas
            value: category_path,
            content: name,
            has_submenu: true,
-            callback: function (value, event, mouseEvent, contextMenu) {
+            callback: function (
+              value,
+              // @ts-expect-error - unused parameter
+              event,
+              // @ts-expect-error - unused parameter
+              mouseEvent,
+              contextMenu
+            ) {
              inner_onMenuAdded(value.value, contextMenu)
            }
          })
@@ -1077,7 +1093,14 @@ export class LGraphCanvas
          value: node.type,
          content: node.title,
          has_submenu: false,
-          callback: function (value, event, mouseEvent, contextMenu) {
+          callback: function (
+            value,
+            // @ts-expect-error - unused parameter
+            event,
+            // @ts-expect-error - unused parameter
+            mouseEvent,
+            contextMenu
+          ) {
            if (!canvas.graph) throw new NullGraphError()

            const first_event = contextMenu.getFirstEvent()
@@ -1102,10 +1125,10 @@ export class LGraphCanvas
        entries.push(entry)
      }

-      // @ts-expect-error Remove param ref_window - unused
      new LiteGraph.ContextMenu(
        entries,
        { event: e, parentMenu: prev_menu },
+        // @ts-expect-error - extra parameter
        ref_window
      )
    }
@@ -1116,6 +1139,7 @@ export class LGraphCanvas

  /** @param _options Parameter is never used */
  static showMenuNodeOptionalOutputs(
+    // @ts-expect-error - unused parameter
    v: unknown,
    /** Unused - immediately overwritten */
    _options: INodeOutputSlot[],
@@ -1200,6 +1224,7 @@ export class LGraphCanvas
  /** @param value Parameter is never used */
  static onShowMenuNodeProperties(
    value: NodeProperty | undefined,
+    // @ts-expect-error - unused parameter
    options: unknown,
    e: MouseEvent,
    prev_menu: ContextMenu<string>,
@@ -1264,9 +1289,13 @@ export class LGraphCanvas
  }

  static onMenuResizeNode(
+    // @ts-expect-error - unused parameter
    value: IContextMenuValue,
+    // @ts-expect-error - unused parameter
    options: IContextMenuOptions,
+    // @ts-expect-error - unused parameter
    e: MouseEvent,
+    // @ts-expect-error - unused parameter
    menu: ContextMenu,
    node: LGraphNode
  ): void {
@@ -1294,8 +1323,10 @@ export class LGraphCanvas
  // TODO refactor :: this is used fot title but not for properties!
  static onShowPropertyEditor(
    item: { property: keyof LGraphNode; type: string },
+    // @ts-expect-error - unused parameter
    options: IContextMenuOptions<string>,
    e: MouseEvent,
+    // @ts-expect-error - unused parameter
    menu: ContextMenu<string>,
    node: LGraphNode
  ): void {
@@ -1370,6 +1401,7 @@ export class LGraphCanvas
    dialog.addEventListener('mouseleave', function () {
      if (LiteGraph.dialog_close_on_mouse_leave) {
        if (!dialog.is_modified && LiteGraph.dialog_close_on_mouse_leave) {
+          // @ts-expect-error - setTimeout type
          dialogCloseTimer = setTimeout(
            dialog.close,
            LiteGraph.dialog_close_on_mouse_leave_delay
@@ -1424,9 +1456,13 @@ export class LGraphCanvas
  }

  static onMenuNodeCollapse(
+    // @ts-expect-error - unused parameter
    value: IContextMenuValue,
+    // @ts-expect-error - unused parameter
    options: IContextMenuOptions,
+    // @ts-expect-error - unused parameter
    e: MouseEvent,
+    // @ts-expect-error - unused parameter
    menu: ContextMenu,
    node: LGraphNode
  ): void {
@@ -1454,9 +1490,13 @@ export class LGraphCanvas
  }

  static onMenuToggleAdvanced(
+    // @ts-expect-error - unused parameter
    value: IContextMenuValue,
+    // @ts-expect-error - unused parameter
    options: IContextMenuOptions,
+    // @ts-expect-error - unused parameter
    e: MouseEvent,
+    // @ts-expect-error - unused parameter
    menu: ContextMenu,
    node: LGraphNode
  ): void {
@@ -1482,7 +1522,9 @@ export class LGraphCanvas
  }

  static onMenuNodeMode(
+    // @ts-expect-error - unused parameter
    value: IContextMenuValue,
+    // @ts-expect-error - unused parameter
    options: IContextMenuOptions,
    e: MouseEvent,
    menu: ContextMenu,
@@ -1527,6 +1569,7 @@ export class LGraphCanvas
  /** @param value Parameter is never used */
  static onMenuNodeColors(
    value: IContextMenuValue<string | null>,
+    // @ts-expect-error - unused parameter
    options: IContextMenuOptions,
    e: MouseEvent,
    menu: ContextMenu<string | null>,
@@ -1588,7 +1631,9 @@ export class LGraphCanvas
  }

  static onMenuNodeShapes(
+    // @ts-expect-error - unused parameter
    value: IContextMenuValue<(typeof LiteGraph.VALID_SHAPES)[number]>,
+    // @ts-expect-error - unused parameter
    options: IContextMenuOptions<(typeof LiteGraph.VALID_SHAPES)[number]>,
    e: MouseEvent,
    menu?: ContextMenu<(typeof LiteGraph.VALID_SHAPES)[number]>,
@@ -1640,9 +1685,13 @@ export class LGraphCanvas
  }

  static onMenuNodeClone(
+    // @ts-expect-error - unused parameter
    value: IContextMenuValue,
+    // @ts-expect-error - unused parameter
    options: IContextMenuOptions,
+    // @ts-expect-error - unused parameter
    e: MouseEvent,
+    // @ts-expect-error - unused parameter
    menu: ContextMenu,
    node: LGraphNode
  ): void {
@@ -3353,10 +3402,10 @@ export class LGraphCanvas
      if (!this.linkConnector.isConnecting) {
        this.dirty_canvas = true

-        // @ts-expect-error Unused param
        this.node_over?.onMouseUp?.(
          e,
          [x - this.node_over.pos[0], y - this.node_over.pos[1]],
+          // @ts-expect-error - extra parameter
          this
        )
        this.node_capturing_input?.onMouseUp?.(e, [
@@ -3405,7 +3454,7 @@ export class LGraphCanvas
    if (!this.graph || !this.allow_dragcanvas) return

    // TODO: Mouse wheel zoom rewrite
-    // @ts-expect-error
+    // @ts-expect-error wheelDeltaY is non-standard property on WheelEvent
    const delta = e.wheelDeltaY ?? e.detail * -60

    this.adjustMouseEvent(e)
@@ -3470,7 +3519,7 @@ export class LGraphCanvas
    if (!graph) return

    let block_default = false
-    // @ts-expect-error
+    // @ts-expect-error EventTarget.localName is not in standard types
    if (e.target.localName == 'input') return

    if (e.type == 'keydown') {
@@ -3508,7 +3557,7 @@ export class LGraphCanvas
        this.pasteFromClipboard({ connectInputs: e.shiftKey })
      } else if (e.key === 'Delete' || e.key === 'Backspace') {
        // delete or backspace
-        // @ts-expect-error
+        // @ts-expect-error EventTarget.localName is not in standard types
        if (e.target.localName != 'input' && e.target.localName != 'textarea') {
          if (this.selectedItems.size === 0) {
            this.#noItemsSelected()
@@ -4372,9 +4421,9 @@ export class LGraphCanvas

    const { ctx, canvas, graph, linkConnector } = this

-    // @ts-expect-error
+    // @ts-expect-error start2D method not in standard CanvasRenderingContext2D
    if (ctx.start2D && !this.viewport) {
-      // @ts-expect-error
+      // @ts-expect-error start2D method not in standard CanvasRenderingContext2D
      ctx.start2D()
      ctx.restore()
      ctx.setTransform(1, 0, 0, 1, 0, 0)
@@ -4824,9 +4873,8 @@ export class LGraphCanvas
          this._bg_img = new Image()
          this._bg_img.name = this.background_image
          this._bg_img.src = this.background_image
-          const that = this
-          this._bg_img.addEventListener('load', function () {
-            that.draw(true, true)
+          this._bg_img.addEventListener('load', () => {
+            this.draw(true, true)
          })
        }

@@ -6037,7 +6085,11 @@ export class LGraphCanvas
  /**
   * draws every group area in the background
   */
-  drawGroups(canvas: HTMLCanvasElement, ctx: CanvasRenderingContext2D): void {
+  drawGroups(
+    // @ts-expect-error - unused parameter
+    canvas: HTMLCanvasElement,
+    ctx: CanvasRenderingContext2D
+  ): void {
    if (!this.graph) return

    const groups = this.graph._groups
@@ -6129,6 +6181,7 @@ export class LGraphCanvas
    function inner_clicked(
      this: LGraphCanvas,
      v: string,
+      // @ts-expect-error - unused parameter
      options: unknown,
      e: MouseEvent
    ) {
@@ -6410,6 +6463,7 @@ export class LGraphCanvas
      optPass || {}
    )
    const dirty = () => this.#dirty()
+    // eslint-disable-next-line @typescript-eslint/no-this-alias
    const that = this
    const { graph } = this
    const { afterRerouteId } = opts
@@ -6572,16 +6626,18 @@ export class LGraphCanvas
        }
        case 'Search':
          if (isFrom) {
-            // @ts-expect-error Subgraph
            opts.showSearchBox(e, {
+              // @ts-expect-error - Subgraph types
              node_from: opts.nodeFrom,
+              // @ts-expect-error - Subgraph types
              slot_from: slotX,
              type_filter_in: fromSlotType
            })
          } else {
-            // @ts-expect-error Subgraph
            opts.showSearchBox(e, {
+              // @ts-expect-error - Subgraph types
              node_to: opts.nodeTo,
+              // @ts-expect-error - Subgraph types
              slot_from: slotX,
              type_filter_out: fromSlotType
            })
@@ -6609,6 +6665,7 @@ export class LGraphCanvas
    event: CanvasPointerEvent,
    multiline?: boolean
  ): HTMLDivElement {
+    // eslint-disable-next-line @typescript-eslint/no-this-alias
    const that = this
    title = title || ''

@@ -6645,6 +6702,7 @@ export class LGraphCanvas
      if (prevent_timeout) return
      if (LiteGraph.dialog_close_on_mouse_leave) {
        if (!dialog.is_modified && LiteGraph.dialog_close_on_mouse_leave) {
+          // @ts-expect-error - setTimeout type
          dialogCloseTimer = setTimeout(
            dialog.close,
            LiteGraph.dialog_close_on_mouse_leave_delay
@@ -6765,7 +6823,7 @@ export class LGraphCanvas
      do_type_filter: LiteGraph.search_filter_enabled,

      // these are default: pass to set initially set values
-      // @ts-expect-error
+      // @ts-expect-error Property missing from interface definition
      type_filter_in: false,

      type_filter_out: false,
@@ -6778,6 +6836,7 @@ export class LGraphCanvas
    Object.assign(options, searchOptions)

    // console.log(options);
+    // eslint-disable-next-line @typescript-eslint/no-this-alias
    const that = this
    const graphcanvas = LGraphCanvas.active_canvas
    const { canvas } = graphcanvas
@@ -6844,6 +6903,7 @@ export class LGraphCanvas

        const hideDelay = options.hide_on_mouse_leave
        const delay = typeof hideDelay === 'number' ? hideDelay : 500
+        // @ts-expect-error - setTimeout type
        timeout_close = setTimeout(dialog.close, delay)
      })
      // if filtering, check focus changed to comboboxes and prevent closing
@@ -6913,6 +6973,7 @@ export class LGraphCanvas
          if (timeout) {
            clearInterval(timeout)
          }
+          // @ts-expect-error - setTimeout type
          timeout = setTimeout(refreshHelper, 10)
          return
        }
@@ -6941,7 +7002,7 @@ export class LGraphCanvas
          opt.innerHTML = aSlots[iK]
          selIn.append(opt)
          if (
-            // @ts-expect-error
+            // @ts-expect-error Property missing from interface definition
            options.type_filter_in !== false &&
            String(options.type_filter_in).toLowerCase() ==
              String(aSlots[iK]).toLowerCase()
@@ -7029,8 +7090,9 @@ export class LGraphCanvas
                iS = options.slot_from.name
                  ? options.node_from.findOutputSlot(options.slot_from.name)
                  : -1
-                // @ts-expect-error change interface check
+                // @ts-expect-error - slot_index property
                if (iS == -1 && options.slot_from.slot_index !== undefined)
+                  // @ts-expect-error - slot_index property
                  iS = options.slot_from.slot_index
                break
              case 'number':
@@ -7073,8 +7135,9 @@ export class LGraphCanvas
                iS = options.slot_from.name
                  ? options.node_to.findInputSlot(options.slot_from.name)
                  : -1
-                // @ts-expect-error change interface check
+                // @ts-expect-error - slot_index property
                if (iS == -1 && options.slot_from.slot_index !== undefined)
+                  // @ts-expect-error - slot_index property
                  iS = options.slot_from.slot_index
                break
              case 'number':
@@ -7175,7 +7238,7 @@ export class LGraphCanvas
          (sIn.value || sOut.value)
        ) {
          // FIXME: Undeclared variable again
-          // @ts-expect-error
+          // @ts-expect-error Variable declared without type annotation
          filtered_extra = []
          for (const i in LiteGraph.registered_node_types) {
            if (
@@ -7184,11 +7247,11 @@ export class LGraphCanvas
                outTypeOverride: sOut && sOut.value ? '*' : false
              })
            ) {
-              // @ts-expect-error
+              // @ts-expect-error Variable declared without type annotation
              filtered_extra.push(i)
            }
          }
-          // @ts-expect-error
+          // @ts-expect-error Variable declared without type annotation
          for (const extraItem of filtered_extra) {
            addResult(extraItem, 'generic_type')
            if (
@@ -7205,14 +7268,14 @@ export class LGraphCanvas
          helper.childNodes.length == 0 &&
          options.show_general_if_none_on_typefilter
        ) {
-          // @ts-expect-error
+          // @ts-expect-error Variable declared without type annotation
          filtered_extra = []
          for (const i in LiteGraph.registered_node_types) {
            if (inner_test_filter(i, { skipFilter: true }))
-              // @ts-expect-error
+              // @ts-expect-error Variable declared without type annotation
              filtered_extra.push(i)
          }
-          // @ts-expect-error
+          // @ts-expect-error Variable declared without type annotation
          for (const extraItem of filtered_extra) {
            addResult(extraItem, 'not_in_filter')
            if (
@@ -7358,7 +7421,7 @@ export class LGraphCanvas
      input = dialog.querySelector('input')
      input?.addEventListener('click', function () {
        dialog.modified()
-        // @ts-expect-error
+        // @ts-expect-error setValue function signature not strictly typed
        setValue(!!input.checked)
      })
    } else {
@@ -7376,7 +7439,7 @@ export class LGraphCanvas
          v = JSON.stringify(v)
        }

-        // @ts-expect-error
+        // @ts-expect-error HTMLInputElement.value expects string but v can be other types
        input.value = v
        input.addEventListener('keydown', function (e) {
          if (e.key == 'Escape') {
@@ -7513,6 +7576,7 @@ export class LGraphCanvas
      if (prevent_timeout) return

      if (!dialog.is_modified && LiteGraph.dialog_close_on_mouse_leave) {
+        // @ts-expect-error - setTimeout type
        dialogCloseTimer = setTimeout(
          dialog.close,
          LiteGraph.dialog_close_on_mouse_leave_delay
@@ -7730,7 +7794,7 @@ export class LGraphCanvas
              className: 'dark',
              callback: inner_clicked
            },
-            // @ts-expect-error
+            // @ts-expect-error ref_window parameter unused in ContextMenu constructor
            ref_window
          )
        })
@@ -7780,8 +7844,8 @@ export class LGraphCanvas
    const inner_refresh = () => {
      // clear
      panel.content.innerHTML = ''
-      // @ts-expect-error ctor props
      panel.addHTML(
+        // @ts-expect-error - desc property
        `<span class='node_type'>${node.type}</span><span class='node_desc'>${node.constructor.desc || ''}</span><span class='separator'></span>`
      )

@@ -8027,9 +8091,13 @@ export class LGraphCanvas
        {
          content: 'Properties Panel',
          callback: function (
+            // @ts-expect-error - unused parameter
            item: any,
+            // @ts-expect-error - unused parameter
            options: any,
+            // @ts-expect-error - unused parameter
            e: any,
+            // @ts-expect-error - unused parameter
            menu: any,
            node: LGraphNode
          ) {
--- a/src/lib/litegraph/src/LGraphNode.ts
+++ b/src/lib/litegraph/src/LGraphNode.ts
@@ -731,7 +731,7 @@ export class LGraphNode

    // Assign onMouseDown implementation
    this.onMouseDown = (
-      // @ts-ignore TODO: Fix after migration to frontend tsconfig rules
+      // @ts-expect-error - CanvasPointerEvent type needs fixing
      e: CanvasPointerEvent,
      pos: Point,
      canvas: LGraphCanvas
@@ -887,7 +887,7 @@ export class LGraphNode
    if (this.inputs)
      o.inputs = this.inputs.map((input) => inputAsSerialisable(input))
    if (this.outputs)
-      // @ts-ignore TODO: Fix after migration to frontend tsconfig rules - widget type mismatch
+      // @ts-expect-error - Output serialization type mismatch
      o.outputs = this.outputs.map((output) => outputAsSerialisable(output))

    if (this.title && this.title != this.constructor.title) o.title = this.title
@@ -1871,7 +1871,7 @@ export class LGraphNode
    }

    const w: IBaseWidget & { type: Type } = {
-      // @ts-expect-error
+      // @ts-expect-error - Type casting for widget type property
      type: type.toLowerCase(),
      name: name,
      value: value,
--- a/src/lib/litegraph/src/LiteGraphGlobal.ts
+++ b/src/lib/litegraph/src/LiteGraphGlobal.ts
@@ -779,6 +779,7 @@ export class LiteGraphGlobal {
        oDOM.addEventListener(sMethod + sEvent, fCall, capture)
      }
      // only pointerevents
+      // falls through
      case 'leave':
      case 'cancel':
      case 'gotpointercapture':
@@ -789,6 +790,7 @@ export class LiteGraphGlobal {
        }
      }
      // not "pointer" || "mouse"
+      // falls through
      default:
        return oDOM.addEventListener(sEvent, fCall, capture)
    }
@@ -829,6 +831,7 @@ export class LiteGraphGlobal {
        }
      }
      // only pointerevents
+      // falls through
      case 'leave':
      case 'cancel':
      case 'gotpointercapture':
@@ -843,6 +846,7 @@ export class LiteGraphGlobal {
        }
      }
      // not "pointer" || "mouse"
+      // falls through
      default:
        return oDOM.removeEventListener(sEvent, fCall, capture)
    }
@@ -949,6 +953,7 @@ export class LiteGraphGlobal {
  extendClass(target: any, origin: any): void {
    for (const i in origin) {
      // copy class properties
+      // eslint-disable-next-line no-prototype-builtins
      if (target.hasOwnProperty(i)) continue
      target[i] = origin[i]
    }
@@ -957,9 +962,11 @@ export class LiteGraphGlobal {
      // copy prototype properties
      for (const i in origin.prototype) {
        // only enumerable
+        // eslint-disable-next-line no-prototype-builtins
        if (!origin.prototype.hasOwnProperty(i)) continue

        // avoid overwriting existing ones
+        // eslint-disable-next-line no-prototype-builtins
        if (target.prototype.hasOwnProperty(i)) continue

        // copy getters
--- a/src/lib/litegraph/src/canvas/FloatingRenderLink.ts
+++ b/src/lib/litegraph/src/canvas/FloatingRenderLink.ts
@@ -192,7 +192,7 @@ export class FloatingRenderLink implements RenderLink {
  }

  connectToRerouteInput(
-    // @ts-ignore TODO: Fix after migration to frontend tsconfig rules
+    // @ts-expect-error - Reroute type needs fixing
    reroute: Reroute,
    { node: inputNode, input }: { node: LGraphNode; input: INodeInputSlot },
    events: CustomEventTarget<LinkConnectorEventMap>
@@ -209,7 +209,7 @@ export class FloatingRenderLink implements RenderLink {
  }

  connectToRerouteOutput(
-    // @ts-ignore TODO: Fix after migration to frontend tsconfig rules
+    // @ts-expect-error - Reroute type needs fixing
    reroute: Reroute,
    outputNode: LGraphNode,
    output: INodeOutputSlot,
--- a/src/lib/litegraph/src/infrastructure/Rectangle.ts
+++ b/src/lib/litegraph/src/infrastructure/Rectangle.ts
@@ -65,8 +65,11 @@ export class Rectangle extends Float64Array {
      : new Rectangle(rect[0], rect[1], rect[2], rect[3])
  }

-  // @ts-ignore TODO: Fix after migration to frontend tsconfig rules - ES2022/ES2023 Float64Array type mismatch
-  override subarray(begin: number = 0, end?: number): Float64Array {
+  override subarray(
+    begin: number = 0,
+    end?: number
+    // @ts-expect-error TypeScript lib typing issue - Float64Array is not generic
+  ): Float64Array<ArrayBuffer> {
    const byteOffset = begin << 3
    const length = end === undefined ? end : end - begin
    return new Float64Array(this.buffer, byteOffset, length)
--- a/src/lib/litegraph/src/litegraph.ts
+++ b/src/lib/litegraph/src/litegraph.ts
@@ -58,6 +58,7 @@ export interface LinkReleaseContextExtended {
  links: ConnectingLink[]
 }

+// eslint-disable-next-line @typescript-eslint/no-empty-object-type
 export interface LiteGraphCanvasEvent extends CustomEvent<CanvasEventDetail> {}

 export interface LGraphNodeConstructor<T extends LGraphNode = LGraphNode> {
--- a/src/lib/litegraph/src/widgets/BaseWidget.ts
+++ b/src/lib/litegraph/src/widgets/BaseWidget.ts
@@ -119,7 +119,6 @@ export abstract class BaseWidget<TWidget extends IBaseWidget = IBaseWidget>

    // `node` has no setter - Object.assign will throw.
    // TODO: Resolve this workaround. Ref: https://github.com/Comfy-Org/litegraph.js/issues/1022
-    // eslint-disable-next-line unused-imports/no-unused-vars
    const {
      node: _,
      // @ts-expect-error Prevent naming conflicts with custom nodes.
@@ -339,7 +338,7 @@ export abstract class BaseWidget<TWidget extends IBaseWidget = IBaseWidget>
   * Correctly and safely typing this is currently not possible (practical?) in TypeScript 5.8.
   */
  createCopyForNode(node: LGraphNode): this {
-    // @ts-expect-error
+    // @ts-expect-error - Constructor type casting for widget cloning
    const cloned: this = new (this.constructor as typeof this)(this, node)
    cloned.value = this.value
    return cloned
--- a/src/lib/litegraph/src/widgets/NumberWidget.ts
+++ b/src/lib/litegraph/src/widgets/NumberWidget.ts
@@ -73,7 +73,9 @@ export class NumberWidget
          // Solve the equation if possible
          try {
            v = eval(v)
-          } catch {}
+          } catch {
+            // Ignore eval errors
+          }
        }
        const newValue = Number(v)
        if (!isNaN(newValue)) {