Discord comments

5 files changed, 130 insertions, 23 deletions

diff --git a/.github/workflows/bun-linux-build.yml b/.github/workflows/bun-linux-build.yml
index 2bc131baf..491fa7085 100644
--- a/.github/workflows/bun-linux-build.yml
+++ b/.github/workflows/bun-linux-build.yml
@@ -153,7 +153,12 @@ jobs:
           name: "Canary (${{github.sha}})"
           tag: "canary"
           artifacts: "${{runner.temp}}/release/bun-${{matrix.tag}}.zip,${{runner.temp}}/release/bun-${{matrix.tag}}-profile.zip"
-
+      - uses: sarisia/actions-status-discord@v1
+        if: always() && github.repository_owner == 'oven-sh' && github.event_name == 'pull_request'
+        with:
+          title: "Build ${{matrix.tag}} ${{job.status}} (${{github.sha}})"
+          webhook: ${{ secrets.DISCORD_WEBHOOK }}
+          status: ${{ job.status }}
   linux-test:
     name: Tests ${{matrix.tag}}
     runs-on: ubuntu-latest
@@ -181,8 +186,8 @@ jobs:
         with:
           name: bun-${{matrix.tag}}
           path: ${{runner.temp}}/release
-      - id: install
-        name: Install
+      - id: install-bun
+        name: Install Bun
         run: |
           cd ${{runner.temp}}/release
           unzip bun-${{matrix.tag}}.zip
@@ -190,6 +195,13 @@ jobs:
           chmod +x bun
           pwd >> $GITHUB_PATH
           ./bun --version
+      - id: install-dependnecies
+        name: Install dependencies
+        run: |
+          sudo apt-get update && sudo apt-get install -y openssl
+          bun install
+          bun install --cwd test
+          bun install --cwd packages/bun-internal-test
       - id: test
         name: Test (node runner)
         env:
@@ -198,11 +210,25 @@ jobs:
           TLS_POSTGRES_DATABASE_URL: ${{ secrets.TLS_POSTGRES_DATABASE_URL }}
         # if: ${{github.event.inputs.use_bun == 'false'}}
         run: |
-          sudo apt-get update && sudo apt-get install -y openssl
-          bun install
-          bun install --cwd test
-          bun install --cwd packages/bun-internal-test
           node packages/bun-internal-test/src/runner.node.mjs || true
+      - uses: sarisia/actions-status-discord@v1
+        if: always() && steps.test.outputs.failing_tests != '' && github.event_name == 'pull_request' && github.repository_owner == 'oven-sh'
+        with:
+          title: "Tests ${{matrix.tag}}"
+          webhook: ${{ secrets.DISCORD_WEBHOOK }}
+          status: ${{ job.status }}
+          description: |
+            ❌ @${{ github.actor }} there are ${{ steps.test.outputs.failing_tests_count }} files with test failures on ${{ matrix.tag }}:
+
+            ${{ steps.test.outputs.failing_tests }}
+
+            **[View test output](https://github.com/oven-sh/bun/actions/runs/${{github.run_id}})**
+      - uses: sarisia/actions-status-discord@v1
+        if: always() && steps.test.outputs.failing_tests == '' && github.event_name == 'pull_request' && github.repository_owner == 'oven-sh'
+        with:
+          title: "Tests ${{matrix.tag}}"
+          webhook: ${{ secrets.DISCORD_WEBHOOK }}
+          status: ${{ job.status }}
       - name: Comment on PR
         if: steps.test.outputs.failing_tests != '' && github.event_name == 'pull_request'
         uses: thollander/actions-comment-pull-request@v2
diff --git a/.github/workflows/bun-mac-aarch64.yml b/.github/workflows/bun-mac-aarch64.yml
index 4bc4d1494..aadb619da 100644
--- a/.github/workflows/bun-mac-aarch64.yml
+++ b/.github/workflows/bun-mac-aarch64.yml
@@ -393,6 +393,12 @@ jobs:
           name: "Canary (${{github.sha}})"
           tag: "canary"
           artifacts: "${{runner.temp}}/release/${{matrix.tag}}.zip,${{runner.temp}}/release/${{matrix.tag}}-profile.zip"
+      - uses: sarisia/actions-status-discord@v1
+        if: always() && github.repository_owner == 'oven-sh' && github.event_name == 'pull_request'
+        with:
+          title: "Build ${{matrix.tag}} ${{job.status}} (${{github.sha}})"
+          webhook: ${{ secrets.DISCORD_WEBHOOK }}
+          status: ${{ job.status }}
   macOS-test:
     name: Tests ${{matrix.tag}}
     runs-on: ${{ matrix.runner }}
@@ -420,8 +426,8 @@ jobs:
         with:
           name: ${{matrix.tag}}
           path: ${{runner.temp}}/release
-      - id: install
-        name: Install
+      - id: install-bun
+        name: Install Bun
         run: |
           cd ${{runner.temp}}/release
           unzip ${{matrix.tag}}.zip
@@ -429,6 +435,12 @@ jobs:
           chmod +x bun
           pwd >> $GITHUB_PATH
           ./bun --version
+      - id: install
+        name: Install dependencies
+        run: |
+          bun install
+          bun install --cwd=test
+          bun install --cwd=packages/bun-internal-test
       - id: test
         name: Test (node runner)
         env:
@@ -437,10 +449,25 @@ jobs:
           TLS_POSTGRES_DATABASE_URL: ${{ secrets.TLS_POSTGRES_DATABASE_URL }}
         # if: ${{github.event.inputs.use_bun == 'false'}}
         run: |
-          bun install
-          bun install --cwd test
-          bun install --cwd packages/bun-internal-test
           node packages/bun-internal-test/src/runner.node.mjs || true
+      - uses: sarisia/actions-status-discord@v1
+        if: always() && steps.test.outputs.failing_tests != '' && github.event_name == 'pull_request'
+        with:
+          title: "Tests ${{matrix.tag}}"
+          webhook: ${{ secrets.DISCORD_WEBHOOK }}
+          status: ${{ job.status }}
+          description: |
+            Hey @${{ github.actor }} there are ${{ steps.test.outputs.failing_tests_count }} files with test failures on ${{ matrix.tag }}:
+
+            ${{ steps.test.outputs.failing_tests }}
+
+            **[View test output](https://github.com/oven-sh/bun/actions/runs/${{github.run_id}})**
+      - uses: sarisia/actions-status-discord@v1
+        if: always() && steps.test.outputs.failing_tests == '' && github.event_name == 'pull_request'
+        with:
+          title: "Tests ${{matrix.tag}}"
+          webhook: ${{ secrets.DISCORD_WEBHOOK }}
+          status: ${{ job.status }}
       - name: Comment on PR
         if: steps.test.outputs.failing_tests != '' && github.event_name == 'pull_request'
         uses: thollander/actions-comment-pull-request@v2
diff --git a/.github/workflows/bun-mac-x64-baseline.yml b/.github/workflows/bun-mac-x64-baseline.yml
index cf3281bd8..3aea31e25 100644
--- a/.github/workflows/bun-mac-x64-baseline.yml
+++ b/.github/workflows/bun-mac-x64-baseline.yml
@@ -397,6 +397,12 @@ jobs:
           name: "Canary (${{github.sha}})"
           tag: "canary"
           artifacts: "${{runner.temp}}/release/${{matrix.tag}}.zip,${{runner.temp}}/release/${{matrix.tag}}-profile.zip"
+      - uses: sarisia/actions-status-discord@v1
+        if: always() && github.repository_owner == 'oven-sh' && github.event_name == 'pull_request'
+        with:
+          title: "Build ${{matrix.tag}} ${{job.status}} (${{github.sha}})"
+          webhook: ${{ secrets.DISCORD_WEBHOOK }}
+          status: ${{ job.status }}
   macOS-test:
     name: Tests ${{matrix.tag}}
     runs-on: ${{ matrix.runner }}
@@ -424,8 +430,8 @@ jobs:
         with:
           name: ${{matrix.tag}}
           path: ${{runner.temp}}/release
-      - id: install
-        name: Install
+      - id: install-bun
+        name: Install Bun
         run: |
           cd ${{runner.temp}}/release
           unzip ${{matrix.tag}}.zip
@@ -433,6 +439,12 @@ jobs:
           chmod +x bun
           pwd >> $GITHUB_PATH
           ./bun --version
+      - id: install
+        name: Install dependencies
+        run: |
+          bun install
+          bun install --cwd=test
+          bun install --cwd=packages/bun-internal-test
       - id: test
         name: Test (node runner)
         env:
@@ -441,10 +453,25 @@ jobs:
           TLS_POSTGRES_DATABASE_URL: ${{ secrets.TLS_POSTGRES_DATABASE_URL }}
         # if: ${{github.event.inputs.use_bun == 'false'}}
         run: |
-          bun install
-          bun install --cwd test
-          bun install --cwd packages/bun-internal-test
           node packages/bun-internal-test/src/runner.node.mjs || true
+      - uses: sarisia/actions-status-discord@v1
+        if: always() && steps.test.outputs.failing_tests != '' && github.event_name == 'pull_request'
+        with:
+          title: "Tests ${{matrix.tag}}"
+          webhook: ${{ secrets.DISCORD_WEBHOOK }}
+          status: ${{ job.status }}
+          description: |
+            Hey @${{ github.actor }} there are ${{ steps.test.outputs.failing_tests_count }} files with test failures on ${{ matrix.tag }}:
+
+            ${{ steps.test.outputs.failing_tests }}
+
+            **[View test output](https://github.com/oven-sh/bun/actions/runs/${{github.run_id}})**
+      - uses: sarisia/actions-status-discord@v1
+        if: always() && steps.test.outputs.failing_tests == '' && github.event_name == 'pull_request'
+        with:
+          title: "Tests ${{matrix.tag}}"
+          webhook: ${{ secrets.DISCORD_WEBHOOK }}
+          status: ${{ job.status }}
       - name: Comment on PR
         if: steps.test.outputs.failing_tests != '' && github.event_name == 'pull_request'
         uses: thollander/actions-comment-pull-request@v2
diff --git a/.github/workflows/bun-mac-x64.yml b/.github/workflows/bun-mac-x64.yml
index a1d8b057a..58dd50a7f 100644
--- a/.github/workflows/bun-mac-x64.yml
+++ b/.github/workflows/bun-mac-x64.yml
@@ -399,6 +399,12 @@ jobs:
           name: "Canary (${{github.sha}})"
           tag: "canary"
           artifacts: "${{runner.temp}}/release/${{matrix.tag}}.zip,${{runner.temp}}/release/${{matrix.tag}}-profile.zip"
+      - uses: sarisia/actions-status-discord@v1
+        if: always() && github.repository_owner == 'oven-sh' && github.event_name == 'pull_request'
+        with:
+          title: "Build ${{matrix.tag}} ${{job.status}} (${{github.sha}})"
+          webhook: ${{ secrets.DISCORD_WEBHOOK }}
+          status: ${{ job.status }}
   macOS-test:
     name: Tests ${{matrix.tag}}
     runs-on: ${{ matrix.runner }}
@@ -426,8 +432,8 @@ jobs:
         with:
           name: ${{matrix.tag}}
           path: ${{runner.temp}}/release
-      - id: install
-        name: Install
+      - id: install-bun
+        name: Install Bun
         run: |
           cd ${{runner.temp}}/release
           unzip ${{matrix.tag}}.zip
@@ -435,6 +441,12 @@ jobs:
           chmod +x bun
           pwd >> $GITHUB_PATH
           ./bun --version
+      - id: install
+        name: Install dependencies
+        run: |
+          bun install
+          bun install --cwd=test
+          bun install --cwd=packages/bun-internal-test
       - id: test
         name: Test (node runner)
         env:
@@ -443,10 +455,25 @@ jobs:
           TLS_POSTGRES_DATABASE_URL: ${{ secrets.TLS_POSTGRES_DATABASE_URL }}
         # if: ${{github.event.inputs.use_bun == 'false'}}
         run: |
-          bun install
-          bun install --cwd test
-          bun install --cwd packages/bun-internal-test
           node packages/bun-internal-test/src/runner.node.mjs || true
+      - uses: sarisia/actions-status-discord@v1
+        if: always() && steps.test.outputs.failing_tests != '' && github.event_name == 'pull_request'
+        with:
+          title: "Tests ${{matrix.tag}}"
+          webhook: ${{ secrets.DISCORD_WEBHOOK }}
+          status: ${{ job.status }}
+          description: |
+            Hey @${{ github.actor }} there are ${{ steps.test.outputs.failing_tests_count }} files with test failures on ${{ matrix.tag }}:
+
+            ${{ steps.test.outputs.failing_tests }}
+
+            **[View test output](https://github.com/oven-sh/bun/actions/runs/${{github.run_id}})**
+      - uses: sarisia/actions-status-discord@v1
+        if: always() && steps.test.outputs.failing_tests == '' && github.event_name == 'pull_request'
+        with:
+          title: "Tests ${{matrix.tag}}"
+          webhook: ${{ secrets.DISCORD_WEBHOOK }}
+          status: ${{ job.status }}
       - name: Comment on PR
         if: steps.test.outputs.failing_tests != '' && github.event_name == 'pull_request'
         uses: thollander/actions-comment-pull-request@v2
diff --git a/test/js/node/fs/cp.test.ts b/test/js/node/fs/cp.test.ts
index eae8a0e77..33b9aa700 100644
--- a/test/js/node/fs/cp.test.ts
+++ b/test/js/node/fs/cp.test.ts
@@ -12,7 +12,7 @@ for (const [name, copy] of impls) {
     try {
       await (copy as any)(...args);
     } catch (e: any) {
-      if (e?.code?.toUpperCase() === "TODO") {
+      if ((e?.code? ?? "").toUpperCase() === "TODO") {
         throw new Error("Expected " + name + "() to throw non TODO error");
       }
       return e;