fix: always normalize journal day number (agent might output wrong day)

Author: Patel230

internal/evolution/git_test.go

@@ -408,9 +408,9 @@ func TestPersistJournalEntry_ZeroDay(t *testing.T) {
 	e.persistJournalEntry(entry, "0")
 
 	data, _ := os.ReadFile(filepath.Join(dir, "docs/JOURNAL.md"))
-	// day 0 means don't replace, so Day 99 should remain
-	if !strings.Contains(string(data), "Day 99") {
-		t.Errorf("should keep original day when day='0', got:\n%s", string(data))
+	// day 0 should normalize agent's wrong day to Day 0
+	if !strings.Contains(string(data), "## Day 0") {
+		t.Errorf("should normalize to Day 0, got:\n%s", string(data))
 	}
 }
 

internal/evolution/phases_communicate.go

@@ -119,10 +119,9 @@ func (e *Engine) persistJournalEntry(journalEntry string, day string) {
 	}
 
 	extracted = strings.TrimSpace(extracted)
-	if dayNum > 0 {
-		dayPattern := regexp.MustCompile(`^(#+)\s*Day\s*\d+`)
-		extracted = dayPattern.ReplaceAllString(extracted, fmt.Sprintf("## Day %d", dayNum))
-	}
+	// Always normalize day number — agent might output wrong day
+	dayPattern := regexp.MustCompile(`^(#+)\s*Day\s*\d+`)
+	extracted = dayPattern.ReplaceAllString(extracted, fmt.Sprintf("## Day %d", dayNum))
 
 	journalPath := filepath.Join(e.repoPath, "docs/JOURNAL.md")
 	_ = os.MkdirAll(filepath.Dir(journalPath), 0o755)