Fix/panic safety (1/8)

examples/parts/arrays.rs

@@ -1,7 +1,4 @@
-use serde::{
-    Deserialize,
-    Serialize,
-};
+use serde::{Deserialize, Serialize};
 use serde_json::json;
 use toon_format::encode_default;
 

examples/parts/arrays_of_arrays.rs

@@ -1,7 +1,4 @@
-use serde::{
-    Deserialize,
-    Serialize,
-};
+use serde::{Deserialize, Serialize};
 use serde_json::json;
 use toon_format::encode_default;
 

examples/parts/decode_strict.rs

@@ -1,8 +1,5 @@
 use serde_json::Value;
-use toon_format::{
-    decode,
-    DecodeOptions,
-};
+use toon_format::{decode, DecodeOptions};
 
 pub fn decode_strict() {
     // Malformed: header says 2 rows, but only 1 provided

examples/parts/delimiters.rs

@@ -1,9 +1,5 @@
 use serde_json::json;
-use toon_format::{
-    encode,
-    Delimiter,
-    EncodeOptions,
-};
+use toon_format::{encode, Delimiter, EncodeOptions};
 
 pub fn delimiters() {
     let data = json!({

examples/parts/mixed_arrays.rs

@@ -1,7 +1,4 @@
-use serde::{
-    Deserialize,
-    Serialize,
-};
+use serde::{Deserialize, Serialize};
 use serde_json::json;
 use toon_format::encode_default;
 

examples/parts/objects.rs

@@ -1,7 +1,4 @@
-use serde::{
-    Deserialize,
-    Serialize,
-};
+use serde::{Deserialize, Serialize};
 use serde_json::json;
 use toon_format::encode_default;
 

examples/parts/round_trip.rs

@@ -1,15 +1,6 @@
-use serde::{
-    Deserialize,
-    Serialize,
-};
-use serde_json::{
-    json,
-    Value,
-};
-use toon_format::{
-    decode_default,
-    encode_default,
-};
+use serde::{Deserialize, Serialize};
+use serde_json::{json, Value};
+use toon_format::{decode_default, encode_default};
 
 #[derive(Debug, Serialize, Deserialize, PartialEq)]
 struct Product {

examples/parts/structs.rs

@@ -1,11 +1,5 @@
-use serde::{
-    Deserialize,
-    Serialize,
-};
-use toon_format::{
-    decode_default,
-    encode_default,
-};
+use serde::{Deserialize, Serialize};
+use toon_format::{decode_default, encode_default};
 
 #[derive(Debug, Serialize, Deserialize, PartialEq)]
 struct User {

examples/parts/tabular.rs

@@ -1,7 +1,4 @@
-use serde::{
-    Deserialize,
-    Serialize,
-};
+use serde::{Deserialize, Serialize};
 use serde_json::json;
 use toon_format::encode_default;
 

src/cli/main.rs

@@ -1,36 +1,17 @@
 use std::{
     fs,
-    io::{
-        self,
-        Read,
-        Write,
-    },
-    path::{
-        Path,
-        PathBuf,
-    },
+    io::{self, Read, Write},
+    path::{Path, PathBuf},
 };
 
-use anyhow::{
-    bail,
-    Context,
-    Result,
-};
+use anyhow::{bail, Context, Result};
 use clap::Parser;
 use comfy_table::Table;
 use serde::Serialize;
 use tiktoken_rs::cl100k_base;
 use toon_format::{
-    decode,
-    encode,
-    types::{
-        DecodeOptions,
-        Delimiter,
-        EncodeOptions,
-        Indent,
-        KeyFoldingMode,
-        PathExpansionMode,
-    },
+    decode, encode,
+    types::{DecodeOptions, Delimiter, EncodeOptions, Indent, KeyFoldingMode, PathExpansionMode},
 };
 
 #[derive(Parser, Debug)]

src/decode/expansion.rs

@@ -2,13 +2,7 @@ use indexmap::IndexMap;
 
 use crate::{
     constants::QUOTED_KEY_MARKER,
-    types::{
-        is_identifier_segment,
-        JsonValue as Value,
-        PathExpansionMode,
-        ToonError,
-        ToonResult,
-    },
+    types::{is_identifier_segment, JsonValue as Value, PathExpansionMode, ToonError, ToonResult},
 };
 
 pub fn should_expand_key(key: &str, mode: PathExpansionMode) -> Option<Vec<String>> {
@@ -90,7 +84,7 @@ pub fn deep_merge_value(
         }
     } else {
         target.insert(first_key.clone(), Value::Object(IndexMap::new()));
-        match target.get_mut(first_key).unwrap() {
+        match target.get_mut(first_key).expect("key was just inserted") {
             Value::Object(obj) => obj,
             _ => unreachable!(),
         }

src/decode/mod.rs

@@ -6,10 +6,7 @@ pub mod validation;
 
 use serde_json::Value;
 
-use crate::types::{
-    DecodeOptions,
-    ToonResult,
-};
+use crate::types::{DecodeOptions, ToonResult};
 
 /// Decode a TOON string into any deserializable type.
 ///

src/decode/parser.rs

@@ -1,29 +1,12 @@
-use serde_json::{
-    Map,
-    Number,
-    Value,
-};
+use serde_json::{Map, Number, Value};
 
 use crate::{
-    constants::{
-        KEYWORDS,
-        MAX_DEPTH,
-        QUOTED_KEY_MARKER,
-    },
+    constants::{KEYWORDS, MAX_DEPTH, QUOTED_KEY_MARKER},
     decode::{
-        scanner::{
-            Scanner,
-            Token,
-        },
+        scanner::{Scanner, Token},
         validation,
     },
-    types::{
-        DecodeOptions,
-        Delimiter,
-        ErrorContext,
-        ToonError,
-        ToonResult,
-    },
+    types::{DecodeOptions, Delimiter, ErrorContext, ToonError, ToonResult},
     utils::validation::validate_depth,
 };
 
@@ -1441,10 +1424,7 @@ mod tests {
 
     #[test]
     fn test_round_trip_parentheses() {
-        use crate::{
-            decode::decode_default,
-            encode::encode_default,
-        };
+        use crate::{decode::decode_default, encode::encode_default};
 
         let original = json!({
             "message": "Mostly Functions (3 of 3)",

src/decode/scanner.rs

@@ -1,8 +1,4 @@
-use crate::types::{
-    Delimiter,
-    ToonError,
-    ToonResult,
-};
+use crate::types::{Delimiter, ToonError, ToonResult};
 
 /// Tokens produced by the scanner during lexical analysis.
 #[derive(Debug, Clone, PartialEq)]
@@ -352,9 +348,10 @@ impl Scanner {
 
         // Leading zeros like "05" are strings, but "0", "0.5", "-0" are numbers
         if s.starts_with('0') && s.len() > 1 {
-            let second_char = s.chars().nth(1).unwrap();
-            if second_char.is_ascii_digit() {
-                return Ok(Token::String(s.to_string(), false));
+            if let Some(second_char) = s.chars().nth(1) {
+                if second_char.is_ascii_digit() {
+                    return Ok(Token::String(s.to_string(), false));
+                }
             }
         }
 
@@ -458,12 +455,18 @@ impl Scanner {
             _ => {}
         }
 
-        if trimmed.starts_with('-') || trimmed.chars().next().unwrap().is_ascii_digit() {
+        if trimmed.starts_with('-')
+            || trimmed
+                .chars()
+                .next()
+                .is_some_and(|c| c.is_ascii_digit())
+        {
             // Leading zeros like "05" are strings
             if trimmed.starts_with('0') && trimmed.len() > 1 {
-                let second_char = trimmed.chars().nth(1).unwrap();
-                if second_char.is_ascii_digit() {
-                    return Ok(Token::String(trimmed.to_string(), false));
+                if let Some(second_char) = trimmed.chars().nth(1) {
+                    if second_char.is_ascii_digit() {
+                        return Ok(Token::String(trimmed.to_string(), false));
+                    }
                 }
             }
 

src/decode/validation.rs

@@ -1,7 +1,4 @@
-use crate::types::{
-    ToonError,
-    ToonResult,
-};
+use crate::types::{ToonError, ToonResult};
 
 /// Validate that array length matches expected value.
 pub fn validate_array_length(expected: usize, actual: usize) -> ToonResult<()> {

src/encode/folding.rs

@@ -1,8 +1,4 @@
-use crate::types::{
-    is_identifier_segment,
-    JsonValue as Value,
-    KeyFoldingMode,
-};
+use crate::types::{is_identifier_segment, JsonValue as Value, KeyFoldingMode};
 
 /// Result of chain analysis for folding.
 pub struct FoldableChain {

src/encode/mod.rs

@@ -7,19 +7,9 @@ use indexmap::IndexMap;
 use crate::{
     constants::MAX_DEPTH,
     types::{
-        EncodeOptions,
-        IntoJsonValue,
-        JsonValue as Value,
-        KeyFoldingMode,
-        ToonError,
-        ToonResult,
-    },
-    utils::{
-        format_canonical_number,
-        normalize,
-        validation::validate_depth,
-        QuotingContext,
+        EncodeOptions, IntoJsonValue, JsonValue as Value, KeyFoldingMode, ToonError, ToonResult,
     },
+    utils::{format_canonical_number, normalize, validation::validate_depth, QuotingContext},
 };
 
 /// Encode any serializable value to TOON format.
@@ -225,7 +215,7 @@ fn write_object_impl(
             writer.write_newline()?;
         }
 
-        let value = &obj[*key];
+        let value = obj.get(*key).expect("key exists in field list");
 
         // Check if this key-value pair can be folded (v1.5 feature)
         // Don't fold if any sibling key is a dotted path starting with this key

src/encode/writer.rs

@@ -1,15 +1,7 @@
 use crate::{
-    types::{
-        Delimiter,
-        EncodeOptions,
-        ToonResult,
-    },
+    types::{Delimiter, EncodeOptions, ToonResult},
     utils::{
-        string::{
-            is_valid_unquoted_key,
-            needs_quoting,
-            quote_string,
-        },
+        string::{is_valid_unquoted_key, needs_quoting, quote_string},
         QuotingContext,
     },
 };