From 8af9c71ef6953471db44acdd56bd707bb85ff4b7 Mon Sep 17 00:00:00 2001
From: Sebastian Kuzminsky <seb@highlab.com>
Date: Sat, 19 Jul 2025 21:10:14 -0600
Subject: [PATCH] `ent done-time ISSUE TIME`: report parse error instead of
 panicking

---
 src/bin/ent/main.rs | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/src/bin/ent/main.rs b/src/bin/ent/main.rs
index 1698954..651e0fc 100644
--- a/src/bin/ent/main.rs
+++ b/src/bin/ent/main.rs
@@ -499,9 +499,13 @@ fn handle_command(
                     let Some(issue) = issues.get_mut_issue(issue_id) else {
                         return Err(anyhow::anyhow!("issue {} not found", issue_id));
                     };
-                    let done_time = chrono::DateTime::parse_from_rfc3339(done_time)
-                        .unwrap()
-                        .with_timezone(&chrono::Local);
+                    let done_time = match chrono::DateTime::parse_from_rfc3339(done_time) {
+                        Ok(done_time) => done_time.with_timezone(&chrono::Local),
+                        Err(e) => {
+                            eprintln!("failed to parse done-time from {}", done_time);
+                            return Err(e.into());
+                        }
+                    };
                     issue.set_done_time(done_time)?;
                 }
                 None => match &issue.done_time {