feat(markdown): add colored headers to table rendering (#506)

- Modified render_table_simple() to accept separate header_style and cell_style parameters
- Table headers now use table_header_text style (bright text with bold modifier)
- Table data cells continue to use table_cell_text style
- Updated renderer.rs to pass the correct styles when rendering markdown tables
- Added test to verify header and cell styling are applied correctly

This ensures that table headers (like 'Aspect', 'Cortex CLI', 'Cursor') are
displayed with distinct styling from the data rows, making tables more readable.

Author: echobt

cortex-core/src/markdown/renderer.rs

@@ -594,8 +594,10 @@ impl<'a> RenderState<'a> {
             table.calculate_column_widths(self.renderer.width);
 
             // Use simple ASCII table format without outer borders
+            // Headers use table_header_text style for colored/bold headers
             let table_lines = render_table_simple(
                 &table,
+                self.renderer.theme.table_header_text,
                 self.renderer.theme.table_cell_text,
                 self.renderer.width,
             );
@@ -863,8 +865,10 @@ impl<'a> RenderState<'a> {
             table.calculate_column_widths(self.renderer.width);
 
             // Use simple ASCII table format without outer borders
+            // Headers use table_header_text style for colored/bold headers
             let table_lines = render_table_simple(
                 &table,
+                self.renderer.theme.table_header_text,
                 self.renderer.theme.table_cell_text,
                 self.renderer.width,
             );

cortex-core/src/markdown/table.rs

@@ -505,12 +505,18 @@ pub fn render_table(
 ///
 /// # Arguments
 /// * `table` - The table to render
-/// * `style` - Style for all text (headers and cells use the same style)
+/// * `header_style` - Style for header text (colored/bold headers)
+/// * `cell_style` - Style for data cell text
 /// * `max_width` - Maximum total width for the table
 ///
 /// # Returns
 /// A vector of `Line`s ready for display in ratatui.
-pub fn render_table_simple(table: &Table, style: Style, max_width: u16) -> Vec<Line<'static>> {
+pub fn render_table_simple(
+    table: &Table,
+    header_style: Style,
+    cell_style: Style,
+    max_width: u16,
+) -> Vec<Line<'static>> {
     // Handle empty table
     if table.is_empty() {
         return Vec::new();
@@ -532,17 +538,22 @@ pub fn render_table_simple(table: &Table, style: Style, max_width: u16) -> Vec<L
 
     let mut lines = Vec::new();
 
-    // Header row (if present)
+    // Header row (if present) - use header_style for colored headers
     if !table.headers.is_empty() {
-        lines.push(render_simple_row(&table.headers, widths, alignments, style));
+        lines.push(render_simple_row(
+            &table.headers,
+            widths,
+            alignments,
+            header_style,
+        ));
 
         // Header separator line: ---+---+---
-        lines.push(render_simple_separator(widths, style));
+        lines.push(render_simple_separator(widths, cell_style));
     }
 
-    // Data rows
+    // Data rows - use cell_style
     for row in &table.rows {
-        lines.push(render_simple_row(row, widths, alignments, style));
+        lines.push(render_simple_row(row, widths, alignments, cell_style));
     }
 
     lines
@@ -1113,7 +1124,7 @@ mod tests {
     #[test]
     fn test_simple_table_empty() {
         let table = Table::default();
-        let lines = render_table_simple(&table, Style::default(), 80);
+        let lines = render_table_simple(&table, Style::default(), Style::default(), 80);
         assert!(lines.is_empty());
     }
 
@@ -1129,7 +1140,7 @@ mod tests {
         builder.end_row();
 
         let table = builder.build();
-        let lines = render_table_simple(&table, Style::default(), 80);
+        let lines = render_table_simple(&table, Style::default(), Style::default(), 80);
 
         // Should have: header, separator, data row = 3 lines (no outer borders)
         assert_eq!(lines.len(), 3);
@@ -1151,7 +1162,7 @@ mod tests {
         builder.end_row();
 
         let table = builder.build();
-        let lines = render_table_simple(&table, Style::default(), 80);
+        let lines = render_table_simple(&table, Style::default(), Style::default(), 80);
 
         // Check that separator line contains + characters
         let separator_content: String = lines[1].spans.iter().map(|s| &*s.content).collect();
@@ -1180,7 +1191,7 @@ mod tests {
         builder.end_row();
 
         let table = builder.build();
-        let lines = render_table_simple(&table, Style::default(), 80);
+        let lines = render_table_simple(&table, Style::default(), Style::default(), 80);
 
         // Should have: 2 data rows only (no header, no separator)
         assert_eq!(lines.len(), 2);
@@ -1200,7 +1211,7 @@ mod tests {
         builder.end_row();
 
         let table = builder.build();
-        let lines = render_table_simple(&table, Style::default(), 80);
+        let lines = render_table_simple(&table, Style::default(), Style::default(), 80);
 
         // First line should be header with | separator
         let header_content: String = lines[0].spans.iter().map(|s| &*s.content).collect();
@@ -1216,4 +1227,44 @@ mod tests {
             "Separator should have + at column intersections"
         );
     }
+
+    #[test]
+    fn test_simple_table_header_styling() {
+        let mut builder = TableBuilder::new();
+        builder.start_header();
+        builder.add_cell("Header1".to_string());
+        builder.add_cell("Header2".to_string());
+        builder.end_header();
+
+        builder.start_row();
+        builder.add_cell("Cell1".to_string());
+        builder.add_cell("Cell2".to_string());
+        builder.end_row();
+
+        let table = builder.build();
+
+        // Use different styles for header and cells
+        let header_style = Style::default().fg(Color::Cyan);
+        let cell_style = Style::default().fg(Color::White);
+
+        let lines = render_table_simple(&table, header_style, cell_style, 80);
+
+        // Header row should have header_style
+        assert!(!lines.is_empty());
+        let header_line = &lines[0];
+        // Check that header spans have the cyan color
+        for span in &header_line.spans {
+            if span.content.contains("Header") {
+                assert_eq!(span.style.fg, Some(Color::Cyan));
+            }
+        }
+
+        // Data row should have cell_style
+        let data_line = &lines[2]; // After header and separator
+        for span in &data_line.spans {
+            if span.content.contains("Cell") {
+                assert_eq!(span.style.fg, Some(Color::White));
+            }
+        }
+    }
 }