fix: warn on non-existent polyfill names instead of throwing error

haoqunjiang · haoqunjiang · commit bbfccba1fbd6 · 2020-06-02T09:49:26.000+08:00
Fixes vuejs#3578
diff --git a/packages/@vue/babel-preset-app/index.js b/packages/@vue/babel-preset-app/index.js
@@ -81,7 +81,14 @@ function getPolyfills (targets, includes) {
   }
 
   const compatData = require('core-js-compat').data
-  return includes.filter(item => isRequired(item, targets, { compatData }))
+  return includes.filter(item => {
+    if (!compatData[item]) {
+      console.warn(`Skipping polyfill ${item} as it's not found in the compatibility table.`)
+      return false
+    }
+
+    return isRequired(item, targets, { compatData })
+  })
 }
 
 module.exports = (context, options = {}) => {

Original file line number	Diff line number	Diff line change
`@@ -81,7 +81,14 @@ function getPolyfills (targets, includes) {`
`81`	`81`	`}`
`82`	`82`
`83`	`83`	`const compatData = require('core-js-compat').data`
`84`		`- return includes.filter(item => isRequired(item, targets, { compatData }))`
	`84`	`+ return includes.filter(item => {`
	`85`	`+ if (!compatData[item]) {`
	`86`	+ console.warn(`Skipping polyfill ${item} as it's not found in the compatibility table.`)
	`87`	`+ return false`
	`88`	`+ }`
	`89`	`+`
	`90`	`+ return isRequired(item, targets, { compatData })`
	`91`	`+ })`
`85`	`92`	`}`
`86`	`93`
`87`	`94`	`module.exports = (context, options = {}) => {`