refactor: Improve logging and update extractor path handling

Author: GeekMasher

src/extractors.rs

@@ -6,12 +6,14 @@ use std::{os::unix::fs::PermissionsExt, path::PathBuf};
 
 async fn fetch_releases(client: &octocrab::Octocrab, repository: &Repository) -> Result<Release> {
     let release = if let Some(rel) = &repository.reference {
+        log::info!("Fetching release by tag: {}", rel);
         client
             .repos(repository.owner.clone(), repository.name.clone())
             .releases()
             .get_by_tag(&rel)
             .await?
     } else {
+        log::info!("Fetching latest release",);
         // Get Latest Release
         client
             .repos(repository.owner.clone(), repository.name.clone())
@@ -20,7 +22,7 @@ async fn fetch_releases(client: &octocrab::Octocrab, repository: &Repository) ->
             .await?
     };
 
-    log::debug!("Release :: {} - {:?}", release.tag_name, release.created_at);
+    log::info!("Release :: {} - {:?}", release.tag_name, release.created_at);
 
     Ok(release)
 }

src/main.rs

@@ -18,7 +18,9 @@ async fn main() -> Result<()> {
 
     debug!("Action :: {action:?}");
 
-    let client = action.octocrab()?;
+    // Use a client without a token to not use Action' token which will fail
+    // as the token won't have access to the CodeQL repository.
+    let octocrab = action.octocrab_without_token()?;
 
     let cwd = action
         .working_directory()
@@ -37,10 +39,6 @@ async fn main() -> Result<()> {
     log::debug!("CodeQL :: {codeql:?}");
 
     if !codeql.is_installed().await {
-        // Use a client without a token to not use Action' token which will fail
-        // as the token won't have access to the CodeQL repository.
-        let octocrab = action.octocrab_without_token()?;
-
         let codeql_version = action.codeql_version();
         log::info!("CodeQL not installed, installing `{codeql_version}`...");
         codeql
@@ -62,18 +60,24 @@ async fn main() -> Result<()> {
         .extractor_repository()
         .context("Failed to get extractor repository")?;
 
-    let extractor_path = cwd.join(".codeql").join("extractors");
+    let extractor_path = codeql_dir.join("extractors");
     if !extractor_path.exists() {
-        std::fs::create_dir(&extractor_path)
+        std::fs::create_dir_all(&extractor_path)
             .with_context(|| format!("Failed to create directory {extractor_path:?}"))?;
         info!("Created Extractor Directory :: {extractor_path:?}");
     }
 
     let mut extractors: Vec<(CodeQLExtractor, RepositoryReference)> = Vec::new();
 
     for extractor_repo in extractor_repos.iter() {
+        log::info!(
+            "Fetching extractor from repository: {} / {}",
+            extractor_repo.owner,
+            extractor_repo.name
+        );
+
         let extractor_path = extractors::fetch_extractor(
-            &client,
+            &octocrab,
             extractor_repo,
             action.attestation(),
             &extractor_path,