Fix/issue bot final (#171)

Author: Heyy-Himanshuu

.github/workflows/issue-assignment.yml

@@ -9,11 +9,11 @@ permissions:
   contents: read
 
 jobs:
-  assign-issue:
-    # Only run on issue comments (not PR comments) and when comment contains @bot assign
+  claim-issue:
+    # Only run on issue comments (not PR comments) and when comment contains @bot claim
     if: |
       !github.event.issue.pull_request &&
-      contains(github.event.comment.body, '@bot assign')
+      contains(github.event.comment.body, '@bot claim')
     runs-on: ubuntu-latest
     # Prevent race conditions - only one job per issue at a time
     concurrency:
@@ -30,12 +30,12 @@ jobs:
             const issue = context.payload.issue;
             const repo = context.repo;
 
-            console.log(`Processing @bot assign from ${commenter} on issue #${issue.number}`);
+            console.log(`Processing @bot claim from ${commenter} on issue #${issue.number}`);
 
             // Validate exact command match
             const commentBody = context.payload.comment.body.trim();
-            if (commentBody !== '@bot assign') {
-              console.log(`Comment "${commentBody}" is not an exact match for "@bot assign", skipping`);
+            if (commentBody !== '@bot claim') {
+              console.log(`Comment "${commentBody}" is not an exact match for "@bot claim", skipping`);
               return;
             }
 
@@ -47,10 +47,9 @@ jobs:
 
             // Check if issue is already assigned
             if (currentIssue.assignees && currentIssue.assignees.length > 0) {
-              const currentAssignee = currentIssue.assignees[0].login;
+              const isAlreadyAssignedToCommenter = currentIssue.assignees.some(a => a.login === commenter);
 
-              // Check if already assigned to the same commenter
-              if (currentAssignee === commenter) {
+              if (isAlreadyAssignedToCommenter) {
                 console.log(`Issue already assigned to commenter ${commenter}`);
                 await github.rest.issues.createComment({
                   ...repo,
@@ -60,28 +59,57 @@ jobs:
                 return;
               }
 
-              console.log(`Issue already assigned to ${currentAssignee}`);
+              // Assigned to someone else
+              const assigneesList = currentIssue.assignees.map(a => `@${a.login}`).join(', ');
+              console.log(`Issue already assigned to ${assigneesList}`);
               await github.rest.issues.createComment({
                 ...repo,
                 issue_number: issue.number,
-                body: `@${commenter} This issue cannot be claimed, as someone else is already working on it.`
+                body: `@${commenter} This issue cannot be claimed, as it is already assigned to ${assigneesList}.`
               });
               return;
             }
 
-            // Check if commenter has any other open assigned issues using Search API
-            const { data: searchResult } = await github.rest.search.issuesAndPullRequests({
-              q: `repo:${repo.owner}/${repo.repo} is:issue is:open assignee:${commenter}`
-            });
+            // Check if commenter has any other open assigned issues
+            // Using ONLY repo-scoped API (listForRepo) - no Search API dependency
+            // This approach works for all users including first-time contributors
+            let otherAssignedIssues = [];
+            let page = 1;
+            let hasMorePages = true;
 
-            // Filter out the current issue and PRs
-            const otherAssignedIssues = searchResult.items.filter(i =>
-              i.number !== issue.number && !i.pull_request
-            );
+            console.log(`Checking for other assigned issues for ${commenter}...`);
 
-            if (otherAssignedIssues.length > 0) {
-              console.log(`User ${commenter} already has ${otherAssignedIssues.length} assigned issues`);
+            while (hasMorePages) {
+              const { data: issues } = await github.rest.issues.listForRepo({
+                ...repo,
+                state: 'open',
+                per_page: 100,
+                page: page
+              });
 
+              if (issues.length === 0) {
+                hasMorePages = false;
+              } else {
+                // Filter: exclude current issue, exclude PRs, find issues assigned to commenter
+                const assignedToUser = issues.filter(i =>
+                  i.number !== issue.number &&
+                  !i.pull_request &&
+                  i.assignees &&
+                  i.assignees.some(a => a.login === commenter)
+                );
+                otherAssignedIssues = otherAssignedIssues.concat(assignedToUser);
+                page++;
+
+                // Stop early if we found assigned issues or reached end of results
+                if (otherAssignedIssues.length > 0 || issues.length < 100) {
+                  hasMorePages = false;
+                }
+              }
+            }
+
+            console.log(`Found ${otherAssignedIssues.length} other assigned issues for ${commenter}`);
+
+            if (otherAssignedIssues.length > 0) {
               await github.rest.issues.createComment({
                 ...repo,
                 issue_number: issue.number,
@@ -129,19 +157,19 @@ jobs:
 
             console.log(`Successfully assigned issue #${issue.number} to ${commenter}`);
 
-  unclaim-issue:
-    # Only run on issue comments (not PR comments) and when comment contains @bot unclaim
+  drop-issue:
+    # Only run on issue comments (not PR comments) and when comment contains @bot drop
     if: |
       !github.event.issue.pull_request &&
-      contains(github.event.comment.body, '@bot unclaim')
+      contains(github.event.comment.body, '@bot drop')
     runs-on: ubuntu-latest
     # Prevent race conditions - only one job per issue at a time
     concurrency:
       group: issue-assign-${{ github.event.issue.number }}
       cancel-in-progress: false
 
     steps:
-      - name: Unclaim Issue
+      - name: Drop Issue
         uses: actions/github-script@v8
         with:
           github-token: ${{ secrets.GITHUB_TOKEN }}
@@ -150,12 +178,12 @@ jobs:
             const issue = context.payload.issue;
             const repo = context.repo;
 
-            console.log(`Processing @bot unclaim from ${commenter} on issue #${issue.number}`);
+            console.log(`Processing @bot drop from ${commenter} on issue #${issue.number}`);
 
             // Validate exact command match
             const commentBody = context.payload.comment.body.trim();
-            if (commentBody !== '@bot unclaim') {
-              console.log(`Comment "${commentBody}" is not an exact match for "@bot unclaim", skipping`);
+            if (commentBody !== '@bot drop') {
+              console.log(`Comment "${commentBody}" is not an exact match for "@bot drop", skipping`);
               return;
             }
 
@@ -221,7 +249,7 @@ jobs:
             await github.rest.issues.createComment({
               ...repo,
               issue_number: issue.number,
-              body: `@${commenter} You have been unassigned from this issue. It's now available for others to claim.`
+              body: `@${commenter} You have dropped this issue. It's now available for others to claim.`
             });
 
             console.log(`Successfully unassigned ${commenter} from issue #${issue.number}`);