Skip to content

PR3 (Draft, Needs IRL Test): Manual reset service - #5

Draft
jbjardine wants to merge 21 commits into
mainfrom
codjardine/pr3-manual-reset-service
Draft

PR3 (Draft, Needs IRL Test): Manual reset service#5
jbjardine wants to merge 21 commits into
mainfrom
codjardine/pr3-manual-reset-service

Conversation

@jbjardine

@jbjardine jbjardine commented Mar 12, 2026

Copy link
Copy Markdown
Owner

Summary

Manual reset service follow-up for Duepi EVO climate entities.

This PR was originally stacked on PR2 and has now been restacked on main after PR2 merged, so the diff stays focused on the manual reset feature only.

Included

  • Add services.yaml with duepi_evo.manual_reset
  • Register the service in integration setup
  • Resolve targeted climate entities to config entries and execute reset
  • Refresh the coordinator after reset
  • Update README with service usage
  • Add focused service tests for success and invalid target handling

Tracking

Validation

  • Docker-only validation completed locally after restacking:
    • pytest tests/test_manual_reset_service.py -q
    • pytest tests -q
    • bash scripts/smoke_migration.sh

IRL test request

If possible, please validate this on a real stove setup.
Please share:

  1. stove model
  2. bridge setup (esp-link / ESPHome / other)
  3. whether manual reset behaves correctly
  4. logs (if available)

jbjardine and others added 5 commits March 12, 2026 08:42
…ing-down-idle

Fix cooling-down HVAC state mapping
Expand the PR84 follow-up tests so the supported status-to-HVAC cases are
visible in one place. This keeps the cooling-down fix on main while making
coverage easier to review and extend.
…ize-tests

Follow up PR84: parametrize HVAC mapping tests
@jbjardine
jbjardine force-pushed the codjardine/pr2-sensors-readonly branch from 104da8d to 6d4f589 Compare March 12, 2026 08:24
@jbjardine
jbjardine force-pushed the codjardine/pr3-manual-reset-service branch from 97ca292 to 1470c8f Compare March 12, 2026 08:26
@jbjardine
jbjardine force-pushed the codjardine/pr3-manual-reset-service branch from 1470c8f to 8922af0 Compare March 12, 2026 08:58
Add a Docker-based smoke test that exercises the upgrade path from the
last stable pre-PR1 release into the current checkout. Keep the legacy
migration phase optional so we can retire it later without losing the
normal Home Assistant smoke test.
@jbjardine
jbjardine force-pushed the codjardine/pr3-manual-reset-service branch from 8922af0 to 110f207 Compare March 12, 2026 12:59
Run Home Assistant containers as the host runner user and give them a
writable HOME/cache under /config. This preserves the phase-one files for
migration while still allowing phase-two replacement on GitHub Actions.
@jbjardine
jbjardine force-pushed the codjardine/pr3-manual-reset-service branch from 110f207 to 663e83f Compare March 12, 2026 13:21
jbjardine and others added 11 commits March 14, 2026 14:32
Add the missing HA 2026 optional development dependencies so the Docker\nsetup matches the reviewer environment and core service validation\nloads cleanly.\n\nGroup climate, sensor, and binary sensor entities under one Home\nAssistant device, mark the dedicated readonly entities as diagnostic,\nand cover the behavior with tests.\n\nCo-Authored-By: Claude <noreply@anthropic.com>
The smoke migration workflow can run on the fork, where the pre-PR1\nbase tag is not present as a local ref. Resolve missing refs by trying\norigin first, then upstream or the canonical repository URL before\ncreating worktrees.\n\nThis keeps the migration smoke test working for fork pushes while\npreserving the existing behavior when the ref is already available.\n\nCo-Authored-By: Claude <noreply@anthropic.com>
…eadonly

PR2 (Needs IRL Test): Dedicated sensors + legacy attr deprecation
@jbjardine
jbjardine force-pushed the codjardine/pr3-manual-reset-service branch from 663e83f to ab2ce5e Compare March 16, 2026 20:01
@jbjardine
jbjardine changed the base branch from codjardine/pr2-sensors-readonly to main March 16, 2026 20:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant