aitasks 1.3.0 → 1.3.1

package/README.md

@@ -72,6 +72,7 @@ export AITASKS_JSON=true
 | Command | Description |
 |---|---|
 | `aitasks init` | Initialize a task database in the current project |
+| `aitasks init --with-review` | Initialize with review enforcement (agents cannot mark done without a passing review) |
 | `aitasks onboard` | Print or inject agent protocol instructions into CLAUDE.md / AGENTS.md |
 
 ### Task Discovery
@@ -79,7 +80,7 @@ export AITASKS_JSON=true
 | Command | Description |
 |---|---|
 | `aitasks list` | List all tasks, sorted by priority |
-| `aitasks list --status ready` | Filter by status (`ready`, `in_progress`, `done`, `blocked`, `needs_review`) |
+| `aitasks list --status ready` | Filter by status (`ready`, `in_progress`, `blocked`, `review`, `done`) |
 | `aitasks next` | Show the highest-priority unblocked ready task |
 | `aitasks next --claim --agent <id>` | Auto-claim and start the best task |
 | `aitasks show <id>` | Full detail on a specific task (includes time tracking) |
@@ -96,9 +97,9 @@ export AITASKS_JSON=true
 | `aitasks start <id...> --agent <id>` | Begin active work on task(s) |
 | `aitasks note <id> <text>` | Add an implementation note |
 | `aitasks check <id> <n> --evidence <text>` | Verify acceptance criterion n |
-| `aitasks done <id...> --agent <id>` | Mark task(s) complete (all criteria must be verified) |
-| `aitasks review <id...> --agent <id>` | Submit for human review |
-| `aitasks reject <id> --reason <text>` | Reject and send back to in_progress |
+| `aitasks done <id...> --agent <id>` | Mark task(s) complete (all criteria must be verified; must be in `review` status if enforcement is on) |
+| `aitasks review <id...> --agent <id>` | Submit task(s) for review (moves to `review` status) |
+| `aitasks reject <id> --reason <text>` | Reject a task in review, send it back to `in_progress` with feedback |
 | `aitasks unclaim <id> --agent <id>` | Release a task back to the pool |
 | `aitasks undo <id>` | Undo the last action on a task |
 | `aitasks delete <id...>` | Delete task(s) - does not require claiming first |
@@ -151,12 +152,50 @@ When you run `aitasks init`, it automatically injects a full agent protocol into
 You can also inject/view it manually:
 
 ```sh
-aitasks onboard               # print to stdout
+aitasks onboard               # print to stdout (reflects current review_required setting)
 aitasks onboard --append      # append to detected agent file
 aitasks onboard --file MY.md  # append to a specific file
 aitasks onboard --json        # output as JSON string
 ```
 
+The injected instructions automatically adapt to the project's review enforcement setting — if `--with-review` is enabled, agents receive the full review workflow (with `aitasks review`, sub-agent approval, and `aitasks reject`) instead of the standard completion flow.
+
+---
+
+## Review Enforcement
+
+Enable a mandatory review gate so agents cannot mark tasks done without an explicit approval step:
+
+```sh
+# Enable at init time
+aitasks init --with-review
+
+# Enable on an existing project (also replaces the agent instructions file with review-aware version)
+aitasks init --with-review  # safe to re-run; updates DB setting and rewrites agent file
+```
+
+**How it works:**
+
+1. Agent completes work and checks all acceptance criteria
+2. Agent submits for review — task moves to `review` status:
+   ```sh
+   aitasks review TASK-001 --agent $AITASKS_AGENT_ID
+   ```
+3. A separate review sub-agent inspects the implementation and either:
+   - **Approves** — moves task to done:
+     ```sh
+     aitasks done TASK-001 --agent review-agent
+     ```
+   - **Rejects** — sends it back to `in_progress` with feedback:
+     ```sh
+     aitasks reject TASK-001 --reason "Missing error handling for 404 case" --agent review-agent
+     ```
+4. If rejected, the agent addresses feedback, re-verifies criteria, and repeats from step 2.
+
+**Enforcement:** When review is required, `aitasks done` and `aitasks update --status done` both block if the task is not already in `review` status. The gate cannot be bypassed.
+
+**Board:** Tasks in `review` status appear in the **IN PROGRESS** section with a `◈` magenta indicator so they're visually distinct from actively worked tasks.
+
 ---
 
 ## Data Storage

package/dist/index.js

@@ -1890,7 +1890,7 @@ var require_commander = __commonJS((exports) => {
 var require_package = __commonJS((exports, module) => {
   module.exports = {
     name: "aitasks",
-    version: "1.3.0",
+    version: "1.3.1",
     description: "CLI task management tool built for AI agents",
     type: "module",
     bin: {
@@ -29071,7 +29071,16 @@ import { join as join3 } from "path";
 import { existsSync as existsSync3 } from "fs";
 
 // src/db/schema.ts
-var SCHEMA_VERSION = 1;
+var SCHEMA_VERSION = 2;
+function runMigrations(db) {
+  const row = db.query(`SELECT value FROM meta WHERE key = 'schema_version'`).get();
+  const current = row ? parseInt(row.value, 10) : 1;
+  if (current < 2) {
+    db.exec(`UPDATE tasks SET status = 'review' WHERE status = 'needs_review'`);
+    db.exec(`INSERT OR IGNORE INTO meta (key, value) VALUES ('review_required', 'false')`);
+    db.exec(`UPDATE meta SET value = '2' WHERE key = 'schema_version'`);
+  }
+}
 function initializeSchema(db) {
   db.exec(`
     CREATE TABLE IF NOT EXISTS meta (
@@ -29126,6 +29135,7 @@ function initializeSchema(db) {
   insert.run("last_task_number", "0");
   insert.run("schema_version", String(SCHEMA_VERSION));
   insert.run("initialized_at", String(Date.now()));
+  insert.run("review_required", "false");
 }
 
 // src/db/backup.ts
@@ -29240,10 +29250,20 @@ function getDb() {
   const db = new Database(dbPath);
   applyPragmas(db);
   checkIntegrity(db, taskieDir);
+  runMigrations(db);
   registerCleanup();
   _db = db;
   return db;
 }
+function getReviewRequired() {
+  const db = getDb();
+  const row = db.query(`SELECT value FROM meta WHERE key = 'review_required'`).get();
+  return row?.value === "true";
+}
+function setReviewRequired(enabled) {
+  const db = getDb();
+  db.run(`INSERT OR REPLACE INTO meta (key, value) VALUES ('review_required', ?)`, [enabled ? "true" : "false"]);
+}
 function createFreshDb(taskieDir) {
   const dbPath = join3(taskieDir, "db.sqlite");
   const db = new Database(dbPath);
@@ -29265,7 +29285,8 @@ import { join as join4 } from "path";
 // src/utils/instructions.ts
 var INSTRUCTIONS_START_MARKER = "<!-- aitasks:instructions -->";
 var INSTRUCTIONS_END_MARKER = "<!-- aitasks:instructions:end -->";
-function getAgentInstructions(version) {
+function getAgentInstructions(version, opts = {}) {
+  const reviewRequired = opts.reviewRequired ?? false;
   return `${INSTRUCTIONS_START_MARKER}
 
 ## AITasks \u2014 Agent Task Protocol (v${version})
@@ -29372,7 +29393,7 @@ aitasks deps TASK-001    # Shows what this task is blocked by and what it blocks
 
 ---
 
-### Completing a Task
+### Completing a Task${reviewRequired ? " \u26A0\uFE0F  REVIEW REQUIRED" : ""}
 
 You MUST verify every acceptance criterion before marking done.
 
@@ -29388,15 +29409,30 @@ You MUST verify every acceptance criterion before marking done.
    aitasks check TASK-001 2 --evidence "integration test suite passes: 12/12 green"
    \`\`\`
 
-3. Mark done (will FAIL if any criterion is unchecked):
+${reviewRequired ? `3. Submit for review \u2014 you CANNOT mark done directly:
+   \`\`\`bash
+   aitasks review TASK-001 --agent $AITASKS_AGENT_ID
+   \`\`\`
+
+4. Spawn a review sub-agent to inspect the implementation and either:
+   - **Approve** (moves task to done):
+     \`\`\`bash
+     aitasks done TASK-001 --agent <review-agent-id>
+     \`\`\`
+   - **Reject** (sends task back to in_progress with feedback):
+     \`\`\`bash
+     aitasks reject TASK-001 --reason "<specific feedback>" --agent <review-agent-id>
+     \`\`\`
+
+5. If rejected: address the feedback, re-check criteria, and repeat from step 3.` : `3. Mark done (will FAIL if any criterion is unchecked):
    \`\`\`bash
    aitasks done TASK-001 --agent $AITASKS_AGENT_ID
    \`\`\`
 
-   If human review is needed instead:
+   If review is needed:
    \`\`\`bash
    aitasks review TASK-001 --agent $AITASKS_AGENT_ID
-   \`\`\`
+   \`\`\``}
 
 ---
 
@@ -29434,7 +29470,8 @@ aitasks unclaim TASK-001 --agent $AITASKS_AGENT_ID --reason "Blocked on missing
 4. Add implementation notes continuously, not just at the end.
 5. If a task needs splitting, create subtasks BEFORE marking parent done.
 6. Your evidence strings must be concrete and verifiable \u2014 not vague affirmations.
-7. Always provide --desc and at least one --ac when creating a task. Both are required.
+7. Always provide --desc and at least one --ac when creating a task. Both are required.${reviewRequired ? `
+8. NEVER move a task to done directly. Always submit for review first with \`aitasks review\`, then spawn a review sub-agent to approve or reject. Tasks cannot be marked done without a passing review.` : ""}
 
 ---
 
@@ -29481,13 +29518,17 @@ function findExistingAgentFile(projectRoot) {
   }
   return null;
 }
-function injectOrCreateAgentFile(projectRoot, version) {
-  const instructions = getAgentInstructions(version);
+function injectOrCreateAgentFile(projectRoot, version, reviewRequired = false, force = false) {
+  const instructions = getAgentInstructions(version, { reviewRequired });
   const existing = findExistingAgentFile(projectRoot);
   if (existing) {
     const content = readFileSync2(existing, "utf8");
     if (instructionsAlreadyPresent(content)) {
-      return { filePath: existing, action: "skipped" };
+      if (!force)
+        return { filePath: existing, action: "skipped" };
+      const replaced = replaceInstructionsBlock(content, instructions);
+      writeFileSync2(existing, replaced, "utf8");
+      return { filePath: existing, action: "appended" };
     }
     const separator = content.endsWith(`
 `) ? `
@@ -29503,8 +29544,25 @@ function injectOrCreateAgentFile(projectRoot, version) {
 `, "utf8");
   return { filePath: newPath, action: "created" };
 }
-function appendToSpecificFile(filePath, version) {
-  const instructions = getAgentInstructions(version);
+function replaceInstructionsBlock(content, newInstructions) {
+  const start = content.indexOf(INSTRUCTIONS_START_MARKER);
+  const end = content.indexOf(INSTRUCTIONS_END_MARKER);
+  if (start === -1 || end === -1)
+    return content;
+  const before = content.slice(0, start).replace(/\n+$/, "");
+  const after = content.slice(end + INSTRUCTIONS_END_MARKER.length).replace(/^\n+/, "");
+  const sep = before.length > 0 ? `
+
+` : "";
+  const tail = after.length > 0 ? `
+
+` + after : `
+`;
+  return before + sep + newInstructions + `
+` + tail;
+}
+function appendToSpecificFile(filePath, version, reviewRequired = false) {
+  const instructions = getAgentInstructions(version, { reviewRequired });
   if (!existsSync4(filePath)) {
     writeFileSync2(filePath, instructions + `
 `, "utf8");
@@ -29525,27 +29583,39 @@ function appendToSpecificFile(filePath, version) {
 }
 
 // src/commands/init.ts
-var initCommand = new Command("init").description("Initialize AITasks in the current project").option("--skip-agent-file", "Skip injecting agent instructions into CLAUDE.md/AGENTS.md/GEMINI.md").action(async (opts) => {
+var initCommand = new Command("init").description("Initialize AITasks in the current project").option("--skip-agent-file", "Skip injecting agent instructions into CLAUDE.md/AGENTS.md/GEMINI.md").option("--with-review", "Enforce review gate: agents cannot mark tasks done without a passing review").action(async (opts) => {
   const root = findProjectRoot();
   const taskieDir = join5(root, ".aitasks");
   if (existsSync5(join5(taskieDir, "db.sqlite"))) {
     console.log(source_default.yellow("  AITasks is already initialized in this project."));
     console.log(source_default.dim(`  DB: ${join5(taskieDir, "db.sqlite")}`));
+    if (opts.withReview) {
+      setReviewRequired(true);
+      console.log(source_default.green("  \u2713") + "  Review enforcement enabled.");
+    }
     if (!opts.skipAgentFile) {
-      const result = injectOrCreateAgentFile(root, getVersion());
+      const result = injectOrCreateAgentFile(root, getVersion(), !!opts.withReview, !!opts.withReview);
       printAgentFileResult(result);
     }
     return;
   }
   mkdirSync(taskieDir, { recursive: true });
   createFreshDb(taskieDir);
+  if (opts.withReview) {
+    setReviewRequired(true);
+  }
   console.log("");
   console.log(source_default.green("  \u2713") + source_default.bold("  AITasks initialized"));
   console.log(source_default.dim(`     Project root : ${root}`));
   console.log(source_default.dim(`     Database      : ${join5(taskieDir, "db.sqlite")}`));
+  if (opts.withReview) {
+    console.log(source_default.magenta("  \u25C8") + source_default.bold("  Review enforcement enabled"));
+    console.log(source_default.dim("     Agents must submit tasks for review before marking done."));
+    console.log(source_default.dim("     Use: aitasks review <id>  \u2192  review sub-agent  \u2192  aitasks done <id>"));
+  }
   console.log("");
   if (!opts.skipAgentFile) {
-    const result = injectOrCreateAgentFile(root, getVersion());
+    const result = injectOrCreateAgentFile(root, getVersion(), !!opts.withReview);
     printAgentFileResult(result);
   }
   console.log(source_default.dim("  Run `aitasks create` to add your first task."));
@@ -30156,7 +30226,7 @@ var PRIORITY_ORDER = `CASE priority
   WHEN 'low'      THEN 3 END`;
 var STATUS_ORDER = `CASE status
   WHEN 'in_progress'  THEN 0
-  WHEN 'needs_review' THEN 1
+  WHEN 'review'       THEN 1
   WHEN 'blocked'      THEN 2
   WHEN 'ready'        THEN 3
   WHEN 'backlog'      THEN 4
@@ -30357,6 +30427,17 @@ function completeTask(taskId, agentId) {
   if (unchecked.length > 0) {
     return { task: null, error: "Not all acceptance criteria are verified", unchecked };
   }
+  if (getReviewRequired() && task.status !== "review") {
+    return {
+      task: null,
+      error: `Review required: submit this task for review first.
+` + `  1. aitasks review <taskId> --agent $AITASKS_AGENT_ID
+` + `  2. Spawn a review sub-agent to inspect the work
+` + `  3. Review agent approves: aitasks done <taskId> --agent <review-agent-id>
+` + `     Review agent rejects: aitasks reject <taskId> --reason "<feedback>"
+` + "  Tasks cannot be moved to Done without a passing review."
+    };
+  }
   const updated = updateTask(taskId, { status: "done", completed_at: Date.now() });
   logEvent({ task_id: taskId, agent_id: agentId, event_type: "completed", payload: {} });
   const pendingRows = db.query(`SELECT id, blocked_by FROM tasks WHERE status != 'done'`).all();
@@ -30442,7 +30523,7 @@ function reviewTask(taskId, agentId) {
   if (task.status !== "in_progress") {
     return { task: null, error: "Task must be in_progress to request review" };
   }
-  const updated = updateTask(taskId, { status: "needs_review" });
+  const updated = updateTask(taskId, { status: "review" });
   logEvent({ task_id: taskId, agent_id: agentId, event_type: "review_requested", payload: {} });
   return { task: updated };
 }
@@ -30450,8 +30531,8 @@ function rejectTask(taskId, reason, agentId) {
   const task = getTask(taskId);
   if (!task)
     return { task: null, error: "Task not found" };
-  if (task.status !== "needs_review") {
-    return { task: null, error: "Task must be in needs_review status to reject" };
+  if (task.status !== "review") {
+    return { task: null, error: "Task must be in review status to reject" };
   }
   updateTask(taskId, { status: "in_progress" });
   addImplementationNote(taskId, `REVIEW REJECTED: ${reason}`, agentId ?? "human");
@@ -38533,7 +38614,7 @@ var STATUS_ICON = {
   ready: "\u25CE",
   in_progress: "\u25B6",
   blocked: "\u2298",
-  needs_review: "\u25C8",
+  review: "\u25C8",
   done: "\u2713"
 };
 
@@ -40114,7 +40195,7 @@ var STATUS_COLORS = {
   ready: "blue",
   in_progress: "yellow",
   blocked: "#FF5C5C",
-  needs_review: "magenta",
+  review: "magenta",
   done: "green"
 };
 var PRIORITY_COLORS = {
@@ -40526,7 +40607,7 @@ var STATUS_COLORS2 = {
   ready: "blue",
   in_progress: "yellow",
   blocked: "#FF5C5C",
-  needs_review: "magenta",
+  review: "magenta",
   done: "green"
 };
 var PRIORITY_COLORS2 = {
@@ -41214,8 +41295,9 @@ var reviewCommand = new Command("review").description("Request human review for
     if (!json) {
       console.log("");
       console.log(source_default.magenta("  \u25C8") + `  ${source_default.bold(task.id)} submitted for review`);
-      console.log(source_default.dim(`     A human can approve with: aitasks done ${task.id}`));
-      console.log(source_default.dim(`     Or reject with: aitasks reject ${task.id} --reason "<reason>"`));
+      console.log(source_default.dim(`     Spawn a review sub-agent to inspect the implementation.`));
+      console.log(source_default.dim(`     Approve: aitasks done ${task.id} --agent <review-agent-id>`));
+      console.log(source_default.dim(`     Reject:  aitasks reject ${task.id} --reason "<feedback>"`));
     }
   }
   if (json) {
@@ -41364,6 +41446,9 @@ var updateCommand = new Command("update").description("Update task fields").argu
   if (Object.keys(changes).length === 0) {
     exitError("No changes specified. Use --help to see available options.", json);
   }
+  if (opts.status === "done" && getReviewRequired() && task.status !== "review") {
+    exitError("Review required: use `aitasks done` which enforces the review gate.\n" + "  Submit for review first: aitasks review <taskId> --agent $AITASKS_AGENT_ID", json);
+  }
   const updated = updateTask(id, changes);
   if (json)
     return jsonOut(true, updated);
@@ -41387,7 +41472,7 @@ var STATUS_COLORS3 = {
   ready: "blue",
   in_progress: "yellow",
   blocked: "#FF5C5C",
-  needs_review: "magenta",
+  review: "magenta",
   done: "green"
 };
 var PRIORITY_COLORS3 = {
@@ -41398,11 +41483,11 @@ var PRIORITY_COLORS3 = {
 };
 function buildTree(tasks) {
   const sorted = [...tasks].sort((a2, b2) => a2.id.localeCompare(b2.id));
-  const inProgressIds = new Set(sorted.filter((t) => t.status === "in_progress").map((t) => t.id));
+  const inProgressIds = new Set(sorted.filter((t) => t.status === "in_progress" || t.status === "review").map((t) => t.id));
   const doneIds = new Set(sorted.filter((t) => t.status === "done").map((t) => t.id));
   const shownIds = new Set;
   const items = [];
-  const ipCount = sorted.filter((t) => t.status === "in_progress").length;
+  const ipCount = sorted.filter((t) => t.status === "in_progress" || t.status === "review").length;
   const doneCount = sorted.filter((t) => t.status === "done").length;
   function push(task, indent, isLastSibling, section, showHeader) {
     items.push({
@@ -41416,13 +41501,13 @@ function buildTree(tasks) {
     shownIds.add(task.id);
   }
   let firstIp = true;
-  for (const task of sorted.filter((t) => t.status === "in_progress" && !t.parent_id)) {
+  for (const task of sorted.filter((t) => (t.status === "in_progress" || t.status === "review") && !t.parent_id)) {
     push(task, 0, false, "in_progress", firstIp);
     firstIp = false;
     const subs = sorted.filter((t) => t.parent_id === task.id);
     subs.forEach((sub, i) => push(sub, 1, i === subs.length - 1, "in_progress", false));
   }
-  for (const task of sorted.filter((t) => t.status === "in_progress" && !!t.parent_id)) {
+  for (const task of sorted.filter((t) => (t.status === "in_progress" || t.status === "review") && !!t.parent_id)) {
     if (!shownIds.has(task.id)) {
       push(task, 0, false, "in_progress", firstIp);
       firstIp = false;
@@ -41666,7 +41751,7 @@ var RightPane = ({ task, width, height, scrollOffset, metricsRef }) => {
     }, i, true, undefined, this))
   }, undefined, false, undefined, this);
 };
-var PICKER_STATUSES = ["backlog", "ready", "in_progress", "blocked", "needs_review", "done"];
+var PICKER_STATUSES = ["backlog", "ready", "in_progress", "blocked", "review", "done"];
 var StatusPicker = ({ task }) => /* @__PURE__ */ jsx_dev_runtime3.jsxDEV(Box_default, {
   flexDirection: "column",
   paddingLeft: 2,
@@ -41877,7 +41962,7 @@ var TreeBoardComponent = ({ getTasks }) => {
         "2": "ready",
         "3": "in_progress",
         "4": "blocked",
-        "5": "needs_review",
+        "5": "review",
         "6": "done"
       };
       const newStatus = statusMap[input];
@@ -42274,10 +42359,10 @@ var STATUS_LABELS = {
   ready: "READY",
   in_progress: "IN PROGRESS",
   blocked: "BLOCKED",
-  needs_review: "NEEDS REVIEW",
+  review: "REVIEW",
   done: "DONE"
 };
-var ALL_STATUSES = ["in_progress", "needs_review", "ready", "blocked", "backlog", "done"];
+var ALL_STATUSES = ["in_progress", "review", "ready", "blocked", "backlog", "done"];
 var StaticCard = ({ task }) => {
   const priColor = PRIORITY_COLORS3[task.priority] ?? "gray";
   return /* @__PURE__ */ jsx_dev_runtime3.jsxDEV(Box_default, {
@@ -42544,13 +42629,14 @@ var exportCommand = new Command("export").description("Export all task data").op
 // src/commands/onboard.ts
 var VERSION = "1.0.0";
 var onboardCommand = new Command("onboard").description("Print or inject agent protocol instructions").option("--append", "Auto-detect and append to CLAUDE.md/AGENTS.md/GEMINI.md").option("--file <path>", "Append to a specific file").option("--json", "Output the instructions as a JSON string").action((opts) => {
-  const instructions = getAgentInstructions(VERSION);
+  const reviewRequired = isInitialized() ? getReviewRequired() : false;
+  const instructions = getAgentInstructions(VERSION, { reviewRequired });
   if (opts.json) {
     console.log(JSON.stringify({ instructions }, null, 2));
     return;
   }
   if (opts.file) {
-    const result2 = appendToSpecificFile(opts.file, VERSION);
+    const result2 = appendToSpecificFile(opts.file, VERSION, reviewRequired);
     const rel = opts.file.replace(process.cwd() + "/", "");
     switch (result2.action) {
       case "created":
@@ -42567,7 +42653,7 @@ var onboardCommand = new Command("onboard").description("Print or inject agent p
   }
   if (opts.append) {
     const root = findProjectRoot();
-    const result2 = injectOrCreateAgentFile(root, VERSION);
+    const result2 = injectOrCreateAgentFile(root, VERSION, reviewRequired);
     const rel = result2.filePath.replace(process.cwd() + "/", "");
     switch (result2.action) {
       case "created":
@@ -42738,7 +42824,7 @@ function getStatusIcon(status) {
     ready: "\u25D0",
     in_progress: "\u25B6",
     blocked: "\u2298",
-    needs_review: "\u25C8",
+    review: "\u25C8",
     done: "\u2713"
   };
   return icons[status] || "\u25CB";
@@ -42778,7 +42864,7 @@ var searchCommand = new Command("search").description("Full-text search across t
     const statusIcon = getStatusIcon2(task.status);
     const priColor = getPriorityColor(task.priority);
     console.log("");
-    console.log(`${source_default.bold.white(task.id)}  ` + `${priColor(task.priority[0].toUpperCase())}  ` + `${statusIcon} ` + highlightMatches(task.title, searchTerms));
+    console.log(`${source_default.bold.white(task.id)}  ` + `${priColor((task.priority[0] ?? "?").toUpperCase())}  ` + `${statusIcon} ` + highlightMatches(task.title, searchTerms));
     const context = getSearchContext(task, searchTerms);
     if (context) {
       console.log(source_default.dim(`  ${context}`));
@@ -42837,7 +42923,7 @@ function getStatusIcon2(status) {
     ready: source_default.blue("\u25D0"),
     in_progress: source_default.yellow("\u25B6"),
     blocked: source_default.hex("#FF5C5C")("\u2298"),
-    needs_review: source_default.magenta("\u25C8"),
+    review: source_default.magenta("\u25C8"),
     done: source_default.green("\u2713")
   };
   return icons[status] || "\u25CB";
@@ -42927,25 +43013,26 @@ var undoCommand = new Command("undo").description("Undo the last action on a tas
   }
 });
 function undoComplete(task) {
+  const newStatus = getReviewRequired() ? "review" : "in_progress";
   updateTask(task.id, {
-    status: "in_progress",
+    status: newStatus,
     completed_at: null
   });
   return {
     success: true,
-    message: `${task.id} reverted from done to in_progress`,
-    data: { task: task.id, previousStatus: "done", newStatus: "in_progress" }
+    message: `${task.id} reverted from done to ${newStatus}`,
+    data: { task: task.id, previousStatus: "done", newStatus }
   };
 }
 function undoReviewRequested(task) {
-  if (task.status !== "needs_review") {
-    return { success: false, message: `${task.id} is not in needs_review status` };
+  if (task.status !== "review") {
+    return { success: false, message: `${task.id} is not in review status` };
   }
   updateTask(task.id, { status: "in_progress" });
   return {
     success: true,
-    message: `${task.id} reverted from needs_review to in_progress`,
-    data: { task: task.id, previousStatus: "needs_review", newStatus: "in_progress" }
+    message: `${task.id} reverted from review to in_progress`,
+    data: { task: task.id, previousStatus: "review", newStatus: "in_progress" }
   };
 }
 function undoStarted(task) {
@@ -43135,4 +43222,4 @@ program2.parseAsync(process.argv).catch((err) => {
   process.exit(1);
 });
 
-//# debugId=26C003F11D02306964756E2164756E21
+//# debugId=B0EAB2DE4FE47CF564756E2164756E21

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "aitasks",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "description": "CLI task management tool built for AI agents",
   "type": "module",
   "bin": {