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

2 files changed, 16 insertions, 18 deletions

diff --git a/src/bin/ngit/sub_commands/checkout.rs b/src/bin/ngit/sub_commands/checkout.rs
index 19e39d0..87f1ff2 100644
--- a/src/bin/ngit/sub_commands/checkout.rs
+++ b/src/bin/ngit/sub_commands/checkout.rs
@@ -24,7 +24,7 @@ use nostr_sdk::{EventId, FromBech32};
 use crate::{
     client::{Client, Connect, fetching_with_report, get_repo_ref_from_cache},
     git::{Repo, RepoActions, str_to_sha1},
-    git_events::{event_to_cover_letter, patch_supports_commit_ids},
+    git_events::{event_to_cover_letter, get_parent_commit_from_patch, patch_supports_commit_ids},
     repo_ref::get_repo_coordinates_when_remote_unknown,
 };
 
@@ -272,13 +272,12 @@ fn checkout_patch(
         );
     }
 
-    let proposal_base_commit = str_to_sha1(&tag_value(
-        most_recent_proposal_patch_chain_or_pr_or_pr_update
-            .last()
-            .context("there should be at least one patch")?,
-        "parent-commit",
-    )?)
-    .context("failed to get valid parent commit id from patch")?;
+    let last_patch = most_recent_proposal_patch_chain_or_pr_or_pr_update
+        .last()
+        .context("there should be at least one patch")?;
+
+    let proposal_base_commit = str_to_sha1(&get_parent_commit_from_patch(last_patch, Some(git_repo))?)
+        .context("failed to get valid parent commit id from patch")?;
 
     let (main_branch_name, _master_tip) = git_repo.get_main_or_master_branch()?;
 
diff --git a/src/bin/ngit/sub_commands/list.rs b/src/bin/ngit/sub_commands/list.rs
index 80eec21..133ac83 100644
--- a/src/bin/ngit/sub_commands/list.rs
+++ b/src/bin/ngit/sub_commands/list.rs
@@ -35,7 +35,7 @@ use crate::{
     git::{Repo, RepoActions, str_to_sha1},
     git_events::{
         commit_msg_from_patch_oneliner, event_is_revision_root, event_to_cover_letter,
-        patch_supports_commit_ids,
+        get_parent_commit_from_patch, patch_supports_commit_ids,
     },
     repo_ref::get_repo_coordinates_when_remote_unknown,
 };
@@ -703,15 +703,14 @@ async fn launch_interactive() -> Result<()> {
             .get_checked_out_branch_name()?
             .eq(&cover_letter.get_branch_name_with_pr_prefix_and_shorthand_id()?);
 
-        let proposal_base_commit = str_to_sha1(&tag_value(
-            most_recent_proposal_patch_chain_or_pr_or_pr_update
-                .last()
-                .context(
-                    "there should be at least one patch as we have already checked for this",
-                )?,
-            "parent-commit",
-        )?)
-        .context("failed to get valid parent commit id from patch")?;
+        let last_patch = most_recent_proposal_patch_chain_or_pr_or_pr_update
+            .last()
+            .context(
+                "there should be at least one patch as we have already checked for this",
+            )?;
+
+        let proposal_base_commit = str_to_sha1(&get_parent_commit_from_patch(last_patch, Some(&git_repo))?)
+            .context("failed to get valid parent commit id from patch")?;
 
         let (main_branch_name, master_tip) = git_repo.get_main_or_master_branch()?;