diff --git a/internal/llm/tools/diagnostics.go b/internal/llm/tools/diagnostics.go index 89ad484c134a30fa4b18526e64a25e37cc0912eb..e48132414ad338635ada6294134e3d7f2d938c7d 100644 --- a/internal/llm/tools/diagnostics.go +++ b/internal/llm/tools/diagnostics.go @@ -245,28 +245,28 @@ func getDiagnostics(filePath string, lsps map[string]*lsp.Client) string { return projectDiagnostics[i] < projectDiagnostics[j] }) - output := "" + var output strings.Builder if len(fileDiagnostics) > 0 { - output += "\n\n" + output.WriteString("\n\n") if len(fileDiagnostics) > 10 { - output += strings.Join(fileDiagnostics[:10], "\n") - output += fmt.Sprintf("\n... and %d more diagnostics", len(fileDiagnostics)-10) + output.WriteString(strings.Join(fileDiagnostics[:10], "\n")) + fmt.Fprintf(&output, "\n... and %d more diagnostics", len(fileDiagnostics)-10) } else { - output += strings.Join(fileDiagnostics, "\n") + output.WriteString(strings.Join(fileDiagnostics, "\n")) } - output += "\n\n" + output.WriteString("\n\n") } if len(projectDiagnostics) > 0 { - output += "\n\n" + output.WriteString("\n\n") if len(projectDiagnostics) > 10 { - output += strings.Join(projectDiagnostics[:10], "\n") - output += fmt.Sprintf("\n... and %d more diagnostics", len(projectDiagnostics)-10) + output.WriteString(strings.Join(projectDiagnostics[:10], "\n")) + fmt.Fprintf(&output, "\n... and %d more diagnostics", len(projectDiagnostics)-10) } else { - output += strings.Join(projectDiagnostics, "\n") + output.WriteString(strings.Join(projectDiagnostics, "\n")) } - output += "\n\n" + output.WriteString("\n\n") } if len(fileDiagnostics) > 0 || len(projectDiagnostics) > 0 { @@ -275,13 +275,13 @@ func getDiagnostics(filePath string, lsps map[string]*lsp.Client) string { projectErrors := countSeverity(projectDiagnostics, "Error") projectWarnings := countSeverity(projectDiagnostics, "Warn") - output += "\n\n" - output += fmt.Sprintf("Current file: %d errors, %d warnings\n", fileErrors, fileWarnings) - output += fmt.Sprintf("Project: %d errors, %d warnings\n", projectErrors, projectWarnings) - output += "\n" + output.WriteString("\n\n") + fmt.Fprintf(&output, "Current file: %d errors, %d warnings\n", fileErrors, fileWarnings) + fmt.Fprintf(&output, "Project: %d errors, %d warnings\n", projectErrors, projectWarnings) + output.WriteString("\n") } - return output + return output.String() } func countSeverity(diagnostics []string, severity string) int { diff --git a/internal/llm/tools/grep.go b/internal/llm/tools/grep.go index e043a71e0bafc77050bf444a02f50c2462018e8d..e6992a4f8c23dc440898298d8f0e5d880b2fdc53 100644 --- a/internal/llm/tools/grep.go +++ b/internal/llm/tools/grep.go @@ -198,35 +198,35 @@ func (g *grepTool) Run(ctx context.Context, call ToolCall) (ToolResponse, error) return ToolResponse{}, fmt.Errorf("error searching files: %w", err) } - var output string + var output strings.Builder if len(matches) == 0 { - output = "No files found" + output.WriteString("No files found") } else { - output = fmt.Sprintf("Found %d matches\n", len(matches)) + fmt.Fprintf(&output, "Found %d matches\n", len(matches)) currentFile := "" for _, match := range matches { if currentFile != match.path { if currentFile != "" { - output += "\n" + output.WriteString("\n") } currentFile = match.path - output += fmt.Sprintf("%s:\n", match.path) + fmt.Fprintf(&output, "%s:\n", match.path) } if match.lineNum > 0 { - output += fmt.Sprintf(" Line %d: %s\n", match.lineNum, match.lineText) + fmt.Fprintf(&output, " Line %d: %s\n", match.lineNum, match.lineText) } else { - output += fmt.Sprintf(" %s\n", match.path) + fmt.Fprintf(&output, " %s\n", match.path) } } if truncated { - output += "\n(Results are truncated. Consider using a more specific path or pattern.)" + output.WriteString("\n(Results are truncated. Consider using a more specific path or pattern.)") } } return WithResponseMetadata( - NewTextResponse(output), + NewTextResponse(output.String()), GrepResponseMetadata{ NumberOfMatches: len(matches), Truncated: truncated, diff --git a/internal/llm/tools/ls.go b/internal/llm/tools/ls.go index 81539abe8db1ce2ccde5b8261d34474ab77ea076..a0426b1393f42bc4729db558641e0c178a680d27 100644 --- a/internal/llm/tools/ls.go +++ b/internal/llm/tools/ls.go @@ -209,7 +209,7 @@ func printNode(builder *strings.Builder, node *TreeNode, level int) { nodeName := node.Name if node.Type == "directory" { - nodeName += string(filepath.Separator) + nodeName = nodeName + string(filepath.Separator) } fmt.Fprintf(builder, "%s- %s\n", indent, nodeName) diff --git a/internal/tui/components/chat/editor/keys.go b/internal/tui/components/chat/editor/keys.go index 51c521244ec8e060333ff3e89328f73252ce5369..0f85249645b976e861a6e9bdff61b7fbebcdbd50 100644 --- a/internal/tui/components/chat/editor/keys.go +++ b/internal/tui/components/chat/editor/keys.go @@ -2,7 +2,6 @@ package editor import ( "github.com/charmbracelet/bubbles/v2/key" - "github.com/charmbracelet/crush/internal/tui/components/core/layout" ) type EditorKeyMap struct { @@ -23,10 +22,18 @@ func DefaultEditorKeyMap() EditorKeyMap { } } +// KeyBindings implements layout.KeyMapProvider +func (k EditorKeyMap) KeyBindings() []key.Binding { + return []key.Binding{ + k.Send, + k.OpenEditor, + } +} + // FullHelp implements help.KeyMap. func (k EditorKeyMap) FullHelp() [][]key.Binding { m := [][]key.Binding{} - slice := layout.KeyMapToSlice(k) + slice := k.KeyBindings() for i := 0; i < len(slice); i += 4 { end := min(i+4, len(slice)) m = append(m, slice[i:end]) diff --git a/internal/tui/components/completions/keys.go b/internal/tui/components/completions/keys.go index c2f7a10d77b12439f18d6bf110c11debaa8b4ba7..fee3a0e574ab926cb4e80d02c0d9d19c0e614edd 100644 --- a/internal/tui/components/completions/keys.go +++ b/internal/tui/components/completions/keys.go @@ -2,7 +2,6 @@ package completions import ( "github.com/charmbracelet/bubbles/v2/key" - "github.com/charmbracelet/crush/internal/tui/components/core/layout" ) type KeyMap struct { @@ -33,10 +32,20 @@ func DefaultKeyMap() KeyMap { } } +// KeyBindings implements layout.KeyMapProvider +func (k KeyMap) KeyBindings() []key.Binding { + return []key.Binding{ + k.Down, + k.Up, + k.Select, + k.Cancel, + } +} + // FullHelp implements help.KeyMap. func (k KeyMap) FullHelp() [][]key.Binding { m := [][]key.Binding{} - slice := layout.KeyMapToSlice(k) + slice := k.KeyBindings() for i := 0; i < len(slice); i += 4 { end := min(i+4, len(slice)) m = append(m, slice[i:end]) diff --git a/internal/tui/components/core/layout/layout.go b/internal/tui/components/core/layout/layout.go index ae43c51bf685134f859bf99ac9cef3d6328cb1e1..a0d0b9c0326de3d6a263eb1707f99b8f61a72b95 100644 --- a/internal/tui/components/core/layout/layout.go +++ b/internal/tui/components/core/layout/layout.go @@ -1,8 +1,6 @@ package layout import ( - "reflect" - "github.com/charmbracelet/bubbles/v2/help" "github.com/charmbracelet/bubbles/v2/key" tea "github.com/charmbracelet/bubbletea/v2" @@ -27,14 +25,7 @@ type Positionable interface { SetPosition(x, y int) tea.Cmd } -func KeyMapToSlice(t any) (bindings []key.Binding) { - typ := reflect.TypeOf(t) - if typ.Kind() != reflect.Struct { - return nil - } - for i := range typ.NumField() { - v := reflect.ValueOf(t).Field(i) - bindings = append(bindings, v.Interface().(key.Binding)) - } - return +// KeyMapProvider defines an interface for types that can provide their key bindings as a slice +type KeyMapProvider interface { + KeyBindings() []key.Binding } diff --git a/internal/tui/components/core/list/keys.go b/internal/tui/components/core/list/keys.go index 92201cebdcc503ea621c75e851381c8ffad3a6ad..9cc628ab912d43aadde09a863d389cdb5dc618ba 100644 --- a/internal/tui/components/core/list/keys.go +++ b/internal/tui/components/core/list/keys.go @@ -2,7 +2,6 @@ package list import ( "github.com/charmbracelet/bubbles/v2/key" - "github.com/charmbracelet/crush/internal/tui/components/core/layout" ) type KeyMap struct { @@ -53,10 +52,26 @@ func DefaultKeyMap() KeyMap { } } +// KeyBindings implements layout.KeyMapProvider +func (k KeyMap) KeyBindings() []key.Binding { + return []key.Binding{ + k.Down, + k.Up, + k.NDown, + k.NUp, + k.DownOneItem, + k.UpOneItem, + k.HalfPageDown, + k.HalfPageUp, + k.Home, + k.End, + } +} + // FullHelp implements help.KeyMap. func (k KeyMap) FullHelp() [][]key.Binding { m := [][]key.Binding{} - slice := layout.KeyMapToSlice(k) + slice := k.KeyBindings() for i := 0; i < len(slice); i += 4 { end := min(i+4, len(slice)) m = append(m, slice[i:end]) diff --git a/internal/tui/components/core/status/keys.go b/internal/tui/components/core/status/keys.go index b16d035bfd0154ff0e8a82af87a67e2d2bd58053..d33abe2d2102ef88bfba6ce8c2d89ea66bd95282 100644 --- a/internal/tui/components/core/status/keys.go +++ b/internal/tui/components/core/status/keys.go @@ -2,7 +2,6 @@ package status import ( "github.com/charmbracelet/bubbles/v2/key" - "github.com/charmbracelet/crush/internal/tui/components/core/layout" ) type KeyMap struct { @@ -33,10 +32,20 @@ func DefaultKeyMap(tabHelp string) KeyMap { } } +// KeyBindings implements layout.KeyMapProvider +func (k KeyMap) KeyBindings() []key.Binding { + return []key.Binding{ + k.Tab, + k.Commands, + k.Sessions, + k.Help, + } +} + // FullHelp implements help.KeyMap. func (k KeyMap) FullHelp() [][]key.Binding { m := [][]key.Binding{} - slice := layout.KeyMapToSlice(k) + slice := k.KeyBindings() for i := 0; i < len(slice); i += 4 { end := min(i+4, len(slice)) m = append(m, slice[i:end]) diff --git a/internal/tui/components/dialogs/commands/keys.go b/internal/tui/components/dialogs/commands/keys.go index 08420f56fd2b57aeaa407a4908bba2e83fc0e569..9685216817c02cdfaab682f94e0f89aa64af365f 100644 --- a/internal/tui/components/dialogs/commands/keys.go +++ b/internal/tui/components/dialogs/commands/keys.go @@ -2,7 +2,6 @@ package commands import ( "github.com/charmbracelet/bubbles/v2/key" - "github.com/charmbracelet/crush/internal/tui/components/core/layout" ) type CommandsDialogKeyMap struct { @@ -38,10 +37,21 @@ func DefaultCommandsDialogKeyMap() CommandsDialogKeyMap { } } +// KeyBindings implements layout.KeyMapProvider +func (k CommandsDialogKeyMap) KeyBindings() []key.Binding { + return []key.Binding{ + k.Select, + k.Next, + k.Previous, + k.Tab, + k.Close, + } +} + // FullHelp implements help.KeyMap. func (k CommandsDialogKeyMap) FullHelp() [][]key.Binding { m := [][]key.Binding{} - slice := layout.KeyMapToSlice(k) + slice := k.KeyBindings() for i := 0; i < len(slice); i += 4 { end := min(i+4, len(slice)) m = append(m, slice[i:end]) @@ -86,10 +96,19 @@ func DefaultArgumentsDialogKeyMap() ArgumentsDialogKeyMap { } } +// KeyBindings implements layout.KeyMapProvider +func (k ArgumentsDialogKeyMap) KeyBindings() []key.Binding { + return []key.Binding{ + k.Confirm, + k.Next, + k.Previous, + } +} + // FullHelp implements help.KeyMap. func (k ArgumentsDialogKeyMap) FullHelp() [][]key.Binding { m := [][]key.Binding{} - slice := layout.KeyMapToSlice(k) + slice := k.KeyBindings() for i := 0; i < len(slice); i += 4 { end := min(i+4, len(slice)) m = append(m, slice[i:end]) diff --git a/internal/tui/components/dialogs/compact/keys.go b/internal/tui/components/dialogs/compact/keys.go index 491b582388593ec33ef2584e503aa3677679c59d..c3dd98e13035085b7d46e7a2e94450b25a7f0d59 100644 --- a/internal/tui/components/dialogs/compact/keys.go +++ b/internal/tui/components/dialogs/compact/keys.go @@ -2,7 +2,6 @@ package compact import ( "github.com/charmbracelet/bubbles/v2/key" - "github.com/charmbracelet/crush/internal/tui/components/core/layout" ) // KeyMap defines the key bindings for the compact dialog. @@ -40,10 +39,21 @@ func DefaultKeyMap() KeyMap { } } +// KeyBindings implements layout.KeyMapProvider +func (k KeyMap) KeyBindings() []key.Binding { + return []key.Binding{ + k.ChangeSelection, + k.Select, + k.Y, + k.N, + k.Close, + } +} + // FullHelp implements help.KeyMap. func (k KeyMap) FullHelp() [][]key.Binding { m := [][]key.Binding{} - slice := layout.KeyMapToSlice(k) + slice := k.KeyBindings() for i := 0; i < len(slice); i += 4 { end := min(i+4, len(slice)) m = append(m, slice[i:end]) diff --git a/internal/tui/components/dialogs/filepicker/keys.go b/internal/tui/components/dialogs/filepicker/keys.go index cc95915c675058e3728d2bd5d1228b638204b093..9f3b706e3cf677b66cbc3136a7b98a466470d949 100644 --- a/internal/tui/components/dialogs/filepicker/keys.go +++ b/internal/tui/components/dialogs/filepicker/keys.go @@ -2,7 +2,6 @@ package filepicker import ( "github.com/charmbracelet/bubbles/v2/key" - "github.com/charmbracelet/crush/internal/tui/components/core/layout" ) // KeyMap defines keyboard bindings for dialog management. @@ -45,10 +44,22 @@ func DefaultKeyMap() KeyMap { } } +// KeyBindings implements layout.KeyMapProvider +func (k KeyMap) KeyBindings() []key.Binding { + return []key.Binding{ + k.Select, + k.Down, + k.Up, + k.Forward, + k.Backward, + k.Close, + } +} + // FullHelp implements help.KeyMap. func (k KeyMap) FullHelp() [][]key.Binding { m := [][]key.Binding{} - slice := layout.KeyMapToSlice(k) + slice := k.KeyBindings() for i := 0; i < len(slice); i += 4 { end := min(i+4, len(slice)) m = append(m, slice[i:end]) diff --git a/internal/tui/components/dialogs/init/keys.go b/internal/tui/components/dialogs/init/keys.go index 163b4869fb477ae34545ef169725b551cd05c6fd..afd82d45ea8b47630c2d5ed1450419ae8d4b4c19 100644 --- a/internal/tui/components/dialogs/init/keys.go +++ b/internal/tui/components/dialogs/init/keys.go @@ -2,7 +2,6 @@ package init import ( "github.com/charmbracelet/bubbles/v2/key" - "github.com/charmbracelet/crush/internal/tui/components/core/layout" ) type KeyMap struct { @@ -38,10 +37,21 @@ func DefaultKeyMap() KeyMap { } } +// KeyBindings implements layout.KeyMapProvider +func (k KeyMap) KeyBindings() []key.Binding { + return []key.Binding{ + k.ChangeSelection, + k.Select, + k.Y, + k.N, + k.Close, + } +} + // FullHelp implements help.KeyMap. func (k KeyMap) FullHelp() [][]key.Binding { m := [][]key.Binding{} - slice := layout.KeyMapToSlice(k) + slice := k.KeyBindings() for i := 0; i < len(slice); i += 4 { end := min(i+4, len(slice)) m = append(m, slice[i:end]) diff --git a/internal/tui/components/dialogs/keys.go b/internal/tui/components/dialogs/keys.go index 027ed6e3a8ad86d2587061085623117a44016d01..c382b7e09e15de04efb5b2520bc490ef9d57b985 100644 --- a/internal/tui/components/dialogs/keys.go +++ b/internal/tui/components/dialogs/keys.go @@ -2,7 +2,6 @@ package dialogs import ( "github.com/charmbracelet/bubbles/v2/key" - "github.com/charmbracelet/crush/internal/tui/components/core/layout" ) // KeyMap defines keyboard bindings for dialog management. @@ -18,10 +17,17 @@ func DefaultKeyMap() KeyMap { } } +// KeyBindings implements layout.KeyMapProvider +func (k KeyMap) KeyBindings() []key.Binding { + return []key.Binding{ + k.Close, + } +} + // FullHelp implements help.KeyMap. func (k KeyMap) FullHelp() [][]key.Binding { m := [][]key.Binding{} - slice := layout.KeyMapToSlice(k) + slice := k.KeyBindings() for i := 0; i < len(slice); i += 4 { end := min(i+4, len(slice)) m = append(m, slice[i:end]) diff --git a/internal/tui/components/dialogs/models/keys.go b/internal/tui/components/dialogs/models/keys.go index f17fdb95e366275660287a17a0df2ffc8e375355..f9c3c549c30c6d95282e88818c247ec80a2f0e4c 100644 --- a/internal/tui/components/dialogs/models/keys.go +++ b/internal/tui/components/dialogs/models/keys.go @@ -2,7 +2,6 @@ package models import ( "github.com/charmbracelet/bubbles/v2/key" - "github.com/charmbracelet/crush/internal/tui/components/core/layout" ) type KeyMap struct { @@ -33,10 +32,20 @@ func DefaultKeyMap() KeyMap { } } +// KeyBindings implements layout.KeyMapProvider +func (k KeyMap) KeyBindings() []key.Binding { + return []key.Binding{ + k.Select, + k.Next, + k.Previous, + k.Close, + } +} + // FullHelp implements help.KeyMap. func (k KeyMap) FullHelp() [][]key.Binding { m := [][]key.Binding{} - slice := layout.KeyMapToSlice(k) + slice := k.KeyBindings() for i := 0; i < len(slice); i += 4 { end := min(i+4, len(slice)) m = append(m, slice[i:end]) diff --git a/internal/tui/components/dialogs/permissions/keys.go b/internal/tui/components/dialogs/permissions/keys.go index de4e595ac273e31be661cccec9d8053bf3c7a379..d626eecf9a819cfb209823f922f96dfb58ea3ca4 100644 --- a/internal/tui/components/dialogs/permissions/keys.go +++ b/internal/tui/components/dialogs/permissions/keys.go @@ -2,7 +2,6 @@ package permissions import ( "github.com/charmbracelet/bubbles/v2/key" - "github.com/charmbracelet/crush/internal/tui/components/core/layout" ) type KeyMap struct { @@ -48,10 +47,23 @@ func DefaultKeyMap() KeyMap { } } +// KeyBindings implements layout.KeyMapProvider +func (k KeyMap) KeyBindings() []key.Binding { + return []key.Binding{ + k.Left, + k.Right, + k.Tab, + k.Select, + k.Allow, + k.AllowSession, + k.Deny, + } +} + // FullHelp implements help.KeyMap. func (k KeyMap) FullHelp() [][]key.Binding { m := [][]key.Binding{} - slice := layout.KeyMapToSlice(k) + slice := k.KeyBindings() for i := 0; i < len(slice); i += 4 { end := min(i+4, len(slice)) m = append(m, slice[i:end]) diff --git a/internal/tui/components/dialogs/quit/keys.go b/internal/tui/components/dialogs/quit/keys.go index 20421fb1c2709f515b31b57b2e984e76f02eb071..3268749b20c703ae1faf7640e253ce557f051c65 100644 --- a/internal/tui/components/dialogs/quit/keys.go +++ b/internal/tui/components/dialogs/quit/keys.go @@ -2,7 +2,6 @@ package quit import ( "github.com/charmbracelet/bubbles/v2/key" - "github.com/charmbracelet/crush/internal/tui/components/core/layout" ) // KeyMap defines the keyboard bindings for the quit dialog. @@ -44,10 +43,22 @@ func DefaultKeymap() KeyMap { } } +// KeyBindings implements layout.KeyMapProvider +func (k KeyMap) KeyBindings() []key.Binding { + return []key.Binding{ + k.LeftRight, + k.EnterSpace, + k.Yes, + k.No, + k.Tab, + k.Close, + } +} + // FullHelp implements help.KeyMap. func (k KeyMap) FullHelp() [][]key.Binding { m := [][]key.Binding{} - slice := layout.KeyMapToSlice(k) + slice := k.KeyBindings() for i := 0; i < len(slice); i += 4 { end := min(i+4, len(slice)) m = append(m, slice[i:end]) diff --git a/internal/tui/components/dialogs/sessions/keys.go b/internal/tui/components/dialogs/sessions/keys.go index 1dfdf43f9e841f45cc4ec6d854117bad63a738ba..a3ca4b31f0c04c491fa7990f7e69ac546f608a7d 100644 --- a/internal/tui/components/dialogs/sessions/keys.go +++ b/internal/tui/components/dialogs/sessions/keys.go @@ -2,7 +2,6 @@ package sessions import ( "github.com/charmbracelet/bubbles/v2/key" - "github.com/charmbracelet/crush/internal/tui/components/core/layout" ) type KeyMap struct { @@ -33,10 +32,20 @@ func DefaultKeyMap() KeyMap { } } +// KeyBindings implements layout.KeyMapProvider +func (k KeyMap) KeyBindings() []key.Binding { + return []key.Binding{ + k.Select, + k.Next, + k.Previous, + k.Close, + } +} + // FullHelp implements help.KeyMap. func (k KeyMap) FullHelp() [][]key.Binding { m := [][]key.Binding{} - slice := layout.KeyMapToSlice(k) + slice := k.KeyBindings() for i := 0; i < len(slice); i += 4 { end := min(i+4, len(slice)) m = append(m, slice[i:end]) diff --git a/internal/tui/keys.go b/internal/tui/keys.go index 2c1d0d038b220d3bc97e78b105caea15f7699149..e36e264f36ca720f717874632e2607006140fbb8 100644 --- a/internal/tui/keys.go +++ b/internal/tui/keys.go @@ -2,7 +2,6 @@ package tui import ( "github.com/charmbracelet/bubbles/v2/key" - "github.com/charmbracelet/crush/internal/tui/components/core/layout" ) type KeyMap struct { @@ -42,7 +41,7 @@ func DefaultKeyMap() KeyMap { // FullHelp implements help.KeyMap. func (k KeyMap) FullHelp() [][]key.Binding { m := [][]key.Binding{} - slice := layout.KeyMapToSlice(k) + slice := k.KeyBindings() for i := 0; i < len(slice); i += 4 { end := min(i+4, len(slice)) m = append(m, slice[i:end]) @@ -50,6 +49,17 @@ func (k KeyMap) FullHelp() [][]key.Binding { return m } +// KeyBindings implements layout.KeyMapProvider +func (k KeyMap) KeyBindings() []key.Binding { + return []key.Binding{ + k.Logs, + k.Quit, + k.Help, + k.Commands, + k.Sessions, + } +} + // ShortHelp implements help.KeyMap. func (k KeyMap) ShortHelp() []key.Binding { return []key.Binding{} diff --git a/internal/tui/page/chat/keys.go b/internal/tui/page/chat/keys.go index 9858a9d5821704334cac75dcdbb1d4ca228234bb..61e8023145262fed2c0a6455dd11ba2b228405d7 100644 --- a/internal/tui/page/chat/keys.go +++ b/internal/tui/page/chat/keys.go @@ -2,7 +2,6 @@ package chat import ( "github.com/charmbracelet/bubbles/v2/key" - "github.com/charmbracelet/crush/internal/tui/components/core/layout" ) type KeyMap struct { @@ -33,10 +32,20 @@ func DefaultKeyMap() KeyMap { } } +// KeyBindings implements layout.KeyMapProvider +func (k KeyMap) KeyBindings() []key.Binding { + return []key.Binding{ + k.NewSession, + k.FilePicker, + k.Cancel, + k.Tab, + } +} + // FullHelp implements help.KeyMap. func (k KeyMap) FullHelp() [][]key.Binding { m := [][]key.Binding{} - slice := layout.KeyMapToSlice(k) + slice := k.KeyBindings() for i := 0; i < len(slice); i += 4 { end := min(i+4, len(slice)) m = append(m, slice[i:end]) diff --git a/internal/tui/page/logs/keys.go b/internal/tui/page/logs/keys.go index 7fc25bf4d399bd30946406bb9a7c7bc5b2747fd1..3c612b949183b4180494bdbb8224dd8eedbbb156 100644 --- a/internal/tui/page/logs/keys.go +++ b/internal/tui/page/logs/keys.go @@ -2,7 +2,6 @@ package logs import ( "github.com/charmbracelet/bubbles/v2/key" - "github.com/charmbracelet/crush/internal/tui/components/core/layout" ) type KeyMap struct { @@ -18,10 +17,17 @@ func DefaultKeyMap() KeyMap { } } +// KeyBindings implements layout.KeyMapProvider +func (k KeyMap) KeyBindings() []key.Binding { + return []key.Binding{ + k.Back, + } +} + // FullHelp implements help.KeyMap. func (k KeyMap) FullHelp() [][]key.Binding { m := [][]key.Binding{} - slice := layout.KeyMapToSlice(k) + slice := k.KeyBindings() for i := 0; i < len(slice); i += 4 { end := min(i+4, len(slice)) m = append(m, slice[i:end])