fix build and auto build preview

Author: dylan-conway

.github/workflows/build-preview.yml

@@ -1,8 +1,8 @@
 name: Preview Build
 
 on:
-  issue_comment:
-    types: [created]
+  pull_request:
+    types: [opened, synchronize, reopened]
   workflow_dispatch:
     inputs:
       pr_number:
@@ -16,98 +16,61 @@ permissions:
   issues: write
 
 concurrency:
-  group: preview-build-pr-${{ github.event.issue.number || inputs.pr_number }}
+  group: preview-build-pr-${{ github.event.pull_request.number || inputs.pr_number }}
   cancel-in-progress: true
 
 jobs:
   trigger:
     name: Trigger Preview Build
     runs-on: ubuntu-latest
-    if: |
-      github.event_name == 'workflow_dispatch' ||
-      (github.event.issue.pull_request && contains(github.event.comment.body, '/build-preview'))
     outputs:
       sha: ${{ steps.pr.outputs.sha }}
       short_sha: ${{ steps.pr.outputs.short_sha }}
       release_tag: ${{ steps.pr.outputs.release_tag }}
-      pr_number: ${{ steps.pr_number.outputs.number }}
+      pr_number: ${{ steps.pr.outputs.pr_number }}
 
     steps:
-      - name: Set PR number
-        id: pr_number
-        run: |
-          if [ "${{ github.event_name }}" = "workflow_dispatch" ]; then
-            echo "number=${{ inputs.pr_number }}" >> $GITHUB_OUTPUT
-          else
-            echo "number=${{ github.event.issue.number }}" >> $GITHUB_OUTPUT
-          fi
-
-      - name: Check permissions
-        if: github.event_name == 'issue_comment'
+      - name: Check org membership
+        if: github.event_name == 'pull_request'
         uses: actions/github-script@v7
         with:
           script: |
-            const { data: permission } = await github.rest.repos.getCollaboratorPermissionLevel({
-              owner: context.repo.owner,
-              repo: context.repo.repo,
-              username: context.payload.comment.user.login
-            });
-
-            if (!['admin', 'write'].includes(permission.permission)) {
-              await github.rest.reactions.createForIssueComment({
-                owner: context.repo.owner,
-                repo: context.repo.repo,
-                comment_id: context.payload.comment.id,
-                content: '-1'
+            try {
+              await github.rest.orgs.checkMembershipForUser({
+                org: 'oven-sh',
+                username: context.actor
               });
-              core.setFailed('❌ Only collaborators with write access can trigger preview builds');
+            } catch (e) {
+              core.setFailed(`${context.actor} is not a member of oven-sh. Preview builds must be triggered manually via workflow_dispatch.`);
             }
 
       - name: Get PR information
         id: pr
         uses: actions/github-script@v7
         with:
           script: |
-            const { data: pr } = await github.rest.pulls.get({
-              owner: context.repo.owner,
-              repo: context.repo.repo,
-              pull_number: ${{ steps.pr_number.outputs.number }}
-            });
+            let sha, prNumber;
 
-            const shortSha = pr.head.sha.substring(0, 8);
-            const releaseTag = `autobuild-preview-pr-${{ steps.pr_number.outputs.number }}-${shortSha}`;
+            if (context.eventName === 'pull_request') {
+              sha = context.payload.pull_request.head.sha;
+              prNumber = context.payload.pull_request.number;
+            } else {
+              const { data: pr } = await github.rest.pulls.get({
+                owner: context.repo.owner,
+                repo: context.repo.repo,
+                pull_number: ${{ inputs.pr_number || 0 }}
+              });
+              sha = pr.head.sha;
+              prNumber = ${{ inputs.pr_number || 0 }};
+            }
 
-            core.setOutput('sha', pr.head.sha);
+            const shortSha = sha.substring(0, 8);
+            const releaseTag = `autobuild-preview-pr-${prNumber}-${shortSha}`;
+
+            core.setOutput('sha', sha);
             core.setOutput('short_sha', shortSha);
             core.setOutput('release_tag', releaseTag);
-
-      - name: React to comment
-        if: github.event_name == 'issue_comment'
-        uses: actions/github-script@v7
-        with:
-          script: |
-            await github.rest.reactions.createForIssueComment({
-              owner: context.repo.owner,
-              repo: context.repo.repo,
-              comment_id: context.payload.comment.id,
-              content: 'eyes'
-            });
-
-      - name: Post starting message
-        if: github.event_name == 'issue_comment'
-        uses: actions/github-script@v7
-        with:
-          script: |
-            const body = "🔄 Preview build started\n\n" +
-              "**Release tag**: `${{ steps.pr.outputs.release_tag }}`\n" +
-              "**Workflow**: [View progress](" + context.serverUrl + "/" + context.repo.owner + "/" + context.repo.repo + "/actions/runs/" + context.runId + ")";
-
-            await github.rest.issues.createComment({
-              owner: context.repo.owner,
-              repo: context.repo.repo,
-              issue_number: ${{ steps.pr_number.outputs.number }},
-              body: body
-            });
+            core.setOutput('pr_number', prNumber);
 
   build:
     name: Build
@@ -146,18 +109,6 @@ jobs:
               body: body
             });
 
-      - name: React to failure
-        if: needs.build.result == 'failure' && github.event_name == 'issue_comment'
-        uses: actions/github-script@v7
-        with:
-          script: |
-            await github.rest.reactions.createForIssueComment({
-              owner: context.repo.owner,
-              repo: context.repo.repo,
-              comment_id: context.payload.comment.id,
-              content: '-1'
-            });
-
       - name: Post failure comment
         if: needs.build.result == 'failure'
         uses: actions/github-script@v7
@@ -179,7 +130,7 @@ jobs:
         with:
           script: |
             const body = "⚠️ Preview build was cancelled\n\n" +
-              "This may have been caused by a newer build being triggered for the same PR.";
+              "This may have been caused by a newer commit being pushed to the same PR.";
 
             await github.rest.issues.createComment({
               owner: context.repo.owner,

Source/bmalloc/mimalloc/CMakeLists.txt

@@ -6,6 +6,10 @@ set(MI_BUILD_STATIC OFF CACHE BOOL "Build mimalloc static library" FORCE)
 set(MI_BUILD_OBJECT ON CACHE BOOL "Build mimalloc object library" FORCE)
 set(MI_BUILD_TESTS OFF CACHE BOOL "Build mimalloc tests" FORCE)
 
+if (USE_BUN_JSC_ADDITIONS)
+    set(MI_USE_CXX ON CACHE BOOL "Use C++ compiler for mimalloc (avoids GCC stdatomic.h incompatibility with clang)" FORCE)
+endif ()
+
 WEBKIT_INCLUDE_CONFIG_FILES_IF_EXISTS()
 
 add_subdirectory(mimalloc)

mac-release.bash

@@ -27,12 +27,7 @@ if [ -n "$ENABLE_SANITIZERS" ]; then
     export ENABLE_ASSERTS="ON";
 fi
 
-export DEFAULT_MALLOC_HEAP_BREAKDOWN=${DEFAULT_MALLOC_HEAP_BREAKDOWN:-"OFF"}
-if [ "$CMAKE_BUILD_TYPE" == "Debug" ]; then
-    export DEFAULT_MALLOC_HEAP_BREAKDOWN="ON"
-fi
-
-ENABLE_MALLOC_HEAP_BREAKDOWN=${ENABLE_MALLOC_HEAP_BREAKDOWN:-$DEFAULT_MALLOC_HEAP_BREAKDOWN}
+ENABLE_MALLOC_HEAP_BREAKDOWN=${ENABLE_MALLOC_HEAP_BREAKDOWN:-"OFF"}
 
 echo "Building for mac: [ $ENABLE_MALLOC_HEAP_BREAKDOWN ] [ $ENABLE_SANITIZERS ]"