@nocobase/plugin-workflow 0.7.5-alpha.1 → 0.7.6-alpha.1

package/lib/client/components/CollectionFieldset.js

@@ -93,27 +93,61 @@ function _objectSpread(target) { for (var i = 1; i < arguments.length; i++) { va
 
 function _defineProperty(obj, key, value) { if (key in obj) { Object.defineProperty(obj, key, { value: value, enumerable: true, configurable: true, writable: true }); } else { obj[key] = value; } return obj; }
 
-// NOTE: observer for watching useProps
+function AssociationInput(props) {
+  var _data$config;
+
+  const _useCollectionManager = (0, _client().useCollectionManager)(),
+        getCollectionFields = _useCollectionManager.getCollectionFields;
+
+  const _useField = (0, _react2().useField)(),
+        path = _useField.path;
+
+  const fieldName = path.segments[path.segments.length - 1];
+
+  const _useForm = (0, _react2().useForm)(),
+        data = _useForm.values;
+
+  const fields = getCollectionFields(data === null || data === void 0 ? void 0 : (_data$config = data.config) === null || _data$config === void 0 ? void 0 : _data$config.collection);
+
+  const _fields$find = fields.find(item => item.name === fieldName),
+        type = _fields$find.type;
+
+  const value = Array.isArray(props.value) ? props.value.join(',') : props.value;
+
+  function onChange(ev) {
+    const trimed = ev.target.value.trim();
+    props.onChange(['belongsTo', 'hasOne'].includes(type) ? trimed : trimed.split(/[,\s]+/));
+  }
+
+  return _react().default.createElement(_antd().Input, _objectSpread(_objectSpread({}, props), {}, {
+    value: value,
+    onChange: onChange
+  }));
+} // NOTE: observer for watching useProps
+
+
 var _default = (0, _react2().observer)(({
   value,
   onChange: _onChange
 }) => {
-  var _data$config, _data$config2;
+  var _data$config2;
 
   const _useTranslation = (0, _reactI18next().useTranslation)(),
         t = _useTranslation.t;
 
   const compile = (0, _client().useCompile)();
 
-  const _useCollectionManager = (0, _client().useCollectionManager)(),
-        getCollection = _useCollectionManager.getCollection,
-        getCollectionFields = _useCollectionManager.getCollectionFields;
+  const _useCollectionManager2 = (0, _client().useCollectionManager)(),
+        getCollection = _useCollectionManager2.getCollection,
+        getCollectionFields = _useCollectionManager2.getCollectionFields;
 
-  const _useForm = (0, _react2().useForm)(),
-        data = _useForm.values;
+  const _useForm2 = (0, _react2().useForm)(),
+        data = _useForm2.values;
 
-  const collectionName = data === null || data === void 0 ? void 0 : (_data$config = data.config) === null || _data$config === void 0 ? void 0 : _data$config.collection;
-  const fields = getCollectionFields(data === null || data === void 0 ? void 0 : (_data$config2 = data.config) === null || _data$config2 === void 0 ? void 0 : _data$config2.collection).filter(field => !field.hidden && (field.uiSchema ? !field.uiSchema['x-read-pretty'] : false) && !['linkTo', 'hasMany', 'hasOne', 'belongsToMany'].includes(field.type));
+  const collectionName = data === null || data === void 0 ? void 0 : (_data$config2 = data.config) === null || _data$config2 === void 0 ? void 0 : _data$config2.collection;
+  const fields = getCollectionFields(collectionName).filter(field => !field.hidden && (field.uiSchema ? !field.uiSchema['x-read-pretty'] : false) // && (!['linkTo', 'hasMany', 'hasOne', 'belongsToMany'].includes(field.type))
+  );
+  const unassignedFields = fields.filter(field => !(field.name in value));
   return _react().default.createElement("fieldset", {
     className: (0, _css().css)`
       margin-top: .5em;
@@ -131,30 +165,20 @@ var _default = (0, _react2().observer)(({
   }, fields.filter(field => field.name in value).map(field => {
     var _field$uiSchema$title, _field$uiSchema;
 
-    const VTypes = _objectSpread({}, _calculators.VariableTypes);
-
-    let operand; // TODO: should refactor to support all types
-
-    if (field.type !== 'belongsTo') {
-      Object.assign(VTypes, {
-        constant: {
-          title: '{{t("Constant")}}',
-          value: 'constant',
-          options: undefined
-        }
-      });
-      operand = typeof value[field.name] === 'string' ? (0, _calculators.parseStringValue)(value[field.name], VTypes) : {
-        type: 'constant',
-        value: value[field.name]
-      };
-    } else {
-      delete VTypes.constant;
-      operand = typeof value[field.name] === 'string' ? (0, _calculators.parseStringValue)(value[field.name], VTypes) : {
-        type: '$context',
-        value: value[field.name]
-      };
-    } // TODO: try to use <ObjectField> to replace this map
+    const VTypes = _objectSpread(_objectSpread({}, ['linkTo', 'hasMany', 'belongsToMany'].includes(field.type) ? {} : _calculators.VariableTypes), {}, {
+      constant: {
+        title: '{{t("Constant")}}',
+        value: 'constant',
+        options: undefined
+      }
+    });
 
+    const operand = typeof value[field.name] === 'string' ? (0, _calculators.parseStringValue)(value[field.name], VTypes) : {
+      type: 'constant',
+      value: value[field.name]
+    }; // constant for associations to use Input, others to use CollectionField
+    // dynamic values only support belongsTo/hasOne association, other association type should disable
+    // TODO: try to use <ObjectField> to replace this map
 
     return _react().default.createElement(_antd().Form.Item, {
       key: field.name,
@@ -187,12 +211,13 @@ var _default = (0, _react2().observer)(({
         type: 'void',
         properties: {
           [field.name]: {
-            'x-component': 'CollectionField'
+            'x-component': ['linkTo', 'belongsTo', 'hasOne', 'hasMany', 'belongsToMany'].includes(field.type) ? 'AssociationInput' : 'CollectionField'
           }
         }
       },
       components: {
-        CollectionField: _client().CollectionField
+        CollectionField: _client().CollectionField,
+        AssociationInput
       }
     }) // ? <SchemaComponent schema={{ ...field.uiSchema, name: field.name }} />
     : null), _react().default.createElement(_antd().Button, {
@@ -206,7 +231,7 @@ var _default = (0, _react2().observer)(({
         _onChange(rest);
       }
     })));
-  }), Object.keys(value).length < fields.length ? _react().default.createElement(_antd().Dropdown, {
+  }), unassignedFields.length ? _react().default.createElement(_antd().Dropdown, {
     overlay: _react().default.createElement(_antd().Menu, {
       onClick: ({
         key
@@ -217,7 +242,7 @@ var _default = (0, _react2().observer)(({
                     max-height: 300px;
                     overflow-y: auto;
                   `
-    }, fields.filter(field => !(field.name in value)).map(field => {
+    }, unassignedFields.map(field => {
       var _field$uiSchema$title2, _field$uiSchema2;
 
       return _react().default.createElement(_antd().Menu.Item, {

package/lib/client/nodes/index.js

@@ -154,9 +154,10 @@ function useUpdateAction() {
           _antd().message.error(t('Node in executed workflow cannot be modified'));
 
           return;
-        }
+        } // TODO: how to do validation separately for each field? especially disabled for dynamic fields?
+        // await form.submit();
+
 
-        yield form.submit();
         yield api.resource('flow_nodes', data.id).update({
           filterByTk: data.id,
           values: {

package/lib/client/triggers/collection.js

@@ -103,7 +103,7 @@ const FieldsSelect = (0, _react2().observer)(props => {
     className: (0, _css().css)`
         min-width: 6em;
       `
-  }), fields.filter(field => !field.hidden && (field.uiSchema ? !field.uiSchema['x-read-pretty'] : true)).map(field => {
+  }), fields.filter(field => !field.hidden && (field.uiSchema ? !field.uiSchema['x-read-pretty'] : true) && !['linkTo', 'hasOne', 'hasMany', 'belongsToMany'].includes(field.type)).map(field => {
     var _field$uiSchema;
 
     return _react().default.createElement(_antd().Select.Option, {

package/lib/server/Plugin.d.ts

@@ -15,6 +15,8 @@ export default class WorkflowPlugin extends Plugin {
     getName(): string;
     load(): Promise<void>;
     toggle(workflow: WorkflowModel, enable?: boolean): void;
-    trigger(workflow: WorkflowModel, context: Object, options?: Transactionable): Promise<ExecutionModel | null>;
+    trigger(workflow: WorkflowModel, context: Object, options?: Transactionable & {
+        context?: any;
+    }): Promise<ExecutionModel | null>;
     createProcessor(execution: ExecutionModel, options?: {}): Processor;
 }

package/lib/server/Plugin.js

@@ -272,7 +272,8 @@ class WorkflowPlugin extends _server().Plugin {
       execution.workflow = workflow;
 
       const processor = _this3.createProcessor(execution, {
-        transaction
+        transaction,
+        _context: options.context
       });
 
       yield processor.start(); // @ts-ignore

package/lib/server/Processor.d.ts

@@ -4,11 +4,12 @@ import ExecutionModel from './models/Execution';
 import JobModel from './models/Job';
 import FlowNodeModel from './models/FlowNode';
 export interface ProcessorOptions extends Transactionable {
+    _context?: any;
     plugin: Plugin;
 }
 export default class Processor {
     execution: ExecutionModel;
-    private options;
+    options: ProcessorOptions;
     static StatusMap: {
         [x: number]: number;
     };

package/lib/server/actions/workflows.js

@@ -75,7 +75,20 @@ function migrateConfig(config, oldToNew) {
         return value.map(item => migrate(item));
 
       case 'string':
-        return value.replace(/(\{\{\$jobsMapByNodeId\.)([\w-]+)/, (_, prefix, id) => `${prefix}${oldToNew.get(Number.parseInt(id, 10)).id}`);
+        const matcher = value.match(/(\{\{\$jobsMapByNodeId\.)([\w-]+)/);
+
+        if (!matcher) {
+          return value;
+        }
+
+        const oldNodeId = Number.parseInt(matcher[2], 10);
+        const newNode = oldToNew.get(oldNodeId);
+
+        if (!newNode) {
+          throw new Error('node configurated for result is not existed');
+        }
+
+        return value.replace(matcher[0], `{{$jobsMapByNodeId.${newNode.id}`);
 
       default:
         return value;
@@ -171,10 +184,18 @@ function _revision() {
             const oldNode = originalNodesMap.get(oldId);
             const newUpstream = oldNode.upstreamId ? oldToNew.get(oldNode.upstreamId) : null;
             const newDownstream = oldNode.downstreamId ? oldToNew.get(oldNode.downstreamId) : null;
+            let migratedConfig;
+
+            try {
+              migratedConfig = migrateConfig(oldNode.config, oldToNew);
+            } catch (err) {
+              return context.throw(400, err.message);
+            }
+
             yield newNode.update({
               upstreamId: (_newUpstream$id = newUpstream === null || newUpstream === void 0 ? void 0 : newUpstream.id) !== null && _newUpstream$id !== void 0 ? _newUpstream$id : null,
               downstreamId: (_newDownstream$id = newDownstream === null || newDownstream === void 0 ? void 0 : newDownstream.id) !== null && _newDownstream$id !== void 0 ? _newDownstream$id : null,
-              config: migrateConfig(oldNode.config, oldToNew)
+              config: migratedConfig
             }, {
               transaction
             });

package/lib/server/instructions/create.js

@@ -27,6 +27,7 @@ var _default = {
       const repo = node.constructor.database.getRepository(collection);
       const options = processor.getParsedValue(params);
       const result = yield repo.create(_objectSpread(_objectSpread({}, options), {}, {
+        context: processor.options._context,
         transaction: processor.transaction
       }));
       return {

package/lib/server/instructions/destroy.js

@@ -27,6 +27,7 @@ var _default = {
       const repo = node.constructor.database.getRepository(collection);
       const options = processor.getParsedValue(params);
       const result = yield repo.destroy(_objectSpread(_objectSpread({}, options), {}, {
+        context: processor.options._context,
         transaction: processor.transaction
       }));
       return {

package/lib/server/instructions/query.js

@@ -28,6 +28,7 @@ var _default = {
       const repo = node.constructor.database.getRepository(collection);
       const options = processor.getParsedValue(params);
       const result = yield (multiple ? repo.find : repo.findOne).call(repo, _objectSpread(_objectSpread({}, options), {}, {
+        context: processor.options._context,
         transaction: processor.transaction
       })); // NOTE: `toJSON()` to avoid getting undefined value from Proxied model instance (#380)
       // e.g. Object.prototype.hasOwnProperty.call(result, 'id') // false

package/lib/server/instructions/update.js

@@ -29,6 +29,7 @@ var _default = {
       const repo = node.constructor.database.getRepository(collection);
       const options = processor.getParsedValue(params);
       const result = yield repo.update(_objectSpread(_objectSpread({}, options), {}, {
+        context: processor.options._context,
         transaction: processor.transaction
       }));
       return {

package/lib/server/triggers/collection.js

@@ -37,6 +37,16 @@ MODE_BITMAP_EVENTS.set(MODE_BITMAP.DESTROY, 'afterDestroy');
 
 function getHookId(workflow, type) {
   return `${type}#${workflow.id}`;
+}
+
+function getFieldRawName(collection, name) {
+  const field = collection.getField(name);
+
+  if (field && field.type === 'belongsTo') {
+    return field.foreignKey;
+  }
+
+  return name;
 } // async function, should return promise
 
 
@@ -49,29 +59,31 @@ function _handler() {
     var _condition$$and;
 
     const _workflow$config3 = workflow.config,
-          collection = _workflow$config3.collection,
+          collectionName = _workflow$config3.collection,
           condition = _workflow$config3.condition,
-          changed = _workflow$config3.changed; // NOTE: if no configured fields changed, do not trigger
+          changed = _workflow$config3.changed;
+    const collection = data.constructor.database.getCollection(collectionName); // NOTE: if no configured fields changed, do not trigger
 
-    if (changed && changed.length && changed.every(name => !data.changed(name))) {// TODO: temp comment out
-      // return;
+    if (changed && changed.length && changed.filter(name => !['linkTo', 'hasOne', 'hasMany', 'belongsToMany'].includes(collection.getField(name).type)).every(name => !data.changedWithAssociations(getFieldRawName(collection, name)))) {
+      // TODO: temp comment out
+      return;
     } // NOTE: if no configured condition match, do not trigger
 
 
     if (condition && ((_condition$$and = condition.$and) === null || _condition$$and === void 0 ? void 0 : _condition$$and.length)) {
       // TODO: change to map filter format to calculation format
       // const calculation = toCalculation(condition);
-      const _data$constructor$dat = data.constructor.database.getCollection(collection),
-            repository = _data$constructor$dat.repository,
-            model = _data$constructor$dat.model;
-
-      const transaction = options.transaction;
+      const repository = collection.repository,
+            model = collection.model;
+      const transaction = options.transaction,
+            context = options.context;
       const count = yield repository.count({
         filter: {
           $and: [condition, {
             [model.primaryKeyAttribute]: data[model.primaryKeyAttribute]
           }]
         },
+        context,
         transaction
       });
 
@@ -83,6 +95,7 @@ function _handler() {
     return this.plugin.trigger(workflow, {
       data: data.get()
     }, {
+      context: options.context,
       transaction: options.transaction
     });
   });

package/lib/server/triggers/schedule.d.ts

@@ -15,9 +15,10 @@ export declare const SCHEDULE_MODE: {
     readonly CONSTANT: 0;
     readonly COLLECTION_FIELD: 1;
 };
+declare function matchNext(this: ScheduleTrigger, workflow: any, now: Date, range?: number): boolean;
 export default class ScheduleTrigger extends Trigger {
-    static CacheRules: ((workflow: any, now: any) => any)[];
-    static TriggerRules: ((workflow: any, now: any) => boolean)[];
+    static CacheRules: (typeof matchNext | ((workflow: any, now: any) => any))[];
+    static TriggerRules: ((workflow: any, now: any) => any)[];
     events: Map<any, any>;
     private timer;
     private cache;
@@ -36,3 +37,4 @@ export default class ScheduleTrigger extends Trigger {
     on(workflow: any): void;
     off(workflow: any): void;
 }
+export {};

package/lib/server/triggers/schedule.js

@@ -56,23 +56,26 @@ ScheduleModes.set(SCHEDULE_MODE.CONSTANT, {
           endsOn = _workflow$config.endsOn,
           repeat = _workflow$config.repeat;
     const timestamp = now.getTime();
+    const startTime = Date.parse(startsOn);
 
-    if (startsOn) {
-      const startTime = Date.parse(startsOn);
-
-      if (!startTime || startTime > timestamp + this.cacheCycle) {
-        return false;
-      }
+    if (!startTime || startTime > timestamp + this.cacheCycle) {
+      return false;
+    }
 
+    if (repeat) {
       if (typeof repeat === 'number' && repeat > this.cacheCycle && (timestamp - startTime) % repeat > this.cacheCycle) {
         return false;
       }
-    }
 
-    if (endsOn) {
-      const endTime = Date.parse(endsOn);
+      if (endsOn) {
+        const endTime = Date.parse(endsOn);
 
-      if (!endTime || endTime <= timestamp) {
+        if (!endTime || endTime <= timestamp) {
+          return false;
+        }
+      }
+    } else {
+      if (startTime < timestamp) {
         return false;
       }
     }
@@ -80,40 +83,45 @@ ScheduleModes.set(SCHEDULE_MODE.CONSTANT, {
     return true;
   },
 
-  trigger(workflow, date) {
+  trigger(workflow, now) {
     const _workflow$config2 = workflow.config,
           startsOn = _workflow$config2.startsOn,
           endsOn = _workflow$config2.endsOn,
           repeat = _workflow$config2.repeat;
+    const timestamp = now.getTime();
+    const startTime = Math.floor(Date.parse(startsOn) / 1000) * 1000;
 
-    if (startsOn && typeof repeat === 'number') {
-      const startTime = Math.floor(Date.parse(startsOn) / 1000) * 1000;
+    if (repeat) {
+      if (typeof repeat === 'number') {
+        if (Math.round(timestamp - startTime) % repeat) {
+          return;
+        }
+      }
 
-      if (Math.round(date.getTime() - startTime) % repeat) {
+      if (endsOn) {
+        const endTime = Date.parse(endsOn);
+
+        if (!endTime || endTime < timestamp) {
+          return false;
+        }
+      }
+    } else {
+      if (startTime !== timestamp) {
         return;
       }
     }
 
     return this.plugin.trigger(workflow, {
-      date
+      date: now
     });
   }
 
 });
 
-function getDateRangeFilter(on, now, dir) {
-  const timestamp = now.getTime();
-  const op = dir < 0 ? _sequelize().Op.lt : _sequelize().Op.gte;
-
+function getOnTimestampWithOffset(on, now) {
   switch (typeof on) {
     case 'string':
-      const time = Date.parse(on);
-
-      if (!time || (dir < 0 ? timestamp < time : time <= timestamp)) {
-        return null;
-      }
-
-      break;
+      return Date.parse(on);
 
     case 'object':
       const field = on.field,
@@ -123,20 +131,17 @@ function getDateRangeFilter(on, now, dir) {
             unit = _on$unit === void 0 ? 1000 : _on$unit;
 
       if (!field) {
-        return {};
+        return null;
       }
 
-      return {
-        [field]: {
-          [op]: new Date(timestamp + offset * unit * dir)
-        }
-      };
+      const timestamp = now.getTime(); // onDate + offset > now
+      // onDate > now - offset
+
+      return timestamp - offset * unit;
 
     default:
-      break;
+      return null;
   }
-
-  return {};
 }
 
 function getDataOptionTime(data, on, dir = 1) {
@@ -151,7 +156,7 @@ function getDataOptionTime(data, on, dir = 1) {
             offset = _on$offset2 === void 0 ? 0 : _on$offset2,
             _on$unit2 = on.unit,
             unit = _on$unit2 === void 0 ? 1000 : _on$unit2;
-      return data[field] ? data[field].getTime() - offset * unit * dir : null;
+      return data.get(field) ? data.get(field).getTime() - offset * unit * dir : null;
 
     default:
       return null;
@@ -189,49 +194,49 @@ ScheduleModes.set(SCHEDULE_MODE.COLLECTION_FIELD, {
     const event = `${collection}.afterSave`;
     const name = getHookId(workflow, event);
 
-    if (!this.events.has(name)) {
-      // NOTE: toggle cache depends on new date
-      const listener = /*#__PURE__*/function () {
-        var _ref = _asyncToGenerator(function* (data, options) {
-          const now = new Date();
-          now.setMilliseconds(0);
-          const timestamp = now.getTime();
-          const startTime = getDataOptionTime(data, startsOn);
-          const endTime = getDataOptionTime(data, endsOn, -1);
-
-          if (!startTime && !repeat) {
-            return;
-          }
+    if (this.events.has(name)) {
+      return;
+    } // NOTE: toggle cache depends on new date
 
-          if (startTime && startTime > timestamp + _this.cacheCycle) {
-            return;
-          }
 
-          if (endTime && endTime <= timestamp) {
-            return;
-          }
+    const listener = /*#__PURE__*/function () {
+      var _ref = _asyncToGenerator(function* (data, options) {
+        const now = new Date();
+        now.setMilliseconds(0);
+        const timestamp = now.getTime();
+        const startTime = getDataOptionTime(data, startsOn);
+        const endTime = getDataOptionTime(data, endsOn, -1);
 
-          if (!nextInCycle.call(_this, workflow, now)) {
-            return;
-          }
+        if (!startTime && !repeat) {
+          return;
+        }
 
-          if (typeof repeat === 'number' && repeat > _this.cacheCycle && (timestamp - startTime) % repeat > _this.cacheCycle) {
-            return;
-          }
+        if (startTime && startTime > timestamp + _this.cacheCycle) {
+          return;
+        }
 
-          console.log('set cache', now);
+        if (endTime && endTime <= timestamp) {
+          return;
+        }
 
-          _this.setCache(workflow);
-        });
+        if (!matchNext.call(_this, workflow, now)) {
+          return;
+        }
 
-        return function listener(_x, _x2) {
-          return _ref.apply(this, arguments);
-        };
-      }();
+        if (typeof repeat === 'number' && repeat > _this.cacheCycle && (timestamp - startTime) % repeat > _this.cacheCycle) {
+          return;
+        }
 
-      this.events.set(name, listener);
-      this.plugin.app.db.on(`${collection}.afterSave`, listener);
-    }
+        _this.setCache(workflow);
+      });
+
+      return function listener(_x, _x2) {
+        return _ref.apply(this, arguments);
+      };
+    }();
+
+    this.events.set(name, listener);
+    this.plugin.app.db.on(event, listener);
   },
 
   off(workflow) {
@@ -242,7 +247,7 @@ ScheduleModes.set(SCHEDULE_MODE.COLLECTION_FIELD, {
     if (this.events.has(name)) {
       const listener = this.events.get(name);
       this.events.delete(name);
-      this.plugin.app.db.off(`${collection}.afterSave`, listener);
+      this.plugin.app.db.off(event, listener);
     }
   },
 
@@ -250,34 +255,60 @@ ScheduleModes.set(SCHEDULE_MODE.COLLECTION_FIELD, {
     var _this2 = this;
 
     return _asyncToGenerator(function* () {
+      const db = _this2.plugin.app.db;
       const _workflow$config4 = workflow.config,
             startsOn = _workflow$config4.startsOn,
             endsOn = _workflow$config4.endsOn,
             repeat = _workflow$config4.repeat,
             collection = _workflow$config4.collection;
-      const starts = getDateRangeFilter(startsOn, now, -1);
+      const timestamp = now.getTime();
+      const startTimestamp = getOnTimestampWithOffset(startsOn, now);
 
-      if (!starts || !Object.keys(starts).length) {
+      if (!startTimestamp) {
         return false;
       }
 
-      const ends = getDateRangeFilter(endsOn, now, 1);
+      const conditions = [{
+        [startsOn.field]: {
+          [_sequelize().Op.lt]: new Date(startTimestamp + _this2.cacheCycle)
+        }
+      }]; // when repeat is number, means repeat after startsOn
+      // (now - startsOn) % repeat <= cacheCycle
 
-      if (!ends) {
-        return false;
-      }
+      if (repeat) {
+        const tsFn = DialectTimestampFnMap[db.options.dialect];
 
-      const conditions = [starts, ends].filter(item => Boolean(Object.keys(item).length)); // when repeat is number, means repeat after startsOn
-      // (now - startsOn) % repeat <= cacheCycle
+        if (typeof repeat === 'number' && repeat > _this2.cacheCycle && tsFn) {
+          conditions.push((0, _sequelize().where)((0, _sequelize().fn)('MOD', (0, _sequelize().literal)(`${Math.round(timestamp / 1000)} - ${tsFn(startsOn.field)}`), Math.round(repeat / 1000)), {
+            [_sequelize().Op.lt]: Math.round(_this2.cacheCycle / 1000)
+          })); // conditions.push(literal(`mod(${timestamp} - ${tsFn(startsOn.field)} * 1000, ${repeat}) < ${this.cacheCycle}`));
+        }
 
-      const db = _this2.plugin.app.db;
-      const tsFn = DialectTimestampFnMap[db.options.dialect];
+        if (endsOn) {
+          const endTimestamp = getOnTimestampWithOffset(endsOn, now);
 
-      if (repeat && typeof repeat === 'number' && repeat > _this2.cacheCycle && tsFn) {
-        const uts = now.getTime();
-        conditions.push((0, _sequelize().where)((0, _sequelize().fn)('MOD', (0, _sequelize().literal)(`${Math.round(uts / 1000)} - ${tsFn(startsOn.field)}`), Math.round(repeat / 1000)), {
-          [_sequelize().Op.lt]: Math.round(_this2.cacheCycle / 1000)
-        })); // conditions.push(literal(`mod(${uts} - ${tsFn(startsOn.field)} * 1000, ${repeat}) < ${this.cacheCycle}`));
+          if (!endTimestamp) {
+            return false;
+          }
+
+          if (typeof endsOn === 'string') {
+            if (endTimestamp <= timestamp) {
+              return false;
+            }
+          } else {
+            conditions.push({
+              [endsOn.field]: {
+                [_sequelize().Op.gte]: new Date(endTimestamp + _this2.interval)
+              }
+            });
+          }
+        }
+      } else {
+        conditions.push({
+          [startsOn.field]: {
+            [_sequelize().Op.gte]: new Date(startTimestamp)
+          }
+        });
       }
 
       const _db$getCollection = db.getCollection(collection),
@@ -292,35 +323,29 @@ ScheduleModes.set(SCHEDULE_MODE.COLLECTION_FIELD, {
     })();
   },
 
-  trigger(workflow, date) {
+  trigger(workflow, now) {
     var _this3 = this;
 
     return _asyncToGenerator(function* () {
-      var _startsOn$offset, _startsOn$unit;
-
       const _workflow$config5 = workflow.config,
-            collection = _workflow$config5.collection,
             startsOn = _workflow$config5.startsOn,
+            repeat = _workflow$config5.repeat,
             endsOn = _workflow$config5.endsOn,
-            repeat = _workflow$config5.repeat;
+            collection = _workflow$config5.collection;
+      const timestamp = now.getTime();
+      const startTimestamp = getOnTimestampWithOffset(startsOn, now);
 
-      if (typeof startsOn !== 'object') {
-        return;
+      if (!startTimestamp) {
+        return false;
       }
 
-      const timestamp = date.getTime();
-      const startTimestamp = timestamp - ((_startsOn$offset = startsOn.offset) !== null && _startsOn$offset !== void 0 ? _startsOn$offset : 0) * ((_startsOn$unit = startsOn.unit) !== null && _startsOn$unit !== void 0 ? _startsOn$unit : 1000);
-      const conditions = [];
+      const conditions = [{
+        [startsOn.field]: {
+          [_sequelize().Op.lt]: new Date(startTimestamp + _this3.interval)
+        }
+      }];
 
-      if (!repeat) {
-        // startsOn exactly equal to now in 1s
-        conditions.push({
-          [startsOn.field]: {
-            [_sequelize().Op.gte]: new Date(startTimestamp),
-            [_sequelize().Op.lt]: new Date(startTimestamp + 1000)
-          }
-        });
-      } else {
+      if (repeat) {
         // startsOn not after now
         conditions.push({
           [startsOn.field]: {
@@ -335,32 +360,32 @@ ScheduleModes.set(SCHEDULE_MODE.COLLECTION_FIELD, {
           })); // conditions.push(literal(`MOD(CAST(${timestamp} AS BIGINT) - CAST((FLOOR(${tsFn(startsOn.field)}) AS BIGINT) * 1000), ${repeat}) = 0`));
         }
 
-        switch (typeof endsOn) {
-          case 'string':
-            const endTime = Date.parse(endsOn);
-
-            if (!endTime || endTime <= timestamp) {
-              return;
-            }
-
-            break;
+        if (endsOn) {
+          const endTimestamp = getOnTimestampWithOffset(endsOn, now);
 
-          case 'object':
-            if (endsOn.field) {
-              var _endsOn$offset, _endsOn$unit;
+          if (!endTimestamp) {
+            return false;
+          }
 
-              conditions.push({
-                [endsOn.field]: {
-                  [_sequelize().Op.gte]: new Date(timestamp - ((_endsOn$offset = endsOn.offset) !== null && _endsOn$offset !== void 0 ? _endsOn$offset : 0) * ((_endsOn$unit = endsOn.unit) !== null && _endsOn$unit !== void 0 ? _endsOn$unit : 1000) + 1000)
-                }
-              });
+          if (typeof endsOn === 'string') {
+            if (endTimestamp <= timestamp) {
+              return false;
             }
-
-            break;
-
-          default:
-            break;
+          } else {
+            conditions.push({
+              [endsOn.field]: {
+                [_sequelize().Op.gte]: new Date(endTimestamp + _this3.interval)
+              }
+            });
+          }
         }
+      } else {
+        // startsOn exactly equal to now in 1s
+        conditions.push({
+          [startsOn.field]: {
+            [_sequelize().Op.gte]: new Date(startTimestamp)
+          }
+        });
       }
 
       const _this3$plugin$app$db$ = _this3.plugin.app.db.getCollection(collection),
@@ -373,12 +398,12 @@ ScheduleModes.set(SCHEDULE_MODE.COLLECTION_FIELD, {
       });
 
       if (instances.length) {
-        console.log(instances.length, 'rows trigger at', date);
+        console.log(instances.length, 'rows trigger at', now);
       }
 
       instances.forEach(item => {
         _this3.plugin.trigger(workflow, {
-          date,
+          date: now,
           data: item.get()
         });
       });
@@ -387,24 +412,16 @@ ScheduleModes.set(SCHEDULE_MODE.COLLECTION_FIELD, {
 
 });
 
-function nextInCycle(workflow, now) {
+function matchNext(workflow, now, range = this.cacheCycle) {
   const repeat = workflow.config.repeat; // no repeat means no need to rerun
   // but if in current cycle, should be put in cache
   // no repeat but in current cycle means startsOn has been configured
   // so we need to more info to determine if necessary config items
 
-  if (!repeat) {
+  if (typeof repeat !== 'string') {
     return true;
   }
 
-  switch (typeof repeat) {
-    case 'string':
-      break;
-
-    default:
-      return true;
-  }
-
   const currentDate = new Date(now);
   currentDate.setMilliseconds(-1);
   const timestamp = now.getTime();
@@ -415,7 +432,7 @@ function nextInCycle(workflow, now) {
 
   let next = interval.next(); // NOTE: cache all workflows will be matched in current cycle
 
-  if (next.getTime() - timestamp <= this.cacheCycle) {
+  if (next.getTime() - timestamp <= range) {
     return true;
   }
 
@@ -452,18 +469,20 @@ class ScheduleTrigger extends _.Trigger {
   }
 
   init() {
-    if (!this.timer) {
-      const now = new Date(); // NOTE: assign to this.timer to avoid duplicated initialization
-
-      this.timer = setTimeout(() => {
-        this.timer = setInterval(this.run, this.interval); // initially trigger
-        // this.onTick(now);
-      }, // NOTE:
-      //  try to align to system time on each second starts,
-      //  after at least 1 second initialized for anything to get ready.
-      //  so jobs in 2 seconds will be missed at first start.
-      1000 - now.getMilliseconds());
+    if (this.timer) {
+      return;
     }
+
+    const now = new Date(); // NOTE: assign to this.timer to avoid duplicated initialization
+
+    this.timer = setTimeout(() => {
+      this.timer = setInterval(this.run, this.interval); // initially trigger
+      // this.onTick(now);
+    }, // NOTE:
+    //  try to align to system time on each second starts,
+    //  after at least 1 second initialized for anything to get ready.
+    //  so jobs in 2 seconds will be missed at first start.
+    1000 - now.getMilliseconds());
   }
 
   onTick(now) {
@@ -631,13 +650,10 @@ class ScheduleTrigger extends _.Trigger {
 }
 
 exports.default = ScheduleTrigger;
-ScheduleTrigger.CacheRules = [// ({ enabled }) => enabled,
-({
+ScheduleTrigger.CacheRules = [({
   config,
   allExecuted
-}) => config.limit ? allExecuted < config.limit : true, ({
-  config
-}) => ['repeat', 'startsOn'].some(key => config[key]), nextInCycle, function (workflow, now) {
+}) => (config.limit ? allExecuted < config.limit : true) && config.startsOn, matchNext, function (workflow, now) {
   const mode = workflow.config.mode;
   const modeHandlers = ScheduleModes.get(mode);
   return modeHandlers.shouldCache.call(this, workflow, now);
@@ -645,28 +661,6 @@ ScheduleTrigger.CacheRules = [// ({ enabled }) => enabled,
 ScheduleTrigger.TriggerRules = [({
   config,
   allExecuted
-}) => config.limit ? allExecuted < config.limit : true, ({
-  config
-}) => ['repeat', 'startsOn'].some(key => config[key]), function (workflow, now) {
-  const repeat = workflow.config.repeat;
-
-  if (typeof repeat !== 'string') {
-    return true;
-  }
-
-  const currentDate = new Date(now);
-  currentDate.setMilliseconds(-1);
-  const timestamp = now.getTime();
-
-  const interval = _cronParser().default.parseExpression(repeat, {
-    currentDate
-  });
-
-  let next = interval.next();
-
-  if (next.getTime() === timestamp) {
-    return true;
-  }
-
-  return false;
+}) => (config.limit ? allExecuted < config.limit : true) && config.startsOn, function (workflow, now) {
+  return matchNext.call(this, workflow, now, 0);
 }];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nocobase/plugin-workflow",
-  "version": "0.7.5-alpha.1",
+  "version": "0.7.6-alpha.1",
   "main": "lib/index.js",
   "license": "Apache-2.0",
   "licenses": [
@@ -10,17 +10,17 @@
     }
   ],
   "dependencies": {
-    "@nocobase/actions": "0.7.5-alpha.1",
-    "@nocobase/client": "0.7.5-alpha.1",
-    "@nocobase/database": "0.7.5-alpha.1",
-    "@nocobase/server": "0.7.5-alpha.1",
-    "@nocobase/utils": "0.7.5-alpha.1",
+    "@nocobase/actions": "0.7.6-alpha.1",
+    "@nocobase/client": "0.7.6-alpha.1",
+    "@nocobase/database": "0.7.6-alpha.1",
+    "@nocobase/server": "0.7.6-alpha.1",
+    "@nocobase/utils": "0.7.6-alpha.1",
     "cron-parser": "4.4.0",
     "json-templates": "^4.2.0",
     "react-js-cron": "^1.4.0"
   },
   "devDependencies": {
-    "@nocobase/test": "0.7.5-alpha.1"
+    "@nocobase/test": "0.7.6-alpha.1"
   },
-  "gitHead": "f6eb27b68185bb0c0b4c2cfca1df84205a9b9173"
+  "gitHead": "f20ce011a9ac516dc6aec110979f063a0e63f923"
 }