refactor(symfony-console): drop the unused Table surface

Table titles, per-column styles and appendRow have no callers: a
plugin-provided command runs inside the worker-side Symfony console
application, so its table rendering resolves against the real PHP
Table rather than this port. Remove those methods together with the
fields they were the only writers of, and the render branches those
fields made unreachable.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

4 files changed, 21 insertions, 330 deletions

diff --git a/crates/shirabe-symfony-console/src/helper/table.rs b/crates/shirabe-symfony-console/src/helper/table.rs
index e2dec0c1..8f82c357 100644
--- a/crates/shirabe-symfony-console/src/helper/table.rs
+++ b/crates/shirabe-symfony-console/src/helper/table.rs
@@ -1,16 +1,11 @@
 //! ref: composer/vendor/symfony/console/Helper/Table.php
 
 use crate::exception::invalid_argument_exception::InvalidArgumentException;
-use crate::exception::runtime_exception::RuntimeException;
 use crate::formatter::output_formatter::OutputFormatter;
 use crate::formatter::wrappable_output_formatter_interface::WrappableOutputFormatterInterface;
-use crate::helper::helper::Helper;
-use crate::helper::table_cell::{TableCell, TableCellOption};
-use crate::helper::table_cell_style::TableCellStyle;
-use crate::helper::table_rows::TableRows;
-use crate::helper::table_separator::TableSeparator;
-use crate::helper::table_style::TableStyle;
-use crate::output::console_section_output::ConsoleSectionOutput;
+use crate::helper::{
+    Helper, TableCell, TableCellOption, TableCellStyle, TableRows, TableSeparator, TableStyle,
+};
 use crate::output::output_interface::OutputInterface;
 use indexmap::IndexMap;
 use shirabe_pcre::preg::Preg;
@@ -228,9 +223,6 @@ fn set_cell(row: &mut Vec<Cell>, index: i64, value: Cell) {
 /// Provides helpers to display a table.
 #[derive(Debug)]
 pub struct Table {
-    header_title: Option<String>,
-    footer_title: Option<String>,
-
     /// Table headers.
     headers: Vec<Row>,
 
@@ -248,13 +240,9 @@ pub struct Table {
 
     style: TableStyle,
 
-    column_styles: IndexMap<i64, TableStyle>,
-
     /// User set column widths.
     column_widths: IndexMap<i64, i64>,
     column_max_widths: IndexMap<i64, i64>,
-
-    rendered: bool,
 }
 
 const SEPARATOR_TOP: i64 = 0;
@@ -282,8 +270,6 @@ impl Table {
         drop(styles_guard);
 
         let mut this = Self {
-            header_title: None,
-            footer_title: None,
             headers: Vec::new(),
             rows: Vec::new(),
             horizontal: false,
@@ -291,10 +277,8 @@ impl Table {
             number_of_columns: None,
             output,
             style: TableStyle::default(),
-            column_styles: IndexMap::new(),
             column_widths: IndexMap::new(),
             column_max_widths: IndexMap::new(),
-            rendered: false,
         };
 
         this.set_style(StyleName::from("default"));
@@ -302,18 +286,8 @@ impl Table {
         this
     }
 
-    /// Sets a style definition.
-    pub fn set_style_definition(name: String, style: TableStyle) {
-        let mut styles_guard = styles().lock().unwrap();
-        if styles_guard.is_none() {
-            *styles_guard = Some(Self::init_styles());
-        }
-
-        styles_guard.as_mut().unwrap().insert(name, style);
-    }
-
     /// Gets a style definition by name.
-    pub fn get_style_definition(
+    pub(crate) fn get_style_definition(
         name: String,
     ) -> anyhow::Result<Result<TableStyle, InvalidArgumentException>> {
         let mut styles_guard = styles().lock().unwrap();
@@ -348,36 +322,10 @@ impl Table {
     }
 
     /// Gets the current table style.
-    pub fn get_style(&self) -> &TableStyle {
+    fn get_style(&self) -> &TableStyle {
         &self.style
     }
 
-    /// Sets table column style.
-    ///
-    /// `$name` is the style name or a TableStyle instance.
-    pub fn set_column_style(
-        &mut self,
-        column_index: i64,
-        name: StyleName,
-    ) -> anyhow::Result<Result<&mut Self, InvalidArgumentException>> {
-        match self.resolve_style(name)? {
-            Ok(style) => {
-                self.column_styles.insert(column_index, style);
-                Ok(Ok(self))
-            }
-            Err(e) => Ok(Err(e)),
-        }
-    }
-
-    /// Gets the current style for a column.
-    ///
-    /// If style was not set, it returns the global table style.
-    pub fn get_column_style(&self, column_index: i64) -> &TableStyle {
-        self.column_styles
-            .get(&column_index)
-            .unwrap_or_else(|| self.get_style())
-    }
-
     /// Sets the minimum width of a column.
     pub fn set_column_width(&mut self, column_index: i64, width: i64) -> &mut Self {
         self.column_widths.insert(column_index, width);
@@ -385,16 +333,6 @@ impl Table {
         self
     }
 
-    /// Sets the minimum width of all columns.
-    pub fn set_column_widths(&mut self, widths: Vec<i64>) -> &mut Self {
-        self.column_widths = IndexMap::new();
-        for (index, width) in widths.into_iter().enumerate() {
-            self.set_column_width(index as i64, width);
-        }
-
-        self
-    }
-
     /// Sets the maximum width of a column.
     ///
     /// Any cell within this column which contents exceeds the specified width will be wrapped into
@@ -433,7 +371,7 @@ impl Table {
         self.add_rows(rows)
     }
 
-    pub fn add_rows(&mut self, rows: Vec<Row>) -> &mut Self {
+    fn add_rows(&mut self, rows: Vec<Row>) -> &mut Self {
         for row in rows {
             self.add_row(row);
         }
@@ -449,50 +387,6 @@ impl Table {
         self
     }
 
-    /// Adds a row to the table, and re-renders the table.
-    pub fn append_row(&mut self, row: Row) -> anyhow::Result<Result<&mut Self, RuntimeException>> {
-        if !Self::output_is_console_section(&self.output) {
-            return Ok(Err(RuntimeException::new(format!(
-                "Output should be an instance of \"{}\" when calling \"{}\".",
-                "Symfony\\Component\\Console\\Output\\ConsoleSectionOutput",
-                "Symfony\\Component\\Console\\Helper\\Table::appendRow",
-            ))));
-        }
-
-        if self.rendered {
-            // TODO(phase-c): downcast output to ConsoleSectionOutput to call clear().
-            let _ = ConsoleSectionOutput::clear;
-            let row_count = self.calculate_row_count();
-            let _ = row_count;
-            todo!()
-        }
-
-        self.add_row(row);
-        self.render();
-
-        Ok(Ok(self))
-    }
-
-    pub fn set_row(&mut self, column: i64, row: Vec<Cell>) -> &mut Self {
-        // PHP indexes $this->rows by arbitrary key; sparse assignment over a positional Vec is not
-        // modeled and has no callers.
-        let _ = (column, row);
-        // TODO(phase-c): sparse `$this->rows[$column] = $row` over a positional row vector.
-        todo!()
-    }
-
-    pub fn set_header_title(&mut self, title: Option<String>) -> &mut Self {
-        self.header_title = title;
-
-        self
-    }
-
-    pub fn set_footer_title(&mut self, title: Option<String>) -> &mut Self {
-        self.footer_title = title;
-
-        self
-    }
-
     pub fn set_horizontal(&mut self, horizontal: bool) -> &mut Self {
         self.horizontal = horizontal;
 
@@ -544,8 +438,6 @@ impl Table {
 
         let mut is_header = !self.horizontal;
         let mut is_first_row = self.horizontal;
-        let mut has_title =
-            self.header_title.is_some() && !self.header_title.as_deref().unwrap_or("").is_empty();
 
         for row_group in &row_groups {
             let mut is_header_separator_rendered = false;
@@ -559,7 +451,7 @@ impl Table {
                 }
 
                 if row.is_table_separator() {
-                    self.render_row_separator(SEPARATOR_MID, None, None);
+                    self.render_row_separator(SEPARATOR_MID);
 
                     continue;
                 }
@@ -569,47 +461,21 @@ impl Table {
                 }
 
                 if is_header && !is_header_separator_rendered {
-                    self.render_row_separator(
-                        if is_header {
-                            SEPARATOR_TOP
-                        } else {
-                            SEPARATOR_TOP_BOTTOM
-                        },
-                        if has_title {
-                            self.header_title.clone()
-                        } else {
-                            None
-                        },
-                        if has_title {
-                            Some(self.style.get_header_title_format())
-                        } else {
-                            None
-                        },
-                    );
-                    has_title = false;
+                    self.render_row_separator(if is_header {
+                        SEPARATOR_TOP
+                    } else {
+                        SEPARATOR_TOP_BOTTOM
+                    });
                     is_header_separator_rendered = true;
                 }
 
                 if is_first_row {
-                    self.render_row_separator(
-                        if is_header {
-                            SEPARATOR_TOP
-                        } else {
-                            SEPARATOR_TOP_BOTTOM
-                        },
-                        if has_title {
-                            self.header_title.clone()
-                        } else {
-                            None
-                        },
-                        if has_title {
-                            Some(self.style.get_header_title_format())
-                        } else {
-                            None
-                        },
-                    );
+                    self.render_row_separator(if is_header {
+                        SEPARATOR_TOP
+                    } else {
+                        SEPARATOR_TOP_BOTTOM
+                    });
                     is_first_row = false;
-                    has_title = false;
                 }
 
                 if self.horizontal {
@@ -631,23 +497,13 @@ impl Table {
                 }
             }
         }
-        self.render_row_separator(
-            SEPARATOR_BOTTOM,
-            self.footer_title.clone(),
-            Some(self.style.get_footer_title_format()),
-        );
+        self.render_row_separator(SEPARATOR_BOTTOM);
 
         self.cleanup();
-        self.rendered = true;
     }
 
     /// Renders horizontal header separator.
-    fn render_row_separator(
-        &self,
-        r#type: i64,
-        title: Option<String>,
-        title_format: Option<String>,
-    ) {
+    fn render_row_separator(&self, r#type: i64) {
         let count = match self.number_of_columns {
             Some(0) | None => return,
             Some(c) => c,
@@ -707,46 +563,6 @@ impl Table {
             column += 1;
         }
 
-        if let Some(title) = title {
-            let title_format = title_format.unwrap();
-            let formatted_title =
-                shirabe_php_shim::sprintf(&title_format, &[PhpMixed::from(title.clone())]);
-            let mut formatted_title = formatted_title;
-            let mut title_length = Helper::width(&self.remove_decoration(&formatted_title));
-            let markup_length = Helper::width(&markup);
-            let limit = markup_length - 4;
-            if title_length > limit {
-                title_length = limit;
-                let format_length = Helper::width(&self.remove_decoration(
-                    &shirabe_php_shim::sprintf(&title_format, &[PhpMixed::from("")]),
-                ));
-                formatted_title = shirabe_php_shim::sprintf(
-                    &title_format,
-                    &[PhpMixed::from(format!(
-                        "{}...",
-                        Helper::substr(&title, 0, Some(limit - format_length - 3))
-                    ))],
-                );
-            }
-
-            let title_start = (markup_length - title_length) / 2;
-            if shirabe_php_shim::mb_detect_encoding(&markup, None, true).is_none() {
-                markup = shirabe_php_shim::substr_replace(
-                    &markup,
-                    &formatted_title,
-                    title_start as usize,
-                    title_length as usize,
-                );
-            } else {
-                markup = format!(
-                    "{}{}{}",
-                    shirabe_php_shim::mb_substr(&markup, 0, Some(title_start), None),
-                    formatted_title,
-                    shirabe_php_shim::mb_substr(&markup, title_start + title_length, None, None),
-                );
-            }
-        }
-
         self.output.borrow().writeln(
             &[shirabe_php_shim::sprintf(
                 &self.style.get_border_format(),
@@ -820,7 +636,7 @@ impl Table {
                 - shirabe_php_shim::mb_strwidth(&cell_str, Some(&encoding));
         }
 
-        let style = self.get_column_style(column);
+        let style = self.get_style();
 
         if cell.is_table_separator() {
             return shirabe_php_shim::sprintf(
@@ -1016,23 +832,6 @@ impl Table {
         TableRows::from_row_groups(row_groups)
     }
 
-    fn calculate_row_count(&mut self) -> i64 {
-        let mut merged = self.headers.clone();
-        merged.push(Row::Separator(TableSeparator::new()));
-        merged.extend(self.rows.clone());
-        let mut number_of_rows = self.build_table_rows(merged).into_row_groups().len() as i64;
-
-        if !self.headers.is_empty() {
-            number_of_rows += 1; // Add row for header separator
-        }
-
-        if !self.rows.is_empty() {
-            number_of_rows += 1; // Add row for footer separator
-        }
-
-        number_of_rows
-    }
-
     /// fill rows that contains rowspan > 1.
     fn fill_next_rows(&self, rows: Vec<Row>, line: i64) -> Vec<Row> {
         let mut rows = rows;
@@ -1406,17 +1205,6 @@ impl Table {
         Helper::remove_decoration(&mut *formatter, string)
     }
 
-    fn output_is_console_section(
-        output: &std::rc::Rc<std::cell::RefCell<dyn OutputInterface>>,
-    ) -> bool {
-        // PHP: $this->output instanceof ConsoleSectionOutput
-        let borrowed = output.borrow();
-        (*borrowed)
-            .as_any()
-            .downcast_ref::<ConsoleSectionOutput>()
-            .is_some()
-    }
-
     fn table_cell_options_colspan(colspan: i64) -> IndexMap<String, TableCellOption> {
         // PHP: ['colspan' => $colspan]
         let mut options = IndexMap::new();
diff --git a/crates/shirabe-symfony-console/src/helper/table_rows.rs b/crates/shirabe-symfony-console/src/helper/table_rows.rs
index 57db5ee8..dea05384 100644
--- a/crates/shirabe-symfony-console/src/helper/table_rows.rs
+++ b/crates/shirabe-symfony-console/src/helper/table_rows.rs
@@ -16,14 +16,6 @@ impl TableRows {
     pub fn from_row_groups(row_groups: Vec<Vec<Row>>) -> Self {
         Self { row_groups }
     }
-
-    pub fn get_iterator(&self) -> std::slice::Iter<'_, Vec<Row>> {
-        self.row_groups.iter()
-    }
-
-    pub fn into_row_groups(self) -> Vec<Vec<Row>> {
-        self.row_groups
-    }
 }
 
 impl<'a> IntoIterator for &'a TableRows {
@@ -34,12 +26,3 @@ impl<'a> IntoIterator for &'a TableRows {
         self.row_groups.iter()
     }
 }
-
-impl IntoIterator for TableRows {
-    type Item = Vec<Row>;
-    type IntoIter = std::vec::IntoIter<Vec<Row>>;
-
-    fn into_iter(self) -> Self::IntoIter {
-        self.row_groups.into_iter()
-    }
-}
diff --git a/crates/shirabe-symfony-console/src/helper/table_separator.rs b/crates/shirabe-symfony-console/src/helper/table_separator.rs
index 99862848..898a6630 100644
--- a/crates/shirabe-symfony-console/src/helper/table_separator.rs
+++ b/crates/shirabe-symfony-console/src/helper/table_separator.rs
@@ -15,9 +15,7 @@ impl TableSeparator {
         Self::new1(IndexMap::new()).expect("TableSeparator default options are always valid")
     }
 
-    pub fn new1(
-        options: IndexMap<String, TableCellOption>,
-    ) -> Result<Self, InvalidArgumentException> {
+    fn new1(options: IndexMap<String, TableCellOption>) -> Result<Self, InvalidArgumentException> {
         Ok(Self {
             inner: TableCell::new("", options)?,
         })
diff --git a/crates/shirabe-symfony-console/src/helper/table_style.rs b/crates/shirabe-symfony-console/src/helper/table_style.rs
index 9566482f..6b0a27a9 100644
--- a/crates/shirabe-symfony-console/src/helper/table_style.rs
+++ b/crates/shirabe-symfony-console/src/helper/table_style.rs
@@ -1,8 +1,5 @@
 //! ref: composer/vendor/symfony/console/Helper/TableStyle.php
 
-use crate::exception::invalid_argument_exception::InvalidArgumentException;
-use crate::exception::logic_exception::LogicException;
-
 /// Defines the styles for a Table.
 #[derive(Debug, Clone)]
 pub struct TableStyle {
@@ -23,8 +20,6 @@ pub struct TableStyle {
     crossing_top_left_bottom_char: String,
     crossing_top_mid_bottom_char: String,
     crossing_top_right_bottom_char: String,
-    header_title_format: String,
-    footer_title_format: String,
     cell_header_format: String,
     cell_row_format: String,
     cell_row_content_format: String,
@@ -52,8 +47,6 @@ impl Default for TableStyle {
             crossing_top_left_bottom_char: "+".to_string(),
             crossing_top_mid_bottom_char: "+".to_string(),
             crossing_top_right_bottom_char: "+".to_string(),
-            header_title_format: "<fg=black;bg=white;options=bold> %s </>".to_string(),
-            footer_title_format: "<fg=black;bg=white;options=bold> %s </>".to_string(),
             cell_header_format: "<info>%s</info>".to_string(),
             cell_row_format: "%s".to_string(),
             cell_row_content_format: " %s ".to_string(),
@@ -64,22 +57,6 @@ impl Default for TableStyle {
 }
 
 impl TableStyle {
-    /// Sets padding character, used for cell padding.
-    pub fn set_padding_char(
-        &mut self,
-        padding_char: String,
-    ) -> anyhow::Result<Result<&mut Self, LogicException>> {
-        if padding_char.is_empty() {
-            return Ok(Err(LogicException::new(
-                "The padding char must not be empty.".to_string(),
-            )));
-        }
-
-        self.padding_char = padding_char;
-
-        Ok(Ok(self))
-    }
-
     /// Gets padding character, used for cell padding.
     pub fn get_padding_char(&self) -> String {
         self.padding_char.clone()
@@ -205,13 +182,6 @@ impl TableStyle {
         self.cell_header_format.clone()
     }
 
-    /// Sets row cell format.
-    pub fn set_cell_row_format(&mut self, cell_row_format: String) -> &mut Self {
-        self.cell_row_format = cell_row_format;
-
-        self
-    }
-
     /// Gets row cell format.
     pub fn get_cell_row_format(&self) -> String {
         self.cell_row_format.clone()
@@ -229,61 +199,13 @@ impl TableStyle {
         self.cell_row_content_format.clone()
     }
 
-    /// Sets table border format.
-    pub fn set_border_format(&mut self, border_format: String) -> &mut Self {
-        self.border_format = border_format;
-
-        self
-    }
-
     /// Gets table border format.
     pub fn get_border_format(&self) -> String {
         self.border_format.clone()
     }
 
-    /// Sets cell padding type.
-    pub fn set_pad_type(
-        &mut self,
-        pad_type: i64,
-    ) -> anyhow::Result<Result<&mut Self, InvalidArgumentException>> {
-        if ![
-            shirabe_php_shim::STR_PAD_LEFT,
-            shirabe_php_shim::STR_PAD_RIGHT,
-            shirabe_php_shim::STR_PAD_BOTH,
-        ]
-        .contains(&pad_type)
-        {
-            return Ok(Err(InvalidArgumentException::new("Invalid padding type. Expected one of (STR_PAD_LEFT, STR_PAD_RIGHT, STR_PAD_BOTH)."
-                .to_string())));
-        }
-
-        self.pad_type = pad_type;
-
-        Ok(Ok(self))
-    }
-
     /// Gets cell padding type.
     pub fn get_pad_type(&self) -> i64 {
         self.pad_type
     }
-
-    pub fn get_header_title_format(&self) -> String {
-        self.header_title_format.clone()
-    }
-
-    pub fn set_header_title_format(&mut self, format: String) -> &mut Self {
-        self.header_title_format = format;
-
-        self
-    }
-
-    pub fn get_footer_title_format(&self) -> String {
-        self.footer_title_format.clone()
-    }
-
-    pub fn set_footer_title_format(&mut self, format: String) -> &mut Self {
-        self.footer_title_format = format;
-
-        self
-    }
 }