coding-agent-harness 1.0.4 → 1.0.5

package/scripts/lib/task-lifecycle/review-gates.mjs

@@ -6,12 +6,17 @@ import {
 import {
   collectReviewRisks,
   isBlockingReviewRisk,
+  parseTaskAuditMetadata,
   parsePhases,
   parseReviewConfirmation,
   readVisualMapContractFile,
 } from "../task-scanner.mjs";
+import {
+  implementationPhases,
+  phaseHasRecordedProgress,
+} from "../phase-kind.mjs";
 
-export function validateLifecycleTransition({ event, currentState, budget, reviewContent = "", reviewTaskKey = "", projectRoot = "", taskDir = "" }) {
+export function validateLifecycleTransition({ event, currentState, budget, reviewContent = "", indexContent = "", reviewTaskKey = "", projectRoot = "", taskDir = "" }) {
   if (event === "task-review" && currentState !== "in_progress") {
     throw new Error(`task-review requires current state in_progress; current state is ${currentState || "unknown"}`);
   }
@@ -24,7 +29,7 @@ export function validateLifecycleTransition({ event, currentState, budget, revie
       const ids = blockingRisks.map((risk) => risk.id || risk.severity).join(", ");
       throw new Error(`Open blocking review findings must be closed before task-complete: ${ids}`);
     }
-    if (!parseReviewConfirmation(reviewContent, { taskKey: reviewTaskKey, projectRoot, taskDir })?.confirmed) {
+    if (!parseReviewConfirmation(reviewContent, { taskKey: reviewTaskKey, taskAudit: parseTaskAuditMetadata(indexContent), projectRoot, taskDir, indexPath: path.join(taskDir, "INDEX.md") })?.confirmed) {
       throw new Error("Human review must be confirmed before task-complete. Run review-confirm first.");
     }
   }
@@ -37,15 +42,13 @@ export function validateReviewEntryGate(taskDir, budget) {
     throw new Error(`task-review requires ${lessonCandidatesFile} before entering human review.`);
   }
   const phases = parsePhases(readVisualMapContractFile(taskDir).content);
-  const actionablePhases = phases.filter((phase) => phase.state !== "skipped");
-  const hasRecordedPhaseProgress = actionablePhases.some(
-    (phase) =>
-      phase.completion > 0 ||
-      ["in_progress", "review", "blocked", "done"].includes(phase.state) ||
-      ["partial", "present", "waived"].includes(phase.evidenceStatus),
-  );
+  const actionablePhases = implementationPhases(phases);
+  if (phases.length > 0 && actionablePhases.length === 0) {
+    throw new Error("task-review requires at least one non-skipped Visual Map execution phase.");
+  }
+  const hasRecordedPhaseProgress = actionablePhases.some(phaseHasRecordedProgress);
   if (actionablePhases.length > 0 && !hasRecordedPhaseProgress) {
-    throw new Error("task-review requires at least one Visual Map phase progress update. Run task-phase before entering human review.");
+    throw new Error("task-review requires at least one Visual Map execution phase progress update. Run task-phase before entering human review.");
   }
 }
 

package/scripts/lib/task-lifecycle/review-submission.mjs

@@ -0,0 +1,63 @@
+import crypto from "node:crypto";
+import fs from "node:fs";
+import path from "node:path";
+import {
+  lessonCandidatesFile,
+  longRunningTaskContractFile,
+  nowTimestamp,
+  readFileSafe,
+  toPosix,
+  visualMapFile,
+} from "../core-shared.mjs";
+import {
+  collectReviewRisks,
+  isBlockingReviewRisk,
+  taskScannerVersion,
+} from "../task-review-model.mjs";
+import { markdownCell } from "./text-utils.mjs";
+
+export function renderAgentReviewSubmission({ target, taskDir, canonicalTaskId, message, evidence }) {
+  const timestamp = nowTimestamp();
+  const submissionId = `ARS-${timestamp.replace(/[^0-9]/g, "").slice(0, 14)}`;
+  const materialsHash = hashTaskMaterials(taskDir);
+  const reviewContent = readFileSafe(path.join(taskDir, "review.md"));
+  const openFindings = collectReviewRisks(reviewContent).filter(isBlockingReviewRisk).length;
+  const evidenceSummary = evidence || message || "Agent submitted task for human review.";
+  return [
+    "## Agent Review Submission",
+    "",
+    "| Field | Value |",
+    "| --- | --- |",
+    `| Submission ID | ${submissionId} |`,
+    `| Submitted At | ${timestamp} |`,
+    "| Submitted By | agent |",
+    `| Task Key | ${canonicalTaskId} |`,
+    `| Materials Checklist Hash | ${materialsHash} |`,
+    `| Evidence Summary | ${markdownCell(evidenceSummary)} |`,
+    `| Open Findings Count | ${openFindings} |`,
+    `| Scanner Version | ${taskScannerVersion} |`,
+    `| Target | TARGET:${toPosix(path.relative(target.projectRoot, taskDir))} |`,
+    "",
+  ].join("\n");
+}
+
+export function replaceAgentReviewSubmission(content, block) {
+  const trimmed = String(content || "").trimEnd();
+  if (/^##\s*(?:Agent Review Submission|Agent 审查提交|Agent 提交审查)\s*$/im.test(trimmed)) {
+    return `${trimmed.replace(/^##\s*(?:Agent Review Submission|Agent 审查提交|Agent 提交审查)\s*$[\s\S]*?(?=^##\s+|(?![\s\S]))/im, `${block.trimEnd()}\n\n`)}\n`;
+  }
+  return `${trimmed}\n\n${block.trimEnd()}\n`;
+}
+
+function hashTaskMaterials(taskDir) {
+  const hash = crypto.createHash("sha256");
+  for (const fileName of ["brief.md", "task_plan.md", visualMapFile, lessonCandidatesFile, "progress.md", "review.md", "findings.md", longRunningTaskContractFile]) {
+    const filePath = path.join(taskDir, fileName);
+    if (!fs.existsSync(filePath)) continue;
+    hash.update(fileName);
+    hash.update("\0");
+    hash.update(readFileSafe(filePath));
+    hash.update("\0");
+  }
+  return hash.digest("hex").slice(0, 16);
+}

package/scripts/lib/task-lifecycle/scaffold-provenance.mjs

@@ -0,0 +1,49 @@
+import path from "node:path";
+import { localizedTemplateSource, todayDate } from "../core-shared.mjs";
+import { markdownCell } from "./text-utils.mjs";
+
+function shellArg(value) {
+  const text = String(value || "");
+  if (/^[A-Za-z0-9._/:=@+-]+$/.test(text)) return text;
+  return `'${text.replaceAll("'", "'\\''")}'`;
+}
+
+function commandPathArg(value, fallback) {
+  const text = String(value || fallback || ".").trim() || ".";
+  if (text === ".") return ".";
+  return path.isAbsolute(text) ? fallback : text;
+}
+
+function renderNewTaskCommand({ taskId, title, locale, budget, longRunning, moduleKey, preset, fromSession, targetInput }) {
+  const parts = ["harness", "new-task"];
+  if (taskId) parts.push(taskId);
+  parts.push("--budget", budget, "--locale", locale);
+  if (title) parts.push("--title", title);
+  if (moduleKey) parts.push("--module", moduleKey);
+  if (preset && preset !== "none") parts.push("--preset", preset);
+  if (fromSession) parts.push("--from-session", commandPathArg(fromSession, "<session.json>"));
+  if (longRunning) parts.push("--long-running");
+  parts.push(commandPathArg(targetInput, "<target>"));
+  return parts.map(shellArg).join(" ");
+}
+
+export function buildScaffoldProvenance({ taskId, normalizedTaskId, title, locale, budget, longRunning, moduleKey, preset, fromSession, targetInput, automaticTaskId = false }) {
+  return {
+    createdBy: "harness new-task",
+    command: markdownCell(renderNewTaskCommand({
+      taskId: automaticTaskId ? "" : taskId || normalizedTaskId,
+      title,
+      locale,
+      budget,
+      longRunning,
+      moduleKey,
+      preset,
+      fromSession,
+      targetInput,
+    })),
+    createdAt: todayDate(),
+    budget,
+    templateSource: localizedTemplateSource("templates/planning/brief.md", locale),
+    exceptionReason: "n/a",
+  };
+}

package/scripts/lib/task-lifecycle/template-files.mjs

@@ -0,0 +1,53 @@
+import { localizedTemplateSource, longRunningTaskContractFile, visualMapFile, lessonCandidatesFile } from "../core-shared.mjs";
+
+export function taskTemplateFiles({ locale = "en-US" } = {}) {
+  return [
+    ["INDEX.md", "templates/planning/INDEX.md"],
+    ["brief.md", "templates/planning/brief.md"],
+    ["task_plan.md", "templates/planning/task_plan.md"],
+    ["execution_strategy.md", "templates/planning/execution_strategy.md"],
+    [visualMapFile, "templates/planning/visual_map.md"],
+    ["findings.md", "templates/planning/findings.md"],
+    [lessonCandidatesFile, "templates/planning/lesson_candidates.md"],
+    ["progress.md", "templates/planning/progress.md"],
+    ["review.md", "templates/planning/review.md"],
+  ].map(([destination, source]) => [destination, localizedTemplateSource(source, locale)]);
+}
+
+export function simpleTaskTemplateFiles({ locale = "en-US" } = {}) {
+  return [
+    ["INDEX.md", "templates/planning/INDEX.md"],
+    ["brief.md", "templates/planning/brief.md"],
+    ["task_plan.md", "templates/planning/task_plan.md"],
+    [visualMapFile, "templates/planning/visual_map.simple.md"],
+    ["progress.md", "templates/planning/progress.md"],
+  ].map(([destination, source]) => [destination, localizedTemplateSource(source, locale)]);
+}
+
+export function optionalTaskTemplateFiles({ locale = "en-US" } = {}) {
+  return [
+    ["references/INDEX.md", "templates/planning/optional/references/INDEX.md"],
+    ["artifacts/INDEX.md", "templates/planning/optional/artifacts/INDEX.md"],
+  ].map(([destination, source]) => [destination, localizedTemplateSource(source, locale)]);
+}
+
+export function moduleTemplateFiles({ locale = "en-US" } = {}) {
+  return [
+    ["brief.md", "templates/planning/module_brief.md"],
+    ["module_plan.md", "templates/planning/module_plan.md"],
+    ["execution_strategy.md", "templates/planning/execution_strategy.md"],
+    [visualMapFile, "templates/planning/visual_map.md"],
+    ["session_prompt.md", "templates/planning/module_session_prompt.md"],
+  ].map(([destination, source]) => [destination, localizedTemplateSource(source, locale)]);
+}
+
+export function taskFilesForBudget({ budget, locale }) {
+  if (budget === "simple") return simpleTaskTemplateFiles({ locale });
+  if (budget === "complex") return [...taskTemplateFiles({ locale }), ...optionalTaskTemplateFiles({ locale })];
+  return taskTemplateFiles({ locale });
+}
+
+export function appendLongRunningContractFile(files, { locale, longRunning }) {
+  if (!longRunning) return files;
+  return [...files, [longRunningTaskContractFile, localizedTemplateSource("templates/planning/long-running-task-contract.md", locale)]];
+}

package/scripts/lib/task-lifecycle.mjs

@@ -5,7 +5,6 @@ import {
   visualMapFile,
   legacyVisualRoadmapFile,
   lessonCandidatesFile,
-  longRunningTaskContractFile,
   allowedTaskStates,
   allowedTaskBudgets,
   allowedPhaseStates,
@@ -15,11 +14,9 @@ import {
   toPosix,
   readFileSafe,
   readBundledTemplate,
-  localizedTemplateSource,
   todayDate,
   localDate,
   datePrefix,
-  nowTimestamp,
   normalizeTaskId,
   renderTaskTemplate,
 } from "./core-shared.mjs";
@@ -35,24 +32,30 @@ import {
 } from "./preset-engine.mjs";
 import {
   collectTasks,
-  collectReviewRisks,
-  isBlockingReviewRisk,
   listTaskPlanPaths,
   parseTaskBudget,
   taskIdForDirectory,
-  taskScannerVersion,
 } from "./task-scanner.mjs";
+import { getColumn, firstColumn, updateMarkdownTableRow } from "./markdown-utils.mjs";
+import { validateLifecycleTransition, validateReviewEntryGate } from "./task-lifecycle/review-gates.mjs";
+import { advanceLifecyclePhase, autoRecordNoLessonCandidateDecision } from "./task-lifecycle/phase-sync.mjs";
+import { confirmTaskReview as confirmTaskReviewWithContext } from "./task-lifecycle/review-confirm.mjs";
+import { appendProgressLog } from "./task-lifecycle/text-utils.mjs";
+import { buildScaffoldProvenance } from "./task-lifecycle/scaffold-provenance.mjs";
+import { buildCreationTaskAudit } from "./task-audit-metadata.mjs";
 import {
-  getColumn,
-  firstColumn,
-  updateMarkdownTableRow,
-} from "./markdown-utils.mjs";
+  renderAgentReviewSubmission,
+  replaceAgentReviewSubmission,
+} from "./task-lifecycle/review-submission.mjs";
 import {
-  validateLifecycleTransition,
-  validateReviewEntryGate,
-} from "./task-lifecycle/review-gates.mjs";
-import { confirmTaskReview as confirmTaskReviewWithContext } from "./task-lifecycle/review-confirm.mjs";
-import { appendProgressLog, markdownCell } from "./task-lifecycle/text-utils.mjs";
+  appendLongRunningContractFile,
+  moduleTemplateFiles,
+  taskFilesForBudget,
+} from "./task-lifecycle/template-files.mjs";
+import {
+  planCreateTaskChanges,
+  refreshPresetCommandAudit,
+} from "./task-lifecycle/create-task-helpers.mjs";
 import {
   beginGovernanceSync,
   commitGovernanceSync,
@@ -62,45 +65,6 @@ import {
   syncTaskGovernance,
 } from "./governance-sync.mjs";
 
-function taskTemplateFiles({ locale = "en-US" } = {}) {
-  return [
-    ["brief.md", "templates/planning/brief.md"],
-    ["task_plan.md", "templates/planning/task_plan.md"],
-    ["execution_strategy.md", "templates/planning/execution_strategy.md"],
-    [visualMapFile, "templates/planning/visual_map.md"],
-    ["findings.md", "templates/planning/findings.md"],
-    [lessonCandidatesFile, "templates/planning/lesson_candidates.md"],
-    ["progress.md", "templates/planning/progress.md"],
-    ["review.md", "templates/planning/review.md"],
-  ].map(([destination, source]) => [destination, localizedTemplateSource(source, locale)]);
-}
-
-function simpleTaskTemplateFiles({ locale = "en-US" } = {}) {
-  return [
-    ["brief.md", "templates/planning/brief.md"],
-    ["task_plan.md", "templates/planning/task_plan.md"],
-    [visualMapFile, "templates/planning/visual_map.md"],
-    ["progress.md", "templates/planning/progress.md"],
-  ].map(([destination, source]) => [destination, localizedTemplateSource(source, locale)]);
-}
-
-function optionalTaskTemplateFiles({ locale = "en-US" } = {}) {
-  return [
-    ["references/INDEX.md", "templates/planning/optional/references/INDEX.md"],
-    ["artifacts/INDEX.md", "templates/planning/optional/artifacts/INDEX.md"],
-  ].map(([destination, source]) => [destination, localizedTemplateSource(source, locale)]);
-}
-
-function moduleTemplateFiles({ locale = "en-US" } = {}) {
-  return [
-    ["brief.md", "templates/planning/module_brief.md"],
-    ["module_plan.md", "templates/planning/module_plan.md"],
-    ["execution_strategy.md", "templates/planning/execution_strategy.md"],
-    [visualMapFile, "templates/planning/visual_map.md"],
-    ["session_prompt.md", "templates/planning/module_session_prompt.md"],
-  ].map(([destination, source]) => [destination, localizedTemplateSource(source, locale)]);
-}
-
 function taskRoot(target, taskId, { moduleKey = "" } = {}) {
   const normalizedTaskId = normalizeTaskId(taskId);
   if (moduleKey) return path.join(target.docsRoot, "09-PLANNING/MODULES", normalizeTaskId(moduleKey), normalizedTaskId);
@@ -175,17 +139,6 @@ function normalizeTaskPresetInput(preset, { targetInput = "" } = {}) {
   return readPresetPackage(normalized, { targetInput }).id;
 }
 
-function taskFilesForBudget({ budget, locale }) {
-  if (budget === "simple") return simpleTaskTemplateFiles({ locale });
-  if (budget === "complex") return [...taskTemplateFiles({ locale }), ...optionalTaskTemplateFiles({ locale })];
-  return taskTemplateFiles({ locale });
-}
-
-function appendLongRunningContractFile(files, { locale, longRunning }) {
-  if (!longRunning) return files;
-  return [...files, [longRunningTaskContractFile, localizedTemplateSource("templates/planning/long-running-task-contract.md", locale)]];
-}
-
 function updateProgressState(content, state, locale) {
   const label = stateLabel(state, locale);
   if (/^##\s*状态[:：][^\n]*/im.test(content)) {
@@ -207,7 +160,31 @@ function bareSlug(datedId) {
   return datedId;
 }
 
-export function createTask(targetInput, taskId, { title = "", locale = "en-US", dryRun = false, moduleKey = "", budget = "standard", longRunning = false, preset = "", fromSession = "", presetArgs = [] } = {}) {
+function automaticTaskSlug(seed) {
+  return normalizeTaskId(seed || "task").slice(0, 48).replace(/-+$/g, "") || "task";
+}
+
+function randomTaskSuffix() {
+  return crypto.randomBytes(4).toString("hex");
+}
+
+function resolveTaskIdentity({ target, taskId, title, presetPackage, moduleKey, automaticTaskId }) {
+  if (!automaticTaskId) {
+    const rawNormalized = normalizeTaskId(taskId || (presetPackage?.task?.defaultTaskId || ""));
+    const normalizedTaskId = ensureDatePrefix(rawNormalized);
+    if (!normalizedTaskId) throw new Error("Missing task id");
+    return { normalizedTaskId, semanticSlug: bareSlug(normalizedTaskId) };
+  }
+
+  const semanticSlug = automaticTaskSlug(title || presetPackage?.task?.defaultTaskId || "task");
+  for (let attempt = 0; attempt < 8; attempt += 1) {
+    const normalizedTaskId = `${localDate()}-${semanticSlug}-${randomTaskSuffix()}`;
+    if (!fs.existsSync(taskRoot(target, normalizedTaskId, { moduleKey }))) return { normalizedTaskId, semanticSlug };
+  }
+  throw new Error(`Unable to allocate automatic task id for: ${semanticSlug}`);
+}
+
+export function createTask(targetInput, taskId, { title = "", locale = "en-US", dryRun = false, moduleKey = "", budget = "standard", longRunning = false, preset = "", fromSession = "", presetArgs = [], automaticTaskId = false } = {}) {
   const requestedPreset = preset || (moduleKey ? "module" : "");
   const normalizedPreset = normalizeTaskPresetInput(requestedPreset, { targetInput });
   const presetPackage = normalizedPreset === "none" ? null : readPresetPackage(normalizedPreset, { targetInput });
@@ -220,15 +197,28 @@ export function createTask(targetInput, taskId, { title = "", locale = "en-US",
   if (presetPackage && !presetPackage.compatibleBudgets.includes(normalizedBudget)) throw new Error(`${normalizedPreset} preset requires --budget ${presetPackage.compatibleBudgets.join("|")}`);
   if (presetPackage?.task?.projectLevelOnly === true && moduleKey) throw new Error(`${normalizedPreset} preset is project-level and cannot be combined with --module`);
   if (presetPackage?.task?.requiresFromSession === true && !fromSession) throw new Error(`${normalizedPreset} preset requires --from-session`);
-  const rawNormalized = normalizeTaskId(taskId || (presetPackage?.task?.defaultTaskId || ""));
-  const normalizedTaskId = ensureDatePrefix(rawNormalized);
-  if (!normalizedTaskId) throw new Error("Missing task id");
-  const semanticSlug = bareSlug(normalizedTaskId);
   const normalizedModuleKey = moduleKey ? normalizeTaskId(moduleKey) : "";
+  const identity = resolveTaskIdentity({ target, taskId, title, presetPackage, moduleKey: normalizedModuleKey, automaticTaskId });
+  const normalizedTaskId = identity.normalizedTaskId;
+  const semanticSlug = identity.semanticSlug;
   const normalizedLocale = normalizeLocale(locale || readCapabilityRegistry(target).locale);
   const taskTitle = title || (normalizedPreset === "legacy-migration" ? "Harness v1 legacy migration" : semanticSlug);
   const directory = taskRoot(target, normalizedTaskId, { moduleKey: normalizedModuleKey });
   if (fs.existsSync(directory)) throw new Error(`Task already exists: ${normalizedTaskId}`);
+  const scaffoldProvenance = buildScaffoldProvenance({
+    taskId,
+    normalizedTaskId,
+    title,
+    locale: normalizedLocale,
+    budget: normalizedBudget,
+    longRunning,
+    moduleKey: normalizedModuleKey,
+    preset: normalizedPreset,
+    fromSession,
+    targetInput: presetInputs?.targetInput || targetInput,
+    automaticTaskId,
+  });
+  const baseTaskAudit = buildCreationTaskAudit(scaffoldProvenance, { projectRoot: target.projectRoot });
   const evaluatedPresetValues = presetPackage ? evaluateTemplateValues(presetPackage, presetInputs.inputs, { taskId: normalizedTaskId, taskTitle, moduleKey: normalizedModuleKey }) : null;
   const presetContext = presetPackage
     ? buildPresetContext({ ...presetPackage, task: { ...(presetPackage.task || {}), kind: presetPackage.task?.kind || "general" } }, {
@@ -240,8 +230,37 @@ export function createTask(targetInput, taskId, { title = "", locale = "en-US",
         evaluatedValues: evaluatedPresetValues,
       })
     : null;
+  const task = {
+    id: taskIdForDirectory(target, directory),
+    shortId: normalizedTaskId,
+    title: taskTitle,
+    module: normalizedModuleKey || null,
+    path: `TARGET:${toPosix(path.relative(target.projectRoot, directory))}`,
+    locale: normalizedLocale,
+    budget: normalizedBudget,
+    kind: presetContext?.kind || "general",
+    preset: normalizedPreset,
+    presetVersion: presetContext?.presetVersion || "",
+    presetAudit: presetContext?.audit || null,
+    migrationTargetLevel: presetContext?.migrationTargetLevel || "",
+    migrationAchievedLevel: presetContext?.migrationAchievedLevel || "",
+    evidenceBundle: presetContext?.evidenceBundle || "",
+    longRunning,
+  };
+  const plannedChanges = planCreateTaskChanges({
+    target,
+    directory,
+    normalizedModuleKey,
+    normalizedLocale,
+    normalizedBudget,
+    longRunning,
+    presetContext,
+    task,
+  });
+  const plannedGovernance = syncTaskGovernance(target, task, { event: "new-task", state: "planned", message: "task registered by CLI", dryRun: true });
+  const plannedWriteScopes = governanceRelativePaths([...plannedChanges, ...plannedGovernance.changes]);
   const changes = [];
-  const governanceContext = beginGovernanceSync(target, { operation: `new-task ${normalizedTaskId}`, dryRun });
+  const governanceContext = beginGovernanceSync(target, { operation: `new-task ${normalizedTaskId}`, dryRun, allowDirtyWorktree: true, allowedRelativePaths: plannedWriteScopes });
   try {
   if (normalizedModuleKey) {
     const moduleDirectory = path.dirname(directory);
@@ -263,6 +282,13 @@ export function createTask(targetInput, taskId, { title = "", locale = "en-US",
           title: normalizedModuleKey,
           locale: normalizedLocale,
           budget: normalizedBudget,
+          moduleKey: normalizedModuleKey,
+          preset: normalizedPreset,
+          presetVersion: presetContext?.presetVersion || "",
+          evidenceBundle: presetContext?.evidenceBundle || "",
+          longRunning,
+          scaffoldProvenance,
+          taskAudit: buildCreationTaskAudit({ ...scaffoldProvenance, templateSource: source }, { projectRoot: target.projectRoot }),
         }),
       );
     }
@@ -288,6 +314,18 @@ export function createTask(targetInput, taskId, { title = "", locale = "en-US",
         title: taskTitle,
         locale: normalizedLocale,
         budget: normalizedBudget,
+        moduleKey: normalizedModuleKey,
+        preset: normalizedPreset,
+        presetVersion: presetContext?.presetVersion || "",
+        evidenceBundle: presetContext?.evidenceBundle || "",
+        longRunning,
+        scaffoldProvenance: {
+          ...scaffoldProvenance,
+          templateSource: source,
+        },
+        taskAudit: destination === "INDEX.md"
+          ? buildCreationTaskAudit({ ...scaffoldProvenance, templateSource: source }, { projectRoot: target.projectRoot })
+          : baseTaskAudit,
       }), presetContext),
     );
   }
@@ -333,23 +371,6 @@ export function createTask(targetInput, taskId, { title = "", locale = "en-US",
       fs.writeFileSync(destinationPath, renderPresetResourceIndex(existing, kind, rows));
     }
   }
-  const task = {
-    id: taskIdForDirectory(target, directory),
-    shortId: normalizedTaskId,
-    title: taskTitle,
-    module: normalizedModuleKey || null,
-    path: `TARGET:${toPosix(path.relative(target.projectRoot, directory))}`,
-    locale: normalizedLocale,
-    budget: normalizedBudget,
-    kind: presetContext?.kind || "general",
-    preset: normalizedPreset,
-    presetVersion: presetContext?.presetVersion || "",
-    presetAudit: presetContext?.audit || null,
-    migrationTargetLevel: presetContext?.migrationTargetLevel || "",
-    migrationAchievedLevel: presetContext?.migrationAchievedLevel || "",
-    evidenceBundle: presetContext?.evidenceBundle || "",
-    longRunning,
-  };
   const governance = syncTaskGovernance(target, task, { event: "new-task", state: "planned", message: "task registered by CLI", dryRun });
   changes.push(...governance.changes);
   const commandWriteScopes = governanceRelativePaths(changes);
@@ -371,21 +392,6 @@ export function createTask(targetInput, taskId, { title = "", locale = "en-US",
   }
 }
 
-function refreshPresetCommandAudit(target, presetContext, { commandWriteScopes = [], dryRun = false } = {}) {
-  const scopes = [...new Set(commandWriteScopes.filter(Boolean))];
-  presetContext.audit = {
-    ...presetContext.audit,
-    presetWriteScopes: presetContext.audit.writeScopes || [],
-    commandWriteScopes: scopes,
-  };
-  for (const evidence of presetContext.evidenceFiles || []) {
-    if (evidence.source !== "preset-audit") continue;
-    evidence.content = `${JSON.stringify(presetContext.audit, null, 2)}\n`;
-    if (dryRun) continue;
-    fs.writeFileSync(path.join(target.projectRoot, evidence.relativePath), evidence.content);
-  }
-}
-
 export function updateTaskLifecycle(targetInput, taskId, { event = "task-log", state = "", message = "", evidence = "" } = {}) {
   const target = normalizeTarget(targetInput);
   const taskDir = resolveTaskDirectory(target, taskId);
@@ -401,6 +407,7 @@ export function updateTaskLifecycle(targetInput, taskId, { event = "task-log", s
     currentState: currentTask?.state || "unknown",
     budget,
     reviewContent: readFileSafe(path.join(taskDir, "review.md")),
+    indexContent: readFileSafe(path.join(taskDir, "INDEX.md")),
     reviewTaskKey: canonicalTaskId,
     projectRoot: target.projectRoot,
     taskDir,
@@ -413,6 +420,8 @@ export function updateTaskLifecycle(targetInput, taskId, { event = "task-log", s
     content = appendProgressLog(content, { event, message, evidence });
     fs.writeFileSync(progressPath, content.endsWith("\n") ? content : `${content}\n`);
     const allowedPaths = [toPosix(path.relative(target.projectRoot, progressPath))];
+    const advancedPhasePath = advanceLifecyclePhase(target, taskDir, event);
+    if (advancedPhasePath) allowedPaths.push(advancedPhasePath);
     if (event === "task-review") {
       const reviewPath = path.join(taskDir, "review.md");
       const reviewContent = readFileSafe(reviewPath);
@@ -430,6 +439,8 @@ export function updateTaskLifecycle(targetInput, taskId, { event = "task-log", s
         ),
       );
       allowedPaths.push(toPosix(path.relative(target.projectRoot, reviewPath)));
+      const lessonDecisionPath = autoRecordNoLessonCandidateDecision(target, taskDir);
+      if (lessonDecisionPath) allowedPaths.push(lessonDecisionPath);
     }
     const task =
       findTaskByDirectory(target, taskDir) ||
@@ -460,50 +471,6 @@ export function confirmTaskReview(targetInput, taskId, { reviewer = "Human Revie
   const taskDir = resolveTaskDirectory(target, taskId);
   return confirmTaskReviewWithContext({ target, taskDir, findTaskByDirectory }, { reviewer, message, confirmText, evidence });
 }
-function renderAgentReviewSubmission({ target, taskDir, canonicalTaskId, message, evidence }) {
-  const timestamp = nowTimestamp();
-  const submissionId = `ARS-${timestamp.replace(/[^0-9]/g, "").slice(0, 14)}`;
-  const materialsHash = hashTaskMaterials(taskDir);
-  const reviewContent = readFileSafe(path.join(taskDir, "review.md"));
-  const openFindings = collectReviewRisks(reviewContent).filter(isBlockingReviewRisk).length;
-  const evidenceSummary = evidence || message || "Agent submitted task for human review.";
-  return [
-    "## Agent Review Submission",
-    "",
-    "| Field | Value |",
-    "| --- | --- |",
-    `| Submission ID | ${submissionId} |`,
-    `| Submitted At | ${timestamp} |`,
-    "| Submitted By | agent |",
-    `| Task Key | ${canonicalTaskId} |`,
-    `| Materials Checklist Hash | ${materialsHash} |`,
-    `| Evidence Summary | ${markdownCell(evidenceSummary)} |`,
-    `| Open Findings Count | ${openFindings} |`,
-    `| Scanner Version | ${taskScannerVersion} |`,
-    `| Target | TARGET:${toPosix(path.relative(target.projectRoot, taskDir))} |`,
-    "",
-  ].join("\n");
-}
-function replaceAgentReviewSubmission(content, block) {
-  const trimmed = String(content || "").trimEnd();
-  if (/^##\s*(?:Agent Review Submission|Agent 审查提交|Agent 提交审查)\s*$/im.test(trimmed)) {
-    return `${trimmed.replace(/^##\s*(?:Agent Review Submission|Agent 审查提交|Agent 提交审查)\s*$[\s\S]*?(?=^##\s+|(?![\s\S]))/im, `${block.trimEnd()}\n\n`)}\n`;
-  }
-  return `${trimmed}\n\n${block.trimEnd()}\n`;
-}
-function hashTaskMaterials(taskDir) {
-  const hash = crypto.createHash("sha256");
-  for (const fileName of ["brief.md", "task_plan.md", visualMapFile, lessonCandidatesFile, "progress.md", "review.md", "findings.md", longRunningTaskContractFile]) {
-    const filePath = path.join(taskDir, fileName);
-    if (!fs.existsSync(filePath)) continue;
-    hash.update(fileName);
-    hash.update("\0");
-    hash.update(readFileSafe(filePath));
-    hash.update("\0");
-  }
-  return hash.digest("hex").slice(0, 16);
-}
-
 export function updateTaskPhase(targetInput, taskId, phaseId, { state = "", completion = "", evidenceStatus = "" } = {}) {
   const target = normalizeTarget(targetInput);
   const taskDir = resolveTaskDirectory(target, taskId);