Merge pull request #143 from hdresearch/ty/remove-history

remove history

Author: AlephNotation

cmd/history.go

@@ -13,7 +13,6 @@ type keyMap struct {
 	Resume  key.Binding
 	Kill    key.Binding
 	Commit  key.Binding
-	History key.Binding
 	Tree    key.Binding
 }
 
@@ -27,21 +26,19 @@ func defaultKeys() keyMap {
 		Resume:  key.NewBinding(key.WithKeys("r"), key.WithHelp("r", "resume VM")),
 		Kill:    key.NewBinding(key.WithKeys("k"), key.WithHelp("k", "delete")),
 		Commit:  key.NewBinding(key.WithKeys("c"), key.WithHelp("c", "commit VM")),
-		History: key.NewBinding(key.WithKeys("h"), key.WithHelp("h", "history")),
 		Tree:    key.NewBinding(key.WithKeys("t"), key.WithHelp("t", "tree")),
 	}
 }
 
 // Implement help.KeyMap interface for bubbles/help
 func (k keyMap) ShortHelp() []key.Binding {
-	return []key.Binding{k.Connect, k.Branch, k.Pause, k.Resume, k.History, k.Tree, k.Switch, k.Quit}
+	return []key.Binding{k.Connect, k.Branch, k.Pause, k.Resume, k.Tree, k.Switch, k.Quit}
 }
 
 func (k keyMap) FullHelp() [][]key.Binding {
 	return [][]key.Binding{
 		{k.Connect, k.Branch, k.Commit},
 		{k.Pause, k.Resume, k.Kill},
-		{k.History, k.Tree},
-		{k.Switch, k.Quit},
+		{k.Tree, k.Switch, k.Quit},
 	}
 }

internal/handlers/history.go

@@ -13,7 +13,6 @@ import (
 	"github.com/hdresearch/vers-cli/internal/app"
 	"github.com/hdresearch/vers-cli/internal/handlers"
 	delsvc "github.com/hdresearch/vers-cli/internal/services/deletion"
-	histSvc "github.com/hdresearch/vers-cli/internal/services/history"
 	svcstatus "github.com/hdresearch/vers-cli/internal/services/status"
 	vmSvc "github.com/hdresearch/vers-cli/internal/services/vm"
 	sshutil "github.com/hdresearch/vers-cli/internal/ssh"
@@ -46,10 +45,6 @@ type actionCompletedMsg struct {
 	text string
 	err  error
 }
-type historyLoadedMsg struct {
-	lines []string
-	err   error
-}
 
 type Model struct {
 	app *app.App
@@ -208,27 +203,6 @@ func (m Model) selectedVMID() (string, bool) {
 	return "", false
 }
 
-func loadHistoryCmd(m Model, vmID string) tea.Cmd {
-	return func() tea.Msg {
-		ctx, cancel := context.WithTimeout(context.Background(), m.app.Timeouts.APIMedium)
-		defer cancel()
-		commits, err := histSvc.GetCommits(ctx, m.app.Client, vmID)
-		if err != nil {
-			return historyLoadedMsg{err: err}
-		}
-		lines := make([]string, 0, len(commits))
-		for _, c := range commits {
-			line := c.ID
-			if c.Alias != "" {
-				line = c.Alias + " (" + c.ID + ")"
-			}
-			line += " | " + c.Author
-			lines = append(lines, line)
-		}
-		return historyLoadedMsg{lines: lines}
-	}
-}
-
 func doCommitCmd(m Model, vmID string, tagCSV string) tea.Cmd {
 	return func() tea.Msg {
 		// Split and trim tags

internal/presenters/history_presenter.go

@@ -32,18 +32,6 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 		}
 		m.loading = true
 		return m, refreshVMsCmd(m)
-	case historyLoadedMsg:
-		m.status = ""
-		if msg.err != nil {
-			m.status = "History error: " + msg.err.Error()
-			m.modalActive = false
-			return m, nil
-		}
-		m.modalText = msg.lines
-		m.modalActive = true
-		m.modalKind = "text"
-		m.modalPrompt = "History:"
-		return m, nil
 
 	case tea.KeyMsg:
 		switch msg.String() {
@@ -119,14 +107,6 @@ func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
 					}
 					return m, nil
 				}
-			case "h":
-				if it, ok := m.vms.SelectedItem().(vmItem); ok {
-					m.modalActive = true
-					m.modalKind = "text"
-					m.modalPrompt = "History:"
-					m.status = "Loading history..."
-					return m, loadHistoryCmd(m, it.ID)
-				}
 			}
 			var cmd tea.Cmd
 			m.vms, cmd = m.vms.Update(msg)