feat: Default to automatically setting commits on release

Author: Luca Forstner

packages/bundler-plugin-core/src/options-mapping.ts

@@ -26,6 +26,11 @@ export function normalizeUserOptions(userOptions: UserOptions) {
       create: userOptions.release?.create ?? true,
       finalize: userOptions.release?.finalize ?? true,
       vcsRemote: userOptions.release?.vcsRemote ?? process.env["SENTRY_VSC_REMOTE"] ?? "origin",
+      setCommits: userOptions.release?.setCommits ?? {
+        auto: true,
+        isDefault: true,
+        repo: undefined, // Just to please type narrowing
+      },
     },
     bundleSizeOptimizations: userOptions.bundleSizeOptimizations,
     reactComponentAnnotation: userOptions.reactComponentAnnotation,

packages/bundler-plugin-core/src/plugins/release-management.ts

@@ -13,7 +13,7 @@ interface ReleaseManagementPluginOptions {
   shouldCreateRelease: boolean;
   shouldFinalizeRelease: boolean;
   include?: string | IncludeEntry | Array<string | IncludeEntry>;
-  setCommitsOption?: SentryCliCommitsOptions;
+  setCommitsOption: SentryCliCommitsOptions | false | { auto: true; isDefault: true };
   deployOptions?: SentryCliNewDeployOptions;
   dist?: string;
   handleRecoverableError: HandleRecoverableErrorFn;
@@ -37,6 +37,7 @@ interface ReleaseManagementPluginOptions {
  * Additionally, if legacy upload options are set, it uploads source maps in the legacy (non-debugId) way.
  */
 export function releaseManagementPlugin({
+  logger,
   releaseName,
   include,
   dist,
@@ -86,8 +87,20 @@ export function releaseManagementPlugin({
           });
         }
 
-        if (setCommitsOption) {
-          await cliInstance.releases.setCommits(releaseName, setCommitsOption);
+        if (setCommitsOption !== false) {
+          try {
+            await cliInstance.releases.setCommits(releaseName, setCommitsOption);
+          } catch (e) {
+            // isDefault being present means that the plugin defaulted to `{ auto: true }` for the setCommitsOptions, meaning that wee should not throw when CLI throws because there is no repo
+            if (!("isDefault" in setCommitsOption)) {
+              throw e;
+            } else {
+              logger.debug(
+                "An error occurred setting commits on release (this message can be ignored unless you commits on release are desired):",
+                e
+              );
+            }
+          }
         }
 
         if (shouldFinalizeRelease) {

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

@@ -200,8 +200,10 @@ export interface Options {
 
     /**
      * Associates the release with its commits in Sentry.
+     *
+     * Defaults to `{ auto: true }`. Set to `false` to disable commit association.
      */
-    setCommits?: SetCommitsOptions;
+    setCommits?: SetCommitsOptions | false;
 
     /**
      * Adds deployment information to the release in Sentry.

packages/bundler-plugin-core/test/option-mappings.test.ts

@@ -23,6 +23,10 @@ describe("normalizeUserOptions()", () => {
         create: true,
         vcsRemote: "origin",
         uploadLegacySourcemaps: "./out",
+        setCommits: {
+          auto: true,
+          isDefault: true,
+        },
       },
       silent: false,
       telemetry: true,
@@ -74,6 +78,10 @@ describe("normalizeUserOptions()", () => {
           sourceMapReference: false,
           stripCommonPrefix: true,
         },
+        setCommits: {
+          auto: true,
+          isDefault: true,
+        },
       },
       silent: false,
       telemetry: true,

packages/dev-utils/src/generate-documentation-table.ts

@@ -154,7 +154,8 @@ errorHandler: (err) => {
       },
       {
         name: "setCommits",
-        fullDescription: "Option to associate the created release with its commits in Sentry.",
+        fullDescription:
+          "Option to associate the created release with its commits in Sentry. Defaults to `{ auto: true }`. Set to `false` to disable.",
         children: [
           {
             name: "previousCommit",

packages/integration-tests/fixtures/telemetry/telemetry.test.ts

@@ -101,7 +101,7 @@ test("rollup bundle telemetry", async () => {
               "upload-legacy-sourcemaps": false,
               "module-metadata": false,
               "inject-build-information": false,
-              "set-commits": "undefined",
+              "set-commits": "auto",
               "finalize-release": true,
               "deploy-options": false,
               "custom-error-handler": false,