Merge branch 'main' into sig/add-ignore-to-inject

Author: s1gr1d

.github/workflows/changelog-preview.yml

@@ -0,0 +1,18 @@
+name: Changelog Preview
+on:
+  pull_request:
+    types:
+      - opened
+      - synchronize
+      - reopened
+      - edited
+      - labeled
+      - unlabeled
+permissions:
+  contents: write
+  pull-requests: write
+
+jobs:
+  changelog-preview:
+    uses: getsentry/craft/.github/workflows/changelog-preview.yml@v2
+    secrets: inherit

.github/workflows/release.yml

@@ -3,14 +3,18 @@ on:
   workflow_dispatch:
     inputs:
       version:
-        description: Version to release
-        required: true
+        description: Version to release (or "auto")
+        required: false
       force:
         description: Force a release even when there are release-blockers (optional)
         required: false
       merge_target:
         description: Target branch to merge into. Uses the default branch as a fallback (optional)
         required: false
+permissions:
+  contents: write
+  pull-requests: write
+
 jobs:
   release:
     runs-on: ubuntu-latest
@@ -27,7 +31,7 @@ jobs:
           token: ${{ steps.token.outputs.token }}
           fetch-depth: 0
       - name: Prepare release
-        uses: getsentry/action-prepare-release@v1
+        uses: getsentry/craft@c6e2f04939b6ee67030588afbb5af76b127d8203 # v2
         env:
           GITHUB_TOKEN: ${{ steps.token.outputs.token }}
         with:

packages/bundler-plugin-core/src/build-plugin-manager.ts

@@ -561,7 +561,7 @@ export function createSentryBuildPluginManager(
                 ...serializeIgnoreOptions(options.sourcemaps?.ignore),
                 ...buildArtifactPaths,
               ],
-              options.debug ?? false
+              options.debug ? "rejectOnError" : false
             );
           } catch (e) {
             sentryScope.captureException('Error in "debugIdInjectionPlugin" writeBundle hook');

packages/bundler-plugin-core/test/build-plugin-manager.test.ts

@@ -439,7 +439,7 @@ describe("createSentryBuildPluginManager", () => {
       );
     });
 
-    it("should pass debug flag when options.debug is true", async () => {
+    it('should pass "rejectOnError" flag when options.debug is true', async () => {
       mockCliExecute.mockResolvedValue(undefined);
 
       const buildPluginManager = createSentryBuildPluginManager(
@@ -460,7 +460,7 @@ describe("createSentryBuildPluginManager", () => {
 
       expect(mockCliExecute).toHaveBeenCalledWith(
         ["sourcemaps", "inject", "--ignore", "node_modules", "/path/to/bundle"],
-        true
+        "rejectOnError"
       );
     });
   });