Add resolved_issue_priority_distribution

Goder-0 · Goder-0 · commit 600f68f1de3e · 2025-09-15T10:15:41.000+09:00
Close #196
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -12,6 +12,8 @@ this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.htm
   `resolvedIssueCount` is added, indicating the number of resolved issues.
   Currently, an issue is defined to be resolved if and only if (1) it is
   "Closed" and (2) status of project item "to-do list" is "Done".
+- Added `resolved_issue_priority_distribution` field to `issueStat` query,
+  which shows the distribution of priorities for resolved issues.
 - Tracing with a filter set by `RUST_LOG` environment variable.
 - Added support for passing the SSH passphrase through the `SSH_PASSPHRASE`
   environment variable.
diff --git a/src/api/issue_stat.rs b/src/api/issue_stat.rs
@@ -31,12 +31,35 @@ impl From<&str> for IssueSize {
         }
     }
 }
+#[derive(Enum, Copy, Clone, Eq, PartialEq, Ord, PartialOrd, Debug)]
+enum IssuePriority {
+    P0,
+    P1,
+    P2,
+    None,
+}
+
+impl From<&str> for IssuePriority {
+    fn from(s: &str) -> Self {
+        match s {
+            "P0" => Self::P0,
+            "P1" => Self::P1,
+            "P2" => Self::P2,
+            _ => Self::None,
+        }
+    }
+}
 
 #[derive(SimpleObject)]
 struct IssueSizeCount {
     size: IssueSize,
     count: usize,
 }
+#[derive(SimpleObject, Debug)]
+struct IssuePriorityCount {
+    priority: IssuePriority,
+    count: usize,
+}
 
 #[derive(InputObject, Debug)]
 pub(crate) struct IssueStatFilter {
@@ -91,6 +114,9 @@ struct IssueStat {
 
     /// The distribution of resolved issues by size.
     resolved_issue_size_distribution: Vec<IssueSizeCount>,
+
+    /// The distribution of priorities for resolved issues.
+    resolved_issue_priority_distribution: Vec<IssuePriorityCount>,
 }
 
 #[Object]
@@ -132,10 +158,29 @@ impl IssueStatQuery {
             .map(|(size, count)| IssueSizeCount { size, count })
             .collect();
 
+        let resolved_issue_priority_distribution = resolved_issues
+            .iter()
+            .fold(BTreeMap::new(), |mut acc, issue| {
+                let priority_str = issue
+                    .project_items
+                    .nodes
+                    .iter()
+                    .find(|p| p.project_title == super::issue::TODO_LIST_PROJECT_TITLE)
+                    .and_then(|p| p.todo_priority.as_deref())
+                    .unwrap_or("None");
+
+                *acc.entry(priority_str.into()).or_insert(0) += 1;
+                acc
+            })
+            .into_iter()
+            .map(|(priority, count)| IssuePriorityCount { priority, count })
+            .collect();
+
         Ok(IssueStat {
             open_issue_count,
             resolved_issue_count,
             resolved_issue_size_distribution,
+            resolved_issue_priority_distribution,
         })
     }
 }
@@ -555,7 +600,6 @@ mod tests {
         let schema = TestSchema::new();
         let owner: &str = "aicers";
         let repo = "github-dashboard-server";
-
         let mut resolved_issues = create_resolved_issues(1..=6);
         // 1 XS
         resolved_issues[0]
@@ -624,4 +668,50 @@ mod tests {
             ])
         );
     }
+
+    #[tokio::test]
+    async fn resolved_issue_priority_distribution() {
+        let schema = TestSchema::new();
+        let owner: &str = "aicers";
+        let repo = "github-dashboard-server";
+
+        let mut resolved_issues = create_resolved_issues(1..=10);
+        // P0: 2
+        resolved_issues[0].project_items.nodes[0].todo_priority = Some("P0".to_string());
+        resolved_issues[1].project_items.nodes[0].todo_priority = Some("P0".to_string());
+        // P1: 3
+        resolved_issues[2].project_items.nodes[0].todo_priority = Some("P1".to_string());
+        resolved_issues[3].project_items.nodes[0].todo_priority = Some("P1".to_string());
+        resolved_issues[4].project_items.nodes[0].todo_priority = Some("P1".to_string());
+        // P2: 1
+        resolved_issues[5].project_items.nodes[0].todo_priority = Some("P2".to_string());
+        // None: 4
+
+        schema
+            .db
+            .insert_issues(resolved_issues, owner, repo)
+            .unwrap();
+
+        let query = r"
+        {
+            issueStat(filter: {}) {
+                resolvedIssuePriorityDistribution {
+                    priority
+                    count
+                }
+            }
+        }";
+        let data = schema.execute(query).await.data.into_json().unwrap();
+        let dist = &data["issueStat"]["resolvedIssuePriorityDistribution"];
+
+        assert_eq!(dist.as_array().unwrap().len(), 4);
+        assert_eq!(dist[0]["priority"], "P0");
+        assert_eq!(dist[0]["count"], 2);
+        assert_eq!(dist[1]["priority"], "P1");
+        assert_eq!(dist[1]["count"], 3);
+        assert_eq!(dist[2]["priority"], "P2");
+        assert_eq!(dist[2]["count"], 1);
+        assert_eq!(dist[3]["priority"], "NONE");
+        assert_eq!(dist[3]["count"], 4);
+    }
 }
