fix: soften canonical parser

Author: dubadub

src/parser/step.rs

@@ -384,8 +384,8 @@ fn cookware<'i>(bp: &mut BlockParser<'_, 'i>) -> Option<Event<'i>> {
             } else {
                 unit.span()
             };
-            bp.error(
-                error!(
+            bp.warn(
+                warning!(
                     "Invalid cookware quantity: unit",
                     label!(span, "remove this"),
                 )
@@ -443,8 +443,8 @@ fn timer<'i>(bp: &mut BlockParser<'_, 'i>) -> Option<Event<'i>> {
     let mut quantity = body.quantity.map(|tokens| {
         let q = parse_quantity(bp, tokens);
         if q.quantity.unit.is_none() {
-            bp.error(
-                error!(
+            bp.warn(
+                warning!(
                     "Invalid timer quantity: missing unit",
                     label!(
                         Span::pos(q.quantity.value.span().end()),
@@ -478,7 +478,7 @@ fn timer<'i>(bp: &mut BlockParser<'_, 'i>) -> Option<Event<'i>> {
         } else {
             Span::pos(name_offset)
         };
-        bp.error(error!(
+        bp.warn(warning!(
             "Invalid timer: neither quantity nor name",
             label!(span, "expected duration or name"),
         ));

tests/general_tests.rs

@@ -247,3 +247,27 @@ fn text_steps_extension() {
         [Content::Text(_)]
     ));
 }
+
+#[test]
+fn timer_missing_unit_warning() {
+    let input = "Cook for ~{30}";
+
+    let parser = CooklangParser::canonical();
+    let result = parser.parse(input);
+
+    // Check that we get a warning, not an error
+    let report = result.report();
+
+    // Verify we have exactly one warning
+    assert_eq!(report.iter().count(), 1);
+
+    // Check that it's a warning, not an error
+    let diag = report.iter().next().unwrap();
+    assert_eq!(diag.severity, cooklang::error::Severity::Warning);
+
+    // Verify the warning message
+    assert!(diag.message.contains("Invalid timer quantity: missing unit"));
+
+    // Should parse successfully (not error)
+    let _r = result.unwrap_output();
+}