canvas library config is now required

client/src/modes/canvas/edit.rs

@@ -145,15 +145,21 @@ async fn execute_canvas_action(
 
 /// NEW: Unified canvas action handler for any CanvasState (LoginState, RegisterState, etc.)
 /// This replaces the old auth_e::execute_edit_action calls with the new canvas library
+/// NEW: Unified canvas action handler for any CanvasState with character fallback
+/// Complete canvas action handler with fallbacks for common keys
+/// Debug version to see what's happening
 async fn handle_canvas_state_edit<S: CanvasState>(
     key: KeyEvent,
     config: &Config,
     state: &mut S,
     ideal_cursor_column: &mut usize,
 ) -> Result<String> {
+    println!("DEBUG: Key pressed: {:?}", key); // DEBUG
+    
     // Try direct key mapping first (same pattern as FormState)
     let canvas_config = canvas::config::CanvasConfig::load();
     if let Some(action_name) = canvas_config.get_edit_action(key.code, key.modifiers) {
+        println!("DEBUG: Canvas config mapped to: {}", action_name); // DEBUG
         let canvas_action = CanvasAction::from_string(action_name);
 
         match ActionDispatcher::dispatch(canvas_action, state, ideal_cursor_column).await {
@@ -170,13 +176,16 @@ async fn handle_canvas_state_edit<S: CanvasState>(
                 return Ok(format!("Context needed: {}", msg));
             }
             Err(_) => {
-                // Fall through to try config mapping
+                println!("DEBUG: Canvas action failed, trying client config"); // DEBUG
             }
         }
+    } else {
+        println!("DEBUG: No canvas config mapping found"); // DEBUG
     }
 
     // Try config-mapped action (same pattern as FormState)
     if let Some(action_str) = config.get_edit_action_for_key(key.code, key.modifiers) {
+        println!("DEBUG: Client config mapped to: {}", action_str); // DEBUG
         let canvas_action = CanvasAction::from_string(&action_str);
         match ActionDispatcher::dispatch(canvas_action, state, ideal_cursor_column).await {
             Ok(ActionResult::Success(msg)) => {
@@ -195,8 +204,34 @@ async fn handle_canvas_state_edit<S: CanvasState>(
                 return Ok(format!("Action failed: {}", e));
             }
         }
+    } else {
+        println!("DEBUG: No client config mapping found"); // DEBUG
     }
 
+    // Character insertion fallback
+    if let KeyCode::Char(c) = key.code {
+        println!("DEBUG: Using character fallback for: {}", c); // DEBUG
+        let canvas_action = CanvasAction::InsertChar(c);
+        match ActionDispatcher::dispatch(canvas_action, state, ideal_cursor_column).await {
+            Ok(ActionResult::Success(msg)) => {
+                return Ok(msg.unwrap_or_default());
+            }
+            Ok(ActionResult::HandledByFeature(msg)) => {
+                return Ok(msg);
+            }
+            Ok(ActionResult::Error(msg)) => {
+                return Ok(format!("Error: {}", msg));
+            }
+            Ok(ActionResult::RequiresContext(msg)) => {
+                return Ok(format!("Context needed: {}", msg));
+            }
+            Err(e) => {
+                return Ok(format!("Character insertion failed: {}", e));
+            }
+        }
+    }
+
+    println!("DEBUG: No action taken for key: {:?}", key); // DEBUG
     Ok(String::new())
 }