Add typo suggestions to MODEL block field errors

Author: treysp

sqlmesh/core/model/common.py

@@ -5,6 +5,7 @@
 from pathlib import Path
 
 from astor import to_source
+from difflib import get_close_matches
 from sqlglot import exp
 from sqlglot.helper import ensure_list
 
@@ -267,13 +268,33 @@ def validate_extra_and_required_fields(
 ) -> None:
     missing_required_fields = klass.missing_required_fields(provided_fields)
     if missing_required_fields:
+        field_names = "'" + "', '".join(missing_required_fields) + "'"
         raise_config_error(
-            f"Missing required fields {missing_required_fields} in the {entity_name}"
+            f"Please add required field{'s' if len(missing_required_fields) > 1 else ''} {field_names} to the {entity_name}."
         )
 
     extra_fields = klass.extra_fields(provided_fields)
     if extra_fields:
-        raise_config_error(f"Invalid extra fields {extra_fields} in the {entity_name}")
+        extra_field_names = "'" + "', '".join(extra_fields) + "'"
+
+        all_fields = klass.all_fields()
+        close_matches = {}
+        for field in extra_fields:
+            matches = get_close_matches(field, all_fields, n=1)
+            if matches:
+                close_matches[field] = matches[0]
+
+        if len(close_matches) == 1:
+            similar_msg = ". Did you mean " + "'" + "', '".join(close_matches.values()) + "'?"
+        else:
+            similar = [
+                f"- {field}: Did you mean '{match}'?" for field, match in close_matches.items()
+            ]
+            similar_msg = "\n\n  " + "\n  ".join(similar) if similar else ""
+
+        raise_config_error(
+            f"Invalid field name{'s' if len(extra_fields) > 1 else ''} present in the {entity_name}: {extra_field_names}{similar_msg}"
+        )
 
 
 def single_value_or_tuple(values: t.Sequence) -> exp.Identifier | exp.Tuple:

sqlmesh/core/model/definition.py

@@ -2156,7 +2156,10 @@ def load_sql_based_model(
         name = get_model_name(path)
 
     if not name:
-        raise_config_error("Model must have a name", path)
+        raise_config_error(
+            "Please add the required 'name' field to the MODEL block at the top of the file.\n\n"
+            + "Learn more at https://sqlmesh.readthedocs.io/en/stable/concepts/models/overview"
+        )
     if "default_catalog" in meta_fields:
         raise_config_error(
             "`default_catalog` cannot be set on a per-model basis. It must be set at the connection level.",
@@ -2400,7 +2403,7 @@ def _create_model(
     **kwargs: t.Any,
 ) -> Model:
     validate_extra_and_required_fields(
-        klass, {"name", *kwargs} - {"grain", "table_properties"}, "model definition"
+        klass, {"name", *kwargs} - {"grain", "table_properties"}, "MODEL block"
     )
 
     for prop in PROPERTIES:

sqlmesh/core/model/kind.py

@@ -1015,7 +1015,9 @@ def create_model_kind(v: t.Any, dialect: str, defaults: t.Dict[str, t.Any]) -> M
                 actual_kind_type, _ = custom_materialization
                 return actual_kind_type(**props)
 
-        validate_extra_and_required_fields(kind_type, set(props), f"model kind '{name}'")
+        validate_extra_and_required_fields(
+            kind_type, set(props), f"MODEL block 'kind {name}(' field"
+        )
         return kind_type(**props)
 
     name = (v.name if isinstance(v, exp.Expression) else str(v)).upper()