Audit: Synchronize keyboard shortcuts and unify Tool implementation with ToolFuture alias (#33)

* Audit: Synchronize keyboard shortcuts and unify Tool implementation with ToolFuture alias

* Update CHANGELOG.md with Documentation Site and Architectural Cleanup details

Author: fam007e

CHANGELOG.md

@@ -23,6 +23,11 @@ All notable changes to this project will be documented in this file.
  - **Optimized Response Handling**: Added 10,000-character truncation to HTTP bodies for memory and rendering efficiency.
  - **Dependency Pruning**: Leaner `Tokio` feature set to reduce binary size and compilation time.
 
+ ### Documentation & DX
+ - **Documentation Site**: Automated `mdBook` deployment to GitHub Pages via CI.
+ - **Shortcut Audit**: Synchronized and documented all global and tool-specific keyboard navigation.
+ - **Architectural Cleanup**: Simplified tool implementation using the unified `ToolFuture` type alias.
+ 
  ## [v2026.02.28] - 2026-02-28
 
 ### Features

Cargo.lock

@@ -69,10 +69,11 @@ You can modify this file to change database paths or API URLs.
 ## Navigation
 
 - **Keyboard:**
-  - `Tab`: Switch between tool tabs.
-  - `Ctrl+F`: Open global search palette.
-  - `Ctrl+C`: Intercepted by some tools for clipboard operations (e.g., copying results). Use `Esc` or `q` to exit tools where applicable.
-  - **Arrow Keys, Enter, etc.:** Used for interacting with the currently selected tool.
+   - `Tab`: Switch between tool tabs.
+   - `Ctrl+F`: Open global search palette.
+   - `Ctrl+Q`: Quit the application.
+   - `Ctrl+C`: Copy the current status message (bottom left) to the clipboard.
+   - **Arrow Keys, Enter, etc.:** Used for interacting with the currently selected tool.
 - **Mouse / Touchpad:**
   - **Click / Tap:** Select a tab to switch to that tool.
 

README.md

@@ -3,9 +3,6 @@ use ratatui::{
     prelude::*,
     widgets::{Block, Borders, Paragraph},
 };
-use std::error::Error;
-use std::future::Future;
-use std::pin::Pin;
 
 use base64::{engine::general_purpose::STANDARD as b64, Engine as _};
 use hex;
@@ -147,10 +144,7 @@ impl super::Tool for EncoderDecoderTool {
         f.render_widget(result_para, chunks[2]);
     }
 
-    fn handle_input(
-        &mut self,
-        key: KeyEvent,
-    ) -> Pin<Box<dyn Future<Output = Result<String, Box<dyn Error>>> + Send + '_>> {
+    fn handle_input(&mut self, key: KeyEvent) -> crate::tools::ToolFuture<'_> {
         Box::pin(async move {
             match key.code {
                 KeyCode::Char('m') if key.modifiers.contains(KeyModifiers::CONTROL) => {

src/tools/encoder_decoder.rs

@@ -5,8 +5,6 @@ use ratatui::{
 };
 use reqwest::{Client, Method};
 use std::error::Error;
-use std::future::Future;
-use std::pin::Pin;
 
 #[derive(Debug, Clone, Copy, PartialEq)]
 enum HttpMethod {
@@ -174,10 +172,7 @@ impl super::Tool for HttpRequestInspectorTool {
         f.render_widget(result_para, chunks[1]);
     }
 
-    fn handle_input(
-        &mut self,
-        key: KeyEvent,
-    ) -> Pin<Box<dyn Future<Output = Result<String, Box<dyn Error>>> + Send + '_>> {
+    fn handle_input(&mut self, key: KeyEvent) -> crate::tools::ToolFuture<'_> {
         Box::pin(async move {
             match key.code {
                 KeyCode::Char('m') if key.modifiers.contains(KeyModifiers::CONTROL) => {

src/tools/http_inspector.rs

@@ -46,9 +46,6 @@ impl JwtDecoderTool {
     }
 }
 
-use std::future::Future;
-use std::pin::Pin;
-
 impl super::Tool for JwtDecoderTool {
     fn name(&self) -> &'static str {
         "JWT Decoder"
@@ -121,10 +118,7 @@ impl super::Tool for JwtDecoderTool {
         f.render_widget(results, chunks[1]);
     }
 
-    fn handle_input(
-        &mut self,
-        key: KeyEvent,
-    ) -> Pin<Box<dyn Future<Output = Result<String, Box<dyn Error>>> + Send + '_>> {
+    fn handle_input(&mut self, key: KeyEvent) -> crate::tools::ToolFuture<'_> {
         Box::pin(async move {
             match key.code {
                 KeyCode::Char(c) => {

src/tools/jwt_decoder.rs

@@ -1,14 +1,14 @@
 use crate::models::github::Organization;
 use crate::secrets::Secrets;
 use reqwest::Client;
+use std::error::Error;
 
 use crossterm::event::{KeyCode, KeyEvent, KeyModifiers};
 use ratatui::{
     prelude::*,
     widgets::{Block, Borders, Paragraph},
 };
 use serde_json;
-use std::error::Error;
 
 pub struct OrgResearchTool {
     input: InputState,
@@ -80,9 +80,6 @@ impl OrgResearchTool {
     }
 }
 
-use std::future::Future;
-use std::pin::Pin;
-
 impl super::Tool for OrgResearchTool {
     fn name(&self) -> &'static str {
         "Org Research"
@@ -174,10 +171,7 @@ impl super::Tool for OrgResearchTool {
         f.render_widget(results, chunks[results_idx]);
     }
 
-    fn handle_input(
-        &mut self,
-        key: KeyEvent,
-    ) -> Pin<Box<dyn Future<Output = Result<String, Box<dyn Error>>> + Send + '_>> {
+    fn handle_input(&mut self, key: KeyEvent) -> crate::tools::ToolFuture<'_> {
         Box::pin(async move {
             match key.code {
                 KeyCode::Up => {

src/tools/org_research.rs

@@ -90,9 +90,6 @@ impl RepoExplorerTool {
     }
 }
 
-use std::future::Future;
-use std::pin::Pin;
-
 impl super::Tool for RepoExplorerTool {
     fn name(&self) -> &'static str {
         "Repo Explorer"
@@ -159,10 +156,7 @@ impl super::Tool for RepoExplorerTool {
         f.render_widget(results, chunks[results_idx]);
     }
 
-    fn handle_input(
-        &mut self,
-        key: KeyEvent,
-    ) -> Pin<Box<dyn Future<Output = Result<String, Box<dyn Error>>> + Send + '_>> {
+    fn handle_input(&mut self, key: KeyEvent) -> crate::tools::ToolFuture<'_> {
         Box::pin(async move {
             match key.code {
                 KeyCode::Enter => self.fetch_repos().await,

src/tools/repo_explorer.rs

@@ -6,8 +6,6 @@ use ratatui::{
 };
 use reqwest::Client;
 use std::error::Error;
-use std::future::Future;
-use std::pin::Pin;
 
 use std::sync::{Arc, Mutex};
 
@@ -241,10 +239,7 @@ impl super::Tool for TokenInspectorTool {
         f.render_widget(results, chunks[1]);
     }
 
-    fn handle_input(
-        &mut self,
-        key: KeyEvent,
-    ) -> Pin<Box<dyn Future<Output = Result<String, Box<dyn Error>>> + Send + '_>> {
+    fn handle_input(&mut self, key: KeyEvent) -> crate::tools::ToolFuture<'_> {
         // We need to clone state for the async move block
         let mut tool_clone = self.clone_state();
         Box::pin(async move {

src/tools/token_inspector.rs

@@ -13,8 +13,6 @@ use ratatui::{
 use unicode_segmentation::UnicodeSegmentation;
 
 use crate::tools::LoadState;
-use std::future::Future;
-use std::pin::Pin;
 
 pub struct UnicodeInspectorTool {
     input: InputState,
@@ -270,10 +268,7 @@ impl super::Tool for UnicodeInspectorTool {
         f.render_widget(results, chunks[4]);
     }
 
-    fn handle_input(
-        &mut self,
-        key: KeyEvent,
-    ) -> Pin<Box<dyn Future<Output = Result<String, Box<dyn Error>>> + Send + '_>> {
+    fn handle_input(&mut self, key: KeyEvent) -> crate::tools::ToolFuture<'_> {
         Box::pin(async move {
             match key.code {
                 KeyCode::Up => {