Address PR feedback

crates/codegen/grammar/src/parser_definition.rs

@@ -59,8 +59,8 @@ impl Visitable for TriviaParserDefinitionRef {
 /// For ambiguous syntaxes this needs to be set to at least N, where N
 /// is the token lookahead required to disambiguate the syntax.
 ///
-// By default, we assume no lookahead is required to recover from
-// unrecognized body between delimiters.
+// By default, we assume no lookahead (0) is required to recover from
+// unrecognized body between delimiters, so it's always triggered.
 #[derive(Clone, Debug, Default)]
 pub struct DelimitedRecoveryTokenThreshold(pub u8);
 

crates/codegen/language/definition/src/model/non_terminals/field.rs

@@ -18,6 +18,9 @@ pub struct FieldDelimiters {
     /// How many tokens have to be matched to trigger the error recovery.
     /// For ambiguous syntaxes this needs to be set to at least N, where N
     /// is the token lookahead required to disambiguate the syntax.
+    ///
+    /// By default, we assume no lookahead (0) is required to recover from
+    /// unrecognized body between delimiters, so it's always triggered.
     pub tokens_matched_acceptance_threshold: Option<u8>,
 }
 

crates/codegen/language/internal_macros/src/derive/spanned.rs

@@ -159,7 +159,7 @@ fn get_spanned_type(input: Type) -> Type {
         }
 
         // External types should also be wrapped in 'Spanned<T>':
-        "bool" | "u8" | "char" | "PathBuf" | "String" | "Version" => {
+        "bool" | "char" | "PathBuf" | "String" | "u8" | "Version" => {
             parse_quote! {
                 crate::internals::Spanned<#input>
             }