@uipath/maestro-builder-sdk 6.18.8 → 6.18.9

package/dist/registry/pull-cli.d.ts

@@ -20,4 +20,25 @@ export declare const CONNECTORS_LIBRARY_VERSION_ENV = "CONNECTORS_LIBRARY_VERSIO
  * rather than guessed at, so a typo cannot fall back to "latest" silently.
  */
 export declare function connectorsLibraryUrl(argv: string[], env?: Record<string, string | undefined>): string;
+/**
+ * Point `current` at an entry, replacing whatever it named.
+ *
+ * Nothing to do when `current` already resolves to the entry — the unchanged
+ * pull, which is most of them.
+ *
+ * Otherwise the new link is made under a temp name and renamed over `current`.
+ * On POSIX that rename is atomic over an existing symlink, so a concurrent
+ * compile never finds the link missing. On Windows `current` is a junction, and
+ * a rename onto an existing junction fails with `EPERM` (flow-builder-sdk#945),
+ * so there the old link is parked under `current.old-<pid>` first, the new one
+ * renamed in, and the parked one removed; a failed rename puts the parked link
+ * back. That leaves a moment where `current` does not exist, which Windows
+ * gives no way to avoid with a junction.
+ *
+ * Links are removed with `unlink`, never a recursive `rm`: the link's target is
+ * the extracted library, and removing the link must not reach into it.
+ *
+ * @param platform - Which branch to take; a parameter so both run in tests.
+ */
+export declare function pointCurrentAt(root: string, sha256: string, platform?: NodeJS.Platform): Promise<void>;
 export declare function run(argv: string[]): Promise<number>;

package/dist/registry/pull-cli.js

@@ -1,6 +1,6 @@
 import { reportResult } from '../cli-result.js';
 import { createHash } from "node:crypto";
-import { mkdir, readFile, rename, rm, stat, symlink, writeFile, } from "node:fs/promises";
+import { mkdir, readFile, realpath, rename, rm, stat, symlink, unlink, writeFile, } from "node:fs/promises";
 import { join, resolve } from "node:path";
 import { writeJsonFile } from "../core/json-file.js";
 import { CURRENT_LINK, LIBRARY_JSON_DIR, LIBRARY_MARKDOWN_DIR, cacheEntry, libraryCacheRoot, referencedConnectorKeys, } from "../core/library-cache.js";
@@ -215,16 +215,81 @@ async function writeSource(libraryPath, archive, sha256, archiveSha256, overlayS
 /**
  * Point `current` at an entry, replacing whatever it named.
  *
- * Written to a temp name and renamed, so a reader never sees the link missing —
- * `rename` over an existing symlink is atomic, while unlink-then-create leaves a
- * window where a concurrent compile finds nothing.
+ * Nothing to do when `current` already resolves to the entry — the unchanged
+ * pull, which is most of them.
+ *
+ * Otherwise the new link is made under a temp name and renamed over `current`.
+ * On POSIX that rename is atomic over an existing symlink, so a concurrent
+ * compile never finds the link missing. On Windows `current` is a junction, and
+ * a rename onto an existing junction fails with `EPERM` (flow-builder-sdk#945),
+ * so there the old link is parked under `current.old-<pid>` first, the new one
+ * renamed in, and the parked one removed; a failed rename puts the parked link
+ * back. That leaves a moment where `current` does not exist, which Windows
+ * gives no way to avoid with a junction.
+ *
+ * Links are removed with `unlink`, never a recursive `rm`: the link's target is
+ * the extracted library, and removing the link must not reach into it.
+ *
+ * @param platform - Which branch to take; a parameter so both run in tests.
  */
-async function pointCurrentAt(root, sha256) {
+export async function pointCurrentAt(root, sha256, platform = process.platform) {
     const link = join(root, CURRENT_LINK);
+    if (await resolvesTo(link, join(root, sha256)))
+        return;
     const temporary = `${link}.tmp-${process.pid}`;
-    await rm(temporary, { recursive: true, force: true });
-    await symlink(sha256, temporary, "junction");
-    await rename(temporary, link);
+    await removeLink(temporary);
+    try {
+        await symlink(sha256, temporary, "junction");
+        if (platform !== "win32") {
+            await rename(temporary, link);
+            return;
+        }
+        const parked = `${link}.old-${process.pid}`;
+        await removeLink(parked);
+        let wasParked = false;
+        try {
+            await rename(link, parked);
+            wasParked = true;
+        }
+        catch (error) {
+            if (error.code !== "ENOENT")
+                throw error;
+        }
+        try {
+            await rename(temporary, link);
+        }
+        catch (error) {
+            if (wasParked)
+                await rename(parked, link).catch(() => undefined);
+            throw error;
+        }
+        if (wasParked)
+            await removeLink(parked);
+    }
+    finally {
+        // Gone already after a successful rename; this is for every way out that
+        // is not one, so a failed pull leaves no `current.tmp-<pid>` behind.
+        await removeLink(temporary);
+    }
+}
+/** Whether `link` and `entry` name the same directory; false when either is missing. */
+async function resolvesTo(link, entry) {
+    try {
+        return (await realpath(link)) === (await realpath(entry));
+    }
+    catch {
+        return false;
+    }
+}
+/** Remove a symlink or junction without following it; a missing one is fine. */
+async function removeLink(path) {
+    try {
+        await unlink(path);
+    }
+    catch (error) {
+        if (error.code !== "ENOENT")
+            throw error;
+    }
 }
 export async function run(argv) {
     if (has(argv, "--help") || has(argv, "-h")) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@uipath/maestro-builder-sdk",
-  "version": "6.18.8",
+  "version": "6.18.9",
   "description": "Build UiPath Flow, Case, and BPMN artifacts by writing TypeScript.",
   "license": "MIT",
   "homepage": "https://docs.uipath.com/maestro",
@@ -92,5 +92,5 @@
     "@types/node": "^22.7.0",
     "esbuild": "^0.28.1"
   },
-  "gitref": "ce161efe130f6a6cdee895d8a7fab2faddcec0fd"
+  "gitref": "6f8e3c88230704c5b5bc0e8983be45ebedd135dc"
 }