@sureshsankaran/ralph-wiggum 0.1.3 → 0.1.4

package/dist/index.js

@@ -45,22 +45,26 @@ function extractPromiseText(text) {
     return match ? match[1].trim().replace(/\s+/g, " ") : null;
 }
 // Atomically increment iteration using file-based locking
-function tryIncrementIteration(stateFilePath, lockFilePath, expectedIteration) {
-    // Try to acquire lock by checking if lock file exists with different iteration
+// Returns the next iteration number if we won the lock, null if another handler is processing
+function tryClaimIteration(stateFilePath, lockFilePath, currentIteration) {
+    const nextIteration = currentIteration + 1;
+    // Check if lock exists for THIS specific iteration transition
     if (existsSync(lockFilePath)) {
         const lockContent = readFileSync(lockFilePath, "utf-8").trim();
-        const lockedIteration = parseInt(lockContent, 10);
-        // If lock exists for same or higher iteration, someone else is handling it
-        if (!isNaN(lockedIteration) && lockedIteration >= expectedIteration) {
+        // Lock format: "from:to" e.g., "1:2" means transitioning from iteration 1 to 2
+        const [fromStr, toStr] = lockContent.split(":");
+        const from = parseInt(fromStr, 10);
+        const to = parseInt(toStr, 10);
+        // If lock exists for this exact transition, someone else is handling it
+        if (from === currentIteration && to === nextIteration) {
             return null;
         }
     }
-    // Write our lock
-    const nextIteration = expectedIteration + 1;
-    writeFileSync(lockFilePath, String(nextIteration));
-    // Double-check we got the lock (simple mutex)
+    // Write our lock with format "from:to"
+    writeFileSync(lockFilePath, `${currentIteration}:${nextIteration}`);
+    // Double-check we got the lock
     const lockCheck = readFileSync(lockFilePath, "utf-8").trim();
-    if (parseInt(lockCheck, 10) !== nextIteration) {
+    if (lockCheck !== `${currentIteration}:${nextIteration}`) {
         return null; // Someone else won the race
     }
     // Update state file
@@ -105,7 +109,7 @@ export const RalphWiggumPlugin = async ({ client, directory }) => {
                         const promiseText = extractPromiseText(part.text);
                         if (promiseText === state.completion_promise) {
                             completionDetected = true;
-                            console.log(`\nRalph loop: Detected <promise>${state.completion_promise}</promise> - task complete!`);
+                            console.log(`\nRalph loop complete! Detected <promise>${state.completion_promise}</promise>`);
                             try {
                                 unlinkSync(stateFilePath);
                             }
@@ -125,9 +129,8 @@ export const RalphWiggumPlugin = async ({ client, directory }) => {
             const isIdle = event.type === "session.status" && event.properties?.status?.type === "idle";
             if (!isIdle)
                 return;
-            // If completion was already detected, don't continue
+            // If completion was already detected, silently ignore
             if (completionDetected) {
-                console.log("\nRalph loop: Completion already detected, not continuing");
                 return;
             }
             const sessionID = event.properties?.sessionID;
@@ -165,17 +168,17 @@ export const RalphWiggumPlugin = async ({ client, directory }) => {
                 catch { }
                 return;
             }
-            // Try to atomically increment - this prevents race conditions
-            const nextIteration = tryIncrementIteration(stateFilePath, lockFilePath, state.iteration);
+            // Try to atomically claim this iteration - this prevents race conditions
+            const nextIteration = tryClaimIteration(stateFilePath, lockFilePath, state.iteration);
             if (nextIteration === null) {
-                // Another event handler won the race, skip this one
+                // Another event handler won the race, skip this one silently
                 return;
             }
-            console.log(`\nRalph loop: Starting iteration ${nextIteration}`);
             // Build system message
             const systemMsg = state.completion_promise
-                ? `Ralph iteration ${nextIteration} | To stop: output <promise>${state.completion_promise}</promise> (ONLY when statement is TRUE - do not lie to exit!)`
-                : `Ralph iteration ${nextIteration} | No completion promise set - loop runs infinitely`;
+                ? `Ralph iteration ${nextIteration} | To stop: output <promise>${state.completion_promise}</promise> (ONLY when TRUE)`
+                : `Ralph iteration ${nextIteration} | No completion promise set`;
+            console.log(`\n${systemMsg}`);
             // Send the same prompt back to continue the session
             try {
                 if (sessionID) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sureshsankaran/ralph-wiggum",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "Ralph Wiggum iterative AI development plugin for OpenCode - continuously loops the same prompt until task completion",
   "type": "module",
   "main": "dist/index.js",