pr feedback

Author: themisvaltinos

sqlmesh/dbt/builtin.py

@@ -171,16 +171,23 @@ def __init__(self, config_dict: t.Dict[str, t.Any]) -> None:
         self._config = config_dict
 
     def __call__(self, *args: t.Any, **kwargs: t.Any) -> str:
-        if args and not kwargs and len(args) == 1 and isinstance(args[0], dict):
-            # Single dict argument: config({"materialized": "table"})
-            self._config.update(args[0])
-        elif not args and kwargs:
-            # Keyword arguments: config(materialized="table")
-            self._config.update(kwargs)
-        elif args:
+        if args and kwargs:
             raise ConfigError(
-                f"Invalid config usage: expected either a single dict or keyword arguments"
+                "Invalid inline model config: cannot mix positional and keyword arguments"
             )
+
+        if args:
+            if len(args) == 1 and isinstance(args[0], dict):
+                # Single dict argument: config({"materialized": "table"})
+                self._config.update(args[0])
+            else:
+                raise ConfigError(
+                    f"Invalid inline model config: expected a single dictionary, got {len(args)} arguments"
+                )
+        elif kwargs:
+            # Keyword arguments: config(materialized="table")
+            self._config.update(kwargs)
+
         return ""
 
     def set(self, name: str, value: t.Any) -> str:

tests/dbt/test_transformation.py

@@ -1075,14 +1075,22 @@ def test_config_dict_syntax():
         config3({"materialized": "table"}, alias="mixed")
         assert False, "Should have raised ConfigError"
     except Exception as e:
-        assert "Invalid config usage" in str(e)
+        assert "cannot mix positional and keyword arguments" in str(e)
 
     # Test nested dicts
     config4 = Config({})
     config4({"meta": {"owner": "data_team", "priority": 1}, "tags": ["daily", "critical"]})
     assert config4._config["meta"]["owner"] == "data_team"
     assert config4._config["tags"] == ["daily", "critical"]
 
+    # Test multiple positional arguments are rejected
+    config4 = Config({})
+    try:
+        config4({"materialized": "table"}, {"alias": "test"})
+        assert False
+    except Exception as e:
+        assert "expected a single dictionary, got 2 arguments" in str(e)
+
 
 def test_config_dict_in_jinja():
     # Test dict syntax directly with Config class