Remove unnecessary /tmp copy + chmod workaround for cursor-agent

[D-K-P]

Summary by CodeRabbit

• Improvements

  • Simplified binary execution by eliminating temporary file workaround and directly using bundled binary for improved native module compatibility.

• Documentation

  • Updated feature documentation to reflect bundled binary support approach.

[coderabbitai]

Walkthrough

The changes remove a temporary file workaround from binary execution by replacing the copy-to-/tmp-and-chmod pattern with direct execution of a bundled node binary. The README documentation is updated to reflect this new approach, and the implementation in extensions/cursor-cli.ts is simplified by removing file copying and permission modification operations while maintaining the same runtime behavior for NDJSON event streaming.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately reflects the main change: removing the /tmp workaround for binary execution and replacing it with direct bundled node usage.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch cursor-cli-demo-improvements

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (2)

cursor-cli-demo/extensions/cursor-cli.ts (2)

59-69: ⚠️ Potential issue | 🟡 Minor

Missing existence check for bundledNode.

entryPoint is validated on Line 62 but bundledNode is not. If the bundled node binary is absent, spawn will throw a generic ENOENT which is harder to debug than a descriptive error message like the one for entryPoint.

Proposed fix

   const entryPoint = `${CURSOR_AGENT_DIR}/index.js`;
   const bundledNode = `${CURSOR_AGENT_DIR}/node`;
 
+  if (!existsSync(bundledNode)) {
+    throw new Error(
+      `Bundled node binary not found at ${bundledNode}. Contents: ${existsSync(CURSOR_AGENT_DIR) ? readdirSync(CURSOR_AGENT_DIR).join(", ") : "N/A"}`,
+    );
+  }
+
   if (!existsSync(entryPoint)) {

---

24-25: ⚠️ Potential issue | 🟡 Minor

Stale comment: still references process.execPath.

Line 24 says "so we can invoke with process.execPath at runtime" but the code now uses the bundled node binary instead. Consider updating to match the new approach.

Proposed fix

-            // Copy the resolved index.js + deps to a fixed path so we can invoke with process.execPath at runtime
+            // Copy the resolved index.js + deps (including bundled node) to a fixed path for runtime invocation