Would a planner recovery help - ComputePathThroughPoses

[mrmara]

Bug report

Required Info:

• Operating System:
  • Ubunt 22.04
• ROS2 Version:
  • humble
• Version or commit hash:
  • humble_main
• DDS implementation:
  • Cyclone

Steps to reproduce issue

Would_a_planner_recovery_help_condition BT node is not working for ComputePathThroughPoses action as it does for ComputePathToPose. This is probably due to a typo:

CODE

That line should probably be
using ThroughActionResult = ThroughAction::Result;

If confirmed, I have a PR ready

[SteveMacenski]

Ah, I agree, please open the PR!

[SteveMacenski]

@claude fix this typo

@mrmara one sec, let me play with claude for a minute. I'm trying to see where its useful to me as a maintainer. tiny fixes like this seem like a good candidate for code gen.

[claude]

Claude encountered an error —— View job

---

I'll analyze this and get back to you.

[SteveMacenski]

@claude fix this typo

[claude]

Claude finished @SteveMacenski's task —— View job • claude/issue-5324-20250702_170128 • Create PR ➔

---

Fixing Typo in would_a_planner_recovery_help_condition.hpp

• Read the current file to understand the typo
• Fix the typo in the type alias definition
• Commit the changes
• Provide PR creation link

✅ Fix completed! Changed ThroughActionResult = Action::Result to ThroughActionResult = ThroughAction::Result on line 32.

This fixes the issue where the Would_a_planner_recovery_help_condition BT node wasn't working properly for ComputePathThroughPoses actions.

---

[SteveMacenski]

Worked on this one! Closing and thanks for the report!