support yarn pnp for nodeAddonIncludes

This solves problems with Yarn PnP and node addon includes.
microsoft · Mar 7, 2021 · 1330360 · 1330360
1 parent d2cb685
commit 1330360
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/Extension/src/LanguageServer/configurations.ts b/Extension/src/LanguageServer/configurations.ts
@@ -406,9 +406,13 @@ export class CppProperties {
         if (!error) {
             try {
                 const pathToNode: string = which.sync("node");
+                const pathToYarn: string = which.sync("yarn");
                 const nodeAddonMap: { [dependency: string]: string } = {
                     "nan": `"${pathToNode}" --no-warnings -e "require('nan')"`,
-                    "node-addon-api": `"${pathToNode}" --no-warnings -p "require('node-addon-api').include"`
+                    "node-addon-api": `"${pathToNode}" --no-warnings -p "require('node-addon-api').include"`,
+                    // support Yarn PnP
+                    "nan": `"${pathToYarn} ${pathToNode}" --no-warnings -e "require('nan')"`,
+                    "node-addon-api": `"${pathToYarn} ${pathToNode}" --no-warnings -p "require('node-addon-api').include"`
                 };
 
                 for (const dep in nodeAddonMap) {