fix: support .join without arguments

Author: merceyz

src/utils.ts

@@ -89,16 +89,14 @@ function getParameterExpressionValue(expr: TSESTree.Expression) {
 		return null;
 	}
 
-	if (!expr.arguments[0]) {
-		return null;
-	}
-
 	if (ASTUtils.getPropertyName(expr.callee) !== "join") {
 		return null;
 	}
 
-	const joinValue = ASTUtils.getStaticValue(expr.arguments[0]);
-	if (typeof joinValue?.value !== "string") {
+	if (
+		expr.arguments[0] != null &&
+		typeof ASTUtils.getStaticValue(expr.arguments[0])?.value !== "string"
+	) {
 		return null;
 	}
 

tests/rules/valid-query.test.ts

@@ -38,6 +38,7 @@ ruleTester.run("valid-query", rule, {
 		"const query = `SELECT * FROM users WHERE id IN (${ids.map(() => '?').join(',')})`;db_users.prepare(query);",
 		"db_users.prepare(`SELECT * FROM users WHERE ${ids.map(() => 'NAME LIKE ? || \\'%\\'').join(' OR ')}`);",
 		"const query = `SELECT * FROM users WHERE ${ids.map(() => 'NAME LIKE ? || \\'%\\'').join(' OR ')}`;db_users.prepare(query);",
+		"db_users.prepare(`SELECT * FROM users WHERE id IN (${ids.map(() => '?').join()})`);",
 	],
 	invalid: [
 		{