fix(native-machine-id): replace bindings dependency with explicit paths (#534)

- Use explicit paths for requiring as that works better with bundlers
- No need to use `bindings` as the alternative is simpler

Signed-off-by: Sebastian Malton <[email protected]>
Co-authored-by: Anna Henningsen <[email protected]>

Fixes: #533

Author: Nokel81

package-lock.json

@@ -20,7 +20,6 @@
   "author": "Compass Team <[email protected]>",
   "gypfile": true,
   "dependencies": {
-    "bindings": "^1.5.0",
     "node-addon-api": "^8.0.0"
   },
   "license": "Apache-2.0",
@@ -53,15 +52,15 @@
     "@mongodb-js/tsconfig-devtools": "^1.0.3",
     "@types/chai": "^4.2.21",
     "@types/mocha": "^9.1.1",
-    "@types/sinon-chai": "^3.2.5",
     "@types/node": "^17.0.35",
+    "@types/sinon-chai": "^3.2.5",
+    "chai": "^4.5.0",
     "eslint": "^7.25.0",
     "gen-esm-wrapper": "^1.1.1",
     "mocha": "^8.4.0",
-    "chai": "^4.5.0",
     "node-machine-id": "^1.1.12",
-    "typescript": "^5.0.4",
-    "ts-node": "^10.9.2"
+    "ts-node": "^10.9.2",
+    "typescript": "^5.0.4"
   },
   "keywords": [
     "machine id",

packages/native-machine-id/package.json

@@ -1,8 +1,24 @@
-import bindings from 'bindings';
 import { createHash } from 'crypto';
 import { promisify } from 'util';
 
-const binding = bindings('native_machine_id');
+type NativeMachineIdModule = {
+  getMachineIdSync: () => string;
+  getMachineIdAsync: (cb: (err: unknown, id: string) => void) => void;
+};
+
+const binding = (() => {
+  try {
+    // eslint-disable-next-line @typescript-eslint/no-var-requires
+    return require('../build/Release/native_machine_id.node') as NativeMachineIdModule;
+  } catch (outerError) {
+    try {
+      // eslint-disable-next-line @typescript-eslint/no-var-requires
+      return require('../build/Debug/native_machine_id.node') as NativeMachineIdModule;
+    } catch {
+      throw outerError;
+    }
+  }
+})();
 
 function getMachineIdFromBindingSync(): string | undefined {
   try {