feat: always use tags in our mirror repos

Author: dervoeti

rust/patchable/src/main.rs

@@ -480,21 +480,7 @@ fn main() -> Result<()> {
                 let mut push_options = git2::PushOptions::new();
                 push_options.remote_callbacks(callbacks);
 
-                // Check if the reference is a tag or branch by inspecting the git repository
-                let refspec = {
-                    let tag_ref = format!("refs/tags/{}", base);
-                    let is_tag = product_repo
-                        .find_reference(&tag_ref)
-                        .is_ok();
-
-                    if is_tag {
-                        format!("{}:refs/tags/{}", base_commit, base)
-                    } else {
-                        // Assume it's a branch as default behavior
-                        format!("{}:refs/heads/{}", base_commit, base)
-                    }
-                };
-
+                let refspec = format!("{}:refs/tags/{}", base_commit, base);
                 tracing::info!(refspec = refspec, "constructed push refspec");
 
                 mirror_remote

rust/patchable/src/repo.rs

@@ -176,9 +176,7 @@ pub fn resolve_and_fetch_commitish(
                     &[commitish],
                     Some(
                         FetchOptions::new()
-                            // Tags need to be present to later determine whether `commitish` is a tag or not
-                            // Patchable needs to know this when initializing a repository with the `--mirrored` option (to construct the refspec)
-                            .download_tags(git2::AutotagOption::Auto)
+                            .update_fetchhead(true)
                             .remote_callbacks(callbacks)
                             // TODO: could be 1, CLI option maybe?
                             .depth(0),
@@ -191,7 +189,8 @@ pub fn resolve_and_fetch_commitish(
                     refs: vec![commitish.to_string()],
                 })?;
             tracing::info!("fetched base commit");
-            repo.revparse_single(commitish)
+            // FETCH_HEAD is written by Remote::fetch to be the last reference fetched
+            repo.revparse_single("FETCH_HEAD")
                 .and_then(|obj| obj.peel_to_commit())
         }
         Err(err) => Err(err),