Make Fronius inverter and controller IDs configurable

[Copilot]

• Add configuration parameters to batcontrol_config_dummy.yaml for Fronius inverter_id and controller_id
• Update FroniusWR class to read inverter_id and controller_id from config with defaults
• Replace hardcoded '1' inverter ID in get_SOC() method
• Replace hardcoded '0' controller ID in get_capacity() method
• Create tests to verify configurable IDs work correctly
• Run linting and testing
• Address code review feedback (remove unused import, refactor tests, remove duplicate code)
• Security scan completed - no vulnerabilities found
• Add ID verification during initialization
• Output complete JSON response when verification fails for debugging
• All tests passing (130 passed)

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Fronius IDs hardcoded</issue_title>
<issue_description>In fronius.py are multiple ID Numbers hardcoded.
For each subtype (Controler, Inverter) the ID must be configured via batcontrol_config.yaml.

In some cases the IDs are varying, so it needs to be a configuration. The current value needs to be the default.</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Fronius IDs hardcoded #238

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[MaStr]

@copilot please add a verification of the given ids against the urls (do a test call to the method) to the Fronius class. If there is an issue, please create some code, that outputs the complete json result of the corresponding call.

[Copilot]

@copilot please add a verification of the given ids against the urls (do a test call to the method) to the Fronius class. If there is an issue, please create some code, that outputs the complete json ...

Added ID verification in commit 0716062. The _verify_fronius_ids() method is now called during initialization and:

• Makes test calls to the powerflow and storage APIs
• Verifies the configured IDs exist in the actual responses
• Logs complete JSON data with available IDs when verification fails
• Raises RuntimeError with available IDs in the error message

Tests added for both valid and invalid ID scenarios.

[Copilot]

Pull request overview

This PR makes Fronius inverter and controller IDs configurable instead of hardcoded, addressing issue #238. The implementation includes configuration parameters with sensible defaults, ID verification during initialization, and comprehensive test coverage.

Key Changes:

• Added configurable fronius_inverter_id (default: '1') and fronius_controller_id (default: '0') parameters
• Implemented ID verification method that validates configured IDs against actual API responses
• Updated get_SOC() and get_capacity() methods to use configurable IDs

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
config/batcontrol_config_dummy.yaml	Added optional configuration parameters for Fronius inverter_id and controller_id with defaults documented
src/batcontrol/inverter/fronius.py	Implemented configurable IDs with string conversion, added _verify_fronius_ids() validation method, updated ID usage in get_SOC() and get_capacity(), minor code formatting improvements
tests/batcontrol/inverter/test_fronius_ids.py	Comprehensive test suite covering default values, custom IDs, type conversion, ID verification in methods, and error handling for invalid IDs