remult 0.27.11 → 0.27.13

package/esm/server/remult-admin.js

@@ -70,6 +70,7 @@ export function buildEntityInfo(options) {
                 valFieldKey,
                 caption: x.caption,
                 relationToOne: relation,
+                inputType: x.inputType,
                 type: x.valueConverter.fieldTypeInDb == 'json'
                     ? 'json'
                     : x.valueType === Number
@@ -88,6 +89,7 @@ export function buildEntityInfo(options) {
                 ids,
                 fields,
                 relations,
+                defaultOrderBy: metadata.options.defaultOrderBy ?? { id: 'asc' },
             });
         }
     }

package/esm/server/remult-api-server.js

@@ -35,8 +35,6 @@ export function createRemultServerCore(options, serverCoreOptions) {
     RemultAsyncLocalStorage.enable();
     {
         let allControllers = [];
-        if (!safeOptions.entities)
-            safeOptions.entities = [...remultStatic.allEntities];
         if (safeOptions.entities)
             allControllers.push(...safeOptions.entities);
         if (safeOptions.controllers)
@@ -77,7 +75,8 @@ export class RemultServerImplementation {
                         started = true;
                         console.time('Schema ensured');
                     };
-                    entitiesMetaData.push(...options.entities.map((e) => remult.repo(e).metadata));
+                    if (options.entities)
+                        entitiesMetaData.push(...options.entities.map((e) => remult.repo(e).metadata));
                     if (dp.ensureSchema) {
                         startConsoleLog();
                         await dp.ensureSchema(entitiesMetaData);
@@ -233,7 +232,7 @@ export class RemultServerImplementation {
                 res.success(await remult.liveQueryStorage.keepAliveAndReturnUnknownQueryIds(await this.coreOptions.getRequestBody(origReq)));
             }));
         }
-        this.options.entities.forEach((e) => {
+        this.options.entities?.forEach((e) => {
             let key = getEntityKey(e);
             if (key != undefined)
                 this.add(key, (c) => {
@@ -328,12 +327,12 @@ export class RemultServerImplementation {
                                         },
                                     });
                                 }
+                                await what(remult, myReq, myRes, genReq, origRes, req);
                             }
                             finally {
                                 remultStatic.asyncContext.setInInitRequest(false);
                             }
                         }
-                        await what(remult, myReq, myRes, genReq, origRes, req);
                     });
             }
             catch (err) {

package/esm/src/data-providers/array-entity-data-provider.js

@@ -293,6 +293,16 @@ export class ArrayEntityDataProvider {
                         }
                     }
                 }
+                if (this.entity.idMetadata.fields.find((x) => newR[names.$dbNameOf(x)] != r[names.$dbNameOf(x)]) &&
+                    this.rows().find((x) => {
+                        for (const f of this.entity.idMetadata.fields) {
+                            if (x[names.$dbNameOf(f)] != newR[names.$dbNameOf(f)])
+                                return false;
+                        }
+                        return true;
+                    })) {
+                    throw Error('id already exists');
+                }
                 this.verifyThatRowHasAllNotNullColumns(newR, names);
                 this.rows()[i] = newR;
                 return Promise.resolve(this.translateFromJson(this.rows()[i], names));
@@ -315,19 +325,24 @@ export class ArrayEntityDataProvider {
         const names = await this.init();
         let j = this.translateToJson(data, names);
         let idf = this.entity.idMetadata.field;
-        if (!(idf instanceof CompoundIdField)) {
-            if (idf.valueConverter.fieldTypeInDb === 'autoincrement') {
-                j[idf.key] = 1;
-                for (const row of this.rows()) {
-                    if (row[idf.key] >= j[idf.key])
-                        j[idf.key] = row[idf.key] + 1;
+        if (!(idf instanceof CompoundIdField) &&
+            idf.valueConverter.fieldTypeInDb === 'autoincrement') {
+            j[idf.key] = 1;
+            for (const row of this.rows()) {
+                if (row[idf.key] >= j[idf.key])
+                    j[idf.key] = row[idf.key] + 1;
+            }
+        }
+        else {
+            if (this.rows().find((x) => {
+                for (const f of this.entity.idMetadata.fields) {
+                    if (x[names.$dbNameOf(f)] != j[names.$dbNameOf(f)])
+                        return false;
                 }
+                return true;
+            })) {
+                throw Error('id already exists');
             }
-            if (j[idf.key])
-                this.rows().forEach((i) => {
-                    if (j[idf.key] == i[idf.key])
-                        throw Error('id already exists');
-                });
         }
         this.verifyThatRowHasAllNotNullColumns(j, names);
         this.rows().push(j);

package/esm/src/data-providers/in-memory-database.js

@@ -12,7 +12,7 @@ export class InMemoryDataProvider {
     }
     rows = {};
     getEntityDataProvider(entity) {
-        let name = entity.key;
+        let name = entity.dbName;
         if (!this.rows[name])
             this.rows[name] = [];
         return new ArrayEntityDataProvider(entity, () => this.rows[name]);

package/esm/src/data-providers/sql-database.js

@@ -502,29 +502,6 @@ class myDummySQLCommand {
         return val.toString();
     }
 }
-async function bulkInsert(array, db) {
-    if (array.length == 0)
-        return;
-    const chunkSize = 250;
-    for (let i = 0; i < array.length; i += chunkSize) {
-        const items = array.slice(i, i + chunkSize);
-        // do whatever
-        const c = db.createCommand();
-        let sql = 'insert into ' +
-            (await items[0]._.metadata.dbName) +
-            ' (' +
-            (await Promise.all(items[0]._.metadata.fields.toArray().map((f) => f.dbName))).join(',') +
-            ') values ';
-        sql += items
-            .map((row) => '(' +
-            row.$.toArray()
-                .map((f) => c.param(f.metadata.valueConverter.toDb(f.value)))
-                .join(', ') +
-            ')')
-            .join(',');
-        await c.execute(sql);
-    }
-}
 export function getRowAfterUpdate(meta, dataProvider, data, id, operation) {
     const idFilter = id !== undefined ? meta.idMetadata.getIdFilter(id) : {};
     return dataProvider
@@ -614,9 +591,9 @@ export async function groupByImpl(options, e, r, orderByNullFirst, limitSyntax)
                 orderBy += ' desc';
             if (orderByNullFirst) {
                 if (x.isDescending)
-                    select += ' nulls last';
+                    orderBy += ' nulls last';
                 else
-                    select += ' nulls first';
+                    orderBy += ' nulls first';
             }
         }
         if (orderBy)

package/esm/src/filter/filter-interfaces.js

@@ -211,6 +211,7 @@ export class Filter {
                                             found = true;
                                             break;
                                         case '$ne':
+                                        case '$not':
                                         case '!=':
                                         case '$nin':
                                             found = true;

package/esm/src/remult3/RepositoryImplementation.js

@@ -713,7 +713,9 @@ export class RepositoryImplementation {
                 //I've used define property to hide this member from console.lo g
                 get: () => row,
             });
+            row.id = row.getId();
             row.saveOriginalData();
+            row.originalId = row.id;
             return result;
         }
     }
@@ -1412,6 +1414,8 @@ export class rowHelperImplementation extends rowHelperBase {
                         updatedRow = (await this.edp.find({ where: await this.getIdFilter() }))[0];
                     }
                     else {
+                        if (this.id === undefined)
+                            throw new Error('Invalid operation, id is undefined');
                         updatedRow = await this.edp.update(this.id, changesOnly);
                     }
                 }
@@ -1488,8 +1492,11 @@ export class rowHelperImplementation extends rowHelperBase {
         }
         this.__assertValidity();
         try {
-            if (doDelete)
+            if (doDelete) {
+                if (this.id === undefined)
+                    throw new Error('Invalid operation, id is undefined');
                 await this.edp.delete(this.id);
+            }
             if (!this.repo._dataProvider.isProxy) {
                 if (this.info.entityInfo.deleted)
                     await this.info.entityInfo.deleted(this.instance, e);
@@ -1979,9 +1986,13 @@ class EntityFullInfo {
         this.fields = r;
         this.caption = buildCaption(entityInfo.caption, this.key, remult, this);
         if (entityInfo.id) {
-            let r = typeof entityInfo.id === 'function'
-                ? entityInfo.id(this.fields)
-                : Object.keys(entityInfo.id).map((x) => this.fields.find(x));
+            let r = typeof entityInfo.id === 'string'
+                ? this.fields.find(entityInfo.id)
+                : Array.isArray(entityInfo.id)
+                    ? entityInfo.id.map((x) => this.fields.find(x))
+                    : typeof entityInfo.id === 'function'
+                        ? entityInfo.id(this.fields)
+                        : Object.keys(entityInfo.id).map((x) => this.fields.find(x));
             if (Array.isArray(r)) {
                 if (r.length > 1)
                     this.idMetadata.field = new CompoundIdField(...r);

package/esm/src/remult3/remult3.js

@@ -19,7 +19,6 @@ export const flags = {
 };
 //p1 - add parameter all to deleteMany, and updateMany
 //p1  filter.apply ApiPreFilter
-//p1 - add not as an overload to not equal
 /*p1 - add id and use uuid by default, but allow changes with Fields.id.defaultIdProvider NO but defaultProvider yes???
   //p1 - replace uuid with crypto.randomUUID and allow custom fallback NO
   //p1 - Add example for luid
@@ -27,7 +26,6 @@ export const flags = {
   //p1 - explain the benefits of changing the default provider for testing in docs.
 */
 //p1 - consider upsert (where,set)
-//p1 TODO - consider id to also support keyof (id:['company','index']) - had problem with | (keyof Partial<entityType>)[] & `entity`
 //p2 - add some kind of options handler that will help with translation etc... like in hagai - something that runs at the stage where options are being built
 /*y1 - https://github.com/remult/remult/discussions/438
      - https://github.com/remult/remult/blob/query-argumets/projects/tests/dbs/test-sql-database.spec.ts#L100-L128
@@ -41,7 +39,6 @@ export const flags = {
 //p1 - tutorial about controller - and mutable controller
 //p1 - docs abount subscription channel
 //p1 - Consider enforcing serial operations on JSON database to prevent data loss
-//p1 - fix sqlite to  support alter table when adding a column with ensure schema = on
 //p1 - add LifecycleEvent to documentation
 //p1 - fix chaining of saving and saved in multiple entity options args
 //p1 - review starter and examples and separate remult * auth from the routes

package/index.d.ts

@@ -17,7 +17,6 @@ export { FieldMetadata, FieldOptions, FieldValidator, ValueConverter, ValueListI
 ValueOrExpression, } from './src/column-interfaces.js';
 export { RestDataProvider } from './src/data-providers/rest-data-provider.js';
 export { InMemoryDataProvider } from './src/data-providers/in-memory-database.js';
-export { WebSqlDataProvider } from './src/data-providers/web-sql-data-provider.js';
 export { SqlDatabase } from './src/data-providers/sql-database.js';
 export { CustomSqlFilterBuilder, dbNamesOf, dbNamesOfOptions, CustomSqlFilterBuilderFunction, EntityDbNames, } from './src/filter/filter-consumer-bridge-to-sql-request.js';
 export { JsonDataProvider, JsonEntityStorage, } from './src/data-providers/json-data-provider.js';

package/index.js

@@ -1,6 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.SubscriptionChannel = exports.InMemoryLiveQueryStorage = exports.remult = exports.ValueConverters = exports.createValueValidator = exports.createValueValidatorWithArgs = exports.createValidatorWithArgs = exports.createValidator = exports.valueValidator = exports.Validators = exports.UrlBuilder = exports.Filter = exports.CompoundIdField = exports.Sort = exports.EventSource = exports.isBackend = exports.withRemult = exports.Remult = exports.Allow = exports.ForbiddenError = exports.ProgressListener = exports.BackendMethod = exports.Controller = exports.JsonEntityOpfsStorage = exports.JsonDataProvider = exports.dbNamesOf = exports.CustomSqlFilterBuilder = exports.SqlDatabase = exports.WebSqlDataProvider = exports.InMemoryDataProvider = exports.RestDataProvider = exports.describeEntity = exports.describeBackendMethods = exports.describeClass = exports.IdEntity = exports.Relations = exports.Fields = exports.Field = exports.getEntityRef = exports.Entity = exports.CaptionTransformer = exports.ValueListInfo = exports.getValueList = exports.ValueListFieldType = exports.getFields = exports.FieldType = exports.ControllerBase = exports.EntityBase = exports.ArrayEntityDataProvider = void 0;
+exports.SubscriptionChannel = exports.InMemoryLiveQueryStorage = exports.remult = exports.ValueConverters = exports.createValueValidator = exports.createValueValidatorWithArgs = exports.createValidatorWithArgs = exports.createValidator = exports.valueValidator = exports.Validators = exports.UrlBuilder = exports.Filter = exports.CompoundIdField = exports.Sort = exports.EventSource = exports.isBackend = exports.withRemult = exports.Remult = exports.Allow = exports.ForbiddenError = exports.ProgressListener = exports.BackendMethod = exports.Controller = exports.JsonEntityOpfsStorage = exports.JsonDataProvider = exports.dbNamesOf = exports.CustomSqlFilterBuilder = exports.SqlDatabase = exports.InMemoryDataProvider = exports.RestDataProvider = exports.describeEntity = exports.describeBackendMethods = exports.describeClass = exports.IdEntity = exports.Relations = exports.Fields = exports.Field = exports.getEntityRef = exports.Entity = exports.CaptionTransformer = exports.ValueListInfo = exports.getValueList = exports.ValueListFieldType = exports.getFields = exports.FieldType = exports.ControllerBase = exports.EntityBase = exports.ArrayEntityDataProvider = void 0;
 exports.repo = repo;
 var array_entity_data_provider_js_1 = require("./src/data-providers/array-entity-data-provider.js");
 Object.defineProperty(exports, "ArrayEntityDataProvider", { enumerable: true, get: function () { return array_entity_data_provider_js_1.ArrayEntityDataProvider; } });
@@ -31,8 +31,6 @@ var rest_data_provider_js_1 = require("./src/data-providers/rest-data-provider.j
 Object.defineProperty(exports, "RestDataProvider", { enumerable: true, get: function () { return rest_data_provider_js_1.RestDataProvider; } });
 var in_memory_database_js_1 = require("./src/data-providers/in-memory-database.js"); //V
 Object.defineProperty(exports, "InMemoryDataProvider", { enumerable: true, get: function () { return in_memory_database_js_1.InMemoryDataProvider; } });
-var web_sql_data_provider_js_1 = require("./src/data-providers/web-sql-data-provider.js"); //V
-Object.defineProperty(exports, "WebSqlDataProvider", { enumerable: true, get: function () { return web_sql_data_provider_js_1.WebSqlDataProvider; } });
 var sql_database_js_1 = require("./src/data-providers/sql-database.js"); //V
 Object.defineProperty(exports, "SqlDatabase", { enumerable: true, get: function () { return sql_database_js_1.SqlDatabase; } });
 var filter_consumer_bridge_to_sql_request_js_1 = require("./src/filter/filter-consumer-bridge-to-sql-request.js");

package/live-query/data-provider-live-query-storage.js

@@ -6,7 +6,6 @@ var context_js_1 = require("../src/context.js");
 var RepositoryImplementation_js_1 = require("../src/remult3/RepositoryImplementation.js");
 var Fields_js_1 = require("../src/remult3/Fields.js");
 var entity_js_1 = require("../src/remult3/entity.js");
-var remult_static_js_1 = require("../src/remult-static.js");
 var initDataProviderOrJson_js_1 = require("../server/initDataProviderOrJson.js");
 var DataProviderLiveQueryStorage = /** @class */ (function () {
     function DataProviderLiveQueryStorage(dataProvider) {
@@ -262,4 +261,3 @@ var LiveQueryStorageEntity = /** @class */ (function (_super) {
     return LiveQueryStorageEntity;
 }(RepositoryImplementation_js_1.EntityBase));
 exports.LiveQueryStorageEntity = LiveQueryStorageEntity;
-remult_static_js_1.remultStatic.allEntities.splice(remult_static_js_1.remultStatic.allEntities.indexOf(LiveQueryStorageEntity), 1);

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "remult",
-    "version": "0.27.11",
+    "version": "0.27.13",
     "description": "A CRUD framework for full-stack TypeScript",
     "homepage": "https://remult.dev",
     "repository": {

package/postgres/schema-builder.d.ts

@@ -1,19 +1,15 @@
 import type { FieldMetadata } from '../src/column-interfaces.js';
-import type { Remult } from '../src/context.js';
 import type { SqlDatabase } from '../src/data-providers/sql-database.js';
 import type { EntityMetadata } from '../src/remult3/remult3.js';
 export declare function postgresColumnSyntax(x: FieldMetadata, dbName: string): string;
-export declare function verifyStructureOfAllEntities(db: SqlDatabase, remult: Remult): Promise<void>;
 export declare class PostgresSchemaBuilder {
     private pool;
     private removeQuotes;
     private whereTableAndSchema;
     private schemaAndName;
     private schemaOnly;
-    verifyStructureOfAllEntities(remult?: Remult): Promise<void>;
     ensureSchema(entities: EntityMetadata[]): Promise<void>;
     createIfNotExist(entity: EntityMetadata): Promise<void>;
-    addColumnIfNotExist<T extends EntityMetadata>(entity: T, c: (e: T) => FieldMetadata): Promise<void>;
     verifyAllColumns<T extends EntityMetadata>(entity: T): Promise<void>;
     specifiedSchema: string;
     constructor(pool: SqlDatabase, schema?: string);

package/postgres/schema-builder.js

@@ -2,11 +2,8 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.PostgresSchemaBuilder = void 0;
 exports.postgresColumnSyntax = postgresColumnSyntax;
-exports.verifyStructureOfAllEntities = verifyStructureOfAllEntities;
 var tslib_1 = require("tslib");
 var filter_consumer_bridge_to_sql_request_js_1 = require("../src/filter/filter-consumer-bridge-to-sql-request.js");
-var remult_proxy_js_1 = require("../src/remult-proxy.js");
-var remult_static_js_1 = require("../src/remult-static.js");
 var RepositoryImplementation_js_1 = require("../src/remult3/RepositoryImplementation.js");
 var valueConverters_js_1 = require("../src/valueConverters.js");
 function postgresColumnSyntax(x, dbName) {
@@ -41,16 +38,6 @@ function postgresColumnSyntax(x, dbName) {
         result += ' varchar' + (x.allowNull ? '' : " default '' not null");
     return result;
 }
-function verifyStructureOfAllEntities(db, remult) {
-    return tslib_1.__awaiter(this, void 0, void 0, function () {
-        return tslib_1.__generator(this, function (_a) {
-            switch (_a.label) {
-                case 0: return [4 /*yield*/, new PostgresSchemaBuilder(db).verifyStructureOfAllEntities(remult)];
-                case 1: return [2 /*return*/, _a.sent()];
-            }
-        });
-    });
-}
 var PostgresSchemaBuilder = /** @class */ (function () {
     function PostgresSchemaBuilder(pool, schema) {
         this.pool = pool;
@@ -106,61 +93,10 @@ var PostgresSchemaBuilder = /** @class */ (function () {
         // Should default to `public`
         return 'public';
     };
-    PostgresSchemaBuilder.prototype.verifyStructureOfAllEntities = function (remult) {
-        return tslib_1.__awaiter(this, void 0, void 0, function () {
-            var completed, entities, _a, _b, entityClass, entity, e, e_1_1;
-            var e_1, _c;
-            return tslib_1.__generator(this, function (_d) {
-                switch (_d.label) {
-                    case 0:
-                        if (!remult) {
-                            remult = remult_proxy_js_1.remult;
-                        }
-                        completed = new Set();
-                        entities = [];
-                        _d.label = 1;
-                    case 1:
-                        _d.trys.push([1, 6, 7, 8]);
-                        _a = tslib_1.__values(tslib_1.__spreadArray([], tslib_1.__read(remult_static_js_1.remultStatic.allEntities), false).reverse()), _b = _a.next();
-                        _d.label = 2;
-                    case 2:
-                        if (!!_b.done) return [3 /*break*/, 5];
-                        entityClass = _b.value;
-                        entity = remult.repo(entityClass).metadata;
-                        return [4 /*yield*/, (0, filter_consumer_bridge_to_sql_request_js_1.dbNamesOf)(entity, this.pool.wrapIdentifier)];
-                    case 3:
-                        e = _d.sent();
-                        if (completed.has(e.$entityName))
-                            return [3 /*break*/, 4];
-                        completed.add(e.$entityName);
-                        entities.push(entity);
-                        _d.label = 4;
-                    case 4:
-                        _b = _a.next();
-                        return [3 /*break*/, 2];
-                    case 5: return [3 /*break*/, 8];
-                    case 6:
-                        e_1_1 = _d.sent();
-                        e_1 = { error: e_1_1 };
-                        return [3 /*break*/, 8];
-                    case 7:
-                        try {
-                            if (_b && !_b.done && (_c = _a.return)) _c.call(_a);
-                        }
-                        finally { if (e_1) throw e_1.error; }
-                        return [7 /*endfinally*/];
-                    case 8: return [4 /*yield*/, this.ensureSchema(entities)];
-                    case 9:
-                        _d.sent();
-                        return [2 /*return*/];
-                }
-            });
-        });
-    };
     PostgresSchemaBuilder.prototype.ensureSchema = function (entities) {
         return tslib_1.__awaiter(this, void 0, void 0, function () {
-            var entities_1, entities_1_1, entity, e, err_1, e_2_1;
-            var e_2, _a;
+            var entities_1, entities_1_1, entity, e, err_1, e_1_1;
+            var e_1, _a;
             return tslib_1.__generator(this, function (_b) {
                 switch (_b.label) {
                     case 0:
@@ -194,14 +130,14 @@ var PostgresSchemaBuilder = /** @class */ (function () {
                         return [3 /*break*/, 1];
                     case 9: return [3 /*break*/, 12];
                     case 10:
-                        e_2_1 = _b.sent();
-                        e_2 = { error: e_2_1 };
+                        e_1_1 = _b.sent();
+                        e_1 = { error: e_1_1 };
                         return [3 /*break*/, 12];
                     case 11:
                         try {
                             if (entities_1_1 && !entities_1_1.done && (_a = entities_1.return)) _a.call(entities_1);
                         }
-                        finally { if (e_2) throw e_2.error; }
+                        finally { if (e_1) throw e_1.error; }
                         return [7 /*endfinally*/];
                     case 12: return [2 /*return*/];
                 }
@@ -267,7 +203,7 @@ var PostgresSchemaBuilder = /** @class */ (function () {
     PostgresSchemaBuilder.prototype.createTableScript = function (entity) {
         return tslib_1.__awaiter(this, void 0, void 0, function () {
             var result, e, _a, _b, x, sql;
-            var e_3, _c;
+            var e_2, _c;
             return tslib_1.__generator(this, function (_d) {
                 switch (_d.label) {
                     case 0:
@@ -290,12 +226,12 @@ var PostgresSchemaBuilder = /** @class */ (function () {
                                 }
                             }
                         }
-                        catch (e_3_1) { e_3 = { error: e_3_1 }; }
+                        catch (e_2_1) { e_2 = { error: e_2_1 }; }
                         finally {
                             try {
                                 if (_b && !_b.done && (_c = _a.return)) _c.call(_a);
                             }
-                            finally { if (e_3) throw e_3.error; }
+                            finally { if (e_2) throw e_2.error; }
                         }
                         result += ",\r\n   primary key (".concat(entity.idMetadata.fields
                             .map(function (f) { return e.$dbNameOf(f); })
@@ -306,49 +242,10 @@ var PostgresSchemaBuilder = /** @class */ (function () {
             });
         });
     };
-    PostgresSchemaBuilder.prototype.addColumnIfNotExist = function (entity, c) {
-        return tslib_1.__awaiter(this, void 0, void 0, function () {
-            var e, cmd, colName, sql, err_2;
-            return tslib_1.__generator(this, function (_a) {
-                switch (_a.label) {
-                    case 0: return [4 /*yield*/, (0, filter_consumer_bridge_to_sql_request_js_1.dbNamesOf)(entity, this.pool.wrapIdentifier)];
-                    case 1:
-                        e = _a.sent();
-                        if ((0, filter_consumer_bridge_to_sql_request_js_1.shouldNotCreateField)(c(entity), e))
-                            return [2 /*return*/];
-                        _a.label = 2;
-                    case 2:
-                        _a.trys.push([2, 7, , 8]);
-                        cmd = this.pool.createCommand();
-                        colName = e.$dbNameOf(c(entity));
-                        return [4 /*yield*/, cmd.execute("SELECT 1 FROM information_schema.columns WHERE " +
-                                "".concat(this.whereTableAndSchema(cmd, e), " ") +
-                                "AND column_name=".concat(cmd.param(colName.toLocaleLowerCase())))];
-                    case 3:
-                        if (!((_a.sent()).rows.length == 0)) return [3 /*break*/, 6];
-                        return [4 /*yield*/, this.getAddColumnScript(entity, c(entity))];
-                    case 4:
-                        sql = _a.sent();
-                        if (PostgresSchemaBuilder.logToConsole)
-                            console.info(sql);
-                        return [4 /*yield*/, this.pool.execute(sql)];
-                    case 5:
-                        _a.sent();
-                        _a.label = 6;
-                    case 6: return [3 /*break*/, 8];
-                    case 7:
-                        err_2 = _a.sent();
-                        console.error(err_2);
-                        return [3 /*break*/, 8];
-                    case 8: return [2 /*return*/];
-                }
-            });
-        });
-    };
     PostgresSchemaBuilder.prototype.verifyAllColumns = function (entity) {
         return tslib_1.__awaiter(this, void 0, void 0, function () {
-            var cmd, e, cols, _a, _b, col, colName, sql, e_4_1, err_3;
-            var e_4, _c;
+            var cmd, e, cols, _a, _b, col, colName, sql, e_3_1, err_2;
+            var e_3, _c;
             return tslib_1.__generator(this, function (_d) {
                 switch (_d.label) {
                     case 0:
@@ -387,19 +284,19 @@ var PostgresSchemaBuilder = /** @class */ (function () {
                         return [3 /*break*/, 4];
                     case 7: return [3 /*break*/, 10];
                     case 8:
-                        e_4_1 = _d.sent();
-                        e_4 = { error: e_4_1 };
+                        e_3_1 = _d.sent();
+                        e_3 = { error: e_3_1 };
                         return [3 /*break*/, 10];
                     case 9:
                         try {
                             if (_b && !_b.done && (_c = _a.return)) _c.call(_a);
                         }
-                        finally { if (e_4) throw e_4.error; }
+                        finally { if (e_3) throw e_3.error; }
                         return [7 /*endfinally*/];
                     case 10: return [3 /*break*/, 12];
                     case 11:
-                        err_3 = _d.sent();
-                        console.error(err_3);
+                        err_2 = _d.sent();
+                        console.error(err_2);
                         return [3 /*break*/, 12];
                     case 12: return [2 /*return*/];
                 }

package/remult-duckdb.d.ts

@@ -4,6 +4,6 @@ import { type FieldMetadata, type EntityMetadata } from './index.js';
 export declare class DuckDBDataProvider extends SqliteCoreDataProvider {
     constructor(db: Database);
     wrapIdentifier(name: string): string;
-    getCreateTableSql(entity: EntityMetadata<any>): Promise<string>;
+    getCreateTableSql(entity: EntityMetadata<any>): Promise<string[]>;
     addColumnSqlSyntax(x: FieldMetadata, dbName: string, isAlterColumn: boolean): string;
 }

package/remult-duckdb.js

@@ -62,7 +62,7 @@ var DuckDBDataProvider = /** @class */ (function (_super) {
                             .join(','), ")");
                         sql +=
                             'create table if not exists ' + e.$entityName + ' (' + result + '\r\n)';
-                        return [2 /*return*/, sql];
+                        return [2 /*return*/, [sql]];
                 }
             });
         });
@@ -70,8 +70,8 @@ var DuckDBDataProvider = /** @class */ (function (_super) {
     DuckDBDataProvider.prototype.addColumnSqlSyntax = function (x, dbName, isAlterColumn) {
         var result = dbName;
         var allowNull = x.allowNull;
-        if (allowNull && isAlterColumn) {
-            allowNull = false;
+        if (!allowNull && isAlterColumn) {
+            allowNull = true;
             console.log('DuckDB does not support altering columns to allow null on existing tables');
         }
         if (x.valueType == Number) {
@@ -81,7 +81,7 @@ var DuckDBDataProvider = /** @class */ (function (_super) {
                 result +=
                     ' ' +
                         x.valueConverter.fieldTypeInDb +
-                        (x.allowNull || isAlterColumn ? '' : ' default 0 not null');
+                        (allowNull ? '' : ' default 0 not null');
         }
         else if (x.valueType == Date) {
             if (!x.valueConverter.fieldTypeInDb)
@@ -93,20 +93,15 @@ var DuckDBDataProvider = /** @class */ (function (_super) {
                 result += ' ' + x.valueConverter.fieldTypeInDb;
         }
         else if (x.valueType == Boolean)
-            result +=
-                ' boolean' +
-                    (x.allowNull || isAlterColumn ? '' : ' default false not null');
+            result += ' boolean' + (allowNull ? '' : ' default false not null');
         else if (x.valueConverter.fieldTypeInDb) {
             result += ' ' + x.valueConverter.fieldTypeInDb;
-            if (!(x.allowNull || isAlterColumn) &&
-                x.valueConverter.fieldTypeInDb == 'integer') {
+            if (!allowNull && x.valueConverter.fieldTypeInDb == 'integer') {
                 result += ' default 0 not null';
             }
         }
         else
-            result +=
-                ' varchar' +
-                    (x.allowNull || isAlterColumn ? '' : " default '' not null");
+            result += ' varchar' + (allowNull ? '' : " default '' not null");
         return result;
     };
     return DuckDBDataProvider;

package/remult-knex/index.d.ts

@@ -1,5 +1,4 @@
 import type { Knex } from 'knex';
-import type { Remult } from '../src/context.js';
 import { type EntityDbNamesBase } from '../src/filter/filter-consumer-bridge-to-sql-request.js';
 import type { FieldMetadata } from '../src/column-interfaces.js';
 import type { DataProvider, EntityDataProvider } from '../src/data-interfaces.js';
@@ -26,7 +25,6 @@ export declare class KnexDataProvider implements DataProvider, HasWrapIdentifier
 export type CustomKnexFilterBuilderFunction = () => Promise<(builder: Knex.QueryBuilder) => void>;
 export declare class KnexSchemaBuilder {
     private knex;
-    verifyStructureOfAllEntities(remult?: Remult): Promise<void>;
     ensureSchema(entities: EntityMetadata<any>[]): Promise<void>;
     createIfNotExist(entity: EntityMetadata): Promise<void>;
     createTableKnexCommand(entity: EntityMetadata, e: EntityDbNamesBase): Knex.SchemaBuilder;

package/remult-knex/index.js

@@ -12,7 +12,6 @@ var RepositoryImplementation_js_1 = require("../src/remult3/RepositoryImplementa
 var sort_js_1 = require("../src/sort.js");
 var valueConverters_js_1 = require("../src/valueConverters.js");
 var repository_internals_js_1 = require("../src/remult3/repository-internals.js");
-var remult_static_js_1 = require("../src/remult-static.js");
 var sql_database_js_1 = require("../src/data-providers/sql-database.js");
 var KnexDataProvider = /** @class */ (function () {
     function KnexDataProvider(knex) {
@@ -784,23 +783,6 @@ var KnexSchemaBuilder = /** @class */ (function () {
         this.knex = knex;
         this.additionalWhere = '';
     }
-    KnexSchemaBuilder.prototype.verifyStructureOfAllEntities = function (remult) {
-        return tslib_1.__awaiter(this, void 0, void 0, function () {
-            var entities;
-            return tslib_1.__generator(this, function (_a) {
-                switch (_a.label) {
-                    case 0:
-                        if (!remult)
-                            remult = remult_proxy_js_1.remult;
-                        entities = remult_static_js_1.remultStatic.allEntities.map(function (x) { return remult.repo(x).metadata; });
-                        return [4 /*yield*/, this.ensureSchema(entities)];
-                    case 1:
-                        _a.sent();
-                        return [2 /*return*/];
-                }
-            });
-        });
-    };
     KnexSchemaBuilder.prototype.ensureSchema = function (entities) {
         return tslib_1.__awaiter(this, void 0, void 0, function () {
             var entities_1, entities_1_1, entity, e, err_2, e_8_1;