chore: improve eligibility logging

Signed-off-by: exploreriii <133720349+exploreriii@users.noreply.github.com>

Author: exploreriii

.github/scripts/lib/eligibility/has-eligibility-01-gfi.js

@@ -1,9 +1,41 @@
+/**
+ * Determines whether a contributor is eligible to be assigned
+ * a Good First Issue (GFI).
+ *
+ * ELIGIBILITY RULES:
+ * - Repository team members (triage / write / maintain / admin)
+ *   always bypass GFI eligibility checks.
+ * - Spam-listed contributors may have at most
+ *   `MAX_OPEN_ISSUES_SPAM_LIST` open GFI assignments.
+ * - All other contributors may have at most
+ *   `MAX_OPEN_ISSUES_NORMAL` open GFI assignments.
+ *
+ * IMPORTANT:
+ * - This helper is policy-only.
+ * - It does NOT assign issues.
+ * - It does NOT post comments.
+ * - Callers must route rejections via `rejectionRouter`.
+ *
+ * @param {Object} params
+ * @param {import('@actions/github').GitHub} params.github
+ * @param {string} params.owner
+ * @param {string} params.repo
+ * @param {string} params.username
+ * @returns {Promise<{
+ *   eligible: boolean,
+ *   reason?: string,
+ *   context?: Object
+ * }>}
+ */
 const { isTeam } = require('../team/has-team');
 const { isOnSpamList } = require('../counts/is-on-spam-list');
-const { countOpenAssignedIssues } = require('../counts/count-opened-assigned-issues');
+const { countOpenAssignedIssues } =
+    require('../counts/count-opened-assigned-issues');
 const REJECTION_REASONS = require('./rejection-reasons');
 
-// Eligibility is configurable to flex with project needs
+// ─────────────────────────────────────────────
+// Policy configuration
+// ─────────────────────────────────────────────
 const MAX_OPEN_ISSUES_NORMAL = 2;
 const MAX_OPEN_ISSUES_SPAM_LIST = 1;
 
@@ -13,10 +45,26 @@ const hasGfiEligibility = async ({
     repo,
     username,
 }) => {
+    console.log('[has-gfi-eligibility] Start', {
+        owner,
+        repo,
+        username,
+    });
+
+    // ─────────────────────────────────────────────
+    // Team members bypass all GFI restrictions
+    // ─────────────────────────────────────────────
     if (await isTeam({ github, owner, repo, username })) {
+        console.log('[has-gfi-eligibility] Bypass: team member', {
+            username,
+        });
+
         return { eligible: true };
     }
 
+    // ─────────────────────────────────────────────
+    // Spam list evaluation
+    // ─────────────────────────────────────────────
     const isSpamListed = await isOnSpamList({
         github,
         owner,
@@ -28,14 +76,28 @@ const hasGfiEligibility = async ({
         ? MAX_OPEN_ISSUES_SPAM_LIST
         : MAX_OPEN_ISSUES_NORMAL;
 
+    // ─────────────────────────────────────────────
+    // Capacity check
+    // ─────────────────────────────────────────────
     const openAssignedCount = await countOpenAssignedIssues({
         github,
         owner,
         repo,
         username,
     });
 
+    console.log('[has-gfi-eligibility] Capacity evaluation', {
+        username,
+        isSpamListed,
+        openAssignedCount,
+        maxAllowed,
+    });
+
     if (openAssignedCount >= maxAllowed) {
+        console.log('[has-gfi-eligibility] Rejected: capacity exceeded', {
+            username,
+        });
+
         return {
             eligible: false,
             reason: REJECTION_REASONS.CAPACITY,
@@ -47,6 +109,13 @@ const hasGfiEligibility = async ({
         };
     }
 
+    // ─────────────────────────────────────────────
+    // Eligible
+    // ─────────────────────────────────────────────
+    console.log('[has-gfi-eligibility] Eligible', {
+        username,
+    });
+
     return { eligible: true };
 };
 

.github/scripts/lib/eligibility/has-eligibility-02-beginner.js

@@ -1,13 +1,37 @@
+/**
+ * Determines whether a contributor is eligible to be assigned
+ * a Beginner issue.
+ *
+ * ELIGIBILITY RULES:
+ * - Team members bypass all checks
+ * - Spam-listed users are never eligible
+ * - Max open assignments enforced
+ * - Must have completed at least N Good First Issues
+ *
+ * @param {Object} params
+ * @param {import('@actions/github').GitHub} params.github
+ * @param {string} params.owner
+ * @param {string} params.repo
+ * @param {string} params.username
+ * @returns {Promise<{
+ *   eligible: boolean,
+ *   reason?: string,
+ *   context?: Object
+ * }>}
+ */
 const { isTeam } = require('../team/has-team');
 const { isOnSpamList } = require('../counts/is-on-spam-list');
-const { hasCompletedGfi } = require('../counts/has-completed-n-01-gfi');
-const { countOpenAssignedIssues } = require('../counts/count-opened-assigned-issues');
+const { hasCompletedGfi } =
+    require('../counts/has-completed-n-01-gfi');
+const { countOpenAssignedIssues } =
+    require('../counts/count-opened-assigned-issues');
 const REJECTION_REASONS = require('./rejection-reasons');
 
-// Configurable
+// ─────────────────────────────────────────────
+// Policy configuration
+// ─────────────────────────────────────────────
 const MAX_OPEN_ASSIGNED_ISSUES = 2;
 const REQUIRED_GFI_COUNT = 1;
-// Spam users are disabled for beginner issues
 
 const hasBeginnerEligibility = async ({
     github,
@@ -21,39 +45,73 @@ const hasBeginnerEligibility = async ({
         username,
     });
 
+    // ─────────────────────────────────────────────
     // Team members bypass everything
+    // ─────────────────────────────────────────────
     if (await isTeam({ github, owner, repo, username })) {
+        console.log('[has-beginner-eligibility] Bypass: team member', {
+            username,
+        });
+
         return { eligible: true };
     }
 
+    // ─────────────────────────────────────────────
     // Spam-listed users are never eligible
-    if (await isOnSpamList({ github, owner, repo, username })) {
+    // ─────────────────────────────────────────────
+    const isSpamListed = await isOnSpamList({
+        github,
+        owner,
+        repo,
+        username,
+    });
+
+    if (isSpamListed) {
+        console.log('[has-beginner-eligibility] Rejected: spam listed', {
+            username,
+        });
+
         return {
             eligible: false,
             reason: REJECTION_REASONS.SPAM,
         };
     }
 
+    // ─────────────────────────────────────────────
     // Capacity check
+    // ─────────────────────────────────────────────
     const openAssignedCount = await countOpenAssignedIssues({
         github,
         owner,
         repo,
         username,
     });
 
+    console.log('[has-beginner-eligibility] Capacity evaluation', {
+        username,
+        openAssignedCount,
+        maxAllowed: MAX_OPEN_ASSIGNED_ISSUES,
+    });
+
     if (openAssignedCount >= MAX_OPEN_ASSIGNED_ISSUES) {
+        console.log('[has-beginner-eligibility] Rejected: capacity exceeded', {
+            username,
+        });
+
         return {
             eligible: false,
             reason: REJECTION_REASONS.CAPACITY,
             context: {
                 openAssignedCount,
                 maxAllowed: MAX_OPEN_ASSIGNED_ISSUES,
+                isSpamListed: false,
             },
         };
     }
 
+    // ─────────────────────────────────────────────
     // GFI prerequisite
+    // ─────────────────────────────────────────────
     const hasRequiredGfi = await hasCompletedGfi({
         github,
         owner,
@@ -62,18 +120,34 @@ const hasBeginnerEligibility = async ({
         requiredCount: REQUIRED_GFI_COUNT,
     });
 
+    console.log('[has-beginner-eligibility] GFI prerequisite', {
+        username,
+        requiredGfiCount: REQUIRED_GFI_COUNT,
+        satisfied: hasRequiredGfi,
+    });
+
     if (!hasRequiredGfi) {
+        console.log('[has-beginner-eligibility] Rejected: missing GFI', {
+            username,
+        });
+
         return {
             eligible: false,
             reason: REJECTION_REASONS.MISSING_GFI,
             context: {
-                completedGfiCount: 0, // safe fallback
+                requiredGfiCount: REQUIRED_GFI_COUNT,
             },
         };
     }
 
+    // ─────────────────────────────────────────────
+    // Eligible
+    // ─────────────────────────────────────────────
+    console.log('[has-beginner-eligibility] Eligible', {
+        username,
+    });
+
     return { eligible: true };
 };
 
 module.exports = { hasBeginnerEligibility };
-