Improve error handling of plugins

src/config/plugin/lib/import.js

@@ -2,8 +2,6 @@ import { pathToFileURL } from 'url'
 
 import isPlainObj from 'is-plain-obj'
 
-import { wrapError } from '../../../error/wrap.js'
-
 import { PluginError } from './error.js'
 
 // Builtin modules are lazy loaded for performance reasons.
@@ -19,8 +17,8 @@ export const importPlugin = async function (
     const { plugin, path } = await IMPORTERS[locationType](location, builtins)
     const pluginA = isPlainObj(plugin) ? { ...plugin } : plugin
     return { plugin: pluginA, path }
-  } catch (error) {
-    throw wrapError(error, `Could not load "${name}"\n`, PluginError)
+  } catch (cause) {
+    throw new PluginError(`Could not load "${name}".`, { cause })
   }
 }
 

src/config/plugin/lib/info.js

@@ -1,5 +1,3 @@
-import { wrapError } from '../../../error/wrap.js'
-
 import { normalizePluginConfig } from './config.js'
 import { PluginError } from './error.js'
 import { importPlugin } from './import.js'
@@ -45,6 +43,6 @@ export const getPluginInfo = async function (pluginConfig, opts) {
 
 const handlePluginError = function (error, originalLocation) {
   return error instanceof PluginError
-    ? wrapError(error, `Invalid plugin "${originalLocation}":`)
+    ? new Error(`Invalid plugin "${originalLocation}".`, { cause: error })
     : error
 }

src/config/plugin/lib/main.js

@@ -1,3 +1,4 @@
+import { mergeErrorCause } from '../../../error/merge/main.js'
 import { allowErrorTypes } from '../../../error/types.js'
 
 import { validateDuplicatePlugins } from './duplicates.js'
@@ -32,7 +33,8 @@ export const getPlugins = async function (pluginConfigs, opts) {
     validateDuplicatePlugins(pluginInfos, optsA)
     return pluginInfos
   } catch (error) {
-    throw allowErrorTypes(error, ErrorTypes)
+    const errorA = mergeErrorCause(error)
+    throw allowErrorTypes(errorA, ErrorTypes)
   }
 }
 

src/config/plugin/lib/module.js

@@ -1,7 +1,5 @@
 import { createRequire } from 'module'
 
-import { wrapError } from '../../../error/wrap.js'
-
 // Resolve Node module names to absolute file paths.
 // We enforce a npm package naming convention for all plugins.
 // TODO: use import.meta.resolve() when available
@@ -13,11 +11,11 @@ export const resolveModuleLocation = function (
 
   try {
     return createRequire(`${cwd}/`).resolve(moduleName)
-  } catch (error) {
-    throw wrapError(
-      error,
+  } catch (cause) {
+    throw new Error(
       `must be ${getBuiltinsError(builtins)}
-This Node module was not found, please ensure it is installed.\n`,
+This Node module was not found, please ensure it is installed:\n`,
+      { cause },
     )
   }
 }

src/config/plugin/lib/normalize.js

@@ -1,4 +1,3 @@
-import { wrapError } from '../../../error/wrap.js'
 import { normalizeInputs } from '../../normalize/lib/main.js'
 
 import { CoreError } from './error.js'
@@ -9,7 +8,8 @@ export const safeNormalizeConfig = async function (config, rules, opts) {
     const { inputs } = await normalizeInputs(config, rules, opts)
     return inputs
   } catch (error) {
-    throw wrapError(error, '', getErrorType(error, opts))
+    const ErrorType = getErrorType(error, opts)
+    throw new ErrorType('', { cause: error })
   }
 }