ci: 👷 Use Bun version with writeStream /fd/ fix

oven-sh/bun#4378 fixed issue causing CI to fail when running changeset

Author: aMediocreDad

.github/workflows/pr.yml

@@ -6,6 +6,8 @@ on:
       - main
   workflow_dispatch:
 
+concurrency: ${{ github.workflow }}-${{ github.ref }}
+
 jobs:
   test:
     name: "Pull Request Linting"
@@ -20,6 +22,7 @@ jobs:
       - name: "Install Dependencies"
         run: bun install
       - name: "Run Linter"
-        run: bun run lint
-      - name: "Format Code"
-        run: bun run format
+        run: bun run test
+      - name: "Test"
+      # remove after Bun test done
+        run: bun run test-script

.github/workflows/publish.yml

@@ -12,27 +12,14 @@ jobs:
 
     steps:
       - uses: actions/checkout@v3
-      - uses: oven-sh/setup-bun@v1
-      - uses: actions/setup-node@v3
-        with:
-          node-version: '*'
-      - name: Install Dependencies
-        run: bun install
-      - name: Lint
-        run: bun lint
-      - name: Build
-        run: bun run build
-
       - name: Package
         run: |
           zip -r forbidden-lands.zip assets fonts lang templates forbidden-lands.js forbidden-lands.css system.json template.json CHANGELOG.md CONTRIBUTING.md LICENSE README.md
-      - name: 🚀 Update Release with Artifacts
+      - name: Update Release with Artifacts
         uses: ncipollo/[email protected]
         with:
           allowUpdates: true
           name: ${{ github.event.release.name }}
-          draft: false
-          prerelease: false
           tag: ${{ github.event.release.tag_name }}
           body: ${{ github.event.release.body }}
           artifacts: './forbidden-lands.zip'

.github/workflows/release.yml

@@ -14,16 +14,13 @@ jobs:
     steps:
       - uses: actions/checkout@v3
       - uses: oven-sh/setup-bun@v1
-      - uses: actions/setup-node@v3
         with:
-          node-version: '*'
+          bun-version: canary
       - name: Install Dependencies
         run: bun install
-      - name: Lint
-        run: bun lint
       - name: Build
         run: bun run build
-      - name: Create Release Pull Request
+      - name: Create Release or Version
         uses: changesets/action@v1
         with:
           commit: 'chore(release): Version Package'

bun.lockb

@@ -24,12 +24,13 @@
 		"format": "rome format --write .",
 		"lint": "bun run test && rome check --apply .",
 		"postinstall": "lefthook install && bun run setup",
-		"release": "node tools/release.js",
+		"release": "bun tools/release.js",
 		"setup": "bun tools/setup.ts",
 		"start": "bun run build --development",
 		"test": "bun run test:types && bunx rome ci .",
 		"test:types": "bunx tsc && bunx tsc -p ./src/tsconfig.json",
-		"version": "node tools/version.js"
+		"version": "bun tools/version.js",
+		"test-script": "bunx changeset version && bunx changeset tag"
 	},
 	"dependencies": {
 		"rpg-awesome": "0.2.0",
@@ -43,7 +44,7 @@
 		"@types/jquery": "3.5.17",
 		"@types/node": "^20.5.7",
 		"@types/showdown": "2.0.1",
-		"bun": "0.8.1",
+		"bun": "canary",
 		"devmoji": "2.3.0",
 		"esbuild": "0.19.2",
 		"esbuild-sass-plugin": "2.13.0",

package.json

@@ -1,5 +1,3 @@
-// Use Bun when Bun supports node:fs/promises fully
-
 import { $, execa } from "execa";
 
 import rootConfig from "../package.json" assert { type: "json" };
@@ -29,8 +27,8 @@ if (exitCode !== 2) {
 
 await $`git checkout --detach`;
 await $`git add --force forbidden-lands.js forbidden-lands.css`;
-await execa("git", ["commit", "-m", "chore(release): :rocket: ${tag}"]);
+await execa("git", ["commit", "-m", `chore(release): ${tag}`]);
 
-await $`npx changeset tag`;
+await $`bunx changeset tag`;
 
 await $`git push --force --follow-tags origin HEAD:refs/heads/${releaseLine}`;

tools/release.js renamed to tools/release.ts

@@ -6,14 +6,14 @@ import { writeFile } from "node:fs/promises";
 /**
  * Convenience function to handle an stderr from execa.
  */
-function handlePossibleError({ stderr }) {
+function handlePossibleError({ stderr }: { stderr: string }) {
 	if (!stderr) return;
 	console.error(stderr);
 	process.exit(1);
 }
 
 // Version package
-await $`npx changeset version`.then(handlePossibleError);
+await $`bunx changeset version`.then(handlePossibleError);
 
 // Import versioned package.json
 const {
@@ -33,4 +33,4 @@ manifest.download = manifest.download.replace(/v\d+\.\d+\.\d+/, `v${version}`);
 await writeFile("system.json", JSON.stringify(manifest, null, "\t") + "\n");
 
 // Format system.json
-await $`npx rome format --write system.json`.then(handlePossibleError);
+await $`bunx rome format --write system.json`.then(handlePossibleError);

tools/version.js renamed to tools/version.ts

@@ -10,6 +10,6 @@
 		"lib": ["ESNext"],
 		"types": []
 	},
-	"include": ["./**/*.ts", "tools/version.js", "tools/release.js"],
+	"include": ["./**/*.ts"],
 	"exclude": ["node_modules", "src"]
 }