refactor: use peel_to_commit

gix 0.66 added peel_to_commit(), which is now used to simplify several reference
to commit function call chains.

Author: jpgrayson

src/cmd/branch/rename.rs

@@ -60,9 +60,7 @@ pub(super) fn dispatch(repo: &gix::Repository, matches: &clap::ArgMatches) -> Re
         let state_commit = repo
             .find_reference(stack.get_stack_refname())
             .expect("just found this stack state reference")
-            .into_fully_peeled_id()?
-            .object()?
-            .try_into_commit()?;
+            .peel_to_commit()?;
         repo.edit_reference(gix::refs::transaction::RefEdit {
             change: gix::refs::transaction::Change::Update {
                 log: gix::refs::transaction::LogChange {

src/cmd/log.rs

@@ -115,12 +115,11 @@ fn run(matches: &ArgMatches) -> Result<()> {
         let simplified_parent_id = stack
             .repo
             .find_reference(stack.get_stack_refname())?
-            .into_fully_peeled_id()?
-            .object()?
-            .to_commit_ref()
-            .parents()
+            .peel_to_commit()?
+            .parent_ids()
             .next()
-            .ok_or_else(|| anyhow!("`{}` does not have any parents", stack.get_stack_refname()))?;
+            .ok_or_else(|| anyhow!("`{}` does not have any parents", stack.get_stack_refname()))?
+            .detach();
 
         let stupid = repo.stupid();
 

src/cmd/pull.rs

@@ -194,15 +194,10 @@ fn run(matches: &ArgMatches) -> Result<()> {
             let remote_name = remote_name.unwrap();
             print_info_message(matches, &format!("Fetching from `{remote_name}`"));
             stupid.user_fetch(&fetch_cmd, &remote_name)?;
-            let fetch_head = repo
+            let target_id = repo
                 .find_reference("FETCH_HEAD")
-                .context("finding `FETCH_HEAD`")?;
-            let target_id = fetch_head
-                .into_fully_peeled_id()
-                .map_err(anyhow::Error::from)
-                .and_then(|id| id.object().map_err(anyhow::Error::from))
-                .and_then(|object| object.peel_tags_to_end().map_err(anyhow::Error::from))
-                .and_then(|object| object.try_into_commit().map_err(anyhow::Error::from))
+                .context("finding `FETCH_HEAD`")?
+                .peel_to_commit()
                 .context("peeling `FETCH_HEAD` to commit")?
                 .id;
             Some(target_id)

src/cmd/undo.rs

@@ -83,10 +83,7 @@ pub(super) fn find_undo_state<'repo>(
         stack
             .repo
             .find_reference(stack.get_stack_refname())?
-            .into_fully_peeled_id()?
-            .object()?
-            .peel_tags_to_end()?
-            .try_into_commit()?,
+            .peel_to_commit()?,
     );
     loop {
         let state = StackState::from_commit(stack.repo, &state_commit)?;

src/stack/stack.rs

@@ -309,9 +309,7 @@ impl<'repo> Stack<'repo> {
         let prev_state_commit = self
             .repo
             .find_reference(&self.stack_refname)?
-            .into_fully_peeled_id()?
-            .object()?
-            .try_into_commit()?;
+            .peel_to_commit()?;
         let prev_state_commit_id = prev_state_commit.id;
         let state = self
             .state

src/stack/transaction/mod.rs

@@ -213,12 +213,10 @@ impl<'repo> ExecuteContext<'repo> {
             } else {
                 reflog_msg
             };
-            let stack_ref = repo.find_reference(stack.get_stack_refname())?;
             let branch_ref_name = stack.get_branch_refname().to_owned();
-            let prev_state_commit = stack_ref
-                .into_fully_peeled_id()?
-                .object()?
-                .try_into_commit()?;
+            let prev_state_commit = repo
+                .find_reference(stack.get_stack_refname())?
+                .peel_to_commit()?;
             let state = stack.state_mut();
             for (patchname, maybe_patch) in &updated_patches {
                 if let Some(patch) = maybe_patch {