bot: update

Signed-off-by: Akshat Kumar <akshat230405@gmail.com>

Author: Akshat8510

.github/scripts/check_advanced_requirement.sh

@@ -12,16 +12,16 @@ if [[ "$PERMISSION" =~ ^(admin|write|triage)$ ]]; then
 fi
 
 # 2. Get counts of completed issues
-#  Use -f q= to handle URL encoding for labels with spaces automatically
+# Use -f q= for native URL encoding
 GFI_QUERY="repo:$REPO is:issue is:closed assignee:$ASSIGNEE -reason:\"not planned\" label:\"good first issue\""
 INT_QUERY="repo:$REPO is:issue is:closed assignee:$ASSIGNEE -reason:\"not planned\" label:\"intermediate\""
 
-GFI_COUNT=$(gh api "search/issues" -f q="$GFI_QUERY" --jq '.total_count')
-INT_COUNT=$(gh api "search/issues" -f q="$INT_QUERY" --jq '.total_count')
+GFI_COUNT=$(gh api "search/issues" -f q="$GFI_QUERY" --jq '.total_count' || echo "0")
+INT_COUNT=$(gh api "search/issues" -f q="$INT_QUERY" --jq '.total_count' || echo "0")
 
-# Ensure counts are numeric (default to 0)
-GFI_COUNT=${GFI_COUNT:-0}
-INT_COUNT=${INT_COUNT:-0}
+# Robust numeric validation
+if ! [[ "$GFI_COUNT" =~ ^[0-9]+$ ]]; then GFI_COUNT=0; fi
+if ! [[ "$INT_COUNT" =~ ^[0-9]+$ ]]; then INT_COUNT=0; fi
 
 echo "Stats for @$ASSIGNEE: GFI: $GFI_COUNT, Intermediate: $INT_COUNT"
 
@@ -35,7 +35,7 @@ else
   # Quote variables to prevent word-splitting
   gh issue edit "$ISSUE_NUMBER" --repo "$REPO" --remove-assignee "$ASSIGNEE"
 
-  # 4. Message
+  # 4. Message Construction
   MSG="Hi @$ASSIGNEE, I cannot assign you to this issue yet.
 
 **Why?**
@@ -45,9 +45,9 @@ Advanced issues involve high-risk changes to the core codebase. They require sig
 - Complete at least **1** 'good first issue' (You have: **$GFI_COUNT**)
 - Complete at least **1** 'intermediate' issue (You have: **$INT_COUNT**)
 
-Please feel free to pick up an [intermediate issue](https://github.com/hiero-ledger/hiero-sdk-python/labels/intermediate) to get started! We look forward to your contributions."
+Please feel free to pick up an [intermediate issue](https://github.com/$REPO/labels/intermediate) to get started! We look forward to your contributions."
 
-  #  Use the variable safely without printf format vulnerabilities
+  # Post the comment safely without printf formatting risks
   gh issue comment "$ISSUE_NUMBER" --repo "$REPO" --body "$MSG"
 
   exit 1

.github/workflows/bot-advanced-check.yml

@@ -2,15 +2,20 @@ name: PythonBot - Advanced Requirement Check
 
 on:
   issues:
-    types: [assigned]
+    types: [assigned, labeled] # Added 'labeled' to catch label changes after assignment
 
 # Explicitly minimal permissions
 permissions:
   issues: write
 
+# Prevent duplicate runners/comments on the same issue
+concurrency:
+  group: ${{ github.workflow }}-${{ github.event.issue.number }}
+  cancel-in-progress: true
+
 jobs:
   check-advanced-qualification:
-    # Most efficient: Skip the runner entirely if the issue isn't 'advanced'
+    # Skip the runner entirely if the issue isn't 'advanced'
     if: contains(github.event.issue.labels.*.name, 'advanced')
     runs-on: ubuntu-latest
     steps:
@@ -22,9 +27,16 @@ jobs:
       - name: Verify User Qualification
         env:
           GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-          ASSIGNEE: ${{ github.event.assignee.login }}
+          # Handle cases where triggered by assignment or by adding a label to an assigned issue
+          ASSIGNEE: ${{ github.event.assignee.login || github.event.issue.assignee.login }}
           ISSUE_NUMBER: ${{ github.event.issue.number }}
           REPO: ${{ github.repository }}
         run: |
+          # If no one is assigned, exit early
+          if [ -z "$ASSIGNEE" ]; then
+            echo "No assignee to check. Skipping."
+            exit 0
+          fi
+          
           chmod +x .github/scripts/check_advanced_requirement.sh
           ./.github/scripts/check_advanced_requirement.sh