@an5/orm 1.0.9 → 1.1.0

package/dist/generator/src/dotnet-generator.js

@@ -6,6 +6,8 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.DotnetGenerator = void 0;
 const fs_1 = __importDefault(require("fs"));
 const path_1 = __importDefault(require("path"));
+const types_1 = require("./types");
+const type_kinds_1 = require("./type-kinds");
 class DotnetGenerator {
     constructor(outputDir) {
         this.outputDir = outputDir;
@@ -14,6 +16,7 @@ class DotnetGenerator {
         }
     }
     generate(models) {
+        this.provider = models[0]?.provider;
         // 1. Generate individual C# entity class files
         for (const model of models) {
             this.generateEntityClass(model);
@@ -25,17 +28,17 @@ class DotnetGenerator {
         // 4. Generate configuration helper class
         this.generateConfigClass();
     }
-    getCsFilterType(fieldType, isOptional) {
-        const lower = fieldType.toLowerCase();
-        if (['datetime', 'datetime2', 'smalldatetime', 'date', 'datetimeoffset'].includes(lower))
-            return 'DateTimeFilter';
-        if (['bool', 'boolean', 'bit'].includes(lower))
-            return 'BoolFilter';
-        if (['int', 'integer', 'smallint', 'tinyint', 'number'].includes(lower))
-            return 'IntFilter';
-        if (['bigint', 'long', 'float', 'real', 'double', 'decimal', 'numeric', 'money', 'smallmoney'].includes(lower))
-            return 'NumberFilter';
-        return 'StringFilter';
+    getCsFilterType(field, isOptional) {
+        // From the declared type: a DECIMAL column took the integer filter because it
+        // arrived as `number`.
+        switch ((0, type_kinds_1.fieldKind)(field, this.provider)) {
+            case 'date': return 'DateTimeFilter';
+            case 'bool': return 'BoolFilter';
+            case 'int':
+            case 'bigint': return 'IntFilter';
+            case 'float': return 'NumberFilter';
+            default: return 'StringFilter';
+        }
     }
     generateOrmTypes(models) {
         let content = `// This file is auto-generated. Do not edit directly.
@@ -110,7 +113,7 @@ namespace An5Orm
 `;
         for (const model of models) {
             const name = this.capitalize(model.name);
-            const csType = (f) => this.mapType(f.type, f.isOptional);
+            const csType = (f) => this.mapType(f, f.isOptional);
             content += `    // ── ${name} ORM Types ──────────────────────────────────────────────────────\n\n`;
             // WhereInput
             content += `    /// <summary>Type-safe WHERE filter for ${name} queries.</summary>\n`;
@@ -119,7 +122,7 @@ namespace An5Orm
             content += `        public List<${name}WhereInput> OR { get; set; }\n`;
             content += `        public ${name}WhereInput NOT { get; set; }\n`;
             for (const f of model.fields) {
-                const ft = this.getCsFilterType(f.type, f.isOptional);
+                const ft = this.getCsFilterType(f, f.isOptional);
                 content += `        public ${ft} ${this.capitalize(f.name)} { get; set; }\n`;
             }
             content += `    }\n\n`;
@@ -135,7 +138,7 @@ namespace An5Orm
             content += `    public class ${name}CreateInput\n    {\n`;
             for (const f of model.fields) {
                 if (!f.isId) {
-                    const ct = this.mapType(f.type, f.isOptional || f.hasDefault);
+                    const ct = this.mapType(f, f.isOptional || f.hasDefault);
                     content += `        public ${ct} ${this.capitalize(f.name)} { get; set; }\n`;
                 }
             }
@@ -145,7 +148,7 @@ namespace An5Orm
             content += `    public class ${name}UpdateInput\n    {\n`;
             for (const f of model.fields) {
                 if (!f.isId) {
-                    const ut = this.mapType(f.type, true); // always nullable for update
+                    const ut = this.mapType(f, true); // always nullable for update
                     content += `        public ${ut} ${this.capitalize(f.name)} { get; set; }\n`;
                 }
             }
@@ -199,44 +202,49 @@ namespace An5Orm
 `;
         fs_1.default.writeFileSync(path_1.default.join(this.outputDir, 'An5Config.cs'), content);
     }
-    mapType(fieldType, isOptional) {
-        let csType = 'string';
-        const lowerType = fieldType.toLowerCase();
-        if (['int', 'integer', 'smallint', 'tinyint', 'number'].includes(lowerType)) {
-            csType = 'int';
-        }
-        else if (['bigint', 'long'].includes(lowerType)) {
-            csType = 'long';
-        }
-        else if (['float', 'real', 'double'].includes(lowerType)) {
-            csType = 'double';
-        }
-        else if (['decimal', 'numeric', 'money', 'smallmoney'].includes(lowerType)) {
-            csType = 'decimal';
-        }
-        else if (['bool', 'boolean', 'bit'].includes(lowerType)) {
-            csType = 'bool';
-        }
-        else if (['datetime', 'datetime2', 'smalldatetime', 'date'].includes(lowerType)) {
-            csType = 'DateTime';
-        }
-        else if (lowerType === 'datetimeoffset') {
-            csType = 'DateTimeOffset';
-        }
-        else if (lowerType === 'time') {
-            csType = 'TimeSpan';
-        }
-        else if (['guid', 'uuid', 'uniqueidentifier'].includes(lowerType)) {
-            csType = 'Guid';
-        }
-        else if (['bytes', 'binary', 'varbinary', 'image'].includes(lowerType)) {
-            csType = 'byte[]';
-        }
-        else if (['xml', 'hierarchyid', 'geography', 'geometry', 'sysname', 'sql_variant', 'sqlvariant', 'rowversion', 'variant'].includes(lowerType)) {
-            csType = 'string';
-        }
-        else if (fieldType.endsWith('[]')) {
-            csType = 'string';
+    /**
+     * The C# type for a field.
+     *
+     * Driven by the declared type, so a `DECIMAL` column is a `decimal` and a `FLOAT`
+     * one a `double`: both arrived as `number` and took the `int` branch.
+     */
+    mapType(field, isOptional) {
+        let csType;
+        switch ((0, type_kinds_1.fieldKind)(field, this.provider)) {
+            case 'int':
+                csType = 'int';
+                break;
+            case 'bigint':
+                csType = 'long';
+                break;
+            case 'float': {
+                // `DECIMAL`/`NUMERIC`/`MONEY` are exact and stay `decimal`; a floating
+                // column is a `double`. Both used to arrive as `number` and became `int`.
+                const base = (field.sqlType ?? field.type).replace(/\([^)]*\)/g, '').trim().toUpperCase();
+                csType = ['DECIMAL', 'NUMERIC', 'MONEY', 'SMALLMONEY'].includes(base) ? 'decimal' : 'double';
+                break;
+            }
+            case 'bool':
+                csType = 'bool';
+                break;
+            case 'date': {
+                const base = (field.sqlType ?? field.type).replace(/\([^)]*\)/g, '').trim().toUpperCase();
+                csType = base === 'DATETIMEOFFSET' ? 'DateTimeOffset' : base === 'TIME' ? 'TimeSpan' : 'DateTime';
+                break;
+            }
+            case 'bytes':
+                csType = 'byte[]';
+                break;
+            case 'vector':
+                csType = 'float[]';
+                break;
+            // JSON, XML, geometry and the rest stay strings: they are read as text and a
+            // Guid column is covered by the string case below via its declared name.
+            default: {
+                const base = (field.sqlType ?? field.type).replace(/\([^)]*\)/g, '').trim().toUpperCase();
+                csType = ['GUID', 'UUID', 'UNIQUEIDENTIFIER'].includes(base) ? 'Guid' : 'string';
+                break;
+            }
         }
         if (isOptional && csType !== 'string' && !csType.endsWith('[]')) {
             csType += '?';
@@ -255,7 +263,7 @@ namespace An5Orm.Entities
 `;
         // Generate fields
         for (const field of model.fields) {
-            const csType = this.mapType(field.type, field.isOptional);
+            const csType = this.mapType(field, field.isOptional);
             content += `        public ${csType} ${this.capitalize(field.name)} { get; set; }\n`;
         }
         // Generate relations
@@ -280,31 +288,138 @@ namespace An5Orm.Entities
 using System;
 using System.Collections.Generic;
 using System.Data;
+using System.Data.Common;
 using System.Reflection;
 using System.Text.Json;
-using Microsoft.Data.SqlClient;
 using An5Orm.Entities;
 
 namespace An5Orm
 {
+    /// <summary>SQL dialects this client can talk to.</summary>
+    public enum An5Dialect
+    {
+        Mssql,
+        Postgres,
+        Sqlite
+    }
+
+    /// <summary>
+    /// Picks the ADO.NET provider and rewrites the connection string.
+    ///
+    /// Everything below works against DbConnection/DbCommand rather than the
+    /// SQL Server types, because SqlClient, Npgsql and Sqlite each derive from
+    /// those but share no other members. Only the SQL that differs per dialect
+    /// is branched on.
+    /// </summary>
+    internal static class An5Provider
+    {
+        public static An5Dialect Detect(string connectionString)
+        {
+            var cs = (connectionString ?? "").Trim().ToLowerInvariant();
+            // SQLite first: a bare path or Data Source= would otherwise fall
+            // through to SQL Server.
+            if (cs.StartsWith("sqlite:") || cs.StartsWith("file:") || cs == ":memory:"
+                || cs.Contains("data source=") || cs.Contains("datasource=")
+                || cs.EndsWith(".db") || cs.EndsWith(".sqlite") || cs.EndsWith(".sqlite3"))
+                return An5Dialect.Sqlite;
+            if (cs.StartsWith("postgres://") || cs.StartsWith("postgresql://") || cs.Contains("host="))
+                return An5Dialect.Postgres;
+            return An5Dialect.Mssql;
+        }
+
+        /// <summary>
+        /// Rewrites the accepted SQLite spellings into the one
+        /// Microsoft.Data.Sqlite reads, which is Data Source=.
+        /// </summary>
+        public static string NormalizeSqlite(string connectionString)
+        {
+            var cs = (connectionString ?? "").Trim();
+            if (cs.Length == 0) return "Data Source=:memory:";
+            if (cs.IndexOf("Data Source", StringComparison.OrdinalIgnoreCase) >= 0
+                || cs.IndexOf("DataSource", StringComparison.OrdinalIgnoreCase) >= 0
+                || cs.IndexOf("Mode", StringComparison.OrdinalIgnoreCase) >= 0)
+                return cs;
+            if (cs == ":memory:") return "Data Source=:memory:";
+            if (cs.StartsWith("sqlite://", StringComparison.OrdinalIgnoreCase))
+                return "Data Source=" + cs.Substring("sqlite://".Length);
+            if (cs.StartsWith("sqlite:", StringComparison.OrdinalIgnoreCase))
+                return "Data Source=" + cs.Substring("sqlite:".Length);
+            if (cs.StartsWith("file:", StringComparison.OrdinalIgnoreCase)) return cs;
+            return "Data Source=" + cs;
+        }
+
+        public static DbConnection Open(string connectionString, An5Dialect dialect)
+        {
+            DbConnection conn = dialect switch
+            {
+                An5Dialect.Postgres => new Npgsql.NpgsqlConnection(connectionString),
+                An5Dialect.Sqlite => new Microsoft.Data.Sqlite.SqliteConnection(NormalizeSqlite(connectionString)),
+                _ => new Microsoft.Data.SqlClient.SqlConnection(connectionString)
+            };
+            conn.Open();
+            if (dialect == An5Dialect.Sqlite)
+            {
+                // WAL keeps readers off the writer's back, and SQLite leaves
+                // foreign keys off even when the schema declares them.
+                using var pragma = conn.CreateCommand();
+                pragma.CommandText = "PRAGMA journal_mode=WAL; PRAGMA foreign_keys=ON;";
+                pragma.ExecuteNonQuery();
+            }
+            return conn;
+        }
+
+        /// <summary>
+        /// Adds a parameter. The name is stored without the @ prefix: Npgsql and
+        /// Sqlite treat a leading @ as part of the name, while SqlClient adds it
+        /// back, so stripping it is the only spelling all three agree on.
+        /// </summary>
+        public static DbParameter Bind(DbCommand cmd, string name, object value)
+        {
+            var p = cmd.CreateParameter();
+            p.ParameterName = (name ?? "").TrimStart('@');
+            p.Value = value ?? DBNull.Value;
+            cmd.Parameters.Add(p);
+            return p;
+        }
+
+        /// <summary>Quotes an identifier for the dialect.</summary>
+        public static string Quote(string name, An5Dialect dialect)
+        {
+            if (name.StartsWith("[") || name.StartsWith("\\"")) return name;
+            if (dialect == An5Dialect.Mssql)
+                return "[" + name.Replace("]", "]]") + "]";
+            return "\\"" + name.Replace("\\"", "\\"\\"") + "\\"";
+        }
+
+        /// <summary>Schema-less engines have no dbo prefix on the generated table names.</summary>
+        public static string TableName(string tableName, An5Dialect dialect)
+        {
+            if (dialect != An5Dialect.Sqlite) return tableName;
+            if (tableName.StartsWith("dbo.", StringComparison.OrdinalIgnoreCase))
+                return tableName.Substring(4);
+            return tableName;
+        }
+    }
+
     public class An5DbContext
     {
         public string ConnectionString { get; }
-        
+        public An5Dialect Dialect { get; }
+
         [ThreadStatic]
-        private static SqlConnection _txConn;
+        private static DbConnection _txConn;
         [ThreadStatic]
-        private static SqlTransaction _tx;
+        private static DbTransaction _tx;
 
         public An5DbContext(string connectionString = null)
         {
             ConnectionString = connectionString ?? An5Config.ConnectionString;
+            Dialect = An5Provider.Detect(ConnectionString);
         }
 
         public An5Transaction BeginTransaction()
         {
-            var conn = new SqlConnection(ConnectionString);
-            conn.Open();
+            var conn = An5Provider.Open(ConnectionString, Dialect);
             var tx = conn.BeginTransaction();
             _txConn = conn;
             _tx = tx;
@@ -313,8 +428,8 @@ namespace An5Orm
                 _tx = null;
             });
         }
-        
-        public static SqlConnection GetActiveConnection(string connectionString, out bool isTx)
+
+        public static DbConnection GetActiveConnection(string connectionString, out bool isTx)
         {
             if (_txConn != null)
             {
@@ -322,30 +437,28 @@ namespace An5Orm
                 return _txConn;
             }
             isTx = false;
-            var conn = new SqlConnection(connectionString);
-            conn.Open();
-            return conn;
+            return An5Provider.Open(connectionString, An5Provider.Detect(connectionString));
         }
-        
-        public static SqlTransaction GetActiveTransaction() => _tx;
+
+        public static DbTransaction GetActiveTransaction() => _tx;
 
         // ── Tables / Repositories ──────────────────────────────────────────────
 `;
         for (const model of models) {
             const name = this.capitalize(model.name);
-            content += `        public TableClient<${model.name}> ${name}s => new TableClient<${model.name}>(ConnectionString, "${model.schemaName}.${model.tableName}");\n`;
+            content += `        public TableClient<${model.name}> ${name}s => new TableClient<${model.name}>(ConnectionString, "${(0, types_1.dottedTableName)(model)}");\n`;
             content += `        public TableClient<${model.name}> ${name} => ${name}s;\n`;
         }
         content += `    }
 
     public class An5Transaction : IDisposable
     {
-        private readonly SqlConnection _conn;
-        private readonly SqlTransaction _tx;
+        private readonly DbConnection _conn;
+        private readonly DbTransaction _tx;
         private readonly Action _cleanup;
         private bool _completed;
 
-        public An5Transaction(SqlConnection conn, SqlTransaction tx, Action cleanup)
+        public An5Transaction(DbConnection conn, DbTransaction tx, Action cleanup)
         {
             _conn = conn;
             _tx = tx;
@@ -380,16 +493,21 @@ namespace An5Orm
     {
         public string ConnectionString { get; }
         public string TableName { get; }
+        public An5Dialect Dialect { get; }
 
         public TableClient(string connectionString, string tableName)
         {
             ConnectionString = connectionString;
-            TableName = tableName;
+            Dialect = An5Provider.Detect(connectionString);
+            // SQLite has no schemas, so the generated dbo. prefix would be part
+            // of the table name and match nothing.
+            TableName = An5Provider.TableName(tableName, Dialect);
         }
 
-        private SqlCommand CreateCommand(SqlConnection conn, string query)
+        private DbCommand CreateCommand(DbConnection conn, string query)
         {
-            var cmd = new SqlCommand(query, conn);
+            var cmd = conn.CreateCommand();
+            cmd.CommandText = query;
             var activeTx = An5DbContext.GetActiveTransaction();
             if (activeTx != null)
             {
@@ -410,7 +528,7 @@ namespace An5Orm
                     {
                         foreach (var kvp in parameters)
                         {
-                            cmd.Parameters.AddWithValue(kvp.Key.StartsWith("@") ? kvp.Key : "@" + kvp.Key, kvp.Value ?? DBNull.Value);
+                            An5Provider.Bind(cmd, kvp.Key, kvp.Value);
                         }
                     }
 
@@ -427,7 +545,7 @@ namespace An5Orm
                                     var val = reader[prop.Name];
                                     if (val != DBNull.Value)
                                     {
-                                        prop.SetValue(item, val);
+                                        SetValue(prop, item, val);
                                     }
                                 }
                             }
@@ -461,7 +579,7 @@ namespace An5Orm
                     {
                         foreach (var kvp in parameters)
                         {
-                            cmd.Parameters.AddWithValue(kvp.Key.StartsWith("@") ? kvp.Key : "@" + kvp.Key, kvp.Value ?? DBNull.Value);
+                            An5Provider.Bind(cmd, kvp.Key, kvp.Value);
                         }
                     }
 
@@ -478,7 +596,7 @@ namespace An5Orm
                                     var val = reader[prop.Name];
                                     if (val != DBNull.Value)
                                     {
-                                        prop.SetValue(item, val);
+                                        SetValue(prop, item, val);
                                     }
                                 }
                             }
@@ -496,11 +614,19 @@ namespace An5Orm
 
         public T FindFirst(string whereClause = null, Dictionary<string, object> parameters = null)
         {
-            string query = $"SELECT TOP 1 * FROM {TableName}";
+            // TOP goes before the table, LIMIT after the WHERE — appending LIMIT
+            // to the SELECT would put it ahead of the predicate.
+            string query = Dialect == An5Dialect.Mssql
+                ? $"SELECT TOP 1 * FROM {TableName}"
+                : $"SELECT * FROM {TableName}";
             if (!string.IsNullOrEmpty(whereClause))
             {
                 query += $" WHERE {whereClause}";
             }
+            if (Dialect != An5Dialect.Mssql)
+            {
+                query += " LIMIT 1";
+            }
 
             var conn = An5DbContext.GetActiveConnection(ConnectionString, out bool isTx);
             try
@@ -511,7 +637,7 @@ namespace An5Orm
                     {
                         foreach (var kvp in parameters)
                         {
-                            cmd.Parameters.AddWithValue(kvp.Key.StartsWith("@") ? kvp.Key : "@" + kvp.Key, kvp.Value ?? DBNull.Value);
+                            An5Provider.Bind(cmd, kvp.Key, kvp.Value);
                         }
                     }
 
@@ -528,7 +654,7 @@ namespace An5Orm
                                     var val = reader[prop.Name];
                                     if (val != DBNull.Value)
                                     {
-                                        prop.SetValue(item, val);
+                                        SetValue(prop, item, val);
                                     }
                                 }
                             }
@@ -554,7 +680,9 @@ namespace An5Orm
             var properties = typeof(T).GetProperties(BindingFlags.Public | BindingFlags.Instance);
             var columns = new List<string>();
             var values = new List<string>();
-            var sqlParams = new List<SqlParameter>();
+            // Collected before the command exists; the provider creates the
+            // concrete parameter type, so binding has to wait for the command.
+            var sqlParams = new List<(string Name, object Value)>();
 
             foreach (var prop in properties)
             {
@@ -563,7 +691,7 @@ namespace An5Orm
                 {
                     columns.Add(prop.Name);
                     values.Add("@" + prop.Name);
-                    sqlParams.Add(new SqlParameter("@" + prop.Name, val));
+                    sqlParams.Add((prop.Name, val));
                 }
             }
 
@@ -573,7 +701,7 @@ namespace An5Orm
             {
                 using (var cmd = CreateCommand(conn, query))
                 {
-                    cmd.Parameters.AddRange(sqlParams.ToArray());
+                    foreach (var (name, value) in sqlParams) An5Provider.Bind(cmd, name, value);
                     cmd.ExecuteNonQuery();
                     if (!isTx)
                     {
@@ -592,7 +720,9 @@ namespace An5Orm
         {
             var properties = typeof(T).GetProperties(BindingFlags.Public | BindingFlags.Instance);
             var sets = new List<string>();
-            var sqlParams = new List<SqlParameter>();
+            // Collected before the command exists; the provider creates the
+            // concrete parameter type, so binding has to wait for the command.
+            var sqlParams = new List<(string Name, object Value)>();
             object idVal = null;
 
             foreach (var prop in properties)
@@ -605,7 +735,7 @@ namespace An5Orm
                 else if (val != null)
                 {
                     sets.Add($"{prop.Name} = @{prop.Name}");
-                    sqlParams.Add(new SqlParameter("@" + prop.Name, val));
+                    sqlParams.Add((prop.Name, val));
                 }
             }
 
@@ -614,14 +744,14 @@ namespace An5Orm
                 throw new InvalidOperationException("Cannot update entity without Id");
             }
 
-            sqlParams.Add(new SqlParameter("@id", idVal));
+            sqlParams.Add(("id", idVal));
             string query = $"UPDATE {TableName} SET {string.Join(", ", sets)} WHERE Id = @id";
             var conn = An5DbContext.GetActiveConnection(ConnectionString, out bool isTx);
             try
             {
                 using (var cmd = CreateCommand(conn, query))
                 {
-                    cmd.Parameters.AddRange(sqlParams.ToArray());
+                    foreach (var (name, value) in sqlParams) An5Provider.Bind(cmd, name, value);
                     cmd.ExecuteNonQuery();
                 }
             }
@@ -640,7 +770,7 @@ namespace An5Orm
             {
                 using (var cmd = CreateCommand(conn, query))
                 {
-                    cmd.Parameters.AddWithValue("@id", id);
+                    An5Provider.Bind(cmd, "id", id);
                     int affected = cmd.ExecuteNonQuery();
                     return affected > 0;
                 }
@@ -663,7 +793,7 @@ namespace An5Orm
                     if (parameters != null)
                     {
                         foreach (var kvp in parameters)
-                            cmd.Parameters.AddWithValue(kvp.Key.StartsWith("@") ? kvp.Key : "@" + kvp.Key, kvp.Value ?? DBNull.Value);
+                            An5Provider.Bind(cmd, kvp.Key, kvp.Value);
                     }
                     var res = cmd.ExecuteScalar();
                     return res != null && res != DBNull.Value ? Convert.ToInt32(res) : 0;
@@ -687,13 +817,15 @@ namespace An5Orm
         {
             if (updateData == null || updateData.Count == 0) return 0;
             var sets = new List<string>();
-            var sqlParams = new List<SqlParameter>();
+            // Bound after the command exists, so the provider can create the
+            // right parameter type; a value tuple carries (name, value) until then.
+            var sqlParams = new List<(string Name, object Value)>();
             int pIndex = 0;
             foreach (var kvp in updateData)
             {
                 string paramName = "@u_" + pIndex++;
                 sets.Add($"{kvp.Key} = {paramName}");
-                sqlParams.Add(new SqlParameter(paramName, kvp.Value ?? DBNull.Value));
+                sqlParams.Add((paramName, kvp.Value));
             }
             string query = $"UPDATE {TableName} SET {string.Join(", ", sets)}";
             if (!string.IsNullOrEmpty(whereClause)) query += $" WHERE {whereClause}";
@@ -702,11 +834,11 @@ namespace An5Orm
             {
                 using (var cmd = CreateCommand(conn, query))
                 {
-                    cmd.Parameters.AddRange(sqlParams.ToArray());
+                    foreach (var (name, value) in sqlParams) An5Provider.Bind(cmd, name, value);
                     if (parameters != null)
                     {
                         foreach (var kvp in parameters)
-                            cmd.Parameters.AddWithValue(kvp.Key.StartsWith("@") ? kvp.Key : "@" + kvp.Key, kvp.Value ?? DBNull.Value);
+                            An5Provider.Bind(cmd, kvp.Key, kvp.Value);
                     }
                     return cmd.ExecuteNonQuery();
                 }
@@ -726,7 +858,7 @@ namespace An5Orm
                     if (parameters != null)
                     {
                         foreach (var kvp in parameters)
-                            cmd.Parameters.AddWithValue(kvp.Key.StartsWith("@") ? kvp.Key : "@" + kvp.Key, kvp.Value ?? DBNull.Value);
+                            An5Provider.Bind(cmd, kvp.Key, kvp.Value);
                     }
                     return cmd.ExecuteNonQuery();
                 }
@@ -746,28 +878,44 @@ namespace An5Orm
 
         public List<T> VectorSearch(List<double> vector, int take = 10, string whereClause = null, Dictionary<string, object> parameters = null, string vectorField = "Embedding", string distanceMetric = "cosine")
         {
-            // 1. Primary path: Native database SQL vector query execution (VECTOR_DISTANCE)
+            // 1. Primary path: native vector query. SQL Server has VECTOR_DISTANCE
+            //    and Postgres has pgvector; SQLite has no vector operator, so it is
+            //    skipped outright rather than issuing SQL that can only fail and
+            //    cost a round trip before falling through.
+            if (Dialect != An5Dialect.Sqlite)
             try
             {
                 var dim = vector.Count;
                 var vecJson = JsonSerializer.Serialize(vector);
-                var sql = $"SELECT TOP ({take}) *, VECTOR_DISTANCE('{distanceMetric}', CAST([{vectorField}] AS VECTOR({dim}, float32)), CAST(@query_vector AS VECTOR({dim}, float32))) AS distance FROM {TableName} WITH (NOLOCK)";
+                var field = An5Provider.Quote(vectorField, Dialect);
+                string sql;
+                if (Dialect == An5Dialect.Postgres)
+                {
+                    var op = distanceMetric.Equals("cosine", StringComparison.OrdinalIgnoreCase) ? "<=>"
+                           : (distanceMetric.Equals("euclidean", StringComparison.OrdinalIgnoreCase) ? "<->" : "<#>");
+                    sql = $"SELECT *, ({field} {op} @query_vector::vector) AS distance FROM {TableName}";
+                }
+                else
+                {
+                    sql = $"SELECT TOP ({take}) *, VECTOR_DISTANCE('{distanceMetric}', CAST({field} AS VECTOR({dim}, float32)), CAST(@query_vector AS VECTOR({dim}, float32))) AS distance FROM {TableName} WITH (NOLOCK)";
+                }
 
                 var p = parameters != null ? new Dictionary<string, object>(parameters) : new Dictionary<string, object>();
                 p["query_vector"] = vecJson;
 
                 if (!string.IsNullOrWhiteSpace(whereClause))
-                    sql += $" WHERE [{vectorField}] IS NOT NULL AND ({whereClause})";
+                    sql += $" WHERE {field} IS NOT NULL AND ({whereClause})";
                 else
-                    sql += $" WHERE [{vectorField}] IS NOT NULL";
+                    sql += $" WHERE {field} IS NOT NULL";
                 sql += " ORDER BY distance ASC";
+                if (Dialect == An5Dialect.Postgres) sql += $" LIMIT {take}";
 
                 var nativeRows = QueryRaw(sql, p);
                 if (nativeRows != null) return nativeRows;
             }
             catch
             {
-                // Fallback to in-memory similarity computation if DB instance lacks native VECTOR_DISTANCE
+                // Fallback to in-memory similarity computation if the database has no native vector support
             }
 
             // 2. Secondary fallback: In-memory similarity computation
@@ -849,7 +997,40 @@ namespace An5Orm
             return dot / (Math.Sqrt(m1) * Math.Sqrt(m2));
         }
 
-        private bool HasColumn(SqlDataReader reader, string columnName)
+        /// <summary>
+        /// Assigns a column to a property, converting when the provider's CLR
+        /// type does not match the property.
+        ///
+        /// Required across dialects, not just for SQLite: SQLite hands back
+        /// dates as TEXT and every integer as Int64, so a DateTime or int
+        /// property would otherwise fail to set. A value that cannot be
+        /// converted is skipped rather than aborting the whole read.
+        /// </summary>
+        private static void SetValue(System.Reflection.PropertyInfo prop, object target, object value)
+        {
+            var wanted = prop.PropertyType;
+            if (wanted.IsInstanceOfType(value))
+            {
+                prop.SetValue(target, value);
+                return;
+            }
+            try
+            {
+                var underlying = Nullable.GetUnderlyingType(wanted);
+                if (underlying != null && value == null) { prop.SetValue(target, null); return; }
+                if (underlying != null)
+                {
+                    prop.SetValue(target, Convert.ChangeType(value, underlying));
+                    return;
+                }
+                if (value is string text && wanted == typeof(Guid)) { prop.SetValue(target, Guid.Parse(text)); return; }
+                if (value is string when && wanted == typeof(DateTime)) { prop.SetValue(target, DateTime.Parse(when, System.Globalization.CultureInfo.InvariantCulture, System.Globalization.DateTimeStyles.RoundtripKind)); return; }
+                prop.SetValue(target, Convert.ChangeType(value, wanted));
+            }
+            catch { }
+        }
+
+        private bool HasColumn(DbDataReader reader, string columnName)
         {
             for (int i = 0; i < reader.FieldCount; i++)
             {