fix: handle None values in metadata dictionary access

Co-Authored-By: Aaron <AJ> Steers <[email protected]>

Author: devin-ai-integration[bot]

airbyte_cdk/qa/checks/metadata.py

@@ -100,7 +100,7 @@ def _run(self, connector: Connector) -> CheckResult:
                 message="Could not infer the language tag from the connector directory",
             )
 
-        current_language_tags = [t for t in connector.metadata.get("tags", []) if t.startswith("language:")]
+        current_language_tags = [t for t in (connector.metadata.get("tags", []) if connector.metadata else []) if t.startswith("language:")]
         if not current_language_tags:
             return self.fail(
                 connector=connector,
@@ -170,7 +170,7 @@ def _run(self, connector: Connector) -> CheckResult:
         Returns:
             CheckResult: The result of the check
         """
-        current_cdk_tags = [t for t in connector.metadata.get("tags", []) if t.startswith("cdk:")]
+        current_cdk_tags = [t for t in (connector.metadata.get("tags", []) if connector.metadata else []) if t.startswith("cdk:")]
         expected_cdk_tag = self.get_expected_cdk_tag(connector)
         if not current_cdk_tags:
             return self.fail(
@@ -217,7 +217,7 @@ def _run(self, connector: Connector) -> CheckResult:
                 message="Can't verify breaking changes deadline: connector version is not defined.",
             )
 
-        breaking_changes = connector.metadata.get("releases", {}).get("breakingChanges")
+        breaking_changes = (connector.metadata.get("releases", {}) if connector.metadata else {}).get("breakingChanges")
 
         if not breaking_changes:
             return self.pass_(
@@ -270,7 +270,7 @@ def _run(self, connector: Connector) -> CheckResult:
         Returns:
             CheckResult: The result of the check
         """
-        max_seconds_between_messages = connector.metadata.get("maxSecondsBetweenMessages")
+        max_seconds_between_messages = connector.metadata.get("maxSecondsBetweenMessages") if connector.metadata else None
         if not max_seconds_between_messages:
             return self.fail(
                 connector=connector,

airbyte_cdk/qa/checks/packaging.py

@@ -280,7 +280,7 @@ def _run(self, connector: Connector) -> CheckResult:
         Returns:
             CheckResult: The result of the check
         """
-        if "dockerImageTag" not in connector.metadata:
+        if not connector.metadata or "dockerImageTag" not in connector.metadata:
             return self.create_check_result(
                 connector=connector,
                 passed=False,
@@ -317,7 +317,7 @@ def _run(self, connector: Connector) -> CheckResult:
         Returns:
             CheckResult: The result of the check
         """
-        if "dockerImageTag" not in connector.metadata:
+        if not connector.metadata or "dockerImageTag" not in connector.metadata:
             return self.create_check_result(
                 connector=connector,
                 passed=False,

airbyte_cdk/qa/checks/security.py

@@ -175,7 +175,7 @@ def _run(self, connector: Connector) -> CheckResult:
                 message=f"Dockerfile file exists. Please remove it and declare the base image in {consts.METADATA_FILE_NAME} file with the `connectorBuildOptions.baseImage` key",
             )
 
-        base_image = connector.metadata.get("connectorBuildOptions", {}).get("baseImage")
+        base_image = (connector.metadata.get("connectorBuildOptions", {}) if connector.metadata else {}).get("baseImage")
         if not base_image:
             return self.create_check_result(
                 connector=connector,