rskim 0.6.0 → 0.7.0

package/bin/skim.js

@@ -7,27 +7,82 @@ function getBinaryPath() {
   const platform = process.platform;
   const arch = process.arch;
 
-  let binaryName = 'skim';
-  if (platform === 'win32') {
-    binaryName = 'skim.exe';
+  // Define supported platforms and their binary paths
+  const supportedPlatforms = {
+    'linux-x64': 'bin/linux/x64/skim',
+    'linux-arm64': 'bin/linux/arm64/skim',
+    'darwin-x64': 'bin/darwin/x64/skim',
+    'darwin-arm64': 'bin/darwin/arm64/skim',
+    'win32-x64': 'bin/win32/x64/skim.exe',
+  };
+
+  const platformKey = `${platform}-${arch}`;
+  const binaryRelativePath = supportedPlatforms[platformKey];
+
+  // Check if platform is supported
+  if (!binaryRelativePath) {
+    console.error(`ERROR: Platform ${platform}-${arch} is not supported by the npm package.`);
+    console.error('');
+    console.error('Supported platforms:');
+    Object.keys(supportedPlatforms).forEach(p => {
+      console.error(`  - ${p}`);
+    });
+    console.error('');
+    console.error('Workaround: Install via Cargo instead:');
+    console.error('  cargo install rskim');
+    console.error('');
+    console.error('If you believe this platform should be supported, please report:');
+    console.error('  https://github.com/dean0x/skim/issues');
+    process.exit(1);
   }
 
-  // Try to find the binary in the package
-  const possiblePaths = [
-    path.join(__dirname, '..', 'bin', platform, arch, binaryName),
-    path.join(__dirname, '..', 'bin', binaryName),
-  ];
+  // Check if binary exists in package
+  const binaryPath = path.join(__dirname, '..', binaryRelativePath);
+  if (!fs.existsSync(binaryPath)) {
+    console.error('ERROR: Binary missing from npm package');
+    console.error(`Expected location: ${binaryPath}`);
+    console.error(`Platform: ${platformKey}`);
+    console.error('');
+    console.error('This is a packaging bug. Please report:');
+    console.error('  https://github.com/dean0x/skim/issues');
+    console.error('');
+    console.error('Workaround: Install via Cargo instead:');
+    console.error('  cargo install rskim');
+    process.exit(1);
+  }
+
+  // Quick check if binary is executable (catches glibc/musl issues)
+  try {
+    const testResult = spawnSync(binaryPath, ['--version'], {
+      timeout: 5000,
+      stdio: 'pipe'
+    });
+
+    if (testResult.error) {
+      console.error('ERROR: Binary exists but cannot be executed');
+      console.error(`Reason: ${testResult.error.message}`);
+      console.error('');
 
-  for (const p of possiblePaths) {
-    if (fs.existsSync(p)) {
-      return p;
+      if (testResult.error.code === 'ENOENT' || testResult.error.message.includes('libc')) {
+        console.error('This may be a C library incompatibility issue.');
+        console.error('Common cause: Alpine Linux (musl) vs glibc binary mismatch');
+        console.error('');
+        console.error('If you are using Alpine Linux, install via Cargo instead:');
+        console.error('  apk add cargo');
+        console.error('  cargo install rskim');
+      } else {
+        console.error('Workaround: Install via Cargo instead:');
+        console.error('  cargo install rskim');
+      }
+      console.error('');
+      console.error('For more help, see: https://github.com/dean0x/skim/issues');
+      process.exit(1);
     }
+  } catch (e) {
+    // Non-fatal, just proceed
   }
 
-  console.error('ERROR: skim binary not found for your platform');
-  console.error(`Platform: ${platform}, Architecture: ${arch}`);
-  console.error('Please report this issue at: https://github.com/dean0x/skim/issues');
-  process.exit(1);
+  return binaryPath;
 }
 
 const binary = getBinaryPath();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rskim",
-  "version": "0.6.0",
+  "version": "0.7.0",
   "description": "Smart code reader - streaming code transformation for AI agents",
   "bin": {
     "skim": "./bin/skim.js"