editor-profile-sync 1.0.3 → 1.0.4

package/lib/editor-cli.js

@@ -59,14 +59,20 @@ export function getEditorCliPath(editor) {
   const cmd = process.platform === "win32" ? "where" : "which";
   try {
     const result = execSync(`${cmd} ${editor.cmd}`, { encoding: "utf-8" });
-    return result.trim().split("\n")[0];
+    const path = result.trim().split("\n")[0];
+    if (path) return path;
   } catch {
-    // On macOS, fall back to app bundle path
-    if (process.platform === "darwin") {
-      return findMacOSCliPath(editor);
-    }
-    return editor.cmd;
+    // Fall through to fallback options
+  }
+
+  // On macOS, fall back to app bundle path
+  if (process.platform === "darwin") {
+    const macPath = findMacOSCliPath(editor);
+    if (macPath) return macPath;
   }
+
+  // Final fallback: use command name (should work if in PATH)
+  return editor.cmd;
 }
 
 function escapeShellArg(arg) {
@@ -77,7 +83,9 @@ function escapeShellArg(arg) {
 function runEditorCmdAsync(editor, args, options = {}) {
   const { ignoreError = false } = options;
   const cliPath = getEditorCliPath(editor);
-  const cmdString = [cliPath, ...args.map(escapeShellArg)].join(" ");
+  const cmdString = [escapeShellArg(cliPath), ...args.map(escapeShellArg)].join(
+    " ",
+  );
 
   return new Promise((resolve, reject) => {
     const child = spawn(cmdString, [], {
@@ -134,6 +142,10 @@ export async function installExtension(editor, extensionId) {
 
 export function uninstallExtension(editor, extensionId) {
   const cliPath = getEditorCliPath(editor);
-  const cmd = [cliPath, "--uninstall-extension", extensionId].join(" ");
+  const cmd = [
+    escapeShellArg(cliPath),
+    "--uninstall-extension",
+    escapeShellArg(extensionId),
+  ].join(" ");
   execSync(cmd, { stdio: "ignore" });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "editor-profile-sync",
-  "version": "1.0.3",
+  "version": "1.0.4",
   "description": "Cross-platform extensions, settings.json, and snippets sync for VS Code-based editors",
   "type": "module",
   "main": "index.js",