From e2a7c81a132607b1430a32d206d3c6c59a2502a0 Mon Sep 17 00:00:00 2001
From: Sebastian Kuzminsky <seb@highlab.com>
Date: Tue, 22 Jul 2025 10:08:12 -0600
Subject: [PATCH] ent assign: speed up setting of assignee

---
 src/bin/ent/main.rs | 58 +++++++++++++++++++++++----------------------
 1 file changed, 30 insertions(+), 28 deletions(-)

diff --git a/src/bin/ent/main.rs b/src/bin/ent/main.rs
index d3975c3..8a947e5 100644
--- a/src/bin/ent/main.rs
+++ b/src/bin/ent/main.rs
@@ -415,35 +415,37 @@ fn handle_command(
         Commands::Assign {
             issue_id,
             new_assignee,
-        } => {
-            let issues = entomologist::database::read_issues_database(issues_database_source)?;
-            let Some(original_issue) = issues.issues.get(issue_id) else {
-                return Err(anyhow::anyhow!("issue {} not found", issue_id));
-            };
-            let old_assignee: String = match &original_issue.assignee {
-                Some(assignee) => assignee.clone(),
-                None => String::from("None"),
-            };
-            println!("issue: {}", issue_id);
-            match new_assignee {
-                Some(new_assignee) => {
-                    let issues_database = entomologist::database::make_issues_database(
-                        issues_database_source,
-                        entomologist::database::IssuesDatabaseAccess::ReadWrite,
-                    )?;
-                    let mut issues =
-                        entomologist::issues::Issues::new_from_dir(&issues_database.dir)?;
-                    let Some(issue) = issues.get_mut_issue(issue_id) else {
-                        return Err(anyhow::anyhow!("issue {} not found", issue_id));
-                    };
-                    println!("assignee: {} -> {}", old_assignee, new_assignee);
-                    issue.set_assignee(new_assignee)?;
-                }
-                None => {
-                    println!("assignee: {}", old_assignee);
-                }
+        } => match new_assignee {
+            Some(new_assignee) => {
+                let issues_database = entomologist::database::make_issues_database(
+                    issues_database_source,
+                    entomologist::database::IssuesDatabaseAccess::ReadWrite,
+                )?;
+                let mut issues = entomologist::issues::Issues::new_from_dir(&issues_database.dir)?;
+                let Some(issue) = issues.get_mut_issue(issue_id) else {
+                    return Err(anyhow::anyhow!("issue {} not found", issue_id));
+                };
+                let old_assignee: String = match &issue.assignee {
+                    Some(assignee) => assignee.clone(),
+                    None => String::from("None"),
+                };
+                issue.set_assignee(new_assignee)?;
+                println!("issue: {}", issue_id);
+                println!("assignee: {} -> {}", old_assignee, new_assignee);
             }
-        }
+            None => {
+                let issues = entomologist::database::read_issues_database(issues_database_source)?;
+                let Some(original_issue) = issues.issues.get(issue_id) else {
+                    return Err(anyhow::anyhow!("issue {} not found", issue_id));
+                };
+                let old_assignee: String = match &original_issue.assignee {
+                    Some(assignee) => assignee.clone(),
+                    None => String::from("None"),
+                };
+                println!("issue: {}", issue_id);
+                println!("assignee: {}", old_assignee);
+            }
+        },
 
         Commands::Tag { issue_id, tag } => match tag {
             Some(tag) => {