Incorrect "suspect signature type" with dictionary unpacking

[Ten0]

What happened?

appdaemon/appdaemon/threading.py

Lines 1242 to 1248 in 02dfb69

	if len(sig.parameters) != callback_args[type]["count"]:
	self.logger.warning(
	"Suspect incorrect signature type for callback %s() in %s, should be %s - discarding",
	funcref.__name__,
	name,
	callback_args[type]["signature"][use_dictionary_unpacking],
	)

fails to gracefully expand arguments of kwargs with dictionary unpacking.

Version

4.4.2

Installation type

Home Assistant add-on

Relevant log output

WARNING AppDaemon: Suspect incorrect signature type for callback on_motion_detected() in my_app, should be f(self, entity, attribute, old, new, **kwargs) - discarding

Relevant code in the app or config file that caused the issue

use_dictionary_unpacking: true

import appdaemon.plugins.hass.hassapi as hass


class MyApp(hass.Hass):

    def initialize(self):
        self.listen_state(
            self.on_motion_detected,
            "binary_sensor.name",
            new="on",
			abc=3
        )

    def on_motion_detected(self, entity, attribute, old, new, abc):
        self.log("Motion!")

Anything else?

It looks like this code could be smarter when use_dictionary_unpacking is involved.
**kwargs is indeed more pythonic to the extent that we can use it for what it's meant for, that is, creating functions that take this dynamic list of arguments and expand them right away.