Skip to content

feat(tui): command palette, diff viewer modals with dedicated patch slot#324

Open
Chukwuebuka-2003 wants to merge 4 commits into
huggingface:mainfrom
Chukwuebuka-2003:tui-phase23-polish
Open

feat(tui): command palette, diff viewer modals with dedicated patch slot#324
Chukwuebuka-2003 wants to merge 4 commits into
huggingface:mainfrom
Chukwuebuka-2003:tui-phase23-polish

Conversation

@Chukwuebuka-2003

Copy link
Copy Markdown
  • Add CommandPaletteScreen modal with search filtering over all registered SlashCommand objects, replacing the old /-insertion approach
  • Add DiffViewerScreen modal with syntax-highlighted diff rendering via render_diff_content()
  • Add last_diff slot to TuiState that stores the full patch at record time via _result_patch, replacing heuristic text-scanning
  • Extract render_diff_content() from _render_patch_body for reuse
  • Swap defaults: command_palette to ctrl+p, model_cycle to ctrl+k
  • Add diff_viewer keybinding (ctrl+l) to TuiKeybindings with serialization in to_json()

This PR closes #323

- Add CommandPaletteScreen modal with search filtering over all registered
  SlashCommand objects, replacing the old /-insertion approach
- Add DiffViewerScreen modal with syntax-highlighted diff rendering
  via render_diff_content()
- Add last_diff slot to TuiState that stores the full patch at record
  time via _result_patch, replacing heuristic text-scanning
- Extract render_diff_content() from _render_patch_body for reuse
- Swap defaults: command_palette to ctrl+p, model_cycle to ctrl+k
- Add diff_viewer keybinding (ctrl+l) to TuiKeybindings with
  serialization in to_json()
@Chukwuebuka-2003

Copy link
Copy Markdown
Author

A Preview

image image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat(tui): command palette, diff viewer, dedicated patch slot

1 participant