chore: move the looking up of repository before the build tool lookup because without repository no build tool is found

This commit also add some useful debug messages for extracting values
from the database for Reproducible Central buildspec generation.

Author: Trong Nhan Mai

src/macaron/build_spec_generator/reproducible_central/rc_build_info.py

@@ -4,6 +4,7 @@
 """This module contains the representation of information needed for Reproducible Central Buildspec generation."""
 
 import logging
+import pprint
 from collections.abc import Sequence
 from dataclasses import dataclass
 
@@ -38,6 +39,12 @@ class RcInternalBuildInfo:
     build_tool_facts: Sequence[BuildToolFacts]
 
 
+def format_build_command_infos(build_command_infos: list[GenericBuildCommandInfo]) -> str:
+    """Return the prettified str format for a list of `GenericBuildCommandInfo` instances."""
+    pretty_formatted_ouput = [pprint.pformat(build_command_info) for build_command_info in build_command_infos]
+    return "\n".join(pretty_formatted_ouput)
+
+
 def get_rc_internal_build_info(
     purl: PackageURL,
     session: Session,
@@ -78,55 +85,69 @@ def get_rc_internal_build_info(
     logger.debug("Latest component ID: %d", latest_component_id)
 
     try:
-        build_tool_facts = lookup_build_tools_check(
-            component_id=latest_component_id,
-            session=session,
-        )
-    except QueryMacaronDatabaseError as lookup_build_tools_error:
+        lookup_component_repository = lookup_repository(latest_component_id, session)
+    except QueryMacaronDatabaseError as lookup_repository_error:
         logger.error(
-            "Unexpected result from querying build tools for %s. Error: %s",
+            "Unexpected result from querying repository information for %s. Error: %s",
             purl.to_string(),
-            lookup_build_tools_error,
+            lookup_repository_error,
         )
         return None
-    if not build_tool_facts:
+    if not lookup_component_repository:
         logger.error(
-            "Cannot find any build tool for PackageURL %s in the database.",
+            "Cannot find any repository information for %s in the database.",
             purl.to_string(),
         )
         return None
-    logger.debug("Build tools discovered from the %s table: %s", BuildToolFacts.__tablename__, build_tool_facts)
+    logger.info(
+        "Repository information for purl %s: url %s, commit %s",
+        purl,
+        lookup_component_repository.remote_path,
+        lookup_component_repository.commit_sha,
+    )
 
     try:
-        lookup_component_repository = lookup_repository(latest_component_id, session)
-    except QueryMacaronDatabaseError as lookup_repository_error:
+        build_tool_facts = lookup_build_tools_check(
+            component_id=latest_component_id,
+            session=session,
+        )
+    except QueryMacaronDatabaseError as lookup_build_tools_error:
         logger.error(
-            "Unexpected result from querying repository information for %s. Error: %s",
+            "Unexpected result from querying build tools for %s. Error: %s",
             purl.to_string(),
-            lookup_repository_error,
+            lookup_build_tools_error,
         )
         return None
-    if not lookup_component_repository:
+    if not build_tool_facts:
         logger.error(
-            "Cannot find any repository information for %s in the database.",
+            "Cannot find any build tool for PackageURL %s in the database.",
             purl.to_string(),
         )
         return None
+    logger.info(
+        "Build tools discovered from the %s table: %s",
+        BuildToolFacts.__tablename__,
+        [fact.build_tool_name for fact in build_tool_facts],
+    )
 
     try:
-        lookup_build_facts = lookup_any_build_command(latest_component_id, session)
+        lookup_build_command_infos = lookup_any_build_command(latest_component_id, session)
     except QueryMacaronDatabaseError as lookup_build_command_error:
         logger.error(
             "Unexpected result from querying all build command information for %s. Error: %s",
             purl.to_string(),
             lookup_build_command_error,
         )
         return None
+    logger.debug(
+        "Build command information discovered\n%s",
+        format_build_command_infos(lookup_build_command_infos),
+    )
 
     return RcInternalBuildInfo(
         purl=purl,
         repository=lookup_component_repository,
         latest_component_id=latest_component_id,
         build_tool_facts=build_tool_facts,
-        generic_build_command_facts=lookup_build_facts,
+        generic_build_command_facts=lookup_build_command_infos,
     )