@bpmn-io/form-js-viewer 1.23.1 → 1.24.1

package/dist/index.cjs

@@ -59,7 +59,26 @@ const getFlavouredFeelVariableNames = (feelString, feelFlavour = 'expression', o
       // if using special depth accessors, use a more complex extraction
       return Array.from(_smartExtractVariableNames(node, depth, specialDepthAccessors));
     }
-    if (depth === 0 && node.name === 'VariableName') return [node.variableName];
+    if (depth === 0 && node.name === 'VariableName') return [_cleanupVariableName(node.variableName)];
+
+    // skip function names - the function name is the only direct VariableName child of a
+    // FunctionInvocation (arguments are nested under Positional/NamedParameters), so drop it
+    if (node.name === 'FunctionInvocation') {
+      const args = node.children.filter(child => child.name !== 'VariableName');
+      return args.reduce((acc, child) => acc.concat(_unfoldVariables(child)), []);
+    }
+
+    // skip type names in instance of expressions
+    if (node.name === 'Type') {
+      return [];
+    }
+
+    // handle quantified (some...satisfies) and for expressions - exclude bound iteration variables
+    if (node.name === 'QuantifiedExpression' || node.name === 'ForExpression') {
+      const boundNames = _collectBoundVariableNames(node);
+      const variables = node.children.reduce((acc, child) => acc.concat(_unfoldVariables(child)), []);
+      return variables.filter(name => !boundNames.includes(name));
+    }
 
     // for any other kind of node, traverse its children and flatten the result
     if (node.children) {
@@ -184,6 +203,9 @@ const _buildSimpleFeelStructureTree = (parseTree, feelString) => {
       if (node.type.name === 'VariableName') {
         nodeRepresentation.variableName = feelString.slice(node.from, node.to);
       }
+      if (node.type.name === 'Name') {
+        nodeRepresentation.variableName = feelString.slice(node.from, node.to);
+      }
       stack.push(nodeRepresentation);
     },
     leave: () => {
@@ -233,6 +255,53 @@ const _extractFilterExpressions = tree => {
   return flattenedExpressionTree;
 };
 
+/**
+ * FEEL keywords that may be incorrectly absorbed into variable names by the parser.
+ * For example, `variable1 not in (...)` parses as VariableName "variable1 not".
+ */
+const FEEL_TRAILING_KEYWORDS = ['not'];
+
+/**
+ * Cleans up a variable name by stripping trailing FEEL keywords that the parser
+ * may have incorrectly included as part of the variable name.
+ *
+ * @param {string} variableName - The raw variable name from the parse tree.
+ * @returns {string} The cleaned variable name.
+ */
+const _cleanupVariableName = variableName => {
+  for (const keyword of FEEL_TRAILING_KEYWORDS) {
+    if (variableName.endsWith(' ' + keyword)) {
+      return variableName.slice(0, -(keyword.length + 1));
+    }
+  }
+  return variableName;
+};
+
+/**
+ * Collects the names of variables bound by iteration constructs (e.g. `for x in ...`, `some x in ...`).
+ * These are locally scoped and should be excluded from the extracted variable set.
+ *
+ * @param {Object} node - A QuantifiedExpression or ForExpression node.
+ * @returns {string[]} The bound variable names.
+ */
+const _collectBoundVariableNames = node => {
+  const boundNames = [];
+  const collect = n => {
+    if (n.name === 'InExpression') {
+      // The Name node inside an InExpression is the bound iteration variable
+      const nameNode = n.children.find(child => child.name === 'Name');
+      if (nameNode && nameNode.variableName) {
+        boundNames.push(nameNode.variableName);
+      }
+    }
+    if (n.children) {
+      n.children.forEach(collect);
+    }
+  };
+  collect(node);
+  return boundNames;
+};
+
 class FeelExpressionLanguage {
   constructor(eventBus) {
     this._eventBus = eventBus;
@@ -5881,7 +5950,7 @@ const EMPTY_ARRAY$1 = [];
  * @returns {import("preact").JSX.Element}
  */
 function FilePicker(props) {
-  /** @type {import("preact/hooks").Ref<HTMLInputElement>} */
+  /** @type {import("preact").RefObject<HTMLInputElement>} */
   const fileInputRef = hooks.useRef(null);
   /** @type {import('../../FileRegistry').FileRegistry} */
   const fileRegistry = useService('fileRegistry', false);
@@ -5971,7 +6040,6 @@ function FilePicker(props) {
       children: [jsxRuntime.jsx("button", {
         type: "button",
         disabled: isInputDisabled,
-        readOnly: readonly,
         className: "fjs-button fjs-filepicker-button",
         onClick: () => {
           fileInputRef.current.click();
@@ -6092,11 +6160,10 @@ function DocumentPreview(props) {
         if (!isValidDocumentEndpoint(finalEndpoint)) {
           return null;
         }
-        const requestInit = getDocumentRequestInit(documentEndpointBuilder, document);
         return jsxRuntime.jsx(DocumentRenderer, {
           documentMetadata: document,
+          documentEndpointBuilder: documentEndpointBuilder,
           endpoint: finalEndpoint,
-          requestInit: requestInit,
           maxHeight: maxHeight,
           domId: `${domId}-${index}`
         }, document.documentId);
@@ -6278,9 +6345,9 @@ function ImageRenderer(props) {
  *
  * @param {Object} props
  * @param {DocumentMetadata} props.documentMetadata
+ * @param {DocumentEndpointBuilder | null} props.documentEndpointBuilder
  * @param {string} props.endpoint
  * @param {string} props.domId
- * @param {RequestInit|undefined} props.requestInit
  * @param {number|undefined} props.maxHeight
  *
  * @returns {import("preact").JSX.Element}
@@ -6288,10 +6355,10 @@ function ImageRenderer(props) {
 function DocumentRenderer(props) {
   const {
     documentMetadata,
+    documentEndpointBuilder,
     endpoint,
     maxHeight,
-    domId,
-    requestInit
+    domId
   } = props;
   const {
     metadata
@@ -6299,6 +6366,8 @@ function DocumentRenderer(props) {
   const [hasError, setHasError] = hooks.useState(false);
   const ref = hooks.useRef(null);
   const isInViewport = useInViewport(ref);
+  const onError = hooks.useCallback(() => setHasError(true), []);
+  const requestInit = hooks.useMemo(() => getDocumentRequestInit(documentEndpointBuilder, documentMetadata), [documentEndpointBuilder, documentMetadata]);
   const singleDocumentContainerClassName = `fjs-${type}-single-document-container`;
   const errorMessageId = `${domId}-error-message`;
   const errorMessage = 'Unable to download document';
@@ -6314,14 +6383,12 @@ function DocumentRenderer(props) {
         url: endpoint,
         alt: metadata.fileName,
         requestInit: requestInit,
-        onError: () => setHasError(true)
+        onError: onError
       }), jsxRuntime.jsx(DownloadButton, {
         endpoint: endpoint,
         fileName: metadata.fileName,
         requestInit: requestInit,
-        onDownloadError: () => {
-          setHasError(true);
-        }
+        onDownloadError: onError
       }), hasError ? jsxRuntime.jsx(Errors, {
         id: errorMessageId,
         errors: [errorMessage]
@@ -6339,7 +6406,7 @@ function DocumentRenderer(props) {
         url: endpoint,
         fileName: metadata.fileName,
         requestInit: requestInit,
-        onError: () => setHasError(true),
+        onError: onError,
         errorMessageId: errorMessageId
       })
     });
@@ -6360,9 +6427,7 @@ function DocumentRenderer(props) {
       endpoint: endpoint,
       fileName: metadata.fileName,
       requestInit: requestInit,
-      onDownloadError: () => {
-        setHasError(true);
-      }
+      onDownloadError: onError
     })]
   });
 }
@@ -7003,10 +7068,10 @@ const MarkdownRendererModule = {
 /**
  * @typedef {import('didi').Injector} Injector
  *
- * @typedef {import('../core/Types').ElementLike} ElementLike
+ * @typedef {import('../core/Types.js').ElementLike} ElementLike
  *
- * @typedef {import('../core/EventBus').default} EventBus
- * @typedef {import('./CommandHandler').default} CommandHandler
+ * @typedef {import('../core/EventBus.js').default} EventBus
+ * @typedef {import('./CommandHandler.js').default} CommandHandler
  *
  * @typedef { any } CommandContext
  * @typedef { {
@@ -7802,7 +7867,6 @@ class RepeatRenderManager {
       }),
       children: [showAdd ? jsxRuntime.jsx("button", {
         type: "button",
-        readOnly: readonly,
         disabled: disabled || readonly,
         class: "fjs-repeat-render-add",
         ref: addButtonRef,
@@ -8045,11 +8109,11 @@ function EventBus() {
  *
  * Returning anything but `undefined` from a listener will stop the listener propagation.
  *
- * @template T
+ * @template {keyof EventMap} EventName
  *
- * @param {string|string[]} events to subscribe to
+ * @param {EventName} events to subscribe to
  * @param {number} [priority=1000] listen priority
- * @param {EventBusEventCallback<T>} callback
+ * @param {EventBusEventCallback<EventMap[EventName]>} callback
  * @param {any} [that] callback context
  */
 /**
@@ -8064,11 +8128,11 @@ function EventBus() {
  *
  * Returning anything but `undefined` from a listener will stop the listener propagation.
  *
- * @template {keyof EventMap} EventName
+ * @template T
  *
- * @param {EventName} events to subscribe to
+ * @param {string|string[]} events to subscribe to
  * @param {number} [priority=1000] listen priority
- * @param {EventBusEventCallback<EventMap[EventName]>} callback
+ * @param {EventBusEventCallback<T>} callback
  * @param {any} [that] callback context
  */
 EventBus.prototype.on = function (events, priority, callback, that) {
@@ -8105,21 +8169,21 @@ EventBus.prototype.on = function (events, priority, callback, that) {
  *
  * Register an event listener that is called only once.
  *
- * @template T
+ * @template {keyof EventMap} EventName
  *
- * @param {string|string[]} events to subscribe to
- * @param {number} [priority=1000] the listen priority
- * @param {EventBusEventCallback<T>} callback
+ * @param {EventName} events to subscribe to
+ * @param {number} [priority=1000] listen priority
+ * @param {EventBusEventCallback<EventMap[EventName]>} callback
  * @param {any} [that] callback context
  */
 /**
  * Register an event listener that is called only once.
  *
- * @template {keyof EventMap} EventName
+ * @template T
  *
- * @param {EventName} events to subscribe to
- * @param {number} [priority=1000] listen priority
- * @param {EventBusEventCallback<EventMap[EventName]>} callback
+ * @param {string|string[]} events to subscribe to
+ * @param {number} [priority=1000] the listen priority
+ * @param {EventBusEventCallback<T>} callback
  * @param {any} [that] callback context
  */
 EventBus.prototype.once = function (events, priority, callback, that) {