Added extension host process that launches from main process

.erb/configs/webpack.config.base.ts

@@ -7,9 +7,14 @@ import TsconfigPathsPlugins from 'tsconfig-paths-webpack-plugin';
 import webpackPaths from './webpack.paths';
 import { dependencies as externals } from '../../release/app/package.json';
 
-const isRenderer =
-  process.env.npm_lifecycle_script?.includes('webpack.config.renderer') ??
-  false;
+let processType: string;
+if (process.env.npm_lifecycle_script?.includes('webpack.config.renderer'))
+  processType = 'renderer';
+else if (
+  process.env.npm_lifecycle_script?.includes('webpack.config.extension-host')
+)
+  processType = 'extension-host';
+else processType = 'main';
 
 const configuration: webpack.Configuration = {
   externals: [...Object.keys(externals || {})],
@@ -61,16 +66,39 @@ const configuration: webpack.Configuration = {
     new webpack.IgnorePlugin({
       checkResource(resource, context) {
         // Don't include stuff from the main folder or @main... in renderer and renderer folder in main folder
-        const exclude = isRenderer
-          ? resource.startsWith('@main') || resource.includes('main/')
-          : resource.startsWith('@renderer') || /renderer\//.test(resource);
+        let exclude = false;
+        switch (processType) {
+          case 'renderer':
+            exclude =
+              resource.startsWith('@main') ||
+              resource.includes('main/') ||
+              resource.startsWith('@extension-host') ||
+              resource.includes('extension-host/') ||
+              resource.startsWith('@node') ||
+              resource.includes('node/');
+            break;
+          case 'extension-host':
+            exclude =
+              resource.startsWith('@main') ||
+              resource.includes('main/') ||
+              resource.startsWith('@renderer') ||
+              resource.includes('renderer/');
+            break;
+          default: // main
+            exclude =
+              resource.startsWith('@renderer') ||
+              /renderer\//.test(resource) ||
+              resource.startsWith('@extension-host') ||
+              resource.includes('extension-host/') ||
+              resource.startsWith('@client') ||
+              resource.includes('client/');
+            break;
+        }
 
         // Log if a file is excluded just fyi
         if (!context.includes('node_modules') && exclude)
           console.log(
-            `${
-              isRenderer ? 'Renderer' : 'Main'
-            }: Resource ${resource}\n\tat context ${context}: ${
+            `${processType}: Resource ${resource}\n\tat context ${context}: ${
               exclude ? 'excluded' : 'included'
             }`,
           );

.erb/configs/webpack.config.extension-host.prod.ts

@@ -0,0 +1,34 @@
+/**
+ * Webpack config for production extension-host process
+ */
+
+import path from 'path';
+import webpack from 'webpack';
+import merge, { mergeWithCustomize } from 'webpack-merge';
+import mainConfig from './webpack.config.main.prod';
+import webpackPaths from './webpack.paths';
+import checkNodeEnv from '../scripts/check-node-env';
+import deleteSourceMaps from '../scripts/delete-source-maps';
+
+checkNodeEnv('production');
+deleteSourceMaps();
+
+const configuration: webpack.Configuration = {
+  entry: {
+    'extension-host': path.join(
+      webpackPaths.srcExtensionHostPath,
+      'extension-host.ts',
+    ),
+  },
+
+  output: {
+    path: webpackPaths.distExtensionHostPath,
+  },
+};
+
+export default mergeWithCustomize({
+  customizeObject(a, b, key) {
+    if (key === 'entry') return b;
+    return merge(a, b);
+  },
+})(mainConfig, configuration);

.erb/configs/webpack.paths.ts

@@ -6,6 +6,7 @@ const dllPath = path.join(__dirname, '../dll');
 
 const srcPath = path.join(rootPath, 'src');
 const srcMainPath = path.join(srcPath, 'main');
+const srcExtensionHostPath = path.join(srcPath, 'extension-host');
 const srcRendererPath = path.join(srcPath, 'renderer');
 const srcSharedPath = path.join(srcPath, 'shared');
 
@@ -17,6 +18,7 @@ const srcNodeModulesPath = path.join(srcPath, 'node_modules');
 
 const distPath = path.join(appPath, 'dist');
 const distMainPath = path.join(distPath, 'main');
+const distExtensionHostPath = path.join(distPath, 'extension-host');
 const distRendererPath = path.join(distPath, 'renderer');
 
 const buildPath = path.join(releasePath, 'build');
@@ -26,6 +28,7 @@ export default {
   dllPath,
   srcPath,
   srcMainPath,
+  srcExtensionHostPath,
   srcRendererPath,
   srcSharedPath,
   releasePath,
@@ -35,6 +38,7 @@ export default {
   srcNodeModulesPath,
   distPath,
   distMainPath,
+  distExtensionHostPath,
   distRendererPath,
   buildPath,
 };

.eslintrc.js

@@ -17,7 +17,8 @@ module.exports = {
     'react/jsx-indent-props': ['warn', 2],
     'comma-dangle': ['error', 'always-multiline'],
     'prettier/prettier': ['warn', { tabWidth: 2, trailingComma: 'all' }],
-    'no-console': 'off',
+    // Should always use our logger
+    'no-console': 'error',
     'react/require-default-props': 'off',
     'no-plusplus': ['error', { allowForLoopAfterthoughts: true }],
     'jsx-a11y/label-has-associated-control': [

.gitignore

@@ -30,3 +30,6 @@ npm-debug.log.*
 
 # Extra VS Code workspaces
 *.code-workspace
+
+# Test node localStorage files
+local-storage

.vscode/launch.json

@@ -2,20 +2,20 @@
   "version": "0.2.0",
   "configurations": [
     {
-      "name": ".NET Core Launch",
+      "name": "Debug .NET Core",
       "type": "coreclr",
       "request": "launch",
-      "preLaunchTask": "build",
+      "preLaunchTask": "npm: build:data",
       "program": "${workspaceFolder}/c-sharp/bin/Debug/net7.0/ParanextDataProvider",
       "args": [],
       "cwd": "${workspaceFolder}",
       "stopAtEntry": false,
       "windows": {
-        "program": "${workspaceFolder}/c-sharp/bin/Debug/net7.0/ParanextDataProvider.exe"
+        "program": "${workspaceFolder}/c-sharp/bin/Debug/net7.0/win-x64/ParanextDataProvider.exe"
       }
     },
     {
-      "name": ".NET Core Attach",
+      "name": "Attach to .NET Core",
       "type": "coreclr",
       "request": "attach",
       "processName": "ParanextDataProvider",
@@ -24,29 +24,41 @@
       }
     },
     {
-      "name": "Electron: Main",
+      "name": "Debug Paranext Core Backend",
       "type": "node",
       "request": "launch",
       "protocol": "inspector",
       "runtimeExecutable": "npm",
       "runtimeArgs": ["run", "start"],
       "env": {
-        "MAIN_ARGS": "--inspect=5858 --remote-debugging-port=9223"
+        "MAIN_ARGS": "--inspect=5858 --remote-debugging-port=9223",
+        "IN_VSCODE": "true"
       }
     },
     {
-      "name": "Electron: Renderer",
+      "name": "Attach to Renderer",
       "type": "chrome",
       "request": "attach",
       "port": 9223,
       "webRoot": "${workspaceFolder}",
       "timeout": 15000
+    },
+    {
+      "name": "Debug Extension Host",
+      "type": "node",
+      "request": "launch",
+      "protocol": "inspector",
+      "runtimeExecutable": "npm",
+      "runtimeArgs": ["run", "start:extension-host"],
+      "env": {
+        "IN_VSCODE": "true"
+      }
     }
   ],
   "compounds": [
     {
-      "name": "Electron: All",
-      "configurations": ["Electron: Main", "Electron: Renderer"]
+      "name": "Debug Paranext Core",
+      "configurations": ["Debug Paranext Core Backend", "Attach to Renderer"]
     }
   ]
 }

README.md

@@ -56,12 +56,6 @@ cd paranext-core
 npm install
 ```
 
-Build the C# code:
-
-```bash
-npm run build:data
-```
-
 ## Starting Development
 
 Start the app in the `dev` environment:
@@ -70,7 +64,7 @@ Start the app in the `dev` environment:
 npm start
 ```
 
-After you run `npm start`, you can edit the Electron and frontend files, and they will hot reload. To edit C# files, you must stop the `npm start` process (or only close Paranext), run `npm run build:data`, and restart `npm start` (or if you only closed Paranext, make a trivial edit to `src/main/main.ts`, and save it to launch Paranext again).
+After you run `npm start` (or, in VSCode, launch `Debug Paranext Core`), you can edit the code, and the relevant processes will hot reload.
 
 ## Packaging for Production