feat: gate cli features (#41)

* feat: gate cli features
* chore: clippy::unnecessary_unwrap
* chore: make `cli` a default feature
* chore: clippy::unnecessary_pass_by_value
* chore: clippy_manual_let_else
* chore: clippy::single_char_pattern
* chore: clippy::explicit_iter_loop
* typo: delimeter -> delimiter
* typo: cloumn -> column
* chore: cargo +nightly fmt

Author: jqnatividad

Cargo.toml

@@ -22,26 +22,44 @@ path = "examples/main.rs"
 [[bin]]
 name = "toon"
 path = "src/cli/main.rs"
+required-features = ["cli"]
+
+[features]
+default = ["cli"]
+cli = [
+    "dep:clap",
+    "dep:anyhow",
+    "dep:tiktoken-rs",
+    "dep:comfy-table",
+    "dep:ratatui",
+    "dep:crossterm",
+    "dep:tui-textarea",
+    "dep:arboard",
+    "dep:syntect",
+    "dep:unicode-width",
+    "dep:chrono",
+]
 
 [dependencies]
 serde = { version = "1.0.228", features = ["derive"] }
 indexmap = "2.0"
 serde_json = { version = "1.0.145", features = ["preserve_order"] }
 thiserror = "2.0.17"
 
-clap = { version = "4.5.11", features = ["derive"] }
-anyhow = "1.0.86"
-tiktoken-rs = "0.9.1"
-comfy-table = "7.1"
-
-# TUI dependencies
-ratatui = "0.29"
-crossterm = "0.28"
-tui-textarea = "0.7"
-arboard = "3.4"
-syntect = "5.2"
-unicode-width = "0.2"
-chrono = "0.4"
+# CLI dependencies (gated behind "cli" feature)
+clap = { version = "4.5.11", features = ["derive"], optional = true }
+anyhow = { version = "1.0.86", optional = true }
+tiktoken-rs = { version = "0.9.1", optional = true }
+comfy-table = { version = "7.1", optional = true }
+
+# TUI dependencies (gated behind "cli" feature)
+ratatui = { version = "0.29", optional = true }
+crossterm = { version = "0.28", optional = true }
+tui-textarea = { version = "0.7", optional = true }
+arboard = { version = "3.4", optional = true }
+syntect = { version = "5.2", optional = true }
+unicode-width = { version = "0.2", optional = true }
+chrono = { version = "0.4", optional = true }
 
 [dev-dependencies]
 datatest-stable = "0.3.3"

src/decode/parser.rs

@@ -651,7 +651,7 @@ impl<'a> Parser<'a> {
 
         if let Some(fields) = fields {
             validation::validate_field_list(&fields)?;
-            self.parse_tabular_array(length, fields, depth, context)
+            self.parse_tabular_array(length, &fields, depth, context)
         } else {
             // Non-tabular arrays as first field of list items require depth adjustment
             // (items at depth +2 relative to hyphen, not the usual +1)
@@ -666,7 +666,7 @@ impl<'a> Parser<'a> {
     fn parse_tabular_array(
         &mut self,
         length: usize,
-        fields: Vec<String>,
+        fields: &[String],
         depth: usize,
         context: ArrayParseContext,
     ) -> ToonResult<Value> {

src/lib.rs

@@ -28,6 +28,7 @@
 pub mod constants;
 pub mod decode;
 pub mod encode;
+#[cfg(feature = "cli")]
 pub mod tui;
 pub mod types;
 pub mod utils;

src/types/delimeter.rs src/types/delimiter.rssrc/types/delimeter.rs renamed to src/types/delimiter.rs

@@ -1,10 +1,10 @@
-mod delimeter;
+mod delimiter;
 mod errors;
 mod folding;
 mod options;
 mod value;
 
-pub use delimeter::Delimiter;
+pub use delimiter::Delimiter;
 pub use errors::{
     ErrorContext,
     ToonError,

src/types/mod.rs

@@ -100,9 +100,10 @@ pub fn is_valid_unquoted_key(key: &str) -> bool {
     }
 
     let mut chars = key.chars();
-    let first = match chars.next() {
-        Some(c) => c,
-        None => return false,
+    let first = if let Some(c) = chars.next() {
+        c
+    } else {
+        return false;
     };
 
     if !first.is_alphabetic() && first != '_' {
@@ -142,7 +143,7 @@ pub fn needs_quoting(s: &str, delimiter: char) -> bool {
         return true;
     }
 
-    if s.starts_with("-") {
+    if s.starts_with('-') {
         return true;
     }
 

src/utils/string.rs

@@ -29,13 +29,13 @@ pub fn validate_field_name(name: &str) -> ToonResult<()> {
 pub fn validate_value(value: &Value) -> ToonResult<()> {
     match value {
         Value::Object(obj) => {
-            for (key, val) in obj.iter() {
+            for (key, val) in obj {
                 validate_field_name(key)?;
                 validate_value(val)?;
             }
         }
         Value::Array(arr) => {
-            for val in arr.iter() {
+            for val in arr {
                 validate_value(val)?;
             }
         }

src/utils/validation.rs

@@ -143,7 +143,7 @@ fn test_tabular_array_errors() {
         let err_str = e.to_string();
         assert!(
             err_str.contains("Parse")
-                || err_str.contains("cloumn")
+                || err_str.contains("column")
                 || err_str.contains("expected")
                 || err_str.contains("primitive"),
             "Error should mention missing field or delimiter: {err_str}"

tests/errors.rs

@@ -77,11 +77,10 @@ fn test_decode_fixtures(path: &Utf8Path, contents: String) -> datatest_stable::R
         let result = decode::<Value>(toon_input, &opts);
 
         if test.should_error {
-            if result.is_ok() {
+            if let Ok(actual_json) = result {
                 return Err(format!(
                     "Test '{}' should have FAILED, but it succeeded with: {:?}",
-                    test_name,
-                    result.unwrap()
+                    test_name, actual_json
                 )
                 .into());
             }