Refactor: Make helper function free & more modular

[Marenz]

No description provided.

[Copilot]

Pull Request Overview

This PR refactors the component-target extraction logic by moving it from a private class method into a standalone helper function and updates the actor start path to use the new helper.

• Removed _get_target_components_from_dispatch method
• Added module-level _convert_target_components function
• Updated _start_actor to call _convert_target_components

Comments suppressed due to low confidence (2)

src/frequenz/dispatch/_actor_dispatcher.py:331

• Add a docstring for _convert_target_components explaining its purpose, parameters, and return values to improve readability and maintainability.

def _convert_target_components(target: ClientTargetComponents) -> TargetComponents:

src/frequenz/dispatch/_actor_dispatcher.py:331

• Introduce unit tests for _convert_target_components to verify correct behavior when target contains integer IDs versus ComponentCategory values.

def _convert_target_components(target: ClientTargetComponents) -> TargetComponents:

[llucax]

Oh, I reviewed this in #155, maybe mark PRs that depend on other PRs as draft?

[llucax]

I guess in this PR you didn't update the client and you can't use the new match-friendly target classes?

[Marenz]

Oh, I reviewed this in #155, maybe mark PRs that depend on other PRs as draft?

I wanted to mark it as draft, but couldn't find the button/function anymore in the UI...
But I did think the WIP commit and the blocked tag gave it away

[Marenz]

I guess in this PR you didn't update the client and you can't use the new match-friendly target classes?

Yeah, doing the BaseId update is a whole different thing again

[llucax]

But I did think the WIP commit and the blocked tag gave it away

You overestimate me...