feat(tui): add edit and delete collection functionality

Author: maniac-en

internal/tui/app/model.go

@@ -1,7 +1,10 @@
 package app
 
 import (
+	"context"
+
 	tea "github.com/charmbracelet/bubbletea"
+	"github.com/maniac-en/req/internal/log"
 	"github.com/maniac-en/req/internal/tui/views"
 )
 
@@ -14,12 +17,13 @@ const (
 )
 
 type Model struct {
-	ctx             *Context
-	mode            ViewMode
-	collectionsView views.CollectionsView
-	addCollectionView views.AddCollectionView
-	width           int
-	height          int
+	ctx                *Context
+	mode               ViewMode
+	collectionsView    views.CollectionsView
+	addCollectionView  views.AddCollectionView
+	editCollectionView views.EditCollectionView
+	width              int
+	height             int
 }
 
 func NewModel(ctx *Context) Model {
@@ -28,6 +32,7 @@ func NewModel(ctx *Context) Model {
 		mode:              CollectionsViewMode,
 		collectionsView:   views.NewCollectionsView(ctx.Collections),
 		addCollectionView: views.NewAddCollectionView(ctx.Collections),
+		// editCollectionView will be created on demand
 	}
 }
 
@@ -42,7 +47,7 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 	case tea.KeyMsg:
 		// Handle global keybinds only when not in filtering mode
 		isFiltering := m.mode == CollectionsViewMode && m.collectionsView.IsFiltering()
-		
+
 		if !isFiltering {
 			switch msg.String() {
 			case "ctrl+c", "q":
@@ -57,6 +62,30 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 					m.mode = AddCollectionViewMode
 					return m, nil
 				}
+			case "e":
+				if m.mode == CollectionsViewMode {
+					// Get selected collection and switch to edit mode
+					if selectedItem := m.collectionsView.GetSelectedItem(); selectedItem != nil {
+						m.mode = EditCollectionViewMode
+						m.editCollectionView = views.NewEditCollectionView(m.ctx.Collections, *selectedItem)
+						return m, nil
+					} else {
+						log.Error("issue getting currently selected collection")
+					}
+				}
+			case "x":
+				if m.mode == CollectionsViewMode {
+					// Delete selected collection
+					if selectedItem := m.collectionsView.GetSelectedItem(); selectedItem != nil {
+						return m, func() tea.Msg {
+							err := m.ctx.Collections.Delete(context.Background(), selectedItem.ID)
+							if err != nil {
+								return views.CollectionDeleteErrorMsg{Err: err}
+							}
+							return views.CollectionDeletedMsg{ID: selectedItem.ID}
+						}
+					}
+				}
 			}
 		}
 	case tea.WindowSizeMsg:
@@ -66,6 +95,16 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 		m.mode = CollectionsViewMode
 		// Reload collections to show any changes
 		return m, m.collectionsView.Init()
+	case views.EditCollectionMsg:
+		m.mode = EditCollectionViewMode
+		m.editCollectionView = views.NewEditCollectionView(m.ctx.Collections, msg.Collection)
+		return m, nil
+	case views.CollectionDeletedMsg:
+		// Collection deleted, reload collections view
+		return m, m.collectionsView.Init()
+	case views.CollectionDeleteErrorMsg:
+		// Delete failed, just continue
+		return m, nil
 	}
 
 	// Forward messages to the appropriate view
@@ -74,8 +113,10 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 		m.collectionsView, cmd = m.collectionsView.Update(msg)
 	case AddCollectionViewMode:
 		m.addCollectionView, cmd = m.addCollectionView.Update(msg)
+	case EditCollectionViewMode:
+		m.editCollectionView, cmd = m.editCollectionView.Update(msg)
 	}
-	
+
 	return m, cmd
 }
 
@@ -85,8 +126,9 @@ func (m Model) View() string {
 		return m.collectionsView.View()
 	case AddCollectionViewMode:
 		return m.addCollectionView.View()
+	case EditCollectionViewMode:
+		return m.editCollectionView.View()
 	default:
 		return m.collectionsView.View()
 	}
 }
-

internal/tui/views/add_collection.go

@@ -130,15 +130,15 @@ type CollectionUpdateErrorMsg struct {
 }
 
 type CollectionDeletedMsg struct {
-	id int64
+	ID int64
 }
 
 type CollectionDeleteErrorMsg struct {
-	err error
+	Err error
 }
 
 type BackToCollectionsMsg struct{}
 
 type EditCollectionMsg struct {
-	collection collections.CollectionEntity
+	Collection collections.CollectionEntity
 }

internal/tui/views/collections.go

@@ -11,22 +11,22 @@ import (
 )
 
 type CollectionsView struct {
-	layout         components.Layout
-	list           components.PaginatedList
+	layout             components.Layout
+	list               components.PaginatedList
 	collectionsManager *collections.CollectionsManager
-	width          int
-	height         int
-	initialized    bool
-	
+	width              int
+	height             int
+	initialized        bool
+
 	// Backend pagination state
-	currentPage    int
-	pageSize       int
-	pagination     crud.PaginationMetadata
+	currentPage int
+	pageSize    int
+	pagination  crud.PaginationMetadata
 }
 
 func NewCollectionsView(collectionsManager *collections.CollectionsManager) CollectionsView {
 	return CollectionsView{
-		layout:            components.NewLayout(),
+		layout:             components.NewLayout(),
 		collectionsManager: collectionsManager,
 	}
 }
@@ -72,43 +72,43 @@ func (v CollectionsView) Update(msg tea.Msg) (CollectionsView, tea.Cmd) {
 		v.width = msg.Width
 		v.height = msg.Height
 		v.layout.SetSize(v.width, v.height)
-		
+
 		// Only set list size if it's been initialized
 		if v.initialized {
 			contentHeight := v.height - 4 // Account for header/footer
 			v.list.SetSize(v.width-4, contentHeight)
 		}
-		
+
 	case collectionsLoaded:
 		items := make([]components.ListItem, len(msg.collections))
 		for i, collection := range msg.collections {
 			items[i] = components.NewCollectionItem(collection)
 		}
-		
+
 		// Update pagination state
 		v.currentPage = msg.currentPage
 		v.pageSize = msg.pageSize
 		v.pagination = msg.pagination
-		
+
 		// Create list with pagination info in title
 		title := fmt.Sprintf("Collections (Page %d/%d)", v.currentPage, v.pagination.TotalPages)
 		v.list = components.NewPaginatedList(items, title)
-		
+
 		if v.width > 0 && v.height > 0 {
 			contentHeight := v.height - 4
 			v.list.SetSize(v.width-4, contentHeight)
 		}
 		v.initialized = true
-		
+
 	case collectionsLoadError:
 		// Handle error - for now just mark as initialized
 		v.initialized = true
-		
+
 	case tea.KeyMsg:
 		if !v.initialized {
 			break
 		}
-		
+
 		// Handle pagination keys only when not filtering
 		if !v.list.IsFiltering() {
 			switch msg.String() {
@@ -130,23 +130,36 @@ func (v CollectionsView) Update(msg tea.Msg) (CollectionsView, tea.Cmd) {
 				return v, nil
 			}
 		}
-		
-		// Always forward keys to the list (handles filtering and navigation)
+
+		// Forward keys to the list (handles filtering and navigation)
 		v.list, cmd = v.list.Update(msg)
-		
+
 	default:
 		if v.initialized {
 			v.list, cmd = v.list.Update(msg)
 		}
 	}
-	
+
 	return v, cmd
 }
 
 func (v CollectionsView) IsFiltering() bool {
 	return v.initialized && v.list.IsFiltering()
 }
 
+func (v CollectionsView) GetSelectedItem() *collections.CollectionEntity {
+	if !v.initialized {
+		return nil
+	}
+	if selectedItem := v.list.SelectedItem(); selectedItem != nil {
+		if collectionItem, ok := selectedItem.(components.CollectionItem); ok {
+			collection := collectionItem.GetCollection()
+			return &collection
+		}
+	}
+	return nil
+}
+
 func (v CollectionsView) View() string {
 	if !v.initialized {
 		return v.layout.FullView(
@@ -157,20 +170,19 @@ func (v CollectionsView) View() string {
 	}
 
 	content := v.list.View()
-	
+
 	// Build instructions with pagination and filter info
-	instructions := "↑↓: navigate • /: filter • enter: edit • d: delete • q: quit"
+	instructions := "↑↓: navigate • /: filter • e: edit • x: delete • q: quit"
 	if !v.list.IsFiltering() {
-		instructions = "↑↓: navigate • a: add • /: filter • enter: edit • d: delete • q: quit"
+		instructions = "↑↓: navigate • a: add • /: filter • e: edit • x: delete • q: quit"
 	}
 	if v.pagination.TotalPages > 1 && !v.list.IsFiltering() {
 		instructions += fmt.Sprintf(" • p/n: prev/next page (%d/%d)", v.currentPage, v.pagination.TotalPages)
 	}
-	
+
 	return v.layout.FullView(
 		"Collections",
 		content,
 		instructions,
 	)
 }
-