refactor utils into helpers.py

ovos_workshop/app.py

@@ -23,7 +23,7 @@
 from ovos_utils.skills.settings import PrivateSettings
 from ovos_utils.sound import wait_while_speaking
 
-from ovos_workshop.skills.base import Intent, IntentBuilder, get_non_properties
+from ovos_workshop.helpers import Intent, IntentBuilder, get_non_properties
 from ovos_workshop.skills.decorators import *
 from ovos_workshop.skills.decorators.killable import killable_event, \
     AbortEvent, AbortQuestion

ovos_workshop/skills/base.py → ovos_workshop/helpers.py

@@ -135,7 +135,7 @@ def __init__(self, name=''):
 
 
 def get_non_properties(obj):
-    """Get attibutes that are not properties from object.
+    """Get attributes that are not properties from object.
 
     Will return members of object class along with bases down to MycroftSkill.
 

ovos_workshop/patches/base_skill.py

@@ -14,31 +14,7 @@
 from mycroft.skills.skill_data import read_vocab_file, load_vocabulary, \
     load_regex
 from mycroft.dialog import load_dialogs
-
-
-def get_non_properties(obj):
-    """Get attibutes that are not properties from object.
-
-    Will return members of object class along with bases down to MycroftSkill.
-
-    Arguments:
-        obj:    object to scan
-
-    Returns:
-        Set of attributes that are not a property.
-    """
-
-    def check_class(cls):
-        """Find all non-properties in a class."""
-        # Current class
-        d = cls.__dict__
-        np = [k for k in d if not isinstance(d[k], property)]
-        # Recurse through base classes excluding MycroftSkill and object
-        for b in [b for b in cls.__bases__ if b not in (object, MycroftSkill)]:
-            np += check_class(b)
-        return np
-
-    return set(check_class(obj.__class__))
+from ovos_workshop.helpers import get_non_properties
 
 
 class MycroftSkill(_MycroftSkill):

ovos_workshop/skills/ovos.py

@@ -11,7 +11,7 @@
 
 from mycroft import dialog
 from mycroft.skills.mycroft_skill.event_container import create_wrapper
-from ovos_workshop.skills.base import get_non_properties, Intent, IntentBuilder
+from ovos_workshop.helpers import get_non_properties, Intent, IntentBuilder
 from ovos_workshop.patches.base_skill import MycroftSkill, FallbackSkill
 from ovos_workshop.skills.decorators.killable import killable_event, \
     AbortEvent, AbortQuestion