feat: implement plugin management menu and associated keybindings

Author: programmersd21

CHANGELOG.md

@@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.1.2]
+
+### Added
+- **Plugin Metadata Display**: Press `Enter` on a plugin in the menu to view full metadata including Name, Description, Author, and Version.
+- **Uninstall Confirmation**: Added safety confirmation dialog before uninstalling plugins with `u` key.
+
 ## [1.1.1]
 
 ### Added

README.md

@@ -175,6 +175,16 @@ Kairo features a **minimalist design system** optimized for clarity and focus.
 | `?` | ❓ Show help menu |
 | `q` | ❌ Quit |
 
+### Plugin Menu Shortcuts
+
+| Shortcut | Action |
+|----------|--------|
+| `enter` | 👁️ View plugin details |
+| `u` | 🗑️ Uninstall plugin |
+| `o` | 📂 Open plugins folder |
+| `r` | 🔄 Reload plugins |
+| `p` / `esc` | ❌ Close menu |
+
 ### View Shortcuts
 
 | Shortcut | View |

internal/app/model.go

@@ -82,7 +82,6 @@ const (
 	ModeHelp
 	ModeThemeMenu
 	ModePluginMenu
-	ModePluginUninstall
 	ModeTagFilter
 )
 
@@ -123,8 +122,6 @@ type Model struct {
 	all   []core.Task
 	tags  []string
 
-	uninstallPluginID string
-
 	statusText string
 	isErr      bool
 
@@ -314,11 +311,6 @@ func (m *Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 		}
 		return m, nil
 
-	case plugin_menu.UninstallConfirmMsg:
-		m.uninstallPluginID = x.ID
-		m.mode = ModePluginUninstall
-		return m, nil
-
 	case plugin_menu.UninstallMsg:
 		if m.plugHost != nil {
 			err := m.plugHost.DeletePlugin(x.ID)
@@ -333,6 +325,11 @@ func (m *Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 			m.rebuildViews()
 			m.rebuildPaletteIndex()
 		}
+		if m.mode == ModePluginMenu {
+			// Stay in plugin menu, refresh handled by SetPlugins above
+		} else {
+			m.mode = ModeList
+		}
 		return m, nil
 
 	case plugin_menu.OpenFolderMsg:
@@ -452,17 +449,6 @@ func (m *Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 	}
 
 	if km, ok := msg.(tea.KeyMsg); ok {
-		if m.mode == ModePluginUninstall {
-			switch km.String() {
-			case "y", "enter":
-				m.mode = ModePluginMenu
-				return m, func() tea.Msg { return plugin_menu.UninstallMsg{ID: m.uninstallPluginID} }
-			case "n", "esc":
-				m.mode = ModePluginMenu
-				return m, nil
-			}
-		}
-
 		if m.mode == ModeConfirmDelete {
 			switch km.String() {
 			case "y", "enter":
@@ -727,7 +713,7 @@ func (m *Model) View() string {
 		content = m.hlp.View()
 	case ModeThemeMenu:
 		content = m.tm.View()
-	case ModePluginMenu, ModePluginUninstall:
+	case ModePluginMenu:
 		content = m.renderMainUI()
 	default:
 		content = m.renderMainUI()
@@ -763,7 +749,7 @@ func (m *Model) renderMainUI() string {
 		body = m.list.View()
 	case ModeDetail:
 		body = m.det.View()
-	case ModePluginMenu, ModePluginUninstall:
+	case ModePluginMenu:
 		body = m.pm.View()
 	default:
 		body = m.list.View()
@@ -892,9 +878,7 @@ func (m *Model) renderFooter() string {
 	case ModeThemeMenu:
 		left = " " + m.s.Muted.Render("enter select • esc/q/"+fk(m.km.CycleTheme)+" close • ↑/↓ navigate")
 	case ModePluginMenu:
-		left = " " + m.s.Muted.Render("x uninstall • esc/q/"+fk(m.km.ManagePlugins)+" close • ↑/↓ navigate")
-	case ModePluginUninstall:
-		left = m.s.BadgeBad.Render(" UNINSTALL? ") + " " + m.s.Muted.Render("y/enter confirm • n/esc cancel")
+		left = " " + m.s.Muted.Render("enter detail • u uninstall • o open folder • r reload • p/"+fk(m.km.ManagePlugins)+" close • ↑/↓ navigate")
 	default:
 		left = " " + m.s.Muted.Render(
 			fk(m.km.Palette)+" "+styles.IconPalette+" • "+

internal/ui/plugin_menu/model.go

@@ -13,14 +13,14 @@ type UninstallMsg struct{ ID string }
 type OpenFolderMsg struct{}
 type ReloadMsg struct{}
 
-type UninstallConfirmMsg struct{ ID string }
-
 type Model struct {
 	styles  styles.Styles
 	width   int
 	height  int
 	plugins []plugins.PluginInfo
 	sel     int
+	detail  *plugins.PluginInfo // nil means in list mode
+	confirm string              // non-empty means showing confirm for uninstalling this ID
 }
 
 func New(s styles.Styles) Model {
@@ -44,6 +44,27 @@ func (m *Model) SetPlugins(ps []plugins.PluginInfo) {
 func (m Model) Update(msg tea.Msg) (Model, tea.Cmd) {
 	switch x := msg.(type) {
 	case tea.KeyMsg:
+		if m.confirm != "" {
+			switch x.String() {
+			case "y":
+				id := m.confirm
+				m.confirm = ""
+				return m, func() tea.Msg { return UninstallMsg{ID: id} }
+			case "n", "esc":
+				m.confirm = ""
+				return m, nil
+			}
+			return m, nil
+		}
+
+		if m.detail != nil {
+			switch x.String() {
+			case "esc", "q", "enter":
+				m.detail = nil
+			}
+			return m, nil
+		}
+
 		switch x.String() {
 		case "esc", "q", "p":
 			return m, func() tea.Msg { return CloseMsg{} }
@@ -55,14 +76,18 @@ func (m Model) Update(msg tea.Msg) (Model, tea.Cmd) {
 			if m.sel < len(m.plugins)-1 {
 				m.sel++
 			}
-		case "o": // Open folder
+		case "enter":
+			if m.sel >= 0 && m.sel < len(m.plugins) {
+				p := m.plugins[m.sel]
+				m.detail = &p
+			}
+		case "o":
 			return m, func() tea.Msg { return OpenFolderMsg{} }
-		case "r": // Reload
+		case "r":
 			return m, func() tea.Msg { return ReloadMsg{} }
-		case "x": // Uninstall key
+		case "u":
 			if m.sel >= 0 && m.sel < len(m.plugins) {
-				id := m.plugins[m.sel].ID
-				return m, func() tea.Msg { return UninstallConfirmMsg{ID: id} }
+				m.confirm = m.plugins[m.sel].ID
 			}
 		}
 	}
@@ -76,6 +101,35 @@ func (m Model) View() string {
 	}
 	cardW := min(80, w-4)
 
+	if m.confirm != "" {
+		return lipgloss.Place(w, m.height, lipgloss.Center, lipgloss.Center,
+			m.styles.Overlay.Width(cardW).Render(
+				lipgloss.JoinVertical(lipgloss.Center,
+					"Uninstall plugin?",
+					"",
+					m.styles.Accent.Render("[y] Yes  [n] No"),
+				),
+			),
+		)
+	}
+
+	if m.detail != nil {
+		p := m.detail
+		content := lipgloss.JoinVertical(lipgloss.Left,
+			m.styles.Title.Render(p.Name),
+			"",
+			"Version: "+p.Version,
+			"Author: "+p.Author,
+			"",
+			p.Description,
+			"",
+			m.styles.Muted.Render("Press Enter/Esc to return"),
+		)
+		return lipgloss.Place(w, m.height, lipgloss.Center, lipgloss.Center,
+			m.styles.Overlay.Width(cardW).Padding(2).Render(content),
+		)
+	}
+
 	var rows []string
 	if len(m.plugins) == 0 {
 		rows = append(rows, m.styles.Muted.Padding(0, 1).Render("No plugins."))