Fix detection of controller events with deCONZ

[EPMatt]

Thank you for taking the time to work on a Pull Request. Your contribution is really appreciated! 🎉
Please don't delete any part of the template, since keeping the provided structure will help maintainers to review your work more rapidly.

Sections marked as * are required and need to be filled in.

Proposed change*

Fix detection of controller events with deCONZ. This preserves the fix for double press events introduced in previous updates.
It works by converting deconz_event events payload into strings, ensuring all RAW events comparisons within the blueprint are valid and always involve two string operands.

Closes #42.

Checklist*

• I followed sections of the Contribution Guidelines relevant to changes I'm proposing.
• I properly tested proposed changes on my system and confirm that they are working as expected.
• I formatted files with Prettier using the command npm run format before submitting my Pull Request.

[EPMatt]

Hi @eloo,
could you please check if this PR solves the issue you reported?

Thanks!

[eloo]

Hi @EPMatt
of course i will give it a try.
I will give feedback till the end of the day.

Thanks a lot

[eloo]

@EPMatt
Hi seems that your change is not fixing the problem.
But i've figured out that when you wrap the trigger_action with string in the condition it seems to work (at least for deconz)

like this

      - conditions: "{{ (trigger_action | string)  in button_short }}"

[EPMatt]

@eloo I took some time to better understand why the fix does not work. I think the issue is in how Home Assistant renders variables in the variables script block. It looks like any numeric-like value, even if explicitly converted to string via the Jinja string filter, is converted into an int, thus making the fix I pushed yesterday completely useless.

I don't know whether this is an expected behaviour, but I'm now opening an issue on HA Core to report the problem.

Thank you very much for your test, that pointed me in the right direction. :)
I'll fix the issue in a few minutes. 👍🏻

[EPMatt]

Hi @eloo,
I've just pushed a fix for this issue.
Could you please check if the automation is now working as expected?

Thanks!

[eloo]

@EPMatt okay yes, an implicit type conversion sounds also unexpected to me.

i will give your fix a try in the evening.

Thanks!

[eloo]

@EPMatt
its working as expected.

thanks

[EPMatt]

Hi @eloo,

Thank you for your feedback, I'm happy to hear that your issue is solved. :)
I'm now updating the changelog and merging this PR.

Thanks again!

[github-actions]

Hi there,

🔒 This pull request has been automatically locked since there has not been any recent activity after it was closed.
Please open a new issue for related bugs. If you want to submit a contribution to the project, you can open a new PR.

Thanks!