Show error when bsconfig cannot be parsed

Author: fhammerschmidt

server/src/server.ts

@@ -531,8 +531,25 @@ function createInterface(msg: p.RequestMessage): m.Message {
   let resPartialPath = filePath.split(projDir)[1];
 
   // The .cmi filename may have a namespace suffix appended.
-  let namespace = utils.getNamespaceNameFromBsConfig(projDir);
-  let suffixToAppend = namespace ? "-" + namespace : "";
+  let namespaceResult = utils.getNamespaceNameFromBsConfig(projDir);
+
+  if (namespaceResult.kind === "error") {
+    let params: p.ShowMessageParams = {
+      type: p.MessageType.Error,
+      message: `Error reading bsconfig file.`,
+    };
+
+    let response: m.NotificationMessage = {
+      jsonrpc: c.jsonrpcVersion,
+      method: "window/showMessage",
+      params,
+    };
+
+    return response;
+  }
+
+  let namespace = namespaceResult.result
+  let suffixToAppend = namespace.length > 0 ? "-" + namespace : "";
 
   let cmiPartialPath = path.join(
     path.dirname(resPartialPath),

server/src/utils.ts

@@ -188,20 +188,31 @@ export const toCamelCase = (text: string): string => {
 
 export const getNamespaceNameFromBsConfig = (
   projDir: p.DocumentUri
-): string | null => {
-  let bsconfigFile = fs.readFileSync(
-    path.join(projDir, c.bsconfigPartialPath),
-    { encoding: "utf-8" }
-  );
-
-  let bsconfig = JSON.parse(bsconfigFile);
-
-  if (bsconfig.namespace === true) {
-    return toCamelCase(bsconfig.name);
-  } else if (typeof bsconfig.namespace === "string") {
-    return toCamelCase(bsconfig.namespace);
-  } else {
-    return null;
+): execResult => {
+  try {
+    let bsconfigFile = fs.readFileSync(
+      path.join(projDir, c.bsconfigPartialPath),
+      { encoding: "utf-8" }
+    );
+
+    let bsconfig = JSON.parse(bsconfigFile);
+    let result = "";
+
+    if (bsconfig.namespace === true) {
+      result = toCamelCase(bsconfig.name);
+    } else if (typeof bsconfig.namespace === "string") {
+      result = toCamelCase(bsconfig.namespace);
+    }
+
+    return {
+      kind: "success",
+      result,
+    };
+  } catch (e) {
+    return {
+      kind: "error",
+      error: e instanceof Error ? e.message : String(e),
+    };
   }
 };