From 3028f847b0945c7102c84bc13db0329cb8fcaace Mon Sep 17 00:00:00 2001
From: Vyacheslav Egorov <vegorov@google.com>
Date: Wed, 30 Aug 2023 12:19:47 +0200
Subject: [PATCH] Filter issues by labels when listing if possible

* If `only-issue-labels` and `only-pr-labels` are not specified
  we limit the listing to `only-labels`.
* If `only-issue-labels` and `only-pr-labels` are specified
  but are the same, we limit the listing to these labels.
* If we don't need to make neither issues nor PRs as stale
  (`days-before-issue-stale` and `days-before-pr-stale` are
  negative) and stale label is the same for issues and PRs
  then we filter by this label.
---
 dist/index.js                   | 21 +++++++++++++++++++++
 src/classes/issues-processor.ts | 27 +++++++++++++++++++++++++++
 2 files changed, 48 insertions(+)

diff --git a/dist/index.js b/dist/index.js
index cfcb005d8..9f4614ee5 100644
--- a/dist/index.js
+++ b/dist/index.js
@@ -676,12 +676,33 @@ class IssuesProcessor {
     getIssues(page) {
         var _a;
         return __awaiter(this, void 0, void 0, function* () {
+            // Compute labels filter.
+            let labels = '';
+            // We can filter issues already when listing them if neither only-pr-labels
+            // nor only-issue-labels are specified or both are the same.
+            if (this.options.onlyPrLabels === '' &&
+                this.options.onlyIssueLabels === '') {
+                labels = this.options.onlyLabels;
+            }
+            else if (this.options.onlyPrLabels == this.options.onlyIssueLabels) {
+                labels = this.options.onlyIssueLabels;
+            }
+            // If we don't have to mark issues and pull requests stale and
+            // both the stale labels are the same then we can use it as a filter:
+            // because we only want to process stale issues.
+            if (!(0, should_mark_when_stale_1.shouldMarkWhenStale)(this._getDaysBeforeIssueStale()) &&
+                !(0, should_mark_when_stale_1.shouldMarkWhenStale)(this._getDaysBeforePrStale()) &&
+                this.options.stalePrLabel === this.options.staleIssueLabel &&
+                !labels.includes(this.options.staleIssueLabel)) {
+                labels += (labels !== '' ? ',' : '') + this.options.staleIssueLabel;
+            }
             try {
                 this.operations.consumeOperation();
                 const issueResult = yield this.client.rest.issues.listForRepo({
                     owner: github_1.context.repo.owner,
                     repo: github_1.context.repo.repo,
                     state: 'open',
+                    labels,
                     per_page: 100,
                     direction: this.options.ascending ? 'asc' : 'desc',
                     page
diff --git a/src/classes/issues-processor.ts b/src/classes/issues-processor.ts
index 31bbb99d6..200d4b497 100644
--- a/src/classes/issues-processor.ts
+++ b/src/classes/issues-processor.ts
@@ -562,12 +562,39 @@ export class IssuesProcessor {
 
   // grab issues from github in batches of 100
   async getIssues(page: number): Promise<Issue[]> {
+    // Compute labels filter.
+    let labels = '';
+
+    // We can filter issues already when listing them if neither only-pr-labels
+    // nor only-issue-labels are specified or both are the same.
+    if (
+      this.options.onlyPrLabels === '' &&
+      this.options.onlyIssueLabels === ''
+    ) {
+      labels = this.options.onlyLabels;
+    } else if (this.options.onlyPrLabels == this.options.onlyIssueLabels) {
+      labels = this.options.onlyIssueLabels;
+    }
+
+    // If we don't have to mark issues and pull requests stale and
+    // both the stale labels are the same then we can use it as a filter:
+    // because we only want to process stale issues.
+    if (
+      !shouldMarkWhenStale(this._getDaysBeforeIssueStale()) &&
+      !shouldMarkWhenStale(this._getDaysBeforePrStale()) &&
+      this.options.stalePrLabel === this.options.staleIssueLabel &&
+      !labels.includes(this.options.staleIssueLabel)
+    ) {
+      labels += (labels !== '' ? ',' : '') + this.options.staleIssueLabel;
+    }
+
     try {
       this.operations.consumeOperation();
       const issueResult = await this.client.rest.issues.listForRepo({
         owner: context.repo.owner,
         repo: context.repo.repo,
         state: 'open',
+        labels,
         per_page: 100,
         direction: this.options.ascending ? 'asc' : 'desc',
         page