settings changed decorator

we should add a decorator for skill settings, instead of the current way
```
    def initialize(self):
        self.settings_change_callback = self.on_settings_changed
```

this would make the code more readable and help avoiding user mistakes (such as assigning the callback before the call to super)