feat: Track created and updated timestamps on issues

Author: mybuddymichael

format_issue.go

@@ -4,6 +4,7 @@ import (
 	"fmt"
 	"io"
 	"strings"
+	"time"
 )
 
 // PrintIssueDetails prints full issue details including ID, Title, Status,
@@ -14,6 +15,12 @@ func PrintIssueDetails(w io.Writer, issue *Issue, store *Store) error {
 	fmt.Fprintf(&b, "\033[1m\033[38;5;5mID\033[0m      %s\n", store.FormatID(issue.ID))
 	fmt.Fprintf(&b, "\033[1m\033[38;5;5mTitle\033[0m   %s\n", issue.Title)
 	fmt.Fprintf(&b, "\033[1m\033[38;5;5mStatus\033[0m  %s\n", issue.Status)
+	if !issue.CreatedAt.IsZero() {
+		fmt.Fprintf(&b, "\033[1m\033[38;5;5mCreated\033[0m %s\n", issue.CreatedAt.Format(time.DateTime))
+	}
+	if !issue.UpdatedAt.IsZero() {
+		fmt.Fprintf(&b, "\033[1m\033[38;5;5mUpdated\033[0m %s\n", issue.UpdatedAt.Format(time.DateTime))
+	}
 	if len(issue.DependsOn) > 0 || len(issue.Blocks) > 0 || len(issue.Comments) > 0 {
 		fmt.Fprintln(&b)
 	}

format_issue_test.go

@@ -358,9 +358,13 @@ func TestPrintIssueDetails_Whitespace(t *testing.T) {
 		t.Errorf("expected output to start with newline")
 	}
 
-	// Should have blank line between Status and Depends on
-	if !strings.Contains(stripped, "Status  open\n\nDepends on") {
-		t.Errorf("expected blank line between Status and Depends on, got: %s", stripped)
+	// Should have timestamps followed by blank line before Depends on
+	if !strings.Contains(stripped, "Updated") {
+		t.Errorf("expected Updated timestamp, got: %s", stripped)
+	}
+	// Check for blank line before Depends on (after timestamps)
+	if !strings.Contains(stripped, "\n\nDepends on") {
+		t.Errorf("expected blank line before Depends on, got: %s", stripped)
 	}
 
 	// Should have blank line between Depends on and Comments

store_core.go

@@ -2,6 +2,7 @@ package main
 
 import (
 	"os"
+	"time"
 
 	"github.com/goccy/go-yaml"
 )
@@ -14,12 +15,14 @@ type Store struct {
 
 // Issue represents a single issue
 type Issue struct {
-	ID        string   `yaml:"id"`
-	Title     string   `yaml:"title"`
-	Status    string   `yaml:"status"`
-	DependsOn []string `yaml:"depends_on,omitempty"`
-	Blocks    []string `yaml:"blocks,omitempty"`
-	Comments  []string `yaml:"comments,omitempty"`
+	ID        string    `yaml:"id"`
+	Title     string    `yaml:"title"`
+	Status    string    `yaml:"status"`
+	CreatedAt time.Time `yaml:"created_at"`
+	UpdatedAt time.Time `yaml:"updated_at"`
+	DependsOn []string  `yaml:"depends_on,omitempty"`
+	Blocks    []string  `yaml:"blocks,omitempty"`
+	Comments  []string  `yaml:"comments,omitempty"`
 }
 
 // NewStore creates a new store with defaults
@@ -59,3 +62,11 @@ func (s *Store) Save(filePath string) error {
 
 	return os.WriteFile(filePath, data, 0o600)
 }
+
+// touch updates the UpdatedAt timestamp for one or more issues
+func (s *Store) touch(issues ...*Issue) {
+	now := time.Now()
+	for _, issue := range issues {
+		issue.UpdatedAt = now
+	}
+}

store_core_test.go

@@ -181,3 +181,55 @@ func TestStoreSaveOrderSorted(t *testing.T) {
 		t.Logf("Content:\n%s", contentStr)
 	}
 }
+
+func TestStoreLoadOldYAML_ZeroTimestamps(t *testing.T) {
+	// Test that old YAML files (without timestamps) load with zero values
+	store := NewStore()
+	issue := &Issue{
+		ID:     "test-123",
+		Title:  "Old issue",
+		Status: "open",
+		// CreatedAt and UpdatedAt not set - simulate old YAML
+	}
+	store.Issues[issue.ID] = issue
+
+	if !issue.CreatedAt.IsZero() {
+		t.Error("Old issue CreatedAt should be zero")
+	}
+	if !issue.UpdatedAt.IsZero() {
+		t.Error("Old issue UpdatedAt should be zero")
+	}
+}
+
+func TestStoreLoadSave_PreservesTimestamps(t *testing.T) {
+	store := NewStore()
+	issue, _ := store.AddIssue("New issue")
+
+	// Save and reload
+	tmpFile := filepath.Join(t.TempDir(), "test.yaml")
+	err := store.Save(tmpFile)
+	if err != nil {
+		t.Fatalf("Save() failed: %v", err)
+	}
+
+	loaded, err := LoadStore(tmpFile)
+	if err != nil {
+		t.Fatalf("LoadStore() failed: %v", err)
+	}
+
+	loadedIssue := loaded.Issues[issue.ID]
+	if loadedIssue.CreatedAt.IsZero() {
+		t.Error("Loaded issue CreatedAt should not be zero")
+	}
+	if loadedIssue.UpdatedAt.IsZero() {
+		t.Error("Loaded issue UpdatedAt should not be zero")
+	}
+
+	// Timestamps should match original (within precision)
+	if !loadedIssue.CreatedAt.Equal(issue.CreatedAt) {
+		t.Error("CreatedAt should survive round-trip")
+	}
+	if !loadedIssue.UpdatedAt.Equal(issue.UpdatedAt) {
+		t.Error("UpdatedAt should survive round-trip")
+	}
+}

store_crud.go

@@ -3,6 +3,7 @@ package main
 import (
 	"fmt"
 	"sort"
+	"time"
 )
 
 // AddIssue creates a new issue with a unique ID and adds it to the store
@@ -23,10 +24,13 @@ func (s *Store) AddIssue(title string) (*Issue, error) {
 		}
 	}
 
+	now := time.Now()
 	issue := &Issue{
-		ID:     id,
-		Title:  title,
-		Status: "open",
+		ID:        id,
+		Title:     title,
+		Status:    "open",
+		CreatedAt: now,
+		UpdatedAt: now,
 	}
 
 	s.Issues[id] = issue
@@ -112,5 +116,6 @@ func (s *Store) UpdateIssueTitle(id, title string) error {
 		return err
 	}
 	issue.Title = title
+	s.touch(issue)
 	return nil
 }

store_crud_test.go

@@ -3,6 +3,7 @@ package main
 import (
 	"strings"
 	"testing"
+	"time"
 )
 
 func TestStoreAddIssue(t *testing.T) {
@@ -198,3 +199,58 @@ func TestStoreResolveIssueID_EmptyPrefix(t *testing.T) {
 		t.Errorf("expected 'abc123', got '%s'", id)
 	}
 }
+
+func TestStoreAddIssue_Timestamps(t *testing.T) {
+	store := NewStore()
+
+	before := time.Now()
+	issue, err := store.AddIssue("Test issue")
+	after := time.Now()
+
+	if err != nil {
+		t.Fatalf("AddIssue() failed: %v", err)
+	}
+
+	// CreatedAt should be set
+	if issue.CreatedAt.IsZero() {
+		t.Error("CreatedAt should not be zero")
+	}
+
+	// UpdatedAt should be set
+	if issue.UpdatedAt.IsZero() {
+		t.Error("UpdatedAt should not be zero")
+	}
+
+	// Both should equal on creation
+	if !issue.CreatedAt.Equal(issue.UpdatedAt) {
+		t.Errorf("CreatedAt and UpdatedAt should be equal on creation, got CreatedAt=%v, UpdatedAt=%v",
+			issue.CreatedAt, issue.UpdatedAt)
+	}
+
+	// Timestamps should be within test execution window
+	if issue.CreatedAt.Before(before) || issue.CreatedAt.After(after) {
+		t.Errorf("CreatedAt should be between %v and %v, got %v", before, after, issue.CreatedAt)
+	}
+}
+
+func TestStoreUpdateIssueTitle_UpdatesTimestamp(t *testing.T) {
+	store := NewStore()
+	issue, _ := store.AddIssue("Original")
+
+	originalCreated := issue.CreatedAt
+	originalUpdated := issue.UpdatedAt
+	time.Sleep(10 * time.Millisecond) // Ensure timestamp differs
+
+	err := store.UpdateIssueTitle(issue.ID, "New title")
+	if err != nil {
+		t.Fatalf("UpdateIssueTitle() failed: %v", err)
+	}
+
+	if !issue.UpdatedAt.After(originalUpdated) {
+		t.Errorf("UpdatedAt should be after original, got original=%v, new=%v", originalUpdated, issue.UpdatedAt)
+	}
+
+	if !issue.CreatedAt.Equal(originalCreated) {
+		t.Error("CreatedAt should not change on update")
+	}
+}

store_lifecycle.go

@@ -9,6 +9,7 @@ func (s *Store) AddComment(id, comment string) error {
 		return err
 	}
 	issue.Comments = append(issue.Comments, comment)
+	s.touch(issue)
 	return nil
 }
 
@@ -22,6 +23,7 @@ func (s *Store) CloseIssue(id, reason string) error {
 	if reason != "" {
 		issue.Comments = append(issue.Comments, fmt.Sprintf("Closed with reason: %s", reason))
 	}
+	s.touch(issue)
 	return nil
 }
 
@@ -32,6 +34,7 @@ func (s *Store) ReopenIssue(id string) error {
 		return fmt.Errorf("issue not found: %s", id)
 	}
 	issue.Status = "open"
+	s.touch(issue)
 	return nil
 }
 

store_lifecycle_test.go

@@ -2,6 +2,7 @@ package main
 
 import (
 	"testing"
+	"time"
 )
 
 func TestStoreCloseIssue(t *testing.T) {
@@ -165,3 +166,52 @@ func TestStoreDeleteIssue_NotFound(t *testing.T) {
 		t.Errorf("expected error '%s', got '%s'", expectedErr, err.Error())
 	}
 }
+
+func TestStoreAddComment_UpdatesTimestamp(t *testing.T) {
+	store := NewStore()
+	issue, _ := store.AddIssue("Test")
+	originalUpdated := issue.UpdatedAt
+	time.Sleep(10 * time.Millisecond)
+
+	err := store.AddComment(issue.ID, "Comment")
+	if err != nil {
+		t.Fatalf("AddComment() failed: %v", err)
+	}
+
+	if !issue.UpdatedAt.After(originalUpdated) {
+		t.Errorf("UpdatedAt should be after original")
+	}
+}
+
+func TestStoreCloseIssue_UpdatesTimestamp(t *testing.T) {
+	store := NewStore()
+	issue, _ := store.AddIssue("Test")
+	originalUpdated := issue.UpdatedAt
+	time.Sleep(10 * time.Millisecond)
+
+	err := store.CloseIssue(issue.ID, "")
+	if err != nil {
+		t.Fatalf("CloseIssue() failed: %v", err)
+	}
+
+	if !issue.UpdatedAt.After(originalUpdated) {
+		t.Errorf("UpdatedAt should be after original")
+	}
+}
+
+func TestStoreReopenIssue_UpdatesTimestamp(t *testing.T) {
+	store := NewStore()
+	issue, _ := store.AddIssue("Test")
+	_ = store.CloseIssue(issue.ID, "")
+	originalUpdated := issue.UpdatedAt
+	time.Sleep(10 * time.Millisecond)
+
+	err := store.ReopenIssue(issue.ID)
+	if err != nil {
+		t.Fatalf("ReopenIssue() failed: %v", err)
+	}
+
+	if !issue.UpdatedAt.After(originalUpdated) {
+		t.Errorf("UpdatedAt should be after original")
+	}
+}

store_relationships.go

@@ -14,6 +14,7 @@ func (s *Store) AddDependency(issueID, dependsOnID string) error {
 
 	issue.DependsOn = append(issue.DependsOn, dependsOnID)
 	blocker.Blocks = append(blocker.Blocks, issueID)
+	s.touch(issue, blocker)
 	return nil
 }
 
@@ -31,6 +32,7 @@ func (s *Store) AddBlocker(issueID, blockedID string) error {
 
 	issue.Blocks = append(issue.Blocks, blockedID)
 	blocked.DependsOn = append(blocked.DependsOn, issueID)
+	s.touch(issue, blocked)
 	return nil
 }
 
@@ -64,6 +66,7 @@ func (s *Store) RemoveDependency(issueID, dependsOnID string) error {
 	}
 	blocker.Blocks = newBlocks
 
+	s.touch(issue, blocker)
 	return nil
 }
 
@@ -97,5 +100,6 @@ func (s *Store) RemoveBlocker(issueID, blockedID string) error {
 	}
 	blocked.DependsOn = newDeps
 
+	s.touch(issue, blocked)
 	return nil
 }