Skip to content

refactor(gateway): reach message, service and action schemas through a TypeSchemaProvider interface - #750

Merged
bburda merged 2 commits into
feat/gateway-core-4-discoveryfrom
feat/gateway-core-5-type-schema
Oct 24, 2026
Merged

bburda merged 2 commits into
feat/gateway-core-4-discoveryfrom
feat/gateway-core-5-type-schema

Conversation

@bburda

@bburda bburda commented Oct 23, 2026

Copy link
Copy Markdown
Collaborator

Pull Request

Summary

The handlers read the schemas of messages, services and actions through TypeSchemaProvider, a ROS-free interface. The ROS side implements it. The copy of type_info for services and actions stays inside its error handling.


Issue


Type

  • Bug fix
  • New feature or tests
  • Breaking change
  • Documentation only

Testing

  • ./scripts/test.sh unit --packages-select ros2_medkit_gateway
  • src/ros2_medkit_gateway/scripts/check_core_purity.sh

Checklist

  • Breaking changes are clearly described (and announced in docs / changelog if needed)
  • Tests were added or updated if needed
  • Docs were updated if behavior or public API changed

@bburda
bburda added this pull request to stack #755 October 23, 2026 16:45
@bburda bburda self-assigned this Oct 23, 2026
Comment thread src/ros2_medkit_gateway/src/ros2/ros2_type_schema_provider.cpp
…a TypeSchemaProvider interface

Handlers read type schemas through a ROS-free provider. The type_info copy
for services and actions stays inside its error handling.
The type schema provider logs a failed lookup under its own logger with the
type name only. The data and operation handlers now log the miss at DEBUG under
rest_server with the topic or type, and the topic transport logs it under the
node logger, so --log-level rest_server:=debug shows which topic has no schema.
@bburda
bburda force-pushed the feat/gateway-core-5-type-schema branch from 87a5bd7 to eb745b5 Compare October 24, 2026 11:33
@bburda
bburda merged commit e9377a0 into main Oct 24, 2026
25 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants