@neriros/ralphy 3.10.10 → 3.10.12

package/dist/mcp/index.js

@@ -24093,6 +24093,12 @@ var HistoryEntrySchema = exports_external.object({
   appVersion: exports_external.string().optional(),
   usage: IterationUsageSchema.partial().optional()
 });
+var RevisionSchema = exports_external.object({
+  version: exports_external.number(),
+  attachmentId: exports_external.string(),
+  sha256: exports_external.string(),
+  trigger: exports_external.string()
+});
 var StateSchema = exports_external.object({
   version: exports_external.literal("2"),
   name: exports_external.string(),
@@ -24141,12 +24147,16 @@ var StateSchema = exports_external.object({
       attachmentId: exports_external.string().nullable().default(null),
       sha256: exports_external.string().nullable().default(null)
     }).default({ attachmentId: null, sha256: null }),
+    designRevisions: exports_external.array(RevisionSchema).default([]),
+    designPdfRevisions: exports_external.array(RevisionSchema).default([]),
     legacyProposalPurged: exports_external.boolean().default(false)
   }).default({
     proposal: { attachmentId: null, sha256: null },
     design: { attachmentId: null, sha256: null },
     proposalPdf: { attachmentId: null, sha256: null },
     designPdf: { attachmentId: null, sha256: null },
+    designRevisions: [],
+    designPdfRevisions: [],
     legacyProposalPurged: false
   }),
   confirmation: exports_external.object({

package/dist/shell/index.js

@@ -18928,8 +18928,8 @@ import { readFileSync } from "fs";
 import { resolve } from "path";
 function getVersion() {
   try {
-    if ("3.10.10")
-      return "3.10.10";
+    if ("3.10.12")
+      return "3.10.12";
   } catch {}
   const dirsToTry = [];
   try {
@@ -89590,7 +89590,7 @@ var init_zod2 = __esm(() => {
 function markersOf(set3) {
   return Array.isArray(set3) ? set3 : [set3];
 }
-var IterationUsageSchema, UsageSchema, HistoryEntrySchema, StateSchema, PhaseFrontmatterSchema;
+var IterationUsageSchema, UsageSchema, HistoryEntrySchema, RevisionSchema, StateSchema, PhaseFrontmatterSchema;
 var init_types2 = __esm(() => {
   init_zod2();
   IterationUsageSchema = exports_external2.object({
@@ -89623,6 +89623,12 @@ var init_types2 = __esm(() => {
     appVersion: exports_external2.string().optional(),
     usage: IterationUsageSchema.partial().optional()
   });
+  RevisionSchema = exports_external2.object({
+    version: exports_external2.number(),
+    attachmentId: exports_external2.string(),
+    sha256: exports_external2.string(),
+    trigger: exports_external2.string()
+  });
   StateSchema = exports_external2.object({
     version: exports_external2.literal("2"),
     name: exports_external2.string(),
@@ -89671,12 +89677,16 @@ var init_types2 = __esm(() => {
         attachmentId: exports_external2.string().nullable().default(null),
         sha256: exports_external2.string().nullable().default(null)
       }).default({ attachmentId: null, sha256: null }),
+      designRevisions: exports_external2.array(RevisionSchema).default([]),
+      designPdfRevisions: exports_external2.array(RevisionSchema).default([]),
       legacyProposalPurged: exports_external2.boolean().default(false)
     }).default({
       proposal: { attachmentId: null, sha256: null },
       design: { attachmentId: null, sha256: null },
       proposalPdf: { attachmentId: null, sha256: null },
       designPdf: { attachmentId: null, sha256: null },
+      designRevisions: [],
+      designPdfRevisions: [],
       legacyProposalPurged: false
     }),
     confirmation: exports_external2.object({
@@ -105470,6 +105480,7 @@ class AgentCoordinator {
   opts;
   workers = [];
   pendingIds = new Set;
+  inFlight = new Set;
   queue = [];
   stopped = false;
   paused = null;
@@ -106036,7 +106047,31 @@ class AgentCoordinator {
     while (this.workers.length + this.pendingIds.size < this.opts.concurrency && this.queue.length > 0) {
       const next = this.queue.shift();
       this.pendingIds.add(next.issue.id);
-      this.launchWorker(next.issue, next.trigger, next.mention);
+      this.track(this.launchWorker(next.issue, next.trigger, next.mention));
+    }
+  }
+  track(p) {
+    this.inFlight.add(p);
+    p.finally(() => {
+      this.inFlight.delete(p);
+    });
+    return p;
+  }
+  async whenSettled() {
+    let consecutiveEmpty = 0;
+    for (let guard = 0;guard < 1000; guard++) {
+      if (this.inFlight.size > 0) {
+        await Promise.allSettled(this.inFlight);
+        consecutiveEmpty = 0;
+      }
+      await new Promise((r) => setTimeout(r, 0));
+      if (this.inFlight.size === 0) {
+        consecutiveEmpty += 1;
+        if (consecutiveEmpty >= WHEN_SETTLED_STABLE_HOPS)
+          return;
+      } else {
+        consecutiveEmpty = 0;
+      }
     }
   }
   async launchWorker(issue2, trigger, mention) {
@@ -106115,6 +106150,7 @@ class AgentCoordinator {
       issueIdentifier: issue2.identifier,
       issue: issue2,
       trigger,
+      ...prep.cwd ? { cwd: prep.cwd } : {},
       kill: handle.kill,
       lastReportedIteration: 0,
       lastSyncedIteration: 0,
@@ -106154,7 +106190,10 @@ class AgentCoordinator {
         this.deps.onLog(`! sync-tasks (launch) failed for ${issue2.identifier}: ${err.message}`, "yellow");
       }
     }
-    handle.exited.then(async (code) => {
+    handle.exited.then((code) => this.track(this.finalizeWorkerExit(worker, issue2, prep, trigger, code)));
+  }
+  async finalizeWorkerExit(worker, issue2, prep, trigger, code) {
+    {
       const idx = this.workers.indexOf(worker);
       if (idx >= 0)
         this.workers.splice(idx, 1);
@@ -106199,10 +106238,10 @@ class AgentCoordinator {
         exit_code: code,
         ok
       });
-      await this.notifyExited(issue2, prep.changeName, code, trigger);
+      await this.notifyExited(issue2, prep.changeName, code, trigger, worker.cwd);
       this.deps.onWorkersChanged();
       this.spawnNext();
-    });
+    }
   }
   async restartWorker(changeName) {
     if (this.stopped)
@@ -106259,7 +106298,7 @@ class AgentCoordinator {
       this.deps.onLog(`! onSteeringAppended failed for ${changeName}: ${err.message}`, "yellow");
     }
   }
-  async notifyExited(issue2, changeName, code, trigger) {
+  async notifyExited(issue2, changeName, code, trigger, workerCwd) {
     const noChanges = code === NO_CHANGES_EXIT;
     const ok = code === 0 || noChanges;
     const changeDir = this.deps.getChangeDir?.(issue2) ?? undefined;
@@ -106279,6 +106318,7 @@ class AgentCoordinator {
         issueIdentifier: issue2.identifier,
         issue: issue2,
         trigger,
+        ...workerCwd ? { cwd: workerCwd } : {},
         kill: () => {},
         lastReportedIteration: 0,
         lastSyncedIteration: 0,
@@ -106378,7 +106418,7 @@ var emptyPrStatus = () => ({
   conflicted: 0,
   ciFailed: 0,
   quarantined: 0
-}), emptyPollResult = () => ({
+}), WHEN_SETTLED_STABLE_HOPS = 3, emptyPollResult = () => ({
   found: 0,
   added: 0,
   buckets: {
@@ -106891,7 +106931,9 @@ async function applyAwaitingMarkerOnce(issue2, statePath, state, deps) {
   }
   state.confirmation.awaitingMarkerAppliedAt = new Date().toISOString();
   try {
-    await writeConfirmationState(statePath, state.stateObj, state.confirmation);
+    const fresh = await readConfirmationState(statePath);
+    fresh.confirmation.awaitingMarkerAppliedAt = state.confirmation.awaitingMarkerAppliedAt;
+    await writeConfirmationState(statePath, fresh.stateObj, fresh.confirmation);
   } catch (err) {
     deps.onLog(`! persist awaitingMarkerAppliedAt for ${issue2.identifier}: ${err.message}`, "yellow");
   }
@@ -106911,7 +106953,9 @@ async function openDraftPrOnce(issue2, statePath, changeName, cwd2, state, deps)
   }
   state.confirmation.earlyDraftPrAt = new Date().toISOString();
   try {
-    await writeConfirmationState(statePath, state.stateObj, state.confirmation);
+    const fresh = await readConfirmationState(statePath);
+    fresh.confirmation.earlyDraftPrAt = state.confirmation.earlyDraftPrAt;
+    await writeConfirmationState(statePath, fresh.stateObj, fresh.confirmation);
   } catch (err) {
     deps.onLog(`! persist earlyDraftPrAt for ${issue2.identifier}: ${err.message}`, "yellow");
   }
@@ -107521,6 +107565,11 @@ var init_linear_resolvers = __esm(() => {
 // apps/agent/src/agent/wire/prepare.ts
 import { mkdir as mkdir8 } from "fs/promises";
 import { join as join27 } from "path";
+function composeAppendPrompt(promptArg, cfgAppendPrompt, workflowPrompt) {
+  return [promptArg || cfgAppendPrompt || "", workflowPrompt].filter(Boolean).join(`
+
+`);
+}
 function createPrepareHelpers(input) {
   const {
     args,
@@ -107617,9 +107666,7 @@ function createPrepareHelpers(input) {
       } catch (err) {
         diag("workflow", `! workflow render failed: ${err.message}`, "yellow");
       }
-      const appendPrompt = [args.prompt || cfg.appendPrompt || "", workflowPrompt].filter(Boolean).join(`
-
-`);
+      const appendPrompt = composeAppendPrompt(args.prompt ?? "", cfg.appendPrompt ?? "", workflowPrompt);
       changeName = await scaffoldChangeForIssue(scaffoldTasksDir, scaffoldStatesDir, issue2, comments, appendPrompt, attachments);
     } else {
       changeName = derivedName;
@@ -107636,6 +107683,7 @@ function createPrepareHelpers(input) {
     }
     return {
       changeName,
+      cwd: workerCwd,
       ...maps.prByChange.has(changeName) ? { prUrl: maps.prByChange.get(changeName) } : {}
     };
   }
@@ -108620,6 +108668,103 @@ function buildTicketDigest(issue2, comments) {
 function retroDepEntry(agentDebug, hook) {
   return agentDebug ? { runRetrospective: hook } : {};
 }
+function computeWantPr(wantPrBase, isAwaiting, isAwaitingConfirmation) {
+  return wantPrBase && !isAwaiting && !isAwaitingConfirmation;
+}
+function computeWantValidateOnly(hasValidateSpec, wantPrBase) {
+  return hasValidateSpec && !wantPrBase;
+}
+function releaseWorkerMaps(maps, changeName) {
+  maps.cwdByChange.delete(changeName);
+  maps.statesDirByChange.delete(changeName);
+  maps.branchByChange.delete(changeName);
+  maps.issueByChange.delete(changeName);
+}
+function buildTaskCmd(args, cfg, changeName) {
+  const engine = args.engineSet ? args.engine : cfg.engine;
+  const model = args.engineSet ? args.model : cfg.model;
+  const c = [
+    process.execPath,
+    process.argv[1] ?? "",
+    "loop",
+    "task",
+    "--name",
+    changeName,
+    "--" + engine,
+    "--model",
+    model
+  ];
+  const maxIter = args.maxIterations || cfg.maxIterationsPerTask;
+  if (maxIter > 0)
+    c.push("--max-iterations", String(maxIter));
+  const maxCost = args.maxCostUsd || cfg.maxCostUsdPerTask;
+  if (maxCost > 0)
+    c.push("--max-cost", String(maxCost));
+  const maxRuntime = args.maxRuntimeMinutes || cfg.maxRuntimeMinutesPerTask;
+  if (maxRuntime > 0)
+    c.push("--max-runtime", String(maxRuntime));
+  const maxFailures = args.maxConsecutiveFailures !== 5 ? args.maxConsecutiveFailures : cfg.maxConsecutiveFailuresPerTask;
+  if (maxFailures !== 5)
+    c.push("--max-failures", String(maxFailures));
+  const delay2 = args.delay || cfg.iterationDelaySeconds;
+  if (delay2 > 0)
+    c.push("--delay", String(delay2));
+  if (args.log || cfg.logRawStream)
+    c.push("--log");
+  if (args.verbose || cfg.taskVerbose)
+    c.push("--verbose");
+  if (args.manualTest || cfg.enableManualTest)
+    c.push("--manual-test");
+  const rp = cfg.openspec.reviewPhase;
+  if (rp.enabled) {
+    c.push("--review-enabled");
+    if (rp.maxRounds !== 1)
+      c.push("--review-max-rounds", String(rp.maxRounds));
+    if (rp.reviewerModel !== undefined)
+      c.push("--review-model", rp.reviewerModel);
+    if (rp.reviewerContextStrategy !== "fresh")
+      c.push("--review-context-strategy", rp.reviewerContextStrategy);
+  }
+  c.push("--from-agent");
+  return c;
+}
+function buildPostTaskInput(input) {
+  const { args, cfg } = input;
+  return {
+    ...input.trigger ? { mode: input.trigger } : {},
+    ...input.prUrl ? { prUrl: input.prUrl } : {},
+    changeName: input.changeName,
+    cwd: input.cwd,
+    projectRoot: input.projectRoot,
+    changeDir: input.changeDir,
+    stateFilePath: input.stateFilePath,
+    branch: input.branch,
+    issue: input.issue,
+    exitCode: input.exitCode,
+    useWorktree: input.useWorktree,
+    wantPr: input.wantPr,
+    wantFixCi: input.wantFixCi,
+    wantAutoMerge: input.wantAutoMerge,
+    wantValidateOnly: input.wantValidateOnly,
+    cfg: {
+      teardownScript: cfg.teardownScript ?? null,
+      prBaseBranch: cfg.prBaseBranch,
+      autoMergeStrategy: cfg.autoMergeStrategy,
+      maxCiFixAttempts: cfg.maxCiFixAttempts,
+      ciPollIntervalSeconds: cfg.ciPollIntervalSeconds,
+      cleanupWorktreeOnSuccess: cfg.cleanupWorktreeOnSuccess,
+      ignoreCiChecks: cfg.ignoreCiChecks,
+      stackPrsOnDependencies: args.stackPrs || cfg.stackPrsOnDependencies,
+      neverTouch: cfg.boundaries.never_touch,
+      metaOnlyFiles: cfg.boundaries.meta_only_files,
+      finalizeNoOpAsDone: cfg.finalizeNoOpAsDone,
+      manualMergeWhenAutoMergeDisabled: cfg.manualMergeWhenAutoMergeDisabled,
+      prDraft: cfg.prDraft,
+      validateCommands: [cfg.commands.test, cfg.commands.lint, cfg.commands.typecheck].filter((c) => Boolean(c))
+    },
+    respawnWorker: input.respawnWorker
+  };
+}
 function createSpawnWorker(input) {
   const {
     args,
@@ -108652,54 +108797,8 @@ function createSpawnWorker(input) {
     onWorkerOutput,
     onWorkerCmd
   } = input;
-  function buildTaskCmdFor(changeName) {
-    const engine = args.engineSet ? args.engine : cfg.engine;
-    const model = args.engineSet ? args.model : cfg.model;
-    const c = [
-      process.execPath,
-      process.argv[1] ?? "",
-      "loop",
-      "task",
-      "--name",
-      changeName,
-      "--" + engine,
-      "--model",
-      model
-    ];
-    const maxIter = args.maxIterations || cfg.maxIterationsPerTask;
-    if (maxIter > 0)
-      c.push("--max-iterations", String(maxIter));
-    const maxCost = args.maxCostUsd || cfg.maxCostUsdPerTask;
-    if (maxCost > 0)
-      c.push("--max-cost", String(maxCost));
-    const maxRuntime = args.maxRuntimeMinutes || cfg.maxRuntimeMinutesPerTask;
-    if (maxRuntime > 0)
-      c.push("--max-runtime", String(maxRuntime));
-    const maxFailures = args.maxConsecutiveFailures !== 5 ? args.maxConsecutiveFailures : cfg.maxConsecutiveFailuresPerTask;
-    if (maxFailures !== 5)
-      c.push("--max-failures", String(maxFailures));
-    const delay2 = args.delay || cfg.iterationDelaySeconds;
-    if (delay2 > 0)
-      c.push("--delay", String(delay2));
-    if (args.log || cfg.logRawStream)
-      c.push("--log");
-    if (args.verbose || cfg.taskVerbose)
-      c.push("--verbose");
-    if (args.manualTest || cfg.enableManualTest)
-      c.push("--manual-test");
-    const rp = cfg.openspec.reviewPhase;
-    if (rp.enabled) {
-      c.push("--review-enabled");
-      if (rp.maxRounds !== 1)
-        c.push("--review-max-rounds", String(rp.maxRounds));
-      if (rp.reviewerModel !== undefined)
-        c.push("--review-model", rp.reviewerModel);
-      if (rp.reviewerContextStrategy !== "fresh")
-        c.push("--review-context-strategy", rp.reviewerContextStrategy);
-    }
-    c.push("--from-agent");
-    return c;
-  }
+  const doPostTask = input.runners?.runPostTask ?? runPostTask;
+  const buildTaskCmdFor = (changeName) => buildTaskCmd(args, cfg, changeName);
   const retroSeen = new Set;
   const runRetrospectiveHook = async (info) => {
     try {
@@ -108777,7 +108876,7 @@ function createSpawnWorker(input) {
       const workerLayout = projectLayout(cwd2);
       const validateSpecPath = join33(workerLayout.changeDir(changeName), "specs", "validate.md");
       const hasValidateSpec = await Bun.file(validateSpecPath).exists();
-      const wantValidateOnly = hasValidateSpec && !wantPrBase;
+      const wantValidateOnly = computeWantValidateOnly(hasValidateSpec, wantPrBase);
       if (hasValidateSpec) {
         try {
           const stateFile = workerLayout.stateFile(changeName);
@@ -108821,10 +108920,10 @@ function createSpawnWorker(input) {
       } catch (err) {
         diag("tasks", `! tasks.md normalization failed: ${err.message}`, "yellow");
       }
-      const wantPr = wantPrBase && !awaitingChangeSet.has(changeName) && !(coordRef.current?.isAwaitingConfirmation(changeName) ?? false);
-      const effectiveCode = await runPostTask({
-        ...trigger ? { mode: trigger } : {},
-        ...prByChange?.get(changeName) ? { prUrl: prByChange.get(changeName) } : {},
+      const wantPr = computeWantPr(wantPrBase, awaitingChangeSet.has(changeName), coordRef.current?.isAwaitingConfirmation(changeName) ?? false);
+      const effectiveCode = await doPostTask(buildPostTaskInput({
+        args,
+        cfg,
         changeName,
         cwd: cwd2,
         projectRoot,
@@ -108838,24 +108937,10 @@ function createSpawnWorker(input) {
         wantFixCi,
         wantAutoMerge,
         wantValidateOnly,
-        cfg: {
-          teardownScript: cfg.teardownScript ?? null,
-          prBaseBranch: cfg.prBaseBranch,
-          autoMergeStrategy: cfg.autoMergeStrategy,
-          maxCiFixAttempts: cfg.maxCiFixAttempts,
-          ciPollIntervalSeconds: cfg.ciPollIntervalSeconds,
-          cleanupWorktreeOnSuccess: cfg.cleanupWorktreeOnSuccess,
-          ignoreCiChecks: cfg.ignoreCiChecks,
-          stackPrsOnDependencies: args.stackPrs || cfg.stackPrsOnDependencies,
-          neverTouch: cfg.boundaries.never_touch,
-          metaOnlyFiles: cfg.boundaries.meta_only_files,
-          finalizeNoOpAsDone: cfg.finalizeNoOpAsDone,
-          manualMergeWhenAutoMergeDisabled: cfg.manualMergeWhenAutoMergeDisabled,
-          prDraft: cfg.prDraft,
-          validateCommands: [cfg.commands.test, cfg.commands.lint, cfg.commands.typecheck].filter((c) => Boolean(c))
-        },
+        ...trigger ? { trigger } : {},
+        ...prByChange?.get(changeName) ? { prUrl: prByChange.get(changeName) } : {},
         respawnWorker: respawn
-      }, {
+      }), {
         cmd: tracedCmd,
         git: gitRunner,
         log: onLog,
@@ -108894,10 +108979,7 @@ function createSpawnWorker(input) {
         },
         resolveDependencyBaseBranch: (issue2) => resolveDependencyBaseBranchImpl(issue2, tracedCmd, cwd2, { apiKey, onLog })
       });
-      cwdByChange.delete(changeName);
-      statesDirByChange.delete(changeName);
-      branchByChange.delete(changeName);
-      issueByChange.delete(changeName);
+      releaseWorkerMaps({ cwdByChange, statesDirByChange, branchByChange, issueByChange }, changeName);
       onWorkerExited(changeName);
       return effectiveCode;
     });
@@ -261725,6 +261807,25 @@ async function readSpecAttachmentsSubtree(statePath) {
   const sidecar = await readSlotSidecar(dirname13(statePath), "specAttachments");
   return sidecar ?? await readInlineSpecAttachments(statePath);
 }
+function asRevisions(value) {
+  if (!Array.isArray(value))
+    return [];
+  const out = [];
+  for (const entry of value) {
+    if (entry && typeof entry === "object" && !Array.isArray(entry)) {
+      const e = entry;
+      if (typeof e.version === "number" && typeof e.attachmentId === "string" && typeof e.sha256 === "string" && typeof e.trigger === "string") {
+        out.push({
+          version: e.version,
+          attachmentId: e.attachmentId,
+          sha256: e.sha256,
+          trigger: e.trigger
+        });
+      }
+    }
+  }
+  return out;
+}
 async function readSpecAttachments(statePath) {
   const sa = await readSpecAttachmentsSubtree(statePath);
   return {
@@ -261743,12 +261844,45 @@ async function readSpecAttachments(statePath) {
     designPdf: {
       attachmentId: sa.designPdf?.attachmentId ?? null,
       sha256: sa.designPdf?.sha256 ?? null
-    }
+    },
+    designRevisions: asRevisions(sa.designRevisions),
+    designPdfRevisions: asRevisions(sa.designPdfRevisions)
   };
 }
 async function persistSlot(statePath, slot, value) {
   await writeField(stateDirOf(statePath), "linear-attachments", `specAttachments.${slot}`, value);
 }
+async function persistRevision(statePath, slot, revisions) {
+  await writeField(stateDirOf(statePath), "linear-attachments", `specAttachments.${REVISIONS_KEY[slot]}`, revisions);
+}
+async function isDesignSealed(stateDir) {
+  try {
+    const pr2 = await readSlotSidecar(stateDir, "pr");
+    const url2 = pr2?.url;
+    if (typeof url2 === "string" && url2.length > 0)
+      return true;
+  } catch {}
+  try {
+    const confirmation = await readSlotSidecar(stateDir, "confirmation");
+    if (confirmation?.earlyDraftPrAt != null)
+      return true;
+  } catch {}
+  return false;
+}
+async function resolveTriggerLabel(stateDir) {
+  try {
+    const flow2 = await readSlotSidecar(stateDir, "flow");
+    const snapshot = flow2?.actorSnapshot;
+    const value = snapshot?.value;
+    if (typeof value === "string" && TRIGGER_LABELS[value])
+      return TRIGGER_LABELS[value];
+  } catch {}
+  return "revision";
+}
+function versionedTitle(slot, n, label) {
+  const base2 = `Ralph design #${n} (${label})`;
+  return slot === "designPdf" ? `${base2} (PDF)` : base2;
+}
 async function adopt(deps, slot) {
   const spec = SLOT_SPECS[slot];
   try {
@@ -261793,6 +261927,65 @@ function extractImplementationSection(tasksMarkdown) {
   return captured.join(`
 `).trim();
 }
+async function syncSlotSealed(deps, slot, sourceBytes, hash2, state) {
+  const spec = SLOT_SPECS[slot];
+  const revisions = state[REVISIONS_KEY[slot]];
+  const v1Sha = state[slot]?.sha256 ?? null;
+  if (hash2 === v1Sha || revisions.some((r) => r.sha256 === hash2)) {
+    deps.log(`  spec-attachments: ${spec.uploadFilename} unchanged (sealed), skipping`, "gray");
+    return;
+  }
+  const n = 2 + revisions.length;
+  const label = await resolveTriggerLabel(stateDirOf(deps.statePath));
+  const title = versionedTitle(slot, n, label);
+  let uploadBytes;
+  try {
+    uploadBytes = await spec.renderBytes(sourceBytes);
+  } catch (err) {
+    deps.log(`! spec-attachments: render ${spec.uploadFilename} (sealed) failed: ${err.message}`, "yellow");
+    return;
+  }
+  let assetUrl;
+  try {
+    const uploaded = await deps.mutations.uploadFileToLinear(deps.apiKey, {
+      filename: spec.uploadFilename,
+      contentType: spec.contentType,
+      bytes: uploadBytes
+    });
+    assetUrl = uploaded.assetUrl;
+  } catch (err) {
+    deps.log(`! spec-attachments: upload ${spec.uploadFilename} (sealed) failed: ${describeLinearError(err)}`, "yellow");
+    return;
+  }
+  let attachmentId = null;
+  try {
+    attachmentId = await deps.mutations.findIssueAttachmentByTitle(deps.apiKey, deps.issueId, title);
+    if (attachmentId) {
+      deps.log(`  spec-attachments: adopted existing ${title} attachment ${attachmentId}`, "gray");
+    }
+  } catch (err) {
+    deps.log(`! spec-attachments: findIssueAttachmentByTitle ${title} failed (treating as no match): ${describeLinearError(err)}`, "yellow");
+    attachmentId = null;
+  }
+  if (!attachmentId) {
+    try {
+      attachmentId = await deps.mutations.createAttachmentForUrl(deps.apiKey, {
+        issueId: deps.issueId,
+        url: assetUrl,
+        title,
+        subtitle: `iteration ${deps.iteration}`
+      });
+    } catch (err) {
+      deps.log(`! spec-attachments: createAttachmentForUrl ${title} failed: ${describeLinearError(err)}`, "yellow");
+      return;
+    }
+    deps.log(`  spec-attachments: created ${title} attachment`, "gray");
+  }
+  await persistRevision(deps.statePath, slot, [
+    ...revisions,
+    { version: n, attachmentId, sha256: hash2, trigger: label }
+  ]);
+}
 async function syncSlot(deps, slot) {
   const spec = SLOT_SPECS[slot];
   const [primaryName, ...trailingNames] = spec.sourceFiles;
@@ -261848,7 +262041,12 @@ ${body}
     offset += p.length;
   }
   const hash2 = sha256Hex(sourceBytes);
-  let current = (await readSpecAttachments(deps.statePath))[slot] ?? EMPTY_SLOT;
+  const state = await readSpecAttachments(deps.statePath);
+  if (await isDesignSealed(stateDirOf(deps.statePath))) {
+    await syncSlotSealed(deps, slot, sourceBytes, hash2, state);
+    return;
+  }
+  let current = state[slot] ?? EMPTY_SLOT;
   if (!current.attachmentId) {
     const { adoptedId } = await adopt(deps, slot);
     if (adoptedId) {
@@ -261950,7 +262148,7 @@ async function syncSpecAttachments(deps) {
     await syncSlot(deps, slot);
   }
 }
-var identityRender = async (b2) => b2, pdfRender = (title) => async (b2) => renderMarkdownToPdf(new TextDecoder().decode(b2), title), SLOT_SPECS, LEGACY_SLOT_TITLES, EMPTY_SLOT;
+var identityRender = async (b2) => b2, pdfRender = (title) => async (b2) => renderMarkdownToPdf(new TextDecoder().decode(b2), title), SLOT_SPECS, LEGACY_SLOT_TITLES, REVISIONS_KEY, EMPTY_SLOT, TRIGGER_LABELS;
 var init_spec_attachments = __esm(() => {
   init_store();
   init_comment_sync();
@@ -261977,7 +262175,16 @@ var init_spec_attachments = __esm(() => {
     proposal: "Ralph proposal",
     proposalPdf: "Ralph proposal (PDF)"
   };
+  REVISIONS_KEY = {
+    design: "designRevisions",
+    designPdf: "designPdfRevisions"
+  };
   EMPTY_SLOT = { attachmentId: null, sha256: null };
+  TRIGGER_LABELS = {
+    review: "review follow-up",
+    "ci-fix": "CI fix",
+    "conflict-fix": "conflict fix"
+  };
 });
 
 // apps/agent/src/agent/wire/comment-sync.ts
@@ -262002,7 +262209,7 @@ function createCommentSyncHooks(input) {
   return {
     enabled: enabled2,
     syncTasks: async (worker, iteration) => {
-      const root = cwdByChange.get(worker.changeName) ?? projectRoot;
+      const root = worker.cwd ?? cwdByChange.get(worker.changeName) ?? projectRoot;
       const layout = projectLayout(root);
       const changeDir = layout.changeDir(worker.changeName);
       const statePath = layout.stateFile(worker.changeName);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@neriros/ralphy",
-  "version": "3.10.10",
+  "version": "3.10.12",
   "description": "An iterative AI task execution framework. Orchestrates multi-phase autonomous work using Claude or Codex engines.",
   "keywords": [
     "agent",