What
Resolve the value_store in-place deserialization placeholder in message_container.
Part of kcenon/common_system#684.
The 2026-05-21 analysis found TODO comments in src/messaging/message_container.cpp indicating that in-place deserialization for value_store is currently a placeholder.
Why
container_system is the shared data/serialization layer for higher-level systems. A placeholder in deserialization behavior can become a hidden correctness or performance gap if downstream systems assume the operation is fully implemented.
Where
src/messaging/message_container.cpp
include/kcenon/container/value_store.h
include/kcenon/container/messaging/message_container.h
- Serializer tests under
tests/
- Messaging integration tests, if present
How
Approach
- Inspect the placeholder code path and confirm whether the intended behavior is still required.
- If required, implement in-place deserialization for
value_store without changing the public API unexpectedly.
- If not required, remove the placeholder path and document the supported alternative.
- Add tests covering round-trip serialization/deserialization and failure handling for malformed input.
- Update README/docs if the behavior is user-facing.
Acceptance Criteria
Verification
What
Resolve the
value_storein-place deserialization placeholder inmessage_container.Part of kcenon/common_system#684.
The 2026-05-21 analysis found TODO comments in
src/messaging/message_container.cppindicating that in-place deserialization forvalue_storeis currently a placeholder.Why
container_systemis the shared data/serialization layer for higher-level systems. A placeholder in deserialization behavior can become a hidden correctness or performance gap if downstream systems assume the operation is fully implemented.Where
src/messaging/message_container.cppinclude/kcenon/container/value_store.hinclude/kcenon/container/messaging/message_container.htests/How
Approach
value_storewithout changing the public API unexpectedly.Acceptance Criteria
value_storein-place deserializationmessage_containerpathVerification
ctestfor container serialization/messaging tests passes