sql-cli 1.72.0

SQL query tool for CSV/JSON with both interactive TUI and non-interactive CLI modes - perfect for exploration and automation
Documentation 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109

# Column Operations Refactoring - Complete

## Summary
Successfully refactored all 4 column operation methods in `enhanced_tui.rs` following the same pattern as the navigation methods simplification. This reduces code complexity and prepares the methods for trait extraction.

## Changes Made

### 1. New Structures Added
- **`ColumnOperationResult`** struct in `viewport_manager.rs` - Unified result type for all column operations
- **`apply_column_operation_result()`** helper in `enhanced_tui.rs` - Single point for state synchronization

### 2. Methods Refactored

All 4 methods now follow the ~8-line pattern:

#### Before (example: hide_current_column)
- **75 lines** of complex logic
- Direct ViewportManager manipulation
- Inline state updates
- Multiple nested conditions

#### After
```rust
pub fn hide_current_column(&mut self) {
    if self.buffer().get_mode() != AppMode::Results {
        return;
    }
    
    let result = self.viewport_manager.borrow_mut()
        .as_mut()
        .map(|vm| vm.hide_current_column_with_result())
        .unwrap_or_else(|| ColumnOperationResult::failure("No viewport manager"));
    
    self.apply_column_operation_result(result);
}
```

### 3. ViewportManager Methods Added
- `hide_current_column_with_result()`
- `unhide_all_columns_with_result()`
- `reorder_column_left_with_result()`
- `reorder_column_right_with_result()`

## Results

### Code Reduction
- **Before**: ~200 lines across 4 methods
- **After**: ~50 lines (4 methods × ~12 lines + helper)
- **Reduction**: ~75% fewer lines

### Benefits Achieved
✅ **Consistency** - All column operations follow same pattern
✅ **Maintainability** - Single point for state updates
✅ **Testability** - Clean result structures easy to test
✅ **Trait Ready** - Minimal dependencies for extraction
✅ **No Regressions** - All 198 tests still pass

## Pattern Established

```rust
// Standard column operation pattern
fn column_operation(&mut self) {
    // 1. Mode check if needed
    if self.buffer().get_mode() != AppMode::Results {
        return;
    }
    
    // 2. Delegate to ViewportManager
    let result = self.viewport_manager.borrow_mut()
        .as_mut()
        .map(|vm| vm.operation_with_result())
        .unwrap_or_else(|| ColumnOperationResult::failure("No viewport manager"));
    
    // 3. Apply result through helper
    self.apply_column_operation_result(result);
}
```

## Testing
- ✅ `cargo build --release` - Builds successfully
- ✅ `cargo fmt` - Code formatted
- ✅ `cargo test` - All 198 tests pass
- ✅ Column operation tests pass

## Next Steps

With both navigation and column operations simplified:

1. **Extract Navigation Trait**
   - Move navigation methods to trait
   - Impl trait for EnhancedTui
   
2. **Extract Column Operations Trait**
   - Move column methods to trait
   - Impl trait for EnhancedTui

3. **Consider Yank Operations**
   - Analyze yank methods
   - Apply same simplification pattern
   - Extract to trait

4. **Module Organization**
   - `traits/navigation.rs`
   - `traits/column_ops.rs`
   - `traits/yank.rs`

## Conclusion

The column operations refactoring is complete and follows the same successful pattern as the navigation simplification. The TUI is becoming more modular with clear separation of concerns, making it easier to maintain and extend while preserving all existing functionality.