mrvn-cli 0.5.20 → 0.5.22

package/dist/marvin-serve.js

@@ -14369,24 +14369,26 @@ function getEffectiveProgress(frontmatter) {
   if (typeof raw === "number") return Math.max(0, Math.min(100, Math.round(raw)));
   return STATUS_PROGRESS_DEFAULTS[frontmatter.status] ?? 0;
 }
-function propagateProgressFromTask(store, taskId) {
+function propagateProgressFromTask(store, taskId, options) {
   const updated = [];
   const task = store.get(taskId);
   if (!task) return updated;
-  if (DONE_STATUSES.has(task.frontmatter.status)) {
-    if (task.frontmatter.progress !== 100) {
-      store.update(taskId, { progress: 100 });
-      updated.push(taskId);
-    }
-  } else if (!task.frontmatter.progressOverride) {
-    const children = store.list({ type: "contribution" }).filter((d) => d.frontmatter.aboutArtifact === taskId);
-    if (children.length > 0) {
-      const avg = children.reduce((sum, c) => sum + getEffectiveProgress(c.frontmatter), 0) / children.length;
-      const progress = Math.round(avg);
-      if (task.frontmatter.progress !== progress) {
-        store.update(taskId, { progress });
+  if (!options?.skipSelf) {
+    if (DONE_STATUSES.has(task.frontmatter.status)) {
+      if (task.frontmatter.progress !== 100) {
+        store.update(taskId, { progress: 100 });
         updated.push(taskId);
       }
+    } else if (!task.frontmatter.progressOverride) {
+      const children = store.list({ type: "contribution" }).filter((d) => d.frontmatter.aboutArtifact === taskId);
+      if (children.length > 0) {
+        const avg = children.reduce((sum, c) => sum + getEffectiveProgress(c.frontmatter), 0) / children.length;
+        const progress = Math.round(avg);
+        if (task.frontmatter.progress !== progress) {
+          store.update(taskId, { progress });
+          updated.push(taskId);
+        }
+      }
     }
   }
   const aboutArtifact = task.frontmatter.aboutArtifact;
@@ -14398,10 +14400,11 @@ function propagateProgressFromTask(store, taskId) {
   }
   return updated;
 }
-function propagateProgressToAction(store, actionId) {
+function propagateProgressToAction(store, actionId, options) {
   const updated = [];
   const action = store.get(actionId);
   if (!action) return updated;
+  if (options?.skipSelf) return updated;
   if (DONE_STATUSES.has(action.frontmatter.status)) {
     if (action.frontmatter.progress !== 100) {
       store.update(actionId, { progress: 100 });
@@ -14586,10 +14589,11 @@ function createActionTools(store) {
         tags: external_exports.array(external_exports.string()).optional().describe("Replace all tags. When provided with sprints, sprint tags are merged into this array."),
         sprints: external_exports.array(external_exports.string()).optional().describe("Sprint IDs to assign (replaces existing sprint tags). E.g. ['SP-001']."),
         workFocus: external_exports.string().optional().describe("Work focus name (e.g. 'Budget UX'). Replaces existing focus:<value> tag."),
-        progress: external_exports.number().nullable().optional().describe("Explicit progress percentage (0-100). Pass null to clear the override and revert to auto-calculation from children.")
+        progress: external_exports.number().nullable().optional().describe("Explicit progress percentage (0-100). Pass null to clear the override and revert to auto-calculation from children."),
+        progressOverride: external_exports.boolean().optional().describe("Control auto-calculation lock. true = lock progress to explicit value, false = allow auto-calculation from children. When omitted: setting progress implies true, null progress implies false.")
       },
       async (args) => {
-        const { id, content, sprints, tags, workFocus, progress, owner, assignee, ...updates } = args;
+        const { id, content, sprints, tags, workFocus, progress, progressOverride, owner, assignee, ...updates } = args;
         if (owner !== void 0) updates.owner = normalizeOwner(owner);
         if (assignee !== void 0) updates.assignee = assignee;
         if (tags !== void 0) {
@@ -14628,13 +14632,17 @@ function createActionTools(store) {
         }
         if (typeof progress === "number") {
           updates.progress = Math.max(0, Math.min(100, Math.round(progress)));
-          updates.progressOverride = true;
+          updates.progressOverride = progressOverride ?? true;
         } else if (progress === null) {
           updates.progressOverride = false;
+        } else if (progressOverride !== void 0) {
+          updates.progressOverride = progressOverride;
         }
         const doc = store.update(id, updates, content);
         if (args.status !== void 0 || typeof progress === "number") {
-          propagateProgressToAction(store, id);
+          propagateProgressToAction(store, id, {
+            skipSelf: typeof progress === "number"
+          });
         }
         return {
           content: [
@@ -17867,10 +17875,11 @@ function createTaskTools(store) {
         priority: external_exports.enum(["critical", "high", "medium", "low"]).optional().describe("New priority"),
         tags: external_exports.array(external_exports.string()).optional().describe("Replace tags (e.g. remove old tags, add new ones)"),
         workFocus: external_exports.string().optional().describe("Work focus name (e.g. 'Budget UX'). Replaces existing focus:<value> tag."),
-        progress: external_exports.number().nullable().optional().describe("Explicit progress percentage (0-100). Overrides auto-calculation from child contributions. Pass null to clear the override and revert to auto-calculation.")
+        progress: external_exports.number().nullable().optional().describe("Explicit progress percentage (0-100). Overrides auto-calculation from child contributions. Pass null to clear the override and revert to auto-calculation."),
+        progressOverride: external_exports.boolean().optional().describe("Control auto-calculation lock. true = lock progress to explicit value, false = allow auto-calculation from children. When omitted: setting progress implies true, null progress implies false.")
       },
       async (args) => {
-        const { id, content, linkedEpic: rawLinkedEpic, tags: userTags, workFocus, progress, ...updates } = args;
+        const { id, content, linkedEpic: rawLinkedEpic, tags: userTags, workFocus, progress, progressOverride, ...updates } = args;
         const warnings = [];
         if (rawLinkedEpic !== void 0) {
           const linkedEpics = normalizeLinkedEpics(rawLinkedEpic);
@@ -17899,13 +17908,17 @@ function createTaskTools(store) {
         }
         if (typeof progress === "number") {
           updates.progress = Math.max(0, Math.min(100, Math.round(progress)));
-          updates.progressOverride = true;
+          updates.progressOverride = progressOverride ?? true;
         } else if (progress === null) {
           updates.progressOverride = false;
+        } else if (progressOverride !== void 0) {
+          updates.progressOverride = progressOverride;
         }
         const doc = store.update(id, updates, content);
         if (args.status !== void 0 || typeof progress === "number") {
-          propagateProgressFromTask(store, id);
+          propagateProgressFromTask(store, id, {
+            skipSelf: typeof progress === "number"
+          });
         }
         const parts = [`Updated task ${doc.frontmatter.id}: ${doc.frontmatter.title}`];
         if (warnings.length > 0) {
@@ -19829,6 +19842,7 @@ function generateProposedActions(issues) {
 // src/skills/builtin/jira/sprint-progress.ts
 import { query as query2 } from "@anthropic-ai/claude-agent-sdk";
 var DONE_STATUSES6 = /* @__PURE__ */ new Set(["done", "closed", "resolved", "obsolete", "wont do", "cancelled"]);
+var PROGRESS_DONE_STATUSES = /* @__PURE__ */ new Set(["done", "closed", "resolved", "obsolete", "wont do"]);
 var BATCH_SIZE = 5;
 var MAX_LINKED_ISSUES = 50;
 var BLOCKED_WEIGHT_RISK_THRESHOLD = 0.3;
@@ -20755,8 +20769,16 @@ async function _assessArtifactRecursive(store, client, host, options, visited, d
     children.push(childReport);
   }
   if (children.length > 0) {
+    const childProgressValues = children.map((c) => {
+      const updates = c.appliedUpdates.length > 0 ? c.appliedUpdates : c.proposedUpdates;
+      const lastStatus = findLast(updates, (u) => u.field === "status");
+      if (lastStatus && PROGRESS_DONE_STATUSES.has(String(lastStatus.proposedValue))) return 100;
+      const lastProgress = findLast(updates, (u) => u.field === "progress");
+      if (lastProgress) return lastProgress.proposedValue;
+      return c.marvinProgress;
+    });
     const rolledUpProgress = Math.round(
-      children.reduce((s, c) => s + c.marvinProgress, 0) / children.length
+      childProgressValues.reduce((s, p) => s + p, 0) / childProgressValues.length
     );
     if (rolledUpProgress !== currentProgress) {
       proposedUpdates.push({
@@ -20771,8 +20793,14 @@ async function _assessArtifactRecursive(store, client, host, options, visited, d
   const signals = buildSignals(commentSignals, linkedIssues, statusDrift, proposedMarvinStatus);
   const appliedUpdates = [];
   if (options.applyUpdates && proposedUpdates.length > 0) {
+    const doneArtifacts = new Set(
+      proposedUpdates.filter((u) => u.field === "status" && PROGRESS_DONE_STATUSES.has(String(u.proposedValue))).map((u) => u.artifactId)
+    );
     for (const update of proposedUpdates) {
       if (update.field === "review") continue;
+      if (update.field === "progress" && doneArtifacts.has(update.artifactId)) {
+        if (update.proposedValue !== 100) continue;
+      }
       try {
         const updatePayload = {
           [update.field]: update.proposedValue,
@@ -20782,12 +20810,14 @@ async function _assessArtifactRecursive(store, client, host, options, visited, d
           updatePayload.progressOverride = false;
         }
         store.update(update.artifactId, updatePayload);
-        const updatedDoc = store.get(update.artifactId);
-        if (updatedDoc) {
-          if (updatedDoc.frontmatter.type === "task") {
-            propagateProgressFromTask(store, update.artifactId);
-          } else if (updatedDoc.frontmatter.type === "action") {
-            propagateProgressToAction(store, update.artifactId);
+        if (update.field === "status") {
+          const updatedDoc = store.get(update.artifactId);
+          if (updatedDoc) {
+            if (updatedDoc.frontmatter.type === "task") {
+              propagateProgressFromTask(store, update.artifactId);
+            } else if (updatedDoc.frontmatter.type === "action") {
+              propagateProgressToAction(store, update.artifactId);
+            }
           }
         }
         appliedUpdates.push(update);
@@ -21092,6 +21122,12 @@ function formatArtifactChild(parts, child, depth) {
     formatArtifactChild(parts, grandchild, depth + 1);
   }
 }
+function findLast(arr, predicate) {
+  for (let i = arr.length - 1; i >= 0; i--) {
+    if (predicate(arr[i])) return arr[i];
+  }
+  return void 0;
+}
 
 // src/skills/builtin/jira/tools.ts
 var JIRA_TYPE = "jira-issue";