Rewrite algorithm to find bsc.exe based on platformPath

Should fix formatting in monorepos.

Fixes #572

Author: Bart Schuurmans

server/src/server.ts

@@ -90,34 +90,32 @@ let findRescriptBinary = (projectRootPath: p.DocumentUri | null) =>
     ? utils.findFilePathFromProjectRoot(projectRootPath, path.join(c.nodeModulesBinDir, c.rescriptBinName))
     : utils.findBinary(extensionConfiguration.binaryPath, c.rescriptBinName);
 
-let findBscBinary = (projectRootPath: p.DocumentUri) => {
-  let rescriptBinaryPath = findRescriptBinary(projectRootPath);
-  if (rescriptBinaryPath !== null) {
-    let rescriptDirPath = path.join(
-      path.dirname(rescriptBinaryPath),
-      "..",
-      "rescript"
-    );
+let findPlatformPath = (projectRootPath: p.DocumentUri | null) => {
+  if (extensionConfiguration.platformPath != null) {
+    return extensionConfiguration.platformPath;
+  }
 
-    let bscBinaryPath = path.join(rescriptDirPath, c.platformDir, c.bscExeName);
+  let rescriptDir = utils.findFilePathFromProjectRoot(projectRootPath, path.join("node_modules", "rescript"));
+  if (rescriptDir == null) {
+    return null;
+  }
 
-    // Workaround for darwinarm64 which has no folder yet in ReScript <= 9.1.4
-    if (
-      process.platform == "darwin" &&
-      process.arch == "arm64" &&
-      !fs.existsSync(bscBinaryPath)
-    ) {
-      bscBinaryPath = path.join(
-        rescriptDirPath,
-        process.platform,
-        c.bscExeName
-      );
-    }
+  let platformPath = path.join(rescriptDir, c.platformDir)
 
-    return bscBinaryPath;
+  // Workaround for darwinarm64 which has no folder yet in ReScript <= 9.1.4
+  if (
+    process.platform == "darwin" &&
+    process.arch == "arm64" &&
+    !fs.existsSync(platformPath)
+  ) {
+    platformPath = path.join(rescriptDir, process.platform);
   }
-  return null;
-};
+
+  return platformPath;
+}
+
+let findBscExeBinary = (projectRootPath: p.DocumentUri | null) =>
+  utils.findBinary(findPlatformPath(projectRootPath), c.bscExeName)
 
 interface CreateInterfaceRequestParams {
   uri: string;
@@ -718,12 +716,12 @@ function format(msg: p.RequestMessage): Array<p.Message> {
   } else {
     // code will always be defined here, even though technically it can be undefined
     let code = getOpenedFileContent(params.textDocument.uri);
+
     let projectRootPath = utils.findProjectRootOfFile(filePath);
-    let bscBinaryPath =
-      projectRootPath === null ? null : findBscBinary(projectRootPath);
+    let bscExeBinaryPath = findBscExeBinary(projectRootPath);
 
     let formattedResult = utils.formatCode(
-      bscBinaryPath,
+      bscExeBinaryPath,
       filePath,
       code,
       extensionConfiguration.allowBuiltInFormatter