Feature: users activity

action.yml

@@ -114,6 +114,10 @@ inputs:
     description: "Shows correlation graphs"
     required: false
     default: false
+  SHOW_ACTIVITY_TIME_GRAPHS:
+    description: "Shows activity time graphs"
+    required: false
+    default: false
 
 outputs:
   JSON_COLLECTION:

package.json

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

src/analytics/index.ts

@@ -1,3 +1,2 @@
 export { sendActionRun } from "./sendActionRun";
 export { sendActionError } from "./sendActionError";
-export { sendDiscussionUsage } from "./sendDiscussionUsage";

src/analytics/sendActionError.ts

@@ -41,6 +41,8 @@ export const sendActionError = (error: Error) => {
       ),
       MERGE_TIME_INTERVALS: getMultipleValuesInput("MERGE_TIME_INTERVALS"),
       USE_CHARTS: getValueAsIs("USE_CHARTS"),
+      SHOW_CORRELATION_GRAPHS: getValueAsIs("SHOW_CORRELATION_GRAPHS"),
+      SHOW_ACTIVITY_TIME_GRAPHS: getValueAsIs("SHOW_ACTIVITY_TIME_GRAPHS"),
     });
   } else {
     mixpanel.track("Anonymous action error", { distinct_id: "anonymous" });

src/analytics/sendActionRun.ts

@@ -39,6 +39,8 @@ export const sendActionRun = () => {
       ),
       MERGE_TIME_INTERVALS: getMultipleValuesInput("MERGE_TIME_INTERVALS"),
       USE_CHARTS: getValueAsIs("USE_CHARTS"),
+      SHOW_CORRELATION_GRAPHS: getValueAsIs("SHOW_CORRELATION_GRAPHS"),
+      SHOW_ACTIVITY_TIME_GRAPHS: getValueAsIs("SHOW_ACTIVITY_TIME_GRAPHS"),
     });
   } else {
     mixpanel.track("Anomymous action run", { distinct_id: "anonymous" });

src/converters/collectData.ts

@@ -12,6 +12,7 @@ import {
   preparePullRequestTimeline,
   prepareResponseTime,
   prepareRequestedReviews,
+  prepareActionsTime,
 } from "./utils";
 import {
   invalidUserLogin,
@@ -47,6 +48,12 @@ export const collectData = (
       )
     );
 
+    prepareActionsTime(
+      pullRequest,
+      data.events[index]?.filter((el) => el),
+      collection
+    );
+
     const closedDate = pullRequest.closed_at
       ? parseISO(pullRequest.closed_at)
       : null;

src/converters/types.ts

@@ -82,6 +82,16 @@ export type Collection = {
   commentsConducted?: number;
   discussions?: Discussion;
   discussionsTypes?: DiscussionType;
+  actionsTime?: Record<
+    string,
+    {
+      opened?: number;
+      merged?: number;
+      approved?: number;
+      changes_requested?: number;
+      commented?: number;
+    }
+  >;
   prSizes?: string[];
   sizes?: Record<
     string,

src/converters/utils/index.ts

@@ -5,3 +5,4 @@ export { prepareDiscussions } from "./prepareDiscussions";
 export { prepareReviews } from "./prepareReviews";
 export { prepareResponseTime } from "./prepareResponseTime";
 export { prepareRequestedReviews } from "./prepareRequestedReviews";
+export { prepareActionsTime } from "./prepareActionsTime";

src/converters/utils/prepareActionsTime.ts

@@ -0,0 +1,64 @@
+import { getHours, parseISO } from "date-fns";
+import { makeComplexRequest } from "../../requests";
+import { Collection } from "../types";
+import { get, set } from "lodash";
+import { invalidUserLogin, reviewedTimelineEvent } from "../constants";
+
+export const prepareActionsTime = (
+  pullRequest: Awaited<
+    ReturnType<typeof makeComplexRequest>
+  >["pullRequestInfo"][number],
+  events: any[] | undefined = [],
+  collection: Record<string, Record<string, Collection>>
+) => {
+  const openingHour = pullRequest?.created_at
+    ? getHours(parseISO(pullRequest?.created_at))
+    : -1;
+  const mergingHour = pullRequest?.merged_at
+    ? getHours(parseISO(pullRequest?.merged_at))
+    : -1;
+
+  events
+    ?.filter((el) => el.event === reviewedTimelineEvent)
+    .map((el) => {
+      const submitHour = el?.submitted_at
+        ? getHours(parseISO(el?.submitted_at))
+        : -1;
+      if (submitHour !== -1) {
+        const keys = ["total", "total", "actionsTime", submitHour, el.state];
+        set(collection, keys, (get(collection, keys, 0) as number) + 1);
+        const userKeys = [
+          el?.user?.login || invalidUserLogin,
+          "total",
+          "actionsTime",
+          submitHour,
+          el.state,
+        ];
+        set(collection, userKeys, (get(collection, userKeys, 0) as number) + 1);
+      }
+    });
+  if (openingHour !== -1) {
+    const keys = ["total", "total", "actionsTime", openingHour, "opened"];
+    set(collection, keys, get(collection, keys, 0) + 1);
+    const userKeys = [
+      pullRequest?.user.login || invalidUserLogin,
+      "total",
+      "actionsTime",
+      openingHour,
+      "opened",
+    ];
+    set(collection, userKeys, get(collection, userKeys, 0) + 1);
+  }
+  if (mergingHour !== -1) {
+    const keys = ["total", "total", "actionsTime", mergingHour, "merged"];
+    set(collection, keys, get(collection, keys, 0) + 1);
+    const userKeys = [
+      pullRequest?.user.login || invalidUserLogin,
+      "total",
+      "actionsTime",
+      openingHour,
+      "merged",
+    ];
+    set(collection, userKeys, get(collection, userKeys, 0) + 1);
+  }
+};

src/createOutput.ts

@@ -11,6 +11,7 @@ import {
 } from "./view/utils";
 import { octokit } from "./octokit";
 import { showStatsTypes } from "./common/constants";
+import { createActivityTimeMarkdown } from "./view/utils/createActivityTimeMarkdown";
 
 export const createOutput = async (
   data: Record<string, Record<string, Collection>>
@@ -75,6 +76,23 @@ export const createOutput = async (
           title: "Correlation Graphs",
         });
       }
+      if (getValueAsIs("SHOW_ACTIVITY_TIME_GRAPHS") === "true") {
+        const activityComment = await octokit.rest.issues.createComment({
+          repo: getValueAsIs("GITHUB_REPO_FOR_ISSUE"),
+          owner: getValueAsIs("GITHUB_OWNER_FOR_ISSUE"),
+          issue_number: issue.data.number,
+          body: createActivityTimeMarkdown(data, users, [
+            {
+              title: "Pull Request report total",
+              link: `${issue.data.html_url}#`,
+            },
+          ]),
+        });
+        comments.push({
+          comment: activityComment,
+          title: "Activity time Graphs",
+        });
+      }
       console.log("Issue successfully created.");
       for (let date of dates) {
         if (date === "total") continue;

src/index.ts

@@ -20,7 +20,6 @@ import { getRateLimit } from "./requests/getRateLimit";
 import {
   sendActionError,
   sendActionRun,
-  sendDiscussionUsage,
 } from "./analytics";
 
 async function main() {
@@ -91,7 +90,6 @@ async function main() {
       }
     );
     const preparedData = collectData(mergedData, teams);
-    sendDiscussionUsage(preparedData);
     console.log("Calculation complete. Generating markdown.");
     await createOutput(preparedData);
 

src/view/utils/createActivityTimeMarkdown.ts

@@ -0,0 +1,18 @@
+import { Collection } from "../../converters";
+import { createActivityXYChart } from "./createActivityXYChart";
+import { createReferences } from "./createReferences";
+
+export const createActivityTimeMarkdown = (
+  data: Record<string, Record<string, Collection>>,
+  users: string[],
+  references: { title: string; link: string }[] = []
+) => {
+  const activity = users
+    .filter(
+      (user) => Object.keys(data[user]?.total?.actionsTime || {}).length > 1
+    )
+    .map((user) => createActivityXYChart(data, user))
+    .join("\n");
+
+  return [createReferences(references)].concat(activity).join("\n").trim();
+};

src/view/utils/createActivityXYChart.ts

@@ -0,0 +1,60 @@
+import { Collection } from "../../converters";
+import { createXYChart } from "./common/createXYChart";
+
+export const createActivityXYChart = (
+  data: Record<string, Record<string, Collection>>,
+  user: string
+) => {
+  const hours = Array.from({ length: 24 }, (_, i) => i.toString());
+  return createXYChart({
+    title: `Activity ${user}`,
+    xAxis: hours,
+    yAxis: {
+      min: 0,
+      max: Math.max(
+        ...Object.values(data[user]?.total?.actionsTime || {}).map((value) =>
+          Math.max(...Object.values(value), 0)
+        ),
+        1
+      ),
+      title: "Amount",
+    },
+    lines: [
+      {
+        name: "Opened",
+        color: "black",
+        values: hours.map(
+          (el) => data[user]?.total?.actionsTime?.[el]?.opened || 0
+        ),
+      },
+      {
+        name: "Merged",
+        color: "purple",
+        values: hours.map(
+          (el) => data[user]?.total?.actionsTime?.[el]?.merged || 0
+        ),
+      },
+      {
+        name: "Approved",
+        color: "green",
+        values: hours.map(
+          (el) => data[user]?.total?.actionsTime?.[el]?.approved || 0
+        ),
+      },
+      {
+        name: "Changes Requested",
+        color: "red",
+        values: hours.map(
+          (el) => data[user]?.total?.actionsTime?.[el]?.changes_requested || 0
+        ),
+      },
+      {
+        name: "Commented",
+        color: "blue",
+        values: hours.map(
+          (el) => data[user]?.total?.actionsTime?.[el]?.commented || 0
+        ),
+      },
+    ],
+  });
+};

src/view/utils/createConfigParamsCode.ts

@@ -18,6 +18,7 @@ ${[
   "TOP_LIST_AMOUNT",
   "AGGREGATE_VALUE_METHODS",
   "SHOW_CORRELATION_GRAPHS",
+  "SHOW_ACTIVITY_TIME_GRAPHS",
   "PERCENTILE",
   "HIDE_USERS",
   "SHOW_USERS",