Added agreed/disagreed result for discussions

README.md

@@ -231,6 +231,8 @@ Click the links for detailed report formats and insights.
 
 To obtain a detailed report on the types of open discussions, it is necessary to include a specific label in the first message of each discussion, enclosed in double square brackets (`[[ ]]`). For example, use `[[Performance issue]]` to categorize a discussion as related to performance issues. The action will then provide a breakdown of discussions based on these labels, allowing for a more targeted and categorized analysis of discussion topics.
 
+In addition, reactions of :+1: and :-1: on the initial message of a discussion will be collected and displayed in the columns for "discussions received" and "discussions conducted." These reactions help to more accurately gauge the usefulness and reception of the discussions initiated.
+
 **Example Usage**:
 
 - In the first comment of a pull request discussion, include a label like `[[Bug]]`, `[[Feature Request]]`, or any custom label of your choice.

package.json

@@ -1,6 +1,6 @@
 {
   "name": "pull-request-analytics-action",
-  "version": "1.8.3",
+  "version": "1.8.4",
   "description": "Generates detailed PR analytics reports within GitHub, focusing on review efficiency and team performance.",
   "main": "build/index.js",
   "scripts": {

src/converters/types.ts

@@ -34,6 +34,11 @@ type PullRequestTimelineInfo = {
   timeToApprove: number;
 };
 
+type Discussion = {
+  received?: DiscussionResult;
+  conducted?: DiscussionResult;
+};
+
 export type Collection = {
   opened: number;
   closed: number;
@@ -48,13 +53,12 @@ export type Collection = {
   timeToMerge?: number[];
   comments: number;
   totalReviewComments: number;
-  discussions?: number;
   reviewComments?: number;
   reviewsConducted?: {
     [key: string]: ReviewTypeStats;
   };
   commentsConducted?: number;
-  discussionsConducted?: number;
+  discussions?: Discussion;
   discussionsTypes?: DiscussionType;
   prSizes?: string[];
   reviewsConductedSize?: PullRequestSize[];

src/converters/utils/prepareDiscussions.ts

@@ -38,6 +38,12 @@ export const prepareDiscussions = (
             total:
               (collection[userLogin][key].discussionsTypes![type]?.conducted
                 ?.total || 0) + 1,
+            agreed:
+              (collection[userLogin][key].discussionsTypes![type]?.conducted
+                ?.agreed || 0) + (discussion.reactions?.["+1"] ? 1 : 0),
+            disagreed:
+              (collection[userLogin][key].discussionsTypes![type]?.conducted
+                ?.disagreed || 0) + (discussion.reactions?.["-1"] ? 1 : 0),
           },
         };
         collection[pullRequestLogin][key].discussionsTypes![type] = {
@@ -46,6 +52,12 @@ export const prepareDiscussions = (
             total:
               (collection[pullRequestLogin][key].discussionsTypes![type]
                 ?.received?.total || 0) + 1,
+            agreed:
+              (collection[userLogin][key].discussionsTypes![type]?.received
+                ?.agreed || 0) + (discussion.reactions?.["+1"] ? 1 : 0),
+            disagreed:
+              (collection[userLogin][key].discussionsTypes![type]?.received
+                ?.disagreed || 0) + (discussion.reactions?.["-1"] ? 1 : 0),
           },
         };
         collection.total[key].discussionsTypes![type] = {
@@ -54,11 +66,23 @@ export const prepareDiscussions = (
             total:
               (collection.total[key].discussionsTypes![type]?.conducted
                 ?.total || 0) + 1,
+            agreed:
+              (collection.total[key].discussionsTypes![type]?.conducted
+                ?.agreed || 0) + (discussion.reactions?.["+1"] ? 1 : 0),
+            disagreed:
+              (collection.total[key].discussionsTypes![type]?.conducted
+                ?.disagreed || 0) + (discussion.reactions?.["-1"] ? 1 : 0),
           },
           received: {
             total:
               (collection.total[key].discussionsTypes![type]?.received?.total ||
                 0) + 1,
+            agreed:
+              (collection.total[key].discussionsTypes![type]?.received
+                ?.agreed || 0) + (discussion.reactions?.["+1"] ? 1 : 0),
+            disagreed:
+              (collection.total[key].discussionsTypes![type]?.received
+                ?.disagreed || 0) + (discussion.reactions?.["-1"] ? 1 : 0),
           },
         };
       });
@@ -85,18 +109,69 @@ export const prepareDiscussions = (
 
     discussions?.forEach((discussion) => {
       const userLogin = discussion.user?.login || invalidUserLogin;
-      collection[userLogin][key].discussionsConducted =
-        (collection[userLogin][key].discussionsConducted || 0) + 1;
-      collection.total[key].discussionsConducted =
-        (collection.total[key].discussionsConducted || 0) + 1;
+      collection[userLogin][key].discussions = {
+        ...collection[userLogin][key].discussions,
+        conducted: {
+          total:
+            (collection[userLogin][key].discussions?.conducted?.total || 0) + 1,
+          agreed:
+            (collection[userLogin][key].discussions?.conducted?.agreed || 0) +
+            (discussion.reactions?.["+1"] ? 1 : 0),
+          disagreed:
+            (collection[userLogin][key].discussions?.conducted?.disagreed ||
+              0) + (discussion.reactions?.["-1"] ? 1 : 0),
+        },
+      };
+      collection.total[key].discussions = {
+        ...collection.total[key].discussions,
+        conducted: {
+          total: (collection.total[key].discussions?.conducted?.total || 0) + 1,
+          agreed:
+            (collection.total[key].discussions?.conducted?.agreed || 0) +
+            (discussion.reactions?.["+1"] ? 1 : 0),
+          disagreed:
+            (collection.total[key].discussions?.conducted?.disagreed || 0) +
+            (discussion.reactions?.["-1"] ? 1 : 0),
+        },
+      };
     });
 
     if (pullRequestLogin) {
-      collection[pullRequestLogin][key]["discussions"] =
-        (discussions?.length || 0) +
-        (collection[pullRequestLogin][key].discussions || 0);
-      collection.total[key]["discussions"] =
-        (discussions?.length || 0) + (collection.total[key].discussions || 0);
+      const agreedDiscussions = discussions?.filter(
+        (discussion) => discussion.reactions?.["+1"]
+      );
+      const disagreedDiscussions = discussions?.filter(
+        (discussion) => discussion.reactions?.["-1"]
+      );
+
+      collection[pullRequestLogin][key].discussions = {
+        ...collection[pullRequestLogin][key].discussions,
+        received: {
+          total:
+            (collection[pullRequestLogin][key].discussions?.received?.total ||
+              0) + (discussions?.length || 0),
+          agreed:
+            (collection[pullRequestLogin][key].discussions?.received?.agreed ||
+              0) + (agreedDiscussions?.length || 0),
+          disagreed:
+            (collection[pullRequestLogin][key].discussions?.received
+              ?.disagreed || 0) + (disagreedDiscussions?.length || 0),
+        },
+      };
+      collection.total[key].discussions = {
+        ...collection.total[key].discussions,
+        received: {
+          total:
+            (collection.total[key].discussions?.received?.total || 0) +
+            (discussions?.length || 0),
+          agreed:
+            (collection.total[key].discussions?.received?.agreed || 0) +
+            (agreedDiscussions?.length || 0),
+          disagreed:
+            (collection.total[key].discussions?.received?.disagreed || 0) +
+            (disagreedDiscussions?.length || 0),
+        },
+      };
     }
   });
 };

src/view/utils/constants.ts

@@ -6,8 +6,10 @@ export const totalOpenedPrsHeader = "Total opened PRs";
 export const additionsDeletionsHeader = "Additions/Deletions";
 export const reviewCommentsHeader = "Total comments";
 export const reviewConductedHeader = "Reviews conducted";
-export const discussionsHeader = "Discussions received";
-export const discussionsConductedHeader = "Discussions conducted";
+export const discussionsHeader =
+  "Agreed / Disagreed / Total discussions received";
+export const discussionsConductedHeader =
+  "Agreed / Disagreed / Total discussions conducted";
 export const commentsConductedHeader = "Comments conducted";
 export const commentsReceivedHeader = "Comments received";
 export const reviewTypesHeader = "Changes requested / Comments / Approvals";

src/view/utils/createPullRequestQualityTable.ts

@@ -29,7 +29,14 @@ export const createPullRequestQualityTable = (
         data["total"]?.[date]?.reviewsConducted?.[user]?.[
           "CHANGES_REQUESTED"
         ]?.toString() || "0",
-        data[user]?.[date]?.discussions?.toString() || "0",
+        `${
+          data[user]?.[date]?.discussions?.received?.agreed?.toString() || "0"
+        } / ${
+          data[user]?.[date]?.discussions?.received?.disagreed?.toString() ||
+          "0"
+        } / ${
+          data[user]?.[date]?.discussions?.received?.total?.toString() || "0"
+        }`,
         data[user]?.[date]?.reviewComments?.toString() || "0",
       ];
     });

src/view/utils/createReviewTable.ts

@@ -24,7 +24,14 @@ export const createReviewTable = (
       return [
         `**${user}**`,
         data[user]?.[date]?.merged?.toString() || "0",
-        data[user]?.[date]?.discussionsConducted?.toString() || "0",
+        `${
+          data[user]?.[date]?.discussions?.conducted?.agreed?.toString() || "0"
+        } / ${
+          data[user]?.[date]?.discussions?.conducted?.disagreed?.toString() ||
+          "0"
+        } / ${
+          data[user]?.[date]?.discussions?.conducted?.total?.toString() || "0"
+        }`,
         data[user]?.[date]?.commentsConducted?.toString() || "0",
         `${sizes
           .map(