[han-xudong]

Reject nonfinite/overflowing SI conversion and make auxiliary executor failures visible. Add allow_bias control, wall-clock diagnostics, and concrete lifecycle/time/support guidance. Pin the checked SDK candidate from netft/netft-cpp#21.

Provide usable SROS2 driver/reader/operator profiles and a bounded Enforce integration check: reader receives samples but cannot send bias, while operator sends exactly one bias to a loopback fake sensor. Profiles include required rosout, parameter_events and own parameter reply permissions.

Validation: final CI 37610033066 passed Noetic, Humble, Jazzy, Kilted, Lyrical and Rolling node/control tests. Jazzy CI and local Kilted passed the real SROS2 allow/deny checks and isolated controller fault tests. No real sensor configuration was changed; no release is published.

Superseded by #23 after the source branch was renamed to fix/ecosystem-improvements. The implementation commit is unchanged; this closed PR retains the original discussion and acceptance history.

[coderabbitai]

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

• Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[codecov]

Codecov Report

❌ Patch coverage is 67.64706% with 22 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
src/core/src/types.cpp	68.18%	3 Missing and 4 partials ⚠️
src/core/src/detail/client_impl.cpp	54.54%	3 Missing and 2 partials ⚠️
src/core/src/detail/time.hpp	58.33%	2 Missing and 3 partials ⚠️
src/ros/unit_conversion.cpp	80.00%	1 Missing and 2 partials ⚠️
src/core/src/detail/xml_config.cpp	33.33%	2 Missing ⚠️

📢 Thoughts on this report? Let us know!