Better error handling

paranext · Nov 13, 2024 · 2a5a503 · 2a5a503
1 parent e28a319
commit 2a5a503
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 6 deletions.
diff --git a/src/declarations/papi-shared-types.ts b/src/declarations/papi-shared-types.ts
@@ -489,6 +489,7 @@ declare module 'papi-shared-types' {
    * ```
    */
   export interface DataProviders {
+    // These are examples. Feel free to take them out if we actually need to provide real ones in core
     'platform.stuff': IDataProvider<StuffDataTypes>;
     'platform.placeholder': IDataProvider<PlaceholderDataTypes>;
   }
@@ -574,6 +575,7 @@ declare module 'papi-shared-types' {
    * ```
    */
   export interface WebViewControllers {
+    // These are examples. Feel free to take them out if we actually need to provide real ones in core
     'platform.stuffWebView': NetworkableObject<{ doStuff(thing: string): Promise<boolean> }>;
     'platform.placeholderWebView': NetworkableObject<{
       runPlaceholderStuff(thing: string): Promise<boolean>;

diff --git a/src/extension-host/services/extension.service.ts b/src/extension-host/services/extension.service.ts
@@ -1206,9 +1206,13 @@ async function reloadExtensions(
   isReloading = false;
   if (shouldReload) {
     (async () => {
-      await platformBibleUtils.wait(RESTART_DELAY_MS);
-      shouldReload = false;
-      reloadExtensions(shouldDeactivateExtensions, shouldEmitDidReloadEvent);
+      try {
+        await platformBibleUtils.wait(RESTART_DELAY_MS);
+        shouldReload = false;
+        await reloadExtensions(shouldDeactivateExtensions, shouldEmitDidReloadEvent);
+      } catch (e) {
+        logger.error(`Subsequent reload after initial reload extensions failed! ${e}`);
+      }
     })();
   }
 

diff --git a/src/main/services/extension-host.service.ts b/src/main/services/extension-host.service.ts
@@ -88,7 +88,6 @@ async function restartExtensionHost(maxWaitTimeInMS: number) {
   }
   // Tells nodemon to restart the process https://github.com/remy/nodemon/blob/HEAD/doc/events.md#using-nodemon-as-child-process
   extensionHost?.send('restart');
-  return undefined;
 }
 
 function hardKillExtensionHost() {

diff --git a/src/shared/services/web-view-provider.service.ts b/src/shared/services/web-view-provider.service.ts
@@ -239,8 +239,8 @@ async function registerWebViewController<WebViewType extends WebViewControllerTy
   });
 
   if (webViewControllersById.has(webViewId))
-    logger.warn(
-      `Web view provider service is setting web view controller with id ${webViewId} (type ${webViewType}) in the map over an existing web view. This is not expected.`,
+    throw new Error(
+      `Web view provider service tried to set web view controller with id ${webViewId} (type ${webViewType}) in the map over an existing web view.`,
     );
   webViewControllersById.set(webViewId, disposableWebViewController);