lee-spec-kit 0.7.8 → 0.7.9

package/dist/index.js

@@ -17875,8 +17875,105 @@ function parseTaskLine(line, index = -1) {
     title: match[4]
   };
 }
+function countLeadingSpaces(line) {
+  const match = line.match(/^(\s*)/);
+  return match?.[1]?.length ?? 0;
+}
+function findTaskBlockEnd(lines, taskLineIndex) {
+  let endIndex = lines.length;
+  for (let index = taskLineIndex + 1; index < lines.length; index++) {
+    if (parseTaskLine(lines[index]) || /^\s*##\s+/.test(lines[index])) {
+      endIndex = index;
+      break;
+    }
+  }
+  return endIndex;
+}
+function isPlaceholderTaskItem(text) {
+  const normalized = text.trim();
+  return normalized === "" || normalized === "-" || /^todo$/i.test(normalized);
+}
+function parseTaskSectionItems(lines, taskLineIndex, headingPattern) {
+  if (taskLineIndex < 0 || taskLineIndex >= lines.length) return void 0;
+  const endIndex = findTaskBlockEnd(lines, taskLineIndex);
+  let sectionHeaderIndex = -1;
+  let sectionHeaderIndent = 0;
+  for (let index = taskLineIndex + 1; index < endIndex; index++) {
+    if (headingPattern.test(lines[index])) {
+      sectionHeaderIndex = index;
+      sectionHeaderIndent = countLeadingSpaces(lines[index]);
+      break;
+    }
+  }
+  if (sectionHeaderIndex === -1) return void 0;
+  const items = [];
+  let placeholderCount = 0;
+  for (let index = sectionHeaderIndex + 1; index < endIndex; index++) {
+    const line = lines[index];
+    if (!line.trim()) continue;
+    const indent = countLeadingSpaces(line);
+    if (indent <= sectionHeaderIndent && /^\s*-\s+/.test(line)) {
+      break;
+    }
+    const match = line.match(/^\s*-\s+(.+?)\s*$/);
+    if (!match) continue;
+    const text = match[1].trim();
+    items.push(text);
+    if (isPlaceholderTaskItem(text)) placeholderCount++;
+  }
+  if (items.length === 0) return void 0;
+  return { items, placeholderCount };
+}
+function parseTaskAcceptance(lines, taskLineIndex) {
+  return parseTaskSectionItems(lines, taskLineIndex, /^\s*-\s+Acceptance:\s*$/);
+}
+function parseTaskChecklist(lines, taskLineIndex) {
+  if (taskLineIndex < 0 || taskLineIndex >= lines.length) return void 0;
+  const endIndex = findTaskBlockEnd(lines, taskLineIndex);
+  let checklistHeaderIndex = -1;
+  let checklistHeaderIndent = 0;
+  for (let index = taskLineIndex + 1; index < endIndex; index++) {
+    if (/^\s*-\s+Checklist:\s*$/.test(lines[index])) {
+      checklistHeaderIndex = index;
+      checklistHeaderIndent = countLeadingSpaces(lines[index]);
+      break;
+    }
+  }
+  if (checklistHeaderIndex === -1) return void 0;
+  let total = 0;
+  let checked = 0;
+  let placeholderCount = 0;
+  for (let index = checklistHeaderIndex + 1; index < endIndex; index++) {
+    const line = lines[index];
+    if (!line.trim()) continue;
+    const indent = countLeadingSpaces(line);
+    if (indent <= checklistHeaderIndent && /^\s*-\s+/.test(line)) {
+      break;
+    }
+    const match = line.match(/^\s*-\s*\[([ xX])\]\s+/);
+    if (!match) continue;
+    total++;
+    if (match[1].toLowerCase() === "x") checked++;
+    const text = line.replace(/^\s*-\s*\[[ xX]\]\s+/, "").trim();
+    if (isPlaceholderTaskItem(text)) placeholderCount++;
+  }
+  if (total === 0) return void 0;
+  return { total, checked, unchecked: total - checked, placeholderCount };
+}
 
 // src/commands/task-run.ts
+function ensureTaskDetailsReady(lines, task) {
+  const acceptance = parseTaskAcceptance(lines, task.index);
+  const checklist = parseTaskChecklist(lines, task.index);
+  const acceptanceHasPlaceholder = !acceptance || acceptance.items.length === 0 || acceptance.placeholderCount > 0;
+  const checklistHasPlaceholder = !checklist || checklist.total === 0 || checklist.placeholderCount > 0;
+  if (acceptanceHasPlaceholder || checklistHasPlaceholder) {
+    throw createCliError(
+      "PRECONDITION_FAILED",
+      `Task "${task.taskId}" still contains Acceptance/Checklist placeholder content. Fill concrete Acceptance items and Checklist items before running task-run.`
+    );
+  }
+}
 function buildTaskRunPrompt(input) {
   const shared = [
     "Read `spec.md`, `plan.md`, and `tasks.md` before editing code.",
@@ -17884,8 +17981,8 @@ function buildTaskRunPrompt(input) {
     "Use additional helper agents only when parallel analysis is clearly worth the extra slot cost.",
     "Keep one writer for overlapping files; do not let multiple sub-agents edit the same files concurrently.",
     "If helper-agent quota is exhausted, continue the task in the main agent instead of blocking progress.",
-    "Update the assigned task status and verification notes in `tasks.md` before leaving this task.",
-    "Mark the task `DONE` only after code changes and verification are complete."
+    "Update the assigned task status, task-local checklist boxes, and verification notes in `tasks.md` before leaving this task.",
+    "Mark the task `DONE` only after code changes and verification are complete. `task-complete` will reject the transition if checklist items remain unchecked."
   ];
   if (input.lang === "ko") {
     return [
@@ -17973,6 +18070,7 @@ async function runTaskRun(featureName, options) {
       `Task "${requestedTaskId}" is already DONE.`
     );
   }
+  ensureTaskDetailsReady(lines, resolvedTask);
   const mode = resolvedTask.status === "TODO" ? "start" : "continue";
   let tasksUpdated = false;
   if (resolvedTask.status === "TODO") {
@@ -18117,6 +18215,13 @@ async function runTaskComplete(featureName, options) {
       `Task "${requestedTaskId}" must be DOING/REVIEW before marking it DONE.`
     );
   }
+  const checklist = parseTaskChecklist(lines, resolvedTask.index);
+  if (checklist && checklist.unchecked > 0) {
+    throw createCliError(
+      "PRECONDITION_FAILED",
+      `Task "${requestedTaskId}" still has unchecked checklist items (${checklist.checked}/${checklist.total}).`
+    );
+  }
   lines[resolvedTask.index] = setTaskStatus2(resolvedTask, "DONE");
   await fs.writeFile(tasksPath, lines.join("\n"), "utf-8");
   const payload = {
@@ -18175,6 +18280,21 @@ function taskCompleteCommand(program2) {
     }
   );
 }
+function collectRepeatableOption(value, previous = []) {
+  return [...previous, value];
+}
+function normalizeTaskDetailItems(values, flagName) {
+  const normalized = (values || []).map((value) => value.trim()).filter(Boolean);
+  for (const value of normalized) {
+    if (value === "-" || /^todo$/i.test(value)) {
+      throw createCliError(
+        "INVALID_ARGUMENT",
+        `${flagName} must contain concrete text, not placeholder values like "-" or "TODO".`
+      );
+    }
+  }
+  return normalized;
+}
 function escapeRegExp8(value) {
   return value.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
 }
@@ -18240,9 +18360,9 @@ function formatTaskBlock(input) {
     `- [TODO][${input.ref}] ${input.taskId} ${input.title}`,
     `  - Date: ${input.recordedAt}`,
     "  - Acceptance:",
-    "    - -",
+    ...input.acceptanceItems.length > 0 ? input.acceptanceItems.map((item) => `    - ${item}`) : ["    - -"],
     "  - Checklist:",
-    "    - [ ] -"
+    ...input.checklistItems.length > 0 ? input.checklistItems.map((item) => `    - [ ] ${item}`) : ["    - [ ] -"]
   ];
 }
 async function resolveTaskFeature(featureName, component) {
@@ -18273,6 +18393,11 @@ async function runTaskAdd(featureName, options) {
   if (!title) {
     throw createCliError("INVALID_ARGUMENT", "`--title` must not be empty.");
   }
+  const acceptanceItems = normalizeTaskDetailItems(
+    options.acceptance,
+    "--acceptance"
+  );
+  const checklistItems = normalizeTaskDetailItems(options.check, "--check");
   const ref = normalizeTaskRef(options.ref);
   if (isPrdRequirementId(ref)) {
     const { definitions } = await scanPrdRequirements(ctx.fs, ctx.config.docsDir);
@@ -18312,7 +18437,14 @@ async function runTaskAdd(featureName, options) {
     nextSectionHeadingIndex
   );
   const recordedAt = getLocalDateString();
-  const blockLines = formatTaskBlock({ ref, taskId, title, recordedAt });
+  const blockLines = formatTaskBlock({
+    ref,
+    taskId,
+    title,
+    recordedAt,
+    acceptanceItems,
+    checklistItems
+  });
   const shouldPrefixBlank = insertIndex > taskListHeadingIndex + 1 && (lines[insertIndex - 1] || "").trim() !== "";
   const shouldSuffixBlank = insertIndex < lines.length && (lines[insertIndex] || "").trim() !== "";
   const insertLines = [
@@ -18343,7 +18475,17 @@ async function runTaskAdd(featureName, options) {
 }
 function taskCommand(program2) {
   const task = program2.command("task").description("Manage tasks");
-  task.command("add [feature-name]").description("Append a new task to the end of Task List").requiredOption("--title <title>", "Task title").requiredOption("--ref <ref>", "Requirement ref: NON-PRD or PRD-FR-001").option("--component <component>", "Component name for multi projects").option("--json", "Output JSON").action(async (featureName, options) => {
+  task.command("add [feature-name]").description("Append a new task to the end of Task List").requiredOption("--title <title>", "Task title").requiredOption("--ref <ref>", "Requirement ref: NON-PRD or PRD-FR-001").option(
+    "--acceptance <text>",
+    "Acceptance item. Repeat to add more than one.",
+    collectRepeatableOption,
+    []
+  ).option(
+    "--check <text>",
+    "Checklist item. Repeat to add more than one.",
+    collectRepeatableOption,
+    []
+  ).option("--component <component>", "Component name for multi projects").option("--json", "Output JSON").action(async (featureName, options) => {
     try {
       await runTaskAdd(featureName, options);
     } catch (error) {