scrumrun 2.7.8 → 2.7.9

package/README.md

@@ -4,7 +4,7 @@
 
 ScrumRun gives an agent a small command surface and a precise project memory: what should be done, how each attempt happened, which decisions constrain the code, and why the architecture exists in its current form.
 
-**Package:** `2.7.8` · **Method target:** `2.0.0` · **Runtime:** Node.js `>=22.13.0` · **License:** MIT
+**Package:** `2.7.9` · **Method target:** `2.0.0` · **Runtime:** Node.js `>=22.13.0` · **License:** MIT
 
 **New here?** Read the [Quickstart](docs/QUICKSTART.md) — first Run in under 10 minutes, no `SPEC.md` reading required. Full docs map in [`docs/INDEX.md`](docs/INDEX.md).
 

package/lib/commands/normalize-legacy.js

@@ -23,6 +23,8 @@ const { RUN_LEDGER_VERSION } = require("../v2/schema");
 
 const RUN_FILENAME_PATTERN = /^RUN-\d{3,}\.md$/;
 const TASK_REF_PATTERN = /^TASK-\d{3,}$/;
+const RUN_REF_PATTERN = /^RUN-\d{3,}$/;
+const DATE_PATTERN = /^\d{4}-\d{2}-\d{2}$/;
 
 function readFrontmatterAndBody(text) {
   const match = String(text || "").match(/^---\r?\n([\s\S]*?)\r?\n---\r?\n?([\s\S]*)$/);
@@ -46,6 +48,11 @@ function extractTitle(body, fallback) {
   return match ? match[1] : fallback;
 }
 
+function extractTaskReference(body) {
+  const match = String(body || "").match(/\b(TASK-\d{3,})\b/);
+  return match ? match[1] : null;
+}
+
 function serializeFrontmatter(frontmatter) {
   const lines = Object.entries(frontmatter).map(([key, value]) => {
     if (value === null || value === undefined) return `${key}: null`;
@@ -71,27 +78,41 @@ function reserveBackupPath(backupDir, runId) {
 
 function needsNormalization(record, body) {
   if (!record) return { needs: true, reason: "frontmatter is malformed or missing" };
+  const identityErrors = [];
+  if (!RUN_REF_PATTERN.test(record.id || "")) identityErrors.push("id is missing or does not match RUN-NNN");
+  if (!DATE_PATTERN.test(record.created || "")) identityErrors.push("created is missing or is not YYYY-MM-DD");
   if (typeof record.task === "string" && record.task && !TASK_REF_PATTERN.test(record.task)) {
-    return { needs: true, reason: `task field does not match TASK-NNN (was ${JSON.stringify(record.task)})` };
+    identityErrors.push(`task field does not match TASK-NNN (was ${JSON.stringify(record.task)})`);
+  }
+  if ((record.task === null || record.task === undefined || record.task === "") && !extractTaskReference(body)) {
+    identityErrors.push("task is missing and cannot be inferred from the legacy body");
   }
   const result = validateRunLedger(record, body);
   const errors = result.errors || [];
-  if (errors.length) return { needs: true, reason: `ledger validation failed (${errors.length} error${errors.length === 1 ? "" : "s"})`, errors };
+  if (errors.length || identityErrors.length) {
+    const allErrors = [...identityErrors, ...errors];
+    const primaryIdentityError = identityErrors.find((error) => error.startsWith("task field")) || identityErrors[0];
+    return { needs: true, reason: primaryIdentityError || `ledger validation failed (${errors.length} error${errors.length === 1 ? "" : "s"})`, errors: allErrors };
+  }
   return { needs: false };
 }
 
-function buildNormalizedBody({ record, originalBody, backupRelative, reason, errors }) {
-  const title = extractTitle(originalBody, `Run ${record.id}`);
+function buildNormalizedBody({ record, runId, originalBody, backupRelative, reason, errors }) {
+  const normalizedId = RUN_REF_PATTERN.test(record.id || "") ? record.id : runId;
+  const normalizedTask = TASK_REF_PATTERN.test(record.task || "") ? record.task : extractTaskReference(originalBody);
+  const snapshotDate = DATE_PATTERN.test(record.created || "")
+    ? record.created
+    : DATE_PATTERN.test(record.updated || "") ? record.updated : new Date().toISOString().slice(0, 10);
+  const title = extractTitle(originalBody, `Run ${normalizedId}`);
   const sha = crypto.createHash("sha256").update(originalBody).digest("hex");
-  const snapshotDate = record.created || new Date().toISOString().slice(0, 10);
   const normalizedRecord = {
-    id: record.id,
+    id: normalizedId,
     kind: "run",
     status: record.status || "partial",
-    created: record.created,
+    created: snapshotDate,
     updated: snapshotDate,
     method: "2.0.0",
-    task: TASK_REF_PATTERN.test(record.task || "") ? record.task : null,
+    task: normalizedTask,
     sprint: record.sprint || null,
     attempt: record.attempt || 1,
     ledger: RUN_LEDGER_VERSION
@@ -157,6 +178,7 @@ function apply(scrumDir, plan) {
     const backupRelative = path.relative(scrumDir, backupPath);
     const normalized = buildNormalizedBody({
       record: frontmatter,
+      runId: entry.id,
       originalBody: raw,
       backupRelative,
       reason: entry.reason,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "scrumrun",
-  "version": "2.7.8",
+  "version": "2.7.9",
   "description": "Evidence-driven Agile runtime and semantic project memory for AI coding agents.",
   "bin": {
     "scrumrun": "bin/scrumrun.js",