precommit black/flake8/isort fixes

Author: craigbarratt

.github/workflows/validate.yml

@@ -9,26 +9,26 @@ jobs:
     runs-on: "ubuntu-latest"
     name: Validate
     steps:
-        - uses: "actions/checkout@v2"
-        - name: HACS validation
-          uses: "hacs/integration/action@master"
-          with:
-            GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-            CATEGORY: "integration"
+      - uses: "actions/checkout@v2"
+      - name: HACS validation
+        uses: "hacs/integration/action@master"
+        with:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          CATEGORY: "integration"
 
-        - name: Hassfest validation
-          uses: "home-assistant/actions/hassfest@master"
+      - name: Hassfest validation
+        uses: "home-assistant/actions/hassfest@master"
 
   style:
     runs-on: "ubuntu-latest"
     name: Check style formatting
     steps:
-        - uses: "actions/checkout@v2"
-        - uses: "actions/setup-python@v1"
-          with:
-            python-version: "3.x"
-        - run: python3 -m pip install black
-        - run: black .
+      - uses: "actions/checkout@v2"
+      - uses: "actions/setup-python@v1"
+        with:
+          python-version: "3.x"
+      - run: python3 -m pip install black
+      - run: black .
 
   pytest:
     runs-on: "ubuntu-latest"

.yamllint

@@ -57,5 +57,4 @@ rules:
     type: unix
   trailing-spaces:
     level: error
-  truthy:
-    level: error
+  truthy: disable

README.md

@@ -682,7 +682,7 @@ previously called `task.unique` with the same `task_name`. The name can be any s
 `task.unique` with the same `task_name`.
 
 Note that `task.unique` applies across all global contexts. It's up to you to use a convention
-for `task_name` that avoids accidential collisions. For example, you could use a prefix of the
+for `task_name` that avoids accidental collisions. For example, you could use a prefix of the
 script file name, so that all `task_unique` calls in `FILENAME.py` use a `task_name` that
 starts with `"FILENAME."`.
 

custom_components/pyscript/function.py

@@ -5,8 +5,6 @@
 import logging
 import traceback
 
-from homeassistant.helpers.service import async_get_all_descriptions
-
 from .const import LOGGER_PATH
 
 _LOGGER = logging.getLogger(LOGGER_PATH + ".handler")
@@ -51,21 +49,25 @@ def __init__(self):
     def init(cls, hass):
         """Initialize Function."""
         cls.hass = hass
-        cls.functions.update({
-            "task.executor": cls.task_executor,
-            "event.fire": cls.event_fire,
-            "task.sleep": cls.async_sleep,
-            "task.unique": cls.task_unique,
-            "service.call": cls.service_call,
-            "service.has_service": cls.service_has_service,
-        })
-        cls.ast_functions.update({
-            "log.debug": lambda ast_ctx: ast_ctx.get_logger().debug,
-            "log.error": lambda ast_ctx: ast_ctx.get_logger().error,
-            "log.info": lambda ast_ctx: ast_ctx.get_logger().info,
-            "log.warning": lambda ast_ctx: ast_ctx.get_logger().warning,
-            "print": lambda ast_ctx: ast_ctx.get_logger().debug,
-        })
+        cls.functions.update(
+            {
+                "task.executor": cls.task_executor,
+                "event.fire": cls.event_fire,
+                "task.sleep": cls.async_sleep,
+                "task.unique": cls.task_unique,
+                "service.call": cls.service_call,
+                "service.has_service": cls.service_has_service,
+            }
+        )
+        cls.ast_functions.update(
+            {
+                "log.debug": lambda ast_ctx: ast_ctx.get_logger().debug,
+                "log.error": lambda ast_ctx: ast_ctx.get_logger().error,
+                "log.info": lambda ast_ctx: ast_ctx.get_logger().info,
+                "log.warning": lambda ast_ctx: ast_ctx.get_logger().warning,
+                "print": lambda ast_ctx: ast_ctx.get_logger().debug,
+            }
+        )
 
     @classmethod
     async def entity_ids(cls, domain=None):
@@ -148,7 +150,11 @@ async def service_completions(cls, root):
         if num_period == 1:
             domain, svc_root = root.split(".")
             if domain in services:
-                words |= {f"{domain}.{svc}" for svc in services[domain] if svc.lower().startswith(svc_root)}
+                words |= {
+                    f"{domain}.{svc}"
+                    for svc in services[domain]
+                    if svc.lower().startswith(svc_root)
+                }
         elif num_period == 0:
             words |= {domain for domain in services if domain.lower().startswith(root)}
 

custom_components/pyscript/global_ctx.py

@@ -17,14 +17,12 @@
 
 _LOGGER = logging.getLogger(LOGGER_PATH + ".global_ctx")
 
+
 class GlobalContext:
     """Define class for global variables and trigger context."""
 
     def __init__(
-        self,
-        name,
-        hass,
-        global_sym_table=None,
+        self, name, hass, global_sym_table=None,
     ):
         """Initialize GlobalContext."""
         self.name = name
@@ -100,12 +98,15 @@ async def trigger_init(self, func):
                         desc = desc[4:].lstrip(" \n\r")
                         file_desc = io.StringIO(desc)
                         service_desc = (
-                            yaml.load(file_desc, Loader=yaml.BaseLoader) or OrderedDict()
+                            yaml.load(file_desc, Loader=yaml.BaseLoader)
+                            or OrderedDict()
                         )
                         file_desc.close()
                     except Exception as exc:
                         self.logger.error(
-                            "Unable to decode yaml doc_string for %s(): %s", func_name, str(exc)
+                            "Unable to decode yaml doc_string for %s(): %s",
+                            func_name,
+                            str(exc),
                         )
                         raise HomeAssistantError(exc)
                 else:
@@ -139,9 +140,7 @@ async def do_service_call(func, ast_ctx, data):
                     return pyscript_service_handler
 
                 self.hass.services.async_register(
-                    DOMAIN,
-                    func_name,
-                    pyscript_service_factory(func_name, func),
+                    DOMAIN, func_name, pyscript_service_factory(func_name, func),
                 )
                 async_set_service_schema(self.hass, DOMAIN, func_name, service_desc)
                 self.services.add(func_name)
@@ -196,7 +195,7 @@ async def do_service_call(func, ast_ctx, data):
                         func_name,
                         self.name,
                         dec_name,
-                        arg_num + 1
+                        arg_num + 1,
                     )
                     del trig_args[dec_name]
                     break
@@ -254,9 +253,7 @@ async def do_service_call(func, ast_ctx, data):
             await self.triggers[func_name].stop()
 
         self.triggers_new[func_name] = TrigInfo(
-            f"{self.name}.{func_name}",
-            trig_args,
-            global_ctx=self,
+            f"{self.name}.{func_name}", trig_args, global_ctx=self,
         )
 
         if self.auto_start:
@@ -313,6 +310,7 @@ class GlobalContextMgr:
     name_seq = 0
 
     def __init__(self):
+        """Report an error if GlobalContextMgr in instantiated."""
         _LOGGER.error("GlobalContextMgr class is not meant to be instantiated")
 
     @classmethod
@@ -321,27 +319,33 @@ def init(cls):
 
         def get_global_ctx_factory(ast_ctx):
             """Generate a pyscript.get_global_ctx() function with given ast_ctx."""
+
             async def get_global_ctx():
                 return ast_ctx.get_global_ctx_name()
+
             return get_global_ctx
 
         def list_global_ctx_factory(ast_ctx):
             """Generate a pyscript.list_global_ctx() function with given ast_ctx."""
+
             async def list_global_ctx():
                 ctx_names = set(cls.contexts.keys())
                 curr_ctx_name = ast_ctx.get_global_ctx_name()
                 ctx_names.discard(curr_ctx_name)
                 return [curr_ctx_name] + sorted(sorted(ctx_names))
+
             return list_global_ctx
 
         def set_global_ctx_factory(ast_ctx):
             """Generate a pyscript.set_global_ctx() function with given ast_ctx."""
+
             async def set_global_ctx(name):
                 global_ctx = cls.get(name)
                 if global_ctx is None:
                     raise NameError(f"global context '{name}' does not exist")
                 ast_ctx.set_global_ctx(global_ctx)
                 ast_ctx.set_logger_name(global_ctx.name)
+
             return set_global_ctx
 
         ast_funcs = {