Skip to content

fix(tools): reject empty topic/message_type on PublishROS2MessageTool - #835

Closed
Bartok9 wants to merge 1 commit into
RobotecAI:mainfrom
Bartok9:aerial/tools-empty-message-type
Closed

Bartok9 wants to merge 1 commit into
RobotecAI:mainfrom
Bartok9:aerial/tools-empty-message-type

Conversation

@Bartok9

@Bartok9 Bartok9 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Summary

Validate topic and message_type on publish tool inputs.

Testing

  • Offline unit tests where ROS not required (uv pytest)
  • AI-assisted; human-reviewed

Claim: bartok
Operator: bartok
Campaign: aerial-drone

Validate publish topic and message_type strings before writable check and send.

Signed-off-by: Bartok9 <danielrpike9@gmail.com>
@maciejmajek

Copy link
Copy Markdown
Member

Covered in #872, min_length=1 on topic and message_type.

@maciejmajek maciejmajek closed this Sep 7, 2026
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