chris/irrigo
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Activity Actions

[IRRIGOAPI-112] Add makeConfirmedOpen to prevent open failures leaving valves stuck on. #8

Merged
chris merged 1 commit from bug/IRRIGOAPI-112 into main 2026-06-25 08:05:17 -06:00
Conversation 0 Commits 1 Files changed 4 +115 -3
chris commented 2026-06-25 08:04:52 -06:00
Owner
Copy link

After openZone exhausts retries, read the relay's actual state before surfacing
the error. If the relay is 'on' (it toggled despite the HA error), treat as a
successful open so the close timer is always scheduled. Mirrors makeConfirmedClose
(API-106) which already handles this pattern on the close side.

Wire the confirmed-open wrapper in bootstrap.ts alongside the existing
makeConfirmedClose so every daemon open path benefits automatically.

After openZone exhausts retries, read the relay's actual state before surfacing the error. If the relay is 'on' (it toggled despite the HA error), treat as a successful open so the close timer is always scheduled. Mirrors makeConfirmedClose (API-106) which already handles this pattern on the close side. Wire the confirmed-open wrapper in bootstrap.ts alongside the existing makeConfirmedClose so every daemon open path benefits automatically.
[IRRIGOAPI-112] Add makeConfirmedOpen to prevent open failures leaving valves stuck on.
All checks were successful
plane-sync / sync (pull_request) Successful in 2s
Details
51c218e00e 
After openZone exhausts retries, read the relay's actual state before surfacing
the error. If the relay is 'on' (it toggled despite the HA error), treat as a
successful open so the close timer is always scheduled. Mirrors makeConfirmedClose
(API-106) which already handles this pattern on the close side.

Wire the confirmed-open wrapper in bootstrap.ts alongside the existing
makeConfirmedClose so every daemon open path benefits automatically.
chris merged commit 8fe258687d into main 2026-06-25 08:05:17 -06:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
chris/irrigo!8
No description provided.