fix(runtime): checking for remote entry type when loading assets

[MadaraUchiha-314]

Description

• Adds the script type check when loading ESM
• This is to avoid the following error when loading ESM modules

• URL to reproduce: https://rollup-plugin-module-federation.netlify.app/packages/examples/project-a/dist/rollup/esm/

Related Issue

Types of changes

• Docs change / refactoring / dependency upgrade
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)

Checklist

• I have added tests to cover my changes.
• All new and existing tests passed.
• I have updated the documentation.

[changeset-bot]

🦋 Changeset detected

Latest commit: 1cc0ef7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 35 packages

Name	Type
@module-federation/runtime	Patch
@module-federation/devtools	Patch
@module-federation/dts-plugin	Patch
@module-federation/nextjs-mf	Patch
@module-federation/node	Patch
@module-federation/runtime-tools	Patch
@module-federation/webpack-bundler-runtime	Patch
@module-federation/modern-js	Patch
@module-federation/enhanced	Patch
@module-federation/rspack	Patch
modernjs-ssr-dynamic-nested-remote	Patch
modernjs-ssr-dynamic-remote-new-version	Patch
modernjs-ssr-dynamic-remote	Patch
modernjs-ssr-host	Patch
modernjs-ssr-nested-remote	Patch
modernjs-ssr-remote-new-version	Patch
modernjs-ssr-remote	Patch
3008-runtime-remote	Patch
host	Patch
host-v5	Patch
host-vue3	Patch
remote1	Patch
remote2	Patch
remote3	Patch
@module-federation/modernjs	Patch
@module-federation/sdk	Patch
@module-federation/managers	Patch
@module-federation/manifest	Patch
@module-federation/third-party-dts-extractor	Patch
@module-federation/bridge-react	Patch
@module-federation/bridge-vue3	Patch
@module-federation/bridge-shared	Patch
@module-federation/bridge-react-webpack-plugin	Patch
@module-federation/esbuild	Patch
@module-federation/utilities	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[netlify]

✅ Deploy Preview for module-federation-docs ready!

Name	Link
🔨 Latest commit	1cc0ef7
🔍 Latest deploy log	https://app.netlify.com/sites/module-federation-docs/deploys/66a854aeb287ac00085a3c56
😎 Deploy Preview	https://deploy-preview-2753--module-federation-docs.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[MadaraUchiha-314]

@ScriptedAlchemy @2heal1 Are these tests flaky or should I look at why these are failing ?

[MadaraUchiha-314]

I see that other PRs also have this problem. Can this be merged ?

[2heal1]

I have a question: Will you change the remote.type as 'module' ? Because we don't add the attr in manifest currently

[ScriptedAlchemy]

@2heal1 isnt the remoteType in manifest "esm" when its esm mode?
I cannot remember the type we set for module

[2heal1]

@ScriptedAlchemy I check the manifest again , the remoteInfo.type will use library.type first , so it can work as expected

[MadaraUchiha-314]

Can we merge this ?

[MadaraUchiha-314]

@ScriptedAlchemy @2heal1 This one liner PR has been waiting for almost 2 weeks. Can we merge it ?