RobotWebTools / rclnodejs / 18834767018

81%

push

github

Add JSON-Safe serialization modes (#1308)

## **Public API Changes**

**New subscription option:**
- `serializationMode: 'default' | 'plain' | 'json'` for `createSubscription()`

**New utility functions:**  
- `rclnodejs.toJSONSafe(obj)` - makes objects JSON-friendly
- `rclnodejs.toJSONString(obj)` - converts directly to JSON string

**Fully backward compatible** - existing code works unchanged.

## **Description**

This adds **message serialization modes** to fix a common headache: ROS sensor messages use `TypedArray`s which don't play nice with JSON.

**The Problem:**
If you try to `JSON.stringify()` a LaserScan message, you get `{}` instead of the actual data. This breaks web APIs, logging, and any workflow that needs JSON.

**The Solution:**
Three modes to handle messages differently:

- **`'default'`** Uses native rclnodejs behavior
- **`'plain'`**: Convert TypedArrays to regular arrays - JSON works!  
- **`'json'`**: Handle everything (TypedArrays, BigInt, NaN) - fully JSON-safe


[#1307](https://github.com/RobotWebTools/rclnodejs/issues/1307)

859 of 1164 branches covered (73.8%)

Branch coverage included in aggregate %.

35 of 57 new or added lines in 3 files covered. (61.4%)

2057 of 2401 relevant lines covered (85.67%)

403.62 hits per line