chore: fix clippy warnings across crate

- Backtick `IndexMap`, `BTreeMap`, `IngredientList` in doc comments.
- Replace `or_insert_with(Vec::new)` with `or_default()` in pantry index.
- Use `contains_key` instead of `get(...).is_some()` in aisle tests.
- Explicit `'_` lifetime on `Vec<Event<'_>>` in parser test helpers.
- Normalize `PantryItem::WithAttributes` with no attributes back to
  `Simple` on parse so write→parse roundtrips preserve the variant.

Author: dubadub

src/aisle.rs

@@ -522,9 +522,9 @@ chili flakes
         let a = parse(input).unwrap();
         let p = a.ingredients_info();
         // All case variants should find the same ingredient
-        assert!(p.get("chili flakes").is_some());
-        assert!(p.get(&"Chili flakes".to_lowercase()).is_some());
-        assert!(p.get(&"CHILI FLAKES".to_lowercase()).is_some());
+        assert!(p.contains_key("chili flakes"));
+        assert!(p.contains_key(&"Chili flakes".to_lowercase()));
+        assert!(p.contains_key(&"CHILI FLAKES".to_lowercase()));
         assert_eq!(
             p.get("chili flakes").unwrap().common_name,
             p.get(&"Chili Flakes".to_lowercase()).unwrap().common_name

src/ingredient_list.rs

@@ -123,7 +123,7 @@ impl IngredientList {
     ///
     /// For each ingredient in the list, if it exists in the pantry with a valid quantity,
     /// subtract that quantity from the required amount. Only subtracts when units match.
-    /// Returns a new IngredientList with the remaining quantities needed.
+    /// Returns a new `IngredientList` with the remaining quantities needed.
     ///
     /// # Arguments
     ///

src/pantry.rs

@@ -42,12 +42,12 @@ use crate::{
 /// format.
 #[derive(Debug, Clone, PartialEq, Eq, Serialize, Deserialize, Default)]
 pub struct PantryConf {
-    /// Map of sections to their items (IndexMap to preserve file order)
+    /// Map of sections to their items (`IndexMap` to preserve file order)
     #[serde(flatten)]
     pub sections: IndexMap<String, Vec<PantryItem>>,
 
     /// Index for fast ingredient lookups (lowercase name -> (section, index))
-    /// Using BTreeMap for better cache locality and predictable iteration
+    /// Using `BTreeMap` for better cache locality and predictable iteration
     #[serde(skip)]
     ingredient_index: BTreeMap<String, Vec<(String, usize)>>,
 }
@@ -187,7 +187,7 @@ impl PantryConf {
                 let lowercase_name = item.name().to_lowercase();
                 self.ingredient_index
                     .entry(lowercase_name)
-                    .or_insert_with(Vec::new)
+                    .or_default()
                     .push((section_name.clone(), idx));
             }
         }
@@ -510,6 +510,22 @@ fn parse_core(
         sections.insert("general".to_string(), general_items);
     }
 
+    // Normalize WithAttributes-with-no-attributes back to Simple, so
+    // `write` → `parse` round-trips preserve the item variant.
+    for items in sections.values_mut() {
+        for item in items.iter_mut() {
+            if let PantryItem::WithAttributes(attrs) = item {
+                if attrs.bought.is_none()
+                    && attrs.expire.is_none()
+                    && attrs.quantity.is_none()
+                    && attrs.low.is_none()
+                {
+                    *item = PantryItem::Simple(attrs.name.clone());
+                }
+            }
+        }
+    }
+
     // Build the index for fast lookups
     let mut ingredient_index = BTreeMap::new();
     for (section_name, items) in &sections {

src/parser/step.rs

@@ -568,7 +568,7 @@ mod tests {
     use crate::{error::SourceReport, parser::token_stream::TokenStream};
     use test_case::test_case;
 
-    fn t(input: &str) -> (Vec<Event>, SourceReport) {
+    fn t(input: &str) -> (Vec<Event<'_>>, SourceReport) {
         let mut tokens = TokenStream::new(input).collect::<Vec<_>>();
         // trim trailing newlines, block splitting should make sure this never
         // reaches the step function

src/parser/text_block.rs

@@ -36,7 +36,7 @@ mod tests {
     use indoc::indoc;
     use test_case::test_case;
 
-    fn t(input: &str) -> (Vec<Event>, SourceReport) {
+    fn t(input: &str) -> (Vec<Event<'_>>, SourceReport) {
         let mut tokens = TokenStream::new(input).collect::<Vec<_>>();
         // trim trailing newlines, block splitting should make sure this never
         // reaches the step function