Narrow `mount()` return type

[hyunbinseo]

Describe the problem

Current mount API returns Record<string, any>

/**
 * Mounts a component to the given target and returns the exports and potentially the props (if compiled with `accessors: true`) of the component.
 * Transitions will play during the initial render unless the `intro` option is set to `false`.
 *
 * @template {Record<string, any>} Props
 * @template {Record<string, any>} Exports
 * @param {ComponentType<SvelteComponent<Props>> | Component<Props, Exports, any>} component
 * @param {MountOptions<Props>} options
 * @returns {Exports}
 */
export function mount(component, options) {
  return _mount(component, options);
}

For cases like wxt, app has to be typed like this:

let app: Record<string, any>;

import { mount, unmount } from 'svelte';
import App from '../lib/App.svelte';

export default defineContentScript({
  main: async (ctx) => {
    let app: Record<string, any>;
    createShadowRootUi(ctx, {
      onMount: (uiContainer) => {
        app = mount(App, { target: uiContainer });
      },
      onRemove: () => unmount(app),
    }).then((ui) => ui.mount());
  },
});

Describe the proposed solution

Provide a Mounted type.

Importance

nice to have

[dummdidumm]

The return type of a function is the exports of that component. So Mounted would just be an alias for Record. As such it doesn't make sense to provide one. Therefore closing.

[hyunbinseo]

Thank you for reviewing.

@dummdidumm would this be the desired code then?

let app: Record<string, any>;

I am currently updating the wxt documentation:

• docs: fix svelte app unmount code wxt-dev/wxt#1673

[dummdidumm]

Yes that looks good 👍