@staticduo/opencode-scheduler 1.3.4 → 1.3.5

package/dist/index.js

@@ -12524,22 +12524,36 @@ function temporaryPath(path) {
 }
 function atomicReplace(path, content, mode, fileSystem) {
   const temporary = temporaryPath(path);
+  let primaryError;
   try {
     fileSystem.writeFile(temporary, content, { mode });
     fileSystem.chmod(temporary, mode);
     fileSystem.rename(temporary, path);
     fileSystem.chmod(path, mode);
+  } catch (error45) {
+    primaryError = error45;
+    throw error45;
   } finally {
-    removeNode(temporary, fileSystem);
+    if (primaryError)
+      bestEffort(() => removeNode(temporary, fileSystem));
+    else
+      removeNode(temporary, fileSystem);
   }
 }
 function atomicSymlink(path, target, fileSystem) {
   const temporary = temporaryPath(path);
+  let primaryError;
   try {
     fileSystem.symlink(target, temporary);
     fileSystem.rename(temporary, path);
+  } catch (error45) {
+    primaryError = error45;
+    throw error45;
   } finally {
-    removeNode(temporary, fileSystem);
+    if (primaryError)
+      bestEffort(() => removeNode(temporary, fileSystem));
+    else
+      removeNode(temporary, fileSystem);
   }
 }
 function removeNode(path, fileSystem) {
@@ -12662,12 +12676,34 @@ function acquireLock(lockRoot, timerUnit, fileSystem, overrides) {
     } catch (error45) {
       if (!isErrorCode(error45, "EEXIST"))
         throw error45;
+      let lockStats;
+      try {
+        lockStats = fileSystem.lstat(lockPath);
+      } catch (inspectError) {
+        if (isErrorCode(inspectError, "ENOENT"))
+          continue;
+        throw inspectError;
+      }
+      if (lockStats.isSymbolicLink()) {
+        throw new Error(`Refusing systemd install lock symlink at ${lockPath}`);
+      }
+      if (!lockStats.isDirectory()) {
+        throw new Error(`Refusing non-directory systemd install lock at ${lockPath}`);
+      }
       const metadata = readLockMetadata(lockPath, fileSystem);
-      const timestamp = metadata.timestamp ?? fileSystem.lstat(lockPath).mtimeMs;
+      const timestamp = metadata.timestamp ?? lockStats.mtimeMs;
       const oldEnough = options.now() - timestamp >= options.staleAfterMs;
       const ownerAlive = metadata.pid !== undefined && options.isPidAlive(metadata.pid);
       if (oldEnough && !ownerAlive) {
-        fileSystem.rm(lockPath, { recursive: true, force: true });
+        const quarantinePath = join(lockRoot, `${timerUnit}.stale-${options.pid}-${options.now()}-${temporaryFileSequence += 1}`);
+        try {
+          fileSystem.rename(lockPath, quarantinePath);
+        } catch (claimError) {
+          if (isErrorCode(claimError, "ENOENT") || isErrorCode(claimError, "EEXIST"))
+            continue;
+          throw claimError;
+        }
+        fileSystem.rm(quarantinePath, { recursive: true, force: true });
         continue;
       }
       if (options.now() - startedAt >= options.timeoutMs) {
@@ -12680,6 +12716,7 @@ function acquireLock(lockRoot, timerUnit, fileSystem, overrides) {
 function installSystemdUnits(request) {
   const fileSystem = request.fileSystem ?? defaultFileSystem;
   const releaseLock = acquireLock(request.lockDir ?? join(request.unitDir, ".opencode-scheduler-locks"), request.timerUnit, fileSystem, request.lock);
+  let primaryError;
   try {
     fileSystem.mkdir(request.unitDir, { recursive: true });
     const servicePath = join(request.unitDir, request.serviceUnit);
@@ -12709,8 +12746,14 @@ function installSystemdUnits(request) {
         bestEffort(() => request.run(`systemctl --user start ${request.timerUnit}`, { stdio: "ignore" }));
       throw error45;
     }
+  } catch (error45) {
+    primaryError = error45;
+    throw error45;
   } finally {
-    releaseLock();
+    if (primaryError)
+      bestEffort(releaseLock);
+    else
+      releaseLock();
   }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@staticduo/opencode-scheduler",
-  "version": "1.3.4",
+  "version": "1.3.5",
   "description": "OpenCode plugin for scheduling recurring jobs using launchd, systemd, Task Scheduler, or cron fallback",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",