electron: use inversify in main process

All the logic for the electron main process currently has to be added to
our generators, making it hard to extend without committing to Theia.

This commit re-arranges the way Electron is launched to allow people to
more easily change the behavior of their application.

Signed-off-by: Paul Maréchal <paul.marechal@ericsson.com>

.travis.yml

@@ -13,7 +13,6 @@ cache:
     - dev-packages/application-manager/node_modules
     - dev-packages/application-package/node_modules
     - dev-packages/cli/node_modules
-    - dev-packages/electron/node_modules
     - examples/api-samples/node_modules
     - examples/browser/node_modules
     - examples/electron/node_modules
@@ -24,6 +23,7 @@ cache:
     - packages/debug/node_modules
     - packages/editor-preview/node_modules
     - packages/editor/node_modules
+    - packages/electron/node_modules
     - packages/file-search/node_modules
     - packages/filesystem/node_modules
     - packages/getting-started/node_modules

CHANGELOG.md

@@ -8,6 +8,7 @@ Breaking Changes:
 
 - [task] Widened the scope of some methods in TaskManager and TaskConfigurations from string to TaskConfigurationScope. This is only breaking for extenders, not callers. [#7928](https://github.com/eclipse-theia/theia/pull/7928)
 - [shell] `ApplicationShell.TrackableWidgetProvider.getTrackableWidgets` is sync to register child widgets in the same tick, use `ApplicationShell.TrackableWidgetProvider.onDidChangeTrackableWidgets` if child widgets are added async
+- [electron] Electron applications can now be configured/extended through inversify. Added new `electronMain` theia extension points to provide inversify container modules.
 
 ## v1.2.0
 

configs/root-compilation.tsconfig.json

@@ -138,6 +138,9 @@
     },
     {
       "path": "../packages/vsx-registry/compile.tsconfig.json"
+    },
+    {
+      "path": "../packages/electron/compile.tsconfig.json"
     }
   ]
 }

dev-packages/application-manager/src/generator/abstract-generator.ts

@@ -48,6 +48,10 @@ export abstract class AbstractGenerator {
         return this.compileModuleImports(modules, 'require');
     }
 
+    protected compileElectronMainModuleImports(modules?: Map<string, string>): string {
+        return modules && this.compileModuleImports(modules, 'require') || '';
+    }
+
     protected compileModuleImports(modules: Map<string, string>, fn: 'import' | 'require'): string {
         if (modules.size === 0) {
             return '';

dev-packages/application-manager/src/generator/backend-generator.ts

@@ -52,18 +52,18 @@ function load(raw) {
     )
 }
 
-function start(port, host, argv) {
+async function start(port, host, argv) {
     if (argv === undefined) {
         argv = process.argv;
     }
 
     const cliManager = container.get(CliManager);
-    return cliManager.initializeCli(argv).then(function () {
-        const application = container.get(BackendApplication);
-        application.use(express.static(path.join(__dirname, '../../lib')));
-        application.use(express.static(path.join(__dirname, '../../lib/index.html')));
-        return application.start(port, host);
-    });
+    await cliManager.initializeCli(argv);
+
+    const application = container.get(BackendApplication);
+    application.use(express.static(path.join(__dirname, '../../lib')));
+    application.use(express.static(path.join(__dirname, '../../lib/index.html')));
+    return application.start(port, host);
 }
 
 module.exports = (port, host, argv) => Promise.resolve()${this.compileBackendModuleImports(backendModules)}