RobotWebTools / rclnodejs / 26731453640

86%

push

github

Add enable_logger_service option for runtime log level control (#1522)

Port rclpy's `LoggingService` to rclnodejs so a node can optionally expose `<node>/get_logger_levels` and `<node>/set_logger_levels` services for runtime log-level inspection and updates, and stabilize CI by wrapping flaky test steps with `nick-fields/retry@v4`.

## Feature
- New `lib/logging_service.js` (`LoggingService`) with idempotent `start()`, UNSET fallback on `getLogger` failure, and per-entry `{ successful, reason }` for set requests.
- New `enableLoggerService` option on `NodeOptions` (default `false`, matches rclpy) with backing field, getter/setter, and TypeScript declaration.
- Wired into `Node` so the service starts after `parameterService` when the option is enabled.

## Tests
- Sinon-based unit tests for `LoggingService` (no ROS runtime needed).
- Extended `NodeOptions` and type tests for the new flag.

## CI
- Wrap test invocations with `nick-fields/retry@v4` (`max_attempts: 2`, `retry_wait_seconds: 10`, `timeout_minutes: 60`) in linux-x64, linux-arm64, windows, and asan workflows. Build-only, docs, publish, and orchestration workflows are intentionally left untouched.

Fix: #1521

1844 of 2342 branches covered (78.74%)

Branch coverage included in aggregate %.

33 of 37 new or added lines in 3 files covered. (89.19%)

3654 of 4089 relevant lines covered (89.36%)

412.36 hits per line