iterate-plugin 3.9.1 → 3.9.2

package/dist/tools/transcript.js

@@ -789,19 +789,28 @@ export async function markFixRolledBackInTranscript(projectRoot, id) {
     const file = transcriptPath(projectRoot);
     if (!existsSync(file))
         return false;
-    try {
-        const parsed = JSON.parse(await readFile(file, 'utf-8'));
-        if (!parsed || typeof parsed !== 'object' || !Array.isArray(parsed.fixes))
+    // Cross-process read-modify-write: serialize on the shared transcript lock
+    // (the same lock `nudge` takes), otherwise two concurrent rollbacks each read
+    // the same base and the second write silently discards the first's flag. The
+    // lock is held across the awaits because `withProjectLock` awaits a
+    // promise-returning callback. Kept fail-safe: any error still returns false.
+    return withProjectLock(projectRoot, 'transcript', async () => {
+        if (!existsSync(file))
             return false;
-        const { config } = loadEffectiveConfig(projectRoot);
-        const approval = config.observatory?.approval ?? 'ask';
-        const builder = rehydrateBuilder(parsed, approval);
-        builder.markFixRolledBack(id);
-        await persist(file, builder.serialize());
-        return true;
-    }
-    catch {
-        // Never let the secondary transcript write break the rollback flow.
-        return false;
-    }
+        try {
+            const parsed = JSON.parse(await readFile(file, 'utf-8'));
+            if (!parsed || typeof parsed !== 'object' || !Array.isArray(parsed.fixes))
+                return false;
+            const { config } = loadEffectiveConfig(projectRoot);
+            const approval = config.observatory?.approval ?? 'ask';
+            const builder = rehydrateBuilder(parsed, approval);
+            builder.markFixRolledBack(id);
+            await persist(file, builder.serialize());
+            return true;
+        }
+        catch {
+            // Never let the secondary transcript write break the rollback flow.
+            return false;
+        }
+    });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "iterate-plugin",
-  "version": "3.9.1",
+  "version": "3.9.2",
   "description": "dsh plugin that turns the iterate skill into an autonomous closed-loop harness with quality command center and experience bank (v3.3). Features: plan -> parallel review xN -> atomic fixes -> validate -> loop -> auto-stop, plus dry-run pure-review mode, quality gate compute/persist, writable experience bank, defense events stream (record + bilingual labels), and native command buttons — with a live quality command center (F8/F9/F10 render real session data), §8 assign-fix instruction, and task_mode indicator wired end to end.",
   "type": "module",
   "license": "MIT",

package/src/tools/transcript.ts

@@ -815,17 +815,25 @@ export async function markFixRolledBackInTranscript(
 ): Promise<boolean> {
   const file = transcriptPath(projectRoot)
   if (!existsSync(file)) return false
-  try {
-    const parsed = JSON.parse(await readFile(file, 'utf-8')) as unknown as TranscriptManifest
-    if (!parsed || typeof parsed !== 'object' || !Array.isArray(parsed.fixes)) return false
-    const { config } = loadEffectiveConfig(projectRoot)
-    const approval = config.observatory?.approval ?? 'ask'
-    const builder = rehydrateBuilder(parsed, approval)
-    builder.markFixRolledBack(id)
-    await persist(file, builder.serialize())
-    return true
-  } catch {
-    // Never let the secondary transcript write break the rollback flow.
-    return false
-  }
+  // Cross-process read-modify-write: serialize on the shared transcript lock
+  // (the same lock `nudge` takes), otherwise two concurrent rollbacks each read
+  // the same base and the second write silently discards the first's flag. The
+  // lock is held across the awaits because `withProjectLock` awaits a
+  // promise-returning callback. Kept fail-safe: any error still returns false.
+  return withProjectLock(projectRoot, 'transcript', async () => {
+    if (!existsSync(file)) return false
+    try {
+      const parsed = JSON.parse(await readFile(file, 'utf-8')) as unknown as TranscriptManifest
+      if (!parsed || typeof parsed !== 'object' || !Array.isArray(parsed.fixes)) return false
+      const { config } = loadEffectiveConfig(projectRoot)
+      const approval = config.observatory?.approval ?? 'ask'
+      const builder = rehydrateBuilder(parsed, approval)
+      builder.markFixRolledBack(id)
+      await persist(file, builder.serialize())
+      return true
+    } catch {
+      // Never let the secondary transcript write break the rollback flow.
+      return false
+    }
+  })
 }