Delegate SQLQueryView string-like columns to column_as_string

The view duplicated the Enum8/Enum16 name resolution and UUID formatting
that column_as_string already does for the interpreter readers. Teach
column_as_string to format UUID too and fold the view's String,
LowCardinality(String), Enum8/16 and UUID arms into one fallback that
calls it. Unsupported types now surface an error from the String read
instead of unreachable!() panicking the TUI.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: azat

src/interpreter/clickhouse.rs

@@ -23,9 +23,10 @@ use std::str::FromStr;
 
 pub type Columns = Block<Complex>;
 
-// clickhouse-rs reads an Enum column as its bare integer (FromSql drops the name mapping), so
-// block.get::<String> fails on it; resolve the name from the Vec carried by the column type.
-// String and LowCardinality(String) fall through to a plain String read (get coerces LC).
+// clickhouse-rs reads an Enum column as its bare integer (FromSql drops the name mapping) and a
+// UUID as uuid::Uuid, so block.get::<String> fails on both; resolve the enum name from the Vec
+// carried by the column type and format the UUID. String and LowCardinality(String) fall through
+// to a plain String read (get coerces LC).
 pub fn column_as_string<K: ColumnType>(block: &Block<K>, row: usize, name: &str) -> Result<String> {
     let column = block
         .columns()
@@ -41,6 +42,7 @@ pub fn column_as_string<K: ColumnType>(block: &Block<K>, row: usize, name: &str)
     Ok(match column.sql_type() {
         SqlType::Enum8(values) => name_of(&values, block.get::<Enum8, _>(row, name)?.internal()),
         SqlType::Enum16(values) => name_of(&values, block.get::<Enum16, _>(row, name)?.internal()),
+        SqlType::Uuid => block.get::<uuid::Uuid, _>(row, name)?.to_string(),
         _ => block.get::<String, _>(row, name)?,
     })
 }

src/view/sql_query_view.rs

@@ -4,13 +4,16 @@ use std::sync::{Arc, Mutex};
 use anyhow::{Result, anyhow};
 use size::{Base, SizeFormatter, Style};
 
-use crate::interpreter::{BackgroundRunner, ContextArc, WorkerEvent, clickhouse::Columns};
+use crate::interpreter::{
+    BackgroundRunner, ContextArc, WorkerEvent,
+    clickhouse::{Columns, column_as_string},
+};
 use crate::view::TableViewItem;
 use crate::view::table_view::TableView;
 use crate::wrap_impl_no_move;
 use chrono::{DateTime, Local};
 use chrono_tz::Tz;
-use clickhouse_rs::types::{Enum8, Enum16, SqlType};
+use clickhouse_rs::types::SqlType;
 use cursive::Cursive;
 use cursive::theme::Color;
 use cursive::utils::markup::StyledString;
@@ -230,31 +233,6 @@ impl SQLQueryView {
                     .find(|c| c.name() == column)
                     .ok_or(anyhow!("Cannot get {} column", column))?;
                 let field = match sql_column.sql_type() {
-                    SqlType::String => Field::String(block.get::<_, _>(i, column)?),
-                    // block.get coerces LowCardinality to its inner type
-                    SqlType::LowCardinality(SqlType::String) => {
-                        Field::String(block.get::<_, _>(i, column)?)
-                    }
-                    SqlType::Uuid => {
-                        Field::String(block.get::<uuid::Uuid, _>(i, column)?.to_string())
-                    }
-                    // Enum values only carry the integer; the name mapping lives in the type
-                    SqlType::Enum8(values) => {
-                        let v = block.get::<Enum8, _>(i, column)?.internal();
-                        let name = values
-                            .iter()
-                            .find(|(_, k)| *k == v)
-                            .map_or_else(|| v.to_string(), |(name, _)| name.clone());
-                        Field::String(name)
-                    }
-                    SqlType::Enum16(values) => {
-                        let v = block.get::<Enum16, _>(i, column)?.internal();
-                        let name = values
-                            .iter()
-                            .find(|(_, k)| *k == v)
-                            .map_or_else(|| v.to_string(), |(name, _)| name.clone());
-                        Field::String(name)
-                    }
                     SqlType::Float64 => Field::Float64(block.get::<_, _>(i, column)?),
                     SqlType::Float32 => Field::Float32(block.get::<_, _>(i, column)?),
                     SqlType::UInt64 => Field::UInt64(block.get::<_, _>(i, column)?),
@@ -270,7 +248,8 @@ impl SQLQueryView {
                             .get::<DateTime<Tz>, _>(i, column)?
                             .with_timezone(&Local),
                     ),
-                    _ => unreachable!("Type for column {} not implemented", column),
+                    // String, LowCardinality(String), Enum8/16 and UUID all render as text
+                    _ => Field::String(column_as_string(&block, i, column)?),
                 };
                 row.0.push(field);
             }