bump version to 0.2.2 and normalize where field names to lowercase for consistent SQL generation

Author: ThomasTJdev

sqlquery.nimble

@@ -1,6 +1,6 @@
 # Package
 
-version       = "0.2.1"
+version       = "0.2.2"
 author        = "ThomasTJdev"
 description   = "SQL query builder and validator - opiniated"
 license       = "MIT"

src/sqlquery/sql_query_core.nim

@@ -264,10 +264,12 @@ proc compileValidateWhereField(
   allowCrossTable: bool,
   contextPrefix: string
 ) =
-  var fieldStr = fieldRaw
-  if fieldStr.startsWith("sql:>"):
+  if fieldRaw.startsWith("sql:>"):
     return
 
+  # Match parseWhereCondition: schema validation is case-insensitive; SQL uses lowercase identifiers.
+  var fieldStr = fieldRaw.toLowerAscii().strip()
+
   # Allow PostgreSQL functions in where clause similarly to runtime parser.
   if "(" in fieldStr:
     let s1 = fieldStr.split("(")[1]
@@ -387,13 +389,16 @@ proc extractColumnRefs(sqlExpr: string): seq[string] =
   return refs
 
 proc parseWhereCondition(condition: WhereSpec, conditionIndex: int, requireTableName = true, table = "", validateSchema = true): tuple[statement: string, params: seq[string]] =
-  var field = condition[0]
+  let fieldRaw = condition[0]
+  let customSQL = fieldRaw.startsWith("sql:>")
+  # Non-custom fields are normalized like parseSelect so validation matches the schema and emitted SQL is stable.
+  var field = if not customSQL: fieldRaw.toLowerAscii().strip() else: fieldRaw
+
   if field == "project_id" and conditionIndex > 0:
     sqlWarning("project_id within a where statement will in 99% of cases be the first condition. That's our indexes!")
 
   let symbol = condition[1]
   let value = condition[2]
-  let customSQL = field.startsWith("sql:>")
   var skipRestValidation = false
 
   if symbol.len() == 0 and value.len() == 0 and not customSQL:

tests/test_sqlquery_generator.nim

@@ -283,6 +283,31 @@ suite "selectQuery":
     check base.sql == "SELECT actions.id, actions.name, actions.status FROM actions LEFT JOIN project as p ON p.id = actions.project_id WHERE actions.project_id = ? AND actions.is_deleted IS NULL GROUP BY p.name"
     check base.params == @["123"]
 
+  test "where field names are normalized to lowercase":
+    let base = selectQuery(
+      table = "actions",
+      select = @["actions.id", "actions.name", "actions.status"],
+      joins = @[("project AS p", LEFTJOIN, @[("p.id", "=", "actions.project_id")])],
+      where = @[("Actions.PROJECT_ID", "=", "123")],
+      groupBy = @["p.name"],
+    )
+    check base.sql == "SELECT actions.id, actions.name, actions.status FROM actions LEFT JOIN project as p ON p.id = actions.project_id WHERE actions.project_id = ? AND actions.is_deleted IS NULL GROUP BY p.name"
+    check base.params == @["123"]
+
+    let grouped = selectQuery(
+      table = "actions",
+      select = @["actions.id"],
+      where = whereAnd(@[
+        whereCond("ACTIONS.Status", "=", "active"),
+        whereOr(@[
+          whereCond("Actions.NAME", "=", "thomas"),
+          whereCond("actions.system", "=", "SYS"),
+        ]),
+      ]),
+    )
+    check grouped.sql == "SELECT actions.id FROM actions WHERE (actions.status = ? AND (actions.name = ? OR actions.system = ?)) AND actions.is_deleted IS NULL"
+    check grouped.params == @["active", "thomas", "SYS"]
+
   test "table with reserved word column name (method) using strings":
     # Test table with column named "method" which is a reserved word in Nim.
     # When using string literals, use the original SQL column name "method".
@@ -743,7 +768,7 @@ suite "WHERE operators extended":
       where = @[("actions.project_id", "=", "123"), ("LOWER(COALESCE(actions.name, ''))", "=", "test")]
     )
 
-    check query.sql == "SELECT actions.id, actions.name FROM actions WHERE actions.project_id = ? AND LOWER(COALESCE(actions.name, '')) = ? AND actions.is_deleted IS NULL"
+    check query.sql == "SELECT actions.id, actions.name FROM actions WHERE actions.project_id = ? AND lower(coalesce(actions.name, '')) = ? AND actions.is_deleted IS NULL"
     check query.params == @["123", "test"]