fix: continue autocomplete after space

Related to camunda/camunda-modeler#5169

Author: Buckwich

src/autocompletion/variable.js

@@ -19,6 +19,9 @@ export function variableCompletion({ variables = [], builtins = [] }) {
 
   const options = getVariableSuggestions(variables, builtins);
 
+  // Allow spaces for autocomplete suggestions should be the same as feel variable/functions names (eg a-z, A-Z, _, 0-9, <space>)
+  const validFor = /^[\w ]*$/;
+
   if (!options.length) {
     return (context) => null;
   }
@@ -36,7 +39,8 @@ export function variableCompletion({ variables = [], builtins = [] }) {
     if (isEmpty(nodeBefore, pos)) {
       return context.explicit ? {
         from: pos,
-        options
+        options,
+        validFor
       } : null;
     }
 
@@ -45,9 +49,15 @@ export function variableCompletion({ variables = [], builtins = [] }) {
       return null;
     }
 
+    // find the full typed input, to continue auto-completion after separated names
+    const typedInput = typeof context.matchBefore === 'function'
+      ? context.matchBefore(/\w[\w ]*$/)
+      : null;
+
     return {
-      from: nodeBefore.from,
-      options
+      from: typedInput ? typedInput.from : nodeBefore.from,
+      options,
+      validFor
     };
   };
 }

test/spec/autocompletion/builtin.spec.js

@@ -106,6 +106,27 @@ describe('autocompletion - built-ins', function() {
   });
 
 
+  it('should complete multi-word prefix (<get o>)', function() {
+
+    const triggerCompletion = setup('get o', [
+      {
+        name: 'get or else',
+        type: 'function',
+        params: [
+          { name: 'value' },
+          { name: 'default' }
+        ]
+      }
+    ]);
+
+    // when
+    const completion = triggerCompletion();
+
+    // then
+    expect(completion).to.exist;
+  });
+
+
   it('should not complete for empty context', function() {
 
     // given

test/spec/autocompletion/variable.spec.js

@@ -108,6 +108,19 @@ describe('autocompletion - variable', function() {
   });
 
 
+  it('should complete variable with underscore and numbers', function() {
+
+    // given
+    const triggerCompletion = setup('test_', [ { name: 'test_123' } ]);
+
+    // when
+    const completion = triggerCompletion();
+
+    // then
+    expect(completion).to.exist;
+  });
+
+
   it('should not complete empty expression', function() {
 
     // given