@openfin/core 30.73.22 → 30.73.24

package/src/api/platform/layout/Factory.js

@@ -20,12 +20,14 @@ const base_1 = require("../../base");
 const splitter_controller_1 = require("./controllers/splitter-controller");
 const view_overlay_1 = require("./utils/view-overlay");
 /**
+ * @PORTED
  * InitLayoutOptions interface
  * @typedef { object } InitLayoutOptions
  * @property { string } [containerId] The id attribute of the container where the window's Layout should be initialized.  If not provided
  * then an element with id `layout-container` is used. We recommend using a div element.
  */
 /**
+ * @PORTED
  * PresetLayoutOptions interface
  * @typedef { object } PresetLayoutOptions
  * @property { LayoutPresetTypes } presetType Which preset layout arrangement to use.
@@ -40,6 +42,7 @@ const view_overlay_1 = require("./utils/view-overlay");
  * @property { LayoutSettings } settings Configuration for certain Layout behaviors. See the LayoutSettings interface.
  */
 /**
+ * @PORTED
  * LayoutItem Interface
  * @typedef { object } LayoutItem Represents the arrangement of Views within a Platform window's Layout.  We do not recommend trying
  * to build Layouts or LayoutItems by hand and instead use calls such as {@link Platform#getSnapshot getSnapshot} or our
@@ -51,6 +54,7 @@ const view_overlay_1 = require("./utils/view-overlay");
  * options of a given component.
  */
 /**
+ * @PORTED
  * LayoutSettings Interface
  * @typedef { object } LayoutSettings Represents a potential ways to customize behavior of your Layout
  * @property { boolean } [constrainDragToHeaders=false] Limits the area to which tabs can be dragged.

package/src/api/platform/layout/Instance.d.ts

@@ -1,9 +1,9 @@
-import * as OpenFin from '../../../OpenFin';
+import type * as OpenFin from '../../../OpenFin';
 import Transport from '../../../transport/transport';
 import { Base } from '../../base';
-import Identity = OpenFin.Identity;
-import InitLayoutOptions = OpenFin.InitLayoutOptions;
-import PresetLayoutOptions = OpenFin.PresetLayoutOptions;
+declare type Identity = OpenFin.Identity;
+declare type InitLayoutOptions = OpenFin.InitLayoutOptions;
+declare type PresetLayoutOptions = OpenFin.PresetLayoutOptions;
 /**
  * @lends Platform#Layout
  */
@@ -48,3 +48,4 @@ export declare class Layout extends Base {
      */
     applyPreset: (options: PresetLayoutOptions) => Promise<void>;
 }
+export {};

package/src/api/platform/layout/controllers/tab-drag-controller.d.ts

@@ -1,4 +1,4 @@
-import * as OpenFin from '../../../../OpenFin';
+import type * as OpenFin from '../../../../OpenFin';
 import { ViewOverlay } from '../utils/view-overlay';
 import { View } from '../../../view';
 /**

package/src/api/platform/layout/shapes.d.ts

@@ -1,5 +1,5 @@
-import * as OpenFin from '../../../OpenFin';
-import LayoutPresetType = OpenFin.LayoutPresetType;
+import type * as OpenFin from '../../../OpenFin';
+declare type LayoutPresetType = OpenFin.LayoutPresetType;
 export interface InitLayoutOptions {
     containerId?: string;
 }
@@ -10,3 +10,4 @@ export interface DragPayload {
     'view-config': OpenFin.ViewCreationOptions;
     'view-identity': [string, string, string];
 }
+export {};

package/src/api/platform/layout/utils/view-overlay.d.ts

@@ -1,4 +1,4 @@
-import * as OpenFin from '../../../../OpenFin';
+import type * as OpenFin from '../../../../OpenFin';
 import Transport from '../../../../transport/transport';
 /**
  * Api client allowing an empty electron BrowserView to be rendered

package/src/api/platform/provider.d.ts

@@ -0,0 +1,162 @@
+import { WindowOptionsChangedEvent } from '../events/window';
+import * as OpenFin from '../../OpenFin';
+/**
+ * This class handles Platform actions. It does not need to be used directly by developers.
+ * However, its methods can be overriden by passing an `overrideCallback` to {@link Platform#init Platform.init}
+ * in order to implement custom Platform behavior. (See {@tutorial Platform.init})
+ *
+ * For an overview of Provider customization, see
+ * {@link https://developers.openfin.co/docs/platform-customization#section-customizing-platform-behavior}.
+ */
+export interface PlatformProvider {
+    /**
+     * Handles requests to create a window in the current platform.
+     * @param { WindowOption } payload Window options for the window to be created.
+     * @param { Identity } [identity] If {@link Platform#createWindow Platform.createWindow} was called, the identity of the caller will be here.
+     * If `createWindow` was called as part of applying a snapshot or creating a view without a target window, `identity` will be undefined.
+     */
+    createWindow(options: OpenFin.PlatformWindowCreationOptions, identity?: OpenFin.Identity): Promise<OpenFin.Window>;
+    /**
+     * Gets the current state of windows and their views and returns a snapshot object containing that info.
+     * @param { undefined } payload Undefined unless you've defined a custom `getSnapshot` protocol.
+     * @param { Identity } identity Identity of the entity that called {@link Platform#getSnapshot Platform.getSnapshot}.
+     * @return { Promise<Snapshot> } Snapshot of current platform state.
+     */
+    getSnapshot(payload: undefined, identity: OpenFin.Identity): Promise<OpenFin.Snapshot>;
+    /**
+     * **NOTE**: Internal use only. It is not recommended to manage the state of individual views.
+     * Gets the current state of a single view and returns an object with the options needed to restore that view as part of a snapshot.
+     * @param { Identity } payload Identity of the view.
+     * @return { Promise<ViewState> }
+     * @internal
+     * @experimental
+     */
+    getViewSnapshot(payload: {
+        viewIdentity: OpenFin.Identity;
+    }): Promise<OpenFin.ViewState>;
+    /**
+     * Called when a snapshot is being applied and some windows in that snapshot would be fully or partially off-screen.
+     * Returns an array of windows with modified positions, such that any off-screen windows are positioned in the top left
+     * corner of the main monitor.
+     * @param { Snapshot } snapshot The snapshot to be applied.
+     * @param { WindowOptions[] } outOfBoundsWindows An array of WindowOptions for any windows that would be off-screen.
+     * @return { Promise<WindowOptions[]> } An array of WindowOptions with their position modified to fit on screen.
+     */
+    positionOutOfBoundsWindows(snapshot: OpenFin.Snapshot, outOfBoundsWindows: OpenFin.WindowCreationOptions[]): Promise<OpenFin.WindowCreationOptions[]>;
+    /**
+     * Handles requests to apply a snapshot to the current Platform.
+     * @param { ApplySnapshotPayload } payload Payload containing the snapshot to be applied, as well as any options.
+     * @param { Identity } [identity] Identity of the entity that called {@link Platform#applySnapshot Platform.applySnapshot}.
+     * Undefined if called internally (e.g. when opening the initial snapshot).
+     * @return { Promise<void> }
+     */
+    applySnapshot(payload: OpenFin.ApplySnapshotPayload, identity?: OpenFin.Identity): Promise<void>;
+    /**
+     * Closes the current Platform and all child windows and views.
+     * @param { undefined } payload Undefined unless you have implemented a custom quite protocol.
+     * @param { Identity } identity Identity of the entity that called {@link Platform#quit Platform.quit}.
+     * @return { Promise<void> }
+     */
+    quit(payload: undefined, identity: OpenFin.Identity): Promise<void>;
+    /**
+     * Closes a view
+     * @param { CloseViewPayload } payload Specifies the `target` view to be closed.
+     * @param { Identity } identity Identity of the entity that called {@link Platform#closeView Platform.closeView}.
+     */
+    closeView(payload: OpenFin.CloseViewPayload, identity?: OpenFin.Identity): Promise<any>;
+    /**
+     * Creates a new view and attaches it to a specified target window.
+     * @param { CreateViewPayload } payload Creation options for the new view.
+     * @param { Identity } identity Identity of the entity that called {@link Platform#createView Platform.createView}.
+     * @return { Promise<void> }
+     */
+    createView(payload: OpenFin.CreateViewPayload, identity: OpenFin.Identity): Promise<OpenFin.View>;
+    /** Handles requests to fetch manifests in the current platform.
+     * @param { FetchManifestPayload } payload Payload containing the manifestUrl to be fetched.
+     * @param { Identity } callerIdentity If {@link Platform#fetchManifest Platform.fetchManifest}
+     * was called, the identity of the caller will be here.
+     * If `fetchManifest` was called internally, `callerIdentity` will be the provider's identity.
+     */
+    fetchManifest(payload: OpenFin.FetchManifestPayload, callerIdentity: OpenFin.Identity): Promise<any>;
+    /**
+     * Replaces a Platform window's layout with a new layout. Any views that were in the old layout but not the new layout will be destroyed.
+     * @param { ReplaceLayoutPayload } payload Contains the `target` window and an `opts` object with a `layout` property to apply.
+     * @param { Identity } [identity] Identity of the entity that called {@link Platform#replaceLayout Platform.replaceLayout}.
+     * Undefined if `replaceLayout` is called internally (e.g. while applying a snapshot).
+     * @return { Promise<void> }
+     */
+    replaceLayout(payload: OpenFin.ReplaceLayoutPayload, identity?: OpenFin.Identity): Promise<void>;
+    replaceView(payload: OpenFin.ReplaceViewPayload, identity?: OpenFin.Identity): Promise<void>;
+    launchIntoPlatform(payload: OpenFin.LaunchIntoPlatformPayload): Promise<void>;
+    /**
+     * Handles requests to set a window's context. `target` may be a window or a view.
+     * If it is a window, that window's `customContext` will be updated.
+     * If it is a view, the `customContext` of that view's current host window will be updated.
+     * @param { SetWindowContextPayload } payload Object containing the requested `context` update,
+     * the `target`'s identity, and the target's `entityType`.
+     * @param { Identity } [identity] Identity of the entity that called {@link Platform#setWindowContext Platform.setWindowContext}.
+     * Undefined if `setWindowContext` is called internally (e.g. while applying a snapshot).
+     * @return { Promise<any> } The new context.
+     */
+    setWindowContext(payload: OpenFin.SetWindowContextPayload, identity?: OpenFin.Identity): Promise<any>;
+    /**
+     * Handles requests to get a window's context. `target` may be a window or a view.
+     * If it is a window, that window's `customContext` will be returned.
+     * If it is a view, the `customContext` of that view's current host window will be returned.
+     * @param { GetWindowContextPayload } payload Object containing the requested `context` update,
+     * the `target`'s identity, and the target's `entityType`.
+     * @param { Identity } [identity] Identity of the entity that called {@link Platform#getWindowContext Platform.getWindowContext}.
+     * Undefined when `getWindowContext` is called internally
+     * (e.g. when getting a window's context for the purpose of raising a "host-context-changed" event on a reparented view).
+     * @return { Promise<any> } The new context.
+     */
+    getWindowContext(payload: OpenFin.GetWindowContextPayload, identity?: OpenFin.Identity): Promise<any>;
+    /**
+     * Called when a window's `customContext` is updated. Responsible for raising the `host-context-updated` event on that window's child views.
+     * @param { WindowOptionsChangedEvent<'window', 'options-changed'> } payload The event payload for the window whose context has changed.
+     * The new context will be contained as `payload.diff.customContext.newVal`.
+     * @return { Promise<HostContextChangedPayload> } The event that it raised.
+     */
+    onWindowContextUpdated(payload: WindowOptionsChangedEvent): Promise<OpenFin.HostContextChangedPayload | undefined>;
+    /**
+     * Closes a Window.
+     * By default it will fire any before unload handler set by a View in the Window.
+     * This can be disabled by setting skipBeforeUnload in the options object of the payload.
+     * This method is called by {@link Platform#closeWindow Platform.closeWindow}.
+     * @param {CloseWindowPayload} payload Object that contains the Window Identity and related options.
+     * @param {Identity} callerIdentity
+     * @returns {Promise<void>}
+     */
+    closeWindow(payload: OpenFin.CloseWindowPayload, callerIdentity: OpenFin.Identity): Promise<void>;
+    /**
+     * Gets all the Views attached to a Window that should close along with it. This is meant to be overridable
+     * in the case where you want to return other Views that may not be attached to the Window that is closing.
+     * @param winId Identity of the Window that is closing
+     * @returns { Promise<View> }
+     */
+    getViewsForWindowClose(windowId: OpenFin.Identity): Promise<OpenFin.View[]>;
+    /**
+     * It takes in an array of Views and returns an object specifying which of them are trying to prevent an unload and which are not.
+     * @param {View[]} views Array of Views
+     * @returns { Promise<ViewStatuses> }
+     */
+    checkViewsForPreventUnload(views: OpenFin.View[]): Promise<OpenFin.ViewStatuses>;
+    /**
+     * Handle the decision of whether a Window or specific View should close when trying to prevent an unload. This is meant to be overridden.
+     * Called in {@link PlatformProvider#closeWindow PlatformProvider.closeWindow}.
+     * Normally you would use this method to show a dialog indicating that there are Views that are trying to prevent an unload.
+     * By default it will always return all Views passed into it as meaning to close.
+     * @param {ViewsPreventingUnloadPayload} payload
+     * @tutorial PlatformProvider.getUserDecisionForBeforeUnload
+     * @returns {Promise<BeforeUnloadUserDecision>}
+     */
+    getUserDecisionForBeforeUnload(payload: OpenFin.ViewsPreventingUnloadPayload): Promise<OpenFin.BeforeUnloadUserDecision>;
+    /**
+     * Handles the closing of a Window and/or its Views. Called in {@link PlatformProvider#closeWindow PlatformProvider.closeWindow}.
+     * The return of {@link PlatformProvider#getUserDecisionForBeforeUnload PlatformProvider.getUserDecisionForBeforeUnload} is passed into this method.
+     * @param {Identity} winId Identity of the Window
+     * @param {BeforeUnloadUserDecision} userDecision Decision object
+     * @returns {Promise<void>}
+     */
+    handleViewsAndWindowClose(windowId: OpenFin.Identity, userDecision: OpenFin.BeforeUnloadUserDecision): Promise<void>;
+}

package/src/api/platform/provider.js

@@ -0,0 +1,2 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });

package/src/api/snapshot-source/Factory.d.ts

@@ -1,7 +1,8 @@
-import * as OpenFin from '../../OpenFin';
+import type * as OpenFin from '../../OpenFin';
 import { Base } from '../base';
 import { SnapshotSource } from './Instance';
 /**
+ * @REMOVED
  * @typedef { object } SnapshotProvider
  * @property {getSnapshot} [getSnapshot]
  * @property {applySnapshot} [applySnapshot]

package/src/api/snapshot-source/Factory.js

@@ -4,6 +4,7 @@ const base_1 = require("../base");
 const Instance_1 = require("./Instance");
 const utils_1 = require("./utils");
 /**
+ * @REMOVED
  * @typedef { object } SnapshotProvider
  * @property {getSnapshot} [getSnapshot]
  * @property {applySnapshot} [applySnapshot]

package/src/api/snapshot-source/Instance.d.ts

@@ -1,4 +1,4 @@
-import * as OpenFin from '../../OpenFin';
+import type * as OpenFin from '../../OpenFin';
 import { Base } from '../base';
 import Transport from '../../transport/transport';
 /**

package/src/api/snapshot-source/utils.d.ts

@@ -1,2 +1,2 @@
-import * as OpenFin from '../../OpenFin';
+import type * as OpenFin from '../../OpenFin';
 export declare const getSnapshotSourceChannelName: (id: OpenFin.ApplicationIdentity) => string;

package/src/api/system/index.d.ts

@@ -1,10 +1,9 @@
-import * as OpenFin from '../../OpenFin';
+import type * as OpenFin from '../../OpenFin';
 import { EmitterBase } from '../base';
 import Transport from '../../transport/transport';
 declare type Identity = OpenFin.Identity;
 declare type ProxyInfo = OpenFin.ProxyInfo;
 declare type ProxyConfig = OpenFin.ProxyConfig;
-declare type SystemEvents = OpenFin.SystemEvents;
 declare type InstalledApps = OpenFin.InstalledApps;
 declare type LogInfo = OpenFin.LogInfo;
 declare type LogLevel = OpenFin.LogLevel;
@@ -21,7 +20,7 @@ declare type PrinterInfo = OpenFin.PrinterInfo;
  * clearing the cache and exiting the runtime as well as listen to <a href="tutorial-System.EventEmitter.html">system events</a>.
  * @namespace
  */
-export default class System extends EmitterBase<SystemEvents> {
+export default class System extends EmitterBase<OpenFin.SystemEvent> {
     constructor(wire: Transport);
     private sendExternalProcessRequest;
     /**

package/src/api/system/index.js

@@ -531,19 +531,27 @@ class System extends base_1.EmitterBase {
                 progressListener(p);
             };
             const cleanListeners = () => {
+                // TODO: fix internal types
+                // @ts-expect-error
                 this.removeListener(dlProgressKey, dlProgress);
             };
-            const dlError = (r, err) => {
-                const error = err || r;
+            const dlError = (payload) => {
                 cleanListeners();
-                reject(new transport_errors_1.RuntimeError(error));
+                const { reason, err: error } = payload;
+                reject(new transport_errors_1.RuntimeError({ reason, error }));
             };
             const dlComplete = () => {
                 cleanListeners();
                 resolve();
             };
+            // TODO: fix internal types
+            // @ts-expect-error
             this.on(dlProgressKey, dlProgress);
+            // TODO: fix internal types
+            // @ts-expect-error
             this.once(dlErrorKey, dlError);
+            // TODO: fix internal types
+            // @ts-expect-error
             this.once(dlCompleteKey, dlComplete);
             const downloadOptions = Object.assign(appAsset, { downloadId });
             this.wire.sendAction('download-asset', downloadOptions).catch((err) => {
@@ -578,19 +586,27 @@ class System extends base_1.EmitterBase {
                 progressListener(p);
             };
             const cleanListeners = () => {
+                // TODO: fix internal types
+                // @ts-expect-error
                 this.removeListener(dlProgressKey, dlProgress);
             };
-            const dlError = (r, err) => {
-                const error = err || r;
+            const dlError = (payload) => {
                 cleanListeners();
-                reject(new transport_errors_1.RuntimeError(error));
+                const { reason, err: error } = payload;
+                reject(new transport_errors_1.RuntimeError({ reason, error }));
             };
             const dlComplete = () => {
                 cleanListeners();
                 resolve();
             };
+            // TODO: fix internal types
+            // @ts-expect-error
             this.on(dlProgressKey, dlProgress);
+            // TODO: fix internal types
+            // @ts-expect-error
             this.once(dlErrorKey, dlError);
+            // TODO: fix internal types
+            // @ts-expect-error
             this.once(dlCompleteKey, dlComplete);
             const downloadOptions = Object.assign(options, { downloadId });
             this.wire.sendAction('download-runtime', downloadOptions).catch((err) => {

package/src/api/view/Factory.d.ts

@@ -1,4 +1,4 @@
-import * as OpenFin from '../../OpenFin';
+import type * as OpenFin from '../../OpenFin';
 import { Base } from '../base';
 /**
  * @lends View

package/src/api/view/Instance.d.ts

@@ -1,10 +1,11 @@
-import * as OpenFin from '../../OpenFin';
+import type * as OpenFin from '../../OpenFin';
 import { WebContents } from '../webcontents/main';
 import Transport from '../../transport/transport';
 import { Layout } from '../platform/layout';
-declare type ViewEvents = OpenFin.ViewEvents;
+import { ViewEvent } from '../events/view';
 declare type UpdatableViewOptions = OpenFin.UpdatableViewOptions;
 /**
+ * @PORTED
  * @typedef {object} View~options
  * @summary View creation options.
  * @desc This is the options object required by {@link View.create View.create}.
@@ -170,7 +171,7 @@ declare type UpdatableViewOptions = OpenFin.UpdatableViewOptions;
  * @alias View
  * @hideconstructor
  */
-export declare class View extends WebContents<ViewEvents> {
+export declare class View extends WebContents<ViewEvent> {
     identity: OpenFin.Identity;
     constructor(wire: Transport, identity: OpenFin.Identity);
     /**
@@ -453,12 +454,15 @@ export declare class View extends WebContents<ViewEvents> {
      */
     triggerBeforeUnload: () => Promise<boolean>;
     /**
+     * **NOTE**: Internal use only.
      * Attaches this view to an HTML element in the current context. The view will resize responsively when the element bounds change.
-     * @param { string } elementId - id of the HTML element to attach the view to.
+     *
+     * @param {HTMLElement} element - HTML element to attach the view to.
      * @return {Function} - Cleanup function that will disconnect the element resize observer.
+     * @internal
      * @experimental
      * @tutorial View.bindToElement
      */
-    bindToElement: (elementId: string) => Promise<() => void>;
+    bindToElement: (element: HTMLElement) => Promise<() => void>;
 }
 export {};

package/src/api/view/Instance.js

@@ -8,6 +8,7 @@ const main_1 = require("../webcontents/main");
 const window_1 = require("../window");
 const bounds_observer_1 = require("../platform/layout/utils/bounds-observer");
 /**
+ * @PORTED
  * @typedef {object} View~options
  * @summary View creation options.
  * @desc This is the options object required by {@link View.create View.create}.
@@ -457,16 +458,18 @@ class View extends main_1.WebContents {
             return message.payload.data;
         };
         /**
+         * **NOTE**: Internal use only.
          * Attaches this view to an HTML element in the current context. The view will resize responsively when the element bounds change.
-         * @param { string } elementId - id of the HTML element to attach the view to.
+         *
+         * @param {HTMLElement} element - HTML element to attach the view to.
          * @return {Function} - Cleanup function that will disconnect the element resize observer.
+         * @internal
          * @experimental
          * @tutorial View.bindToElement
          */
-        this.bindToElement = async (elementId) => {
-            const element = document.getElementById(elementId);
+        this.bindToElement = async (element) => {
             if (!element) {
-                throw new Error(`Element matching id: ${elementId} not found.`);
+                throw new Error('Element not found.');
             }
             const onChange = async (bounds) => this.setBounds(bounds);
             return (0, bounds_observer_1.observeBounds)(element, onChange);

package/src/api/webcontents/main.d.ts

@@ -1,14 +1,29 @@
-import * as OpenFin from '../../OpenFin';
+import type * as OpenFin from '../../OpenFin';
 import { EmitterBase } from '../base';
 import Transport from '../../transport/transport';
-import { WebContentsEventMapping } from '../events/webcontents';
-declare type ImageFormat = 'bmp' | 'jpg' | 'png';
+import { BaseEvent } from '../events/base';
+/**
+ * Configuration for page capture.
+ */
 export interface CapturePageOptions {
+    /**
+     * The area of the window to be captured.
+     */
     area?: OpenFin.Rectangle;
-    format?: ImageFormat;
+    /**
+     * @defaultValue 'png'
+     *
+     * The format of the captured image.  Can be 'png', 'jpg', or 'bmp'.
+     */
+    format?: 'bmp' | 'jpg' | 'png';
+    /**
+     * @defaultValue 100
+     *
+     * Quality of JPEG image. Between 0 - 100.
+     */
     quality?: number;
 }
-export declare class WebContents<T extends WebContentsEventMapping> extends EmitterBase<T> {
+export declare class WebContents<T extends BaseEvent> extends EmitterBase<T> {
     entityType: string;
     constructor(wire: Transport, identity: OpenFin.Identity, entityType: string);
     capturePage(options?: CapturePageOptions): Promise<string>;
@@ -35,4 +50,3 @@ export declare class WebContents<T extends WebContentsEventMapping> extends Emit
     inspectServiceWorker(): Promise<void>;
     showPopupWindow(options: OpenFin.PopupOptions): Promise<OpenFin.PopupResult>;
 }
-export {};

package/src/api/webcontents/main.js

@@ -92,6 +92,8 @@ class WebContents extends base_1.EmitterBase {
                     throw new Error(`Something went wrong during onPopupReady execution: ${error}`);
                 }
             };
+            // TODO: fix typing (internal)
+            // @ts-expect-error
             await this.once('popup-ready', readyListener);
         }
         const { payload: tryCreatePayload } = await this.wire.sendAction('try-create-popup-window', {
@@ -130,9 +132,17 @@ class WebContents extends base_1.EmitterBase {
                 await options.onPopupResult(normalizePopupResult(payload));
             };
             const teardownListener = async () => {
+                // TODO: fix typing (internal)
+                // @ts-expect-error
                 await this.removeListener('popup-result', dispatchResultListener);
             };
+            // TODO: fix typing (internal)
+            // @ts-expect-error
             await this.on('popup-result', dispatchResultListener);
+            // TODO: fix typing (internal)
+            // hilariously this does not need a ts-expect-error - this is gap in type soundness
+            // should investigate - probably due to `teardownListener` taking a void argument
+            // which might play nicely with the `never` type?  huh...
             await this.once('popup-teardown', teardownListener);
         }
         const { payload } = await this.wire.sendAction('show-popup-window', {

package/src/api/window/Factory.d.ts

@@ -1,4 +1,4 @@
-import * as OpenFin from '../../OpenFin';
+import type * as OpenFin from '../../OpenFin';
 import { Base } from '../base';
 /**
  * @lends Window