ocx-cursor 0.3.0 → 0.3.2

package/bin/ocx-cursor.mjs

@@ -112,7 +112,7 @@ async function main() {
   }
   if (command === "update") {
     process.stdout.write("Updating OpenCodex Cursor Bridge from npm...\n");
-    updateService();
+    await updateService();
     return;
   }
   if (command === "sync") {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ocx-cursor",
-  "version": "0.3.0",
+  "version": "0.3.2",
   "description": "OpenCodex companion service for Cursor custom models",
   "keywords": [
     "opencodex",

package/src/install.mjs

@@ -1,7 +1,8 @@
 import { execFileSync } from "node:child_process";
 import { randomBytes } from "node:crypto";
-import { access, chmod, copyFile, cp, lstat, mkdir, readFile, readlink, rename, rm, symlink, writeFile } from "node:fs/promises";
-import { dirname, join, resolve } from "node:path";
+import { access, chmod, copyFile, cp, lstat, mkdir, mkdtemp, readFile, readlink, rename, rm, symlink, writeFile } from "node:fs/promises";
+import { tmpdir } from "node:os";
+import { dirname, join, relative, resolve, sep } from "node:path";
 import { fileURLToPath } from "node:url";
 import {
   cliLinkFile,
@@ -97,13 +98,18 @@ export async function prepareInstallSecret() {
   };
 }
 
+export function shouldCopyPackagePath(source, packageRoot = sourcePackageRoot) {
+  const segments = relative(packageRoot, source).split(sep);
+  return !segments.includes("node_modules") && !segments.includes(".git");
+}
+
 async function copyPackage() {
   if (resolve(sourcePackageRoot) === resolve(installedPackageRoot)) return;
   const temporary = `${installedPackageRoot}.tmp-${process.pid}`;
   await rm(temporary, { recursive: true, force: true });
   await cp(sourcePackageRoot, temporary, {
     recursive: true,
-    filter: (source) => !source.includes("/node_modules/") && !source.includes("/.git/"),
+    filter: (source) => shouldCopyPackagePath(source),
   });
   await rm(installedPackageRoot, { recursive: true, force: true });
   await rename(temporary, installedPackageRoot);
@@ -176,16 +182,22 @@ export async function installService() {
   return { installRoot, launchAgentFile, cliLinkFile, secretStatus };
 }
 
-export function updateService(options = {}) {
+export async function updateService(options = {}) {
   const execute = options.execFileSync || execFileSync;
-  execute(options.npmCommand || "npm", [
-    "exec",
-    "--yes",
-    "--package=ocx-cursor@latest",
-    "--",
-    "ocx-cursor",
-    "install",
-  ], { stdio: "inherit" });
+  const ownsTemporaryDirectory = !options.temporaryDirectory;
+  const temporaryDirectory = options.temporaryDirectory || await mkdtemp(join(tmpdir(), "ocx-cursor-update-"));
+  try {
+    execute(options.npmCommand || "npm", [
+      "exec",
+      "--yes",
+      "--package=ocx-cursor@latest",
+      "--",
+      "ocx-cursor",
+      "install",
+    ], { cwd: temporaryDirectory, stdio: "inherit" });
+  } finally {
+    if (ownsTemporaryDirectory) await rm(temporaryDirectory, { recursive: true, force: true });
+  }
 }
 
 export async function uninstallService() {