feat(receive): add plugin support for receive cmd - #164
Conversation
Fully autogenerated via: - plan: Add support for the plugins (non-verbose, someip, rewrite, can) to the "receive" command.
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #164 +/- ##
==========================================
+ Coverage 83.31% 83.40% +0.09%
==========================================
Files 40 40
Lines 21396 21568 +172
==========================================
+ Hits 17826 17989 +163
- Misses 3570 3579 +9 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Autogenerated via: Add a unit test to see whether the non-verbose plugin works for receive command and converts a non-verbose message to visible msg in ascii output.
Makes no sense to have receive with can plugin. Clippy warnings fixed as well.
There was a problem hiding this comment.
Pull request overview
This pull request adds plugin support (NonVerbose, SomeIp, Rewrite) to the "receive" command, bringing feature parity with the "convert" command. The PR also includes refactoring of serial port initialization logic and adds a comprehensive test for the NonVerbose plugin integration.
Changes:
- Added command-line arguments for NonVerbose, SomeIp, and Rewrite plugins to the receive command
- Implemented plugin processing thread that intercepts messages between receiver and forwarder/output
- Refactored serial port opening logic into platform-specific helper functions
- Added test case to verify NonVerbose plugin functionality in the receive command
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/utils/unzip.rs | Code formatting improvement - multi-line function call |
| src/utils/ipdltmsgreceiver.rs | Refactored serial port opening into platform-specific functions (Windows/Unix) |
| src/bin/adlt/receive.rs | Added plugin support infrastructure with command-line args, initialization, processing thread, and test coverage |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Fully autogenerated via: