Add configurable TUI key bindings via config file (#6)

## Summary

- Add `[keys]` section to TOML config for overriding TUI key bindings
- Action names are context-prefixed (`cursor_`, `grid_`, `detail_`,
`search_`, `global_`) for clarity
- Single string or array of strings supported; only overridden keys
change, others keep defaults
- Help text auto-updates to reflect configured bindings

## Implementation

- `KeysConfig` type with custom `UnmarshalTOML` normalizing single
strings to `[]string`
- `ApplyKeysConfig()` maps config action names to `KeyMap` fields via
registry
- `NewModel` accepts optional `KeysConfig` parameter
- Config merge overlays per-action (child overrides parent)

## Test plan

- [x] Config parsing tests: single string, array, mixed, no `[keys]`
section, merge overlay/nil
- [x] `ApplyKeysConfig` tests: single/multiple actions, unknown ignored,
nil config, help text updated, all 16 actions recognized
- [x] All existing tests updated and passing
- [x] `mise run lint` — 0 issues

Closes #3

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>

Author: seletz

.odoo-work-cli.toml

@@ -26,3 +26,30 @@ filters = [
     { field = "company_id.name", op = "=", value = "digitalgedacht GmbH" },
     { field = "stage_id.name", op = "=", value = "Umsetzung" },
 ]
+
+[keys]
+# Cursor movement (shared across grid, detail, search views)
+cursor_up = ["up", "k"]
+cursor_down = ["down", "j"]
+
+# Grid view
+grid_next_col = ["right", "l"]
+grid_prev_col = ["left", "h"]
+grid_enter = ["enter"]
+grid_search = ["/"]
+
+# Detail view
+detail_edit = ["e"]
+detail_add = ["a"]
+detail_delete = ["d"]
+
+# Search view
+search_toggle = ["ctrl+a"]
+
+# Global (available in all non-modal views)
+global_prev_week = ["p"]
+global_next_week = ["n"]
+global_back = ["esc"]
+global_refresh = ["r"]
+global_help = ["?"]
+global_quit = ["q", "ctrl+c"]

cmd/odoo-work-cli/main.go

@@ -582,7 +582,7 @@ var tuiCmd = &cobra.Command{
 			return err
 		}
 
-		m := tui.NewModel(client, tui.MondayTime{Time: monday}, cfg.Hours, cfg.Bundesland)
+		m := tui.NewModel(client, tui.MondayTime{Time: monday}, cfg.Hours, cfg.Bundesland, cfg.Keys)
 		p := tea.NewProgram(m)
 		_, err = p.Run()
 		return err

internal/config/config.go

@@ -7,6 +7,40 @@ import (
 	"github.com/BurntSushi/toml"
 )
 
+// KeysConfig maps action names to key binding strings.
+// Each action maps to one or more key strings (e.g. "q", "ctrl+c").
+type KeysConfig map[string][]string
+
+// UnmarshalTOML normalizes TOML values: a single string becomes a
+// one-element slice, and an array of strings stays as-is.
+func (kc *KeysConfig) UnmarshalTOML(data interface{}) error {
+	m, ok := data.(map[string]interface{})
+	if !ok {
+		return fmt.Errorf("keys: expected table, got %T", data)
+	}
+	result := make(KeysConfig, len(m))
+	for action, val := range m {
+		switch v := val.(type) {
+		case string:
+			result[action] = []string{v}
+		case []interface{}:
+			keys := make([]string, 0, len(v))
+			for _, elem := range v {
+				s, ok := elem.(string)
+				if !ok {
+					return fmt.Errorf("keys.%s: expected string element, got %T", action, elem)
+				}
+				keys = append(keys, s)
+			}
+			result[action] = keys
+		default:
+			return fmt.Errorf("keys.%s: expected string or array, got %T", action, val)
+		}
+	}
+	*kc = result
+	return nil
+}
+
 // ConfigReader provides access to configuration values.
 type ConfigReader interface {
 	OdooURL() string
@@ -64,6 +98,7 @@ type Config struct {
 	Password   string                 `toml:"-"`
 	Models     map[string]ModelConfig `toml:"models"`
 	Hours      HoursLimits            `toml:"hours"`
+	Keys       KeysConfig             `toml:"keys"`
 	Bundesland string                 `toml:"bundesland"` // German federal state for holidays (e.g. "Bayern")
 }
 
@@ -175,6 +210,14 @@ func (c *Config) Merge(other *Config) {
 	if other.Hours.WeeklyHigh != 0 {
 		c.Hours.WeeklyHigh = other.Hours.WeeklyHigh
 	}
+	if other.Keys != nil {
+		if c.Keys == nil {
+			c.Keys = make(KeysConfig)
+		}
+		for action, keys := range other.Keys {
+			c.Keys[action] = keys
+		}
+	}
 	if other.Models != nil {
 		if c.Models == nil {
 			c.Models = make(map[string]ModelConfig)

internal/config/config_test.go

@@ -563,6 +563,151 @@ func TestMerge_HoursLimits(t *testing.T) {
 	}
 }
 
+func TestKeysConfig_UnmarshalTOML_SingleString(t *testing.T) {
+	content := `
+[keys]
+quit = "q"
+edit = "e"
+`
+	dir := t.TempDir()
+	path := filepath.Join(dir, "config.toml")
+	if err := os.WriteFile(path, []byte(content), 0o644); err != nil {
+		t.Fatal(err)
+	}
+
+	cfg, err := LoadFromTOML(path)
+	if err != nil {
+		t.Fatalf("unexpected error: %v", err)
+	}
+	if cfg.Keys == nil {
+		t.Fatal("Keys is nil")
+	}
+	if got := cfg.Keys["quit"]; len(got) != 1 || got[0] != "q" {
+		t.Errorf("Keys[quit] = %v, want [q]", got)
+	}
+	if got := cfg.Keys["edit"]; len(got) != 1 || got[0] != "e" {
+		t.Errorf("Keys[edit] = %v, want [e]", got)
+	}
+}
+
+func TestKeysConfig_UnmarshalTOML_Array(t *testing.T) {
+	content := `
+[keys]
+quit = ["q", "ctrl+c"]
+up = ["up", "k"]
+`
+	dir := t.TempDir()
+	path := filepath.Join(dir, "config.toml")
+	if err := os.WriteFile(path, []byte(content), 0o644); err != nil {
+		t.Fatal(err)
+	}
+
+	cfg, err := LoadFromTOML(path)
+	if err != nil {
+		t.Fatalf("unexpected error: %v", err)
+	}
+	if got := cfg.Keys["quit"]; len(got) != 2 || got[0] != "q" || got[1] != "ctrl+c" {
+		t.Errorf("Keys[quit] = %v, want [q ctrl+c]", got)
+	}
+	if got := cfg.Keys["up"]; len(got) != 2 || got[0] != "up" || got[1] != "k" {
+		t.Errorf("Keys[up] = %v, want [up k]", got)
+	}
+}
+
+func TestKeysConfig_UnmarshalTOML_Mixed(t *testing.T) {
+	content := `
+[keys]
+quit = ["q", "ctrl+c"]
+edit = "e"
+`
+	dir := t.TempDir()
+	path := filepath.Join(dir, "config.toml")
+	if err := os.WriteFile(path, []byte(content), 0o644); err != nil {
+		t.Fatal(err)
+	}
+
+	cfg, err := LoadFromTOML(path)
+	if err != nil {
+		t.Fatalf("unexpected error: %v", err)
+	}
+	if got := cfg.Keys["quit"]; len(got) != 2 {
+		t.Errorf("Keys[quit] = %v, want 2 elements", got)
+	}
+	if got := cfg.Keys["edit"]; len(got) != 1 || got[0] != "e" {
+		t.Errorf("Keys[edit] = %v, want [e]", got)
+	}
+}
+
+func TestLoadFromTOML_NoKeysSection(t *testing.T) {
+	content := `url = "https://odoo.example.com"`
+	dir := t.TempDir()
+	path := filepath.Join(dir, "config.toml")
+	if err := os.WriteFile(path, []byte(content), 0o644); err != nil {
+		t.Fatal(err)
+	}
+
+	cfg, err := LoadFromTOML(path)
+	if err != nil {
+		t.Fatalf("unexpected error: %v", err)
+	}
+	if cfg.Keys != nil {
+		t.Errorf("Keys = %v, want nil when [keys] section absent", cfg.Keys)
+	}
+}
+
+func TestMerge_Keys(t *testing.T) {
+	base := &Config{
+		Keys: KeysConfig{
+			"quit": {"q", "ctrl+c"},
+			"edit": {"e"},
+		},
+	}
+	overlay := &Config{
+		Keys: KeysConfig{
+			"quit": {"ctrl+q"},
+		},
+	}
+
+	base.Merge(overlay)
+
+	if got := base.Keys["quit"]; len(got) != 1 || got[0] != "ctrl+q" {
+		t.Errorf("Keys[quit] = %v, want [ctrl+q] (overlay replaces)", got)
+	}
+	if got := base.Keys["edit"]; len(got) != 1 || got[0] != "e" {
+		t.Errorf("Keys[edit] = %v, want [e] (base preserved)", got)
+	}
+}
+
+func TestMerge_Keys_NilOverlay(t *testing.T) {
+	base := &Config{
+		Keys: KeysConfig{
+			"quit": {"q"},
+		},
+	}
+	overlay := &Config{}
+
+	base.Merge(overlay)
+
+	if got := base.Keys["quit"]; len(got) != 1 || got[0] != "q" {
+		t.Errorf("Keys[quit] = %v, want [q] (nil overlay preserves base)", got)
+	}
+}
+
+func TestMerge_Keys_NilBase(t *testing.T) {
+	base := &Config{}
+	overlay := &Config{
+		Keys: KeysConfig{
+			"quit": {"ctrl+q"},
+		},
+	}
+
+	base.Merge(overlay)
+
+	if got := base.Keys["quit"]; len(got) != 1 || got[0] != "ctrl+q" {
+		t.Errorf("Keys[quit] = %v, want [ctrl+q]", got)
+	}
+}
+
 func TestMerge_FiltersSameFieldOverride(t *testing.T) {
 	base := &Config{
 		Models: map[string]ModelConfig{

internal/tui/keymap.go

@@ -1,6 +1,11 @@
 package tui
 
-import "charm.land/bubbles/v2/key"
+import (
+	"strings"
+
+	"charm.land/bubbles/v2/key"
+	"github.com/seletz/odoo-work-cli/internal/config"
+)
 
 // KeyMap defines key bindings for the TUI.
 type KeyMap struct {
@@ -13,8 +18,6 @@ type KeyMap struct {
 	Refresh      key.Binding
 	Help         key.Binding
 	Quit         key.Binding
-	PrevWeek     key.Binding
-	NextWeek     key.Binding
 	Enter        key.Binding
 	Back         key.Binding
 	Edit         key.Binding
@@ -94,6 +97,85 @@ func DefaultKeyMap() KeyMap {
 	}
 }
 
+// actionHelpDesc maps config action names to their help description text.
+// Action names are prefixed with the context they apply to:
+//   - cursor_  : shared cursor movement (grid, detail, search)
+//   - grid_    : grid view actions
+//   - detail_  : detail view actions
+//   - search_  : search view actions
+//   - global_  : actions available in all non-modal views
+var actionHelpDesc = map[string]string{
+	"cursor_up":        "up",
+	"cursor_down":      "down",
+	"grid_next_col":    "next day",
+	"grid_prev_col":    "prev day",
+	"grid_enter":       "detail",
+	"grid_search":      "search",
+	"detail_edit":      "edit",
+	"detail_add":       "add",
+	"detail_delete":    "delete",
+	"search_toggle":    "toggle filter",
+	"global_quit":      "quit",
+	"global_help":      "help",
+	"global_refresh":   "refresh",
+	"global_back":      "back",
+	"global_prev_week": "prev week",
+	"global_next_week": "next week",
+}
+
+// ApplyKeysConfig overrides key bindings in km from the given config.
+// Unknown action names are silently ignored. Returns the modified KeyMap.
+func ApplyKeysConfig(km KeyMap, cfg config.KeysConfig) KeyMap {
+	if cfg == nil {
+		return km
+	}
+	for action, keys := range cfg {
+		desc, ok := actionHelpDesc[action]
+		if !ok {
+			continue
+		}
+		binding := key.NewBinding(
+			key.WithKeys(keys...),
+			key.WithHelp(strings.Join(keys, "/"), desc),
+		)
+		switch action {
+		case "cursor_up":
+			km.Up = binding
+		case "cursor_down":
+			km.Down = binding
+		case "grid_next_col":
+			km.NextCol = binding
+		case "grid_prev_col":
+			km.PrevCol = binding
+		case "grid_enter":
+			km.Enter = binding
+		case "grid_search":
+			km.Search = binding
+		case "detail_edit":
+			km.Edit = binding
+		case "detail_add":
+			km.Add = binding
+		case "detail_delete":
+			km.Delete = binding
+		case "search_toggle":
+			km.SearchToggle = binding
+		case "global_quit":
+			km.Quit = binding
+		case "global_help":
+			km.Help = binding
+		case "global_refresh":
+			km.Refresh = binding
+		case "global_back":
+			km.Back = binding
+		case "global_prev_week":
+			km.Left = binding
+		case "global_next_week":
+			km.Right = binding
+		}
+	}
+	return km
+}
+
 // ShortHelp returns key bindings for the short help view.
 func (k KeyMap) ShortHelp() []key.Binding {
 	return []key.Binding{k.Up, k.Down, k.NextCol, k.Left, k.Right, k.Enter, k.Edit, k.Add, k.Delete, k.Search, k.Refresh, k.Help, k.Quit}