crossplane-function-pythonic 0.6.3__py3-none-any.whl → 0.7.0__py3-none-any.whl

crossplane/pythonic/__about__.py

@@ -1,8 +1,6 @@
 from importlib.metadata import PackageNotFoundError, version
 
 try:
-    __version__ = version(
-        "crossplane-function-pythonic"
-    )  # Use the exact [project.name] from pyproject.toml
+    __version__ = version("crossplane-function-pythonic")
 except PackageNotFoundError:
     __version__ = "0.0.0"

crossplane/pythonic/auto_ready.py

@@ -1,4 +1,13 @@
-def resource_ready(resource):
+from typing import TYPE_CHECKING
+
+if TYPE_CHECKING:
+    from . import (
+        composite,
+        protobuf,
+    )
+
+
+def resource_ready(resource: composite.Resource) -> protobuf.Value | None:
     if not resource.observed:
         return None
     return _checks.get(
@@ -7,9 +16,9 @@ def resource_ready(resource):
 
 
 class ReadyCondition:
-    def ready(self, resource):
+    def ready(self, resource: composite.Resource) -> protobuf.Value | None:
         ready = resource.conditions.Ready
-        if not ready._find_condition():
+        if not ready._get_condition():
             return None
         if ready.status:
             return resource.observed.metadata.name
@@ -18,7 +27,7 @@ class ReadyCondition:
         return resource.status.notReadyCondition
 
 
-_checks = {}
+_checks: dict[tuple, Check] = {}
 _check_default = ReadyCondition()
 
 
@@ -29,12 +38,12 @@ class Check:
         if hasattr(cls, "apiVersion"):
             _checks[(cls.apiVersion, cls.__name__)] = cls()
 
-    def ready(self, resource):
+    def ready(self, resource: composite.Resource) -> protobuf.Value | None:
         raise NotImplementedError()
 
 
 class AlwaysReady(Check):
-    def ready(self, resource):
+    def ready(self, resource: composite.Resource) -> protobuf.Value:
         return resource.observed.metadata.name
 
 
@@ -53,7 +62,7 @@ class ConfigMap(AlwaysReady):
 class CronJob(Check):
     apiVersion = "batch/v1"
 
-    def ready(self, resource):
+    def ready(self, resource: composite.Resource) -> protobuf.Value:
         if resource.observed.spec.suspend and len(resource.observed.spec.suspend):
             return resource.observed.metadata.name
         if not resource.status.lastScheduleTime:
@@ -72,7 +81,7 @@ class CronJob(Check):
 class DaemonSet(Check):
     apiVersion = "apps/v1"
 
-    def ready(self, resource):
+    def ready(self, resource: composite.Resource) -> protobuf.Value:
         scheduled = resource.status.desiredNumberScheduled
         if not scheduled:
             return scheduled
@@ -88,7 +97,7 @@ class DaemonSet(Check):
 class Deployment(Check):
     apiVersion = "apps/v1"
 
-    def ready(self, resource):
+    def ready(self, resource: composite.Resource) -> protobuf.Value:
         replicas = resource.observed.spec.replicas or 1
         for field in ("updatedReplicas", "availableReplicas"):
             value = resource.status[field]
@@ -109,7 +118,7 @@ class Deployment(Check):
 class HorizontalPodAutoscaler(Check):
     apiVersion = "autoscaling/v2"
 
-    def ready(self, resource):
+    def ready(self, resource: composite.Resource) -> protobuf.Value:
         for type in (
             "FailedGetScale",
             "FailedUpdateScale",
@@ -121,22 +130,22 @@ class HorizontalPodAutoscaler(Check):
         for type in ("ScalingActive", "ScalingLimited"):
             if resource.conditions[type].status:
                 return resource.observed.metadata.name
-        return resource.status.notScalingActiveOrLimiited
+        return resource.status.notScalingActiveOrLimited
 
 
 class Ingress(Check):
     apiVersion = "networking.k8s.io/v1"
 
-    def ready(self, resource):
+    def ready(self, resource: composite.Resource) -> protobuf.Value:
         if not len(resource.status.loadBalancer.ingress):
-            return resource.status.noLoadBalanceIngresses
+            return resource.status.noLoadBalancerIngresses
         return resource.observed.metadata.name
 
 
 class Job(Check):
     apiVersion = "batch/v1"
 
-    def ready(self, resource):
+    def ready(self, resource: composite.Resource) -> protobuf.Value:
         for type in ("Failed", "Suspended"):
             if resource.conditions[type].status:
                 return resource.status[f"is{type}"]
@@ -157,7 +166,7 @@ class Namespace(AlwaysReady):
 class PersistentVolumeClaim(Check):
     apiVersion = "v1"
 
-    def ready(self, resource):
+    def ready(self, resource: composite.Resource) -> protobuf.Value:
         if resource.status.phase != "Bound":
             return resource.status.phaseNotBound
         return resource.observed.metadata.name
@@ -166,7 +175,7 @@ class PersistentVolumeClaim(Check):
 class Pod(Check):
     apiVersion = "v1"
 
-    def ready(self, resource):
+    def ready(self, resource: composite.Resource) -> protobuf.Value:
         if resource.status.phase == "Succeeded":
             return resource.observed.metadata.name
         if (
@@ -179,9 +188,9 @@ class Pod(Check):
 
 
 class ReplicaSet(Check):
-    apiVersion = "v1"
+    apiVersion = "apps/v1"
 
-    def ready(self, resource):
+    def ready(self, resource: composite.Resource) -> protobuf.Value:
         if int(resource.status.observedGeneration) < int(
             resource.observed.metadata.generation
         ):
@@ -195,7 +204,7 @@ class ReplicaSet(Check):
         if int(resource.status.availableReplicas) < int(
             resource.observed.spec.replicas or 1
         ):
-            return resource.status.tooFewavailableReplicas
+            return resource.status.tooFewAvailableReplicas
         return resource.observed.metadata.name
 
 
@@ -214,7 +223,7 @@ class Secret(AlwaysReady):
 class Service(Check):
     apiVersion = "v1"
 
-    def ready(self, resource):
+    def ready(self, resource: composite.Resource) -> protobuf.Value:
         if resource.observed.spec.type != "LoadBalancer":
             return resource.observed.metadata.name
         if not len(resource.status.loadBalancer.ingress):
@@ -229,7 +238,7 @@ class ServiceAccount(AlwaysReady):
 class StatefulSet(Check):
     apiVersion = "apps/v1"
 
-    def ready(self, resource):
+    def ready(self, resource: composite.Resource) -> protobuf.Value:
         replicas = resource.observed.spec.replicas or 1
         for field in ("readyReplicas", "currentReplicas"):
             value = resource.status[field]
@@ -238,5 +247,5 @@ class StatefulSet(Check):
             if replicas != value:
                 return resource.status[f"{field}NotReplicas"]
         if resource.status.currentRevision != resource.status.updateRevision:
-            return resource.status.currentRevisionNotUpdateReivsion
+            return resource.status.currentRevisionNotUpdateRevision
         return resource.observed.metadata.name

crossplane/pythonic/command.py

@@ -1,27 +1,32 @@
+import argparse
 import logging
 import pathlib
 import sys
 
 
 class Command:
-    name = None
-    command = None
-    description = None
+    name: str | None = None
+    command: str | None = None
+    help: str | None = None
+    description: str | None = None
 
     @classmethod
-    def create(cls, subparsers):
-        parser = subparsers.add_parser(
-            cls.name, help=cls.help, description=cls.description
-        )
-        parser.set_defaults(command=cls)
-        cls.add_parser_arguments(parser)
+    def create(
+        cls, subparsers: argparse._SubParsersAction[argparse.ArgumentParser]
+    ) -> None:
+        if cls.name is not None:
+            parser: argparse.ArgumentParser = subparsers.add_parser(
+                cls.name, help=cls.help, description=cls.description
+            )
+            parser.set_defaults(command=cls)
+            cls.add_parser_arguments(parser)
 
     @classmethod
-    def add_parser_arguments(cls, parser):
+    def add_parser_arguments(cls, parser: argparse.ArgumentParser) -> None:
         pass
 
     @classmethod
-    def add_function_arguments(cls, parser):
+    def add_function_arguments(cls, parser: argparse.ArgumentParser) -> None:
         parser.add_argument(
             "--debug",
             "-d",
@@ -66,14 +71,14 @@ class Command:
             help="Enable Crossplane V1 compatibility mode",
         )
 
-    def __init__(self, args=None):
-        self.args = args
+    def __init__(self, args: argparse.Namespace | None = None) -> None:
+        self.args = args if args is not None else argparse.Namespace()
         self.initialize()
 
-    def initialize(self):
+    def initialize(self) -> None:
         pass
 
-    def initialize_function(self):
+    def initialize_function(self) -> None:
         formatter = Formatter(self.args.log_name_width)
         handler = logging.StreamHandler(sys.stdout)
         handler.setFormatter(formatter)
@@ -98,15 +103,16 @@ class Command:
         if self.args.allow_oversize_protos:
             from google.protobuf.internal import api_implementation
 
-            if api_implementation._c_module:
-                api_implementation._c_module.SetAllowOversizeProtos(True)
+            c_module = getattr(api_implementation, "_c_module", None)
+            if c_module:
+                c_module.SetAllowOversizeProtos(True)
 
-    async def run(self):
+    async def run(self) -> None:
         raise NotImplementedError()
 
 
 class Formatter(logging.Formatter):
-    def __init__(self, name_width):
+    def __init__(self, name_width: int):
         super().__init__(
             f"[{{asctime}}.{{msecs:03.0f}}] {{sname:{name_width}.{name_width}}} [{{levelname:8.8}}] {{message}}",
             "%Y-%m-%d %H:%M:%S",
@@ -114,16 +120,17 @@ class Formatter(logging.Formatter):
         )
         self.name_width = name_width
 
-    def format(self, record):
-        record.sname = record.name
-        extra = len(record.sname) - self.name_width
+    def format(self, record: logging.LogRecord):
+        sname = record.name
+        extra = len(sname) - self.name_width
         if extra > 0:
-            names = record.sname.split(".")
+            names = sname.split(".")
             for ix, name in enumerate(names):
                 if len(name) > extra:
                     names[ix] = name[extra:]
                     break
                 names[ix] = name[:1]
                 extra -= len(name) - 1
-            record.sname = ".".join(names)
+            sname = ".".join(names)
+        record.sname = sname
         return super().format(record)