I don't believe this is correct either. I'm working on a project that has some Vue code, and some plain JS. If I implement your change I see this despite having some Vue files in play:

[webpack-cli] Error: [VueLoaderPlugin Error] No matching rule for .vue files found.Make sure there is at least one root-level rule that matches .vue or .vue.html files

Your change, however, does get me further down the road in solving the error in the noted lined tickets.

I've tracked this down to a problem in my own configuration. I have a rule supplied by Shakapacker that looks like this:

  test:/\.html$/,exclude:/\.(js|mjs|jsx|ts|tsx)$/,type:'asset/source'}

This rule trips thefoo.vue.html test, which, in turn, confuses vue-loader.

I'm not a Vue expert by any stretch of the imagination, so I'm unclear if thefoo.vue.html test is valid or not, but it seems like I need to intercept the rule above and improve the exclude regex.

Good luck on your own journey!