feat: handle non-executable errors way better

Author: AlecDusheck

javascript/bin/bounty.js

@@ -32,6 +32,27 @@ if (!fs.existsSync(binaryPath)) {
     process.exit(1);
 }
 
+// Check executable permissions on Unix-like systems
+if (platform !== 'win32') {
+    try {
+        const stat = fs.statSync(binaryPath);
+        // Check if file is executable for user
+        if ((stat.mode & 0o100) === 0) {
+            try {
+                fs.chmodSync(binaryPath, 0o755);
+            } catch (error) {
+                console.error('\nError: Binary is not executable and could not set permissions automatically.');
+                console.error(`Please run: sudo chmod +x "${binaryPath}"\n`);
+                process.exit(1);
+            }
+        }
+    } catch (error) {
+        console.error('\nError: Could not check binary permissions.');
+        console.error(`Please run: sudo chmod +x "${binaryPath}"\n`);
+        process.exit(1);
+    }
+}
+
 // Execute the binary with the same arguments
 const result = spawnSync(binaryPath, process.argv.slice(2), {
     stdio: 'inherit'

javascript/package.json

@@ -3,14 +3,14 @@
   "version": "1.0.0",
   "description": "CLI tool for managing GitHub bounties",
   "bin": {
-    "bounty": "./bin/bounty.js",
-    "@bountybot/cli": "./bin/bounty.js"
+    "bounty": "./bin/bounty.js"
   },
   "scripts": {
     "prepare": "node scripts/build.js",
     "build": "node scripts/build.js",
-    "build:release": "node scripts/build.js --release"
-    },
+    "build:release": "node scripts/build.js --release",
+    "postinstall": "node scripts/postinstall.js"
+  },
   "files": [
     "bin",
     "scripts"
@@ -23,8 +23,10 @@
   "devDependencies": {
     "fs-extra": "^11.2.0"
   },
+  "dependencies": {
+    "fs-extra": "^11.2.0"
+  },
   "publishConfig": {
-    "access": "public",
-    "preserveSymlinks": true
+    "access": "public"
   }
 }