Better handling of publicPath: 'auto'

[AndreiSoroka]

What kind of change does this PR introduce? (check at least one)

• Bugfix
• Feature
• Code style update
• Refactor
• Docs
• Underlying tools
• Other, please describe:

Does this PR introduce a breaking change? (check one)

• Yes
• No

Other information:

For module-federation need to use output auto

const { defineConfig } = require('@vue/cli-service') const webpack = require('webpack') module.exports = defineConfig({ configureWebpack: { output: { publicPath: 'auto', // <- ERROR: Avoid modifying webpack output.publicPath directly. Use the "publicPath" option instead. }, optimization: { splitChunks: false, }, plugins: [ new webpack.container.ModuleFederationPlugin({ name: 'vue_cli_demo', filename: 'remoteEntry.js', exposes: { './HelloWorld.vue': './src/components/HelloWorld.vue', }, shared: { vue: { singleton: true, }, }, }), ], }, transpileDependencies: true })

Fix problem for

• external_"app2":1 Uncaught (in promise) ScriptExternalLoadError: Loading script failed. module-federation/module-federation-examples#307
• webpack module federation doesn't work after vue-cli@5.0.0-beta.3 #6823

after that I'll add example here

• Add example vue3 cli module-federation/module-federation-examples#1632

[hansonfang]

https://next.cli.vuejs.org/config/#publicpath VueConfig has option for this.

[AndreiSoroka]

@hansonfang

Local: http://localhost:8082/auto

But... I'm checking at the moment, seems MF is working...

Last time it confused me and I went looking for a workaround

[AndreiSoroka]

@hansonfang updated MR

[AndreiSoroka]

@hansonfang what do you think about current Mr? (after update)

[AndreiSoroka]

FYI: @sodatea ?

[hansonfang]

@AndreiSoroka sorry I'm not member of contributor group. Maybe ask others to determine.

[haoqunjiang]

Thanks for the PR.
But I think there are still many places that we need to fix for publicPath: 'auto' to work correctly.

For example, process.env.BASE_URL should be set to '' in this case. So does the baseUrl argument for genHistoryApiFallbackRewrites.

After applying those changes, I think it would be more consistent to replace the options.publicPath here with process.env.BASE_URL, too.

[AndreiSoroka]

thanks. I will update mr on this week

[AndreiSoroka]

@sodatea Hi, can you re-review the code?