Battery Boost UI: disable button when battery on hold

[naltatis]

fixes #28419

Only addresses UI. Boost API can still be called. Added behavior note to the docs.

[sourcery-ai]

Hey - I've found 2 issues, and left some high level feedback:

• The UI treats the button as logically unavailable in batteryHold but doesn’t actually disable it (:disabled remains driven only by disabled), which may be confusing for accessibility and screen readers given the PR title; consider wiring batteryHold into the disabled state or otherwise exposing this state via ARIA (e.g. aria-disabled).
• You introduced the BATTERY_MODE enum but still type the batteryMode prop as String in Loadpoint.vue, Loadpoints.vue, Site.vue, and Vehicle.vue; tightening these to PropType<BATTERY_MODE> would give you better type safety and prevent accidental misuse.

Prompt for AI Agents

Please address the comments from this code review:

## Overall Comments
- The UI treats the button as logically unavailable in `batteryHold` but doesn’t actually disable it (`:disabled` remains driven only by `disabled`), which may be confusing for accessibility and screen readers given the PR title; consider wiring `batteryHold` into the disabled state or otherwise exposing this state via ARIA (e.g. `aria-disabled`).
- You introduced the `BATTERY_MODE` enum but still type the `batteryMode` prop as `String` in `Loadpoint.vue`, `Loadpoints.vue`, `Site.vue`, and `Vehicle.vue`; tightening these to `PropType<BATTERY_MODE>` would give you better type safety and prevent accidental misuse.

## Individual Comments

### Comment 1
<location path="assets/js/components/Loadpoints/Loadpoints.vue" line_range="91" />
<code_context>
 		batteryBoostLimit: { type: Number, default: 100 },
 		batteryConfigured: Boolean,
 		batterySoc: Number,
+		batteryMode: String,

 		// session
</code_context>
<issue_to_address>
**suggestion:** Align `batteryMode` prop typing with `BATTERY_MODE` enum for better type safety.

Here, and in `Loadpoint.vue`/`Vehicle.vue`, `batteryMode` is typed as a plain `String`, so you lose the `BATTERY_MODE` enum guarantees from `BatteryBoostButton.vue`. Please update these props to use `String as PropType<BATTERY_MODE>` (or equivalent) to preserve type safety and ensure consistent use of `BATTERY_MODE` across components.
</issue_to_address>

### Comment 2
<location path="server/mcp/openapi.md" line_range="485" />
<code_context>
 ## setLoadpointBatteryBoost

-Enable or disable battery boost. When active, the maximum available home battery power is added until the home battery is drained to configured SoC limit.
+Enable or disable battery boost. When active, the maximum available home battery power is added until the home battery is drained to configured SoC limit. Note: boost will not work while the battery is on hold (e.g. during fast charging or planned charging with discharge prevention enabled).

 **Tags:** loadpoints
</code_context>
<issue_to_address>
**nitpick (typo):** Consider adding 'the' before 'configured SoC limit' for grammatical correctness.

Change the phrase to "drained to the configured SoC limit." for improved readability; the rest of the note reads clearly.

```suggestion
Enable or disable battery boost. When active, the maximum available home battery power is added until the home battery is drained to the configured SoC limit. Note: boost will not work while the battery is on hold (e.g. during fast charging or planned charging with discharge prevention enabled).
```
</issue_to_address>

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}