perf: simpler error handling

src/index.ts

@@ -91,7 +91,6 @@ export default (options: Options = {}): Plugin => {
         deps: new Set(),
         assets: new Map<string, Uint8Array>(),
         warn: this.warn.bind(this),
-        error: this.error.bind(this),
         plugin: this,
         options: {},
       };

src/loaders/less/index.ts

@@ -11,7 +11,7 @@ const loader: Loader<LESSLoaderOptions> = {
   test: /\.less$/i,
   async process({ code, map }) {
     const less = await loadModule("less");
-    if (!less) this.error("You need to install `less` package in order to process Less files");
+    if (!less) throw new Error("You need to install `less` package in order to process Less files");
 
     const res = await less.render(code, {
       ...this.options,

src/loaders/postcss/index.ts

@@ -53,7 +53,7 @@ const loader: Loader<PostCSSLoaderOptions> = {
   async process({ code, map, extracted }) {
     const { options } = this;
 
-    const config = await loadConfig(this.id, options.config).catch(this.error);
+    const config = await loadConfig(this.id, options.config);
 
     const plugins = [
       ...[

src/loaders/sass/index.ts

@@ -13,7 +13,7 @@ const loader: Loader<SASSLoaderOptions> = {
   async process({ code, map }) {
     const { options } = this;
 
-    const [sass, type] = await loadSass(options.impl).catch(this.error);
+    const [sass, type] = await loadSass(options.impl);
 
     // `fibers` doesn't work in testing
     const useFibers = options.fibers ?? (type === "sass" && process.env.NODE_ENV !== "test");

src/loaders/stylus.ts

@@ -11,7 +11,7 @@ const loader: Loader<StylusLoaderOptions> = {
   async process({ code, map }) {
     const stylus = await loadModule("stylus");
     if (!stylus)
-      this.error("You need to install `stylus` package in order to process Stylus files");
+      throw new Error("You need to install `stylus` package in order to process Stylus files");
 
     const style = stylus(code, { ...this.options })
       .set("filename", this.id)

src/types.ts

@@ -152,8 +152,6 @@ export type LoaderContext<TLoaderOptions = ObjectWithUnknownProps> = {
   readonly assets: Map<string, Uint8Array>;
   /** Function for emitting a waring */
   readonly warn: PluginContext["warn"];
-  /** Function for emitting an error */
-  readonly error: PluginContext["error"];
   /** https://rollupjs.org/guide/en#plugin-context */
   readonly plugin: PluginContext;
 };