bpmnlint 11.0.0 → 11.1.0

package/config/all.js

@@ -1,4 +1,5 @@
 const allRules = [
+  'ad-hoc-sub-process',
   'conditional-flows',
   'end-event-required',
   'event-sub-process-typed-start-event',

package/config/recommended.js

@@ -1,5 +1,6 @@
 module.exports = {
   rules: {
+    'ad-hoc-sub-process': 'error',
     'conditional-flows': 'error',
     'end-event-required': 'error',
     'event-sub-process-typed-start-event': 'error',

package/lib/resolver/node-resolver.js

@@ -117,18 +117,5 @@ NodeResolver.prototype.normalizePkg = function(pkg) {
 // helpers ////////////////////
 
 function createScopedRequire(cwd) {
-
-  // shim createRequireFromPath for Node < 10.12
-  // shim createRequireFromPath for Node < 12.2.0
-  const createRequireFromPath = Module.createRequire || Module.createRequireFromPath || (filename => {
-    const mod = new Module(filename, null);
-
-    mod.filename = filename;
-    mod.paths = Module._nodeModulePaths(path.dirname(filename));
-    mod._compile('module.exports = require;', filename);
-
-    return mod.exports;
-  });
-
-  return createRequireFromPath(path.join(cwd, '__placeholder__.js'));
+  return Module.createRequire(path.join(cwd, '__placeholder__.js'));
 }

package/lib/test-rule.js

@@ -2,7 +2,8 @@ const traverse = require('./traverse');
 
 const {
   isArray,
-  isObject
+  isObject,
+  isFunction
 } = require('min-dash');
 
 
@@ -57,10 +58,12 @@ class Reporter {
 module.exports = function testRule({ moddleRoot, rule }) {
   const reporter = new Reporter({ rule, moddleRoot });
 
-  const check = rule.check;
+  const check = rule.check || {};
 
-  const enter = check && check.enter || check;
-  const leave = check && check.leave;
+  const leave = 'leave' in check ? check.leave : undefined;
+  const enter = 'enter' in check ? check.enter : (
+    isFunction(check) ? check : undefined
+  );
 
   if (!enter && !leave) {
     throw new Error('no check implemented');

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "bpmnlint",
   "description": "Validate your BPMN diagrams based on configurable lint rules",
-  "version": "11.0.0",
+  "version": "11.1.0",
   "main": "lib/index.js",
   "keywords": [
     "bpmnlint",

package/rules/ad-hoc-sub-process.js

@@ -0,0 +1,45 @@
+const {
+  is,
+  isAny
+} = require('bpmnlint-utils');
+
+/**
+ * A rule that ensures that an Ad Hoc Sub Process is valid according to the BPMN spec:
+ *
+ * - No start or end events
+ * - Every intermediate event has an outgoing sequence flow
+ */
+module.exports = function() {
+
+  function check(node, reporter) {
+
+    if (!is(node, 'bpmn:AdHocSubProcess')) {
+      return;
+    }
+
+    const flowElements = node.flowElements || [];
+
+    flowElements.forEach(function(flowElement) {
+
+      if (is(flowElement, 'bpmn:StartEvent')) {
+        reporter.report(flowElement.id, 'A <Start Event> is not allowed in <Ad Hoc Sub Process>');
+      }
+
+      if (is(flowElement, 'bpmn:EndEvent')) {
+        reporter.report(flowElement.id, 'An <End Event> is not allowed in <Ad Hoc Sub Process>');
+      }
+
+      if (isAny(flowElement, [ 'bpmn:IntermediateCatchEvent', 'bpmn:IntermediateThrowEvent' ])) {
+        if (!flowElement.outgoing || flowElement.outgoing.length === 0) {
+          reporter.report(flowElement.id, 'An intermediate event inside <Ad Hoc Sub Process> must have an outgoing sequence flow');
+        }
+      }
+
+    });
+  }
+
+  return {
+    check
+  };
+
+};