Skip to content

Comments

Clean up PumpSteer sensor attributes and expand diagnostics#197

Merged
JohanAlvedal merged 1 commit intocodex/01from
codex/clean-up-sensor.pumpsteer-attributes
Jan 20, 2026
Merged

Clean up PumpSteer sensor attributes and expand diagnostics#197
JohanAlvedal merged 1 commit intocodex/01from
codex/clean-up-sensor.pumpsteer-attributes

Conversation

@JohanAlvedal
Copy link
Owner

Motivation

  • Rensa upp sensor.pumpsteer så att UI visar endast användarvänliga fält och inte interna kontroll-/pris-/PI-variabler, samtidigt som full intern status görs tillgänglig via diagnostics för felsökning.
  • Flytta interna data ur UI-attributen för bättre läsbarhet och för att undvika att exponera stora debug-arrayer eller potentiellt känslig information.

Description

  • Begränsar attributen som sensor.pumpsteer exponerar genom att förenkla _build_attributes i custom_components/pumpsteer/sensor/sensor.py så att endast användarorienterade fält finns kvar (t.ex. mode, status, decision_reason, current_price, price_category, price_interval_minutes, target_temperature, indoor_temperature, outdoor_temperature, aggressiveness, inertia, holiday_mode, forecast_available, block_state, in_price_block, brake_offset_c).
  • Samlar och serialiserar alla interna värden (block-, ramp-, price- och PI-interna) i en diagnostics-payload i async_update och skickar denna vidare till diagnostics-samlingen istället för att lägga dem i UI-attribut.
  • Lagt till hjälpfunktioner ._get_decision_reason, ._serialize_block och ._serialize_block_window för tydlig formatering av beslutstext och blockdata för diagnostics.
  • Uppdaterar custom_components/pumpsteer/diagnostics.py för att returnera internal_state och att redigera konfidentiella fält i config entry med redact_data (CONF_ACCESS_TOKEN, CONF_API_KEY, CONF_CLIENT_ID, CONF_CLIENT_SECRET, CONF_PASSWORD, CONF_TOKEN, CONF_USERNAME).
  • Anpassat tester i tests/test_const.py för den nya attributsignaturen och hjälpfunktionerna.

Testing

  • Körde pytest lokalt mot projektet; alla tester kördes och avslutade utan fel.
  • Resultat: pytest — 31 passed.

Codex Task

@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@JohanAlvedal JohanAlvedal merged commit 40e8f25 into codex/01 Jan 20, 2026
1 of 4 checks passed
@JohanAlvedal JohanAlvedal deleted the codex/clean-up-sensor.pumpsteer-attributes branch January 29, 2026 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant