linode-cli 5.56.2__tar.gz → 5.56.3__tar.gz

{linode_cli-5.56.2/linode_cli.egg-info → linode_cli-5.56.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.2
 Name: linode-cli
-Version: 5.56.2
+Version: 5.56.3
 Summary: The official command-line interface for interacting with the Linode API.
 Author-email: "Akamai Technologies Inc." <developers@linode.com>
 License: BSD-3-Clause

{linode_cli-5.56.2 → linode_cli-5.56.3/linode_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.2
 Name: linode-cli
-Version: 5.56.2
+Version: 5.56.3
 Summary: The official command-line interface for interacting with the Linode API.
 Author-email: "Akamai Technologies Inc." <developers@linode.com>
 License: BSD-3-Clause

{linode_cli-5.56.2 → linode_cli-5.56.3}/linodecli/baked/request.py

@@ -2,9 +2,13 @@
 Request details for a CLI Operation
 """
 
+from typing import List, Optional
+
+from openapi3.paths import MediaType
 from openapi3.schemas import Schema
 
 from linodecli.baked.parsing import simplify_description
+from linodecli.baked.response import OpenAPIResponse
 from linodecli.baked.util import _aggregate_schema_properties
 
 
@@ -13,16 +17,16 @@ class OpenAPIRequestArg:
     A single argument to a request as defined by a Schema in the OpenAPI spec
     """
 
-    def __init__(
+    def __init__(  # pylint: disable=too-many-arguments
         self,
-        name,
-        schema,
-        required,
-        prefix=None,
-        is_parent=False,
-        parent=None,
-        depth=0,
-    ):  # pylint: disable=too-many-arguments
+        name: str,
+        schema: Schema,
+        required: bool,
+        prefix: Optional[str] = None,
+        is_parent: bool = False,
+        parent: Optional[str] = None,
+        depth: int = 0,
+    ) -> None:
         """
         Parses a single Schema node into a argument the CLI can use when making
         requests.
@@ -120,9 +124,14 @@ class OpenAPIRequestArg:
             )
 
 
-def _parse_request_model(schema, prefix=None, parent=None, depth=0):
+def _parse_request_model(
+    schema: Schema,
+    prefix: Optional[str] = None,
+    parent: Optional[str] = None,
+    depth: int = 0,
+) -> List[OpenAPIRequestArg]:
     """
-    Parses a schema into a list of OpenAPIRequest objects
+    Parses an OpenAPI schema into a list of OpenAPIRequest objects
     :param schema: The schema to parse as a request model
     :type schema: openapi3.Schema
     :param prefix: The prefix to add to all keys in this schema, as a json path
@@ -143,6 +152,7 @@ def _parse_request_model(schema, prefix=None, parent=None, depth=0):
         return args
 
     for k, v in properties.items():
+        # Handle nested objects which aren't read-only and have properties
         if (
             v.type == "object"
             and not v.readOnly
@@ -159,6 +169,8 @@ def _parse_request_model(schema, prefix=None, parent=None, depth=0):
                 # parent arguments.
                 depth=depth,
             )
+
+        # Handle arrays of objects that not marked as JSON
         elif (
             v.type == "array"
             and v.items
@@ -209,7 +221,7 @@ class OpenAPIRequest:
     on the MediaType object of a requestBody portion of an OpenAPI Operation
     """
 
-    def __init__(self, request):
+    def __init__(self, request: MediaType) -> None:
         """
         :param request: The request's MediaType object in the OpenAPI spec,
                         corresponding to the application/json data the endpoint
@@ -256,7 +268,7 @@ class OpenAPIFilteringRequest:
     endpoints where filters are accepted.
     """
 
-    def __init__(self, response_model):
+    def __init__(self, response_model: OpenAPIResponse) -> None:
         """
         :param response_model: The parsed response model whose properties may be
                                filterable.

{linode_cli-5.56.2 → linode_cli-5.56.3}/linodecli/baked/response.py

@@ -2,7 +2,10 @@
 Converting the processed OpenAPI Responses into something the CLI can work with
 """
 
+from typing import Optional
+
 from openapi3.paths import MediaType
+from openapi3.schemas import Schema
 
 from linodecli.baked.util import _aggregate_schema_properties
 
@@ -30,7 +33,13 @@ class OpenAPIResponseAttr:
     from it.
     """
 
-    def __init__(self, name, schema, prefix=None, nested_list_depth=0):
+    def __init__(
+        self,
+        name: str,
+        schema: Schema,
+        prefix: Optional[str] = None,
+        nested_list_depth: int = 0,
+    ) -> None:
         """
         :param name: The key that held this schema in the properties list, representing
                      its name in a response.
@@ -84,10 +93,13 @@ class OpenAPIResponseAttr:
             self.item_type = schema.items.type
 
     @property
-    def path(self):
+    def path(self) -> str:
         """
         This is a helper for filterable fields to return the json path to this
         element in a response.
+
+        :returns: The json path to the element in a response.
+        :rtype: str
         """
         return self.name
 
@@ -129,6 +141,7 @@ class OpenAPIResponseAttr:
             value = str(value)
             color = self.color_map.get(value) or self.color_map["default_"]
             value = f"[{color}]{value}[/]"
+        # Convert None value to an empty string for better display
         if value is None:
             # Prints the word None if you don't change it
             value = ""
@@ -194,12 +207,14 @@ def _parse_response_model(schema, prefix=None, nested_list_depth=0):
         elif v.type == "object":
             attrs += _parse_response_model(v, prefix=pref)
         elif v.type == "array" and v.items.type == "object":
+            # Parse arrays for objects recursively and increase the nesting depth
             attrs += _parse_response_model(
                 v.items,
                 prefix=pref,
                 nested_list_depth=nested_list_depth + 1,
             )
         else:
+            # Handle any other simple types
             attrs.append(
                 OpenAPIResponseAttr(
                     k, v, prefix=prefix, nested_list_depth=nested_list_depth
@@ -215,7 +230,7 @@ class OpenAPIResponse:
     responses section of an OpenAPI Operation
     """
 
-    def __init__(self, response: MediaType):
+    def __init__(self, response: MediaType) -> None:
         """
         :param response: The response's MediaType object in the OpenAPI spec,
                           corresponding to the application/json response type
@@ -287,15 +302,22 @@ class OpenAPIResponse:
 
         nested_lists = [c.strip() for c in self.nested_list.split(",")]
         result = []
+
         for nested_list in nested_lists:
             path_parts = nested_list.split(".")
+
             if not isinstance(json, list):
                 json = [json]
+
             for cur in json:
+                # Get the nested list using the path
                 nlist_path = cur
                 for p in path_parts:
                     nlist_path = nlist_path.get(p)
                 nlist = nlist_path
+
+                # For each item in the nested list,
+                # combine the parent properties with the nested item
                 for item in nlist:
                     cobj = {k: v for k, v in cur.items() if k != path_parts[0]}
                     cobj["_split"] = path_parts[-1]

{linode_cli-5.56.2 → linode_cli-5.56.3}/linodecli/configuration/config.py

@@ -82,10 +82,7 @@ class CLIConfig:
         :returns: The `default-user` username or an empty string.
         :rtype: str
         """
-        if self.config.has_option("DEFAULT", "default-user"):
-            return self.config.get("DEFAULT", "default-user")
-
-        return ""
+        return self.config.get("DEFAULT", "default-user", fallback="")
 
     def set_user(self, username: str):
         """
@@ -153,15 +150,11 @@ class CLIConfig:
         :rtype: str
         """
         if self.used_env_token:
-            return os.environ.get(ENV_TOKEN_NAME, None)
+            return os.getenv(ENV_TOKEN_NAME, None)
 
-        if self.config.has_option(
-            self.username or self.default_username(), "token"
-        ):
-            return self.config.get(
-                self.username or self.default_username(), "token"
-            )
-        return ""
+        return self.config.get(
+            self.username or self.default_username(), "token", fallback=""
+        )
 
     def get_value(self, key: str) -> Optional[Any]:
         """
@@ -180,12 +173,9 @@ class CLIConfig:
                   current user.
         :rtype: any
         """
-        username = self.username or self.default_username()
-
-        if not self.config.has_option(username, key):
-            return None
-
-        return self.config.get(username, key)
+        return self.config.get(
+            self.username or self.default_username(), key, fallback=None
+        )
 
     def get_bool(self, key: str) -> bool:
         """
@@ -204,12 +194,10 @@ class CLIConfig:
                   current user.
         :rtype: any
         """
-        username = self.username or self.default_username()
-
-        if not self.config.has_option(username, key):
-            return False
 
-        return self.config.getboolean(username, key)
+        return self.config.getboolean(
+            self.username or self.default_username(), key, fallback=False
+        )
 
     # plugin methods - these are intended for plugins to utilize to store their
     # own persistent config information
@@ -255,13 +243,10 @@ class CLIConfig:
                 "No running plugin to retrieve configuration for!"
             )
 
-        username = self.username or self.default_username()
+        username = self.username or self.default_username() or "DEFAULT"
         full_key = f"plugin-{self.running_plugin}-{key}"
 
-        if not self.config.has_option(username, full_key):
-            return None
-
-        return self.config.get(username, full_key)
+        return self.config.get(username, full_key, fallback=None)
 
     # TODO: this is more of an argparsing function than it is a config function
     # might be better to move this to argparsing during refactor and just have
@@ -308,11 +293,8 @@ class CLIConfig:
             # these don't get included in the updated namespace
             if key.startswith("plugin-"):
                 continue
-            value = None
-            if self.config.has_option(username, key):
-                value = self.config.get(username, key)
-            else:
-                value = ns_dict[key]
+
+            value = self.config.get(username, key, fallback=ns_dict.get(key))
 
             if not value:
                 continue
@@ -553,7 +535,7 @@ class CLIConfig:
 
         if len(users) == 0:
             # config is new or _really_ old
-            token = self.config.get("DEFAULT", "token")
+            token = self.config.get("DEFAULT", "token", fallback=None)
 
             if token is not None:
                 # there's a token in the config - configure that user

{linode_cli-5.56.2 → linode_cli-5.56.3}/linodecli/configuration/helpers.py

@@ -299,6 +299,4 @@ def _config_get_with_default(
     :returns: The value pulled from the config or the default value.
     :rtype: Any
     """
-    return (
-        config.get(user, field) if config.has_option(user, field) else default
-    )
+    return config.get(user, field, fallback=default)

{linode_cli-5.56.2 → linode_cli-5.56.3}/linodecli/version.py

@@ -2,4 +2,4 @@
 The version of the Linode CLI.
 """
 
-__version__ = "v5.56.2"
+__version__ = "v5.56.3"