mrvn-cli 0.5.22 → 0.5.24

package/dist/marvin-serve.js

@@ -20732,11 +20732,12 @@ async function _assessArtifactRecursive(store, client, host, options, visited, d
     }
   }
   const primaryHasComments = jiraKey ? (jiraIssues.get(jiraKey)?.comments.length ?? 0) > 0 : false;
+  let commentAnalysisProgress = null;
   if (depth < MAX_LLM_DEPTH && jiraKey && primaryHasComments) {
     const estimatedChars = estimateCommentTextSize(jiraIssues, linkedJiraIssues, linkedIssueSignals);
     if (estimatedChars <= MAX_LLM_COMMENT_CHARS) {
       try {
-        const summary = await analyzeSingleArtifactComments(
+        const analysis = await analyzeSingleArtifactComments(
           fm.id,
           fm.title,
           jiraKey,
@@ -20745,7 +20746,20 @@ async function _assessArtifactRecursive(store, client, host, options, visited, d
           linkedJiraIssues,
           linkedIssueSignals
         );
-        commentSummary = summary;
+        commentSummary = analysis.summary;
+        commentAnalysisProgress = analysis.progressEstimate;
+        if (commentAnalysisProgress !== null) {
+          const hasExplicitProgress = "progress" in fm && typeof fm.progress === "number";
+          if (!hasExplicitProgress && !fm.progressOverride && commentAnalysisProgress !== currentProgress) {
+            proposedUpdates.push({
+              artifactId: fm.id,
+              field: "progress",
+              currentValue: currentProgress,
+              proposedValue: commentAnalysisProgress,
+              reason: `Comment analysis estimates ${commentAnalysisProgress}% progress`
+            });
+          }
+        }
       } catch (err) {
         errors.push(`Comment analysis failed: ${err instanceof Error ? err.message : String(err)}`);
       }
@@ -20826,6 +20840,23 @@ async function _assessArtifactRecursive(store, client, host, options, visited, d
       }
     }
   }
+  if (options.applyUpdates) {
+    const assessmentSummary = buildAssessmentSummary(
+      commentSummary,
+      commentAnalysisProgress,
+      signals,
+      children,
+      linkedIssues
+    );
+    try {
+      store.update(fm.id, {
+        assessmentSummary,
+        lastAssessedAt: assessmentSummary.generatedAt
+      });
+    } catch (err) {
+      errors.push(`Failed to persist assessment summary: ${err instanceof Error ? err.message : String(err)}`);
+    }
+  }
   return {
     artifactId: fm.id,
     title: fm.title,
@@ -20843,6 +20874,7 @@ async function _assessArtifactRecursive(store, client, host, options, visited, d
     progressDrift,
     commentSignals,
     commentSummary,
+    commentAnalysisProgress,
     linkedIssues,
     linkedIssueSignals,
     children,
@@ -20935,13 +20967,15 @@ function estimateCommentTextSize(jiraIssues, linkedJiraIssues, linkedIssueSignal
 }
 var SINGLE_ARTIFACT_COMMENT_PROMPT = `You are a delivery management assistant analyzing Jira comments for a single work item.
 
-Produce a 2-3 sentence progress summary covering:
-- What work has been completed
-- What is pending or blocked
-- Any decisions, handoffs, or scheduling mentioned
-- Relevant context from linked issue comments (if provided)
+Analyze the comments and produce:
+1. A 2-3 sentence progress summary covering: what work has been completed, what is pending or blocked, any decisions/handoffs/scheduling mentioned, and relevant context from linked issue comments (if provided).
+2. A progress estimate (0-100%) based on evidence in the comments \u2014 e.g., if comments indicate all items have been triaged into tasks, or implementation is complete pending review, estimate accordingly. If you cannot determine progress from the comments, set progressEstimate to null.
 
-Return ONLY the summary text, no JSON or formatting.`;
+Return a JSON object with this exact structure:
+{"summary": "your 2-3 sentence summary", "progressEstimate": 75}
+
+Use null for progressEstimate if the comments don't provide enough evidence to estimate.
+IMPORTANT: Only return the JSON object, no other text.`;
 async function analyzeSingleArtifactComments(artifactId, title, jiraKey, jiraStatus, jiraIssues, linkedJiraIssues, linkedIssueSignals) {
   const promptParts = [];
   const primaryData = jiraIssues.get(jiraKey);
@@ -20964,7 +20998,7 @@ ${commentTexts}`);
     promptParts.push(`### Linked: ${signal.sourceKey} (${signal.linkType})
 ${commentTexts}`);
   }
-  if (promptParts.length === 0) return null;
+  if (promptParts.length === 0) return { summary: null, progressEstimate: null };
   const prompt = promptParts.join("\n\n");
   const result = query2({
     prompt,
@@ -20981,11 +21015,25 @@ ${commentTexts}`);
         (b) => b.type === "text"
       );
       if (textBlock) {
-        return textBlock.text.trim();
+        return parseCommentAnalysis(textBlock.text.trim());
       }
     }
   }
-  return null;
+  return { summary: null, progressEstimate: null };
+}
+function parseCommentAnalysis(text) {
+  const parsed = parseLlmJson(text);
+  if (parsed && typeof parsed.summary === "string") {
+    const progressEstimate2 = typeof parsed.progressEstimate === "number" && parsed.progressEstimate >= 0 && parsed.progressEstimate <= 100 ? Math.round(parsed.progressEstimate) : null;
+    return { summary: parsed.summary, progressEstimate: progressEstimate2 };
+  }
+  let progressEstimate = null;
+  const pctMatch = text.match(/(\d{1,3})%/);
+  if (pctMatch) {
+    const pct = parseInt(pctMatch[1], 10);
+    if (pct >= 0 && pct <= 100) progressEstimate = pct;
+  }
+  return { summary: text, progressEstimate };
 }
 function emptyArtifactReport(artifactId, errors) {
   return {
@@ -21005,6 +21053,7 @@ function emptyArtifactReport(artifactId, errors) {
     progressDrift: false,
     commentSignals: [],
     commentSummary: null,
+    commentAnalysisProgress: null,
     linkedIssues: [],
     linkedIssueSignals: [],
     children: [],
@@ -21014,6 +21063,33 @@ function emptyArtifactReport(artifactId, errors) {
     errors
   };
 }
+function buildAssessmentSummary(commentSummary, commentAnalysisProgress, signals, children, linkedIssues) {
+  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 childDoneCount = children.filter((c, i) => {
+    const updates = c.appliedUpdates.length > 0 ? c.appliedUpdates : c.proposedUpdates;
+    const lastStatus = findLast(updates, (u) => u.field === "status");
+    const effectiveStatus = lastStatus ? String(lastStatus.proposedValue) : c.marvinStatus;
+    return DONE_STATUSES6.has(effectiveStatus);
+  }).length;
+  const childRollupProgress = children.length > 0 ? Math.round(childProgressValues.reduce((s, p) => s + p, 0) / childProgressValues.length) : null;
+  return {
+    generatedAt: (/* @__PURE__ */ new Date()).toISOString(),
+    commentSummary,
+    commentAnalysisProgress,
+    signals,
+    childCount: children.length,
+    childDoneCount,
+    childRollupProgress,
+    linkedIssueCount: linkedIssues.length
+  };
+}
 function formatArtifactReport(report) {
   const parts = [];
   parts.push(`# Artifact Assessment \u2014 ${report.artifactId}`);
@@ -21042,6 +21118,9 @@ function formatArtifactReport(report) {
   if (report.commentSummary) {
     parts.push(`## Comments`);
     parts.push(report.commentSummary);
+    if (report.commentAnalysisProgress !== null) {
+      parts.push(`  \u{1F4CA} Comment-derived progress estimate: ${report.commentAnalysisProgress}%`);
+    }
     parts.push("");
   }
   if (report.children.length > 0) {