Question about proper setup of behavior tree services

Making an AI that crashes through walls to find the player and attack. Currently I have the detect bricks and destroy bricks in a service. Would it be more proper to just have the detection in the service and then extend the destroying bricks to a dedicated task?

Yes that would be the proper way to do it,
Services should primarily be used to update the state of the AI (scan for objects in range, monitor health, etc) through Blackboard Keys.

Tasks should then be used to act on those states (go to object, hide from enemies, etc)

separating those allows for creating reusable components and is considered best practice.

Quick Start Guide - Services

Quick Start Guide - Tasks

If this helped you out please don’t forget to mark it as the answer to the question.