[jsweep] Clean add_labels.cjs (#9005)

* jsweep: Clean add_labels.cjs

- Simplified item number determination with ternary expression
- Consolidated error handling for invalid/missing item numbers
- Removed unnecessary intermediate variables
- Used spread operator for context.repo
- Used shorthand property for contextType
- Created comprehensive test file with 13 test cases
- Reduced from 146 to 133 lines (13 lines saved)

* Fix add_labels.test.cjs test framework and empty labels behavior (#9006)

* Initial plan

* Fix add_labels.test.cjs to use vitest and fix empty labels test expectation

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

* Apply prettier formatting to add_labels.cjs (single-line ternary expressions)

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>

* Add changeset [skip-ci]

---------

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Copilot <198982749+Copilot@users.noreply.github.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Co-authored-by: Peli de Halleux <pelikhan@users.noreply.github.com>

Author: 3 people

.changeset/patch-clean-add-labels-jsweep.md

@@ -47,38 +47,20 @@ async function main(config = {}) {
 
     processedCount++;
 
-    const item = message;
-
     // Determine target issue/PR number
-    let itemNumber;
-    if (item.item_number !== undefined) {
-      itemNumber = parseInt(String(item.item_number), 10);
-      if (isNaN(itemNumber)) {
-        core.warning(`Invalid item number: ${item.item_number}`);
-        return {
-          success: false,
-          error: `Invalid item number: ${item.item_number}`,
-        };
-      }
-    } else {
-      // Use context issue or PR if available
-      const contextIssue = context.payload?.issue?.number;
-      const contextPR = context.payload?.pull_request?.number;
-      itemNumber = contextIssue || contextPR;
-
-      if (!itemNumber) {
-        core.warning("No item_number provided and not in issue/PR context");
-        return {
-          success: false,
-          error: "No issue/PR number available",
-        };
-      }
+    const itemNumber = message.item_number !== undefined ? parseInt(String(message.item_number), 10) : context.payload?.issue?.number || context.payload?.pull_request?.number;
+
+    if (!itemNumber || isNaN(itemNumber)) {
+      const errorMsg = message.item_number !== undefined ? `Invalid item number: ${message.item_number}` : "No item_number provided and not in issue/PR context";
+      core.warning(errorMsg);
+      return {
+        success: false,
+        error: message.item_number !== undefined ? `Invalid item number: ${message.item_number}` : "No issue/PR number available",
+      };
     }
 
-    // Determine context type
     const contextType = context.payload?.pull_request ? "pull request" : "issue";
-
-    const requestedLabels = item.labels ?? [];
+    const requestedLabels = message.labels ?? [];
     core.info(`Requested labels: ${JSON.stringify(requestedLabels)}`);
 
     // Use validation helper to sanitize and validate labels
@@ -118,8 +100,7 @@ async function main(config = {}) {
 
     try {
       await github.rest.issues.addLabels({
-        owner: context.repo.owner,
-        repo: context.repo.repo,
+        ...context.repo,
         issue_number: itemNumber,
         labels: uniqueLabels,
       });
@@ -130,7 +111,7 @@ async function main(config = {}) {
         success: true,
         number: itemNumber,
         labelsAdded: uniqueLabels,
-        contextType: contextType,
+        contextType,
       };
     } catch (error) {
       const errorMessage = getErrorMessage(error);