chore: Make npm scripts cross-platform compatible (#179)

carlhoerberg · claude · baelter · web-flow · commit 7e18dccf314f · 2025-11-28T08:26:31.000+01:00
* chore: Make npm scripts cross-platform compatible Replace Unix-specific shell commands with Node.js scripts to ensure compatibility across Windows, macOS, and Linux: - Replace `rm -rf` with fs.rmSync() in prebuild script - Replace `echo` redirect with fs.writeFileSync() in postbuild script - Replace macOS-specific `sed` command with Node.js file operations in version script All scripts use Node.js built-in APIs (fs, path) with no additional dependencies. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com> * fixup! chore: Make npm scripts cross-platform compatible * format --------- Co-authored-by: Claude <noreply@anthropic.com> Co-authored-by: Anders Bälter <anders@84codes.com>
diff --git a/package.json b/package.json
@@ -38,12 +38,12 @@
     "test-browser": "vitest --config ./vitest.config.browser.ts",
     "update-changelog": "node scripts/update-changelog.js",
     "create-tag": "node scripts/release-tag.js",
-    "prebuild": "rm -rf dist lib types",
+    "prebuild": "node scripts/clean.js",
     "build": "tsc && tsc --module commonjs --outDir lib/cjs && tsc --emitDeclarationOnly --removeComments false && rollup -c",
-    "postbuild": "echo '{\"type\": \"commonjs\"}' > lib/cjs/package.json",
+    "postbuild": "node scripts/write-cjs-package.js",
     "prepare": "npm run build",
     "preversion": "npm run test:local",
-    "version": "sed -i '' \"s/VERSION = .*/VERSION = \\\"$npm_package_version\\\"/\" src/amqp-base-client.ts && npm run update-changelog && npm run format  && git add -A",
+    "version": "node scripts/update-version.js && npm run update-changelog && npm run format && git add -A",
     "postversion": "npm run create-tag && git push && git push --tags",
     "release": "npm version patch --no-git-tag-version",
     "release:minor": "npm version minor --no-git-tag-version",
diff --git a/scripts/clean.js b/scripts/clean.js
@@ -0,0 +1,4 @@
+import { rmSync } from "fs"
+;["dist", "lib", "types"].forEach((dir) => {
+  rmSync(dir, { recursive: true, force: true })
+})
diff --git a/scripts/update-version.js b/scripts/update-version.js
@@ -0,0 +1,12 @@
+import { readFileSync, writeFileSync } from "fs"
+
+const version = process.env.npm_package_version
+if (!version) {
+  console.error("Error: npm_package_version not found")
+  process.exit(1)
+}
+
+const file = "src/amqp-base-client.ts"
+const content = readFileSync(file, "utf8")
+const updated = content.replace(/VERSION = ".*"/, `VERSION = "${version}"`)
+writeFileSync(file, updated)
diff --git a/scripts/write-cjs-package.js b/scripts/write-cjs-package.js
@@ -0,0 +1,6 @@
+import { writeFileSync, mkdirSync } from "fs"
+import { dirname } from "path"
+
+const path = "lib/cjs/package.json"
+mkdirSync(dirname(path), { recursive: true })
+writeFileSync(path, JSON.stringify({ type: "commonjs" }, null, 2) + "\n")
