lalph 0.1.69 → 0.1.70

package/dist/cli.mjs

@@ -141989,6 +141989,7 @@ var Prd = class extends Service()("lalph/Prd", { make: gen(function* () {
 		const yaml = yield* fs.readFileString(prdFile);
 		return PrdIssue.arrayFromYaml(yaml);
 	});
+	const syncSemaphore = makeSemaphoreUnsafe(1);
 	const mergableGithubPrs = gen(function* () {
 		const updated = yield* readPrd;
 		const prs = empty$11();
@@ -142008,7 +142009,7 @@ var Prd = class extends Service()("lalph/Prd", { make: gen(function* () {
 			issueId: issue.id,
 			state: "todo"
 		});
-	});
+	}, syncSemaphore.withPermit);
 	const mergeConflictInstruction = "Next step: Rebase PR and resolve merge conflicts.";
 	const flagUnmergable = fnUntraced(function* (options) {
 		const issue = current.find((entry) => entry.id === options.issueId);
@@ -142048,7 +142049,7 @@ var Prd = class extends Service()("lalph/Prd", { make: gen(function* () {
 	let current = yield* source.issues;
 	yield* fs.writeFileString(prdFile, PrdIssue.arrayToYaml(current));
 	const updatedIssues = /* @__PURE__ */ new Map();
-	const sync$2 = gen(function* () {
+	const sync$2 = syncSemaphore.withPermit(gen(function* () {
 		const updated = yield* readPrd;
 		if (!(updated.length !== current.length || updated.some((u, i) => u.isChangedComparedTo(current[i])))) return;
 		const githubPrs = /* @__PURE__ */ new Map();
@@ -142081,7 +142082,7 @@ var Prd = class extends Service()("lalph/Prd", { make: gen(function* () {
 				githubPrNumber: prNumber
 			});
 		})));
-	}).pipe(uninterruptible);
+	}).pipe(uninterruptible));
 	const updateSyncHandle = yield* make$25();
 	const updateSync = gen(function* () {
 		const tempFile = yield* fs.makeTempFileScoped();
@@ -142101,7 +142102,7 @@ var Prd = class extends Service()("lalph/Prd", { make: gen(function* () {
 		path: prdFile,
 		mergableGithubPrs,
 		maybeRevertIssue,
-		revertUpdatedIssues: gen(function* () {
+		revertUpdatedIssues: syncSemaphore.withPermit(gen(function* () {
 			for (const issue of updatedIssues.values()) {
 				if (issue.state === "done") continue;
 				yield* source.updateIssue({
@@ -142109,7 +142110,7 @@ var Prd = class extends Service()("lalph/Prd", { make: gen(function* () {
 					state: "todo"
 				});
 			}
-		}),
+		})),
 		flagUnmergable,
 		findById
 	};
@@ -142442,7 +142443,7 @@ const commandSource = make$27("source").pipe(withDescription("Select the issue s
 
 //#endregion
 //#region package.json
-var version = "0.1.69";
+var version = "0.1.70";
 
 //#endregion
 //#region src/cli.ts

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "lalph",
   "type": "module",
-  "version": "0.1.69",
+  "version": "0.1.70",
   "publishConfig": {
     "access": "public"
   },

package/src/Prd.ts

@@ -53,6 +53,8 @@ export class Prd extends ServiceMap.Service<
       return PrdIssue.arrayFromYaml(yaml)
     })
 
+    const syncSemaphore = Effect.makeSemaphoreUnsafe(1)
+
     const mergableGithubPrs = Effect.gen(function* () {
       const updated = yield* readPrd
       const prs = Array.empty<{ issueId: string; prNumber: number }>()
@@ -76,7 +78,7 @@ export class Prd extends ServiceMap.Service<
         issueId: issue.id!,
         state: "todo",
       })
-    })
+    }, syncSemaphore.withPermit)
 
     const mergeConflictInstruction =
       "Next step: Rebase PR and resolve merge conflicts."
@@ -134,64 +136,66 @@ export class Prd extends ServiceMap.Service<
 
     const updatedIssues = new Map<string, PrdIssue>()
 
-    const sync = Effect.gen(function* () {
-      const updated = yield* readPrd
-      const anyChanges =
-        updated.length !== current.length ||
-        updated.some((u, i) => u.isChangedComparedTo(current[i]!))
-      if (!anyChanges) {
-        return
-      }
+    const sync = syncSemaphore.withPermit(
+      Effect.gen(function* () {
+        const updated = yield* readPrd
+        const anyChanges =
+          updated.length !== current.length ||
+          updated.some((u, i) => u.isChangedComparedTo(current[i]!))
+        if (!anyChanges) {
+          return
+        }
 
-      const githubPrs = new Map<string, number>()
-      const toRemove = new Set(
-        current.filter((i) => i.id !== null).map((i) => i.id!),
-      )
+        const githubPrs = new Map<string, number>()
+        const toRemove = new Set(
+          current.filter((i) => i.id !== null).map((i) => i.id!),
+        )
 
-      for (const issue of updated) {
-        toRemove.delete(issue.id!)
+        for (const issue of updated) {
+          toRemove.delete(issue.id!)
 
-        if (issue.id === null) {
-          yield* source.createIssue(issue)
-          continue
-        }
+          if (issue.id === null) {
+            yield* source.createIssue(issue)
+            continue
+          }
 
-        if (issue.githubPrNumber) {
-          githubPrs.set(issue.id, issue.githubPrNumber)
-        }
+          if (issue.githubPrNumber) {
+            githubPrs.set(issue.id, issue.githubPrNumber)
+          }
 
-        const existing = current.find((i) => i.id === issue.id)
-        if (!existing || !existing.isChangedComparedTo(issue)) continue
+          const existing = current.find((i) => i.id === issue.id)
+          if (!existing || !existing.isChangedComparedTo(issue)) continue
 
-        yield* source.updateIssue({
-          issueId: issue.id,
-          title: issue.title,
-          description: issue.description,
-          state: issue.state,
-          blockedBy: issue.blockedBy,
-        })
+          yield* source.updateIssue({
+            issueId: issue.id,
+            title: issue.title,
+            description: issue.description,
+            state: issue.state,
+            blockedBy: issue.blockedBy,
+          })
 
-        updatedIssues.set(issue.id, issue)
-      }
+          updatedIssues.set(issue.id, issue)
+        }
 
-      yield* Effect.forEach(
-        toRemove,
-        (issueId) => source.cancelIssue(issueId),
-        { concurrency: "unbounded" },
-      )
+        yield* Effect.forEach(
+          toRemove,
+          (issueId) => source.cancelIssue(issueId),
+          { concurrency: "unbounded" },
+        )
 
-      current = yield* source.issues
-      yield* fs.writeFileString(
-        prdFile,
-        PrdIssue.arrayToYaml(
-          current.map((issue) => {
-            const prNumber = githubPrs.get(issue.id!)
-            if (!prNumber) return issue
-            return new PrdIssue({ ...issue, githubPrNumber: prNumber })
-          }),
-        ),
-      )
-    }).pipe(Effect.uninterruptible)
+        current = yield* source.issues
+        yield* fs.writeFileString(
+          prdFile,
+          PrdIssue.arrayToYaml(
+            current.map((issue) => {
+              const prNumber = githubPrs.get(issue.id!)
+              if (!prNumber) return issue
+              return new PrdIssue({ ...issue, githubPrNumber: prNumber })
+            }),
+          ),
+        )
+      }).pipe(Effect.uninterruptible),
+    )
 
     const updateSyncHandle = yield* FiberHandle.make()
     const updateSync = Effect.gen(function* () {
@@ -237,15 +241,17 @@ export class Prd extends ServiceMap.Service<
       path: prdFile,
       mergableGithubPrs,
       maybeRevertIssue,
-      revertUpdatedIssues: Effect.gen(function* () {
-        for (const issue of updatedIssues.values()) {
-          if (issue.state === "done") continue
-          yield* source.updateIssue({
-            issueId: issue.id!,
-            state: "todo",
-          })
-        }
-      }),
+      revertUpdatedIssues: syncSemaphore.withPermit(
+        Effect.gen(function* () {
+          for (const issue of updatedIssues.values()) {
+            if (issue.state === "done") continue
+            yield* source.updateIssue({
+              issueId: issue.id!,
+              state: "todo",
+            })
+          }
+        }),
+      ),
       flagUnmergable,
       findById,
     }