fix: clarify format's type to infer getDbStatement's return type

also refactor getDbStatement logic to always return a string

Signed-off-by: Naseem <naseem@transit.app>

plugins/node/opentelemetry-plugin-mysql/src/utils.ts

@@ -76,15 +76,19 @@ function getJDBCString(
  */
 export function getDbStatement(
   query: string | Query | QueryOptions,
-  format: Function,
+  format: (
+    sql: string,
+    values: any[],
+    stringifyObjects?: boolean,
+    timeZone?: string
+  ) => string,
   values?: any[]
 ) {
   if (typeof query === 'string') {
     return values ? format(query, values) : query;
-  } else if (typeof query === 'object') {
-    if (query.values) {
-      return format(query.sql, query.values);
-    }
-    return values ? format(query.sql, values) : query.sql;
+  } else {
+    return values || query.values
+      ? format(query.sql, values || query.values)
+      : query.sql;
   }
 }