RobotWebTools / rclnodejs / 24328049899

86%

push

github

Add rmw_zenoh_cpp as RMW vendor and update dist (#1487)

**Changes:**

- **`lib/rmw.js`**
  - Added `ZENOH: 'rmw_zenoh_cpp'` to `RMWNames` constants
  - Added missing `iron`, `jazzy`, and `kilted` distro entries to `DefaultRosRMWNameMap` (all default to `rmw_fastrtps_cpp`)
  - Added safe fallback to `getRMWName()` — returns `rmw_fastrtps_cpp` when the distro is not in the map and `RMW_IMPLEMENTATION` is unset, preventing `undefined` from causing incorrect feature gating

- **`test/test-subscription-content-filter.js`**
  - Updated `isContentFilteringSupported()` to exclude `rmw_zenoh_cpp`, which does not support content filtering

**Context:**

`rmw_zenoh_cpp` was promoted to Tier 1 in ROS 2 Kilted Kaiju and is actively referenced in `rclpy` tests for conditional behavior. rclnodejs was missing it entirely from its RMW vendor registry. The `DefaultRosRMWNameMap` was also missing entries for multiple known distros (`iron`, `jazzy`, `kilted`), and `getRMWName()` had no fallback for unmapped distros.

Fix: #1486

1567 of 1987 branches covered (78.86%)

Branch coverage included in aggregate %.

3 of 3 new or added lines in 1 file covered. (100.0%)

3182 of 3544 relevant lines covered (89.79%)

451.55 hits per line