fix(quality): resolve clippy warnings for 2.0 release

- Fix Cargo.toml: empty_enums → empty_enum (correct lint name)
- Fix CellBuffer Debug impl: use finish_non_exhaustive()
- Fix doc comments: use backticks for code in docs
- Fix doc formatting: proper paragraph separation
- Auto-fix uninlined_format_args across computebrick/*.rs

Quality metrics:
- Clippy: 0 errors, 0 warnings
- Tests: 16,102 passed, 0 failed

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

Author: noahgift

Cargo.toml

@@ -38,7 +38,7 @@ cloned_instead_of_copied = "warn"
 cognitive_complexity = "allow"  # Test functions can be complex
 default_trait_access = "allow"  # Tests can use Default::default() explicitly
 doc_markdown = "warn"
-empty_enums = "warn"
+empty_enum = "warn"
 enum_variant_names = "warn"
 expl_impl_clone_on_copy = "warn"
 explicit_deref_methods = "warn"

src/computebrick/api.rs

@@ -51,7 +51,7 @@ pub fn plot(
 /// * `value` - Current value
 /// * `max` - Maximum value
 /// * `label` - Optional label
-/// * `gradient` - Optional gradient colors ["start", "end"]
+/// * `gradient` - Optional gradient colors `["start", "end"]`
 ///
 /// # Example
 /// ```ruchy

src/computebrick/canvas.rs

@@ -410,7 +410,7 @@ impl fmt::Debug for CellBuffer {
         f.debug_struct("CellBuffer")
             .field("width", &self.width)
             .field("height", &self.height)
-            .finish()
+            .finish_non_exhaustive()
     }
 }
 
@@ -506,7 +506,7 @@ impl Canvas for CellBuffer {
     }
 
     fn draw_braille(&mut self, x: f32, y: f32, pattern: u8, color: Color) {
-        let ch = char::from_u32(0x2800 + pattern as u32).unwrap_or(' ');
+        let ch = char::from_u32(0x2800 + u32::from(pattern)).unwrap_or(' ');
         self.set_char(x as usize, y as usize, ch, color, Color::TRANSPARENT);
     }
 }

src/computebrick/render.rs

@@ -10,7 +10,7 @@ use std::io::{self, Write};
 pub enum RenderBackend {
     /// Direct terminal via crossterm.
     Terminal,
-    /// WebAssembly Canvas2D.
+    /// WebAssembly `Canvas2D`.
     WasmCanvas,
     /// WebAssembly WebGL.
     WasmWebGL,
@@ -216,11 +216,11 @@ impl DiffRenderer {
         match self.color_mode {
             ColorMode::TrueColor => {
                 let (r, g, b) = color.to_rgb8();
-                write!(output, "\x1b[38;2;{};{};{}m", r, g, b)?;
+                write!(output, "\x1b[38;2;{r};{g};{b}m")?;
             }
             ColorMode::Color256 => {
                 let idx = color_to_256(color);
-                write!(output, "\x1b[38;5;{}m", idx)?;
+                write!(output, "\x1b[38;5;{idx}m")?;
             }
             ColorMode::Color16 | ColorMode::Color8 => {
                 let idx = color_to_16(color);
@@ -236,11 +236,11 @@ impl DiffRenderer {
         match self.color_mode {
             ColorMode::TrueColor => {
                 let (r, g, b) = color.to_rgb8();
-                write!(output, "\x1b[48;2;{};{};{}m", r, g, b)?;
+                write!(output, "\x1b[48;2;{r};{g};{b}m")?;
             }
             ColorMode::Color256 => {
                 let idx = color_to_256(color);
-                write!(output, "\x1b[48;5;{}m", idx)?;
+                write!(output, "\x1b[48;5;{idx}m")?;
             }
             ColorMode::Color16 | ColorMode::Color8 => {
                 let idx = color_to_16(color);
@@ -273,7 +273,7 @@ impl Default for DiffRenderer {
     }
 }
 
-/// Clone a CellBuffer (for diff comparison).
+/// Clone a `CellBuffer` (for diff comparison).
 fn clone_buffer(buffer: &CellBuffer) -> CellBuffer {
     let mut new_buf = CellBuffer::new(buffer.width(), buffer.height());
     for y in 0..buffer.height() {
@@ -302,9 +302,9 @@ fn color_to_256(color: Color) -> u8 {
     }
 
     // Map to 6x6x6 color cube (indices 16-231)
-    let r_idx = (r as u16 * 5 / 255) as u8;
-    let g_idx = (g as u16 * 5 / 255) as u8;
-    let b_idx = (b as u16 * 5 / 255) as u8;
+    let r_idx = (u16::from(r) * 5 / 255) as u8;
+    let g_idx = (u16::from(g) * 5 / 255) as u8;
+    let b_idx = (u16::from(b) * 5 / 255) as u8;
 
     16 + 36 * r_idx + 6 * g_idx + b_idx
 }

src/computebrick/widgets.rs

@@ -145,7 +145,7 @@ impl BrailleGraph {
         if right_dots >= 3 { pattern |= 0x10; } // dot 5
         if right_dots >= 4 { pattern |= 0x08; } // dot 4
 
-        char::from_u32(0x2800 + pattern as u32).unwrap_or(' ')
+        char::from_u32(0x2800 + u32::from(pattern)).unwrap_or(' ')
     }
 
     /// Get block character for value.
@@ -534,7 +534,7 @@ impl Table {
     }
 
     fn compute_column_widths(headers: &[String], rows: &[Vec<String>]) -> Vec<usize> {
-        let mut widths: Vec<usize> = headers.iter().map(|h| h.len()).collect();
+        let mut widths: Vec<usize> = headers.iter().map(std::string::String::len).collect();
 
         for row in rows {
             for (i, cell) in row.iter().enumerate() {

src/frontend/parser/expressions_helpers/string_operations.rs

@@ -112,7 +112,8 @@ fn flush_text_part(parts: &mut Vec<StringPart>, current: &mut String) {
 /// - `{expr}` - Simple expression
 /// - `{expr:format}` - Expression with format specifier
 /// - `{}` - Empty placeholder (positional argument)
-/// Fixed: Don't confuse :: (path separator) with : (format specifier)
+///
+/// Note: Don't confuse `::` (path separator) with `:` (format specifier)
 fn parse_interpolation(expr_str: &str) -> Result<StringPart> {
     use crate::frontend::parser::Parser;
 

src/runtime/eval_array.rs

@@ -602,15 +602,15 @@ fn eval_array_sort(arr: &Arc<[Value]>) -> Result<Value, InterpreterError> {
 }
 
 /// PIPELINE-001: Reverse array order
-/// Enables: arr |> reverse or arr.reverse()
+/// Enables: arr |> reverse or `arr.reverse()`
 fn eval_array_reverse(arr: &Arc<[Value]>) -> Result<Value, InterpreterError> {
     let mut reversed = arr.to_vec();
     reversed.reverse();
     Ok(Value::Array(Arc::from(reversed)))
 }
 
 /// BOOK-200: Sum all numeric elements in array
-/// Enables: [1, 2, 3] |> sum() => 6
+/// Enables: [1, 2, 3] |> `sum()` => 6
 fn eval_array_sum(arr: &Arc<[Value]>) -> Result<Value, InterpreterError> {
     let mut int_sum: i64 = 0;
     let mut float_sum: f64 = 0.0;
@@ -639,7 +639,7 @@ fn eval_array_sum(arr: &Arc<[Value]>) -> Result<Value, InterpreterError> {
 }
 
 /// BOOK-200: Compute product of all numeric elements
-/// Enables: [1, 2, 3] |> product() => 6
+/// Enables: [1, 2, 3] |> `product()` => 6
 fn eval_array_product(arr: &Arc<[Value]>) -> Result<Value, InterpreterError> {
     if arr.is_empty() {
         return Ok(Value::Integer(1)); // Identity for multiplication
@@ -672,7 +672,7 @@ fn eval_array_product(arr: &Arc<[Value]>) -> Result<Value, InterpreterError> {
 }
 
 /// BOOK-200: Find minimum numeric element
-/// Enables: [3, 1, 4] |> min() => 1
+/// Enables: [3, 1, 4] |> `min()` => 1
 fn eval_array_min(arr: &Arc<[Value]>) -> Result<Value, InterpreterError> {
     if arr.is_empty() {
         return Ok(Value::Nil);
@@ -705,7 +705,7 @@ fn eval_array_min(arr: &Arc<[Value]>) -> Result<Value, InterpreterError> {
 }
 
 /// BOOK-200: Find maximum numeric element
-/// Enables: [3, 1, 4] |> max() => 4
+/// Enables: [3, 1, 4] |> `max()` => 4
 fn eval_array_max(arr: &Arc<[Value]>) -> Result<Value, InterpreterError> {
     if arr.is_empty() {
         return Ok(Value::Nil);
@@ -768,7 +768,7 @@ fn eval_array_skip(arr: &Arc<[Value]>, count: &Value) -> Result<Value, Interpret
 }
 
 /// BOOK-200: Zip two arrays together into array of tuples
-/// Enables: [1, 2].zip(["a", "b"]) => [(1, "a"), (2, "b")]
+/// Enables: `[1, 2].zip(["a", "b"]) => [(1, "a"), (2, "b")]`
 fn eval_array_zip(arr: &Arc<[Value]>, other: &Value) -> Result<Value, InterpreterError> {
     match other {
         Value::Array(other_arr) => {

src/runtime/eval_string_methods.rs

@@ -194,7 +194,7 @@ fn eval_string_repeat(s: &str, count: &Value) -> Result<Value, InterpreterError>
 /// Cyclomatic complexity: 2
 fn eval_string_append(s: &Arc<str>, suffix: &Value) -> Result<Value, InterpreterError> {
     if let Value::String(suffix_str) = suffix {
-        Ok(Value::from_string(format!("{}{}", s, suffix_str)))
+        Ok(Value::from_string(format!("{s}{suffix_str}")))
     } else {
         Err(InterpreterError::RuntimeError(
             "append expects string argument".to_string(),

src/runtime/interpreter.rs

@@ -7311,7 +7311,7 @@ impl Interpreter {
         result
             .into_iter()
             .take(provided_count)
-            .filter_map(|opt| opt)
+            .flatten()
             .collect()
     }