clawd/MMM-HomeAssistantDisplay
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove debug log spam from hot path entirely
Some checks failed
Release Drafter / update_release_draft (push) Failing after 4s
Details

Browse Source 
The conditional config.debuglogging checks were not working reliably
in node_helper context. Removed debug logging from onStateChangedEvent
entirely (it fires on every entity state change - multiple times per
second for power sensors). WS disconnect promoted to info level since
it's actually useful.
This commit is contained in:
Clawd
2026-02-23 20:11:51 +00:00
parent a2955ceab4
commit 8831ee9e13
1 changed files with 1 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
node_helper.js
View File

@@ -224,14 +224,8 @@ async function reconnectWebsocket(payload) {
}
function onStateChangedEvent(event) {
if (config.debuglogging) {
this.logger.debug(`Got state change for ${event.data.entity_id}`);
}
for (const connection in this.connections) {
if (this.connections[connection].entities.includes(event.data.entity_id)) {
if (config.debuglogging) {
this.logger.debug(`Found listening connection (${connection}) for entity ${event.data.entity_id}`);
}
this.sendSocketNotification("CHANGED_STATE", {
identifier: connection,
cause: event.data.entity_id,
@@ -243,9 +237,7 @@ function onStateChangedEvent(event) {
function onWebsocketCloseEvent(event) {
for (const connection in this.connections) {
if (event.target == this.connections[connection].websocket.rawClient.ws) {
if (config.debuglogging) {
this.logger.debug(`Hass WS Disconnected (${connection})`);
}
this.logger.info(`Hass WS Disconnected (${connection})`);
this.sendSocketNotification("HASSWS_DISCONNECTED", {
identifier: connection,
});