nightralph 0.0.33 → 0.0.35

package/dist/docs-templates/issue-tracker.md

@@ -6,18 +6,26 @@ Issues and specs for this repo live as markdown files in `.scratch/`.
 
 - One feature per directory: `.scratch/<feature-slug>/`
 - The spec is `.scratch/<feature-slug>/spec.md`
-- Implementation issues are one file per ticket at `.scratch/<feature-slug>/issues/<NN>-<slug>.md`, numbered from `01`, never a single combined tickets file
-- Triage state is recorded as a `Status:` line near the top of each issue file (see `triage-labels.md` for the role strings)
-- Dependencies are recorded as a `**Blocked by:**` line near the top of each issue file, listing the numbers of the tickets that must complete first, or `None`
-- Comments and conversation history append to the bottom of the file under a `## Comments` heading
+- Implementation issues are one file per ticket at
+  `.scratch/<feature-slug>/issues/<NN>-<slug>.md`, numbered from `01`,
+  never a single combined tickets file
+- Triage state is recorded as a `**Status:**` line near the top of each
+  issue file (see `triage-labels.md` for the role strings)
+- Dependencies are recorded as a `**Blocked by:**` line near the top of
+  each issue file, listing the numbers of the tickets that must complete
+  first, or `None`
+- Comments and conversation history append to the bottom of the file under
+  a `## Comments` heading
 
 ## When a skill says "publish to the issue tracker"
 
-Create a new file under `.scratch/<feature-slug>/` (creating the directory if needed).
+Create a new file under `.scratch/<feature-slug>/` (creating the directory
+if needed).
 
 ## When a skill says "fetch the relevant ticket"
 
-Read the file at the referenced path. The user will normally pass the path or the issue number directly.
+Read the file at the referenced path. The user will normally pass the path
+or the issue number directly.
 
 ## Tickets are executed within hours, not weeks
 

package/dist/index.js

@@ -1152,30 +1152,110 @@ function findReadyTickets(tickets) {
   });
 }
 __name(findReadyTickets, "findReadyTickets");
-var COMPLETED_RE = /^\[x\]\s+(.+)$/i;
+var COMPLETED_RE = /^\s*[-*+]?\s*\[x\]\s+(.+)$/i;
 var TOOL_USE_RE = /^\[[\w.]+\]/;
+var CHECKBOX_RE = /^(\s*[-*+] )\[( |x|X)\]( .*)$/;
+var UNCHECKED_BOX_RE = /^(\s*[-*+] )\[( )\]/;
+function normalizeItemText(s) {
+  return s.replace(/\s+/g, " ").trim().toLowerCase();
+}
+__name(normalizeItemText, "normalizeItemText");
+function stripCheckboxPrefix(s) {
+  return s.replace(/^\s*[-*+]?\s*\[[ xX]\]\s+/, "");
+}
+__name(stripCheckboxPrefix, "stripCheckboxPrefix");
+function parseChecklistItems(lines) {
+  const items = [];
+  let inFence = false;
+  let i = 0;
+  while (i < lines.length) {
+    if (/^\s*```/.test(lines[i])) {
+      inFence = !inFence;
+      i++;
+      continue;
+    }
+    if (inFence) {
+      i++;
+      continue;
+    }
+    const m = lines[i].match(CHECKBOX_RE);
+    if (!m) {
+      i++;
+      continue;
+    }
+    const start = i;
+    let text = m[3];
+    i++;
+    while (i < lines.length && /^\s+\S/.test(lines[i]) && !CHECKBOX_RE.test(lines[i])) {
+      text += " " + lines[i].trim();
+      i++;
+    }
+    items.push({
+      start,
+      text: normalizeItemText(text)
+    });
+  }
+  return items;
+}
+__name(parseChecklistItems, "parseChecklistItems");
 function checkItems(ticket, completedItems) {
   if (completedItems.length === 0) return;
-  let body = ticket.body;
-  for (const item of completedItems) {
-    const escaped = item.replace(
-      /[.*+?^${}()|[\]\\]/g,
-      "\\$&"
-    );
-    body = body.replace(
-      new RegExp(
-        `- \\[ \\] ${escaped}`,
-        "i"
-      ),
-      `- [x] ${item}`
+  let body;
+  try {
+    body = readFileSync4(ticket.filepath, "utf8");
+  } catch {
+    body = ticket.body;
+  }
+  const wanted = new Set(
+    completedItems.map(
+      (item) => normalizeItemText(stripCheckboxPrefix(item))
+    )
+  );
+  const lines = body.split("\n");
+  let changed = false;
+  for (const item of parseChecklistItems(lines)) {
+    if (!wanted.has(item.text)) continue;
+    const flipped = lines[item.start].replace(
+      UNCHECKED_BOX_RE,
+      "$1[x]"
     );
+    if (flipped !== lines[item.start]) {
+      lines[item.start] = flipped;
+      changed = true;
+    }
   }
+  if (!changed) return;
+  body = lines.join("\n");
   writeFileSync3(ticket.filepath, body);
   ticket.body = body;
 }
 __name(checkItems, "checkItems");
+function checkAllItems(body) {
+  const lines = body.split("\n");
+  let inFence = false;
+  for (let i = 0; i < lines.length; i++) {
+    if (/^\s*```/.test(lines[i])) {
+      inFence = !inFence;
+      continue;
+    }
+    if (inFence) continue;
+    lines[i] = lines[i].replace(
+      UNCHECKED_BOX_RE,
+      "$1[x]"
+    );
+  }
+  return lines.join("\n");
+}
+__name(checkAllItems, "checkAllItems");
 function markDone(ticket) {
-  const updated = ticket.body.replace(
+  let body;
+  try {
+    body = readFileSync4(ticket.filepath, "utf8");
+  } catch {
+    body = ticket.body;
+  }
+  body = checkAllItems(body);
+  const updated = body.replace(
     /\*\*Status:\*\*\s*.+/i,
     "**Status:** done"
   );
@@ -1590,9 +1670,16 @@ function spawnAgent(opts) {
   proc.stdout.pipe(logStream);
   const piFormatter = isPi ? createPiStreamFormatter() : null;
   const completedItems = [];
+  const seenItems = /* @__PURE__ */ new Set();
   function collectCompleted(text) {
-    const match = text.match(COMPLETED_RE);
-    if (match) completedItems.push(match[1]);
+    for (const line of text.split("\n")) {
+      const match = line.match(COMPLETED_RE);
+      if (!match) continue;
+      const item = match[1].trim();
+      if (!item || seenItems.has(item)) continue;
+      seenItems.add(item);
+      completedItems.push(item);
+    }
   }
   __name(collectCompleted, "collectCompleted");
   const rl = createInterface({ input: proc.stdout });