From 9fd30715980b16697473198466c28c14a9e04b67 Mon Sep 17 00:00:00 2001
From: Mattt Zmuda <mattt@replicate.com>
Date: Mon, 18 Mar 2024 04:39:52 -0700
Subject: [PATCH 1/2] Extract tarball generation into separate build job

---
 .github/workflows/ci.yml | 54 ++++++++++++++++++++++++++++------------
 1 file changed, 38 insertions(+), 16 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index c6ca04e..b6bd576 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -28,8 +28,32 @@ jobs:
       - run: npm run lint
 
 
+  # Build a production tarball and use it to run the integration
+  build:
+    runs-on: ubuntu-latest
+
+    outputs:
+      tarball-name: ${{ steps.pack.outputs.tarball-name }}
+
+    steps:
+      - uses: actions/checkout@v3
+      - name: Use Node.js
+        uses: actions/setup-node@v3
+        with:
+          node-version: 20.x
+          cache: "npm"
+      - name: Build tarball
+        id: pack
+        run: |
+          echo "::set-output name=tarball-name::$(npm --loglevel error pack)"
+      - uses: actions/upload-artifact@v3
+        with:
+          name: package-tarball
+          path: ${{ steps.pack.outputs.tarball-name }}
+
+
   integration-node:
-    needs: test
+    needs: [test, build]
     runs-on: ubuntu-latest
 
     env:
@@ -47,21 +71,22 @@ jobs:
 
     steps:
       - uses: actions/checkout@v3
+      - uses: actions/download-artifact@v3
+        with:
+          name: package-tarball
       - name: Use Node.js ${{ matrix.node-version }}
         uses: actions/setup-node@v3
         with:
           node-version: ${{ matrix.node-version }}
           cache: "npm"
-      # Build a production tarball and run the integration tests against it.
       - run: |
-          PKG_TARBALL=$(npm --loglevel error pack)
           npm --prefix integration/${{ matrix.suite }} install
-          npm --prefix integration/${{ matrix.suite }} install "file:/./$PKG_TARBALL"
+          npm --prefix integration/${{ matrix.suite }} install "./${{ needs.build.outputs.tarball-name }}"
           npm --prefix integration/${{ matrix.suite }} test
 
 
   integration-edge:
-    needs: test
+    needs: [test, build]
     runs-on: ubuntu-latest
 
     env:
@@ -74,22 +99,23 @@ jobs:
 
     steps:
       - uses: actions/checkout@v3
+      - uses: actions/download-artifact@v3
+        with:
+          name: package-tarball
       - name: Use Node.js ${{ matrix.node-version }}
         uses: actions/setup-node@v3
         with:
           node-version: ${{ matrix.node-version }}
           cache: "npm"
-      # Build a production tarball and run the integration tests against it.
       - run: |
           test "${{ matrix.suite }}" = "cloudflare-worker" && echo "REPLICATE_API_TOKEN=${{ secrets.REPLICATE_API_TOKEN }}" > integration/${{ matrix.suite }}/.dev.vars
-          PKG_TARBALL=$(npm --loglevel error pack)
           npm --prefix integration/${{ matrix.suite }} install
-          npm --prefix integration/${{ matrix.suite }} install "file:/./$PKG_TARBALL"
+          npm --prefix integration/${{ matrix.suite }} install "./${{ needs.build.outputs.tarball-name }}"
           npm --prefix integration/${{ matrix.suite }} test
 
 
   integration-bun:
-    needs: test
+    needs: [test, build]
     runs-on: ubuntu-latest
 
     env:
@@ -98,23 +124,19 @@ jobs:
     strategy:
       matrix:
         bun-version: [1.0.11]
-        node-version: [20.x]
         suite: [bun]
 
     steps:
       - uses: actions/checkout@v3
-      - name: Use Node.js ${{ matrix.node-version }}
-        uses: actions/setup-node@v3
+      - uses: actions/download-artifact@v3
         with:
-          node-version: ${{ matrix.node-version }}
-          cache: "npm"
+          name: package-tarball
       - name: Use Bun ${{ matrix.bun-version }}
         uses: oven-sh/setup-bun@v1
         with:
           bun-version: ${{ matrix.bun-version }}
       - run: |
-          PKG_TARBALL=$(npm --loglevel error pack)
           cd integration/${{ matrix.suite }}
           bun uninstall replicate
-          bun install "file:../../$PKG_TARBALL"
+          bun install "file:../../${{ needs.build.outputs.tarball-name }}"
           bun test

From a7f98f35a922b87298c805994356dc8ec6318b40 Mon Sep 17 00:00:00 2001
From: Mattt Zmuda <mattt@replicate.com>
Date: Mon, 18 Mar 2024 04:43:55 -0700
Subject: [PATCH 2/2] Replace use of deprecated ::set-output

See https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
---
 .github/workflows/ci.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index b6bd576..b3dbba4 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -45,7 +45,7 @@ jobs:
       - name: Build tarball
         id: pack
         run: |
-          echo "::set-output name=tarball-name::$(npm --loglevel error pack)"
+          echo "tarball-name=$(npm --loglevel error pack)" >> $GITHUB_OUTPUT
       - uses: actions/upload-artifact@v3
         with:
           name: package-tarball