fix: index known files by default, index other files explicitly

app/lib/file-context/file-context.js

@@ -106,7 +106,7 @@ module.exports = class FileContext extends EventEmitter {
   /**
    * Handle file opened.
    *
-   * @param { { uri: string, value: string } } fileProps
+   * @param { { uri: string, value: string, processor?: string } } fileProps
    */
   fileOpened(fileProps) {
     return this._indexer.fileOpened(fileProps);
@@ -115,7 +115,7 @@ module.exports = class FileContext extends EventEmitter {
   /**
    * Handle file content changed.
    *
-   * @param { { uri: string, value: string } } fileProps
+   * @param { { uri: string, value: string, processor?: string } } fileProps
    */
   fileContentChanged(fileProps) {
     return this._indexer.fileContentChanged(fileProps);

app/lib/file-context/indexer.js

@@ -97,16 +97,17 @@ module.exports = class Indexer {
   /**
    * @param { string } uri
    * @param { string } [localValue]
+   * @param { string } [processor]
    */
-  add(uri, localValue) {
+  add(uri, localValue, processor) {
     uri = toFileUrl(uri);
 
-    this._logger.info('indexer:add', uri, localValue);
+    this._logger.info('indexer:add', uri, localValue, processor);
 
     let indexItem = this.items.get(uri);
 
     if (!indexItem) {
-      indexItem = createIndexItem({ uri, localValue });
+      indexItem = createIndexItem({ uri, localValue, processor });
 
       this.items.set(uri, indexItem);
     }
@@ -126,35 +127,35 @@ module.exports = class Indexer {
   /**
    * Notify file opened
    *
-   * @param { { uri: string, value: string } } fileProps
+   * @param { { uri: string, value: string, processor?: string } } fileProps
    */
   fileOpened(fileProps) {
-
     const {
+      processor,
       uri,
       value
     } = fileProps;
 
     this._emit('file-opened', uri);
 
-    return this.add(toFileUrl(uri), value);
+    return this.add(toFileUrl(uri), value, processor);
   }
 
   /**
    * Notify file content changed
    *
-   * @param { { uri: string, value: string } } fileProps
+   * @param { { uri: string, value: string, processor?: string } } fileProps
    */
   fileContentChanged(fileProps) {
-
     const {
+      processor,
       uri,
       value
     } = fileProps;
 
     this._emit('file-content-changed', uri);
 
-    return this.add(toFileUrl(uri), value);
+    return this.add(toFileUrl(uri), value, processor);
   }
 
   /**
@@ -345,7 +346,8 @@ module.exports = class Indexer {
 /**
  * @param { {
  *  uri: string,
- *  localValue?: string
+ *  localValue?: string,
+ *  processor?: string
  * } } item
  *
  * @return {IndexItem}
@@ -355,6 +357,7 @@ function createIndexItem(item) {
   const {
     uri,
     localValue,
+    processor,
     ...rest
   } = item;
 
@@ -373,7 +376,8 @@ function createIndexItem(item) {
       this.localValue = value;
     },
     file,
-    localValue
+    localValue,
+    processor
   };
 
 }

app/lib/file-context/processor.js

@@ -38,6 +38,16 @@ module.exports = class Processor {
   process(item) {
     this._logger.info('processor:process', item.uri);
 
+    if (item.processor) {
+      const processor = this._processors.find(processor => processor.id === item.processor);
+
+      if (processor) {
+        return processor.process(item);
+      }
+
+      this._logger.warn('processor:process', `Processor with id ${ item.processor } not found`);
+    }
+
     const processor = this._processors.find(processor => processor.extensions.includes(getFileExtension(item.file.path)));
 
     if (!processor) {

app/lib/file-context/processors/bpmnProcessor.js

@@ -21,7 +21,8 @@ const {
 } = require('./util');
 
 module.exports = {
-  extensions: [ '.bpmn', '.xml' ],
+  id: 'bpmn',
+  extensions: [ '.bpmn' ],
   process: async (item) => {
     let rootElement, ids, linkedIds;
 

app/lib/file-context/processors/dmnProcessor.js

@@ -18,6 +18,7 @@ const {
 } = require('./util');
 
 module.exports = {
+  id: 'dmn',
   extensions: [ '.dmn' ],
   process: async (item) => {
     let rootElement, ids;

app/lib/file-context/processors/formProcessor.js

@@ -11,6 +11,7 @@
 const assert = require('node:assert');
 
 module.exports = {
+  id: 'form',
   extensions: [ '.form' ],
   process: async (item) => {
     let formId;

app/lib/file-context/processors/processApplicationProcessor.js

@@ -9,6 +9,7 @@
  */
 
 module.exports = {
+  id: 'processApplication',
   extensions: [ '.process-application' ],
   process: async (item) => {
     return {

app/lib/file-context/types.d.ts

@@ -22,6 +22,7 @@ export type IndexItem = {
   file: File,
   localValue?: any,
   metadata: Metadata,
+  processor?: string,
   uri: string,
   value: any,
   [key: string]: any

app/lib/index.js

@@ -224,45 +224,62 @@ renderer.on('system-clipboard:write-text', function(options, done) {
 });
 
 // file context //////////
-renderer.on('file-context:add-root', function(filePath, done) {
+renderer.on('file-context:add-root', function(options, done) {
+  const { filePath } = options;
+
   fileContext.addRoot(filePath);
 
   done(null);
 });
 
-renderer.on('file-context:remove-root', function(filePath, done) {
+renderer.on('file-context:remove-root', function(options, done) {
+  const { filePath } = options;
+
   fileContext.removeRoot(filePath);
 
   done(null);
 });
 
-renderer.on('file-context:file-opened', function(filePath, value, done) {
-  fileContext.fileOpened({ uri: toFileUrl(filePath), value });
-
-  done(null);
-});
+renderer.on('file-context:file-opened', function(options, done) {
+  const {
+    filePath,
+    processor,
+    value
+  } = options;
 
-renderer.on('file-context:file-content-changed', function(filePath, value, done) {
-  fileContext.fileContentChanged({ uri: toFileUrl(filePath), value });
+  fileContext.fileOpened({
+    processor,
+    uri: toFileUrl(filePath),
+    value
+  });
 
   done(null);
 });
 
-renderer.on('file-context:file-closed', function(filePath, done) {
-  fileContext.fileClosed(toFileUrl(filePath));
-
-  done(null);
-});
+renderer.on('file-context:file-content-changed', function(options, done) {
+  const {
+    filePath,
+    processor,
+    value
+  } = options;
 
-renderer.on('file:content-changed', function(filePath, contents, done) {
   fileContext.fileContentChanged({
+    processor,
     uri: toFileUrl(filePath),
-    value: contents
+    value
   });
 
   done(null);
 });
 
+renderer.on('file-context:file-closed', function(options, done) {
+  const { filePath } = options;
+
+  fileContext.fileClosed(toFileUrl(filePath));
+
+  done(null);
+});
+
 // filesystem //////////
 
 renderer.on('file:read', function(filePath, options = {}, done) {

client/src/app/App.js

@@ -2109,18 +2109,38 @@ export class App extends PureComponent {
 
   _onTabOpened(tab) {
     if (!this.isUnsaved(tab)) {
-      this.getGlobal('backend').send('file-context:file-opened', tab.file.path, undefined);
+      const {
+        file,
+        type
+      } = tab;
+
+      this.getGlobal('backend').send('file-context:file-opened', {
+        filePath: file.path,
+        processor: getProcessor(type)
+      });
     }
   }
 
   _onTabClosed(tab) {
     if (!this.isUnsaved(tab)) {
-      this.getGlobal('backend').send('file-context:file-closed', tab.file.path);
+      const { file } = tab;
+
+      this.getGlobal('backend').send('file-context:file-closed', {
+        filePath: file.path,
+      });
     }
   }
 
   _onTabSaved(tab) {
-    this.getGlobal('backend').send('file-context:file-content-changed', tab.file.path, undefined);
+    const {
+      file,
+      type
+    } = tab;
+
+    this.getGlobal('backend').send('file-context:file-content-changed', {
+      filePath: file.path,
+      processor: getProcessor(type)
+    });
   }
 
   render() {
@@ -2537,3 +2557,19 @@ function failSafe(fn, errorHandler) {
     }
   };
 }
+
+function getProcessor(type) {
+  if (type === 'cloud-bpmn') {
+    return 'bpmn';
+  }
+
+  if (type === 'cloud-dmn') {
+    return 'dmn';
+  }
+
+  if (type === 'cloud-form') {
+    return 'form';
+  }
+
+  return null;
+}