fix(macros): slightly improve unsupported type error message

dyc3 · dyc3 · commit 29a286584818 · 2025-05-13T18:26:31.000-04:00
diff --git a/sqlx-macros-core/src/query/args.rs b/sqlx-macros-core/src/query/args.rs
@@ -66,7 +66,12 @@ pub fn quote_args<DB: DatabaseExt>(
                                         i + 1,
                                     )
                                 } else {
-                                    format!("unsupported type {} for param #{}", param_ty, i + 1)
+                                    format!(
+                                        "no built in mapping found for type {} for param #{}; \
+                                        a type override may be required, see documentation for details",
+                                        param_ty,
+                                        i + 1
+                                    )
                                 }
                             })?
                             .parse::<TokenStream>()
diff --git a/sqlx-macros-core/src/query/output.rs b/sqlx-macros-core/src/query/output.rs
@@ -236,7 +236,7 @@ fn get_column_type<DB: DatabaseExt>(i: usize, column: &DB::Column) -> TokenStrea
             let message =
                 if let Some(feature_gate) = <DB as TypeChecking>::get_feature_gate(type_info) {
                     format!(
-                        "optional sqlx feature `{feat}` required for type {ty} of {col}",
+                        "SQLx feature `{feat}` required for type {ty} of {col}",
                         ty = &type_info,
                         feat = feature_gate,
                         col = DisplayColumn {
@@ -246,7 +246,8 @@ fn get_column_type<DB: DatabaseExt>(i: usize, column: &DB::Column) -> TokenStrea
                     )
                 } else {
                     format!(
-                        "unsupported type {ty} of {col}",
+                        "no built in mapping found for type {ty} of {col}; \
+                        a type override may be required, see documentation for details",
                         ty = type_info,
                         col = DisplayColumn {
                             idx: i,

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,12 @@ pub fn quote_args<DB: DatabaseExt>(`
`66`	`66`	`i + 1,`
`67`	`67`	`)`
`68`	`68`	`} else {`
`69`		`- format!("unsupported type {} for param #{}", param_ty, i + 1)`
	`69`	`+ format!(`
	`70`	`+ "no built in mapping found for type {} for param #{}; \`
	`71`	`+ a type override may be required, see documentation for details",`
	`72`	`+ param_ty,`
	`73`	`+ i + 1`
	`74`	`+ )`
`70`	`75`	`}`
`71`	`76`	`})?`
`72`	`77`	`.parse::<TokenStream>()`