@neriros/ralphy 2.17.2 → 2.18.0

package/README.md

@@ -173,6 +173,7 @@ A default `ralphy.config.json` is written on first run with every defaulted sett
       "getTodo": { "filter": [{ "type": "status", "value": "Todo" }] },
       "getInProgress": { "filter": [{ "type": "status", "value": "In Progress" }] },
       "getConflicted": { "filter": [{ "type": "label", "value": "ralph:conflicted" }] },
+      "getReview": { "filter": [{ "type": "label", "value": "ralph:review" }] },
       "setInProgress": { "type": "status", "value": "In Progress" },
       "setDone": {
         "apply": [
@@ -183,6 +184,7 @@ A default `ralphy.config.json` is written on first run with every defaulted sett
       "setError": { "type": "label", "value": "ralph:error" },
       "setConflicted": { "type": "label", "value": "ralph:conflicted" },
       "clearConflicted": { "type": "label", "value": "ralph:conflicted" },
+      "clearReview": { "type": "label", "value": "ralph:review" },
     },
   },
   "useWorktree": true,
@@ -205,9 +207,11 @@ A default `ralphy.config.json` is written on first run with every defaulted sett
 
 Linear is the source of truth for which issues Ralph has touched. Each `linear.indicators` key names a lifecycle event:
 
-- `getTodo` / `getInProgress` / `getConflicted` — `{ filter: [...] }` selectors used to find issues to pick up, resume, or repair.
+- `getTodo` / `getInProgress` / `getConflicted` / `getReview` — `{ filter: [...] }` selectors used to find issues to pick up, resume, repair, or follow up on after review.
 - `setInProgress` / `setDone` / `setError` / `setConflicted` — single marker `{ type, value }` or `{ apply: [...] }` for multi-marker.
-- `clearConflicted` — labels to remove once a conflicted PR is fixed (status removal is not supported).
+- `clearConflicted` / `clearReview` — labels to remove once a conflicted PR is fixed or a review-mode issue is picked back up (status removal is not supported).
+
+**Review follow-ups.** When a Linear issue is in a "done" state and a reviewer adds the `getReview` marker (typically a label like `ralph:review` after leaving comments), Ralph picks it up, applies `setInProgress`, removes the `clearReview` label so the same trigger doesn't re-fire, fetches the comment thread, filters out Ralph's own comments, and prepends those reviewer comments as a new task at the top of `tasks.md`. The worker addresses them in the same change branch and `setDone` is re-applied on success.
 
 Marker types are `"label"` or `"status"`. Combine markers under `apply` when one event needs to set multiple — e.g. `setDone` flipping a status _and_ adding a "shipped" label.
 

package/dist/cli/index.js

@@ -35029,8 +35029,8 @@ import { readFileSync as readFileSync2 } from "fs";
 import { resolve } from "path";
 function getVersion() {
   try {
-    if ("2.17.2")
-      return "2.17.2";
+    if ("2.18.0")
+      return "2.18.0";
   } catch {}
   const dirsToTry = [];
   try {
@@ -35356,13 +35356,20 @@ var init_cli = __esm(() => {
     "getTodo",
     "getInProgress",
     "getConflicted",
+    "getReview",
     "setInProgress",
     "setDone",
     "setError",
     "setConflicted",
-    "clearConflicted"
+    "clearConflicted",
+    "clearReview"
+  ]);
+  GET_KEYS = new Set([
+    "getTodo",
+    "getInProgress",
+    "getConflicted",
+    "getReview"
   ]);
-  GET_KEYS = new Set(["getTodo", "getInProgress", "getConflicted"]);
   HELP_TEXT = [
     `ralph v${VERSION}`,
     "",
@@ -35404,8 +35411,9 @@ var init_cli = __esm(() => {
     "                          --indicator getTodo:status:Todo",
     "                          --indicator setDone:label:shipped",
     "                          --indicator setDone:status:Done   (combined with above \u2192 multi-marker)",
-    "                          Keys: getTodo, getInProgress, getConflicted,",
-    "                                setInProgress, setDone, setError, setConflicted, clearConflicted",
+    "                          Keys: getTodo, getInProgress, getConflicted, getReview,",
+    "                                setInProgress, setDone, setError, setConflicted,",
+    "                                clearConflicted, clearReview",
     "                          Types: label, status",
     "  --create-pr             Push the worker branch and open a GitHub PR on success (needs --worktree)",
     "  --fix-ci                After opening the PR, re-run on CI failures until green (needs --create-pr)",
@@ -39468,6 +39476,99 @@ var init_types2 = __esm(() => {
   });
 });
 
+// apps/cli/src/agent/worktree.ts
+import { basename, join as join3 } from "path";
+import { homedir } from "os";
+import { exists } from "fs/promises";
+function worktreesDir(projectRoot) {
+  return join3(homedir(), ".ralph", basename(projectRoot), "worktrees");
+}
+function branchForChange(changeName) {
+  return `ralph/${changeName}`;
+}
+async function createWorktree(projectRoot, changeName, runner) {
+  const dir = worktreesDir(projectRoot);
+  const cwd2 = join3(dir, changeName);
+  const branch = branchForChange(changeName);
+  const list = await runner.run(["worktree", "list", "--porcelain"], projectRoot);
+  if (list.stdout.includes(`worktree ${cwd2}
+`)) {
+    return { cwd: cwd2, branch };
+  }
+  let branchExists = true;
+  try {
+    await runner.run(["rev-parse", "--verify", "--quiet", `refs/heads/${branch}`], projectRoot);
+  } catch {
+    branchExists = false;
+  }
+  const cmd = branchExists ? ["worktree", "add", cwd2, branch] : ["worktree", "add", "-b", branch, cwd2];
+  await runner.run(cmd, projectRoot);
+  return { cwd: cwd2, branch };
+}
+async function removeWorktree(projectRoot, cwd2, runner) {
+  await runner.run(["worktree", "remove", "--force", cwd2], projectRoot);
+}
+async function isWorktreeSafeToRemove(cwd2, base2, runner) {
+  const status = await runner.run(["status", "--porcelain"], cwd2);
+  const dirty = status.stdout.trim();
+  let unpushedCommits = "";
+  try {
+    const log2 = await runner.run(["log", "--oneline", `${base2}..HEAD`, "--no-merges"], cwd2);
+    unpushedCommits = log2.stdout.trim();
+  } catch {
+    unpushedCommits = "<unknown: failed to compare against base>";
+  }
+  if (dirty && unpushedCommits) {
+    return {
+      safe: false,
+      reason: "uncommitted changes AND unpushed commits present",
+      dirty,
+      unpushedCommits
+    };
+  }
+  if (dirty) {
+    return {
+      safe: false,
+      reason: "uncommitted or untracked files present",
+      dirty,
+      unpushedCommits
+    };
+  }
+  if (unpushedCommits) {
+    return {
+      safe: false,
+      reason: `commits ahead of ${base2} were not pushed/PR'd`,
+      dirty,
+      unpushedCommits
+    };
+  }
+  return { safe: true, dirty, unpushedCommits };
+}
+async function seedWorktreeMcpConfig(projectRoot, worktreeCwd) {
+  const dst = join3(worktreeCwd, ".mcp.json");
+  const src = join3(projectRoot, ".mcp.json");
+  const source = await exists(dst) ? dst : await exists(src) ? src : null;
+  if (!source)
+    return;
+  let parsed;
+  try {
+    parsed = await Bun.file(source).json();
+  } catch {
+    return;
+  }
+  const servers = parsed.mcpServers;
+  if (servers && typeof servers === "object") {
+    for (const cfg of Object.values(servers)) {
+      if (Array.isArray(cfg.args)) {
+        cfg.args = cfg.args.map((a) => typeof a === "string" && a.startsWith(".ralph/") ? join3(projectRoot, a) : a);
+      }
+    }
+  }
+  await Bun.write(dst, JSON.stringify(parsed, null, 2) + `
+`);
+}
+var init_worktree = () => {};
+
 // node_modules/.bun/react@18.3.1/node_modules/react/cjs/react-jsx-dev-runtime.development.js
 var require_react_jsx_dev_runtime_development = __commonJS((exports) => {
   var React10 = __toESM(require_react());
@@ -59333,8 +59434,8 @@ var init_node = __esm(() => {
 });
 
 // packages/telemetry/src/index.ts
-import { homedir } from "os";
-import { join as join6 } from "path";
+import { homedir as homedir2 } from "os";
+import { join as join7 } from "path";
 import { randomUUID } from "crypto";
 function setDefaultProperties(props) {
   defaultProps = { ...defaultProps, ...props };
@@ -59342,7 +59443,7 @@ function setDefaultProperties(props) {
 async function init() {
   if (!enabled)
     return;
-  const idPath = join6(homedir(), ".ralph", ".telemetry-id");
+  const idPath = join7(homedir2(), ".ralph", ".telemetry-id");
   const idFile = Bun.file(idPath);
   if (await idFile.exists()) {
     distinctId = (await idFile.text()).trim();
@@ -59417,12 +59518,12 @@ ${fence}`;
 }
 
 // apps/cli/src/agent/config.ts
-import { join as join10 } from "path";
+import { join as join11 } from "path";
 function stripJsonComments(text) {
   return text.replace(/\/\/[^\n]*/g, "");
 }
 async function loadRalphyConfig(projectRoot) {
-  const path = join10(projectRoot, "ralphy.config.json");
+  const path = join11(projectRoot, "ralphy.config.json");
   const file = Bun.file(path);
   if (!await file.exists()) {
     return RalphyConfigSchema.parse({});
@@ -59432,7 +59533,7 @@ async function loadRalphyConfig(projectRoot) {
   return RalphyConfigSchema.parse(raw);
 }
 async function ensureRalphyConfig(projectRoot) {
-  const path = join10(projectRoot, "ralphy.config.json");
+  const path = join11(projectRoot, "ralphy.config.json");
   const file = Bun.file(path);
   if (await file.exists())
     return path;
@@ -59533,6 +59634,10 @@ var MarkerSchema, GetIndicatorSchema, SetIndicatorSchema, IndicatorsSchema, Ralp
       // Issues whose PR has a merge conflict (Ralph will attempt a re-fix run).
       // "getConflicted": { "filter": [{ "type": "label", "value": "ralph:conflict" }] },
 
+      // Done issues with new review comments to address (Ralph will re-open
+      // and prepend a task that ingests the non-Ralph comments).
+      // "getReview": { "filter": [{ "type": "label", "value": "ralph:review" }] },
+
       // Applied when Ralph picks up an issue.
       // "setInProgress": { "type": "label", "value": "ralph:in-progress" },
 
@@ -59546,7 +59651,10 @@ var MarkerSchema, GetIndicatorSchema, SetIndicatorSchema, IndicatorsSchema, Ralp
       // "setConflicted": { "type": "label", "value": "ralph:conflict" },
 
       // Label removed once the conflict is fixed (status removal is not supported here).
-      // "clearConflicted": { "type": "label", "value": "ralph:conflict" }
+      // "clearConflicted": { "type": "label", "value": "ralph:conflict" },
+
+      // Label removed when Ralph picks up a review-mode issue (status removal not supported).
+      // "clearReview": { "type": "label", "value": "ralph:review" }
     }
   }
 }
@@ -59568,24 +59676,28 @@ var init_config = __esm(() => {
     getTodo: GetIndicatorSchema.optional(),
     getInProgress: GetIndicatorSchema.optional(),
     getConflicted: GetIndicatorSchema.optional(),
+    getReview: GetIndicatorSchema.optional(),
     setInProgress: SetIndicatorSchema.optional(),
     setDone: SetIndicatorSchema.optional(),
     setError: SetIndicatorSchema.optional(),
     setConflicted: SetIndicatorSchema.optional(),
-    clearConflicted: SetIndicatorSchema.optional()
+    clearConflicted: SetIndicatorSchema.optional(),
+    clearReview: SetIndicatorSchema.optional()
   }).superRefine((value, ctx) => {
-    const clear = value.clearConflicted;
-    if (!clear)
-      return;
-    const markers = "apply" in clear ? clear.apply : [clear];
-    for (const m of markers) {
-      if (m.type !== "label") {
-        ctx.addIssue({
-          code: exports_external.ZodIssueCode.custom,
-          path: ["clearConflicted"],
-          message: "clearConflicted markers must be label-typed (status removal is not supported)"
-        });
-        return;
+    for (const key of ["clearConflicted", "clearReview"]) {
+      const clear = value[key];
+      if (!clear)
+        continue;
+      const markers = "apply" in clear ? clear.apply : [clear];
+      for (const m of markers) {
+        if (m.type !== "label") {
+          ctx.addIssue({
+            code: exports_external.ZodIssueCode.custom,
+            path: [key],
+            message: `${key} markers must be label-typed (status removal is not supported)`
+          });
+          break;
+        }
       }
     }
   });
@@ -59634,8 +59746,8 @@ var init_config = __esm(() => {
 
 // packages/log/src/log.ts
 import { appendFile } from "fs/promises";
-import { join as join11, dirname as dirname4 } from "path";
-import { homedir as homedir2 } from "os";
+import { join as join12, dirname as dirname4 } from "path";
+import { homedir as homedir3 } from "os";
 import { mkdir as mkdir2 } from "fs/promises";
 function fmt(type, text) {
   return `[${new Date().toISOString()}] [${type}] ${text}
@@ -59680,25 +59792,25 @@ async function initWorkerLog(logFile) {
 var ANSI_RE, AGENT_LOG_PATH;
 var init_log = __esm(() => {
   ANSI_RE = /\x1b(?:\[[0-9;]*[A-Za-z]|\][^\x07\x1b]*(?:\x07|\x1b\\)|.)/g;
-  AGENT_LOG_PATH = join11(homedir2(), ".ralph", "agent-mode.log");
+  AGENT_LOG_PATH = join12(homedir3(), ".ralph", "agent-mode.log");
   mkdir2(dirname4(AGENT_LOG_PATH), { recursive: true }).catch(() => {
     return;
   });
 });
 
 // packages/core/src/layout.ts
-import { join as join12 } from "path";
+import { join as join13 } from "path";
 function projectLayout(root) {
-  const statesDir = join12(root, ".ralph", "tasks");
-  const tasksDir = join12(root, "openspec", "changes");
+  const statesDir = join13(root, ".ralph", "tasks");
+  const tasksDir = join13(root, "openspec", "changes");
   return {
     root,
     statesDir,
     tasksDir,
-    agentStateFile: join12(root, ".ralph", "agent-state.json"),
-    changeDir: (name) => join12(tasksDir, name),
-    taskStateDir: (name) => join12(statesDir, name),
-    stateFile: (name) => join12(statesDir, name, STATE_FILE2)
+    agentStateFile: join13(root, ".ralph", "agent-state.json"),
+    changeDir: (name) => join13(tasksDir, name),
+    taskStateDir: (name) => join13(statesDir, name),
+    stateFile: (name) => join13(statesDir, name, STATE_FILE2)
   };
 }
 var STATE_FILE2 = ".ralph-state.json";
@@ -59753,7 +59865,15 @@ function buildIssueFilter(spec) {
         where.and = [{ state: current }, noStatus];
     }
     if (labels.length > 0) {
-      where.labels = { ...where.labels, every: { name: { nin: labels } } };
+      const includeLabels = where.labels;
+      const excludeLabels = { every: { name: { nin: labels } } };
+      if (includeLabels === undefined) {
+        where.labels = excludeLabels;
+      } else {
+        const existingAnd = where.and ?? [];
+        where.and = [...existingAnd, { labels: includeLabels }, { labels: excludeLabels }];
+        delete where.labels;
+      }
     }
   }
   return where;
@@ -59949,19 +60069,21 @@ class AgentCoordinator {
     let todo = [];
     let inProgress = [];
     let conflicted = [];
+    let review = [];
     try {
-      [todo, inProgress, conflicted] = await Promise.all([
+      [todo, inProgress, conflicted, review] = await Promise.all([
         this.deps.fetchTodo(),
         this.deps.fetchInProgress(),
-        this.deps.fetchConflicted()
+        this.deps.fetchConflicted(),
+        this.deps.fetchReview()
       ]);
     } catch (err) {
       this.deps.onLog(`! Linear poll failed: ${err.message}`, "red");
       capture("agent_linear_poll_failed", { error: err.message });
       return { found: 0, added: 0 };
     }
-    if (todo.length + inProgress.length + conflicted.length > 0) {
-      this.deps.onLog(`  poll: ${todo.length} todo, ${inProgress.length} in-progress, ${conflicted.length} conflicted`, "gray");
+    if (todo.length + inProgress.length + conflicted.length + review.length > 0) {
+      this.deps.onLog(`  poll: ${todo.length} todo, ${inProgress.length} in-progress, ${conflicted.length} conflicted, ${review.length} review`, "gray");
     }
     const queuedIds = new Set(this.queue.map((q) => q.issue.id));
     const activeIds = new Set(this.workers.map((w) => w.issueId));
@@ -59996,6 +60118,16 @@ class AgentCoordinator {
       added += 1;
       this.deps.onLog(`  \u21B3 ${issue.identifier} queued (conflict-fix)`, "gray");
     }
+    for (const issue of review) {
+      if (atTicketLimit())
+        break;
+      if (!eligible(issue.id))
+        continue;
+      this.queue.push({ issue, mode: "review" });
+      queuedIds.add(issue.id);
+      added += 1;
+      this.deps.onLog(`  \u21B3 ${issue.identifier} queued (review)`, "gray");
+    }
     for (const issue of todo) {
       if (atTicketLimit())
         break;
@@ -60012,7 +60144,8 @@ class AgentCoordinator {
       const modeRank = {
         resume: 0,
         "conflict-fix": 1,
-        fresh: 2
+        review: 2,
+        fresh: 3
       };
       this.queue.sort((a, b) => {
         const pa = a.issue.priority === 0 ? Infinity : a.issue.priority;
@@ -60025,7 +60158,7 @@ class AgentCoordinator {
     this.spawnNext();
     await this.scanDoneForConflicts();
     await this.reportProgress();
-    const found = todo.length + inProgress.length + conflicted.length;
+    const found = todo.length + inProgress.length + conflicted.length + review.length;
     return { found, added };
   }
   dependenciesResolved(issue) {
@@ -60169,6 +60302,26 @@ class AgentCoordinator {
         });
       }
     }
+    if (mode === "review" && this.opts.clearReview) {
+      try {
+        await this.deps.removeIndicator(issue, this.opts.clearReview);
+        this.deps.onLog(`  ${issue.identifier}: clearReview applied`, "gray");
+      } catch (err) {
+        this.deps.onLog(`! Linear clearReview failed for ${issue.identifier}: ${err.message}`, "yellow");
+        capture("agent_indicator_failed", {
+          indicator: "clearReview",
+          issue_identifier: issue.identifier,
+          error: err.message
+        });
+      }
+    }
+    if (mode === "review" && this.opts.postComments !== false) {
+      try {
+        await this.deps.postComment(issue, `\uD83D\uDD01 Ralph picked up new review comments. Tracking change: \`${prep.changeName}\``);
+      } catch (err) {
+        this.deps.onLog(`! Linear review comment failed for ${issue.identifier}: ${err.message}`, "yellow");
+      }
+    }
     if (mode === "fresh" && this.opts.postComments !== false) {
       let alreadyPosted = false;
       try {
@@ -60308,7 +60461,7 @@ var init_coordinator = __esm(() => {
 });
 
 // apps/cli/src/agent/scaffold.ts
-import { join as join13 } from "path";
+import { join as join14 } from "path";
 import { mkdir as mkdir3 } from "fs/promises";
 function changeNameForIssue(issue) {
   const slug = issue.title.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-+|-+$/g, "").slice(0, 40);
@@ -60316,10 +60469,10 @@ function changeNameForIssue(issue) {
 }
 async function scaffoldChangeForIssue(tasksDir, statesDir, issue, comments = [], appendPrompt = "") {
   const name = changeNameForIssue(issue);
-  const changeDir = join13(tasksDir, name);
-  const stateDir = join13(statesDir, name);
+  const changeDir = join14(tasksDir, name);
+  const stateDir = join14(statesDir, name);
   await mkdir3(changeDir, { recursive: true });
-  await mkdir3(join13(changeDir, "specs"), { recursive: true });
+  await mkdir3(join14(changeDir, "specs"), { recursive: true });
   await mkdir3(stateDir, { recursive: true });
   const commentsBlock = comments.length > 0 ? [
     "",
@@ -60371,106 +60524,13 @@ async function scaffoldChangeForIssue(tasksDir, statesDir, issue, comments = [],
     ""
   ].join(`
 `);
-  await Bun.write(join13(changeDir, "proposal.md"), proposal);
-  await Bun.write(join13(changeDir, "tasks.md"), tasks);
-  await Bun.write(join13(changeDir, "design.md"), design);
+  await Bun.write(join14(changeDir, "proposal.md"), proposal);
+  await Bun.write(join14(changeDir, "tasks.md"), tasks);
+  await Bun.write(join14(changeDir, "design.md"), design);
   return name;
 }
 var init_scaffold = () => {};
 
-// apps/cli/src/agent/worktree.ts
-import { basename, join as join14 } from "path";
-import { homedir as homedir3 } from "os";
-import { exists } from "fs/promises";
-function worktreesDir(projectRoot) {
-  return join14(homedir3(), ".ralph", basename(projectRoot), "worktrees");
-}
-function branchForChange(changeName) {
-  return `ralph/${changeName}`;
-}
-async function createWorktree(projectRoot, changeName, runner) {
-  const dir = worktreesDir(projectRoot);
-  const cwd2 = join14(dir, changeName);
-  const branch = branchForChange(changeName);
-  const list = await runner.run(["worktree", "list", "--porcelain"], projectRoot);
-  if (list.stdout.includes(`worktree ${cwd2}
-`)) {
-    return { cwd: cwd2, branch };
-  }
-  let branchExists = true;
-  try {
-    await runner.run(["rev-parse", "--verify", "--quiet", `refs/heads/${branch}`], projectRoot);
-  } catch {
-    branchExists = false;
-  }
-  const cmd = branchExists ? ["worktree", "add", cwd2, branch] : ["worktree", "add", "-b", branch, cwd2];
-  await runner.run(cmd, projectRoot);
-  return { cwd: cwd2, branch };
-}
-async function removeWorktree(projectRoot, cwd2, runner) {
-  await runner.run(["worktree", "remove", "--force", cwd2], projectRoot);
-}
-async function isWorktreeSafeToRemove(cwd2, base2, runner) {
-  const status = await runner.run(["status", "--porcelain"], cwd2);
-  const dirty = status.stdout.trim();
-  let unpushedCommits = "";
-  try {
-    const log2 = await runner.run(["log", "--oneline", `${base2}..HEAD`, "--no-merges"], cwd2);
-    unpushedCommits = log2.stdout.trim();
-  } catch {
-    unpushedCommits = "<unknown: failed to compare against base>";
-  }
-  if (dirty && unpushedCommits) {
-    return {
-      safe: false,
-      reason: "uncommitted changes AND unpushed commits present",
-      dirty,
-      unpushedCommits
-    };
-  }
-  if (dirty) {
-    return {
-      safe: false,
-      reason: "uncommitted or untracked files present",
-      dirty,
-      unpushedCommits
-    };
-  }
-  if (unpushedCommits) {
-    return {
-      safe: false,
-      reason: `commits ahead of ${base2} were not pushed/PR'd`,
-      dirty,
-      unpushedCommits
-    };
-  }
-  return { safe: true, dirty, unpushedCommits };
-}
-async function seedWorktreeMcpConfig(projectRoot, worktreeCwd) {
-  const dst = join14(worktreeCwd, ".mcp.json");
-  const src = join14(projectRoot, ".mcp.json");
-  const source = await exists(dst) ? dst : await exists(src) ? src : null;
-  if (!source)
-    return;
-  let parsed;
-  try {
-    parsed = await Bun.file(source).json();
-  } catch {
-    return;
-  }
-  const servers = parsed.mcpServers;
-  if (servers && typeof servers === "object") {
-    for (const cfg of Object.values(servers)) {
-      if (Array.isArray(cfg.args)) {
-        cfg.args = cfg.args.map((a) => typeof a === "string" && a.startsWith(".ralph/") ? join14(projectRoot, a) : a);
-      }
-    }
-  }
-  await Bun.write(dst, JSON.stringify(parsed, null, 2) + `
-`);
-}
-var init_worktree = () => {};
-
 // apps/cli/src/agent/pr.ts
 function defaultTitle(issue) {
   return `${issue.identifier}: ${issue.title}`;
@@ -61024,6 +61084,30 @@ function mergeIndicators(cfg, cli) {
   }
   return out;
 }
+function isRalphComment(body) {
+  const trimmed = body.trimStart();
+  return /^(\uD83E\uDD16|\uD83D\uDD04|\u2705|\u2717|\u26A0|\uD83D\uDD01)\s*Ralph\b/.test(trimmed);
+}
+function buildReviewTaskBody(comments, url) {
+  if (comments.length === 0) {
+    return `No non-Ralph reviewer comments were found on ${url}. Recheck the issue manually before continuing.`;
+  }
+  const blocks = comments.map((c) => {
+    const author = c.user?.name ?? "unknown";
+    return `**${author}** \u2014 ${c.createdAt}
+
+${c.body.trim()}`;
+  });
+  return [
+    `Reviewer comments left on the Linear issue (${url}):`,
+    "",
+    ...blocks,
+    "",
+    "Address every concrete request above. If a comment is ambiguous, note",
+    "your interpretation in proposal.md `## Steering` before acting."
+  ].join(`
+`);
+}
 function unionMarkers(...sets) {
   const out = [];
   const seen = new Set;
@@ -61063,6 +61147,7 @@ function buildAgentCoordinator(input) {
   const team = args.linearTeam || cfg.linear.team;
   const assignee = args.linearAssignee || cfg.linear.assignee;
   const excludeFromTodo = unionMarkers(indicators.setDone, indicators.setError, indicators.setConflicted);
+  const excludeFromReview = unionMarkers(indicators.setInProgress, indicators.setError, indicators.setConflicted);
   const gitRunner = input.runners?.git ?? bunGitRunner;
   const cmdRunner = input.runners?.cmd ?? bunCmdRunner;
   const stateCache = new Map;
@@ -61236,7 +61321,8 @@ function buildAgentCoordinator(input) {
   async function prepare(issue, mode) {
     const { workerCwd, scaffoldTasksDir, scaffoldStatesDir, branch } = await setupWorktree(issue);
     let changeName;
-    if (mode === "fresh") {
+    const isFresh = mode === "fresh";
+    if (isFresh) {
       let comments = [];
       try {
         comments = await fetchIssueComments(apiKey, issue.id);
@@ -61256,7 +61342,24 @@ function buildAgentCoordinator(input) {
     issueByChange.set(changeName, issue);
     if (branch)
       branchByChange.set(changeName, branch);
-    if (mode === "conflict-fix") {
+    if (mode === "review") {
+      const wtLayout = projectLayout(workerCwd);
+      const tasksFile = join16(wtLayout.changeDir(changeName), "tasks.md");
+      let comments = [];
+      try {
+        comments = await fetchIssueComments(apiKey, issue.id);
+      } catch (err) {
+        onLog(`! Linear comment fetch failed for ${issue.identifier}: ${err.message}`, "yellow");
+      }
+      const reviewerComments = comments.filter((c) => !isRalphComment(c.body));
+      const body = buildReviewTaskBody(reviewerComments, issue.url);
+      try {
+        await prependFixTask(tasksFile, "Address reviewer comments", body);
+      } catch (err) {
+        onLog(`! could not prepend review task: ${err.message}`, "red");
+      }
+      await reactivateState2(wtLayout.stateFile(changeName), changeName);
+    } else if (mode === "conflict-fix") {
       const wtLayout = projectLayout(workerCwd);
       const tasksFile = join16(wtLayout.changeDir(changeName), "tasks.md");
       const prUrl = prByChange.get(changeName);
@@ -61531,6 +61634,7 @@ PR: ${prUrl}` : ""
     fetchTodo: () => fetchByGet(indicators.getTodo, excludeFromTodo),
     fetchInProgress: () => fetchByGet(indicators.getInProgress, []),
     fetchConflicted: () => fetchByGet(indicators.getConflicted, []),
+    fetchReview: () => fetchByGet(indicators.getReview, excludeFromReview),
     fetchDoneCandidates,
     prepare,
     spawnWorker,
@@ -61559,6 +61663,7 @@ PR: ${prUrl}` : ""
     ...indicators.setError !== undefined ? { setError: indicators.setError } : {},
     ...indicators.setConflicted !== undefined ? { setConflicted: indicators.setConflicted } : {},
     ...indicators.clearConflicted !== undefined ? { clearConflicted: indicators.clearConflicted } : {},
+    ...indicators.clearReview !== undefined ? { clearReview: indicators.clearReview } : {},
     postComments: cfg.linear.postComments,
     commentEveryIterations: cfg.linear.updateEveryIterations,
     ...args.maxTickets > 0 ? { maxTickets: args.maxTickets } : {}
@@ -61585,6 +61690,9 @@ function describeIndicators(indicators, team, assignee) {
   if (indicators.getConflicted) {
     parts.push(`conflicted=[${indicators.getConflicted.filter.map((m) => `${m.type}:${m.value}`).join(",")}]`);
   }
+  if (indicators.getReview) {
+    parts.push(`review=[${indicators.getReview.filter.map((m) => `${m.type}:${m.value}`).join(",")}]`);
+  }
   return parts.join(", ");
 }
 var bunGitRunner, bunCmdRunner;
@@ -66963,7 +67071,8 @@ function ensureState(changeDir) {
 
 // apps/cli/src/components/TaskList.tsx
 var import_react22 = __toESM(require_react(), 1);
-import { join as join3 } from "path";
+import { join as join4 } from "path";
+init_worktree();
 var jsx_dev_runtime = __toESM(require_jsx_dev_runtime(), 1);
 function countTaskItems(content) {
   const checked = (content.match(/^- \[x\]/gm) ?? []).length;
@@ -66976,10 +67085,10 @@ function buildRows(statesDir, projectRoot) {
   const seenNames = new Set;
   const sources = [{ dir: statesDir, label: "main" }];
   if (projectRoot) {
-    const worktreesRoot = join3(projectRoot, ".ralph", "worktrees");
+    const worktreesRoot = worktreesDir(projectRoot);
     for (const wt of storage.list(worktreesRoot)) {
       sources.push({
-        dir: join3(worktreesRoot, wt, ".ralph", "tasks"),
+        dir: join4(worktreesRoot, wt, ".ralph", "tasks"),
         label: `wt:${wt}`
       });
     }
@@ -66988,7 +67097,7 @@ function buildRows(statesDir, projectRoot) {
     for (const entry of storage.list(dir)) {
       if (seenNames.has(entry))
         continue;
-      const raw = storage.read(join3(dir, entry, ".ralph-state.json"));
+      const raw = storage.read(join4(dir, entry, ".ralph-state.json"));
       if (raw === null)
         continue;
       let state;
@@ -67004,7 +67113,7 @@ function buildRows(statesDir, projectRoot) {
 `).find((l) => l.trim() !== "") ?? "";
       let progress = "\u2014";
       let progressStyled = true;
-      const tasksContent = storage.read(join3(dir, entry, "tasks.md"));
+      const tasksContent = storage.read(join4(dir, entry, "tasks.md"));
       if (tasksContent !== null) {
         const { checked, unchecked } = countTaskItems(tasksContent);
         const total = checked + unchecked;
@@ -67145,7 +67254,7 @@ function TaskList({ statesDir, projectRoot }) {
 }
 
 // apps/cli/src/components/TaskStatus.tsx
-import { join as join4 } from "path";
+import { join as join5 } from "path";
 var jsx_dev_runtime2 = __toESM(require_jsx_dev_runtime(), 1);
 var HEAVY_RULE = "============================================";
 var LIGHT_RULE = "--------------------------------------------";
@@ -67156,7 +67265,7 @@ function TaskStatus({ state, stateDir }) {
   const time = Math.round(state.usage.total_duration_ms / 1000 * 10) / 10 + "s";
   const artifacts = OPENSPEC_ARTIFACTS.map((name) => ({
     name,
-    exists: storage.read(join4(stateDir, name)) !== null
+    exists: storage.read(join5(stateDir, name)) !== null
   }));
   const recent = state.history.slice(-10);
   return /* @__PURE__ */ jsx_dev_runtime2.jsxDEV(Box_default, {
@@ -67299,7 +67408,7 @@ function TaskStatus({ state, stateDir }) {
 
 // apps/cli/src/components/TaskLoop.tsx
 var import_react56 = __toESM(require_react(), 1);
-import { join as join9 } from "path";
+import { join as join10 } from "path";
 
 // node_modules/.bun/@inkjs+ui@2.0.0+5b84dde3d6cd3930/node_modules/@inkjs/ui/build/components/badge/badge.js
 var import_react24 = __toESM(require_react(), 1);
@@ -70863,7 +70972,7 @@ function StopMessage({
 
 // apps/cli/src/hooks/useLoop.ts
 var import_react55 = __toESM(require_react(), 1);
-import { join as join8 } from "path";
+import { join as join9 } from "path";
 
 // packages/engine/src/spawn.ts
 var {spawn: bunSpawn } = globalThis.Bun;
@@ -70873,7 +70982,7 @@ var spawn = bunSpawn;
 import { createWriteStream } from "fs";
 import { mkdtemp, unlink, mkdir } from "fs/promises";
 import { dirname as dirname2 } from "path";
-import { join as join5 } from "path";
+import { join as join6 } from "path";
 import { tmpdir } from "os";
 
 // packages/engine/src/feed-events.ts
@@ -71606,7 +71715,7 @@ function buildCodexArgs() {
   return ["exec", "--json", "--color", "never", "--dangerously-bypass-approvals-and-sandbox", "-"];
 }
 async function runInteractive(model, prompt, taskDir) {
-  const promptFile = taskDir ? join5(taskDir, "_interactive_prompt.md") : join5(await mkdtemp(join5(tmpdir(), "ralph-")), "prompt.md");
+  const promptFile = taskDir ? join6(taskDir, "_interactive_prompt.md") : join6(await mkdtemp(join6(tmpdir(), "ralph-")), "prompt.md");
   await Bun.write(promptFile, prompt);
   try {
     const cmd = [
@@ -71632,7 +71741,7 @@ async function runInteractive(model, prompt, taskDir) {
       stderr: "inherit"
     });
     const exitCode = await proc.exited;
-    const doneFile = taskDir ? join5(taskDir, "_interactive_done") : null;
+    const doneFile = taskDir ? join6(taskDir, "_interactive_done") : null;
     if (doneFile && await Bun.file(doneFile).exists()) {
       return { exitCode: 0, usage: null, sessionId: null, rateLimited: false };
     }
@@ -71849,12 +71958,12 @@ function commitTaskDir(taskDir, message) {
 init_src();
 
 // packages/core/src/loop.ts
-import { join as join7 } from "path";
+import { join as join8 } from "path";
 var STEERING_MAX_LINES = 20;
 function buildTaskPrompt(state, taskDir) {
   const storage = getStorage();
   let prompt = "";
-  const steeringContent = storage.read(join7(taskDir, "steering.md"));
+  const steeringContent = storage.read(join8(taskDir, "steering.md"));
   if (steeringContent !== null) {
     const steeringLines = steeringContent.split(`
 `).filter((line) => !line.startsWith("#")).filter((line) => line.trim()).slice(0, STEERING_MAX_LINES);
@@ -71873,7 +71982,7 @@ function buildTaskPrompt(state, taskDir) {
 `;
     }
   }
-  const tasksContent = storage.read(join7(taskDir, "tasks.md"));
+  const tasksContent = storage.read(join8(taskDir, "tasks.md"));
   if (tasksContent !== null) {
     const section = firstUnchecked(tasksContent);
     if (section) {
@@ -71888,7 +71997,7 @@ function buildTaskPrompt(state, taskDir) {
       prompt += `---
 
 `;
-      prompt += `**Tracking progress**: as you finish each item above, edit ` + `\`${join7(taskDir, "tasks.md")}\` and change its \`- [ ]\` to ` + `\`- [x]\` in the same commit. The loop reads this file between ` + `iterations and stops when no \`- [ ]\` items remain \u2014 if you do ` + `not tick the box, the next iteration will repeat this task.
+      prompt += `**Tracking progress**: as you finish each item above, edit ` + `\`${join8(taskDir, "tasks.md")}\` and change its \`- [ ]\` to ` + `\`- [x]\` in the same commit. The loop reads this file between ` + `iterations and stops when no \`- [ ]\` items remain \u2014 if you do ` + `not tick the box, the next iteration will repeat this task.
 
 `;
     }
@@ -71909,7 +72018,7 @@ function buildTaskPrompt(state, taskDir) {
 `;
   }
   if (state.manualTest) {
-    const tasksContent2 = storage.read(join7(taskDir, "tasks.md"));
+    const tasksContent2 = storage.read(join8(taskDir, "tasks.md"));
     const hasUncheckedTasks = tasksContent2 !== null && /^- \[ \]/m.test(tasksContent2);
     if (!hasUncheckedTasks) {
       const hasManualTestSection = tasksContent2 !== null && /^## Manual Testing/m.test(tasksContent2);
@@ -71958,7 +72067,7 @@ When all tasks are complete and all files are committed, push your branch and op
 }
 function checkStopSignal(taskDir, stateDir) {
   const storage = getStorage();
-  const stopFile = join7(taskDir, "STOP");
+  const stopFile = join8(taskDir, "STOP");
   const reason = storage.read(stopFile);
   if (reason === null)
     return null;
@@ -72033,7 +72142,7 @@ function updateStateIteration(stateDir, result2, startedAt, engine, model, usage
 }
 function appendSteeringMessage(taskDir, message) {
   const storage = getStorage();
-  const steeringPath = join7(taskDir, "steering.md");
+  const steeringPath = join8(taskDir, "steering.md");
   const existing = storage.read(steeringPath);
   const updated = existing ? `${message}
 
@@ -72100,11 +72209,11 @@ function useLoop(opts) {
       setLogLines((prev) => [...prev, { id: nextId(), kind: "feed", event }]);
     };
     runWithContext(createDefaultContext(), async () => {
-      const stateDir = join8(opts.statesDir, opts.name);
-      const tasksDir = join8(opts.tasksDir, opts.name);
+      const stateDir = join9(opts.statesDir, opts.name);
+      const tasksDir = join9(opts.tasksDir, opts.name);
       const storage = getStorage();
       let currentState;
-      const existingStateRaw = storage.read(join8(stateDir, ".ralph-state.json"));
+      const existingStateRaw = storage.read(join9(stateDir, ".ralph-state.json"));
       if (existingStateRaw !== null) {
         currentState = readState(stateDir);
         if (currentState.engine !== opts.engine || currentState.model !== opts.model) {
@@ -72151,7 +72260,7 @@ function useLoop(opts) {
           setStopReason(stop);
           break;
         }
-        const tasksContent = storage.read(join8(tasksDir, "tasks.md"));
+        const tasksContent = storage.read(join9(tasksDir, "tasks.md"));
         if (tasksContent !== null) {
           const remaining = countUnchecked(tasksContent);
           addInfo(`tasks.md: ${remaining} unchecked item${remaining === 1 ? "" : "s"} remaining`);
@@ -72191,7 +72300,7 @@ function useLoop(opts) {
             model: opts.model,
             prompt,
             logFlag: opts.log,
-            logFile: join8(stateDir, "log.json"),
+            logFile: join9(stateDir, "log.json"),
             taskDir: tasksDir,
             interactive: false,
             onFeedEvent: addFeedEvent,
@@ -72214,7 +72323,7 @@ function useLoop(opts) {
               model: opts.model,
               prompt: buildSteeringPrompt(steerMessage),
               logFlag: opts.log,
-              logFile: join8(stateDir, "log.json"),
+              logFile: join9(stateDir, "log.json"),
               taskDir: tasksDir,
               onFeedEvent: addResumeFeedEvent,
               signal: resumeController.signal,
@@ -72409,7 +72518,7 @@ function TaskLoop({ opts }) {
   }, [loop.isRunning, exit]);
   if (!loop.state)
     return null;
-  const stateDir = join9(opts.statesDir, opts.name);
+  const stateDir = join10(opts.statesDir, opts.name);
   return /* @__PURE__ */ jsx_dev_runtime8.jsxDEV(Box_default, {
     flexDirection: "column",
     children: [

package/dist/mcp/index.js

@@ -3047,10 +3047,13 @@ var require_data = __commonJS((exports, module) => {
   };
 });
 
-// node_modules/.bun/fast-uri@3.1.0/node_modules/fast-uri/lib/utils.js
+// node_modules/.bun/fast-uri@3.1.2/node_modules/fast-uri/lib/utils.js
 var require_utils = __commonJS((exports, module) => {
   var isUUID = RegExp.prototype.test.bind(/^[\da-f]{8}-[\da-f]{4}-[\da-f]{4}-[\da-f]{4}-[\da-f]{12}$/iu);
   var isIPv4 = RegExp.prototype.test.bind(/^(?:(?:25[0-5]|2[0-4]\d|1\d{2}|[1-9]\d|\d)\.){3}(?:25[0-5]|2[0-4]\d|1\d{2}|[1-9]\d|\d)$/u);
+  var isHexPair = RegExp.prototype.test.bind(/^[\da-f]{2}$/iu);
+  var isUnreserved = RegExp.prototype.test.bind(/^[\da-z\-._~]$/iu);
+  var isPathCharacter = RegExp.prototype.test.bind(/^[\da-z\-._~!$&'()*+,;=:@/]$/iu);
   function stringArrayToHexStripped(input) {
     let acc = "";
     let code = 0;
@@ -3244,27 +3247,77 @@ var require_utils = __commonJS((exports, module) => {
     }
     return output.join("");
   }
-  function normalizeComponentEncoding(component, esc2) {
-    const func = esc2 !== true ? escape : unescape;
-    if (component.scheme !== undefined) {
-      component.scheme = func(component.scheme);
-    }
-    if (component.userinfo !== undefined) {
-      component.userinfo = func(component.userinfo);
-    }
-    if (component.host !== undefined) {
-      component.host = func(component.host);
+  var HOST_DELIMS = { "@": "%40", "/": "%2F", "?": "%3F", "#": "%23", ":": "%3A" };
+  var HOST_DELIM_RE = /[@/?#:]/g;
+  var HOST_DELIM_NO_COLON_RE = /[@/?#]/g;
+  function reescapeHostDelimiters(host, isIP) {
+    const re = isIP ? HOST_DELIM_NO_COLON_RE : HOST_DELIM_RE;
+    re.lastIndex = 0;
+    return host.replace(re, (ch) => HOST_DELIMS[ch]);
+  }
+  function normalizePercentEncoding(input, decodeUnreserved = false) {
+    if (input.indexOf("%") === -1) {
+      return input;
     }
-    if (component.path !== undefined) {
-      component.path = func(component.path);
+    let output = "";
+    for (let i = 0;i < input.length; i++) {
+      if (input[i] === "%" && i + 2 < input.length) {
+        const hex = input.slice(i + 1, i + 3);
+        if (isHexPair(hex)) {
+          const normalizedHex = hex.toUpperCase();
+          const decoded = String.fromCharCode(parseInt(normalizedHex, 16));
+          if (decodeUnreserved && isUnreserved(decoded)) {
+            output += decoded;
+          } else {
+            output += "%" + normalizedHex;
+          }
+          i += 2;
+          continue;
+        }
+      }
+      output += input[i];
     }
-    if (component.query !== undefined) {
-      component.query = func(component.query);
+    return output;
+  }
+  function normalizePathEncoding(input) {
+    let output = "";
+    for (let i = 0;i < input.length; i++) {
+      if (input[i] === "%" && i + 2 < input.length) {
+        const hex = input.slice(i + 1, i + 3);
+        if (isHexPair(hex)) {
+          const normalizedHex = hex.toUpperCase();
+          const decoded = String.fromCharCode(parseInt(normalizedHex, 16));
+          if (decoded !== "." && isUnreserved(decoded)) {
+            output += decoded;
+          } else {
+            output += "%" + normalizedHex;
+          }
+          i += 2;
+          continue;
+        }
+      }
+      if (isPathCharacter(input[i])) {
+        output += input[i];
+      } else {
+        output += escape(input[i]);
+      }
     }
-    if (component.fragment !== undefined) {
-      component.fragment = func(component.fragment);
+    return output;
+  }
+  function escapePreservingEscapes(input) {
+    let output = "";
+    for (let i = 0;i < input.length; i++) {
+      if (input[i] === "%" && i + 2 < input.length) {
+        const hex = input.slice(i + 1, i + 3);
+        if (isHexPair(hex)) {
+          output += "%" + hex.toUpperCase();
+          i += 2;
+          continue;
+        }
+      }
+      output += escape(input[i]);
     }
-    return component;
+    return output;
   }
   function recomposeAuthority(component) {
     const uriTokens = [];
@@ -3279,7 +3332,7 @@ var require_utils = __commonJS((exports, module) => {
         if (ipV6res.isIPV6 === true) {
           host = `[${ipV6res.escapedHost}]`;
         } else {
-          host = component.host;
+          host = reescapeHostDelimiters(host, false);
         }
       }
       uriTokens.push(host);
@@ -3293,7 +3346,10 @@ var require_utils = __commonJS((exports, module) => {
   module.exports = {
     nonSimpleDomain,
     recomposeAuthority,
-    normalizeComponentEncoding,
+    reescapeHostDelimiters,
+    normalizePercentEncoding,
+    normalizePathEncoding,
+    escapePreservingEscapes,
     removeDotSegments,
     isIPv4,
     isUUID,
@@ -3302,7 +3358,7 @@ var require_utils = __commonJS((exports, module) => {
   };
 });
 
-// node_modules/.bun/fast-uri@3.1.0/node_modules/fast-uri/lib/schemes.js
+// node_modules/.bun/fast-uri@3.1.2/node_modules/fast-uri/lib/schemes.js
 var require_schemes = __commonJS((exports, module) => {
   var { isUUID } = require_utils();
   var URN_REG = /([\da-z][\d\-a-z]{0,31}):((?:[\w!$'()*+,\-.:;=@]|%[\da-f]{2})+)/iu;
@@ -3476,13 +3532,13 @@ var require_schemes = __commonJS((exports, module) => {
   };
 });
 
-// node_modules/.bun/fast-uri@3.1.0/node_modules/fast-uri/index.js
+// node_modules/.bun/fast-uri@3.1.2/node_modules/fast-uri/index.js
 var require_fast_uri = __commonJS((exports, module) => {
-  var { normalizeIPv6, removeDotSegments, recomposeAuthority, normalizeComponentEncoding, isIPv4, nonSimpleDomain } = require_utils();
+  var { normalizeIPv6, removeDotSegments, recomposeAuthority, normalizePercentEncoding, normalizePathEncoding, escapePreservingEscapes, reescapeHostDelimiters, isIPv4, nonSimpleDomain } = require_utils();
   var { SCHEMES, getSchemeHandler } = require_schemes();
   function normalize(uri, options) {
     if (typeof uri === "string") {
-      uri = serialize(parse6(uri, options), options);
+      uri = normalizeString(uri, options);
     } else if (typeof uri === "object") {
       uri = parse6(serialize(uri, options), options);
     }
@@ -3548,19 +3604,9 @@ var require_fast_uri = __commonJS((exports, module) => {
     return target;
   }
   function equal(uriA, uriB, options) {
-    if (typeof uriA === "string") {
-      uriA = unescape(uriA);
-      uriA = serialize(normalizeComponentEncoding(parse6(uriA, options), true), { ...options, skipEscape: true });
-    } else if (typeof uriA === "object") {
-      uriA = serialize(normalizeComponentEncoding(uriA, true), { ...options, skipEscape: true });
-    }
-    if (typeof uriB === "string") {
-      uriB = unescape(uriB);
-      uriB = serialize(normalizeComponentEncoding(parse6(uriB, options), true), { ...options, skipEscape: true });
-    } else if (typeof uriB === "object") {
-      uriB = serialize(normalizeComponentEncoding(uriB, true), { ...options, skipEscape: true });
-    }
-    return uriA.toLowerCase() === uriB.toLowerCase();
+    const normalizedA = normalizeComparableURI(uriA, options);
+    const normalizedB = normalizeComparableURI(uriB, options);
+    return normalizedA !== undefined && normalizedB !== undefined && normalizedA.toLowerCase() === normalizedB.toLowerCase();
   }
   function serialize(cmpts, opts) {
     const component = {
@@ -3586,12 +3632,12 @@ var require_fast_uri = __commonJS((exports, module) => {
       schemeHandler.serialize(component, options);
     if (component.path !== undefined) {
       if (!options.skipEscape) {
-        component.path = escape(component.path);
+        component.path = escapePreservingEscapes(component.path);
         if (component.scheme !== undefined) {
           component.path = component.path.split("%3A").join(":");
         }
       } else {
-        component.path = unescape(component.path);
+        component.path = normalizePercentEncoding(component.path);
       }
     }
     if (options.reference !== "suffix" && component.scheme) {
@@ -3626,7 +3672,16 @@ var require_fast_uri = __commonJS((exports, module) => {
     return uriTokens.join("");
   }
   var URI_PARSE = /^(?:([^#/:?]+):)?(?:\/\/((?:([^#/?@]*)@)?(\[[^#/?\]]+\]|[^#/:?]*)(?::(\d*))?))?([^#?]*)(?:\?([^#]*))?(?:#((?:.|[\n\r])*))?/u;
-  function parse6(uri, opts) {
+  function getParseError(parsed, matches) {
+    if (matches[2] !== undefined && parsed.path && parsed.path[0] !== "/") {
+      return 'URI path must start with "/" when authority is present.';
+    }
+    if (typeof parsed.port === "number" && (parsed.port < 0 || parsed.port > 65535)) {
+      return "URI port is malformed.";
+    }
+    return;
+  }
+  function parseWithStatus(uri, opts) {
     const options = Object.assign({}, opts);
     const parsed = {
       scheme: undefined,
@@ -3637,6 +3692,7 @@ var require_fast_uri = __commonJS((exports, module) => {
       query: undefined,
       fragment: undefined
     };
+    let malformedAuthorityOrPort = false;
     let isIP = false;
     if (options.reference === "suffix") {
       if (options.scheme) {
@@ -3657,6 +3713,11 @@ var require_fast_uri = __commonJS((exports, module) => {
       if (isNaN(parsed.port)) {
         parsed.port = matches[5];
       }
+      const parseError = getParseError(parsed, matches);
+      if (parseError !== undefined) {
+        parsed.error = parsed.error || parseError;
+        malformedAuthorityOrPort = true;
+      }
       if (parsed.host) {
         const ipv4result = isIPv4(parsed.host);
         if (ipv4result === false) {
@@ -3695,14 +3756,18 @@ var require_fast_uri = __commonJS((exports, module) => {
             parsed.scheme = unescape(parsed.scheme);
           }
           if (parsed.host !== undefined) {
-            parsed.host = unescape(parsed.host);
+            parsed.host = reescapeHostDelimiters(unescape(parsed.host), isIP);
           }
         }
         if (parsed.path) {
-          parsed.path = escape(unescape(parsed.path));
+          parsed.path = normalizePathEncoding(parsed.path);
         }
         if (parsed.fragment) {
-          parsed.fragment = encodeURI(decodeURIComponent(parsed.fragment));
+          try {
+            parsed.fragment = encodeURI(decodeURIComponent(parsed.fragment));
+          } catch {
+            parsed.error = parsed.error || "URI malformed";
+          }
         }
       }
       if (schemeHandler && schemeHandler.parse) {
@@ -3711,7 +3776,29 @@ var require_fast_uri = __commonJS((exports, module) => {
     } else {
       parsed.error = parsed.error || "URI can not be parsed.";
     }
-    return parsed;
+    return { parsed, malformedAuthorityOrPort };
+  }
+  function parse6(uri, opts) {
+    return parseWithStatus(uri, opts).parsed;
+  }
+  function normalizeString(uri, opts) {
+    return normalizeStringWithStatus(uri, opts).normalized;
+  }
+  function normalizeStringWithStatus(uri, opts) {
+    const { parsed, malformedAuthorityOrPort } = parseWithStatus(uri, opts);
+    return {
+      normalized: malformedAuthorityOrPort ? uri : serialize(parsed, opts),
+      malformedAuthorityOrPort
+    };
+  }
+  function normalizeComparableURI(uri, opts) {
+    if (typeof uri === "string") {
+      const { normalized, malformedAuthorityOrPort } = normalizeStringWithStatus(uri, opts);
+      return malformedAuthorityOrPort ? undefined : normalized;
+    }
+    if (typeof uri === "object") {
+      return serialize(uri, opts);
+    }
   }
   var fastUri = {
     SCHEMES,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@neriros/ralphy",
-  "version": "2.17.2",
+  "version": "2.18.0",
   "description": "An iterative AI task execution framework. Orchestrates multi-phase autonomous work using Claude or Codex engines.",
   "keywords": [
     "agent",
@@ -57,33 +57,35 @@
     "prepublishOnly": "bun run build:publish"
   },
   "devDependencies": {
-    "@commitlint/cli": "^20.4.3",
-    "@commitlint/config-conventional": "^20.4.3",
+    "@commitlint/cli": "^20.5.3",
+    "@commitlint/config-conventional": "^20.5.3",
     "@fission-ai/openspec": "latest",
-    "@modelcontextprotocol/sdk": "^1.12.0",
+    "@modelcontextprotocol/sdk": "^1.29.0",
     "@nx/devkit": "^22.7.1",
     "@nx/js": "^22.7.1",
-    "@secretlint/secretlint-rule-preset-recommend": "^11.3.1",
+    "@secretlint/secretlint-rule-preset-recommend": "^11.7.1",
     "@swc-node/register": "^1.11.1",
-    "@swc/core": "^1.15.18",
+    "@swc/core": "^1.15.33",
     "@total-typescript/ts-reset": "^0.6.1",
-    "@types/node": "^22.0.0",
-    "bun-types": "^1.3.0",
-    "chalk": "^5.4.0",
-    "dependency-cruiser": "^17.3.8",
+    "@types/node": "^22.19.18",
+    "bun-types": "^1.3.13",
+    "chalk": "^5.6.2",
+    "dependency-cruiser": "^17.4.0",
     "husky": "^9.1.7",
-    "knip": "^5.85.0",
-    "lint-staged": "^16.3.2",
+    "knip": "^5.88.1",
+    "lint-staged": "^16.4.0",
     "nx": "22.5.3",
     "oxc-parser": "^0.126.0",
     "oxfmt": "^0.36.0",
-    "oxlint": "^1.51.0",
-    "secretlint": "^11.3.1",
-    "typescript": "^5.8.0",
-    "zod": "^3.24.0"
+    "oxlint": "^1.63.0",
+    "secretlint": "^11.7.1",
+    "typescript": "^5.9.3",
+    "zod": "^3.25.76"
   },
   "overrides": {
+    "@babel/plugin-transform-modules-systemjs": "^7.29.4",
     "axios": "^1.15.1",
+    "fast-uri": "^3.1.2",
     "minimatch": "^10.2.3"
   },
   "engines": {