feat: webpack no longer uses unplugin (#870)

Author: timfish

packages/bundler-plugin-core/src/index.ts

@@ -495,3 +495,5 @@ export type { Logger } from "./logger";
 export type { Options, SentrySDKBuildFlags } from "./types";
 export { CodeInjection, replaceBooleanFlagsInCode, stringToUUID } from "./utils";
 export { createSentryBuildPluginManager } from "./build-plugin-manager";
+export { generateGlobalInjectorCode, generateModuleMetadataInjectorCode } from "./utils";
+export { createDebugIdUploadFunction } from "./debug-id-upload";

packages/webpack-plugin/package.json

@@ -54,7 +54,6 @@
   },
   "dependencies": {
     "@sentry/bundler-plugin-core": "4.9.1",
-    "unplugin": "1.0.1",
     "uuid": "^9.0.0"
   },
   "devDependencies": {

packages/webpack-plugin/rollup.config.js

@@ -3,7 +3,7 @@ import babel from "@rollup/plugin-babel";
 import packageJson from "./package.json";
 import modulePackage from "module";
 
-const input = ["src/index.ts", "src/webpack5.ts"];
+const input = ["src/index.ts", "src/webpack5.ts", "src/component-annotation-transform.ts"];
 
 const extensions = [".ts"];
 

packages/webpack-plugin/src/component-annotation-transform.ts

@@ -0,0 +1,43 @@
+// Webpack loader for component annotation transform
+// Based on unplugin v1.0.1 transform loader pattern
+
+export default async function transform(
+  this: {
+    async: () => (err: Error | null, content?: string, sourceMap?: unknown) => void;
+    resourcePath: string;
+    query: {
+      transform?: (
+        code: string,
+        id: string
+      ) => Promise<{ code: string; map?: unknown } | null | undefined | string>;
+    };
+  },
+  source: string,
+  map: unknown
+): Promise<void> {
+  const callback = this.async();
+  const { transform: transformFn } = this.query;
+
+  if (!transformFn) {
+    return callback(null, source, map);
+  }
+
+  try {
+    const id = this.resourcePath;
+    const result = await transformFn(source, id);
+
+    if (result == null) {
+      callback(null, source, map);
+    } else if (typeof result === "string") {
+      callback(null, result, map);
+    } else {
+      callback(null, result.code, result.map || map);
+    }
+  } catch (error) {
+    if (error instanceof Error) {
+      callback(error);
+    } else {
+      callback(new Error(String(error)));
+    }
+  }
+}

packages/webpack-plugin/src/index.ts

@@ -1,4 +1,4 @@
-import { SentryWebpackPluginOptions, sentryWebpackUnpluginFactory } from "./webpack4and5";
+import { SentryWebpackPluginOptions, sentryWebpackPluginFactory } from "./webpack4and5";
 
 // eslint-disable-next-line @typescript-eslint/ban-ts-comment
 // @ts-ignore webpack is a peer dep
@@ -8,14 +8,12 @@ const BannerPlugin = webpack4or5?.BannerPlugin || webpack4or5?.default?.BannerPl
 
 const DefinePlugin = webpack4or5?.DefinePlugin || webpack4or5?.default?.DefinePlugin;
 
-const sentryUnplugin = sentryWebpackUnpluginFactory({
-  BannerPlugin,
-  DefinePlugin,
-});
-
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 export const sentryWebpackPlugin: (options?: SentryWebpackPluginOptions) => any =
-  sentryUnplugin.webpack;
+  sentryWebpackPluginFactory({
+    BannerPlugin,
+    DefinePlugin,
+  });
 
 export { sentryCliBinaryExists } from "@sentry/bundler-plugin-core";