Catch the error when loading library failed (#1071)

Currently, `interfaces.d.ts`, a declaration file for ROS2 messages, will be generated when running `npm install`, which needs to require the `.js` files of messages and the `ref-napi` C++ addon gets loaded.

There could be a corner case that the `ref-napi` C++ addon is cross-compiled for arm arch, so loading `binding.node` dynamically failed on x86 platform, see details #1066.

This patch catches the error when loading the library failed, and if it happens, we will use a prebuild `interfaces.d.ts` instead. Meanwhiles, this patch moves out generating `interfaces.d.ts` file out of `generate-messages` command because it already executed.

Fix: #1066

Author: minggangw

.npmignore

@@ -10,7 +10,6 @@ CONTRIBUTORS.md
 Dockerfile
 cpplint.py
 suppr.txt
-types/interfaces.d.ts
 generated/
 build/
 coverage/

package.json

@@ -17,15 +17,16 @@
     "build:dev": "node-gyp -j 16 build --debug",
     "rebuild": "npm run clean && node-gyp -j 16 rebuild",
     "rebuild:dev": "npm run clean && node-gyp -j 16 rebuild --debug",
-    "generate-messages": "node scripts/generate_messages.js && node scripts/generate_tsd.js",
-    "generate-messages:dev": "node scripts/generate_messages.js --debug && node scripts/generate_tsd.js",
+    "generate-messages": "node scripts/generate_messages.js",
+    "generate-messages:dev": "node scripts/generate_messages.js --debug",
+    "generate-tsd-messages": "node scripts/generate_tsd.js",
     "clean": "node-gyp clean && rimraf ./generated",
     "install": "npm run rebuild",
     "postinstall": "npm run generate-messages",
     "docs": "cd docs && make",
     "test": "nyc node --expose-gc ./scripts/run_test.js && npx tsd",
     "lint": "eslint && node ./scripts/cpplint.js",
-    "format": "clang-format -i -style=file ./src/*.cpp ./src/*.hpp && prettier --write \"{lib,rosidl_gen,rostsd_gen,rosidl_parser,types,example,test,scripts,benchmark}/**/*.{js,md,ts}\" ./*.{js,md,ts}",
+    "format": "clang-format -i -style=file ./src/*.cpp ./src/*.hpp && prettier --write \"{lib,rosidl_gen,rostsd_gen,rosidl_parser,types,example,test,scripts,benchmark,rostsd_gen}/**/*.{js,md,ts}\" ./*.{js,md,ts}",
     "prepare": "husky",
     "coverage": "cat ./coverage/lcov.info | ./node_modules/coveralls/bin/coveralls.js"
   },

rostsd_gen/index.js

@@ -25,6 +25,7 @@ declare module "rclnodejs" {
 
 'use strict';
 
+const os = require('os');
 const path = require('path');
 const fs = require('fs');
 const loader = require('../lib/interface_loader.js');
@@ -34,7 +35,10 @@ async function generateAll() {
   // load pkg and interface info (msgs and srvs)
   const generatedPath = path.join(__dirname, '../generated/');
   const pkgInfos = getPkgInfos(generatedPath);
-
+  if (pkgInfos.length === 0) {
+    console.log('No package found, prebuild interfaces.d.ts will be used.');
+    return;
+  }
   // write interfaces.d.ts file
   const interfacesFilePath = path.join(__dirname, '../types/interfaces.d.ts');
   const fd = fs.openSync(interfacesFilePath, 'w');
@@ -69,8 +73,15 @@ function getPkgInfos(rootDir) {
         })
       )
         continue;
-
-      const rosInterface = loader.loadInterface(typeClass);
+      let rosInterface = null;
+      try {
+        rosInterface = loader.loadInterface(typeClass);
+      } catch (e) {
+        console.log(
+          `${e.message}, please make sure it's built for ${os.arch()} platform correctly.`
+        );
+        return pkgInfos;
+      }
 
       if (!pkgInfo.subfolders.has(typeClass.type)) {
         pkgInfo.subfolders.set(typeClass.type, []);