@seanmars/tospec 0.19.0-beta.8 → 0.20.0

package/dist/utils/task-progress.js

@@ -1,26 +1,41 @@
 import { promises as fs } from 'fs';
 import path from 'path';
 import { resolveArtifactOutputs, resolveSchema } from '../core/artifact-graph/index.js';
+import { buildCodeFenceMask, normalizeDocument, stripBom } from '../core/parsers/requirement-text.js';
 import { resolveSchemaForChange } from './change-metadata.js';
 /**
- * `- [ ] text` / `- [x] text`, capturing leading indentation, the checkbox state
- * and the text. Indented (nested) tasks count exactly like top-level ones, so a
- * subtask is never invisible to progress; the indent is kept only so the
+ * `- [ ] text` / `- [x] text`, capturing indentation, checkbox state and text.
+ * Nested tasks count like top-level ones; the indent is kept only so the
  * dashboard can render the nesting it found.
  *
- * Limitation: line-based, so a `- [ ]` inside an indented code block in tasks.md
- * miscounts. A real markdown parse is the fix if that ever bites.
+ * The bullet class and the required whitespace after `]` are GFM's deliberately:
+ * GitHub's rendering is the only view of tasks.md the author can check against,
+ * so "tasks tospec sees" must equal "tasks GitHub draws". Too narrow and
+ * `+ [ ] 1.2 …` is a box nobody can count; too wide and `- [ ]2.6 …` is a task
+ * nobody can tick.
+ *
+ * Fences are masked by the callers (`buildCodeFenceMask`), not here: a tasks.md
+ * documenting the task format puts real `- [ ]` lines in a fence, which would
+ * leave the archive gate permanently incomplete.
  */
-const TASK_PATTERN = /^([ \t]*)[-*]\s+\[([\sxX])\]\s*(.*)$/;
+const TASK_PATTERN = /^([ \t]*)[-*+][ \t]+\[([ \txX])\](?:[ \t]+(.*))?$/;
 export function parseTasksFromContent(content) {
     const tasks = [];
-    content.split(/\r?\n/).forEach((raw, i) => {
+    // normalizeDocument so a leading BOM cannot hide the first line:
+    // `<BOM>- [ ] 1.1` fails TASK_PATTERN's `^` anchor.
+    const lines = normalizeDocument(content).split('\n');
+    const fenced = buildCodeFenceMask(lines);
+    lines.forEach((raw, i) => {
+        if (fenced[i])
+            return;
         const match = TASK_PATTERN.exec(raw);
         if (!match)
             return;
         tasks.push({
             line: i + 1,
-            text: match[3].trim(),
+            // Undefined when the marker is the whole line (`- [ ]`), which GFM draws
+            // as a checkbox: an empty task, not a missing one.
+            text: (match[3] ?? '').trim(),
             done: match[2].toLowerCase() === 'x',
             indent: match[1].replace(/\t/g, '  ').length,
         });
@@ -28,10 +43,10 @@ export function parseTasksFromContent(content) {
     return tasks;
 }
 /**
- * Identifies the change's tracked-tasks artifact: the artifact whose `generates`
- * equals the schema's `apply.tracks` value, falling back to the artifact with id
- * `tasks` when no `apply` block declares what it tracks. (`apply.tracks` is a
- * filename that *selects* the artifact; the glob is that artifact's `generates`.)
+ * The change's tracked-tasks artifact: the one whose `generates` equals the
+ * schema's `apply.tracks` value, falling back to the artifact with id `tasks`
+ * when no `apply` block declares what it tracks. (`apply.tracks` *selects* the
+ * artifact; the glob is that artifact's `generates`.)
  */
 function findTrackedTasksArtifact(schema) {
     const tracks = schema.apply?.tracks;
@@ -41,10 +56,8 @@ function findTrackedTasksArtifact(schema) {
     return schema.artifacts.find((a) => a.id === 'tasks');
 }
 /**
- * Resolves the tracked-tasks artifact's output glob for a change, or undefined
- * when the schema cannot be resolved or no tracked-tasks artifact exists.
- * `resolveSchema` throws on an unresolvable/misnamed schema; we swallow that so
- * the caller falls back to a single top-level `tasks.md` and never crashes.
+ * Undefined when the schema is unresolvable or has no tracked-tasks artifact, so
+ * the caller falls back to a single top-level `tasks.md` rather than crashing.
  */
 function resolveTrackedTasksGlob(changeDir, projectRoot) {
     try {
@@ -57,12 +70,9 @@ function resolveTrackedTasksGlob(changeDir, projectRoot) {
     }
 }
 /**
- * Absolute paths of the files holding a change's tracked tasks: every file
- * matched by the tracked-tasks artifact's `generates` glob — the same
- * file-resolution `tospec status` uses (`resolveArtifactOutputs`) — so tasks are
- * no longer blind to nested `tasks.md` files (#1202). Falls back to a single
- * top-level `tasks.md` when the schema is unresolvable, no tracked-tasks
- * artifact is found, or the glob matches no file.
+ * Absolute paths of the files holding a change's tracked tasks, through the same
+ * `resolveArtifactOutputs` glob resolution `tospec status` uses. Falls back to a
+ * single top-level `tasks.md` when nothing resolves.
  */
 export function resolveTaskFiles(changeDir, projectRoot) {
     const generates = resolveTrackedTasksGlob(changeDir, projectRoot);
@@ -74,9 +84,8 @@ export function resolveTaskFiles(changeDir, projectRoot) {
     return [path.join(changeDir, 'tasks.md')];
 }
 /**
- * Lists every task of a change, in file order. Files that can't be read (absent
- * `tasks.md`, or vanished between glob and read) contribute nothing rather than
- * throwing.
+ * Every task of a change, in file order. Files that cannot be read contribute
+ * nothing rather than throwing.
  */
 export async function listTasksForChange(changesDir, changeName, projectRoot) {
     const changeDir = path.join(changesDir, changeName);
@@ -107,22 +116,34 @@ export async function getTaskProgressForChange(changesDir, changeName, projectRo
  */
 export async function setTaskDone(file, line, done) {
     const content = await fs.readFile(file, 'utf-8');
-    const lines = content.split('\n');
+    // BOM stripped so `line` means the same line it did in `parseTasksFromContent`,
+    // which numbers from BOM-free text. Splitting on '\n' leaves each line's '\r'
+    // attached, which is what preserves CRLF through the rejoin below.
+    const lines = stripBom(content).split('\n');
     const raw = lines[line - 1];
-    // Splitting on '\n' is what preserves CRLF through the rejoin below, so the
-    // line still carries its '\r' here. Match without it — `TASK_PATTERN`'s tail
-    // cannot cross a line terminator — but keep writing `raw`, endings intact.
-    if (raw === undefined || !TASK_PATTERN.test(raw.replace(/\r$/, ''))) {
+    // A fenced line is "not a task" for the same reason it is not counted: it is a
+    // quoted example, and writing to it edits documentation. The UI draws no
+    // checkbox there, so reaching here means a stale client — hence the caller's 409.
+    const fenced = buildCodeFenceMask(lines);
+    if (raw === undefined || fenced[line - 1] || !TASK_PATTERN.test(raw.replace(/\r$/, ''))) {
         throw new Error(`Line ${line} of ${path.basename(file)} is not a task`);
     }
-    lines[line - 1] = raw.replace(/\[[\sxX]\]/, done ? '[x]' : '[ ]');
+    // Same box class as TASK_PATTERN, so this rewrites exactly the marker the line
+    // matched on; the first `[…]` on a task line is always it.
+    lines[line - 1] = raw.replace(/\[[ \txX]\]/, done ? '[x]' : '[ ]');
     await fs.writeFile(file, lines.join('\n'), 'utf-8');
 }
+/**
+ * "Tasks done", not "Complete": this counts checkboxes, while `tospec status`
+ * answers whether the change's artifacts exist. A change with only a finished
+ * `tasks.md` read as `✓ Complete` in `list` and `1/4 artifacts complete` in
+ * `status`, and the word was what made them look contradictory.
+ */
 export function formatTaskStatus(progress) {
     if (progress.total === 0)
         return 'No tasks';
     if (progress.completed === progress.total)
-        return '✓ Complete';
+        return '✓ Tasks done';
     return `${progress.completed}/${progress.total} tasks`;
 }
 //# sourceMappingURL=task-progress.js.map

package/dist/utils/task-progress.js.map

@@ -1 +1 @@
-{"version":3,"file":"task-progress.js","sourceRoot":"","sources":["../../src/utils/task-progress.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,IAAI,EAAE,EAAE,MAAM,IAAI,CAAC;AACpC,OAAO,IAAI,MAAM,MAAM,CAAC;AAExB,OAAO,EAAE,sBAAsB,EAAE,aAAa,EAAE,MAAM,iCAAiC,CAAC;AACxF,OAAO,EAAE,sBAAsB,EAAE,MAAM,sBAAsB,CAAC;AAE9D;;;;;;;;GAQG;AACH,MAAM,YAAY,GAAG,sCAAsC,CAAC;AAkB5D,MAAM,UAAU,qBAAqB,CAAC,OAAe;IACnD,MAAM,KAAK,GAAiB,EAAE,CAAC;IAC/B,OAAO,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,CAAC,GAAG,EAAE,CAAC,EAAE,EAAE;QACxC,MAAM,KAAK,GAAG,YAAY,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACrC,IAAI,CAAC,KAAK;YAAE,OAAO;QACnB,KAAK,CAAC,IAAI,CAAC;YACT,IAAI,EAAE,CAAC,GAAG,CAAC;YACX,IAAI,EAAE,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE;YACrB,IAAI,EAAE,KAAK,CAAC,CAAC,CAAC,CAAC,WAAW,EAAE,KAAK,GAAG;YACpC,MAAM,EAAE,KAAK,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,KAAK,EAAE,IAAI,CAAC,CAAC,MAAM;SAC7C,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IACH,OAAO,KAAK,CAAC;AACf,CAAC;AAED;;;;;GAKG;AACH,SAAS,wBAAwB,CAAC,MAAkB;IAClD,MAAM,MAAM,GAAG,MAAM,CAAC,KAAK,EAAE,MAAM,CAAC;IACpC,IAAI,MAAM,IAAI,IAAI,EAAE,CAAC;QACnB,OAAO,MAAM,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,SAAS,KAAK,MAAM,CAAC,CAAC;IAC9D,CAAC;IACD,OAAO,MAAM,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,OAAO,CAAC,CAAC;AACxD,CAAC;AAED;;;;;GAKG;AACH,SAAS,uBAAuB,CAAC,SAAiB,EAAE,WAAmB;IACrE,IAAI,CAAC;QACH,MAAM,UAAU,GAAG,sBAAsB,CAAC,SAAS,EAAE,SAAS,EAAE,WAAW,CAAC,CAAC;QAC7E,MAAM,MAAM,GAAG,aAAa,CAAC,UAAU,EAAE,WAAW,CAAC,CAAC;QACtD,OAAO,wBAAwB,CAAC,MAAM,CAAC,EAAE,SAAS,CAAC;IACrD,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,SAAS,CAAC;IACnB,CAAC;AACH,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,gBAAgB,CAAC,SAAiB,EAAE,WAAmB;IACrE,MAAM,SAAS,GAAG,uBAAuB,CAAC,SAAS,EAAE,WAAW,CAAC,CAAC;IAClE,IAAI,SAAS,EAAE,CAAC;QACd,MAAM,KAAK,GAAG,sBAAsB,CAAC,SAAS,EAAE,SAAS,CAAC,CAAC;QAC3D,IAAI,KAAK,CAAC,MAAM,GAAG,CAAC;YAAE,OAAO,KAAK,CAAC;IACrC,CAAC;IACD,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,UAAU,CAAC,CAAC,CAAC;AAC5C,CAAC;AAED;;;;GAIG;AACH,MAAM,CAAC,KAAK,UAAU,kBAAkB,CACtC,UAAkB,EAClB,UAAkB,EAClB,WAAmB;IAEnB,MAAM,SAAS,GAAG,IAAI,CAAC,IAAI,CAAC,UAAU,EAAE,UAAU,CAAC,CAAC;IACpD,MAAM,KAAK,GAAe,EAAE,CAAC;IAC7B,KAAK,MAAM,IAAI,IAAI,gBAAgB,CAAC,SAAS,EAAE,WAAW,CAAC,EAAE,CAAC;QAC5D,IAAI,OAAe,CAAC;QACpB,IAAI,CAAC;YACH,OAAO,GAAG,MAAM,EAAE,CAAC,QAAQ,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC;QAC7C,CAAC;QAAC,MAAM,CAAC;YACP,SAAS;QACX,CAAC;QACD,KAAK,MAAM,IAAI,IAAI,qBAAqB,CAAC,OAAO,CAAC;YAAE,KAAK,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,GAAG,IAAI,EAAE,CAAC,CAAC;IACnF,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED,qFAAqF;AACrF,MAAM,CAAC,KAAK,UAAU,wBAAwB,CAC5C,UAAkB,EAClB,UAAkB,EAClB,WAAmB;IAEnB,MAAM,KAAK,GAAG,MAAM,kBAAkB,CAAC,UAAU,EAAE,UAAU,EAAE,WAAW,CAAC,CAAC;IAC5E,OAAO,EAAE,KAAK,EAAE,KAAK,CAAC,MAAM,EAAE,SAAS,EAAE,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,MAAM,EAAE,CAAC;AAChF,CAAC;AAED;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,WAAW,CAAC,IAAY,EAAE,IAAY,EAAE,IAAa;IACzE,MAAM,OAAO,GAAG,MAAM,EAAE,CAAC,QAAQ,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC;IACjD,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAClC,MAAM,GAAG,GAAG,KAAK,CAAC,IAAI,GAAG,CAAC,CAAC,CAAC;IAC5B,4EAA4E;IAC5E,6EAA6E;IAC7E,2EAA2E;IAC3E,IAAI,GAAG,KAAK,SAAS,IAAI,CAAC,YAAY,CAAC,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC,EAAE,CAAC;QACpE,MAAM,IAAI,KAAK,CAAC,QAAQ,IAAI,OAAO,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,gBAAgB,CAAC,CAAC;IAC1E,CAAC;IACD,KAAK,CAAC,IAAI,GAAG,CAAC,CAAC,GAAG,GAAG,CAAC,OAAO,CAAC,YAAY,EAAE,IAAI,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC;IAClE,MAAM,EAAE,CAAC,SAAS,CAAC,IAAI,EAAE,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,OAAO,CAAC,CAAC;AACtD,CAAC;AAED,MAAM,UAAU,gBAAgB,CAAC,QAAsB;IACrD,IAAI,QAAQ,CAAC,KAAK,KAAK,CAAC;QAAE,OAAO,UAAU,CAAC;IAC5C,IAAI,QAAQ,CAAC,SAAS,KAAK,QAAQ,CAAC,KAAK;QAAE,OAAO,YAAY,CAAC;IAC/D,OAAO,GAAG,QAAQ,CAAC,SAAS,IAAI,QAAQ,CAAC,KAAK,QAAQ,CAAC;AACzD,CAAC"}
+{"version":3,"file":"task-progress.js","sourceRoot":"","sources":["../../src/utils/task-progress.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,IAAI,EAAE,EAAE,MAAM,IAAI,CAAC;AACpC,OAAO,IAAI,MAAM,MAAM,CAAC;AAExB,OAAO,EAAE,sBAAsB,EAAE,aAAa,EAAE,MAAM,iCAAiC,CAAC;AACxF,OAAO,EAAE,kBAAkB,EAAE,iBAAiB,EAAE,QAAQ,EAAE,MAAM,qCAAqC,CAAC;AACtG,OAAO,EAAE,sBAAsB,EAAE,MAAM,sBAAsB,CAAC;AAE9D;;;;;;;;;;;;;;GAcG;AACH,MAAM,YAAY,GAAG,mDAAmD,CAAC;AAkBzE,MAAM,UAAU,qBAAqB,CAAC,OAAe;IACnD,MAAM,KAAK,GAAiB,EAAE,CAAC;IAC/B,iEAAiE;IACjE,oDAAoD;IACpD,MAAM,KAAK,GAAG,iBAAiB,CAAC,OAAO,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IACrD,MAAM,MAAM,GAAG,kBAAkB,CAAC,KAAK,CAAC,CAAC;IACzC,KAAK,CAAC,OAAO,CAAC,CAAC,GAAG,EAAE,CAAC,EAAE,EAAE;QACvB,IAAI,MAAM,CAAC,CAAC,CAAC;YAAE,OAAO;QACtB,MAAM,KAAK,GAAG,YAAY,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACrC,IAAI,CAAC,KAAK;YAAE,OAAO;QACnB,KAAK,CAAC,IAAI,CAAC;YACT,IAAI,EAAE,CAAC,GAAG,CAAC;YACX,yEAAyE;YACzE,mDAAmD;YACnD,IAAI,EAAE,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE;YAC7B,IAAI,EAAE,KAAK,CAAC,CAAC,CAAC,CAAC,WAAW,EAAE,KAAK,GAAG;YACpC,MAAM,EAAE,KAAK,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,KAAK,EAAE,IAAI,CAAC,CAAC,MAAM;SAC7C,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IACH,OAAO,KAAK,CAAC;AACf,CAAC;AAED;;;;;GAKG;AACH,SAAS,wBAAwB,CAAC,MAAkB;IAClD,MAAM,MAAM,GAAG,MAAM,CAAC,KAAK,EAAE,MAAM,CAAC;IACpC,IAAI,MAAM,IAAI,IAAI,EAAE,CAAC;QACnB,OAAO,MAAM,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,SAAS,KAAK,MAAM,CAAC,CAAC;IAC9D,CAAC;IACD,OAAO,MAAM,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,OAAO,CAAC,CAAC;AACxD,CAAC;AAED;;;GAGG;AACH,SAAS,uBAAuB,CAAC,SAAiB,EAAE,WAAmB;IACrE,IAAI,CAAC;QACH,MAAM,UAAU,GAAG,sBAAsB,CAAC,SAAS,EAAE,SAAS,EAAE,WAAW,CAAC,CAAC;QAC7E,MAAM,MAAM,GAAG,aAAa,CAAC,UAAU,EAAE,WAAW,CAAC,CAAC;QACtD,OAAO,wBAAwB,CAAC,MAAM,CAAC,EAAE,SAAS,CAAC;IACrD,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,SAAS,CAAC;IACnB,CAAC;AACH,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,gBAAgB,CAAC,SAAiB,EAAE,WAAmB;IACrE,MAAM,SAAS,GAAG,uBAAuB,CAAC,SAAS,EAAE,WAAW,CAAC,CAAC;IAClE,IAAI,SAAS,EAAE,CAAC;QACd,MAAM,KAAK,GAAG,sBAAsB,CAAC,SAAS,EAAE,SAAS,CAAC,CAAC;QAC3D,IAAI,KAAK,CAAC,MAAM,GAAG,CAAC;YAAE,OAAO,KAAK,CAAC;IACrC,CAAC;IACD,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,UAAU,CAAC,CAAC,CAAC;AAC5C,CAAC;AAED;;;GAGG;AACH,MAAM,CAAC,KAAK,UAAU,kBAAkB,CACtC,UAAkB,EAClB,UAAkB,EAClB,WAAmB;IAEnB,MAAM,SAAS,GAAG,IAAI,CAAC,IAAI,CAAC,UAAU,EAAE,UAAU,CAAC,CAAC;IACpD,MAAM,KAAK,GAAe,EAAE,CAAC;IAC7B,KAAK,MAAM,IAAI,IAAI,gBAAgB,CAAC,SAAS,EAAE,WAAW,CAAC,EAAE,CAAC;QAC5D,IAAI,OAAe,CAAC;QACpB,IAAI,CAAC;YACH,OAAO,GAAG,MAAM,EAAE,CAAC,QAAQ,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC;QAC7C,CAAC;QAAC,MAAM,CAAC;YACP,SAAS;QACX,CAAC;QACD,KAAK,MAAM,IAAI,IAAI,qBAAqB,CAAC,OAAO,CAAC;YAAE,KAAK,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,GAAG,IAAI,EAAE,CAAC,CAAC;IACnF,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED,qFAAqF;AACrF,MAAM,CAAC,KAAK,UAAU,wBAAwB,CAC5C,UAAkB,EAClB,UAAkB,EAClB,WAAmB;IAEnB,MAAM,KAAK,GAAG,MAAM,kBAAkB,CAAC,UAAU,EAAE,UAAU,EAAE,WAAW,CAAC,CAAC;IAC5E,OAAO,EAAE,KAAK,EAAE,KAAK,CAAC,MAAM,EAAE,SAAS,EAAE,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,MAAM,EAAE,CAAC;AAChF,CAAC;AAED;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,WAAW,CAAC,IAAY,EAAE,IAAY,EAAE,IAAa;IACzE,MAAM,OAAO,GAAG,MAAM,EAAE,CAAC,QAAQ,CAAC,IAAI,EAAE,OAAO,CAAC,CAAC;IACjD,gFAAgF;IAChF,8EAA8E;IAC9E,mEAAmE;IACnE,MAAM,KAAK,GAAG,QAAQ,CAAC,OAAO,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAC5C,MAAM,GAAG,GAAG,KAAK,CAAC,IAAI,GAAG,CAAC,CAAC,CAAC;IAC5B,+EAA+E;IAC/E,yEAAyE;IACzE,kFAAkF;IAClF,MAAM,MAAM,GAAG,kBAAkB,CAAC,KAAK,CAAC,CAAC;IACzC,IAAI,GAAG,KAAK,SAAS,IAAI,MAAM,CAAC,IAAI,GAAG,CAAC,CAAC,IAAI,CAAC,YAAY,CAAC,IAAI,CAAC,GAAG,CAAC,OAAO,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC,EAAE,CAAC;QACxF,MAAM,IAAI,KAAK,CAAC,QAAQ,IAAI,OAAO,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,gBAAgB,CAAC,CAAC;IAC1E,CAAC;IACD,+EAA+E;IAC/E,2DAA2D;IAC3D,KAAK,CAAC,IAAI,GAAG,CAAC,CAAC,GAAG,GAAG,CAAC,OAAO,CAAC,aAAa,EAAE,IAAI,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC;IACnE,MAAM,EAAE,CAAC,SAAS,CAAC,IAAI,EAAE,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,OAAO,CAAC,CAAC;AACtD,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,gBAAgB,CAAC,QAAsB;IACrD,IAAI,QAAQ,CAAC,KAAK,KAAK,CAAC;QAAE,OAAO,UAAU,CAAC;IAC5C,IAAI,QAAQ,CAAC,SAAS,KAAK,QAAQ,CAAC,KAAK;QAAE,OAAO,cAAc,CAAC;IACjE,OAAO,GAAG,QAAQ,CAAC,SAAS,IAAI,QAAQ,CAAC,KAAK,QAAQ,CAAC;AACzD,CAAC"}

package/dist/utils/timestamp.d.ts

@@ -1,15 +1,12 @@
 /**
- * Formats a Date as `yyyyMMdd_HHmmss` in local time (24-hour, zero-padded).
- *
- * The single source of CLI-generated timestamps used for naming files and
- * directories (archive dirs, ticket files) — agents must never assemble the
- * timestamp themselves, so the format lives in exactly one place.
+ * `yyyyMMdd_HHmmss` in local time. The single source of the stamps that name
+ * archive dirs and ticket files — agents must never assemble one by hand.
  */
 export declare function formatTimestamp(date: Date): string;
 /**
- * Formats a Date as `yyyy-MM-dd` in local time. Use this for date-only metadata
- * instead of `toISOString().split('T')[0]`, which truncates in UTC and records
- * the wrong calendar day near local midnight.
+ * A Date as `yyyy-MM-dd` in local time. For date-only metadata, instead of
+ * `toISOString().split('T')[0]`, which truncates in UTC and records the wrong
+ * calendar day near local midnight.
  */
 export declare function formatLocalDate(date?: Date): string;
 //# sourceMappingURL=timestamp.d.ts.map

package/dist/utils/timestamp.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"timestamp.d.ts","sourceRoot":"","sources":["../../src/utils/timestamp.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AACH,wBAAgB,eAAe,CAAC,IAAI,EAAE,IAAI,GAAG,MAAM,CASlD;AAED;;;;GAIG;AACH,wBAAgB,eAAe,CAAC,IAAI,GAAE,IAAiB,GAAG,MAAM,CAG/D"}
+{"version":3,"file":"timestamp.d.ts","sourceRoot":"","sources":["../../src/utils/timestamp.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,wBAAgB,eAAe,CAAC,IAAI,EAAE,IAAI,GAAG,MAAM,CASlD;AAED;;;;GAIG;AACH,wBAAgB,eAAe,CAAC,IAAI,GAAE,IAAiB,GAAG,MAAM,CAG/D"}

package/dist/utils/timestamp.js

@@ -1,9 +1,6 @@
 /**
- * Formats a Date as `yyyyMMdd_HHmmss` in local time (24-hour, zero-padded).
- *
- * The single source of CLI-generated timestamps used for naming files and
- * directories (archive dirs, ticket files) — agents must never assemble the
- * timestamp themselves, so the format lives in exactly one place.
+ * `yyyyMMdd_HHmmss` in local time. The single source of the stamps that name
+ * archive dirs and ticket files — agents must never assemble one by hand.
  */
 export function formatTimestamp(date) {
     const pad = (n) => String(n).padStart(2, '0');
@@ -16,9 +13,9 @@ export function formatTimestamp(date) {
     return `${year}${month}${day}_${hours}${minutes}${seconds}`;
 }
 /**
- * Formats a Date as `yyyy-MM-dd` in local time. Use this for date-only metadata
- * instead of `toISOString().split('T')[0]`, which truncates in UTC and records
- * the wrong calendar day near local midnight.
+ * A Date as `yyyy-MM-dd` in local time. For date-only metadata, instead of
+ * `toISOString().split('T')[0]`, which truncates in UTC and records the wrong
+ * calendar day near local midnight.
  */
 export function formatLocalDate(date = new Date()) {
     const pad = (n) => String(n).padStart(2, '0');

package/dist/utils/timestamp.js.map

@@ -1 +1 @@
-{"version":3,"file":"timestamp.js","sourceRoot":"","sources":["../../src/utils/timestamp.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AACH,MAAM,UAAU,eAAe,CAAC,IAAU;IACxC,MAAM,GAAG,GAAG,CAAC,CAAS,EAAU,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IAC9D,MAAM,IAAI,GAAG,IAAI,CAAC,WAAW,EAAE,CAAC;IAChC,MAAM,KAAK,GAAG,GAAG,CAAC,IAAI,CAAC,QAAQ,EAAE,GAAG,CAAC,CAAC,CAAC;IACvC,MAAM,GAAG,GAAG,GAAG,CAAC,IAAI,CAAC,OAAO,EAAE,CAAC,CAAC;IAChC,MAAM,KAAK,GAAG,GAAG,CAAC,IAAI,CAAC,QAAQ,EAAE,CAAC,CAAC;IACnC,MAAM,OAAO,GAAG,GAAG,CAAC,IAAI,CAAC,UAAU,EAAE,CAAC,CAAC;IACvC,MAAM,OAAO,GAAG,GAAG,CAAC,IAAI,CAAC,UAAU,EAAE,CAAC,CAAC;IACvC,OAAO,GAAG,IAAI,GAAG,KAAK,GAAG,GAAG,IAAI,KAAK,GAAG,OAAO,GAAG,OAAO,EAAE,CAAC;AAC9D,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,eAAe,CAAC,IAAI,GAAS,IAAI,IAAI,EAAE;IACrD,MAAM,GAAG,GAAG,CAAC,CAAS,EAAU,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IAC9D,OAAO,GAAG,IAAI,CAAC,WAAW,EAAE,IAAI,GAAG,CAAC,IAAI,CAAC,QAAQ,EAAE,GAAG,CAAC,CAAC,IAAI,GAAG,CAAC,IAAI,CAAC,OAAO,EAAE,CAAC,EAAE,CAAC;AACpF,CAAC"}
+{"version":3,"file":"timestamp.js","sourceRoot":"","sources":["../../src/utils/timestamp.ts"],"names":[],"mappings":"AAAA;;;GAGG;AACH,MAAM,UAAU,eAAe,CAAC,IAAU;IACxC,MAAM,GAAG,GAAG,CAAC,CAAS,EAAU,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IAC9D,MAAM,IAAI,GAAG,IAAI,CAAC,WAAW,EAAE,CAAC;IAChC,MAAM,KAAK,GAAG,GAAG,CAAC,IAAI,CAAC,QAAQ,EAAE,GAAG,CAAC,CAAC,CAAC;IACvC,MAAM,GAAG,GAAG,GAAG,CAAC,IAAI,CAAC,OAAO,EAAE,CAAC,CAAC;IAChC,MAAM,KAAK,GAAG,GAAG,CAAC,IAAI,CAAC,QAAQ,EAAE,CAAC,CAAC;IACnC,MAAM,OAAO,GAAG,GAAG,CAAC,IAAI,CAAC,UAAU,EAAE,CAAC,CAAC;IACvC,MAAM,OAAO,GAAG,GAAG,CAAC,IAAI,CAAC,UAAU,EAAE,CAAC,CAAC;IACvC,OAAO,GAAG,IAAI,GAAG,KAAK,GAAG,GAAG,IAAI,KAAK,GAAG,OAAO,GAAG,OAAO,EAAE,CAAC;AAC9D,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,eAAe,CAAC,IAAI,GAAS,IAAI,IAAI,EAAE;IACrD,MAAM,GAAG,GAAG,CAAC,CAAS,EAAU,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IAC9D,OAAO,GAAG,IAAI,CAAC,WAAW,EAAE,IAAI,GAAG,CAAC,IAAI,CAAC,QAAQ,EAAE,GAAG,CAAC,CAAC,IAAI,GAAG,CAAC,IAAI,CAAC,OAAO,EAAE,CAAC,EAAE,CAAC;AACpF,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@seanmars/tospec",
-  "version": "0.19.0-beta.8",
+  "version": "0.20.0",
   "description": "Spec-driven development CLI for structured requirements and issue workflows",
   "license": "MIT",
   "homepage": "https://www.npmjs.com/package/@seanmars/tospec",
@@ -53,7 +53,6 @@
     "skills": "node build.js && node scripts/gen-skills.mjs",
     "templates": "node scripts/gen-templates.mjs",
     "dev": "tsc --watch",
-    "test": "vitest run",
-    "prepublish": "node build.js"
+    "test": "vitest run"
   }
 }

package/schemas/issue/schema.yaml

@@ -32,7 +32,14 @@ artifacts:
       directory level). REMOVED lists each name as a bullet with Reason and
       Migration fields; RENAMED writes a FROM:/TO: pair, e.g. "- FROM: `<old
       name>`" / "- TO: `<new name>`". If the bug stems from a wrong spec, fix
-      that Requirement with MODIFIED.
+      that Requirement with MODIFIED — pasting the full updated block, since a
+      MODIFIED that omits a scenario the current spec still has is rejected. To
+      drop or rename a scenario on purpose, omit it and declare it under
+      "## REMOVED Scenarios" as "- Requirement: `<requirement name>`" /
+      "- Scenario: `<scenario name>`" / "- Reason: ...".
+      Include only the sections this fix needs: the template ships ADDED and
+      keeps the optional sections in a fenced example inside a comment. A
+      placeholder section pasted in and left unfilled fails validation.
 apply:
   requires: [task]
   tracks: task.md

package/schemas/issue/templates/spec.md

@@ -16,9 +16,18 @@ edit that spec's Purpose directly instead. -->
 - **THEN** [outcome]
 - **AND** [additional outcome]
 
-<!-- MODIFIED, REMOVED, and RENAMED Requirements may also be used.
-MODIFIED must contain the full updated requirement block. -->
-
+<!-- MODIFIED, REMOVED, RENAMED Requirements and REMOVED Scenarios are optional
+sections. Add only the ones this change needs, copied from the fenced example
+below. The fence keeps the example out of the parser; a section copied out of
+it and left as-is fails validation, because a bracketed `[name]` is rejected as
+a requirement, scenario or entry name — so does the ADDED block above until its
+placeholders are replaced.
+
+MODIFIED must contain the full updated requirement block — omitting a scenario
+the current spec still has is rejected, because archiving would delete it. To
+drop or rename one on purpose, declare it under ## REMOVED Scenarios.
+
+```markdown
 ## REMOVED Requirements
 
 - `[name]`
@@ -29,3 +38,11 @@ MODIFIED must contain the full updated requirement block. -->
 
 - FROM: `[old name]`
 - TO: `[new name]`
+
+## REMOVED Scenarios
+
+- Requirement: `[requirement the scenario belongs to]`
+- Scenario: `[scenario name, omitted from the MODIFIED block above]`
+- Reason: [why this scenario no longer applies]
+```
+-->

package/schemas/sdd/schema.yaml

@@ -22,6 +22,10 @@ artifacts:
       requiredSections: [Why, What Changes, Impact]
       minSectionLength:
         Why: 50
+      # The ceiling is advice about where detail belongs: a Why this long is
+      # usually design.md content, or a change that wants splitting.
+      maxSectionLength:
+        Why: 1000
   - id: specs
     # Exactly one directory level. A deeper path (specs/<area>/<capability>/spec.md)
     # is rejected by the validator: archive's merge only reads one level, so a nested
@@ -38,11 +42,20 @@ artifacts:
       ### Requirement: <name> (state the obligation with SHALL or MUST in the
       body, not only in the heading) /
       #### Scenario: <name> (four #, WHEN/THEN/AND bullets).
-      MODIFIED must paste the full updated requirement block.
+      MODIFIED must paste the full updated requirement block. A MODIFIED that
+      omits a scenario the current spec still has is rejected, because it would
+      delete it silently; to remove or rename one deliberately, drop it from the
+      MODIFIED block and declare it under "## REMOVED Scenarios" as
+      "- Requirement: `<requirement name>`" / "- Scenario: `<scenario name>`" /
+      "- Reason: ...". A rename is that declaration plus the new scenario in the
+      same MODIFIED block.
       REMOVED lists each removed name as a bullet with Reason and Migration
       fields, e.g. "- <name>" / "- Reason: ..." / "- Migration: ...".
       RENAMED writes a FROM:/TO: pair per entry, e.g. "- FROM: `<old name>`" /
       "- TO: `<new name>`".
+      Include only the sections this change needs: the template ships ADDED
+      and keeps the optional sections in a fenced example inside a comment. A
+      placeholder section pasted in and left unfilled fails validation.
       A new capability's delta must open with ## Purpose (one sentence on why this
       capability exists); archive carries it into the newly created main spec. Do
       not add Purpose to an existing capability's delta (edit the main spec's
@@ -83,6 +96,12 @@ artifacts:
       Each task group = one narrow, complete path (independently verifiable).
       Every implementation task is preceded by its own "write the test first" task
       (TDD red before green).
+      Number tasks `<group>.<n>` under a `## <group>.` heading. Validation checks
+      that a task's leading number matches the group it sits under and that no ID
+      is used twice — the two mistakes that make "continue with 2.3" point at the
+      wrong item. Gaps in a sequence (1.1, 1.3) are not checked and are fine:
+      deleting a task mid-list is normal and renumbering the rest would invalidate
+      every reference already made to them.
 apply:
   requires: [tasks]
   tracks: tasks.md

package/schemas/sdd/templates/spec.md

@@ -16,9 +16,18 @@ edit that spec's Purpose directly instead. -->
 - **THEN** [outcome]
 - **AND** [additional outcome]
 
-<!-- MODIFIED, REMOVED, and RENAMED Requirements may also be used.
-MODIFIED must contain the full updated requirement block. -->
-
+<!-- MODIFIED, REMOVED, RENAMED Requirements and REMOVED Scenarios are optional
+sections. Add only the ones this change needs, copied from the fenced example
+below. The fence keeps the example out of the parser; a section copied out of
+it and left as-is fails validation, because a bracketed `[name]` is rejected as
+a requirement, scenario or entry name — so does the ADDED block above until its
+placeholders are replaced.
+
+MODIFIED must contain the full updated requirement block — omitting a scenario
+the current spec still has is rejected, because archiving would delete it. To
+drop or rename one on purpose, declare it under ## REMOVED Scenarios.
+
+```markdown
 ## REMOVED Requirements
 
 - `[name]`
@@ -29,3 +38,11 @@ MODIFIED must contain the full updated requirement block. -->
 
 - FROM: `[old name]`
 - TO: `[new name]`
+
+## REMOVED Scenarios
+
+- Requirement: `[requirement the scenario belongs to]`
+- Scenario: `[scenario name, omitted from the MODIFIED block above]`
+- Reason: [why this scenario no longer applies]
+```
+-->