ace-tool-rs 0.1.7 → 0.1.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ace-tool-rs",
-  "version": "0.1.7",
+  "version": "0.1.8",
   "description": "MCP server for codebase indexing, semantic search, and prompt enhancement",
   "repository": {
     "type": "git",

package/run.js

@@ -7,6 +7,12 @@ const https = require("https");
 const os = require("os");
 const crypto = require("crypto");
 
+// Note: We do not use 'kexec' to replace the process because the available packages 
+// are largely unmaintained or have compatibility issues with modern Node.js versions.
+// Instead, we spawn the child and forward signals/stdio.
+// Crucially, we ensure the wrapper ONLY writes to stderr (logs, extraction output)
+// so that the Rust binary's stdout (MCP JSON-RPC) is the only thing on stdout.
+
 const PACKAGE_NAME = "ace-tool-rs";
 const REPO_OWNER = "missdeer";
 const REPO_NAME = "ace-tool-rs";
@@ -203,7 +209,7 @@ async function getReleaseByTag(version) {
   } catch (error) {
     // If the specific version tag doesn't exist, fall back to latest
     if (error.message.includes("404")) {
-      console.log(
+      console.error(
         `Release v${version} not found, falling back to latest release...`
       );
       return getLatestRelease();
@@ -308,8 +314,9 @@ function downloadToFile(url, destPath, options = {}, redirectCount = 0) {
 
 async function extractTarGz(archivePath, destDir) {
   return new Promise((resolve, reject) => {
+    // Redirect stdout to stderr to prevent MCP corruption
     const tar = spawn("tar", ["-xzf", archivePath, "-C", destDir], {
-      stdio: "inherit",
+      stdio: ["ignore", process.stderr, process.stderr],
     });
     tar.on("close", (code) => {
       if (code === 0) resolve();
@@ -329,6 +336,7 @@ async function extractZip(archivePath, destDir) {
   return new Promise((resolve, reject) => {
     // Escape paths for PowerShell: escape backticks and single quotes
     const escapePath = (p) => p.replace(/`/g, "``").replace(/'/g, "''");
+    // Redirect stdout to stderr to prevent MCP corruption
     const unzipProcess = spawn(
       "powershell",
       [
@@ -338,7 +346,7 @@ async function extractZip(archivePath, destDir) {
         "-Command",
         `Expand-Archive -LiteralPath '${escapePath(archivePath)}' -DestinationPath '${escapePath(destDir)}' -Force`,
       ],
-      { stdio: "inherit" }
+      { stdio: ["ignore", process.stderr, process.stderr] }
     );
     unzipProcess.on("close", (code) => {
       if (code === 0) resolve();
@@ -416,7 +424,7 @@ async function downloadAndExtract(cacheDir) {
   // Try to acquire lock
   if (!acquireLock(lockPath)) {
     // Wait for other process to complete
-    console.log("Another process is downloading, waiting...");
+    console.error("Another process is downloading, waiting...");
     let attempts = 0;
     while (!fs.existsSync(binaryPath) && attempts < 60) {
       await new Promise((resolve) => setTimeout(resolve, 1000));
@@ -436,7 +444,7 @@ async function downloadAndExtract(cacheDir) {
 
     // Get release for the specific version (with retry)
     const version = getPackageVersion();
-    console.log(`Downloading ${PACKAGE_NAME} v${version}...`);
+    console.error(`Downloading ${PACKAGE_NAME} v${version}...`);
 
     const release = await withRetry(() => getReleaseByTag(version));
     const asset = release.assets.find((a) => a.name === assetName);
@@ -467,7 +475,7 @@ async function downloadAndExtract(cacheDir) {
     );
 
     // Extract to temporary directory
-    console.log("Extracting...");
+    console.error("Extracting...");
     fs.mkdirSync(tempExtractDir, { recursive: true });
 
     if (assetName.endsWith(".zip")) {
@@ -496,7 +504,7 @@ async function downloadAndExtract(cacheDir) {
     fs.unlinkSync(tempArchive);
     fs.rmSync(tempExtractDir, { recursive: true, force: true });
 
-    console.log(`Installed ${PACKAGE_NAME} to ${binaryPath}`);
+    console.error(`Installed ${PACKAGE_NAME} to ${binaryPath}`);
     return binaryPath;
   } catch (error) {
     console.error(`Failed to download ${PACKAGE_NAME}: ${error.message}`);