From fcff4541e1f36b6575596c353637b25aeae9bdcf Mon Sep 17 00:00:00 2001
From: DanConwayDev <DanConwayDev@protonmail.com>
Date: Wed, 18 Feb 2026 14:48:20 +0000
Subject: feat: handle missing optional patch tags for pr/ flow

- Add mbox_parser module to extract metadata from patch content
- Extract author/committer from From: and Date: headers when tags missing
- Extract commit message body as fallback for description tag
- Implement best-guess parent commit logic using committer timestamps
- Update patch_supports_commit_ids to accept mbox-parseable patches
- Enable patches without optional tags to appear as pr/ branches
---
 src/lib/git_events.rs | 50 +++++++++++++++++++++++++++++++++++++++++++++-----
 1 file changed, 45 insertions(+), 5 deletions(-)

(limited to 'src/lib/git_events.rs')

diff --git a/src/lib/git_events.rs b/src/lib/git_events.rs
index e907a9b..b39e797 100644
--- a/src/lib/git_events.rs
+++ b/src/lib/git_events.rs
@@ -40,6 +40,30 @@ pub fn get_commit_id_from_patch(event: &Event) -> Result<String> {
     }
 }
 
+pub fn get_parent_commit_from_patch(
+    event: &Event,
+    git_repo: Option<&Repo>,
+) -> Result<String> {
+    if let Ok(parent) = tag_value(event, "parent-commit") {
+        return Ok(parent);
+    }
+
+    let metadata = crate::mbox_parser::parse_mbox_patch(&event.content)
+        .context("failed to parse patch for timestamp")?;
+    let timestamp = metadata.committer_timestamp.unwrap_or(metadata.author_timestamp);
+
+    if let Some(repo) = git_repo {
+        if let Some(best_guess) = repo
+            .find_best_guess_parent_commit(timestamp)
+            .context("failed to find best guess parent commit")?
+        {
+            return Ok(best_guess.to_string());
+        }
+    }
+
+    bail!("no parent-commit tag and could not determine best guess parent")
+}
+
 pub fn get_event_root(event: &nostr::Event) -> Result<EventId> {
     Ok(EventId::parse(
         event
@@ -88,11 +112,27 @@ pub fn event_is_revision_root(event: &Event) -> bool {
 }
 
 pub fn patch_supports_commit_ids(event: &Event) -> bool {
-    event.kind.eq(&Kind::GitPatch)
-        && event
-            .tags
-            .iter()
-            .any(|t| !t.as_slice().is_empty() && t.as_slice()[0].eq("commit-pgp-sig"))
+    if !event.kind.eq(&Kind::GitPatch) {
+        return false;
+    }
+
+    if event
+        .tags
+        .iter()
+        .any(|t| !t.as_slice().is_empty() && t.as_slice()[0].eq("commit-pgp-sig"))
+    {
+        return true;
+    }
+
+    if event
+        .tags
+        .iter()
+        .any(|t| !t.as_slice().is_empty() && t.as_slice()[0].eq("parent-commit"))
+    {
+        return true;
+    }
+
+    crate::mbox_parser::parse_mbox_patch(&event.content).is_ok()
 }
 
 pub fn event_is_valid_pr_or_pr_update(event: &Event) -> bool {
-- 
cgit v1.2.3