refactor(behavior_path_planner): renaming and comment collision check debug

[zulfaqar-azmi-t4]

Description

🤖 Generated by Copilot at 2335c67

This pull request improves the debug information and messages for the lane change module of the behavior path planner. It modifies the CollisionCheckDebug struct and the functions that use it in utils.hpp, safety_check.cpp, and debug.cpp to use more descriptive and consistent names and fields. It also updates the interface.cpp and normal.cpp files to use the modified struct and provide more informative and consistent debug messages.

Tests performed

Compile autoware

Effects on system behavior

Not applicable.

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

• I've confirmed the contribution guidelines.
• The PR follows the pull request guidelines.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

• The PR follows the pull request guidelines.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

• There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

[codecov]

Codecov Report

Patch coverage: 18.18% and project coverage change: -0.06% ⚠️

Comparison is base (b5e2c13) 15.08% compared to head (6cb69d6) 15.03%.
Report is 18 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4749      +/-   ##
==========================================
- Coverage   15.08%   15.03%   -0.06%     
==========================================
  Files        1563     1563              
  Lines      107896   108601     +705     
  Branches    33111    33602     +491     
==========================================
+ Hits        16272    16324      +52     
- Misses      73868    74424     +556     
- Partials    17756    17853      +97     

Flag	Coverage Δ		*Carryforward flag
differential	13.14% <18.18%> (?)
total	15.08% <ø> (-0.01%)	⬇️	Carriedforward from b5e2c13

*This pull request uses carry forward flags. Click here to find out more.

Files Changed	Coverage Δ
...clude/behavior_path_planner/marker_utils/utils.hpp	0.00% <ø> (ø)
...ath_planner/src/marker_utils/lane_change/debug.cpp	0.00% <0.00%> (ø)
...planner/src/scene_module/lane_change/interface.cpp	6.35% <0.00%> (+0.02%)	⬆️
...th_planner/src/scene_module/lane_change/normal.cpp	3.99% <0.00%> (-0.01%)	⬇️
...ner/src/utils/path_safety_checker/safety_check.cpp	26.85% <57.14%> (-5.69%)	⬇️

... and 6 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[kyoichi-sugahara]

@zulfaqar-azmi-t4
Very good 👍 super thanks!
Could you move visualize function to utils.cpp?

[zulfaqar-azmi-t4]

@kyoichi-sugahara
Thank you. Sure, I will move the functions via another PR.

[kyoichi-sugahara]

LGTM