fix: Set executable permissions on ripgrep binaries at runtime (#267)

Zip archives don't reliably preserve Unix file permissions across platforms.
This causes EACCES errors when ripgrep binaries are extracted without
executable permissions on macOS/Linux.

Solution: ripgrep-wrapper.js now sets 0o755 permissions at runtime when
the module is first loaded, ensuring binaries are always executable
regardless of how they were extracted.

Fixes: spawn rg-aarch64-apple-darwin EACCES on macOS

Author: wonderwhy-er

scripts/build-mcpb.cjs

@@ -160,7 +160,9 @@ try {
         const src = path.join(ripgrepBinSrc, binary);
         const dest = path.join(ripgrepBinDest, binary);
         fs.copyFileSync(src, dest);
-        // Preserve executable permissions
+        // Set executable permissions (for development/testing)
+        // Note: Zip archives don't preserve permissions reliably, so ripgrep-wrapper.js
+        // also sets permissions at runtime to ensure compatibility after extraction
         if (!binary.endsWith('.exe')) {
             fs.chmodSync(dest, 0o755);
         }

scripts/ripgrep-wrapper.js

@@ -34,15 +34,32 @@ const binaryName = isWindows ? `rg-${target}.exe` : `rg-${target}`;
 // __dirname is lib/, so go up one level to reach bin/
 const rgPath = path.join(__dirname, '..', 'bin', binaryName);
 
-// Verify binary exists
+// Verify binary exists and ensure executable permissions
 if (!fs.existsSync(rgPath)) {
   // Try fallback to original rg location
   const fallbackPath = path.join(__dirname, '..', 'bin', isWindows ? 'rg.exe' : 'rg');
   if (fs.existsSync(fallbackPath)) {
+    // Ensure executable permissions on Unix systems
+    if (!isWindows) {
+      try {
+        fs.chmodSync(fallbackPath, 0o755);
+      } catch (err) {
+        // Ignore permission errors - might not have write access
+      }
+    }
     module.exports.rgPath = fallbackPath;
   } else {
     throw new Error(`ripgrep binary not found for platform ${target}: ${rgPath}`);
   }
 } else {
+  // Ensure executable permissions on Unix systems
+  // This fixes issues when extracting from zip archives that don't preserve permissions
+  if (!isWindows) {
+    try {
+      fs.chmodSync(rgPath, 0o755);
+    } catch (err) {
+      // Ignore permission errors - might not have write access
+    }
+  }
   module.exports.rgPath = rgPath;
 }