OpenAI Controller for Reactor

[tamorgen]

@toggledbits
I just upgraded to version MSR 24293, bare metal running on Fedora. Upon restart, I am getting a error banner:

TypeError: Cannot read properties of undefined (reading 'value')
    at Rulesets.getRulesets (https://*************************/client/Rulesets.js:111:38)
    at async StatusTab.updateSetRules (https://*************************/reactor/en-US/lib/js/reactor-ui-status.js:276:28)
    at async StatusTab.drawSetRules (https://*************************/reactor/en-US/lib/js/reactor-ui-status.js:270:13)
You may report this error, but do not screen shot it. Copy-paste the text. Remember to include a description of the operation you were performing in as much detail as possible. Report using the Reactor Bug Tracker (in your left navigation) or at the SmartHome Community.

I followed the new directions about npm

npm i --no-save --no-package-lock --omit dev

Any idea what the issue is?

[toggledbits]

Hard refresh the browser, and if the error persists, press F12 (or whatever works for your browser) to open the browser console and see if there are any other error messages logged. That error suggests it can't read the Ruleset index, but that's a pretty basic function. If you can see your Rulesets listed in the left navigation, the index is fine.

[tamorgen]

Hard refresh on Chrome did not solve the issue, however, the rulesets are listed on the left side navigation, as you stated.

Dev console shows one error in Rulesets.js

Running:

MSR latest-24293-ea42a81d
ZWaveJSController 0.1.24293
Fedora Linux 40 (Server Edition)

Home Assistant
Core 2024.10.3
Supervisor 2024.10.2
Operating System 13.2
Frontend 20240906.0

Chrome: Windows, Version 129.0.6668.59 (Official Build) (32-bit)

[toggledbits]

OK. Looks like a nuisance race condition. After acknowledging the error, if you're not getting any other errors, don't worry about it. I'll have a fix along in a bit.

[tamorgen]

@toggledbits,
Okay, thanks. I don't receive any other errors, and it only appears when I initially load the page or do a refresh.

[gwp1]

@toggledbits hate to pile on... I got this:

    at _ClientAPI.sendrpc (https://192.168.1.241:8554/client/ClientAPI.js:309:45)
    at _ClientAPI.echo (https://192.168.1.241:8554/client/ClientAPI.js:649:31)
    at HTMLAnchorElement.handleTabClick (https://192.168.1.241:8554/reactor/en-US/lib/js/reactor-ui-core.js:132:13)
    at HTMLAnchorElement.dispatch (https://192.168.1.241:8554/node_modules/jquery/dist/jquery.min.js:2:40035)
    at v.handle (https://192.168.1.241:8554/node_modules/jquery/dist/jquery.min.js:2:38006)

This was after using Tools > Restart Reactor because I wanted to ensure everything was good back and forth between Hubitat and MSR. I'm missing capabilities from my Roborock that were there prior to the release upgrade, specifically appScene and appRoomClean and appRoomResume. No updates have been made to the app in Hubitat, those capabilities are still there.

[toggledbits]

@gwp1 said in Error on start for MSR after update to 24293:

@toggledbits hate to pile on... I got this:

Rules of engagement haven't changed. That's clearly redacted content you posted, with no source... pop-up modal dialog? In the browser log stream? Where's the first line of the error saying what the error actually is?

@gwp1 said in Error on start for MSR after update to 24293:

I'm missing capabilities from my Roborock that were there prior to the release upgrade, specifically appScene and appRoomClean and appRoomResume.

Open a separate thread for a separate problem. That said, those aren't Reactor capabilities, so be really detailed (per the posting guidelines) with what you give me there. Thank you.

[gwp1]

@toggledbits yes, I'd planned to split out the Roborock issue but included it above in case it provided value to troubleshooting.

To the original issue, the content is from a pop-up, yes.

Steps to reproduce: Tools > Restart Reactor

Result:

Following the guidance, I did not screenshot it initially. Clicking on the text auto-highlights it; I present it here:

ReferenceError: payload is not defined
    at _ClientAPI.sendrpc (https://192.168.1.241:8554/client/ClientAPI.js:309:45)
    at _ClientAPI.echo (https://192.168.1.241:8554/client/ClientAPI.js:649:31)
    at HTMLAnchorElement.handleTabClick (https://192.168.1.241:8554/reactor/en-US/lib/js/reactor-ui-core.js:132:13)
    at HTMLAnchorElement.dispatch (https://192.168.1.241:8554/node_modules/jquery/dist/jquery.min.js:2:40035)
    at v.handle (https://192.168.1.241:8554/node_modules/jquery/dist/jquery.min.js:2:38006)

As to the separate issue... doing another Testing > Restart Reactor to obtain the above screenshot appears to have fixed the issue - the missing capabilities/commands are back and the rules are no longer erroring.

[toggledbits]

OK. That's benign, I think. I'm guessing you are switching to another tab pretty quickly, while the restart and browser-side reconnect are still happening. When you switch tabs, there's a query made to the Reactor service (which isn't running at that moment), and that's what's failing. If you wait on the Tools tab until Reactor reports it's connected again, then switch tabs, I'll bet you don't get the error.

[gwp1]

@toggledbits Yes, if I am patient and don't touch anything until MSR is showing green/CONNECTED it does not occur.

Patience is a tall order before 0900 EDT sans coffee on a Monday...

[toggledbits]

As I said, it's a benign message. I'll make sure it's not so picky on the next build.

[tamorgen]

modules:composer.user-said-in, @gwp1, Error on start for MSR after update to 24293

Patience is a tall order before 0900 EDT sans coffee on a Monday

Get some coffee

Glad I'm not the only one that had the error. I thought I broke something.

[toggledbits]

By the way, just so you all know, all of these errors were happening before this build, you just didn't see them. The code structure that allowed the race condition that opened this thread has been there since 2021; few would get it, and maybe not even consistently (the nature of race conditions), and when it happened, it would just be logged in the browser console (which most people don't have up) but otherwise not much effect. Same with the post-restart tab switch message -- logged quietly in a place nobody ever looked. When I added the global error handling, a bunch of things crawled out of the woodwork... as they say, sunlight is a good disinfectant. A ton of things were found and fixed by that before releasing this build, but there will always be differences in the way I approach the app vs how everyone else does, so now your eyes are on things I wouldn't easily see. Alarming, maybe, but not harmful so far, and finding these things and fixing them really improves code quality. I suspect we'll have a few more of these in the coming weeks, but just report and keep on doing what you're doing (in the app and generally).

[gwp1]

@toggledbits ignorance is bliss