bpmn-elements 7.0.0 → 8.1.0

package/dist/src/activity/Activity.js

@@ -23,19 +23,19 @@ var _Errors = require("../error/Errors");
 
 function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }
 
-const activityDefSymbol = Symbol.for('activityDefinition');
-const bpmnIoSymbol = Symbol.for('bpmnIo');
-const consumingSymbol = Symbol.for('consuming');
-const countersSymbol = Symbol.for('counters');
-const eventDefinitionsSymbol = Symbol.for('eventDefinitions');
-const execSymbol = Symbol.for('exec');
-const executeMessageSymbol = Symbol.for('executeMessage');
-const extensionsSymbol = Symbol.for('extensions');
-const flagsSymbol = Symbol.for('flags');
-const flowsSymbol = Symbol.for('flows');
-const formatterSymbol = Symbol.for('formatter');
-const messageHandlersSymbol = Symbol.for('messageHandlers');
-const stateMessageSymbol = Symbol.for('stateMessage');
+const kActivityDef = Symbol.for('activityDefinition');
+const kBpmnIo = Symbol.for('bpmnIo');
+const kConsuming = Symbol.for('consuming');
+const kCounters = Symbol.for('counters');
+const kEventDefinitions = Symbol.for('eventDefinitions');
+const kExec = Symbol.for('exec');
+const kExecuteMessage = Symbol.for('executeMessage');
+const kExtensions = Symbol.for('extensions');
+const kFlags = Symbol.for('flags');
+const kFlows = Symbol.for('flows');
+const kFormatter = Symbol.for('formatter');
+const kMessageHandlers = Symbol.for('messageHandlers');
+const kStateMessage = Symbol.for('stateMessage');
 var _default = Activity;
 exports.default = _default;
 
@@ -50,7 +50,7 @@ function Activity(Behaviour, activityDef, context) {
     attachedTo: attachedToRef,
     eventDefinitions
   } = behaviour;
-  this[activityDefSymbol] = activityDef;
+  this[kActivityDef] = activityDef;
   this.id = id;
   this.type = type;
   this.name = name;
@@ -62,7 +62,7 @@ function Activity(Behaviour, activityDef, context) {
   this.logger = context.environment.Logger(type.toLowerCase());
   this.environment = context.environment;
   this.context = context;
-  this[countersSymbol] = {
+  this[kCounters] = {
     taken: 0,
     discarded: 0
   };
@@ -89,7 +89,7 @@ function Activity(Behaviour, activityDef, context) {
   const inboundAssociations = context.getInboundAssociations(id);
   const inboundTriggers = attachedToActivity ? [attachedToActivity] : inboundSequenceFlows.slice();
   const outboundSequenceFlows = context.getOutboundSequenceFlows(id);
-  const flows = this[flowsSymbol] = {
+  const flows = this[kFlows] = {
     inboundSequenceFlows,
     inboundAssociations,
     inboundJoinFlows: [],
@@ -99,7 +99,7 @@ function Activity(Behaviour, activityDef, context) {
   };
   const isForCompensation = !!behaviour.isForCompensation;
   const isParallelJoin = activityDef.isParallelGateway && flows.inboundSequenceFlows.length > 1;
-  this[flagsSymbol] = {
+  this[kFlags] = {
     isEnd: flows.outboundSequenceFlows.length === 0,
     isStart: flows.inboundSequenceFlows.length === 0 && !attachedTo && !behaviour.triggeredByEvent && !isForCompensation,
     isSubProcess: activityDef.isSubProcess,
@@ -110,8 +110,8 @@ function Activity(Behaviour, activityDef, context) {
     isParallelJoin,
     isThrowing: activityDef.isThrowing
   };
-  this[execSymbol] = {};
-  this[messageHandlersSymbol] = {
+  this[kExec] = {};
+  this[kMessageHandlers] = {
     onInbound: isParallelJoin ? this._onJoinInbound.bind(this) : this._onInbound.bind(this),
     onRunMessage: this._onRunMessage.bind(this),
     onApiMessage: this._onApiMessage.bind(this),
@@ -144,7 +144,7 @@ function Activity(Behaviour, activityDef, context) {
     }
   }
 
-  this[eventDefinitionsSymbol] = eventDefinitions && eventDefinitions.map(ed => new ed.Behaviour(this, ed, this.context));
+  this[kEventDefinitions] = eventDefinitions && eventDefinitions.map(ed => new ed.Behaviour(this, ed, this.context));
 }
 
 const proto = Activity.prototype;
@@ -152,7 +152,7 @@ Object.defineProperty(proto, 'counters', {
   enumerable: true,
 
   get() {
-    return { ...this[countersSymbol]
+    return { ...this[kCounters]
     };
   }
 
@@ -161,7 +161,7 @@ Object.defineProperty(proto, 'execution', {
   enumerable: true,
 
   get() {
-    return this[execSymbol].execution;
+    return this[kExec].execution;
   }
 
 });
@@ -169,7 +169,7 @@ Object.defineProperty(proto, 'executionId', {
   enumerable: true,
 
   get() {
-    return this[execSymbol].executionId;
+    return this[kExec].executionId;
   }
 
 });
@@ -177,8 +177,8 @@ Object.defineProperty(proto, 'bpmnIo', {
   enumerable: true,
 
   get() {
-    if (bpmnIoSymbol in this) return this[bpmnIoSymbol];
-    const bpmnIo = this[bpmnIoSymbol] = new _BpmnIO.default(this, this.context);
+    if (kBpmnIo in this) return this[kBpmnIo];
+    const bpmnIo = this[kBpmnIo] = new _BpmnIO.default(this, this.context);
     return bpmnIo;
   }
 
@@ -187,8 +187,8 @@ Object.defineProperty(proto, 'extensions', {
   enumerable: true,
 
   get() {
-    if (extensionsSymbol in this) return this[extensionsSymbol];
-    const extensions = this[extensionsSymbol] = this.context.loadExtensions(this);
+    if (kExtensions in this) return this[kExtensions];
+    const extensions = this[kExtensions] = this.context.loadExtensions(this);
     return extensions;
   }
 
@@ -197,10 +197,10 @@ Object.defineProperty(proto, 'formatter', {
   enumerable: true,
 
   get() {
-    let formatter = this[formatterSymbol];
+    let formatter = this[kFormatter];
     if (formatter) return formatter;
     const broker = this.broker;
-    formatter = this[formatterSymbol] = new _MessageFormatter.Formatter({
+    formatter = this[kFormatter] = new _MessageFormatter.Formatter({
       id: this.id,
       broker,
       logger: this.logger
@@ -213,7 +213,7 @@ Object.defineProperty(proto, 'isRunning', {
   enumerable: true,
 
   get() {
-    if (!this[consumingSymbol]) return false;
+    if (!this[kConsuming]) return false;
     return !!this.status;
   }
 
@@ -222,7 +222,7 @@ Object.defineProperty(proto, 'outbound', {
   enumerable: true,
 
   get() {
-    return this[flowsSymbol].outboundSequenceFlows;
+    return this[kFlows].outboundSequenceFlows;
   }
 
 });
@@ -230,7 +230,7 @@ Object.defineProperty(proto, 'inbound', {
   enumerable: true,
 
   get() {
-    return this[flowsSymbol].inboundSequenceFlows;
+    return this[kFlows].inboundSequenceFlows;
   }
 
 });
@@ -238,7 +238,7 @@ Object.defineProperty(proto, 'isEnd', {
   enumerable: true,
 
   get() {
-    return this[flagsSymbol].isEnd;
+    return this[kFlags].isEnd;
   }
 
 });
@@ -246,7 +246,7 @@ Object.defineProperty(proto, 'isStart', {
   enumerable: true,
 
   get() {
-    return this[flagsSymbol].isStart;
+    return this[kFlags].isStart;
   }
 
 });
@@ -254,7 +254,7 @@ Object.defineProperty(proto, 'isSubProcess', {
   enumerable: true,
 
   get() {
-    return this[flagsSymbol].isSubProcess;
+    return this[kFlags].isSubProcess;
   }
 
 });
@@ -262,7 +262,7 @@ Object.defineProperty(proto, 'isMultiInstance', {
   enumerable: true,
 
   get() {
-    return this[flagsSymbol].isMultiInstance;
+    return this[kFlags].isMultiInstance;
   }
 
 });
@@ -270,7 +270,7 @@ Object.defineProperty(proto, 'isThrowing', {
   enumerable: true,
 
   get() {
-    return this[flagsSymbol].isThrowing;
+    return this[kFlags].isThrowing;
   }
 
 });
@@ -278,7 +278,7 @@ Object.defineProperty(proto, 'isForCompensation', {
   enumerable: true,
 
   get() {
-    return this[flagsSymbol].isForCompensation;
+    return this[kFlags].isForCompensation;
   }
 
 });
@@ -286,7 +286,7 @@ Object.defineProperty(proto, 'triggeredByEvent', {
   enumerable: true,
 
   get() {
-    return this[activityDefSymbol].triggeredByEvent;
+    return this[kActivityDef].triggeredByEvent;
   }
 
 });
@@ -294,7 +294,7 @@ Object.defineProperty(proto, 'attachedTo', {
   enumerable: true,
 
   get() {
-    const attachedToId = this[flagsSymbol].attachedTo;
+    const attachedToId = this[kFlags].attachedTo;
     if (!attachedToId) return null;
     return this.getActivityById(attachedToId);
   }
@@ -304,13 +304,13 @@ Object.defineProperty(proto, 'eventDefinitions', {
   enumerable: true,
 
   get() {
-    return this[eventDefinitionsSymbol];
+    return this[kEventDefinitions];
   }
 
 });
 
 proto.activate = function activate() {
-  if (this[flagsSymbol].isForCompensation) return;
+  if (this[kFlags].isForCompensation) return;
   return this._consumeInbound();
 };
 
@@ -322,7 +322,7 @@ proto.deactivate = function deactivate() {
 
 proto.init = function init(initContent) {
   const id = this.id;
-  const exec = this[execSymbol];
+  const exec = this[kExec];
   const executionId = exec.initExecutionId = exec.initExecutionId || (0, _shared.getUniqueId)(id);
   this.logger.debug(`<${id}> initialized with executionId <${executionId}>`);
 
@@ -334,7 +334,7 @@ proto.init = function init(initContent) {
 proto.run = function run(runContent) {
   const id = this.id;
   if (this.isRunning) throw new Error(`activity <${id}> is already running`);
-  const exec = this[execSymbol];
+  const exec = this[kExec];
   const executionId = exec.executionId = exec.initExecutionId || (0, _shared.getUniqueId)(id);
   exec.initExecutionId = null;
 
@@ -356,9 +356,9 @@ proto.recover = function recover(state) {
   if (!state) return;
   this.stopped = state.stopped;
   this.status = state.status;
-  const exec = this[execSymbol];
+  const exec = this[kExec];
   exec.executionId = state.executionId;
-  this[countersSymbol] = { ...this[countersSymbol],
+  this[kCounters] = { ...this[kCounters],
     ...state.counters
   };
 
@@ -371,7 +371,7 @@ proto.recover = function recover(state) {
 };
 
 proto.resume = function resume() {
-  if (this[consumingSymbol]) {
+  if (this[kConsuming]) {
     throw new Error(`cannot resume running activity <${this.id}>`);
   }
 
@@ -391,26 +391,26 @@ proto.resume = function resume() {
 
 proto.discard = function discard(discardContent) {
   if (!this.status) return this._runDiscard(discardContent);
-  const execution = this[execSymbol].execution;
+  const execution = this[kExec].execution;
   if (execution && !execution.completed) return execution.discard();
 
   this._deactivateRunConsumers();
 
   const broker = this.broker;
   broker.getQueue('run-q').purge();
-  broker.publish('run', 'run.discard', (0, _messageHelper.cloneContent)(this[stateMessageSymbol].content));
+  broker.publish('run', 'run.discard', (0, _messageHelper.cloneContent)(this[kStateMessage].content));
 
   this._consumeRunQ();
 };
 
 proto.stop = function stop() {
-  if (!this[consumingSymbol]) return;
+  if (!this[kConsuming]) return;
   return this.getApi().stop();
 };
 
 proto.next = function next() {
   if (!this.environment.settings.step) return;
-  const stateMessage = this[stateMessageSymbol];
+  const stateMessage = this[kStateMessage];
   if (!stateMessage) return;
   if (this.status === 'executing') return false;
   if (this.status === 'formatting') return false;
@@ -426,13 +426,13 @@ proto.shake = function shake() {
 };
 
 proto.evaluateOutbound = function evaluateOutbound(fromMessage, discardRestAtTake, callback) {
-  return this[flowsSymbol].outboundEvaluator.evaluate(fromMessage, discardRestAtTake, callback);
+  return this[kFlows].outboundEvaluator.evaluate(fromMessage, discardRestAtTake, callback);
 };
 
 proto.getState = function getState() {
   const msg = this._createMessage();
 
-  const exec = this[execSymbol];
+  const exec = this[kExec];
   return { ...msg,
     executionId: exec.executionId,
     stopped: this.stopped,
@@ -445,9 +445,9 @@ proto.getState = function getState() {
 };
 
 proto.getApi = function getApi(message) {
-  const execution = this[execSymbol].execution;
+  const execution = this[kExec].execution;
   if (execution && !execution.completed) return execution.getApi(message);
-  return (0, _Api.ActivityApi)(this.broker, message || this[stateMessageSymbol]);
+  return (0, _Api.ActivityApi)(this.broker, message || this[kStateMessage]);
 };
 
 proto.getActivityById = function getActivityById(elementId) {
@@ -455,7 +455,7 @@ proto.getActivityById = function getActivityById(elementId) {
 };
 
 proto._runDiscard = function runDiscard(discardContent) {
-  const exec = this[execSymbol];
+  const exec = this[kExec];
   const executionId = exec.executionId = exec.initExecutionId || (0, _shared.getUniqueId)(this.id);
   exec.initExecutionId = null;
 
@@ -473,7 +473,7 @@ proto._runDiscard = function runDiscard(discardContent) {
 proto._discardRun = function discardRun() {
   const status = this.status;
   if (!status) return;
-  const execution = this[execSymbol].execution;
+  const execution = this[kExec].execution;
   if (execution && !execution.completed) return;
 
   switch (status) {
@@ -488,7 +488,7 @@ proto._discardRun = function discardRun() {
   if (this.extensions) this.extensions.deactivate();
   const broker = this.broker;
   broker.getQueue('run-q').purge();
-  broker.publish('run', 'run.discard', (0, _messageHelper.cloneContent)(this[stateMessageSymbol].content));
+  broker.publish('run', 'run.discard', (0, _messageHelper.cloneContent)(this[kStateMessage].content));
 
   this._consumeRunQ();
 };
@@ -506,28 +506,28 @@ proto._shakeOutbound = function shakeOutbound(sourceMessage) {
     type: 'shake'
   });
 
-  if (this[flagsSymbol].isEnd) {
+  if (this[kFlags].isEnd) {
     return broker.publish('event', 'activity.shake.end', message.content, {
       persistent: false,
       type: 'shake'
     });
   }
 
-  for (const flow of this[flowsSymbol].outboundSequenceFlows) flow.shake(message);
+  for (const flow of this[kFlows].outboundSequenceFlows) flow.shake(message);
 };
 
 proto._consumeInbound = function consumeInbound() {
   if (this.status) return;
   const inboundQ = this.broker.getQueue('inbound-q');
 
-  if (this[flagsSymbol].isParallelJoin) {
-    return inboundQ.consume(this[messageHandlersSymbol].onInbound, {
+  if (this[kFlags].isParallelJoin) {
+    return inboundQ.consume(this[kMessageHandlers].onInbound, {
       consumerTag: '_run-on-inbound',
       prefetch: 1000
     });
   }
 
-  return inboundQ.consume(this[messageHandlersSymbol].onInbound, {
+  return inboundQ.consume(this[kMessageHandlers].onInbound, {
     consumerTag: '_run-on-inbound'
   });
 };
@@ -543,6 +543,7 @@ proto._onInbound = function onInbound(routingKey, message) {
   switch (routingKey) {
     case 'association.take':
     case 'flow.take':
+    case 'activity.restart':
     case 'activity.enter':
       return this.run({
         message: content.message,
@@ -580,7 +581,7 @@ proto._onJoinInbound = function onJoinInbound(routingKey, message) {
     inboundSequenceFlows,
     inboundJoinFlows,
     inboundTriggers
-  } = this[flowsSymbol];
+  } = this[kFlows];
   const idx = inboundJoinFlows.findIndex(msg => msg.content.id === content.id);
   inboundJoinFlows.push(message);
   if (idx > -1) return;
@@ -630,7 +631,7 @@ proto._onInboundEvent = function onInboundEvent(routingKey, message) {
     case 'activity.enter':
     case 'activity.discard':
       {
-        if (content.id === this[flagsSymbol].attachedTo) {
+        if (content.id === this[kFlags].attachedTo) {
           inboundQ.queueMessage(fields, (0, _messageHelper.cloneContent)(content), properties);
         }
 
@@ -655,7 +656,7 @@ proto._onInboundEvent = function onInboundEvent(routingKey, message) {
 
     case 'association.complete':
       {
-        if (!this[flagsSymbol].isForCompensation) break;
+        if (!this[kFlags].isForCompensation) break;
         inboundQ.queueMessage(fields, (0, _messageHelper.cloneContent)(content), properties);
         const compensationId = `${(0, _shared.brokerSafeId)(id)}_${(0, _shared.brokerSafeId)(content.sequenceId)}`;
 
@@ -671,9 +672,9 @@ proto._onInboundEvent = function onInboundEvent(routingKey, message) {
 };
 
 proto._consumeRunQ = function consumeRunQ() {
-  if (this[consumingSymbol]) return;
-  this[consumingSymbol] = true;
-  this.broker.getQueue('run-q').assertConsumer(this[messageHandlersSymbol].onRunMessage, {
+  if (this[kConsuming]) return;
+  this[kConsuming] = true;
+  this.broker.getQueue('run-q').assertConsumer(this[kMessageHandlers].onRunMessage, {
     exclusive: true,
     consumerTag: '_activity-run'
   });
@@ -709,7 +710,7 @@ proto._continueRunMessage = function continueRunMessage(routingKey, message) {
   const correlationId = message.properties.correlationId;
   const id = this.id;
   const step = this.environment.settings.step;
-  this[stateMessageSymbol] = message;
+  this[kStateMessage] = message;
 
   switch (routingKey) {
     case 'run.enter':
@@ -718,7 +719,7 @@ proto._continueRunMessage = function continueRunMessage(routingKey, message) {
         this.status = 'entered';
 
         if (!isRedelivered) {
-          this[execSymbol].execution = null;
+          this[kExec].execution = null;
         }
 
         if (this.extensions) this.extensions.activate((0, _messageHelper.cloneMessage)(message), this);
@@ -733,7 +734,7 @@ proto._continueRunMessage = function continueRunMessage(routingKey, message) {
       {
         this.logger.debug(`<${id}> discard`, isRedelivered ? 'redelivered' : '');
         this.status = 'discard';
-        this[execSymbol].execution = null;
+        this[kExec].execution = null;
         if (this.extensions) this.extensions.activate((0, _messageHelper.cloneMessage)(message), this);
         if (this.bpmnIo) this.bpmnIo.activate(message);
 
@@ -771,7 +772,7 @@ proto._continueRunMessage = function continueRunMessage(routingKey, message) {
         const execution = this.execution;
 
         if (!isRedelivered && execution) {
-          this[executeMessageSymbol] = message;
+          this[kExecuteMessage] = message;
           return execution.passthrough(message);
         }
       }
@@ -779,12 +780,12 @@ proto._continueRunMessage = function continueRunMessage(routingKey, message) {
     case 'run.execute':
       {
         this.status = 'executing';
-        this[executeMessageSymbol] = message;
-        this.broker.getQueue('execution-q').assertConsumer(this[messageHandlersSymbol].onExecutionMessage, {
+        this[kExecuteMessage] = message;
+        this.broker.getQueue('execution-q').assertConsumer(this[kMessageHandlers].onExecutionMessage, {
           exclusive: true,
           consumerTag: '_activity-execution'
         });
-        const exec = this[execSymbol];
+        const exec = this[kExec];
         if (!exec.execution) exec.execution = new _ActivityExecution.default(this, this.context);
 
         if (isRedelivered) {
@@ -802,9 +803,8 @@ proto._continueRunMessage = function continueRunMessage(routingKey, message) {
     case 'run.end':
       {
         if (this.status === 'end') break;
-        this[countersSymbol].taken++;
+        this[kCounters].taken++;
         this.status = 'end';
-        if (isRedelivered) break;
         return this._doRunLeave(message, false, () => {
           this._publishEvent('end', content, {
             correlationId
@@ -828,7 +828,7 @@ proto._continueRunMessage = function continueRunMessage(routingKey, message) {
     case 'run.discarded':
       {
         this.logger.debug(`<${content.executionId} (${id})> discarded`);
-        this[countersSymbol].discarded++;
+        this[kCounters].discarded++;
         this.status = 'discarded';
         content.outbound = undefined;
 
@@ -886,7 +886,7 @@ proto._continueRunMessage = function continueRunMessage(routingKey, message) {
 };
 
 proto._onExecutionMessage = function onExecutionMessage(routingKey, message) {
-  const executeMessage = this[executeMessageSymbol];
+  const executeMessage = this[kExecuteMessage];
   const content = (0, _messageHelper.cloneContent)({ ...executeMessage.content,
     ...message.content,
     executionId: executeMessage.content.executionId,
@@ -949,8 +949,8 @@ proto._onExecutionMessage = function onExecutionMessage(routingKey, message) {
 
 proto._ackRunExecuteMessage = function ackRunExecuteMessage() {
   if (this.environment.settings.step) return;
-  const executeMessage = this[executeMessageSymbol];
-  this[executeMessageSymbol] = null;
+  const executeMessage = this[kExecuteMessage];
+  this[kExecuteMessage] = null;
   executeMessage.ack();
 };
 
@@ -977,9 +977,9 @@ proto._doRunLeave = function doRunLeave(message, isDiscarded, onOutbound) {
       });
     }
 
-    this.broker.publish('run', 'run.leave', (0, _messageHelper.cloneContent)(content, { ...(outbound.length ? {
+    this.broker.publish('run', 'run.leave', (0, _messageHelper.cloneContent)(content, { ...(outbound.length && {
         outbound
-      } : undefined)
+      })
     }), {
       correlationId
     });
@@ -988,12 +988,12 @@ proto._doRunLeave = function doRunLeave(message, isDiscarded, onOutbound) {
 };
 
 proto._doOutbound = function doOutbound(fromMessage, isDiscarded, callback) {
-  const outboundSequenceFlows = this[flowsSymbol].outboundSequenceFlows;
+  const outboundSequenceFlows = this[kFlows].outboundSequenceFlows;
   if (!outboundSequenceFlows.length) return callback(null, []);
   const fromContent = fromMessage.content;
   let discardSequence = fromContent.discardSequence;
 
-  if (isDiscarded && !discardSequence && this[flagsSymbol].attachedTo && fromContent.inbound && fromContent.inbound[0]) {
+  if (isDiscarded && !discardSequence && this[kFlags].attachedTo && fromContent.inbound && fromContent.inbound[0]) {
     discardSequence = [fromContent.inbound[0].id];
   }
 
@@ -1031,9 +1031,9 @@ proto._doRunOutbound = function doRunOutbound(outboundList, content, discardSequ
     this.broker.publish('run', 'run.outbound.' + action, (0, _messageHelper.cloneContent)(content, {
       flow: { ...outboundFlow,
         sequenceId: (0, _shared.getUniqueId)(`${flowId}_${action}`),
-        ...(discardSequence ? {
+        ...(discardSequence && {
           discardSequence: discardSequence.slice()
-        } : undefined)
+        })
       }
     }));
   }
@@ -1043,7 +1043,7 @@ proto._doRunOutbound = function doRunOutbound(outboundList, content, discardSequ
 
 proto._onResumeMessage = function onResumeMessage(message) {
   message.ack();
-  const stateMessage = this[stateMessageSymbol];
+  const stateMessage = this[kStateMessage];
   const {
     fields
   } = stateMessage;
@@ -1076,9 +1076,9 @@ proto._publishEvent = function publishEvent(state, content, properties = {}) {
 };
 
 proto._onStop = function onStop(message) {
-  const running = this[consumingSymbol];
+  const running = this[kConsuming];
   this.stopped = true;
-  this[consumingSymbol] = false;
+  this[kConsuming] = false;
   const broker = this.broker;
   broker.cancel('_activity-run');
   broker.cancel('_activity-api');
@@ -1094,11 +1094,11 @@ proto._onStop = function onStop(message) {
 };
 
 proto._consumeApi = function consumeApi() {
-  const executionId = this[execSymbol].executionId;
+  const executionId = this[kExec].executionId;
   if (!executionId) return;
   const broker = this.broker;
   broker.cancel('_activity-api');
-  broker.subscribeTmp('api', `activity.*.${executionId}`, this[messageHandlersSymbol].onApiMessage, {
+  broker.subscribeTmp('api', `activity.*.${executionId}`, this[kMessageHandlers].onApiMessage, {
     noAck: true,
     consumerTag: '_activity-api',
     priority: 100
@@ -1131,18 +1131,18 @@ proto._createMessage = function createMessage(override) {
   const result = { ...override,
     id: this.id,
     type: this.type,
-    ...(name ? {
+    ...(name && {
       name
-    } : undefined),
-    ...(status ? {
+    }),
+    ...(status && {
       status
-    } : undefined),
-    ...(parent ? {
+    }),
+    ...(parent && {
       parent: (0, _messageHelper.cloneParent)(parent)
-    } : undefined)
+    })
   };
 
-  for (const [flag, value] of Object.entries(this[flagsSymbol])) {
+  for (const [flag, value] of Object.entries(this[kFlags])) {
     if (value) result[flag] = value;
   }
 
@@ -1150,7 +1150,7 @@ proto._createMessage = function createMessage(override) {
 };
 
 proto._getOutboundSequenceFlowById = function getOutboundSequenceFlowById(flowId) {
-  return this[flowsSymbol].outboundSequenceFlows.find(flow => flow.id === flowId);
+  return this[kFlows].outboundSequenceFlows.find(flow => flow.id === flowId);
 };
 
 proto._resumeExtensions = function resumeExtensions(message, callback) {
@@ -1171,7 +1171,7 @@ proto._deactivateRunConsumers = function _deactivateRunConsumers() {
   broker.cancel('_activity-api');
   broker.cancel('_activity-run');
   broker.cancel('_activity-execution');
-  this[consumingSymbol] = false;
+  this[kConsuming] = false;
 };
 
 function OutboundEvaluator(activity, outboundFlows) {
@@ -1316,9 +1316,9 @@ OutboundEvaluator.prototype.completed = function completed(err) {
 
   for (const flow of Object.values(result)) {
     evaluationResult.push({ ...flow,
-      ...(message !== undefined ? {
+      ...(message !== undefined && {
         message
-      } : undefined)
+      })
     });
   }
 
@@ -1329,8 +1329,8 @@ function formatFlowAction(flow, options) {
   return { ...options,
     id: flow.id,
     action: options.action,
-    ...(flow.isDefault ? {
+    ...(flow.isDefault && {
       isDefault: true
-    } : undefined)
+    })
   };
 }