Add gemini-triaged label for issue-labeler and only run the labeler if no area: label is present on the issue

github-actions/issue-labeling/BUILD.bazel

@@ -20,6 +20,7 @@ ts_project(
         ":node_modules/@octokit/rest",
         ":node_modules/@types/node",
         "//github-actions:utils",
+        "//ng-dev/pr/common/labels",
     ],
 )
 

github-actions/issue-labeling/lib/issue-labeling.ts

@@ -4,6 +4,7 @@ import {GoogleGenAI} from '@google/genai';
 import {Octokit} from '@octokit/rest';
 import {ANGULAR_ROBOT, getAuthTokenFor, revokeActiveInstallationToken} from '../../utils.js';
 import {components} from '@octokit/openapi-types';
+import {miscLabels} from '../../../ng-dev/pr/common/labels/index.js';
 
 export class IssueLabeling {
   static run = async () => {
@@ -38,8 +39,17 @@ export class IssueLabeling {
     // Initialize labels and issue data
     await this.initialize();
 
-    const ai = this.getGenerativeAI();
+    // Determine if the issue already has an area label, if it does we can exit early.
+    if (
+      this.issueData?.labels.some((label) =>
+        (typeof label === 'string' ? label : label.name)?.startsWith('area: '),
+      )
+    ) {
+      this.coreService.info('Issue already has an area label. Skipping.');
+      return;
+    }
 
+    const ai = this.getGenerativeAI();
     const prompt = `
 You are a helper for an open source repository.
 Your task is to allow the user to categorize the issue with an "area: " label.
@@ -71,6 +81,7 @@ If no area label applies, respond with "none".
 
       if (this.repoAreaLabels.has(text)) {
         await this.addLabel(text);
+        await this.addLabel(miscLabels.GEMINI_TRIAGED.name);
       } else {
         this.coreService.info(
           `Generated label "${text}" is not in the list of valid area labels or is "ambiguous"/"none".`,

github-actions/issue-labeling/lib/main.spec.ts

@@ -33,6 +33,7 @@ describe('IssueLabeling', () => {
         data: {
           title: 'Tough Issue',
           body: 'Complex Body',
+          labels: [],
         },
       }),
     );
@@ -117,4 +118,19 @@ describe('IssueLabeling', () => {
     expect(issueLabeling).toBeDefined();
     expect(mockCore.getInput).not.toHaveBeenCalled(); // until run is called
   });
+
+  it('should skip labeling when issue already has an area label', async () => {
+    mockGit.issues.get.and.resolveTo({
+      data: {
+        title: 'Tough Issue',
+        body: 'Complex Body',
+        labels: [{name: 'area: core'}],
+      },
+    });
+
+    await issueLabeling.run();
+
+    expect(mockGit.issues.addLabels).not.toHaveBeenCalled();
+    expect(mockCore.info).toHaveBeenCalledWith('Issue already has an area label. Skipping.');
+  });
 });

ng-dev/pr/common/labels/index.ts

@@ -6,6 +6,7 @@ import {priorityLabels} from './priority.js';
 import {featureLabels} from './feature.js';
 import {requiresLabels} from './requires.js';
 import {Label, LabelParams} from './base.js';
+import {miscLabels} from './misc.js';
 
 export const allLabels = {
   ...managedLabels,
@@ -15,10 +16,19 @@ export const allLabels = {
   ...priorityLabels,
   ...featureLabels,
   ...requiresLabels,
+  ...miscLabels,
 };
 
 // Ensures that all labels in `allLabels` properly implement `Label`.
 const _typeCheckEnforceAllLabels: Record<PropertyKey, Label<LabelParams>> = allLabels;
 
-export {managedLabels, actionLabels, mergeLabels, targetLabels, priorityLabels, requiresLabels};
+export {
+  managedLabels,
+  actionLabels,
+  mergeLabels,
+  targetLabels,
+  priorityLabels,
+  requiresLabels,
+  miscLabels,
+};
 export {Label};

ng-dev/pr/common/labels/misc.ts

@@ -21,4 +21,8 @@ export const miscLabels = createTypedObject(MiscLabel)({
     description:
       'Label noting that a pull request will automatically be managed and rebased by renovate',
   },
+  GEMINI_TRIAGED: {
+    name: 'gemini-triaged',
+    description: 'Label noting that an issue has been triaged by gemini',
+  },
 });