feat(frontmatter): track status transitions and ensure created_at is set

Author: tikazyq

rust/leanspec-cli/tests/update.rs

@@ -44,6 +44,21 @@ fn test_update_status_transition() {
     let content = read_file(&cwd.join("specs").join("001-my-spec").join("README.md"));
     let fm = parse_frontmatter(&content);
     assert_eq!(fm.get("status").and_then(|v| v.as_str()), Some("complete"));
+
+    // Should record transitions for status changes
+    let transitions = fm.get("transitions");
+    assert!(transitions.is_some(), "expected transitions in frontmatter");
+    if let Some(serde_yaml::Value::Sequence(seq)) = transitions {
+        // planned -> in-progress -> complete
+        assert!(seq.len() >= 2, "expected at least 2 transitions, got {}", seq.len());
+        let last = seq.last().and_then(|v| v.as_mapping());
+        let last_status = last
+            .and_then(|m| m.get("status"))
+            .and_then(|v| v.as_str());
+        assert_eq!(last_status, Some("complete"));
+    } else {
+        panic!("transitions should be a YAML sequence");
+    }
 }
 
 #[test]

rust/leanspec-core/src/parsers/frontmatter.rs

@@ -272,6 +272,8 @@ impl FrontmatterParser {
         updates: &std::collections::HashMap<String, serde_yaml::Value>,
     ) -> Result<String, ParseError> {
         let (mut frontmatter, body) = self.parse(content)?;
+        let previous_status = frontmatter.status;
+        let now = Utc::now();
 
         // Apply updates
         for (key, value) in updates {
@@ -307,12 +309,25 @@ impl FrontmatterParser {
             }
         }
 
+        // Ensure created_at exists for velocity tracking
+        if frontmatter.created_at.is_none() {
+            frontmatter.created_at = Some(now);
+        }
+
         // Update timestamps
-        frontmatter.updated_at = Some(Utc::now());
+        frontmatter.updated_at = Some(now);
+
+        // Track status transitions
+        if frontmatter.status != previous_status {
+            frontmatter.transitions.push(StatusTransition {
+                status: frontmatter.status,
+                at: now,
+            });
+        }
 
         // Set completed_at if status changed to complete
         if frontmatter.status == SpecStatus::Complete && frontmatter.completed_at.is_none() {
-            frontmatter.completed_at = Some(Utc::now());
+            frontmatter.completed_at = Some(now);
         }
 
         Ok(self.stringify(&frontmatter, &body))