diff options
| author | Dax <[email protected]> | 2025-07-23 20:30:46 -0400 |
|---|---|---|
| committer | GitHub <[email protected]> | 2025-07-23 20:30:46 -0400 |
| commit | 96866e52ce4b28d8f380d84722f411e786781f51 (patch) | |
| tree | fbe8df4e706510b3158d1659d8aa54b0c533c849 /packages/tui | |
| parent | 507c975e929d1d8bd58ae26e7bbcf9537daaf5b6 (diff) | |
| download | opencode-96866e52ce4b28d8f380d84722f411e786781f51.tar.gz opencode-96866e52ce4b28d8f380d84722f411e786781f51.zip | |
basic undo feature (#1268)
Co-authored-by: adamdotdevin <[email protected]>
Co-authored-by: Jay V <[email protected]>
Co-authored-by: Aiden Cline <[email protected]>
Co-authored-by: Andrew Joslin <[email protected]>
Co-authored-by: GitHub Action <[email protected]>
Co-authored-by: Tobias Walle <[email protected]>
Diffstat (limited to 'packages/tui')
| -rw-r--r-- | packages/tui/internal/app/app.go | 23 | ||||
| -rw-r--r-- | packages/tui/internal/app/prompt.go | 68 | ||||
| -rw-r--r-- | packages/tui/internal/commands/command.go | 17 | ||||
| -rw-r--r-- | packages/tui/internal/components/chat/editor.go | 50 | ||||
| -rw-r--r-- | packages/tui/internal/components/chat/messages.go | 240 | ||||
| -rw-r--r-- | packages/tui/internal/tui/tui.go | 13 | ||||
| -rw-r--r-- | packages/tui/sdk/.stats.yml | 8 | ||||
| -rw-r--r-- | packages/tui/sdk/api.md | 2 | ||||
| -rw-r--r-- | packages/tui/sdk/config.go | 8 | ||||
| -rw-r--r-- | packages/tui/sdk/session.go | 37 | ||||
| -rw-r--r-- | packages/tui/sdk/session_test.go | 51 |
11 files changed, 485 insertions, 32 deletions
diff --git a/packages/tui/internal/app/app.go b/packages/tui/internal/app/app.go index 0556750a0..df4e209cf 100644 --- a/packages/tui/internal/app/app.go +++ b/packages/tui/internal/app/app.go @@ -52,6 +52,13 @@ type SessionCreatedMsg = struct { Session *opencode.Session } type SessionSelectedMsg = *opencode.Session +type MessageRevertedMsg struct { + Session opencode.Session + Message Message +} +type SessionUnrevertedMsg struct { + Session opencode.Session +} type SessionLoadedMsg struct{} type ModelSelectedMsg struct { Provider opencode.Provider @@ -174,6 +181,16 @@ func New( return app, nil } +func (a *App) Keybind(commandName commands.CommandName) string { + command := a.Commands[commandName] + kb := command.Keybindings[0] + key := kb.Key + if kb.RequiresLeader { + key = a.Config.Keybinds.Leader + " " + kb.Key + } + return key +} + func (a *App) Key(commandName commands.CommandName) string { t := theme.CurrentTheme() base := styles.NewStyle().Background(t.Background()).Foreground(t.Text()).Bold(true).Render @@ -183,11 +200,7 @@ func (a *App) Key(commandName commands.CommandName) string { Faint(true). Render command := a.Commands[commandName] - kb := command.Keybindings[0] - key := kb.Key - if kb.RequiresLeader { - key = a.Config.Keybinds.Leader + " " + kb.Key - } + key := a.Keybind(commandName) return base(key) + muted(" "+command.Description) } diff --git a/packages/tui/internal/app/prompt.go b/packages/tui/internal/app/prompt.go index 9176b7df5..282ced704 100644 --- a/packages/tui/internal/app/prompt.go +++ b/packages/tui/internal/app/prompt.go @@ -1,6 +1,7 @@ package app import ( + "errors" "time" "github.com/sst/opencode-sdk-go" @@ -109,6 +110,73 @@ func (p Prompt) ToMessage( } } +func (m Message) ToPrompt() (*Prompt, error) { + switch m.Info.(type) { + case opencode.UserMessage: + text := "" + attachments := []*attachment.Attachment{} + for _, part := range m.Parts { + switch p := part.(type) { + case opencode.TextPart: + if p.Synthetic { + continue + } + text += p.Text + " " + case opencode.FilePart: + switch p.Source.Type { + case "file": + attachments = append(attachments, &attachment.Attachment{ + ID: p.ID, + Type: "file", + Display: p.Source.Text.Value, + URL: p.URL, + Filename: p.Filename, + MediaType: p.Mime, + StartIndex: int(p.Source.Text.Start), + EndIndex: int(p.Source.Text.End), + Source: &attachment.FileSource{ + Path: p.Source.Path, + Mime: p.Mime, + }, + }) + case "symbol": + r := p.Source.Range.(opencode.SymbolSourceRange) + attachments = append(attachments, &attachment.Attachment{ + ID: p.ID, + Type: "symbol", + Display: p.Source.Text.Value, + URL: p.URL, + Filename: p.Filename, + MediaType: p.Mime, + StartIndex: int(p.Source.Text.Start), + EndIndex: int(p.Source.Text.End), + Source: &attachment.SymbolSource{ + Path: p.Source.Path, + Name: p.Source.Name, + Kind: int(p.Source.Kind), + Range: attachment.SymbolRange{ + Start: attachment.Position{ + Line: int(r.Start.Line), + Char: int(r.Start.Character), + }, + End: attachment.Position{ + Line: int(r.End.Line), + Char: int(r.End.Character), + }, + }, + }, + }) + } + } + } + return &Prompt{ + Text: text, + Attachments: attachments, + }, nil + } + return nil, errors.New("unknown message type") +} + func (m Message) ToSessionChatParams() []opencode.SessionChatParamsPartUnion { parts := []opencode.SessionChatParamsPartUnion{} for _, part := range m.Parts { diff --git a/packages/tui/internal/commands/command.go b/packages/tui/internal/commands/command.go index 6015ab85b..8d1f3a6c4 100644 --- a/packages/tui/internal/commands/command.go +++ b/packages/tui/internal/commands/command.go @@ -138,7 +138,8 @@ const ( MessagesLastCommand CommandName = "messages_last" MessagesLayoutToggleCommand CommandName = "messages_layout_toggle" MessagesCopyCommand CommandName = "messages_copy" - MessagesRevertCommand CommandName = "messages_revert" + MessagesUndoCommand CommandName = "messages_undo" + MessagesRedoCommand CommandName = "messages_redo" AppExitCommand CommandName = "app_exit" ) @@ -348,9 +349,16 @@ func LoadFromConfig(config *opencode.Config) CommandRegistry { Keybindings: parseBindings("<leader>y"), }, { - Name: MessagesRevertCommand, - Description: "revert message", + Name: MessagesUndoCommand, + Description: "undo last message", + Keybindings: parseBindings("<leader>u"), + Trigger: []string{"undo"}, + }, + { + Name: MessagesRedoCommand, + Description: "redo message", Keybindings: parseBindings("<leader>r"), + Trigger: []string{"redo"}, }, { Name: AppExitCommand, @@ -365,7 +373,8 @@ func LoadFromConfig(config *opencode.Config) CommandRegistry { json.Unmarshal(marshalled, &keybinds) for _, command := range defaults { // Remove share/unshare commands if sharing is disabled - if config.Share == opencode.ConfigShareDisabled && (command.Name == SessionShareCommand || command.Name == SessionUnshareCommand) { + if config.Share == opencode.ConfigShareDisabled && + (command.Name == SessionShareCommand || command.Name == SessionUnshareCommand) { continue } if keybind, ok := keybinds[string(command.Name)]; ok && keybind != "" { diff --git a/packages/tui/internal/components/chat/editor.go b/packages/tui/internal/components/chat/editor.go index 4a5212786..4311a4ca7 100644 --- a/packages/tui/internal/components/chat/editor.go +++ b/packages/tui/internal/components/chat/editor.go @@ -21,6 +21,7 @@ import ( "github.com/sst/opencode/internal/commands" "github.com/sst/opencode/internal/components/dialog" "github.com/sst/opencode/internal/components/textarea" + "github.com/sst/opencode/internal/components/toast" "github.com/sst/opencode/internal/styles" "github.com/sst/opencode/internal/theme" "github.com/sst/opencode/internal/util" @@ -57,6 +58,7 @@ type editorComponent struct { historyIndex int // -1 means current (not in history) currentText string // Store current text when navigating history pasteCounter int + reverted bool } func (m *editorComponent) Init() tea.Cmd { @@ -122,10 +124,34 @@ func (m *editorComponent) Update(msg tea.Msg) (tea.Model, tea.Cmd) { } // Maximize editor responsiveness for printable characters if msg.Text != "" { + m.reverted = false m.textarea, cmd = m.textarea.Update(msg) cmds = append(cmds, cmd) return m, tea.Batch(cmds...) } + case app.MessageRevertedMsg: + if msg.Session.ID == m.app.Session.ID { + switch msg.Message.Info.(type) { + case opencode.UserMessage: + prompt, err := msg.Message.ToPrompt() + if err != nil { + return m, toast.NewErrorToast("Failed to revert message") + } + m.RestoreFromPrompt(*prompt) + m.textarea.MoveToEnd() + m.reverted = true + return m, nil + } + } + case app.SessionUnrevertedMsg: + if msg.Session.ID == m.app.Session.ID { + if m.reverted { + updated, cmd := m.Clear() + m = updated.(*editorComponent) + return m, cmd + } + return m, nil + } case tea.PasteMsg: text := string(msg) @@ -646,21 +672,14 @@ func NewEditorComponent(app *app.App) EditorComponent { return m } -// RestoreFromHistory restores a message from history at the given index -func (m *editorComponent) RestoreFromHistory(index int) { - if index < 0 || index >= len(m.app.State.MessageHistory) { - return - } - - entry := m.app.State.MessageHistory[index] - +func (m *editorComponent) RestoreFromPrompt(prompt app.Prompt) { m.textarea.Reset() - m.textarea.SetValue(entry.Text) + m.textarea.SetValue(prompt.Text) // Sort attachments by start index in reverse order (process from end to beginning) // This prevents index shifting issues - attachmentsCopy := make([]*attachment.Attachment, len(entry.Attachments)) - copy(attachmentsCopy, entry.Attachments) + attachmentsCopy := make([]*attachment.Attachment, len(prompt.Attachments)) + copy(attachmentsCopy, prompt.Attachments) for i := 0; i < len(attachmentsCopy)-1; i++ { for j := i + 1; j < len(attachmentsCopy); j++ { @@ -677,6 +696,15 @@ func (m *editorComponent) RestoreFromHistory(index int) { } } +// RestoreFromHistory restores a message from history at the given index +func (m *editorComponent) RestoreFromHistory(index int) { + if index < 0 || index >= len(m.app.State.MessageHistory) { + return + } + entry := m.app.State.MessageHistory[index] + m.RestoreFromPrompt(entry) +} + func getMediaTypeFromExtension(ext string) string { switch strings.ToLower(ext) { case ".jpg": diff --git a/packages/tui/internal/components/chat/messages.go b/packages/tui/internal/components/chat/messages.go index cbea349ca..e498de9f0 100644 --- a/packages/tui/internal/components/chat/messages.go +++ b/packages/tui/internal/components/chat/messages.go @@ -1,6 +1,7 @@ package chat import ( + "context" "fmt" "log/slog" "slices" @@ -11,6 +12,7 @@ import ( "github.com/charmbracelet/x/ansi" "github.com/sst/opencode-sdk-go" "github.com/sst/opencode/internal/app" + "github.com/sst/opencode/internal/commands" "github.com/sst/opencode/internal/components/dialog" "github.com/sst/opencode/internal/components/toast" "github.com/sst/opencode/internal/layout" @@ -31,6 +33,8 @@ type MessagesComponent interface { GotoTop() (tea.Model, tea.Cmd) GotoBottom() (tea.Model, tea.Cmd) CopyLastMessage() (tea.Model, tea.Cmd) + UndoLastMessage() (tea.Model, tea.Cmd) + RedoLastMessage() (tea.Model, tea.Cmd) } type messagesComponent struct { @@ -161,10 +165,22 @@ func (m *messagesComponent) Update(msg tea.Msg) (tea.Model, tea.Cmd) { m.tail = true m.loading = true return m, m.renderView() + case app.SessionUnrevertedMsg: + if msg.Session.ID == m.app.Session.ID { + m.cache.Clear() + m.tail = true + return m, m.renderView() + } + case app.MessageRevertedMsg: + if msg.Session.ID == m.app.Session.ID { + m.cache.Clear() + m.tail = true + return m, m.renderView() + } case opencode.EventListResponseEventSessionUpdated: if msg.Properties.Info.ID == m.app.Session.ID { - m.header = m.renderHeader() + cmds = append(cmds, m.renderView()) } case opencode.EventListResponseEventMessageUpdated: if msg.Properties.Info.SessionID == m.app.Session.ID { @@ -205,7 +221,6 @@ type renderCompleteMsg struct { } func (m *messagesComponent) renderView() tea.Cmd { - if m.rendering { slog.Debug("pending render, skipping") m.dirty = true @@ -233,6 +248,9 @@ func (m *messagesComponent) renderView() tea.Cmd { width := m.width // always use full width + reverted := false + revertedMessageCount := 0 + revertedToolCount := 0 lastAssistantMessage := "zzzzzzzzzzzzzzzzzzzzzzzzzzzzzzzz" for _, msg := range slices.Backward(m.app.Messages) { if assistant, ok := msg.Info.(opencode.AssistantMessage); ok { @@ -246,6 +264,17 @@ func (m *messagesComponent) renderView() tea.Cmd { switch casted := message.Info.(type) { case opencode.UserMessage: + if casted.ID == m.app.Session.Revert.MessageID { + reverted = true + revertedMessageCount = 1 + revertedToolCount = 0 + continue + } + if reverted { + revertedMessageCount++ + continue + } + for partIndex, part := range message.Parts { switch part := part.(type) { case opencode.TextPart: @@ -324,10 +353,18 @@ func (m *messagesComponent) renderView() tea.Cmd { } case opencode.AssistantMessage: + if casted.ID == m.app.Session.Revert.MessageID { + reverted = true + revertedMessageCount = 1 + revertedToolCount = 0 + } hasTextPart := false for partIndex, p := range message.Parts { switch part := p.(type) { case opencode.TextPart: + if reverted { + continue + } hasTextPart = true finished := part.Time.End > 0 remainingParts := message.Parts[partIndex+1:] @@ -406,6 +443,10 @@ func (m *messagesComponent) renderView() tea.Cmd { blocks = append(blocks, content) } case opencode.ToolPart: + if reverted { + revertedToolCount++ + continue + } if !m.showToolDetails { if !hasTextPart { orphanedToolCalls = append(orphanedToolCalls, part) @@ -472,7 +513,7 @@ func (m *messagesComponent) renderView() tea.Cmd { } } - if error != "" { + if error != "" && !reverted { error = styles.NewStyle().Width(width - 6).Render(error) error = renderContentBlock( m.app, @@ -491,6 +532,44 @@ func (m *messagesComponent) renderView() tea.Cmd { } } + if revertedMessageCount > 0 || revertedToolCount > 0 { + messagePlural := "" + toolPlural := "" + if revertedMessageCount != 1 { + messagePlural = "s" + } + if revertedToolCount != 1 { + toolPlural = "s" + } + revertedStyle := styles.NewStyle(). + Background(t.BackgroundPanel()). + Foreground(t.TextMuted()) + + content := revertedStyle.Render(fmt.Sprintf( + "%d message%s reverted, %d tool call%s reverted", + revertedMessageCount, + messagePlural, + revertedToolCount, + toolPlural, + )) + hintStyle := styles.NewStyle().Background(t.BackgroundPanel()).Foreground(t.Text()) + hint := hintStyle.Render(m.app.Keybind(commands.MessagesRedoCommand)) + hint += revertedStyle.Render(" (or /redo) to restore") + + content += "\n" + hint + content = styles.NewStyle(). + Background(t.BackgroundPanel()). + Width(width - 6). + Render(content) + content = renderContentBlock( + m.app, + content, + width, + WithBorderColor(t.BackgroundPanel()), + ) + blocks = append(blocks, content) + } + final := []string{} clipboard := []string{} var selection *selection @@ -522,7 +601,11 @@ func (m *messagesComponent) renderView() tea.Cmd { middle := strings.TrimRight(ansi.Strip(ansi.Cut(line, left, right)), " ") suffix := ansi.Cut(line, left+ansi.StringWidth(middle), width) clipboard = append(clipboard, middle) - line = prefix + styles.NewStyle().Background(t.Accent()).Foreground(t.BackgroundPanel()).Render(middle) + suffix + line = prefix + styles.NewStyle(). + Background(t.Accent()). + Foreground(t.BackgroundPanel()). + Render(ansi.Strip(middle)) + + suffix } final = append(final, line) } @@ -773,6 +856,155 @@ func (m *messagesComponent) CopyLastMessage() (tea.Model, tea.Cmd) { return m, tea.Batch(cmds...) } +func (m *messagesComponent) UndoLastMessage() (tea.Model, tea.Cmd) { + after := float64(0) + var revertedMessage app.Message + reversedMessages := []app.Message{} + for i := len(m.app.Messages) - 1; i >= 0; i-- { + reversedMessages = append(reversedMessages, m.app.Messages[i]) + switch casted := m.app.Messages[i].Info.(type) { + case opencode.UserMessage: + if casted.ID == m.app.Session.Revert.MessageID { + after = casted.Time.Created + } + case opencode.AssistantMessage: + if casted.ID == m.app.Session.Revert.MessageID { + after = casted.Time.Created + } + } + if m.app.Session.Revert.PartID != "" { + for _, part := range m.app.Messages[i].Parts { + switch casted := part.(type) { + case opencode.TextPart: + if casted.ID == m.app.Session.Revert.PartID { + after = casted.Time.Start + } + case opencode.ToolPart: + // TODO: handle tool parts + } + } + } + } + + messageID := "" + for _, msg := range reversedMessages { + switch casted := msg.Info.(type) { + case opencode.UserMessage: + if after > 0 && casted.Time.Created >= after { + continue + } + messageID = casted.ID + revertedMessage = msg + } + if messageID != "" { + break + } + } + + if messageID == "" { + return m, nil + } + + return m, func() tea.Msg { + response, err := m.app.Client.Session.Revert( + context.Background(), + m.app.Session.ID, + opencode.SessionRevertParams{ + MessageID: opencode.F(messageID), + }, + ) + if err != nil { + slog.Error("Failed to undo message", "error", err) + return toast.NewErrorToast("Failed to undo message") + } + if response == nil { + return toast.NewErrorToast("Failed to undo message") + } + return app.MessageRevertedMsg{Session: *response, Message: revertedMessage} + } +} + +func (m *messagesComponent) RedoLastMessage() (tea.Model, tea.Cmd) { + before := float64(0) + var revertedMessage app.Message + for _, message := range m.app.Messages { + switch casted := message.Info.(type) { + case opencode.UserMessage: + if casted.ID == m.app.Session.Revert.MessageID { + before = casted.Time.Created + } + case opencode.AssistantMessage: + if casted.ID == m.app.Session.Revert.MessageID { + before = casted.Time.Created + } + } + if m.app.Session.Revert.PartID != "" { + for _, part := range message.Parts { + switch casted := part.(type) { + case opencode.TextPart: + if casted.ID == m.app.Session.Revert.PartID { + before = casted.Time.Start + } + case opencode.ToolPart: + // TODO: handle tool parts + } + } + } + } + + messageID := "" + for _, msg := range m.app.Messages { + switch casted := msg.Info.(type) { + case opencode.UserMessage: + if casted.Time.Created <= before { + continue + } + messageID = casted.ID + revertedMessage = msg + } + if messageID != "" { + break + } + } + + if messageID == "" { + return m, func() tea.Msg { + // unrevert back to original state + response, err := m.app.Client.Session.Unrevert( + context.Background(), + m.app.Session.ID, + ) + if err != nil { + slog.Error("Failed to unrevert session", "error", err) + return toast.NewErrorToast("Failed to redo message") + } + if response == nil { + return toast.NewErrorToast("Failed to redo message") + } + return app.SessionUnrevertedMsg{Session: *response} + } + } + + return m, func() tea.Msg { + // calling revert on a "later" message is like a redo + response, err := m.app.Client.Session.Revert( + context.Background(), + m.app.Session.ID, + opencode.SessionRevertParams{ + MessageID: opencode.F(messageID), + }, + ) + if err != nil { + slog.Error("Failed to redo message", "error", err) + return toast.NewErrorToast("Failed to redo message") + } + if response == nil { + return toast.NewErrorToast("Failed to redo message") + } + return app.MessageRevertedMsg{Session: *response, Message: revertedMessage} + } +} + func NewMessagesComponent(app *app.App) MessagesComponent { vp := viewport.New() vp.KeyMap = viewport.KeyMap{} diff --git a/packages/tui/internal/tui/tui.go b/packages/tui/internal/tui/tui.go index 752e805ea..ee0b22b48 100644 --- a/packages/tui/internal/tui/tui.go +++ b/packages/tui/internal/tui/tui.go @@ -470,6 +470,10 @@ func (a Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) { case app.SessionCreatedMsg: a.app.Session = msg.Session return a, util.CmdHandler(app.SessionLoadedMsg{}) + case app.MessageRevertedMsg: + if msg.Session.ID == a.app.Session.ID { + a.app.Session = &msg.Session + } case app.ModelSelectedMsg: a.app.Provider = &msg.Provider a.app.Model = &msg.Model @@ -1045,7 +1049,14 @@ func (a Model) executeCommand(command commands.Command) (tea.Model, tea.Cmd) { updated, cmd := a.messages.CopyLastMessage() a.messages = updated.(chat.MessagesComponent) cmds = append(cmds, cmd) - case commands.MessagesRevertCommand: + case commands.MessagesUndoCommand: + updated, cmd := a.messages.UndoLastMessage() + a.messages = updated.(chat.MessagesComponent) + cmds = append(cmds, cmd) + case commands.MessagesRedoCommand: + updated, cmd := a.messages.RedoLastMessage() + a.messages = updated.(chat.MessagesComponent) + cmds = append(cmds, cmd) case commands.AppExitCommand: return a, tea.Quit } diff --git a/packages/tui/sdk/.stats.yml b/packages/tui/sdk/.stats.yml index f9f86831f..da0b08b5a 100644 --- a/packages/tui/sdk/.stats.yml +++ b/packages/tui/sdk/.stats.yml @@ -1,4 +1,4 @@ -configured_endpoints: 24 -openapi_spec_url: https://storage.googleapis.com/stainless-sdk-openapi-specs/opencode%2Fopencode-9574184bd9e916aa69eae8e26e0679556038d3fcfb4009a445c97c6cc3e4f3ee.yml -openapi_spec_hash: 93ba1215ab0dc853a1691b049cc47d75 -config_hash: 09e4835d57ec7ed0b2d316c6815bcf0a +configured_endpoints: 26 +openapi_spec_url: https://storage.googleapis.com/stainless-sdk-openapi-specs/opencode%2Fopencode-1efc45c35b58e88b0550fbb0c7a204ef66522742f87c9e29c76a18b120c0d945.yml +openapi_spec_hash: 5e15d85e4704624f9b13bae1c71aa416 +config_hash: 1ae82c93499b9f0b9ba828b8919f9cb3 diff --git a/packages/tui/sdk/api.md b/packages/tui/sdk/api.md index 1f5191cad..0bb72433c 100644 --- a/packages/tui/sdk/api.md +++ b/packages/tui/sdk/api.md @@ -114,8 +114,10 @@ Methods: - <code title="post /session/{id}/message">client.Session.<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go#SessionService.Chat">Chat</a>(ctx <a href="https://pkg.go.dev/context">context</a>.<a href="https://pkg.go.dev/context#Context">Context</a>, id <a href="https://pkg.go.dev/builtin#string">string</a>, body <a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go">opencode</a>.<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go#SessionChatParams">SessionChatParams</a>) (<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go">opencode</a>.<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go#AssistantMessage">AssistantMessage</a>, <a href="https://pkg.go.dev/builtin#error">error</a>)</code> - <code title="post /session/{id}/init">client.Session.<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go#SessionService.Init">Init</a>(ctx <a href="https://pkg.go.dev/context">context</a>.<a href="https://pkg.go.dev/context#Context">Context</a>, id <a href="https://pkg.go.dev/builtin#string">string</a>, body <a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go">opencode</a>.<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go#SessionInitParams">SessionInitParams</a>) (<a href="https://pkg.go.dev/builtin#bool">bool</a>, <a href="https://pkg.go.dev/builtin#error">error</a>)</code> - <code title="get /session/{id}/message">client.Session.<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go#SessionService.Messages">Messages</a>(ctx <a href="https://pkg.go.dev/context">context</a>.<a href="https://pkg.go.dev/context#Context">Context</a>, id <a href="https://pkg.go.dev/builtin#string">string</a>) ([]<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go">opencode</a>.<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go#SessionMessagesResponse">SessionMessagesResponse</a>, <a href="https://pkg.go.dev/builtin#error">error</a>)</code> +- <code title="post /session/{id}/revert">client.Session.<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go#SessionService.Revert">Revert</a>(ctx <a href="https://pkg.go.dev/context">context</a>.<a href="https://pkg.go.dev/context#Context">Context</a>, id <a href="https://pkg.go.dev/builtin#string">string</a>, body <a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go">opencode</a>.<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go#SessionRevertParams">SessionRevertParams</a>) (<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go">opencode</a>.<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go#Session">Session</a>, <a href="https://pkg.go.dev/builtin#error">error</a>)</code> - <code title="post /session/{id}/share">client.Session.<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go#SessionService.Share">Share</a>(ctx <a href="https://pkg.go.dev/context">context</a>.<a href="https://pkg.go.dev/context#Context">Context</a>, id <a href="https://pkg.go.dev/builtin#string">string</a>) (<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go">opencode</a>.<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go#Session">Session</a>, <a href="https://pkg.go.dev/builtin#error">error</a>)</code> - <code title="post /session/{id}/summarize">client.Session.<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go#SessionService.Summarize">Summarize</a>(ctx <a href="https://pkg.go.dev/context">context</a>.<a href="https://pkg.go.dev/context#Context">Context</a>, id <a href="https://pkg.go.dev/builtin#string">string</a>, body <a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go">opencode</a>.<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go#SessionSummarizeParams">SessionSummarizeParams</a>) (<a href="https://pkg.go.dev/builtin#bool">bool</a>, <a href="https://pkg.go.dev/builtin#error">error</a>)</code> +- <code title="post /session/{id}/unrevert">client.Session.<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go#SessionService.Unrevert">Unrevert</a>(ctx <a href="https://pkg.go.dev/context">context</a>.<a href="https://pkg.go.dev/context#Context">Context</a>, id <a href="https://pkg.go.dev/builtin#string">string</a>) (<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go">opencode</a>.<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go#Session">Session</a>, <a href="https://pkg.go.dev/builtin#error">error</a>)</code> - <code title="delete /session/{id}/share">client.Session.<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go#SessionService.Unshare">Unshare</a>(ctx <a href="https://pkg.go.dev/context">context</a>.<a href="https://pkg.go.dev/context#Context">Context</a>, id <a href="https://pkg.go.dev/builtin#string">string</a>) (<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go">opencode</a>.<a href="https://pkg.go.dev/github.com/sst/opencode-sdk-go#Session">Session</a>, <a href="https://pkg.go.dev/builtin#error">error</a>)</code> # Tui diff --git a/packages/tui/sdk/config.go b/packages/tui/sdk/config.go index 0461cba87..96759fc47 100644 --- a/packages/tui/sdk/config.go +++ b/packages/tui/sdk/config.go @@ -510,8 +510,12 @@ type KeybindsConfig struct { MessagesPageUp string `json:"messages_page_up,required"` // Navigate to previous message MessagesPrevious string `json:"messages_previous,required"` - // Revert message + // Redo message + MessagesRedo string `json:"messages_redo,required"` + // @deprecated use messages_undo. Revert message MessagesRevert string `json:"messages_revert,required"` + // Undo message + MessagesUndo string `json:"messages_undo,required"` // List available models ModelList string `json:"model_list,required"` // Create/update AGENTS.md @@ -565,7 +569,9 @@ type keybindsConfigJSON struct { MessagesPageDown apijson.Field MessagesPageUp apijson.Field MessagesPrevious apijson.Field + MessagesRedo apijson.Field MessagesRevert apijson.Field + MessagesUndo apijson.Field ModelList apijson.Field ProjectInit apijson.Field SessionCompact apijson.Field diff --git a/packages/tui/sdk/session.go b/packages/tui/sdk/session.go index 8426b64eb..86d46000d 100644 --- a/packages/tui/sdk/session.go +++ b/packages/tui/sdk/session.go @@ -112,6 +112,18 @@ func (r *SessionService) Messages(ctx context.Context, id string, opts ...option return } +// Revert a message +func (r *SessionService) Revert(ctx context.Context, id string, body SessionRevertParams, opts ...option.RequestOption) (res *Session, err error) { + opts = append(r.Options[:], opts...) + if id == "" { + err = errors.New("missing required id parameter") + return + } + path := fmt.Sprintf("session/%s/revert", id) + err = requestconfig.ExecuteNewRequest(ctx, http.MethodPost, path, body, &res, opts...) + return +} + // Share a session func (r *SessionService) Share(ctx context.Context, id string, opts ...option.RequestOption) (res *Session, err error) { opts = append(r.Options[:], opts...) @@ -136,6 +148,18 @@ func (r *SessionService) Summarize(ctx context.Context, id string, body SessionS return } +// Restore all reverted messages +func (r *SessionService) Unrevert(ctx context.Context, id string, opts ...option.RequestOption) (res *Session, err error) { + opts = append(r.Options[:], opts...) + if id == "" { + err = errors.New("missing required id parameter") + return + } + path := fmt.Sprintf("session/%s/unrevert", id) + err = requestconfig.ExecuteNewRequest(ctx, http.MethodPost, path, nil, &res, opts...) + return +} + // Unshare the session func (r *SessionService) Unshare(ctx context.Context, id string, opts ...option.RequestOption) (res *Session, err error) { opts = append(r.Options[:], opts...) @@ -988,7 +1012,7 @@ func (r sessionTimeJSON) RawJSON() string { type SessionRevert struct { MessageID string `json:"messageID,required"` - Part float64 `json:"part,required"` + PartID string `json:"partID"` Snapshot string `json:"snapshot"` JSON sessionRevertJSON `json:"-"` } @@ -996,7 +1020,7 @@ type SessionRevert struct { // sessionRevertJSON contains the JSON metadata for the struct [SessionRevert] type sessionRevertJSON struct { MessageID apijson.Field - Part apijson.Field + PartID apijson.Field Snapshot apijson.Field raw string ExtraFields map[string]apijson.Field @@ -2010,6 +2034,15 @@ func (r SessionInitParams) MarshalJSON() (data []byte, err error) { return apijson.MarshalRoot(r) } +type SessionRevertParams struct { + MessageID param.Field[string] `json:"messageID,required"` + PartID param.Field[string] `json:"partID"` +} + +func (r SessionRevertParams) MarshalJSON() (data []byte, err error) { + return apijson.MarshalRoot(r) +} + type SessionSummarizeParams struct { ModelID param.Field[string] `json:"modelID,required"` ProviderID param.Field[string] `json:"providerID,required"` diff --git a/packages/tui/sdk/session_test.go b/packages/tui/sdk/session_test.go index 5d7c55cad..807f19563 100644 --- a/packages/tui/sdk/session_test.go +++ b/packages/tui/sdk/session_test.go @@ -197,6 +197,35 @@ func TestSessionMessages(t *testing.T) { } } +func TestSessionRevertWithOptionalParams(t *testing.T) { + t.Skip("skipped: tests are disabled for the time being") + baseURL := "http://localhost:4010" + if envURL, ok := os.LookupEnv("TEST_API_BASE_URL"); ok { + baseURL = envURL + } + if !testutil.CheckTestServer(t, baseURL) { + return + } + client := opencode.NewClient( + option.WithBaseURL(baseURL), + ) + _, err := client.Session.Revert( + context.TODO(), + "id", + opencode.SessionRevertParams{ + MessageID: opencode.F("msg"), + PartID: opencode.F("prt"), + }, + ) + if err != nil { + var apierr *opencode.Error + if errors.As(err, &apierr) { + t.Log(string(apierr.DumpRequest(true))) + } + t.Fatalf("err should be nil: %s", err.Error()) + } +} + func TestSessionShare(t *testing.T) { t.Skip("skipped: tests are disabled for the time being") baseURL := "http://localhost:4010" @@ -248,6 +277,28 @@ func TestSessionSummarize(t *testing.T) { } } +func TestSessionUnrevert(t *testing.T) { + t.Skip("skipped: tests are disabled for the time being") + baseURL := "http://localhost:4010" + if envURL, ok := os.LookupEnv("TEST_API_BASE_URL"); ok { + baseURL = envURL + } + if !testutil.CheckTestServer(t, baseURL) { + return + } + client := opencode.NewClient( + option.WithBaseURL(baseURL), + ) + _, err := client.Session.Unrevert(context.TODO(), "id") + if err != nil { + var apierr *opencode.Error + if errors.As(err, &apierr) { + t.Log(string(apierr.DumpRequest(true))) + } + t.Fatalf("err should be nil: %s", err.Error()) + } +} + func TestSessionUnshare(t *testing.T) { t.Skip("skipped: tests are disabled for the time being") baseURL := "http://localhost:4010" |
