fix(api): resolve TypeScript errors in webviewWindow.ts

packages/api/package.json

@@ -55,6 +55,7 @@
     "eslint-plugin-security": "3.0.1",
     "fast-glob": "3.3.3",
     "globals": "^16.2.0",
+    "prettier": "^3.5.3",
     "rollup": "4.50.1",
     "tslib": "^2.8.1",
     "typescript": "^5.8.3",

packages/api/src/webviewWindow.ts

@@ -15,30 +15,32 @@ import type { EventName, EventCallback, UnlistenFn } from './event'
 import { invoke } from './core'
 import type { Color, DragDropEvent } from './webview'
 
+// Internal-only type to avoid @ts-expect-error everywhere
+interface InternalWebviewOptions
+  extends Omit<WebviewOptions, 'x' | 'y' | 'width' | 'height'> {
+  skip?: boolean
+}
+
 /**
  * Get an instance of `Webview` for the current webview window.
  *
  * @since 2.0.0
  */
 function getCurrentWebviewWindow(): WebviewWindow {
   const webview = getCurrentWebview()
-  // @ts-expect-error `skip` is not defined in the public API but it is handled by the constructor
-  return new WebviewWindow(webview.label, { skip: true })
+  return new WebviewWindow(webview.label, { skip: true } as any)
 }
 
 /**
- * Gets a list of instances of `Webview` for all available webview windows.
- *
- * @since 2.0.0
+ * Gets a list of instances of `Webview` for all available webviews.
  */
 async function getAllWebviewWindows(): Promise<WebviewWindow[]> {
   return invoke<string[]>('plugin:window|get_all_windows').then((windows) =>
     windows.map(
       (w) =>
         new WebviewWindow(w, {
-          // @ts-expect-error `skip` is not defined in the public API but it is handled by the constructor
           skip: true
-        })
+        } as any)
     )
   )
 }
@@ -52,7 +54,6 @@ class WebviewWindow {
   /** Local event listeners. */
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
   listeners: Record<string, Array<EventCallback<any>>>
-
   /**
    * Creates a new {@link Window} hosting a {@link Webview}.
    * @example
@@ -78,14 +79,16 @@ class WebviewWindow {
       & WindowOptions = {}
   ) {
     this.label = label
-    // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment
     this.listeners = Object.create(null)
 
-    // @ts-expect-error `skip` is not a public API so it is not defined in WebviewOptions
-    if (!options?.skip) {
+    const internalOptions = options as InternalWebviewOptions & {
+      skip?: boolean
+    }
+
+    if (!internalOptions?.skip) {
       invoke('plugin:webview|create_webview_window', {
         options: {
-          ...options,
+          ...internalOptions,
           parent:
             typeof options.parent === 'string'
               ? options.parent
@@ -109,30 +112,27 @@ class WebviewWindow {
    * @param label The webview label.
    * @returns The Webview instance to communicate with the webview or null if the webview doesn't exist.
    */
+
   static async getByLabel(label: string): Promise<WebviewWindow | null> {
     const webview =
       (await getAllWebviewWindows()).find((w) => w.label === label) ?? null
     if (webview) {
-      // @ts-expect-error `skip` is not defined in the public API but it is handled by the constructor
-      return new WebviewWindow(webview.label, { skip: true })
+      return new WebviewWindow(webview.label, { skip: true } as any)
     }
     return null
   }
 
   /**
    * Get an instance of `Webview` for the current webview.
    */
+
   static getCurrent(): WebviewWindow {
     return getCurrentWebviewWindow()
   }
 
-  /**
-   * Gets a list of instances of `Webview` for all available webviews.
-   */
   static async getAll(): Promise<WebviewWindow[]> {
     return getAllWebviewWindows()
   }
-
   /**
    * Listen to an emitted event on this webivew window.
    *
@@ -152,15 +152,16 @@ class WebviewWindow {
    * @returns A promise resolving to a function to unlisten to the event.
    * Note that removing the listener is required if your listener goes out of scope e.g. the component is unmounted.
    */
+
   async listen<T>(
     event: EventName,
     handler: EventCallback<T>
   ): Promise<UnlistenFn> {
     if (this._handleTauriEvent(event, handler)) {
       return () => {
-        // eslint-disable-next-line security/detect-object-injection
         const listeners = this.listeners[event]
-        listeners.splice(listeners.indexOf(handler), 1)
+        const idx = listeners.indexOf(handler)
+        if (idx >= 0) listeners.splice(idx, 1)
       }
     }
     return listen(event, handler, {
@@ -187,15 +188,16 @@ class WebviewWindow {
    * @returns A promise resolving to a function to unlisten to the event.
    * Note that removing the listener is required if your listener goes out of scope e.g. the component is unmounted.
    */
+
   async once<T>(
     event: EventName,
     handler: EventCallback<T>
   ): Promise<UnlistenFn> {
     if (this._handleTauriEvent(event, handler)) {
       return () => {
-        // eslint-disable-next-line security/detect-object-injection
         const listeners = this.listeners[event]
-        listeners.splice(listeners.indexOf(handler), 1)
+        const idx = listeners.indexOf(handler)
+        if (idx >= 0) listeners.splice(idx, 1)
       }
     }
     return once(event, handler, {
@@ -206,27 +208,20 @@ class WebviewWindow {
   /**
    * Set the window and webview background color.
    *
-   * #### Platform-specific:
-   *
-   * - **Android / iOS:** Unsupported for the window layer.
-   * - **macOS / iOS**: Not implemented for the webview layer.
-   * - **Windows**:
-   *   - alpha channel is ignored for the window layer.
-   *   - On Windows 7, alpha channel is ignored for the webview layer.
-   *   - On Windows 8 and newer, if alpha channel is not `0`, it will be ignored.
-   *
-   * @returns A promise indicating the success or failure of the operation.
-   *
    * @since 2.1.0
    */
   async setBackgroundColor(color: Color): Promise<void> {
-    return invoke('plugin:window|set_background_color', { color }).then(() => {
-      return invoke('plugin:webview|set_webview_background_color', { color })
-    })
+    try {
+      await invoke('plugin:window|set_background_color', { color })
+      await invoke('plugin:webview|set_webview_background_color', { color })
+    } catch (err) {
+      const error = new Error('Failed to set background color')
+      ;(error as any).cause = err
+      throw error
+    }
   }
 }
 
-// Order matters, we use window APIs by default
 applyMixins(WebviewWindow, [Window, Webview])
 
 /** Extends a base class by other specified classes, without overriding existing properties */
@@ -243,12 +238,12 @@ function applyMixins(
         typeof baseClass.prototype === 'object'
         && baseClass.prototype
         && name in baseClass.prototype
-      )
+      ) {
         return
+      }
       Object.defineProperty(
         baseClass.prototype,
         name,
-        // eslint-disable-next-line
         Object.getOwnPropertyDescriptor(extendedClass.prototype, name)
           ?? Object.create(null)
       )