add support for rewrite and ext options

README.md

@@ -75,6 +75,8 @@ Adding the following to your workflow will create a new Sentry release and tell
 |`projects`|Space-separated list of paths of projects. When omitted, falls back to the environment variable `SENTRY_PROJECT` to determine the project.|-|
 |`url_prefix`|Adds a prefix to source map urls after stripping them.|-|
 |`strip_common_prefix`|Will remove a common prefix from uploaded filenames. Useful for removing a path that is build-machine-specific.|`false`|
+|`rewrite`|Enables rewriting of matching sourcemaps so that indexed maps are flattened and missing sources are inlined if possible.|`true`|
+|`ext`|Space-separated list of file extensions to be considered for source map upload. By default: js, map, jsbundle, bundle.|-|
 |`working_directory`|Directory to collect sentry release information from. Useful when collecting information from a non-standard checkout directory.|-|
 
 ### Examples
@@ -88,6 +90,18 @@ Adding the following to your workflow will create a new Sentry release and tell
         sourcemaps: './lib'
     ```
 
+- Upload source maps with custom file extensions and disable rewriting.
+
+    ```yaml
+    - uses: getsentry/action-release@v1
+      with:
+        environment: 'production'
+        sourcemaps: './build'
+        url_prefix: '~/'
+        rewrite: false
+        ext: 'js map'
+    ```
+
 - Create a new Sentry release for the `production` environment of your project at version `v1.0.1`.
 
     ```yaml

__tests__/main.test.ts

@@ -10,6 +10,7 @@ import {
   getSetCommitsOption,
   getProjects,
   getUrlPrefixOption,
+  getExtOption,
 } from '../src/options';
 
 describe('options', () => {
@@ -209,6 +210,26 @@ describe('options', () => {
       expect(getUrlPrefixOption()).toEqual('build');
     });
   });
+
+  describe('getExtOption', () => {
+    afterEach(() => {
+      delete process.env['INPUT_EXT'];
+    });
+
+    test('should return undefined when ext is omitted', () => {
+      expect(getExtOption()).toBeUndefined();
+    });
+
+    test('should return array when ext is provided', () => {
+      process.env['INPUT_EXT'] = 'js map';
+      expect(getExtOption()).toEqual(['js', 'map']);
+    });
+
+    test('should filter out empty strings', () => {
+      process.env['INPUT_EXT'] = 'js  map';
+      expect(getExtOption()).toEqual(['js', 'map']);
+    });
+  });
 });
 
 // shows how the runner will run a javascript action with env / stdout protocol

action.yml

@@ -41,6 +41,13 @@ inputs:
   strip_common_prefix:
     description: 'Will remove a common prefix from uploaded filenames. Useful for removing a path that is build-machine-specific.'
     required: false
+  rewrite:
+    description: 'Enables rewriting of matching sourcemaps so that indexed maps are flattened and missing sources are inlined if possible. Defaults to true.'
+    required: false
+    default: true
+  ext:
+    description: 'Space-separated list of file extensions to be considered for source map upload. By default: js, map, jsbundle, bundle.'
+    required: false
   working_directory:
     description: 'Directory to collect sentry release information from. Useful when collecting information from a non-standard checkout directory.'
     required: false

lib/main.js

@@ -48,6 +48,8 @@ const process = __importStar(require("process"));
         const projects = options.getProjects();
         const urlPrefix = options.getUrlPrefixOption();
         const stripCommonPrefix = options.getBooleanOption('strip_common_prefix', false);
+        const rewrite = options.getBooleanOption('rewrite', true);
+        const ext = options.getExtOption();
         const version = yield options.getVersion();
         const workingDirectory = options.getWorkingDirectory();
         core.debug(`Version is ${version}`);
@@ -75,6 +77,8 @@ const process = __importStar(require("process"));
                     dist,
                     urlPrefix,
                     stripCommonPrefix,
+                    rewrite,
+                    ext,
                 };
                 return cli.uploadSourceMaps(version, sourceMapOptions);
             })));

lib/options.js

@@ -28,7 +28,7 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
     });
 };
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.getWorkingDirectory = exports.getUrlPrefixOption = exports.getProjects = exports.checkEnvironmentVariables = exports.getSetCommitsOption = exports.getBooleanOption = exports.getDist = exports.getSourcemaps = exports.getStartedAt = exports.getEnvironment = exports.getVersion = void 0;
+exports.getExtOption = exports.getWorkingDirectory = exports.getUrlPrefixOption = exports.getProjects = exports.checkEnvironmentVariables = exports.getSetCommitsOption = exports.getBooleanOption = exports.getDist = exports.getSourcemaps = exports.getStartedAt = exports.getEnvironment = exports.getVersion = void 0;
 const core = __importStar(require("@actions/core"));
 const cli_1 = require("./cli");
 /**
@@ -182,3 +182,10 @@ exports.getUrlPrefixOption = () => {
 exports.getWorkingDirectory = () => {
     return core.getInput('working_directory');
 };
+exports.getExtOption = () => {
+    const extOption = core.getInput('ext');
+    if (!extOption) {
+        return undefined;
+    }
+    return extOption.split(' ').filter(ext => ext.length > 0);
+};

src/main.ts

@@ -24,6 +24,8 @@ import * as process from 'process';
       'strip_common_prefix',
       false
     );
+    const rewrite = options.getBooleanOption('rewrite', true);
+    const ext = options.getExtOption();
     const version = await options.getVersion();
     const workingDirectory = options.getWorkingDirectory();
 
@@ -56,6 +58,8 @@ import * as process from 'process';
             dist,
             urlPrefix,
             stripCommonPrefix,
+            rewrite,
+            ext,
           };
           return cli.uploadSourceMaps(version, sourceMapOptions);
         })

src/options.ts

@@ -182,3 +182,12 @@ export const getUrlPrefixOption = (): string => {
 export const getWorkingDirectory = (): string => {
   return core.getInput('working_directory');
 };
+
+export const getExtOption = (): string[] | undefined => {
+  const extOption: string = core.getInput('ext');
+  if (!extOption) {
+    return undefined;
+  }
+
+  return extOption.split(' ').filter(ext => ext.length > 0);
+};