Clean up code structure to use try/catch/await

Author: andreiborza

packages/webpack-plugin/src/webpack4and5.ts

@@ -281,25 +281,26 @@ export function sentryWebpackPluginFactory({
             const freeGlobalDependencyOnBuildArtifacts = createDependencyOnBuildArtifacts();
             const upload = createDebugIdUploadFunction({ sentryBuildPluginManager });
 
-            sentryBuildPluginManager
-              .createRelease()
-              .then(async () => {
+            const run = async (): Promise<void> => {
+              try {
+                await sentryBuildPluginManager.createRelease();
                 if (sourcemapsEnabled && options.sourcemaps?.disable !== "disable-upload") {
                   const outputPath = compilation.outputOptions.path ?? path.resolve();
                   const buildArtifacts = Object.keys(compilation.assets).map((asset) =>
                     path.join(outputPath, asset)
                   );
                   await upload(buildArtifacts);
                 }
-              })
-              .finally(async () => {
+              } finally {
                 freeGlobalDependencyOnBuildArtifacts();
                 await sentryBuildPluginManager.deleteArtifacts();
-              })
-              .then(
-                () => callback(),
-                (err: Error) => callback(err)
-              );
+              }
+            };
+
+            run().then(
+              () => callback(),
+              (err: Error) => callback(err)
+            );
           }
         );