Add TUI Phase 2B: browse filters, dashboard view, and column updates

Add 4 new browse filters (priority cycle, project text input, my-items
toggle, sort cycle), two-line filter bar, replace EFFORT column with
color-coded STATUS, add POSTED BY for wide terminals, and a new "Me"
dashboard view with claimed/in-review/completed sections.

Key fixes: toggling Mine ON resets status to "all" to avoid empty results
when items aren't in "open" state. Project filter value stored as plain
string field decoupled from textinput state to survive value-receiver
copies across bubbletea update cycles.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: julianknutsen

internal/commons/commons_test.go

@@ -241,6 +241,110 @@ func TestFormatTagsJSON(t *testing.T) {
 	}
 }
 
+func TestBuildBrowseQuery_MyItems(t *testing.T) {
+	t.Parallel()
+	f := BrowseFilter{Priority: -1, MyItems: "my-rig"}
+	q := BuildBrowseQuery(f)
+	if !strings.Contains(q, "(posted_by = 'my-rig' OR claimed_by = 'my-rig')") {
+		t.Errorf("MyItems should produce OR clause, got:\n%s", q)
+	}
+	// MyItems should suppress separate PostedBy/ClaimedBy.
+	if strings.Contains(q, "AND posted_by =") || strings.Contains(q, "AND claimed_by =") {
+		t.Error("MyItems should suppress separate posted_by/claimed_by conditions")
+	}
+}
+
+func TestBuildBrowseQuery_MyItems_OverridesPostedClaimedBy(t *testing.T) {
+	t.Parallel()
+	f := BrowseFilter{Priority: -1, MyItems: "my-rig", PostedBy: "other", ClaimedBy: "other"}
+	q := BuildBrowseQuery(f)
+	if !strings.Contains(q, "(posted_by = 'my-rig' OR claimed_by = 'my-rig')") {
+		t.Errorf("MyItems should take priority, got:\n%s", q)
+	}
+	if strings.Contains(q, "posted_by = 'other'") {
+		t.Error("PostedBy should be ignored when MyItems is set")
+	}
+}
+
+func TestBuildBrowseQuery_SortPriority(t *testing.T) {
+	t.Parallel()
+	f := BrowseFilter{Priority: -1, Sort: SortPriority}
+	q := BuildBrowseQuery(f)
+	if !strings.Contains(q, "ORDER BY priority ASC, created_at DESC") {
+		t.Errorf("SortPriority should order by priority, got:\n%s", q)
+	}
+}
+
+func TestBuildBrowseQuery_SortNewest(t *testing.T) {
+	t.Parallel()
+	f := BrowseFilter{Priority: -1, Sort: SortNewest}
+	q := BuildBrowseQuery(f)
+	if !strings.Contains(q, "ORDER BY created_at DESC") {
+		t.Errorf("SortNewest should order by created_at DESC, got:\n%s", q)
+	}
+	if strings.Contains(q, "priority ASC") {
+		t.Error("SortNewest should not include priority ordering")
+	}
+}
+
+func TestBuildBrowseQuery_SortAlpha(t *testing.T) {
+	t.Parallel()
+	f := BrowseFilter{Priority: -1, Sort: SortAlpha}
+	q := BuildBrowseQuery(f)
+	if !strings.Contains(q, "ORDER BY title ASC") {
+		t.Errorf("SortAlpha should order by title ASC, got:\n%s", q)
+	}
+}
+
+func TestBuildBrowseQuery_PriorityFilter(t *testing.T) {
+	t.Parallel()
+	f := BrowseFilter{Priority: 1}
+	q := BuildBrowseQuery(f)
+	if !strings.Contains(q, "priority = 1") {
+		t.Errorf("Priority=1 should filter by priority, got:\n%s", q)
+	}
+}
+
+func TestValidPriorities(t *testing.T) {
+	t.Parallel()
+	got := ValidPriorities()
+	want := []int{-1, 0, 1, 2, 3, 4}
+	if len(got) != len(want) {
+		t.Fatalf("len = %d, want %d", len(got), len(want))
+	}
+	for i := range want {
+		if got[i] != want[i] {
+			t.Errorf("index %d: got %d, want %d", i, got[i], want[i])
+		}
+	}
+}
+
+func TestPriorityLabel(t *testing.T) {
+	t.Parallel()
+	if got := PriorityLabel(-1); got != "all" {
+		t.Errorf("PriorityLabel(-1) = %q, want %q", got, "all")
+	}
+	if got := PriorityLabel(0); got != "P0" {
+		t.Errorf("PriorityLabel(0) = %q, want %q", got, "P0")
+	}
+	if got := PriorityLabel(3); got != "P3" {
+		t.Errorf("PriorityLabel(3) = %q, want %q", got, "P3")
+	}
+}
+
+func TestSortLabel(t *testing.T) {
+	t.Parallel()
+	if got := SortLabel(SortPriority); got != "priority" {
+		t.Errorf("SortLabel(SortPriority) = %q", got)
+	}
+	if got := SortLabel(SortNewest); got != "newest" {
+		t.Errorf("SortLabel(SortNewest) = %q", got)
+	}
+	if got := SortLabel(SortAlpha); got != "alpha" {
+		t.Errorf("SortLabel(SortAlpha) = %q", got)
+	}
+}
+
 func TestFormatTagsJSON_RoundTrip(t *testing.T) {
 	t.Parallel()
 	tags := []string{"it's", "go", `say "hi"`}

internal/commons/queries.go

@@ -5,6 +5,35 @@ import (
 	"strings"
 )
 
+// SortOrder defines browse result ordering.
+type SortOrder int
+
+// Sort order constants for browse results.
+const (
+	SortPriority SortOrder = iota
+	SortNewest
+	SortAlpha
+)
+
+// ValidSortOrders returns all sort modes.
+func ValidSortOrders() []SortOrder {
+	return []SortOrder{SortPriority, SortNewest, SortAlpha}
+}
+
+// SortLabel returns a human-readable label for a sort order.
+func SortLabel(s SortOrder) string {
+	switch s {
+	case SortPriority:
+		return "priority"
+	case SortNewest:
+		return "newest"
+	case SortAlpha:
+		return "alpha"
+	default:
+		return "priority"
+	}
+}
+
 // BrowseFilter holds filter parameters for querying the wanted board.
 type BrowseFilter struct {
 	Status    string
@@ -15,6 +44,8 @@ type BrowseFilter struct {
 	PostedBy  string
 	ClaimedBy string
 	Search    string
+	MyItems   string    // rig handle for OR filter (posted_by OR claimed_by); empty = disabled
+	Sort      SortOrder // result ordering
 }
 
 // WantedSummary holds the columns returned by BrowseWanted.
@@ -56,11 +87,16 @@ func BuildBrowseQuery(f BrowseFilter) string {
 	if f.Priority >= 0 {
 		conditions = append(conditions, fmt.Sprintf("priority = %d", f.Priority))
 	}
-	if f.PostedBy != "" {
-		conditions = append(conditions, fmt.Sprintf("posted_by = '%s'", EscapeSQL(f.PostedBy)))
-	}
-	if f.ClaimedBy != "" {
-		conditions = append(conditions, fmt.Sprintf("claimed_by = '%s'", EscapeSQL(f.ClaimedBy)))
+	if f.MyItems != "" {
+		escaped := EscapeSQL(f.MyItems)
+		conditions = append(conditions, fmt.Sprintf("(posted_by = '%s' OR claimed_by = '%s')", escaped, escaped))
+	} else {
+		if f.PostedBy != "" {
+			conditions = append(conditions, fmt.Sprintf("posted_by = '%s'", EscapeSQL(f.PostedBy)))
+		}
+		if f.ClaimedBy != "" {
+			conditions = append(conditions, fmt.Sprintf("claimed_by = '%s'", EscapeSQL(f.ClaimedBy)))
+		}
 	}
 	if f.Search != "" {
 		conditions = append(conditions, fmt.Sprintf("title LIKE '%%%s%%'", EscapeSQL(f.Search)))
@@ -70,7 +106,15 @@ func BuildBrowseQuery(f BrowseFilter) string {
 	if len(conditions) > 0 {
 		query += " WHERE " + strings.Join(conditions, " AND ")
 	}
-	query += " ORDER BY priority ASC, created_at DESC"
+
+	switch f.Sort {
+	case SortNewest:
+		query += " ORDER BY created_at DESC"
+	case SortAlpha:
+		query += " ORDER BY title ASC"
+	default:
+		query += " ORDER BY priority ASC, created_at DESC"
+	}
 	limit := f.Limit
 	if limit <= 0 {
 		limit = 50
@@ -205,6 +249,65 @@ func TypeLabel(typ string) string {
 	return typ
 }
 
+// ValidPriorities returns the browse filter priority cycle values.
+// -1 means "all" (unfiltered).
+func ValidPriorities() []int {
+	return []int{-1, 0, 1, 2, 3, 4}
+}
+
+// PriorityLabel returns a human-readable label for a priority filter value.
+func PriorityLabel(pri int) string {
+	if pri < 0 {
+		return "all"
+	}
+	return fmt.Sprintf("P%d", pri)
+}
+
+// DashboardData holds the sections for the "me" dashboard view.
+type DashboardData struct {
+	Claimed   []WantedSummary // status=claimed, claimed_by=me
+	InReview  []WantedSummary // status=in_review, posted_by=me
+	Completed []WantedSummary // status=completed, claimed_by=me, limit 5
+}
+
+// QueryMyDashboard fetches personal dashboard data for the given handle.
+func QueryMyDashboard(dbDir, handle string) (*DashboardData, error) {
+	escaped := EscapeSQL(handle)
+	data := &DashboardData{}
+
+	// Claimed items.
+	claimedQ := fmt.Sprintf(
+		"SELECT id, title, COALESCE(project,'') as project, COALESCE(type,'') as type, priority, COALESCE(posted_by,'') as posted_by, status, COALESCE(effort_level,'medium') as effort_level FROM wanted WHERE status = 'claimed' AND claimed_by = '%s' ORDER BY priority ASC, created_at DESC LIMIT 50",
+		escaped)
+	csv, err := DoltSQLQuery(dbDir, claimedQ)
+	if err != nil {
+		return nil, fmt.Errorf("dashboard claimed: %w", err)
+	}
+	data.Claimed = parseWantedSummaries(csv)
+
+	// In-review items (posted by me, awaiting my review).
+	reviewQ := fmt.Sprintf(
+		"SELECT id, title, COALESCE(project,'') as project, COALESCE(type,'') as type, priority, COALESCE(posted_by,'') as posted_by, status, COALESCE(effort_level,'medium') as effort_level FROM wanted WHERE status = 'in_review' AND posted_by = '%s' ORDER BY priority ASC, created_at DESC LIMIT 50",
+		escaped)
+	csv, err = DoltSQLQuery(dbDir, reviewQ)
+	if err != nil {
+		return nil, fmt.Errorf("dashboard in_review: %w", err)
+	}
+	data.InReview = parseWantedSummaries(csv)
+
+	// Recent completions.
+	completedQ := fmt.Sprintf(
+		"SELECT id, title, COALESCE(project,'') as project, COALESCE(type,'') as type, priority, COALESCE(posted_by,'') as posted_by, status, COALESCE(effort_level,'medium') as effort_level FROM wanted WHERE status = 'completed' AND claimed_by = '%s' ORDER BY created_at DESC LIMIT 5",
+		escaped)
+	csv, err = DoltSQLQuery(dbDir, completedQ)
+	if err != nil {
+		return nil, fmt.Errorf("dashboard completed: %w", err)
+	}
+	data.Completed = parseWantedSummaries(csv)
+
+	return data, nil
+}
+
 // BrowseWantedBranchAware wraps BrowseWanted with branch overlay in PR mode.
 func BrowseWantedBranchAware(dbDir, mode, rigHandle string, f BrowseFilter) ([]WantedSummary, error) {
 	items, err := BrowseWanted(dbDir, f)