fix(search): Byte error - attempted to delete negative search input

sqlk/Cargo.toml

@@ -1,7 +1,7 @@
 
 [package]
 name = "sqlk"      
-version = "0.1.4"   
+version = "0.1.5"   
 authors = ["Seth <seth@planetbun.com>"] 
 edition = "2024"        
 description = "A terminal-based PostgreSQL query execution and visualization tool with vim-like navigation." 
@@ -54,4 +54,5 @@ uuid = "1.18.0"
 cli-clipboard = "0.4.0"
 
 [dev-dependencies]
+insta = "1.43.1"
 tempfile = "3.8"

sqlk/src/application/events/mod.rs

@@ -183,3 +183,4 @@ impl App {
         Ok(())
     }
 }
+

sqlk/src/application/modes/mod.rs

@@ -24,20 +24,23 @@ impl App {
             }
             KeyCode::Enter => {
                 if let Some(viewer) = &mut self.table_viewer {
-                    viewer.search( &self.search_input, &mut self.ui);
+                    viewer.search(&self.search_input, &mut self.ui);
                 }
-                self.current_mode = AppMode::TableViewer;
+                    // Correctly reset search input and cursor position
+    self.search_input.clear();
+    self.search_cursor_position = 0;
+    self.current_mode = AppMode::TableViewer;
             }
             KeyCode::Char(c) => {
                 // Insert character at cursor position
                 self.search_input.insert(self.search_cursor_position.into(), c);
                 self.search_cursor_position += 1;
             }
             KeyCode::Backspace => {
-                // Remove character at cursor position
                 if self.search_cursor_position > 0 {
+                    self.search_input
+                        .remove(Into::<usize>::into(self.search_cursor_position) - 1);
                     self.search_cursor_position -= 1;
-                    self.search_input.remove(self.search_cursor_position.into());
                 }
             }
             KeyCode::Esc => {

sqlk/src/application/state/mod.rs

@@ -82,7 +82,7 @@ impl App {
             query_parser: QueryParser::new(),
             query_blocks: Vec::new(),
             is_querying: false,
-            search_cursor_position: 0
+            search_cursor_position: 0,
         };
 
         Ok(app)

sqlk/src/config/db/mod.rs

@@ -66,7 +66,7 @@ impl DatabaseType {
         match self {
             DatabaseType::PostgreSQL => 5432,
             DatabaseType::MySQL => 3306,
-            DatabaseType::SQLite => 0, 
+            DatabaseType::SQLite => 0,
         }
     }
 }

sqlk/src/config/loader/mod.rs

@@ -29,7 +29,7 @@ impl ConfigLoader {
         let config_path = self.config_dir.join("config.toml");
         if config_path.exists() {
             config = self.load_config_file(&config_path)?;
-            config.env_file = env_file.to_path_buf(); 
+            config.env_file = env_file.to_path_buf();
         }
 
         config.database = self.load_database_config(env_file)?;

sqlk/src/config/parser/mod.rs

@@ -18,7 +18,13 @@ impl DatabaseUrlParser {
     fn extract_value(value_part: &str) -> Option<String> {
         let trimmed = value_part.trim();
 
-        if trimmed.starts_with('"') && trimmed.ends_with('"') && trimmed.len() >= 2 && trimmed.starts_with('\'') && trimmed.ends_with('\'') && trimmed.len() >= 2 {
+        if trimmed.starts_with('"')
+            && trimmed.ends_with('"')
+            && trimmed.len() >= 2
+            && trimmed.starts_with('\'')
+            && trimmed.ends_with('\'')
+            && trimmed.len() >= 2
+        {
             Some(trimmed[1..trimmed.len() - 1].to_string())
         } else if !trimmed.is_empty() {
             Some(trimmed.to_string())

sqlk/src/database/postgres/formatting.rs

@@ -44,9 +44,7 @@ impl PostgresFormatter {
             .columns()
             .iter()
             .enumerate()
-            .map(|(idx, _col)| {
-                self.get_column_type_from_rows(&rows, idx)
-            })
+            .map(|(idx, _col)| self.get_column_type_from_rows(&rows, idx))
             .collect();
 
         let mut result_rows = Vec::new();

sqlk/src/main.rs

@@ -15,3 +15,4 @@ async fn main() -> Result<()> {
 
     Ok(())
 }
+

sqlk/src/matrix/colours/mod.rs

@@ -55,7 +55,7 @@ impl ColorCalculator {
                 let b = b1 as f32 + (b2 as f32 - b1 as f32) * factor;
                 Color::Rgb(r as u8, g as u8, b as u8)
             }
-            _ => from, 
+            _ => from,
         }
     }
 }

sqlk/src/table_viewer/chart/mod.rs

@@ -66,8 +66,7 @@ impl TableViewer {
                 let mut temp_items: Vec<_> =
                     sorted_items.iter().take(max_items - 1).cloned().collect();
 
-                if let Some(pos) = temp_items.iter().position(|(l, _)| l == value_to_highlight)
-                {
+                if let Some(pos) = temp_items.iter().position(|(l, _)| l == value_to_highlight) {
                     let highlighted = temp_items.remove(pos);
                     temp_items.insert(0, highlighted);
                 }
@@ -144,7 +143,7 @@ impl TableViewer {
             self.show_chart = false;
             self.chart_data = None;
         } else {
-            let max_items = 20; 
+            let max_items = 20;
 
             let value_to_highlight = self.get_current_cell_value().unwrap_or_default();
 
@@ -169,7 +168,7 @@ impl TableViewer {
             ));
             lines.push("├─────────────────────────────────────────┤".to_string());
 
-            let label_width = width.saturating_sub(25).max(10); 
+            let label_width = width.saturating_sub(25).max(10);
             let bar_char = "█";
 
             for item in &chart_data.items {

sqlk/src/table_viewer/navigation/mod.rs

@@ -51,7 +51,7 @@ impl TableViewer {
             KeyCode::Char('N') => self.prev_search_match(1),
 
             KeyCode::Char('c') => {
-                let bar_width = 30; 
+                let bar_width = 30;
                 self.toggle_chart(bar_width);
             }
 

sqlk/src/table_viewer/search/mod.rs

@@ -1,6 +1,9 @@
 use regex::Regex;
 
-use crate::{table_viewer::{CellPosition, TableViewer}, ui::{ToastType, UI}};
+use crate::{
+    table_viewer::{CellPosition, TableViewer},
+    ui::{ToastType, UI},
+};
 
 #[derive(Debug, Default)]
 pub struct SearchState {
@@ -22,7 +25,7 @@ impl TableViewer {
         let Ok(re) = Regex::new(&format!("(?i){}", term)) else {
             return;
         };
-        
+
         for (row_idx, row) in self.data.rows.iter().enumerate() {
             for (col_idx, cell) in row.iter().enumerate() {
                 if re.is_match(cell) {
@@ -34,7 +37,10 @@ impl TableViewer {
             }
         }
 
-        ui.add_toast(format!("Found {:?} matches", self.search_state.matches.iter().len()), ToastType::Info);
+        ui.add_toast(
+            format!("Found {:?} matches", self.search_state.matches.iter().len()),
+            ToastType::Info,
+        );
 
         if !self.search_state.matches.is_empty() {
             self.jump_to_match(0);

sqlk/src/ui/home/mod.rs

@@ -126,7 +126,7 @@ impl UI {
                         .add_modifier(Modifier::BOLD),
                 )),
                 status_line,
-                Line::from(""), 
+                Line::from(""),
                 Line::from(Span::styled(
                     "Lines:",
                     Style::default()

sqlk/src/ui/search/mod.rs

@@ -80,13 +80,11 @@ impl UI {
         f.render_widget(help_text, chunks[2]);
 
         // Position the cursor at the end of the input text
-        f.set_cursor_position(
-            Position {
+        f.set_cursor_position(Position {
             // x-coordinate: start of the block + border + text length
             x: chunks[1].x + app.search_cursor_position + 1,
             // y-coordinate: start of the block + top border
             y: chunks[1].y + 1,
-            }
-        );
+        });
     }
 }