feat(gatsby): Add optional deleteSourcemapsAfterUpload (#13610)

Related #13582
This work adds the `deleteSourcemapsAfterUpload` option to the Gatsby
plugin, allowing it to be passed to the Webpack plugin to set the
`sourceMapFilesToDeleteAfterUpload` without exposing the API.

This simplifies our workflow by eliminating the need to apply a Yarn
patch to modify the package whenever we want to use
`filesToDeleteAfterUpload`.

Before submitting a pull request, please take a look at our

[Contributing](https://github.com/getsentry/sentry-javascript/blob/master/CONTRIBUTING.md)
guidelines and verify:

- [x] If you've added code that should be tested, please add tests.
- [x] Ensure your code lints and the test suite passes (`yarn lint`) &
(`yarn test`).

---------

Co-authored-by: Luca Forstner <[email protected]>

Author: julianCast

packages/gatsby/README.md

@@ -65,6 +65,24 @@ module.exports = {
 };
 ```
 
+Additionally, you can delete source map files after they have been uploaded by setting the `deleteSourcemapsAfterUpload`
+option to be `true`.
+
+```javascript
+module.exports = {
+  // ...
+  plugins: [
+    {
+      resolve: '@sentry/gatsby',
+      options: {
+        deleteSourcemapsAfterUpload: true,
+      },
+    },
+    // ...
+  ],
+};
+```
+
 ## Links
 
 - [Official SDK Docs](https://docs.sentry.io/quickstart/)

packages/gatsby/gatsby-node.js

@@ -7,12 +7,15 @@ const SENTRY_USER_CONFIG = ['./sentry.config.js', './sentry.config.ts'];
 exports.onCreateWebpackConfig = ({ getConfig, actions }, options) => {
   const enableClientWebpackPlugin = options.enableClientWebpackPlugin !== false;
   if (process.env.NODE_ENV === 'production' && enableClientWebpackPlugin) {
+    const deleteSourcemapsAfterUpload = options.deleteSourcemapsAfterUpload === true;
     actions.setWebpackConfig({
       plugins: [
         sentryWebpackPlugin({
           sourcemaps: {
             // Only include files from the build output directory
             assets: ['./public/**'],
+            // Delete source files after uploading
+            filesToDeleteAfterUpload: deleteSourcemapsAfterUpload ? ['./public/**/*.map'] : undefined,
             // Ignore files that aren't users' source code related
             ignore: [
               'polyfill-*', // related to polyfills

packages/gatsby/test/gatsby-node.test.ts

@@ -1,5 +1,12 @@
+import { sentryWebpackPlugin } from '@sentry/webpack-plugin';
 import { onCreateWebpackConfig } from '../gatsby-node';
 
+jest.mock('@sentry/webpack-plugin', () => ({
+  sentryWebpackPlugin: jest.fn().mockReturnValue({
+    apply: jest.fn(),
+  }),
+}));
+
 describe('onCreateWebpackConfig', () => {
   let originalNodeEnv: string | undefined;
 
@@ -12,6 +19,10 @@ describe('onCreateWebpackConfig', () => {
     process.env.NODE_ENV = originalNodeEnv;
   });
 
+  afterEach(() => {
+    jest.clearAllMocks();
+  });
+
   it('sets a webpack config', () => {
     const actions = {
       setWebpackConfig: jest.fn(),
@@ -36,4 +47,24 @@ describe('onCreateWebpackConfig', () => {
 
     expect(actions.setWebpackConfig).toHaveBeenCalledTimes(0);
   });
+
+  it('sets sourceMapFilesToDeleteAfterUpload when provided in options', () => {
+    const actions = {
+      setWebpackConfig: jest.fn(),
+    };
+
+    const getConfig = jest.fn();
+
+    onCreateWebpackConfig({ actions, getConfig }, { deleteSourcemapsAfterUpload: true });
+
+    expect(actions.setWebpackConfig).toHaveBeenCalledTimes(1);
+
+    expect(sentryWebpackPlugin).toHaveBeenCalledWith(
+      expect.objectContaining({
+        sourcemaps: expect.objectContaining({
+          filesToDeleteAfterUpload: ['./public/**/*.map'],
+        }),
+      }),
+    );
+  });
 });