Merge pull request #125 from ArjinAlbay/main

Add 'Add to Kanban' quick action for Quick Wins issues

Author: ArjinAlbay

src/app/action-required/page.tsx

@@ -66,6 +66,20 @@ function formatTimeAgo(dateString: string): string {
   return `${Math.floor(diffInSeconds / 2592000)}mo ago`;
 }
 
+function getStaleRowClassName(daysOld: number | undefined): string {
+  if (!daysOld) return "";
+
+  if (daysOld > 14) {
+    return "bg-red-50 dark:bg-red-950/20 hover:bg-red-100 dark:hover:bg-red-950/30";
+  }
+
+  if (daysOld > 7) {
+    return "bg-yellow-50 dark:bg-yellow-950/20 hover:bg-yellow-100 dark:hover:bg-yellow-950/30";
+  }
+
+  return "";
+}
+
 function ActionRequiredContent() {
   const {
     assignedItems,
@@ -279,7 +293,10 @@ function ActionRequiredContent() {
           </TableHeader>
           <TableBody>
             {items.map((item: ActionItem) => (
-              <TableRow key={item.id}>
+              <TableRow
+                key={item.id}
+                className={type === "stale" ? getStaleRowClassName(item.daysOld) : ""}
+              >
                 <TableCell>
                   <div className="flex items-center gap-2">
                     <div className="min-w-0 flex-1">

src/components/quick-wins/QuickWinsTable.tsx

@@ -1,6 +1,6 @@
 "use client";
 
-import { useState, useMemo } from "react";
+import { useState, useMemo, useCallback } from "react";
 import {
   useReactTable,
   getCoreRowModel,
@@ -45,6 +45,7 @@ import {
 
 import { createColumns } from "./columns";
 import type { GitHubIssue } from "@/types/quickWins";
+import { useKanbanStore } from "@/stores/kanban";
 
 interface QuickWinsTableProps {
   data: GitHubIssue[];
@@ -70,7 +71,21 @@ export function QuickWinsTable({
 
   const [languageFilter, setLanguageFilter] = useState<string>("all");
 
-  const columns = useMemo(() => createColumns(), []);
+  const addTask = useKanbanStore((state) => state.addTask);
+
+  const handleAddToKanban = useCallback((issue: GitHubIssue) => {
+    addTask({
+      title: issue.title,
+      description: `From ${issue.repository}`,
+      type: "github-issue",
+      priority: issue.priority,
+      githubUrl: issue.url,
+      labels: issue.labels.map((label) => label.name),
+      columnId: "todo",
+    });
+  }, [addTask]);
+
+  const columns = useMemo(() => createColumns({ onAddToKanban: handleAddToKanban }), [handleAddToKanban]);
 
   const filteredData = useMemo(() => {
     let filtered = data;

src/components/quick-wins/columns.tsx

@@ -2,10 +2,14 @@ import { ColumnDef } from "@tanstack/react-table";
 import { Badge } from "@/components/ui/badge";
 import { Button } from "@/components/ui/button";
 import { Avatar, AvatarFallback, AvatarImage } from "@/components/ui/avatar";
-import { ExternalLink, Calendar } from "lucide-react";
+import { ExternalLink, Calendar, ListPlus } from "lucide-react";
 import type { GitHubIssue } from "@/types/quickWins";
 
-export const createColumns = (): ColumnDef<GitHubIssue>[] => [
+interface CreateColumnsOptions {
+  onAddToKanban?: (issue: GitHubIssue) => void;
+}
+
+export const createColumns = (options?: CreateColumnsOptions): ColumnDef<GitHubIssue>[] => [
   {
     accessorKey: "title",
     header: ({ column }) => (
@@ -157,4 +161,24 @@ export const createColumns = (): ColumnDef<GitHubIssue>[] => [
     },
     enableSorting: false,
   },
+  {
+    id: "actions",
+    header: "Actions",
+    cell: ({ row }) => {
+      const issue = row.original;
+      return (
+        <Button
+          size="sm"
+          variant="outline"
+          onClick={() => options?.onAddToKanban?.(issue)}
+          disabled={!options?.onAddToKanban}
+          className="gap-1"
+        >
+          <ListPlus className="w-4 h-4" />
+          Add to Kanban
+        </Button>
+      );
+    },
+    enableSorting: false,
+  },
 ];

src/lib/api/github-graphql-client.ts

@@ -148,6 +148,8 @@ interface PullRequest {
   comments: {
     totalCount: number;
   };
+  additions?: number;
+  deletions?: number;
   __typename?: string;
 }
 
@@ -163,6 +165,8 @@ interface PullRequestWithReviews extends PullRequest {
 
 interface StalePullRequest extends Omit<PullRequest, "assignees"> {
   reviewDecision: string | null;
+  additions?: number;
+  deletions?: number;
 }
 
 class GitHubGraphQLClient {
@@ -521,7 +525,7 @@ class GitHubGraphQLClient {
             }
           }
         }
-        # Assigned Pull Requests 
+        # Assigned Pull Requests
         pullRequests(states: OPEN, first: 50) {
           nodes {
             __typename
@@ -552,6 +556,8 @@ class GitHubGraphQLClient {
             comments {
               totalCount
             }
+            additions
+            deletions
           }
         }
       }
@@ -587,6 +593,8 @@ class GitHubGraphQLClient {
               totalCount
             }
             reviewDecision
+            additions
+            deletions
           }
         }
       }
@@ -622,6 +630,8 @@ class GitHubGraphQLClient {
               totalCount
             }
             reviewDecision
+            additions
+            deletions
           }
         }
       }
@@ -690,6 +700,8 @@ class GitHubGraphQLClient {
                 }
               }
             }
+            additions
+            deletions
           }
         }
       }
@@ -733,6 +745,8 @@ class GitHubGraphQLClient {
                 }
               }
             }
+            additions
+            deletions
           }
         }
       }
@@ -857,6 +871,11 @@ class GitHubGraphQLClient {
     const labelNames = labels.map((l) => l.name);
 
     const isPR = this.isPullRequest(item);
+    const commentCount = item.comments?.totalCount || 0;
+
+    const prSize = isPR && "additions" in item && "deletions" in item
+      ? { additions: item.additions, deletions: item.deletions }
+      : undefined;
 
     return {
       id: item.id,
@@ -869,11 +888,17 @@ class GitHubGraphQLClient {
         avatarUrl: item.author?.avatarUrl || "",
       },
       labels,
-      priority: this.calculateActionPriority(labelNames, daysOld, mentionType),
+      priority: this.calculateActionPriority(
+        labelNames,
+        daysOld,
+        mentionType,
+        commentCount,
+        prSize
+      ),
       daysOld,
       createdAt: item.createdAt,
       updatedAt: item.updatedAt,
-      comments: item.comments?.totalCount || 0,
+      comments: commentCount,
       stars: item.repository.stargazerCount || 0,
       ...(mentionType && { mentionType }),
     };
@@ -882,10 +907,36 @@ class GitHubGraphQLClient {
   private calculateActionPriority(
     labels: string[],
     daysOld: number,
-    mentionType?: "mention" | "review_request" | "comment"
+    mentionType?: "mention" | "review_request" | "comment",
+    commentCount = 0,
+    prSize?: { additions?: number; deletions?: number }
   ): "urgent" | "high" | "medium" | "low" {
-    if (mentionType === "review_request") {
-      return "urgent";
+    let score = 0;
+
+    const statusScores = {
+      review_request: 50,
+      mention: 30,
+      comment: 25,
+    };
+    if (mentionType && statusScores[mentionType]) {
+      score += statusScores[mentionType];
+    } else {
+      score += 20;
+    }
+
+    score += Math.min(commentCount * 2, 30);
+
+    if (prSize && (prSize.additions || prSize.deletions)) {
+      const totalChanges = (prSize.additions || 0) + (prSize.deletions || 0);
+      if (totalChanges > 1000) {
+        score += 20;
+      } else if (totalChanges > 500) {
+        score += 15;
+      } else if (totalChanges > 100) {
+        score += 10;
+      } else {
+        score += 5;
+      }
     }
 
     const lowerLabels = labels.map((l) => l.toLowerCase());
@@ -896,20 +947,33 @@ class GitHubGraphQLClient {
           l.includes("critical") || l.includes("urgent") || l.includes("p0")
       )
     ) {
-      return "urgent";
-    }
-    if (
+      score += 40;
+    } else if (
       lowerLabels.some(
         (l) => l.includes("high") || l.includes("p1") || l.includes("bug")
       )
     ) {
-      return "high";
+      score += 25;
+    } else if (
+      lowerLabels.some(
+        (l) =>
+          l.includes("low") || l.includes("p3") || l.includes("enhancement")
+      )
+    ) {
+      score -= 10;
     }
 
-    if (daysOld > 14) return "urgent";
-    if (daysOld > 7) return "high";
-    if (daysOld > 3) return "medium";
+    if (daysOld > 14) {
+      score *= 1.5;
+    } else if (daysOld > 7) {
+      score *= 1.3;
+    } else if (daysOld > 3) {
+      score *= 1.1;
+    }
 
+    if (score >= 100) return "urgent";
+    if (score >= 70) return "high";
+    if (score >= 40) return "medium";
     return "low";
   }
 }