rclnodejs 0.30.0 → 0.32.0

package/lib/time.js

@@ -17,7 +17,7 @@
 const rclnodejs = require('bindings')('rclnodejs');
 const Duration = require('./duration.js');
 const ClockType = require('./clock_type.js');
-const int64 = require('int64-napi');
+const S_TO_NS = 10n ** 9n;
 
 /**
  * @class - Class representing a Time in ROS
@@ -26,16 +26,20 @@ const int64 = require('int64-napi');
 class Time {
   /**
    * Create a Time.
-   * @param {number|string} [seconds=0] - The second part of the time.
-   * @param {number|string} [nanoseconds=0] - The nanosecond part of the time.
+   * @param {bigint} [seconds=0] - The second part of the time.
+   * @param {bigint} [nanoseconds=0] - The nanosecond part of the time.
    * @param {ClockType} [clockType=Clock.ClockType.SYSTEM_TIME] - The clock type.
    */
-  constructor(seconds = 0, nanoseconds = 0, clockType = ClockType.SYSTEM_TIME) {
-    if (typeof seconds !== 'number' && typeof seconds !== 'string') {
+  constructor(
+    seconds = 0n,
+    nanoseconds = 0n,
+    clockType = ClockType.SYSTEM_TIME
+  ) {
+    if (typeof seconds !== 'bigint') {
       throw new TypeError('Invalid argument of seconds');
     }
 
-    if (typeof nanoseconds !== 'number' && typeof nanoseconds !== 'string') {
+    if (typeof nanoseconds !== 'bigint') {
       throw new TypeError('Invalid argument of nanoseconds');
     }
 
@@ -43,25 +47,22 @@ class Time {
       throw new TypeError('Invalid argument of clockType');
     }
 
-    if (
-      int64.lt(seconds, 0) ||
-      (typeof seconds === 'string' && seconds.startsWith('-'))
-    ) {
+    if (seconds < 0n) {
       throw new RangeError('seconds value must not be negative');
     }
 
-    if (
-      int64.lt(nanoseconds, 0) ||
-      (typeof nanoseconds === 'string' && nanoseconds.startsWith('-'))
-    ) {
+    if (nanoseconds < 0n) {
       throw new RangeError('nanoseconds value must not be negative');
     }
 
-    this._nanoseconds = int64.from(seconds).multiply(1e9).add(nanoseconds);
-    this._handle = rclnodejs.createTimePoint(
-      this._nanoseconds.toString(),
-      clockType
-    );
+    const total = seconds * S_TO_NS + nanoseconds;
+    if (total >= 2n ** 63n) {
+      throw new RangeError(
+        'Total nanoseconds value is too large to store in C time point.'
+      );
+    }
+    this._nanoseconds = total;
+    this._handle = rclnodejs.createTimePoint(this._nanoseconds, clockType);
     this._clockType = clockType;
   }
 
@@ -80,22 +81,11 @@ class Time {
    * Get the nanosecond part of the time.
    * @name Time#get:nanoseconds
    * @function
-   * @return {number|string} - value in nanosecond, if the value is greater than Number.MAX_SAFE_INTEGER (2^53-1), will be presented in string of decimal format.
+   * @return {bigint} - value in nanosecond.
    */
 
   get nanoseconds() {
-    let str = rclnodejs.getNanoseconds(this._handle);
-    let nano;
-
-    if (str.startsWith('-')) {
-      nano = int64.negative(int64.from(str));
-    } else {
-      nano = int64.from(str);
-    }
-    if (Number.isFinite(nano.toNumber())) {
-      return nano.toNumber();
-    }
-    return nano.toString();
+    return rclnodejs.getNanoseconds(this._handle);
   }
 
   /**
@@ -106,9 +96,11 @@ class Time {
    */
 
   get secondsAndNanoseconds() {
-    const seconds = int64.from(this._nanoseconds).divide(1e9).toNumber();
-    const nanoseconds = int64.from(this._nanoseconds).mod(1e9).toNumber();
-    return { seconds, nanoseconds };
+    const nanoseconds = this._nanoseconds;
+    return {
+      seconds: nanoseconds / S_TO_NS,
+      nanoseconds: nanoseconds % S_TO_NS,
+    };
   }
 
   /**
@@ -119,8 +111,8 @@ class Time {
   add(other) {
     if (other instanceof Duration) {
       return new Time(
-        0,
-        int64.add(this._nanoseconds, other.nanoseconds).toString(),
+        0n,
+        this._nanoseconds + other.nanoseconds,
         this._clockType
       );
     }
@@ -137,14 +129,11 @@ class Time {
       if (other._clockType !== this._clockType) {
         throw new TypeError("Can't subtract times with different clock types");
       }
-      return new Duration(
-        0,
-        int64.subtract(this._nanoseconds, other._nanoseconds).toString()
-      );
+      return new Duration(0n, this._nanoseconds - other._nanoseconds);
     } else if (other instanceof Duration) {
       return new Time(
-        0,
-        int64.subtract(this._nanoseconds, other._nanoseconds).toString(),
+        0n,
+        this._nanoseconds - other._nanoseconds,
         this._clockType
       );
     }
@@ -161,7 +150,7 @@ class Time {
       if (other._clockType !== this._clockType) {
         throw new TypeError("Can't compare times with different clock types");
       }
-      return this._nanoseconds.eq(other.nanoseconds);
+      return this._nanoseconds === other.nanoseconds;
     }
     throw new TypeError('Invalid argument');
   }
@@ -176,7 +165,7 @@ class Time {
       if (other._clockType !== this._clockType) {
         throw new TypeError("Can't compare times with different clock types");
       }
-      return this._nanoseconds.ne(other.nanoseconds);
+      return this._nanoseconds !== other.nanoseconds;
     }
   }
 
@@ -190,7 +179,7 @@ class Time {
       if (other._clockType !== this._clockType) {
         throw new TypeError("Can't compare times with different clock types");
       }
-      return this._nanoseconds.lt(other.nanoseconds);
+      return this._nanoseconds < other.nanoseconds;
     }
     throw new TypeError('Invalid argument');
   }
@@ -205,7 +194,7 @@ class Time {
       if (other._clockType !== this._clockType) {
         throw new TypeError("Can't compare times with different clock types");
       }
-      return this._nanoseconds.lte(other.nanoseconds);
+      return this._nanoseconds <= other.nanoseconds;
     }
     throw new TypeError('Invalid argument');
   }
@@ -220,7 +209,7 @@ class Time {
       if (other._clockType !== this._clockType) {
         throw new TypeError("Can't compare times with different clock types");
       }
-      return this._nanoseconds.gt(other.nanoseconds);
+      return this._nanoseconds > other.nanoseconds;
     }
     throw new TypeError('Invalid argument');
   }
@@ -235,7 +224,7 @@ class Time {
       if (other._clockType !== this._clockType) {
         throw new TypeError("Can't compare times with different clock types");
       }
-      return this._nanoseconds.gte(other.nanoseconds);
+      return this._nanoseconds >= other.nanoseconds;
     }
     throw new TypeError('Invalid argument');
   }
@@ -261,7 +250,7 @@ class Time {
    * @return {Time} Return the created Time object.
    */
   static fromMsg(msg, clockType = ClockType.ROS_TIME) {
-    return new Time(msg.sec, msg.nanosec, clockType);
+    return new Time(BigInt(msg.sec), BigInt(msg.nanosec), clockType);
   }
 }
 

package/lib/time_source.js

@@ -17,7 +17,6 @@
 const rclnodejs = require('bindings')('rclnodejs');
 const { Clock, ROSClock } = require('./clock.js');
 const { ClockType } = Clock;
-const Node = require('./node.js');
 const { Parameter, ParameterType } = require('./parameter.js');
 const Time = require('./time.js');
 
@@ -37,7 +36,7 @@ class TimeSource {
     this._node = node;
     this._associatedClocks = [];
     this._clockSubscription = undefined;
-    this._lastTimeSet = new Time(0, 0, ClockType.ROS_TIME);
+    this._lastTimeSet = new Time(0n, 0n, ClockType.ROS_TIME);
     this._isRosTimeActive = false;
 
     if (this._node) {

package/lib/timer.js

@@ -29,7 +29,7 @@ class Timer {
   }
 
   /**
-   * @type {number}
+   * @type {bigint} - The period of the timer in nanoseconds.
    */
   get period() {
     return this._period;
@@ -73,18 +73,18 @@ class Timer {
 
   /**
    * Get the interval since the last call of this timer.
-   * @return {number} - the interval value - ms.
+   * @return {bigint} - the interval value in nanoseconds.
    */
   timeSinceLastCall() {
-    return parseInt(rclnodejs.timerGetTimeSinceLastCall(this._handle), 10);
+    return rclnodejs.timerGetTimeSinceLastCall(this._handle);
   }
 
   /**
    * Get the interval until the next call will happen.
-   * @return {number} - the interval value - ms.
+   * @return {bigint} - the interval value in nanoseconds.
    */
   timeUntilNextCall() {
-    return parseInt(rclnodejs.timerGetTimeUntilNextCall(this._handle), 10);
+    return rclnodejs.timerGetTimeUntilNextCall(this._handle);
   }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "rclnodejs",
-  "version": "0.30.0",
+  "version": "0.32.0",
   "description": "ROS2.0 JavaScript client with Node.js",
   "main": "index.js",
   "types": "types/index.d.ts",
@@ -17,15 +17,14 @@
     "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",
-    "generate-messages:dev": "node scripts/generate_messages.js --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",
     "clean": "node-gyp clean && rimraf ./generated",
     "install": "npm run rebuild",
     "postinstall": "npm run generate-messages",
     "docs": "cd docs && make",
-    "test": "node --expose-gc ./scripts/run_test.js && npm run dtslint",
-    "dtslint": "node scripts/generate_tsd.js",
-    "lint": "eslint --max-warnings=0 --ext js,ts index.js types scripts lib example rosidl_gen rosidl_parser test benchmark/rclnodejs && node ./scripts/cpplint.js",
+    "test": "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}",
     "prepare": "husky"
   },
@@ -43,48 +42,43 @@
     "type": "git",
     "url": "git+https://github.com/RobotWebTools/rclnodejs.git"
   },
-  "//": "Pin deep-equal to ^1.1.1",
   "devDependencies": {
-    "@babel/eslint-parser": "^7.25.9",
+    "@eslint/js": "^10.0.0",
+    "@types/node": "^22.13.9",
     "@typescript-eslint/eslint-plugin": "^8.18.0",
     "@typescript-eslint/parser": "^8.18.0",
-    "babel-eslint": "^10.1.0",
     "clang-format": "^1.8.0",
     "commander": "^13.1.0",
-    "deep-equal": "^1.1.1",
+    "deep-equal": "^2.2.3",
     "eslint": "^9.16.0",
-    "eslint-config-prettier": "^10.0.1",
+    "eslint-config-prettier": "^10.0.2",
     "eslint-plugin-prettier": "^5.2.1",
+    "globals": "^16.0.0",
     "husky": "^9.1.7",
     "jsdoc": "^4.0.4",
     "lint-staged": "^15.2.10",
     "mocha": "^11.0.2",
+    "rimraf": "^6.0.1",
     "sinon": "^19.0.2",
     "tree-kill": "^1.2.2",
-    "typescript": "^5.7.2"
+    "tsd": "^0.31.2",
+    "typescript": "^5.7.2",
+    "uuid": "^11.0.3"
   },
-  "//": "Pin int64-napi to ^1.0.2",
   "dependencies": {
     "@rclnodejs/ref-array-di": "^1.2.2",
     "@rclnodejs/ref-napi": "^4.0.0",
     "@rclnodejs/ref-struct-di": "^1.1.1",
-    "array.prototype.flat": "^1.3.2",
     "bindings": "^1.5.0",
     "compare-versions": "^6.1.1",
     "debug": "^4.4.0",
     "dot": "^1.1.3",
-    "dtslint": "^4.2.1",
     "fs-extra": "^11.2.0",
-    "json-bigint": "^1.0.0",
-    "int64-napi": "^1.0.2",
     "is-close": "^1.3.3",
-    "mkdirp": "^3.0.1",
-    "mz": "^2.7.0",
+    "json-bigint": "^1.0.0",
     "nan": "^2.22.0",
-    "rimraf": "^6.0.1",
-    "uuid": "^11.0.3",
-    "walk": "^2.3.15",
-    "prettier": "^3.4.2"
+    "prettier": "^3.4.2",
+    "walk": "^2.3.15"
   },
   "husky": {
     "hooks": {
@@ -99,6 +93,9 @@
       "clang-format -i -style=file"
     ]
   },
+  "tsd": {
+    "directory": "test/types"
+  },
   "engines": {
     "node": ">= 16.13.0"
   }

package/rosidl_gen/action_msgs.js

@@ -12,7 +12,6 @@
 
 'use strict';
 
-/* eslint-disable camelcase */
 const GOAL_ID_FIELD = {
   name: 'goal_id',
   type: {

package/rosidl_gen/filter.js

@@ -1,3 +1,17 @@
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+'use strict';
+
 const fs = require('fs');
 const path = require('path');
 const os = require('os');
@@ -73,7 +87,6 @@ const RosPackageFilters = {
 
     if (!fs.existsSync(blocklistPath)) return;
 
-    // eslint-disable-next-line
     let blocklistData = JSON.parse(fs.readFileSync(blocklistPath, 'utf8'));
 
     let filters = blocklistData.map((pkgFilterData) => {

package/rosidl_gen/generator.json

@@ -1,6 +1,6 @@
 {
   "name": "rosidl-generator",
-  "version": "0.5.0",
+  "version": "0.5.1",
   "description": "Generate JavaScript object from ROS IDL(.msg) files",
   "main": "index.js",
   "authors": [

package/rosidl_gen/index.js

@@ -64,7 +64,6 @@ async function generateAll(forcedGenerating) {
 
 const generator = {
   version() {
-    // eslint-disable-next-line
     return fse.readJsonSync(path.join(__dirname, 'generator.json')).version;
   },
 

package/rosidl_gen/message_translator.js

@@ -14,10 +14,6 @@
 
 'use strict';
 
-/* eslint-disable max-depth */
-
-const debug = require('debug')('rclnodejs:message_translator');
-
 function isTypedArray(value) {
   return ArrayBuffer.isView(value) && !(value instanceof DataView);
 }
@@ -39,7 +35,6 @@ function copyMsgObject(msg, obj) {
           // It's an array
           if (typeof obj[i][0] === 'object') {
             // It's an array of objects: converting to ROS message objects
-
             // 1. Extract the element-type first
             // 2. Build the array by translate every elements
             let msgArray = [];
@@ -66,8 +61,7 @@ function copyMsgObject(msg, obj) {
 function verifyMessage(message, obj) {
   if (message.constructor.isROSArray) {
     // It's a ROS message array
-    //  Note: there won't be any JavaScript array in message
-
+    // Note: there won't be any JavaScript array in message.
     if (!Array.isArray(obj)) {
       return false;
     }
@@ -131,13 +125,12 @@ function toPlainObject(message, enableTypedArray = true) {
 
   if (message.constructor.isROSArray) {
     // It's a ROS message array
-    //  Note: there won't be any JavaScript array in message
+    // Note: there won't be any JavaScript array in message.
     let array = [];
     message.data.forEach((e) => {
       array.push(toPlainObject(e, enableTypedArray)); // Translate every elements
     });
     return array;
-    // eslint-disable-next-line no-else-return
   } else {
     // It's a ROS message
     const def = message.constructor.ROSMessageDef;

package/rosidl_gen/packages.js

@@ -20,7 +20,6 @@ const readline = require('readline');
 const path = require('path');
 const walk = require('walk');
 const os = require('os');
-const flat = require('array.prototype.flat');
 const pkgFilters = require('../rosidl_gen/filter.js');
 
 const fsp = fs.promises;
@@ -193,8 +192,7 @@ async function findAmentPackagesInDirectory(dir) {
     pkgs.map((pkg) => getPackageDefinitionsFiles(pkg, dir))
   );
 
-  // Support flat() method for nodejs < 11.
-  const rosFiles = Array.prototype.flat ? files.flat() : flat(files);
+  const rosFiles = files.flat();
   const pkgMap = new Map();
   await Promise.all(
     rosFiles.map((filePath) => addInterfaceInfos(filePath, dir, pkgMap))

package/rosidl_gen/primitive_types.js

@@ -18,8 +18,6 @@ const ref = require('@rclnodejs/ref-napi');
 const StructType = require('@rclnodejs/ref-struct-di')(ref);
 const rclnodejs = require('bindings')('rclnodejs');
 
-/* eslint-disable camelcase */
-
 const StringRefStruct = StructType({
   data: ref.types.CString,
   size: ref.types.size_t,
@@ -65,5 +63,3 @@ module.exports = {
   wstring: StringRefStruct,
   initString: initString,
 };
-
-/* eslint-enable camelcase */

package/rosidl_gen/templates/message.dot

@@ -824,6 +824,10 @@ class {{=arrayWrapper}} {
   get classType() {
     return {{=arrayWrapper}};
   }
+
+  toPlainObject(enableTypedArray) {
+    return translator.toPlainObject(this, enableTypedArray);
+  }
 }
 
 {{? it.spec.constants != undefined && it.spec.constants.length}}

package/rostsd_gen/index.js

@@ -1,6 +1,3 @@
-/* eslint-disable max-depth */
-/* eslint-disable no-sync */
-/* eslint-disable camelcase */
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
 // You may obtain a copy of the License at

package/scripts/cpplint.js

@@ -18,10 +18,9 @@ const exec = require('child_process').exec;
 
 const cmd = 'wget -nc ';
 const cpplintUrl =
-  'https://raw.githubusercontent.com/google/styleguide' +
-  '/gh-pages/cpplint/cpplint.py';
+  'https://raw.githubusercontent.com/cpplint/cpplint/refs/heads/develop/cpplint.py';
 const root = `${__dirname}/../src`;
-const args = `--extensions=cpp,h,hpp,cc ${root}/*`;
+const args = `--filter=-build/include_subdir,-whitespace/indent_namespace --extensions=cpp,h,hpp,cc ${root}/*`;
 
 console.log('Downloading the cpplint...');
 exec(cmd + cpplintUrl, (err, stdout, stderr) => {
@@ -29,7 +28,7 @@ exec(cmd + cpplintUrl, (err, stdout, stderr) => {
     console.log(`Downloading failed: ${stderr}`);
   } else {
     console.log('Running the cpplint...');
-    exec('python cpplint.py ' + args, (err, stdout, stderr) => {
+    exec('python3 cpplint.py ' + args, (err, stdout, stderr) => {
       console.log(stdout);
       if (err) {
         console.log(stderr);

package/scripts/generate_messages.js

@@ -1,6 +1,5 @@
 #!/usr/bin/env node
 
-/* eslint-disable camelcase */
 // Copyright (c) 2018 Intel Corporation. All rights reserved.
 
 // Licensed under the Apache License, Version 2.0 (the "License");
@@ -15,6 +14,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+/* eslint-disable camelcase */
 'use strict';
 
 const generator = require('../rosidl_gen/index.js');

package/scripts/npmjs-readme.md

@@ -45,7 +45,7 @@ npm i rclnodejs@x.y.z
 
 |                                      RCLNODEJS Version                                       |                                                             Compatible ROS 2 LTS                                                             |
 | :------------------------------------------------------------------------------------------: | :------------------------------------------------------------------------------------------------------------------------------------------: |
-| latest version (currently [v0.30.0](https://github.com/RobotWebTools/rclnodejs/tree/0.30.0)) | [Humble](https://github.com/RobotWebTools/rclnodejs/tree/humble-hawksbill)<br>[Jazzy](https://github.com/RobotWebTools/rclnodejs/tree/jazzy) |
+| latest version (currently [v0.32.0](https://github.com/RobotWebTools/rclnodejs/tree/0.32.0)) | [Humble](https://github.com/RobotWebTools/rclnodejs/tree/humble-hawksbill)<br>[Jazzy](https://github.com/RobotWebTools/rclnodejs/tree/jazzy) |
 
 ## Documentation
 

package/scripts/ros_distro.js

@@ -1,3 +1,15 @@
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+
+//     http://www.apache.org/licenses/LICENSE-2.0
+
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
 'use strict';
 
 const DistroUtils = require('../lib/distro');

package/scripts/run_test.js

@@ -19,8 +19,6 @@ const Mocha = require('mocha');
 const os = require('os');
 const path = require('path');
 
-let rootDir = path.dirname(__dirname);
-
 fs.remove(path.join(path.dirname(__dirname), 'generated'), (err) => {
   if (!err) {
     let mocha = new Mocha();

package/src/executor.hpp

@@ -12,8 +12,8 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#ifndef RCLNODEJS_EXECUTOR_HPP_
-#define RCLNODEJS_EXECUTOR_HPP_
+#ifndef SRC_EXECUTOR_HPP_
+#define SRC_EXECUTOR_HPP_
 
 #include <rcl/wait.h>
 #include <uv.h>
@@ -74,4 +74,4 @@ class Executor {
 
 }  // namespace rclnodejs
 
-#endif
+#endif  // SRC_EXECUTOR_HPP_

package/src/handle_manager.hpp

@@ -12,8 +12,8 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#ifndef RCLNODEJS_HANDLE_MANAGER_HPP_
-#define RCLNODEJS_HANDLE_MANAGER_HPP_
+#ifndef SRC_HANDLE_MANAGER_HPP_
+#define SRC_HANDLE_MANAGER_HPP_
 
 #include <nan.h>
 #include <rcl/wait.h>
@@ -132,4 +132,4 @@ class HandleManager {
 
 }  // namespace rclnodejs
 
-#endif
+#endif  // SRC_HANDLE_MANAGER_HPP_

package/src/macros.hpp

@@ -12,8 +12,8 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#ifndef RCLNODEJS_MARCOS_HPP_
-#define RCLNODEJS_MARCOS_HPP_
+#ifndef SRC_MACROS_HPP_
+#define SRC_MACROS_HPP_
 
 #include "rcutils/logging_macros.h"
 
@@ -43,4 +43,4 @@
 #define RCLNODEJS_DEBUG(...)
 #endif
 
-#endif
+#endif  // SRC_MACROS_HPP_

package/src/rcl_action_bindings.hpp

@@ -12,8 +12,8 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#ifndef RCLNODEJS_RCL_ACTION_BINDINGS_HPP_
-#define RCLNODEJS_RCL_ACTION_BINDINGS_HPP_
+#ifndef SRC_RCL_ACTION_BINDINGS_HPP_
+#define SRC_RCL_ACTION_BINDINGS_HPP_
 
 #include <nan.h>
 #include <rcl/rcl.h>
@@ -28,4 +28,4 @@ extern std::vector<BindingMethod> action_binding_methods;
 
 }  // namespace rclnodejs
 
-#endif
+#endif  // SRC_RCL_ACTION_BINDINGS_HPP_