@ai-dev-methodologies/rlp-desk 0.14.1 → 0.14.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ai-dev-methodologies/rlp-desk",
-  "version": "0.14.1",
+  "version": "0.14.3",
   "description": "Fresh-context iterative loops for Claude Code — autonomous task completion with independent verification",
   "scripts": {
     "postinstall": "node scripts/postinstall.js",

package/src/node/polling/signal-poller.mjs

@@ -127,6 +127,11 @@ export async function pollForSignal(
     capturePane = defaultCapturePane,
     sendKeys = defaultSendKeys,
     log = () => {},
+    // v0.14.2 Bug Report #4 Fix-D: optional legacy fallback path checked
+    // only after the canonical signalFile last-chance read fails. Caller
+    // (campaign-main-loop) supplies the pre-v0.13.0 .claude/ralph-desk
+    // memos path; signal-poller stays read-only and never migrates.
+    legacySignalFile = null,
   } = {},
 ) {
   const deadline = Date.now() + timeoutMs;
@@ -249,7 +254,23 @@ export async function pollForSignal(
     const rawContent = await readFile(signalFile);
     return JSON.parse(rawContent);
   } catch {
-    // fall through to TimeoutError
+    // fall through
+  }
+
+  // v0.14.2 Bug Report #4 Fix-D: codex sometimes lands the verdict at the
+  // pre-v0.13.0 legacy path (.claude/ralph-desk/memos/...) instead of the
+  // canonical .rlp-desk/memos/. If the caller passed `legacySignalFile`,
+  // try that path before declaring timeout — same semantics as the
+  // canonical last-chance read. campaign-main-loop is responsible for
+  // migrating the file into the canonical location after observing it;
+  // signal-poller stays read-only.
+  if (legacySignalFile) {
+    try {
+      const rawContent = await readFile(legacySignalFile);
+      return JSON.parse(rawContent);
+    } catch {
+      // fall through to TimeoutError
+    }
   }
 
   throw new TimeoutError(`Timed out waiting for valid JSON signal at ${signalFile}`);

package/src/node/prompts/prompt-assembler.mjs

@@ -180,6 +180,11 @@ export async function assembleVerifierPrompt({
   verifiedUs = [],
   autonomousMode = false,
   conflictLogPath = '',
+  // v0.14.2 Bug Report #4 Fix-E: when supplied, the assembled prompt
+  // ends with a strong "MUST write verdict to <absolute_path>" rule so
+  // codex (which sometimes infers the legacy .claude/ralph-desk path
+  // from CWD) lands the verdict where the leader is polling.
+  verdictWritePath = '',
 } = {}) {
   const basePrompt = await readRequiredFile(promptBase, 'Verifier prompt base file');
   const promptLines = [
@@ -209,5 +214,19 @@ export async function assembleVerifierPrompt({
     appendAutonomousModeSection(promptLines, { conflictLogPath, verifier: true });
   }
 
+  if (verdictWritePath) {
+    promptLines.push('');
+    promptLines.push('---');
+    promptLines.push('## CRITICAL: Verdict file write path (v0.14.2)');
+    promptLines.push('');
+    promptLines.push('Write `verify-verdict.json` ONLY to this absolute path:');
+    promptLines.push('');
+    promptLines.push(`    ${verdictWritePath}`);
+    promptLines.push('');
+    promptLines.push('DO NOT write to `.claude/ralph-desk/memos/` — that path is deprecated since');
+    promptLines.push('v0.13.0. The leader polls only the absolute path above; writing elsewhere');
+    promptLines.push('triggers BLOCKED `verifier_dead` even though your verdict is correct.');
+  }
+
   return `${promptLines.join('\n')}\n`;
 }

package/src/node/runner/campaign-main-loop.mjs

@@ -43,6 +43,31 @@ const MODEL_UPGRADES = {
   'gpt-5.3-codex-spark:xhigh': 'BLOCKED',
 };
 
+// v0.14.2 Bug Report #4 Fix-D: codex occasionally lands the verdict at the
+// pre-v0.13.0 `.claude/ralph-desk/memos/` path despite prompt instructions.
+// signal-poller's `legacySignalFile` last-chance branch returns the parsed
+// verdict in memory; these two helpers move the file into the canonical
+// .rlp-desk/memos/ location AFTER the polling loop succeeds, so analytics
+// archival + sentinel hygiene remain consistent.
+export async function _verdictMigrationNeeded(paths) {
+  if (!paths?.legacyVerdictFile || !paths?.verdictFile) return false;
+  // Migration is only meaningful when the legacy file exists AND the
+  // canonical file does not. If both exist, canonical wins (already
+  // observed) and we leave legacy alone.
+  let legacyExists = false;
+  let canonicalExists = false;
+  try { legacyExists = await exists(paths.legacyVerdictFile); } catch {}
+  try { canonicalExists = await exists(paths.verdictFile); } catch {}
+  return legacyExists && !canonicalExists;
+}
+
+export async function _migrateLegacyVerdict(paths) {
+  if (!paths?.legacyVerdictFile || !paths?.verdictFile) return false;
+  await fs.mkdir(path.dirname(paths.verdictFile), { recursive: true });
+  await fs.rename(paths.legacyVerdictFile, paths.verdictFile);
+  return true;
+}
+
 // v0.13.0: legacy .claude/ralph-desk/ guidance for run mode (no auto-mv).
 export function detectLegacyDeskInRunMode(rootDir, env = process.env) {
   const legacyPath = path.join(rootDir, LEGACY_DESK_REL);
@@ -76,6 +101,12 @@ function buildPaths(rootDir, slug, env = process.env) {
     doneClaimFile: path.join(deskRoot, 'memos', `${slug}-done-claim.json`),
     signalFile: path.join(deskRoot, 'memos', `${slug}-iter-signal.json`),
     verdictFile: path.join(deskRoot, 'memos', `${slug}-verify-verdict.json`),
+    // v0.14.2 Bug Report #4 Fix-D: codex sometimes lands the verdict at the
+    // pre-v0.13.0 legacy path. We track the absolute legacy location so the
+    // signal-poller last-chance read can fall back to it before declaring
+    // timeout. Always rooted at <project>/.claude/ralph-desk/memos/, even
+    // when RLP_DESK_RUNTIME_DIR overrides the canonical deskRoot.
+    legacyVerdictFile: path.join(rootDir, '.claude', 'ralph-desk', 'memos', `${slug}-verify-verdict.json`),
     blockedSentinel: path.join(deskRoot, 'memos', `${slug}-blocked.md`),
     completeSentinel: path.join(deskRoot, 'memos', `${slug}-complete.md`),
     contextFile: path.join(deskRoot, 'context', `${slug}-latest.md`),
@@ -376,6 +407,11 @@ async function dispatchVerifier({
     verifyMode: 'per-us',
     usId,
     verifiedUs: state.verified_us,
+    // v0.14.2 Fix-E: hand the absolute canonical verdict path to the
+    // verifier prompt. assembleVerifierPrompt appends a "CRITICAL: write
+    // verdict to <path>" footer so codex does not infer the legacy
+    // .claude/ralph-desk/memos/ location from CWD.
+    verdictWritePath: paths.verdictFile,
   });
   const fileName = suffix
     ? `${suffix}.verifier-prompt.md`
@@ -1452,7 +1488,21 @@ async function _runCampaignBody(slug, options, paths, rootDir) {
         mode: parseModelFlag(verifierModel, 'verifier').engine,
         paneId: state.verifier_pane_id,
         timeoutMs: iterTimeoutMs,
+        // v0.14.2 Fix-D: codex sometimes writes the verdict at the legacy
+        // .claude/ralph-desk/memos/ path. signal-poller's last-chance read
+        // tries this fallback before timing out.
+        legacySignalFile: paths.legacyVerdictFile,
       });
+      // v0.14.2 Fix-D continued: if the verdict came from the legacy path,
+      // migrate it into the canonical location so the rest of the pipeline
+      // (analytics archival, sentinels, status) sees a single canonical
+      // file. Best-effort — any rename failure is logged but does not
+      // re-throw because we already have the parsed verdict in memory.
+      if (await _verdictMigrationNeeded(paths)) {
+        await _migrateLegacyVerdict(paths).catch((migrateErr) => {
+          console.error('[v0.14.2] legacy verdict migration failed:', migrateErr?.message ?? migrateErr);
+        });
+      }
       validateArtifact(verdict, {
         expectedSlug: slug,
         iterationFloor: state.iteration,

package/src/node/runner/prompt-dismisser.mjs

@@ -36,13 +36,22 @@ const DEFAULT_NO_RE = /\[y\/N\]|\(yes\/no,\s*default\s+no\)|[Dd]efault[: ]+[Nn]o
 const ACTIVE_TASK_RE =
   /esc to interrupt|background terminal running|^\s*[·✻]\s+[A-Za-z]+(\.{3}|…)/m;
 
-// v0.14.1: codex post-work idle UI markers. NOT a permission prompt — the
-// codex CLI has finished its task and is waiting for the next user input.
-// Pattern: a divider line "─ Worked for Xm Ys ─", a "› " input prompt, and
-// a status bar "Context X% left". Sources: BOS Bug Report #3 (2026-05-04).
+// v0.14.1 / v0.14.2: codex post-work idle UI markers. NOT a permission prompt
+// — the codex CLI has finished its task and is waiting for the next user
+// input. Sources: BOS Bug Report #3 (2026-05-04) + #4 (2026-05-05).
 // Treat this as "task done, idle awaiting input"; callers should harvest
 // the verdict file rather than escalate as `prompt_blocked`.
-export const CODEX_IDLE_RE = /─\s*Worked for \d+m \d+s\s*─|Context \d+%\s*left/;
+//
+// v0.14.2 relaxation (Bug #4): the v0.14.1 strict "─ Worked for Xm Ys ─"
+// regex required the surrounding horizontal rule to match. tmux capture
+// truncation occasionally dropped those rules so the pattern missed in
+// production. Match on multiple independent markers; ANY one is enough.
+//   1. "Worked for Xm Ys"           — duration line, codex-only
+//   2. "Context X%left" (no space)  — status bar; tolerate wrap removal
+//   3. "gpt-X.Y reasoning · branch" — codex idle status line
+//   4. codex default suggestions    — only printed at the idle prompt
+export const CODEX_IDLE_RE =
+  /Worked for \d+m \d+s|Context \d+%\s*left|gpt-\d+(\.\d+)? (low|medium|high|xhigh) ·|Improve documentation in @|Summarize recent commits|Explain (this )?code/;
 export function isCodexIdleUi(paneText) {
   if (typeof paneText !== 'string' || paneText.length === 0) return false;
   return CODEX_IDLE_RE.test(paneText);