remult 0.27.5 → 0.27.7

package/esm/src/filter/filter-consumer-bridge-to-sql-request.js

@@ -285,13 +285,14 @@ export async function fieldDbName(f, meta, wrapIdentifier = (x) => x, forceSqlEx
             let result;
             if (typeof f.options.sqlExpression === 'function') {
                 if (f[sqlExpressionInProgressKey] && !forceSqlExpression) {
-                    return "recursive sqlExpression call for field '" + f.key + "'. ";
+                    return "recursive sqlExpression call for field '" + f.key + "'. \0";
                 }
                 try {
                     ;
                     f[sqlExpressionInProgressKey] = true;
                     result = await f.options.sqlExpression(meta);
-                    f.options.sqlExpression = () => result;
+                    if (!result.includes('\0'))
+                        f.options.sqlExpression = () => result;
                 }
                 finally {
                     delete f[sqlExpressionInProgressKey];

package/esm/src/live-query/SubscriptionServer.js

@@ -12,7 +12,7 @@ export class LiveQueryPublisher {
     runPromise(p) { }
     debugFileSaver = (x) => { };
     async itemChanged(entityKey, changes) {
-        await this.liveQueryStorage().forEach(entityKey, async ({ query: q, setData }) => {
+        await this.runPromise(this.liveQueryStorage().forEach(entityKey, async ({ query: q, setData }) => {
             let query = { ...q.data };
             await this.performWithContext(query.requestJson, entityKey, async (repo) => {
                 const messages = [];
@@ -61,7 +61,7 @@ export class LiveQueryPublisher {
                 if (messages.length > 0)
                     this.subscriptionServer().publishMessage(q.id, messages);
             });
-        });
+        }));
     }
 }
 export class InMemoryLiveQueryStorage {

package/esm/src/remult3/RepositoryImplementation.js

@@ -1348,7 +1348,7 @@ export class rowHelperImplementation extends rowHelperBase {
                             await listener.saved?.(this.instance, isNew);
                         }
                 }
-                await this.repo._remult.liveQueryPublisher.itemChanged(this.repo.metadata.key, [{ id: this.getId(), oldId: this.getOriginalId(), deleted: false }]);
+                this.repo._remult.liveQueryPublisher.itemChanged(this.repo.metadata.key, [{ id: this.getId(), oldId: this.getOriginalId(), deleted: false }]);
                 this.saveOriginalData();
                 this._isNew = false;
                 return this.instance;
@@ -1420,7 +1420,9 @@ export class rowHelperImplementation extends rowHelperBase {
                 for (const listener of this.repo.listeners) {
                     await listener.deleted?.(this.instance);
                 }
-            await this.repo._remult.liveQueryPublisher.itemChanged(this.repo.metadata.key, [{ id: this.getId(), oldId: this.getOriginalId(), deleted: true }]);
+            this.repo._remult.liveQueryPublisher.itemChanged(this.repo.metadata.key, [
+                { id: this.getId(), oldId: this.getOriginalId(), deleted: true },
+            ]);
             this._wasDeleted = true;
         }
         catch (err) {

package/esm/src/remult3/remult3.js

@@ -1,51 +1,37 @@
 export const flags = {
     error500RetryCount: 4,
 };
-//y1 - Hacker News for the tutorial?
+//p1 - add transactional option to backend method
+//p1 - add parameter all to deleteMany, and updateMany
+//p1  filter.apply ApiPreFilter
+//p1 - min, max, avg, sum, countDistinct - noam come up with an api
+/*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
+  //p1 - add example for nanoid
+  //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
-     //p2 - allow preprocess to replace filter values - for example replace $contains on a specific field, with specific other sql - useful for full text search and other scenarios
      //y1 - consider sql expression gets a dbnames of it's own (that already has the "tableName" defined correctly) maybe also the filter translator
+     //p2 - allow preprocess to replace filter values - for example replace $contains on a specific field, with specific other sql - useful for full text search and other scenarios
      //y2 - soft-delete-discussion https://discord.com/channels/975754286384418847/1230386433093533698/1230386433093533698
 */
-//y1 - add parameter all to deleteMany, and updateMany
-//p1 - fix query docs to also explain how it can be used for infinite scroll and pagination.
-//p1 - when like doesn't have a string - don't send it to the db
-//y1 - 'test expression columns without aliases'
-//y1 - live query update waits for last
-//p1 - add luid - it's cool for order by. - NO
-//p1 - replace uuid with crypto.randomUUID and allow custom fallback NO
-//p1 - add nanoid - NO
-//p1 - add id and use luid by default, but allow changes with Fields.id.defaultIdProvider NO but defaultProvider yes
-/*p1
- * field, fieldType ,ValueListFieldType, - return optional parameter for decorators
- * options to remultExpress tc... should be mandatory
- * return value converter to previous typing
- * add breaking change notification about findOne and findFirst and findId
- */
-//p1 - fix query docs to also explain how it can be used for infinite scroll and pagination.
-//y1 - 'test expression columns without aliases'
-//y1 - ts issue with fields - when a field is optional, it's $ is also optional and thats a problem
+//p2 - fix query docs to also explain how it can be used for infinite scroll and pagination.
+//p2 - when like doesn't have a string - don't send it to the db
 //p1 - vite 5.3 supports ts5 decorators - check and adapt.
-//p1 - see if I can fix the no alias problem in selects
-//p1 - add luid - it's cool for order by.
-//p1 - replace uuid with crypto.randomUUID and allow custom fallback
-//p1 - add nanoid
-//p1 - add id and use luid by default, but allow changes with Fields.id.defaultIdProvider
-//p1 - explain the benefits of changing the default provider for testing in docs.
+//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 - fix app custom filters example for multiple filters.
-//p1 - add not!!!!
 //p1 - add LifecycleEvent to documentation
-//p1 - Fix hagai and starter project to latest ts
 //p1 - fix chaining of saving and saved in multiple entity options args
 //p1 - review starter and examples and separate remult * auth from the routes
-//p1 - min, max, avg, sum
-//p1 - consider upsert (where,set)
 //y1 - live query with count #436
 //y1 TODO - In the esm version of our tutorial - the imports are automatically .ts and not .js in react and not in vue
-//y1 TODO - consider id to also support keyof (id:['company','index']) - had problem with | (keyof Partial<entityType>)[] & `entity`
-//y1 TODO - readonly key: keyof MembersOnly<entityType> & string
 //y1 TODO - fix remult admin not to load the html into memory until used
 //y2 - talk about insert / update / delete with relations
 /*
@@ -63,15 +49,10 @@ const result = await repo.batch(x=>({
 }))
 */
 //y1 - wait a second to close stream -see pr
-//p1 - Consider enforcing serial operations on JSON database to prevent data loss
 //p1 - prepare the createEntity discussion
-//p1 - return error on connection error
 //p1 - article on displayValue including it's definition for entities that is used in relations
 //p1 - article auth.js with express - played with it, requires type="module" and a few more configurations - https://github.com/noam-honig/express-auth
-//y2 - currently for string fields we default for 255 in knex (which is their default) why not just use text (varchar max) - and only use exact length when one is provided?
 //p2 - create foreign key constraints in user code - https://codesandbox.io/p/devbox/fk-validator-tdshcs
-//p2 - admin url1 - pocketbase use /_ for the admin //{allowed?:Allowed,url?:string} also checkout the admin* as it should probably be something else and didn't work in hono
-//y2 - fix validators cache
 //y2 - should we validate relations
 //y1 - dependency of live query tables  live query refresh of view on table update
 //p1 - see why mongo tests do not run anymore
@@ -91,7 +72,6 @@ const result = await repo.batch(x=>({
   - delete restrict (implicit, or user selected - and if so, how) (delete & update of id)
 
 */
-//y1 - tried to upgrade vitest, nuxt tests are failing with loading uuid - sounds familiar?
 //y1 - talk about the parameter issue with backend methods
 //y2 - livequery for findfirst (@JY)
 /*y2 -
@@ -108,7 +88,6 @@ const result = await repo.batch(x=>({
 //y2 - Fix problem with promise all in sql expression recurssion - when using PromiseAll in row relation loading, some sql expressions appear is recursion call even if they are not
 //p2 - when subscribe is forbidden - the query still runs after the renew process
 //p2 - 'update tasks set  where id = $1
-//p2 - type metadata.key - to keyof entity - based on cwis input
 //y2 - remove __dirname from tutorials
 //p2 - when value changes for field with error, clear error - so the user will feel comfortable
 //p2 - allowApiUpdate should be false for include in api false

package/esm/src/server-action.js

@@ -89,7 +89,7 @@ export class myServerAction extends Action {
     async execute(info, remult, res) {
         let result = { data: {} };
         let ds = remult.dataProvider;
-        await doTransaction(remult, async () => {
+        await decideTransaction(remult, this.options, async () => {
             if (!remult.isAllowedForInstance(undefined, this.options.allowed))
                 throw new ForbiddenError();
             info.args = await prepareReceivedArgs(this.types(), info.args, remult, ds, res);
@@ -112,7 +112,24 @@ export function Controller(key) {
         return target;
     };
 }
-/** Indicates that the decorated methods runs on the backend. See: [Backend Methods](https://remult.dev/docs/backendMethods.html) */
+/**
+ * Decorator indicating that the decorated method runs on the backend.
+ * It allows the method to be invoked from the frontend while ensuring that the execution happens on the server side.
+ * By default, the method runs within a database transaction, meaning it will either complete entirely or fail without making any partial changes.
+ * This behavior can be controlled using the `transactional` option in the `BackendMethodOptions`.
+ *
+ * For more details, see: [Backend Methods](https://remult.dev/docs/backendMethods.html).
+ *
+ * @param options - Configuration options for the backend method, including permissions, routing, and transactional behavior.
+ *
+ * @example
+ * ```typescript
+ * @BackendMethod({ allowed: true })
+ * async someBackendMethod() {
+ *   // method logic here
+ * }
+ * ```
+ */
 export function BackendMethod(options) {
     return (target, context, descriptor) => {
         const key = typeof context === 'string' ? context : context.name.toString();
@@ -147,7 +164,7 @@ export function BackendMethod(options) {
                 }
                 else
                     return await originalMethod.apply(
-                    //@ts-expect-error I specifically referred to the this of the original function - so it'll be sent inside
+                    //@ts-ignore
                     this, args);
             };
             registerAction(target, result);
@@ -181,7 +198,7 @@ export function BackendMethod(options) {
                         try {
                             let remult = req;
                             let r;
-                            await doTransaction(remult, async () => {
+                            await decideTransaction(remult, options, async () => {
                                 d.args = await prepareReceivedArgs(getTypes(), d.args, remult, remult.dataProvider, res);
                                 if (remultStatic.allEntities.includes(constructor)) {
                                     let repo = remult.repo(constructor);
@@ -406,3 +423,10 @@ export async function prepareReceivedArgs(types, args, remult, ds, res) {
     return args;
 }
 export const classBackendMethodsArray = Symbol.for('classBackendMethodsArray');
+//
+async function decideTransaction(remult, options, what) {
+    if (options.transactional === undefined || options.transactional === true)
+        return await doTransaction(remult, what);
+    else
+        await what(remult.dataProvider);
+}

package/package.json

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