mrvn-cli 0.5.16 → 0.5.17

package/dist/index.d.ts

@@ -16,18 +16,39 @@ interface MarvinUserConfig {
 interface GitConfig {
     remote?: string;
 }
-interface ConditionalJiraStatusEntry {
-    default: string[];
-    inSprint?: string[];
+/**
+ * Conditional status mapping: Jira status → Marvin status with sprint context.
+ * Used when a Jira status should map differently based on sprint membership.
+ */
+interface ConditionalJiraStatusMapping {
+    default: string;
+    inSprint?: string;
 }
-interface JiraStatusMap {
-    [marvinStatus: string]: string[] | ConditionalJiraStatusEntry;
+/**
+ * Flat Jira→Marvin status map (spec format).
+ * Keys are Jira status names, values are Marvin status strings or conditional objects.
+ *
+ * Example:
+ *   "In Progress": in-progress
+ *   "To Do":
+ *     default: backlog
+ *     inSprint: ready
+ */
+interface FlatJiraStatusMap {
+    [jiraStatus: string]: string | ConditionalJiraStatusMapping;
+}
+/**
+ * Legacy Marvin→Jira[] status map (nested format).
+ * Keys are Marvin status names, values are arrays of Jira status names.
+ */
+interface LegacyJiraStatusMap {
+    [marvinStatus: string]: string[];
 }
 interface JiraProjectConfig {
     projectKey?: string;
-    statusMap?: {
-        action?: JiraStatusMap;
-        task?: JiraStatusMap;
+    statusMap?: FlatJiraStatusMap | {
+        action?: LegacyJiraStatusMap;
+        task?: LegacyJiraStatusMap;
     };
 }
 interface MarvinProjectConfig {

package/dist/index.js

@@ -14505,11 +14505,25 @@ import { tool as tool2 } from "@anthropic-ai/claude-agent-sdk";
 
 // src/storage/progress.ts
 var DONE_STATUSES = /* @__PURE__ */ new Set(["done", "closed", "resolved", "cancelled"]);
+var STATUS_PROGRESS_DEFAULTS = {
+  done: 100,
+  closed: 100,
+  resolved: 100,
+  obsolete: 100,
+  "wont do": 100,
+  cancelled: 100,
+  review: 80,
+  "in-progress": 40,
+  ready: 5,
+  blocked: 10,
+  backlog: 0,
+  open: 0
+};
 function getEffectiveProgress(frontmatter) {
   if (DONE_STATUSES.has(frontmatter.status)) return 100;
   const raw = frontmatter.progress;
   if (typeof raw === "number") return Math.max(0, Math.min(100, Math.round(raw)));
-  return 0;
+  return STATUS_PROGRESS_DEFAULTS[frontmatter.status] ?? 0;
 }
 function propagateProgressFromTask(store, taskId) {
   const updated = [];
@@ -25092,35 +25106,62 @@ var DEFAULT_TASK_STATUS_MAP = {
   blocked: ["Blocked"],
   backlog: ["To Do", "Open", "Backlog", "New"]
 };
-function isConditionalEntry(value) {
-  return !Array.isArray(value) && typeof value === "object" && "default" in value;
+function isLegacyFormat(statusMap) {
+  if (!statusMap || typeof statusMap !== "object") return false;
+  const keys = Object.keys(statusMap);
+  if (!keys.every((k) => k === "action" || k === "task")) return false;
+  for (const key of keys) {
+    const val = statusMap[key];
+    if (typeof val !== "object" || val === null) return false;
+    for (const innerVal of Object.values(val)) {
+      if (!Array.isArray(innerVal)) return false;
+      if (!innerVal.every((v) => typeof v === "string")) return false;
+    }
+  }
+  return true;
 }
-function buildStatusLookup(configMap, defaults, inSprint = false) {
-  const map2 = configMap ?? defaults;
+function buildLegacyLookup(legacyMap) {
   const lookup = /* @__PURE__ */ new Map();
-  for (const [marvinStatus, value] of Object.entries(map2)) {
-    const statuses = isConditionalEntry(value) ? value.default : value;
-    for (const js of statuses) {
+  for (const [marvinStatus, jiraStatuses] of Object.entries(legacyMap)) {
+    for (const js of jiraStatuses) {
       lookup.set(js.toLowerCase(), marvinStatus);
     }
   }
-  if (inSprint) {
-    for (const [marvinStatus, value] of Object.entries(map2)) {
-      if (isConditionalEntry(value) && value.inSprint) {
-        for (const js of value.inSprint) {
-          lookup.set(js.toLowerCase(), marvinStatus);
-        }
-      }
+  return lookup;
+}
+function buildFlatLookup(flatMap, inSprint) {
+  const lookup = /* @__PURE__ */ new Map();
+  for (const [jiraStatus, value] of Object.entries(flatMap)) {
+    if (typeof value === "string") {
+      lookup.set(jiraStatus.toLowerCase(), value);
+    } else {
+      const resolved = inSprint && value.inSprint ? value.inSprint : value.default;
+      lookup.set(jiraStatus.toLowerCase(), resolved);
     }
   }
   return lookup;
 }
-function mapJiraStatusForAction(status, configMap, inSprint) {
-  const lookup = buildStatusLookup(configMap, DEFAULT_ACTION_STATUS_MAP, inSprint ?? false);
+function normalizeStatusMap(statusMap) {
+  if (!statusMap) return {};
+  if (isLegacyFormat(statusMap)) {
+    return { legacy: statusMap };
+  }
+  return { flat: statusMap };
+}
+function mapJiraStatusForAction(status, resolved, inSprint = false) {
+  if (resolved.flat) {
+    const lookup2 = buildFlatLookup(resolved.flat, inSprint);
+    return lookup2.get(status.toLowerCase()) ?? "open";
+  }
+  const lookup = buildLegacyLookup(resolved.legacy?.action ?? DEFAULT_ACTION_STATUS_MAP);
   return lookup.get(status.toLowerCase()) ?? "open";
 }
-function mapJiraStatusForTask(status, configMap, inSprint) {
-  const lookup = buildStatusLookup(configMap, DEFAULT_TASK_STATUS_MAP, inSprint ?? false);
+function mapJiraStatusForTask(status, resolved, inSprint = false) {
+  if (resolved.flat) {
+    const lookup2 = buildFlatLookup(resolved.flat, inSprint);
+    return lookup2.get(status.toLowerCase()) ?? "backlog";
+  }
+  const lookup = buildLegacyLookup(resolved.legacy?.task ?? DEFAULT_TASK_STATUS_MAP);
   return lookup.get(status.toLowerCase()) ?? "backlog";
 }
 function isInActiveSprint(store, tags) {
@@ -25181,7 +25222,8 @@ async function fetchJiraStatus(store, client, host, artifactId, statusMap) {
     try {
       const issue2 = await client.getIssueWithLinks(jiraKey);
       const inSprint = isInActiveSprint(store, doc.frontmatter.tags);
-      const proposedStatus = artifactType === "task" ? mapJiraStatusForTask(issue2.fields.status.name, statusMap?.task, inSprint) : mapJiraStatusForAction(issue2.fields.status.name, statusMap?.action, inSprint);
+      const resolved = statusMap ?? {};
+      const proposedStatus = artifactType === "task" ? mapJiraStatusForTask(issue2.fields.status.name, resolved, inSprint) : mapJiraStatusForAction(issue2.fields.status.name, resolved, inSprint);
       const currentStatus = doc.frontmatter.status;
       const linkedIssues = [];
       if (issue2.fields.subtasks) {
@@ -25639,7 +25681,8 @@ async function processIssue(issue2, client, host, dateRange, jiraKeyToArtifacts,
       const jiraStatus = issue2.fields.status?.name;
       if (jiraStatus) {
         const inSprint = store ? isInActiveSprint(store, fm.tags) : false;
-        proposedStatus = artifactType === "task" ? mapJiraStatusForTask(jiraStatus, statusMap?.task, inSprint) : mapJiraStatusForAction(jiraStatus, statusMap?.action, inSprint);
+        const resolved = statusMap ?? {};
+        proposedStatus = artifactType === "task" ? mapJiraStatusForTask(jiraStatus, resolved, inSprint) : mapJiraStatusForAction(jiraStatus, resolved, inSprint);
       }
     }
     marvinArtifacts.push({
@@ -25768,20 +25811,6 @@ var COMPLEXITY_WEIGHTS = {
   "very-complex": 8
 };
 var DEFAULT_WEIGHT = 3;
-var STATUS_PROGRESS_DEFAULTS = {
-  done: 100,
-  closed: 100,
-  resolved: 100,
-  obsolete: 100,
-  "wont do": 100,
-  cancelled: 100,
-  review: 80,
-  "in-progress": 40,
-  ready: 5,
-  backlog: 0,
-  open: 0
-};
-var BLOCKED_DEFAULT_PROGRESS = 10;
 function resolveWeight(complexity) {
   if (complexity && complexity in COMPLEXITY_WEIGHTS) {
     return { weight: COMPLEXITY_WEIGHTS[complexity], weightSource: "complexity" };
@@ -25797,9 +25826,6 @@ function resolveProgress(frontmatter, commentAnalysisProgress) {
     return { progress: Math.max(0, Math.min(100, Math.round(commentAnalysisProgress))), progressSource: "comment-analysis" };
   }
   const status = frontmatter.status;
-  if (status === "blocked") {
-    return { progress: BLOCKED_DEFAULT_PROGRESS, progressSource: "status-default" };
-  }
   const defaultProgress = STATUS_PROGRESS_DEFAULTS[status] ?? 0;
   return { progress: defaultProgress, progressSource: "status-default" };
 }
@@ -25892,7 +25918,8 @@ async function assessSprintProgress(store, client, host, options = {}) {
     if (jiraData) {
       jiraStatus = jiraData.issue.fields.status.name;
       const inSprint = isInActiveSprint(store, fm.tags);
-      proposedMarvinStatus = fm.type === "task" ? mapJiraStatusForTask(jiraStatus, options.statusMap?.task, inSprint) : mapJiraStatusForAction(jiraStatus, options.statusMap?.action, inSprint);
+      const resolved = options.statusMap ?? {};
+      proposedMarvinStatus = fm.type === "task" ? mapJiraStatusForTask(jiraStatus, resolved, inSprint) : mapJiraStatusForAction(jiraStatus, resolved, inSprint);
       const subtasks = jiraData.issue.fields.subtasks ?? [];
       if (subtasks.length > 0) {
         jiraSubtaskProgress = computeSubtaskProgress(subtasks);
@@ -25923,6 +25950,20 @@ async function assessSprintProgress(store, client, host, options = {}) {
         proposedValue: jiraSubtaskProgress,
         reason: `Jira ${jiraKey} subtask progress is ${jiraSubtaskProgress}%`
       });
+    } else if (statusDrift && proposedMarvinStatus && !fm.progressOverride) {
+      const hasExplicitProgress = "progress" in fm && typeof fm.progress === "number";
+      if (!hasExplicitProgress) {
+        const proposedProgress = STATUS_PROGRESS_DEFAULTS[proposedMarvinStatus] ?? 0;
+        if (proposedProgress !== currentProgress) {
+          proposedUpdates.push({
+            artifactId: fm.id,
+            field: "progress",
+            currentValue: currentProgress,
+            proposedValue: proposedProgress,
+            reason: `Status changing to "${proposedMarvinStatus}" \u2192 default progress ${proposedProgress}%`
+          });
+        }
+      }
     }
     const tags = fm.tags ?? [];
     const focusTag = tags.find((t) => t.startsWith("focus:"));
@@ -26301,7 +26342,7 @@ function findByJiraKey(store, jiraKey) {
 function createJiraTools(store, projectConfig) {
   const jiraUserConfig = loadUserConfig().jira;
   const defaultProjectKey = projectConfig?.jira?.projectKey;
-  const statusMap = projectConfig?.jira?.statusMap;
+  const statusMap = normalizeStatusMap(projectConfig?.jira?.statusMap);
   return [
     // --- Local read tools ---
     tool20(
@@ -26928,22 +26969,31 @@ function createJiraTools(store, projectConfig) {
           const s = issue2.fields.status.name;
           statusCounts.set(s, (statusCounts.get(s) ?? 0) + 1);
         }
-        const actionMap = statusMap?.action ?? DEFAULT_ACTION_STATUS_MAP;
-        const taskMap = statusMap?.task ?? DEFAULT_TASK_STATUS_MAP;
         const actionLookup = /* @__PURE__ */ new Map();
-        for (const [marvin, value] of Object.entries(actionMap)) {
-          const jiraStatuses = Array.isArray(value) ? value : value.default;
-          for (const js of jiraStatuses) actionLookup.set(js.toLowerCase(), marvin);
-          if (!Array.isArray(value) && value.inSprint) {
-            for (const js of value.inSprint) actionLookup.set(js.toLowerCase(), `${marvin} (inSprint)`);
-          }
-        }
         const taskLookup = /* @__PURE__ */ new Map();
-        for (const [marvin, value] of Object.entries(taskMap)) {
-          const jiraStatuses = Array.isArray(value) ? value : value.default;
-          for (const js of jiraStatuses) taskLookup.set(js.toLowerCase(), marvin);
-          if (!Array.isArray(value) && value.inSprint) {
-            for (const js of value.inSprint) taskLookup.set(js.toLowerCase(), `${marvin} (inSprint)`);
+        if (statusMap.flat) {
+          for (const [jiraStatus, value] of Object.entries(statusMap.flat)) {
+            const lower = jiraStatus.toLowerCase();
+            if (typeof value === "string") {
+              actionLookup.set(lower, value);
+              taskLookup.set(lower, value);
+            } else {
+              actionLookup.set(lower, value.default);
+              taskLookup.set(lower, value.default);
+              if (value.inSprint) {
+                actionLookup.set(lower, `${value.default} / ${value.inSprint} (inSprint)`);
+                taskLookup.set(lower, `${value.default} / ${value.inSprint} (inSprint)`);
+              }
+            }
+          }
+        } else {
+          const actionMap = statusMap.legacy?.action ?? DEFAULT_ACTION_STATUS_MAP;
+          const taskMap = statusMap.legacy?.task ?? DEFAULT_TASK_STATUS_MAP;
+          for (const [marvin, jiraStatuses] of Object.entries(actionMap)) {
+            for (const js of jiraStatuses) actionLookup.set(js.toLowerCase(), marvin);
+          }
+          for (const [marvin, jiraStatuses] of Object.entries(taskMap)) {
+            for (const js of jiraStatuses) taskLookup.set(js.toLowerCase(), marvin);
           }
         }
         const parts = [
@@ -26965,25 +27015,20 @@ function createJiraTools(store, projectConfig) {
           if (!taskTarget) unmappedTask.push(status);
         }
         if (unmappedAction.length > 0 || unmappedTask.length > 0) {
+          const allUnmapped = [.../* @__PURE__ */ new Set([...unmappedAction, ...unmappedTask])];
           parts.push("");
           parts.push("To fix unmapped statuses, add jira.statusMap to .marvin/config.yaml:");
           parts.push("  jira:");
           parts.push("    statusMap:");
-          if (unmappedAction.length > 0) {
-            parts.push("      action:");
-            parts.push(`        # Map these: ${unmappedAction.join(", ")}`);
-            parts.push("        # <marvin-status>: [<jira-status>, ...]");
-          }
-          if (unmappedTask.length > 0) {
-            parts.push("      task:");
-            parts.push(`        # Map these: ${unmappedTask.join(", ")}`);
-            parts.push("        # <marvin-status>: [<jira-status>, ...]");
+          for (const s of allUnmapped) {
+            parts.push(`      "${s}": <marvin-status>`);
           }
+          parts.push("  # Supported marvin statuses: done, in-progress, review, ready, blocked, backlog, open");
         } else {
           parts.push("");
           parts.push("All statuses are mapped.");
         }
-        const usingConfig = statusMap?.action || statusMap?.task;
+        const usingConfig = statusMap.flat || statusMap.legacy;
         parts.push("");
         parts.push(usingConfig ? "Using status maps from .marvin/config.yaml." : "Using built-in default status maps (no jira.statusMap in config).");
         return {
@@ -32372,7 +32417,7 @@ async function jiraSyncCommand(artifactId, options = {}) {
     );
     return;
   }
-  const statusMap = project.config.jira?.statusMap;
+  const statusMap = normalizeStatusMap(project.config.jira?.statusMap);
   const label = artifactId ? `Checking ${artifactId} against Jira...` : "Checking all Jira-linked actions/tasks...";
   console.log(chalk20.dim(label));
   if (options.dryRun) {
@@ -32489,9 +32534,7 @@ async function jiraStatusesCommand(projectKey) {
     return;
   }
   console.log(chalk20.dim(`Fetching statuses from Jira project ${resolvedProjectKey}...`));
-  const statusMap = project.config.jira?.statusMap;
-  const actionMap = statusMap?.action ?? DEFAULT_ACTION_STATUS_MAP;
-  const taskMap = statusMap?.task ?? DEFAULT_TASK_STATUS_MAP;
+  const statusMap = normalizeStatusMap(project.config.jira?.statusMap);
   const email3 = jiraUserConfig?.email ?? process.env.JIRA_EMAIL;
   const apiToken = jiraUserConfig?.apiToken ?? process.env.JIRA_API_TOKEN;
   const auth = "Basic " + Buffer.from(`${email3}:${apiToken}`).toString("base64");
@@ -32515,14 +32558,28 @@ async function jiraStatusesCommand(projectKey) {
     statusCounts.set(s, (statusCounts.get(s) ?? 0) + 1);
   }
   const actionLookup = /* @__PURE__ */ new Map();
-  for (const [marvin, value] of Object.entries(actionMap)) {
-    const jiraStatuses = Array.isArray(value) ? value : value.default;
-    for (const js of jiraStatuses) actionLookup.set(js.toLowerCase(), marvin);
-  }
   const taskLookup = /* @__PURE__ */ new Map();
-  for (const [marvin, value] of Object.entries(taskMap)) {
-    const jiraStatuses = Array.isArray(value) ? value : value.default;
-    for (const js of jiraStatuses) taskLookup.set(js.toLowerCase(), marvin);
+  if (statusMap.flat) {
+    for (const [jiraStatus, value] of Object.entries(statusMap.flat)) {
+      const lower = jiraStatus.toLowerCase();
+      if (typeof value === "string") {
+        actionLookup.set(lower, value);
+        taskLookup.set(lower, value);
+      } else {
+        const label = value.inSprint ? `${value.default} / ${value.inSprint} (inSprint)` : value.default;
+        actionLookup.set(lower, label);
+        taskLookup.set(lower, label);
+      }
+    }
+  } else {
+    const actionMap = statusMap.legacy?.action ?? DEFAULT_ACTION_STATUS_MAP;
+    const taskMap = statusMap.legacy?.task ?? DEFAULT_TASK_STATUS_MAP;
+    for (const [marvin, jiraStatuses] of Object.entries(actionMap)) {
+      for (const js of jiraStatuses) actionLookup.set(js.toLowerCase(), marvin);
+    }
+    for (const [marvin, jiraStatuses] of Object.entries(taskMap)) {
+      for (const js of jiraStatuses) taskLookup.set(js.toLowerCase(), marvin);
+    }
   }
   console.log(
     `
@@ -32545,14 +32602,11 @@ Found ${chalk20.bold(String(statusCounts.size))} distinct statuses in ${chalk20.
     console.log(chalk20.yellow("\nSome statuses are unmapped. Add jira.statusMap to .marvin/config.yaml:"));
     console.log(chalk20.dim("  jira:"));
     console.log(chalk20.dim("    statusMap:"));
-    console.log(chalk20.dim("      action:"));
-    console.log(chalk20.dim('        <marvin-status>: ["<Jira Status>", ...]'));
-    console.log(chalk20.dim("      task:"));
-    console.log(chalk20.dim('        <marvin-status>: ["<Jira Status>", ...]'));
+    console.log(chalk20.dim('      "<Jira Status>": <marvin-status>'));
   } else {
     console.log(chalk20.green("\nAll statuses are mapped."));
   }
-  const usingConfig = statusMap?.action || statusMap?.task;
+  const usingConfig = statusMap.flat || statusMap.legacy;
   console.log(
     chalk20.dim(
       usingConfig ? "\nUsing status maps from .marvin/config.yaml." : "\nUsing built-in default status maps (no jira.statusMap in config)."
@@ -32587,7 +32641,7 @@ async function jiraDailyCommand(options) {
   const today = (/* @__PURE__ */ new Date()).toISOString().slice(0, 10);
   const fromDate = options.from ?? today;
   const toDate = options.to ?? fromDate;
-  const statusMap = proj.config.jira?.statusMap;
+  const statusMap = normalizeStatusMap(proj.config.jira?.statusMap);
   const rangeLabel = fromDate === toDate ? fromDate : `${fromDate} to ${toDate}`;
   console.log(
     chalk20.dim(`Fetching Jira daily summary for ${resolvedProjectKey} \u2014 ${rangeLabel}...`)
@@ -32704,7 +32758,7 @@ function createProgram() {
   const program = new Command();
   program.name("marvin").description(
     "AI-powered product development assistant with Product Owner, Delivery Manager, and Technical Lead personas"
-  ).version("0.5.16");
+  ).version("0.5.17");
   program.command("init").description("Initialize a new Marvin project in the current directory").action(async () => {
     await initCommand();
   });