Conversation
…ass state change.
There was a problem hiding this comment.
Pull request overview
This pull request addresses issue #73 where the sidebar configuration was not being reflected correctly due to Home Assistant state changes during startup. The PR implements safeguards to prevent sidebar updates when Home Assistant is not fully running, avoiding unnecessary re-renders and configuration loss.
Changes:
- Added HA_STATE enum and validation to prevent sidebar operations when Home Assistant is not in RUNNING state
- Implemented a
shouldUpdateoverride on the ha-sidebar component to block re-renders during HA startup - Enhanced diff checking with href validation to detect invalid sidebar items
- Improved configuration processing order by moving settings manipulation after new items are added
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| src/constants/dom.ts | Added HA_STATE enum with state constants and SHOULD_UPDATE_PROPS (unused) |
| src/sidebar-organizer.ts | Added state checking in run() method, shouldUpdate override for ha-sidebar, improved diff validation, reordered config processing |
| src/utilities/model/store.ts | Added _haNotRunningToast() method to notify users when HA is starting up, added BOTTOM_GRID to config update |
| src/components/sidebar-dialog.ts | Improved console logging formatting for consistency |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
🎉 This PR is included in version 3.0.2-beta.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
## [3.0.2](v3.0.1...v3.0.2) (2026-02-20) ### Bug Fixes * **download:** update download method for companion app integration ([#79](#79)) ([7571f2e](7571f2e)) * **sidebar:** add method to handle sidebar DOM updates during Home Assistant state changes ([#76](#76)) ([d3c2833](d3c2833)), closes [#73](#73)
|
🎉 This PR is included in version 3.0.2 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Implement a method to prevent sidebar updates when Home Assistant is not running, avoid unnecessary re-renders.
closes: #73