feat(status): add --reason to all pr and issue status commands

All status transitions (pr close/reopen/ready/draft, issue close/reopen/
resolved) now accept an optional --reason flag stored in the event content.

5 files changed, 41 insertions, 26 deletions

diff --git a/skills/ngit/SKILL.md b/skills/ngit/SKILL.md
index b097cdf..c316c1e 100644
--- a/skills/ngit/SKILL.md
+++ b/skills/ngit/SKILL.md
@@ -119,10 +119,10 @@ git push origin main      # push to nostr remote records the merge event
 ### Lifecycle
 
 ```bash
-ngit pr close <ID|nevent>
-ngit pr reopen <ID|nevent>
-ngit pr ready <ID|nevent>   # mark draft as ready for review
-ngit pr draft <ID|nevent>   # convert back to draft
+ngit pr close <ID|nevent> --reason "blocked by upstream"
+ngit pr reopen <ID|nevent> --reason "fix was incomplete"
+ngit pr ready <ID|nevent> --reason "addressed review feedback"
+ngit pr draft <ID|nevent> --reason "needs more work"
 ngit pr label <ID|nevent> --label bug --label enhancement
 ```
 
@@ -138,9 +138,9 @@ ngit issue view <ID|nevent> --json
 ngit issue view <ID|nevent> --json --comments
 ngit issue comment <ID|nevent> --body "Reproduced on v2.1"
 ngit issue comment <ID|nevent> --body "Thanks!" --reply-to <comment-ID|nevent>
-ngit issue close <ID|nevent> --reason "wontfix"        # closed without resolution
+ngit issue close <ID|nevent> --reason "wontfix"
 ngit issue resolved <ID|nevent> --reason "fixed in abc123"
-ngit issue reopen <ID|nevent>
+ngit issue reopen <ID|nevent> --reason "regression in v2.3"
 ngit issue label <ID|nevent> --label bug --label enhancement
 ```
 
diff --git a/src/bin/ngit/cli.rs b/src/bin/ngit/cli.rs
index 37d85a2..a240597 100644
--- a/src/bin/ngit/cli.rs
+++ b/src/bin/ngit/cli.rs
@@ -248,6 +248,9 @@ pub enum PrCommands {
         /// Proposal event-id (hex) or nevent (bech32)
         #[arg(value_name = "ID|nevent")]
         id: String,
+        /// Optional reason stored in event content
+        #[arg(long)]
+        reason: Option<String>,
         /// Use local cache only, skip network fetch
         #[arg(long)]
         offline: bool,
@@ -257,6 +260,9 @@ pub enum PrCommands {
         /// Proposal event-id (hex) or nevent (bech32)
         #[arg(value_name = "ID|nevent")]
         id: String,
+        /// Optional reason stored in event content
+        #[arg(long)]
+        reason: Option<String>,
         /// Use local cache only, skip network fetch
         #[arg(long)]
         offline: bool,
@@ -266,6 +272,9 @@ pub enum PrCommands {
         /// Proposal event-id (hex) or nevent (bech32)
         #[arg(value_name = "ID|nevent")]
         id: String,
+        /// Optional reason stored in event content
+        #[arg(long)]
+        reason: Option<String>,
         /// Use local cache only, skip network fetch
         #[arg(long)]
         offline: bool,
@@ -275,6 +284,9 @@ pub enum PrCommands {
         /// Proposal event-id (hex) or nevent (bech32)
         #[arg(value_name = "ID|nevent")]
         id: String,
+        /// Optional reason stored in event content
+        #[arg(long)]
+        reason: Option<String>,
         /// Use local cache only, skip network fetch
         #[arg(long)]
         offline: bool,
@@ -414,6 +426,9 @@ pub enum IssueCommands {
         /// Issue event-id (hex) or nevent (bech32)
         #[arg(value_name = "ID|nevent")]
         id: String,
+        /// Optional reason stored in event content
+        #[arg(long)]
+        reason: Option<String>,
         /// Use local cache only, skip network fetch
         #[arg(long)]
         offline: bool,
diff --git a/src/bin/ngit/main.rs b/src/bin/ngit/main.rs
index a0cb3e6..3686011 100644
--- a/src/bin/ngit/main.rs
+++ b/src/bin/ngit/main.rs
@@ -102,17 +102,17 @@ async fn main() {
                 PrCommands::Send(sub_args) => {
                     sub_commands::send::launch(&cli, sub_args, false).await
                 }
-                PrCommands::Close { id, offline } => {
-                    sub_commands::pr_status::launch_close(id, *offline).await
+                PrCommands::Close { id, reason, offline } => {
+                    sub_commands::pr_status::launch_close(id, *offline, reason.as_deref()).await
                 }
-                PrCommands::Reopen { id, offline } => {
-                    sub_commands::pr_status::launch_reopen(id, *offline).await
+                PrCommands::Reopen { id, reason, offline } => {
+                    sub_commands::pr_status::launch_reopen(id, *offline, reason.as_deref()).await
                 }
-                PrCommands::Ready { id, offline } => {
-                    sub_commands::pr_status::launch_ready(id, *offline).await
+                PrCommands::Ready { id, reason, offline } => {
+                    sub_commands::pr_status::launch_ready(id, *offline, reason.as_deref()).await
                 }
-                PrCommands::Draft { id, offline } => {
-                    sub_commands::pr_status::launch_draft(id, *offline).await
+                PrCommands::Draft { id, reason, offline } => {
+                    sub_commands::pr_status::launch_draft(id, *offline, reason.as_deref()).await
                 }
                 PrCommands::Comment {
                     id,
@@ -188,8 +188,8 @@ async fn main() {
                     sub_commands::issue_status::launch_resolved(id, *offline, reason.as_deref())
                         .await
                 }
-                IssueCommands::Reopen { id, offline } => {
-                    sub_commands::issue_status::launch_reopen(id, *offline).await
+                IssueCommands::Reopen { id, reason, offline } => {
+                    sub_commands::issue_status::launch_reopen(id, *offline, reason.as_deref()).await
                 }
                 IssueCommands::Comment {
                     id,
diff --git a/src/bin/ngit/sub_commands/issue_status.rs b/src/bin/ngit/sub_commands/issue_status.rs
index 840ab8e..99b4fa6 100644
--- a/src/bin/ngit/sub_commands/issue_status.rs
+++ b/src/bin/ngit/sub_commands/issue_status.rs
@@ -183,8 +183,8 @@ pub async fn launch_close(id: &str, offline: bool, reason: Option<&str>) -> Resu
     launch_status(id, offline, Kind::GitStatusClosed, "closed", reason).await
 }
 
-pub async fn launch_reopen(id: &str, offline: bool) -> Result<()> {
-    launch_status(id, offline, Kind::GitStatusOpen, "reopened", None).await
+pub async fn launch_reopen(id: &str, offline: bool, reason: Option<&str>) -> Result<()> {
+    launch_status(id, offline, Kind::GitStatusOpen, "reopened", reason).await
 }
 
 pub async fn launch_resolved(id: &str, offline: bool, reason: Option<&str>) -> Result<()> {
diff --git a/src/bin/ngit/sub_commands/pr_status.rs b/src/bin/ngit/sub_commands/pr_status.rs
index 12aafb7..4a51bb3 100644
--- a/src/bin/ngit/sub_commands/pr_status.rs
+++ b/src/bin/ngit/sub_commands/pr_status.rs
@@ -193,18 +193,18 @@ async fn launch_status(
     Ok(())
 }
 
-pub async fn launch_close(id: &str, offline: bool) -> Result<()> {
-    launch_status(id, offline, Kind::GitStatusClosed, "closed", None).await
+pub async fn launch_close(id: &str, offline: bool, reason: Option<&str>) -> Result<()> {
+    launch_status(id, offline, Kind::GitStatusClosed, "closed", reason).await
 }
 
-pub async fn launch_reopen(id: &str, offline: bool) -> Result<()> {
-    launch_status(id, offline, Kind::GitStatusOpen, "reopened", None).await
+pub async fn launch_reopen(id: &str, offline: bool, reason: Option<&str>) -> Result<()> {
+    launch_status(id, offline, Kind::GitStatusOpen, "reopened", reason).await
 }
 
-pub async fn launch_ready(id: &str, offline: bool) -> Result<()> {
-    launch_status(id, offline, Kind::GitStatusOpen, "marked as ready", None).await
+pub async fn launch_ready(id: &str, offline: bool, reason: Option<&str>) -> Result<()> {
+    launch_status(id, offline, Kind::GitStatusOpen, "marked as ready", reason).await
 }
 
-pub async fn launch_draft(id: &str, offline: bool) -> Result<()> {
-    launch_status(id, offline, Kind::GitStatusDraft, "converted to draft", None).await
+pub async fn launch_draft(id: &str, offline: bool, reason: Option<&str>) -> Result<()> {
+    launch_status(id, offline, Kind::GitStatusDraft, "converted to draft", reason).await
 }