Debug performance around the new checkout

Author: Byron

crates/but-testing/src/args.rs

@@ -147,7 +147,7 @@ pub enum Subcommands {
     /// As StackIDs are going away, BranchDetails would be the part that remains.
     StackDetails {
         /// The ID of the stack to list details for.
-        id: StackId,
+        id: Option<StackId>,
     },
     /// Returns detailed
     BranchDetails {

crates/but-testing/src/command/mod.rs

@@ -196,15 +196,19 @@ pub mod stacks {
         }
     }
 
-    pub fn details(id: StackId, current_dir: &Path, v3: bool) -> anyhow::Result<()> {
+    pub fn details(id: Option<StackId>, current_dir: &Path, v3: bool) -> anyhow::Result<()> {
         let project = project_from_path(current_dir)?;
         let ctx = CommandContext::open(&project, AppSettings::default())?;
         let details = if v3 {
             let meta = ref_metadata_toml(ctx.project())?;
             let repo = ctx.gix_repo_for_merging_non_persisting()?;
-            but_workspace::stack_details_v3(id.into(), &repo, &meta)
+            but_workspace::stack_details_v3(id, &repo, &meta)
         } else {
-            but_workspace::stack_details(&project.gb_dir(), id, &ctx)
+            but_workspace::stack_details(
+                &project.gb_dir(),
+                id.context("a StackID is needed for the old implementation")?,
+                &ctx,
+            )
         }?;
         debug_print(details)
     }

crates/but-workspace/src/stacks.rs

@@ -441,14 +441,18 @@ pub fn stack_details_v3(
             .into_iter()
             .find_map(|(id, stack)| (id == stack_id).then_some(stack)))
     }
-    let ref_info_options = ref_info::Options {
+    let mut ref_info_options = ref_info::Options {
         // TODO(perf): make this so it can be enabled for a specific stack-id.
         expensive_commit_info: true,
         traversal: meta.graph_options(),
     };
     let mut stack = match stack_id {
         None => {
             // assume single-branch mode.
+            // Make sure the UI isn't overwhelmed, this currently happens easily on some repos where a lot of commits
+            // would otherwise be returned. The problem is that then the workspace might not be correct, but there isn't
+            // another way that still allows to extend the range via gas-stations. Maybe one day we won't need this.
+            ref_info_options.traversal.hard_limit = Some(500);
             let mut info = head_info(repo, meta, ref_info_options)?;
             if info.stacks.len() != 1 {
                 bail!(