Add ignored_labels config

config/services.yaml

@@ -74,6 +74,8 @@ parameters:
                 - 'App\Subscriber\RewriteUnwantedPhrasesSubscriber'
                 - 'App\Subscriber\AllowEditFromMaintainerSubscriber'
             secret: '%env(SYMFONY_AI_SECRET)%'
+            ignored_labels:
+                - 'Run examples'
 
         symfony/ux:
             subscribers:

src/Model/Repository.php

@@ -1,4 +1,4 @@
 <?php
 
 namespace App\Model;
 
@@ -10,12 +10,14 @@
 class Repository
 {
     /**
-     * @param string|null $secret the webhook secret used by GitHub
+     * @param string|null   $secret        the webhook secret used by GitHub
+     * @param list<string>  $ignoredLabels labels that should not be auto-applied from PR/issue titles
      */
     public function __construct(
         private readonly string $vendor,
         private readonly string $name,
         private readonly ?string $secret = null,
+        private readonly array $ignoredLabels = [],
     ) {
     }
 
@@ -44,6 +46,14 @@
         );
     }
 
+    /**
+     * @return list<string>
+     */
+    public function getIgnoredLabels(): array
+    {
+        return $this->ignoredLabels;
+    }
+
     public function getFullName(): string
     {
         return sprintf('%s/%s', $this->getVendor(), $this->getName());

src/Service/LabelNameExtractor.php

@@ -54,6 +54,9 @@ public function extractLabels(string $title, Repository $repository): array
             }
         }
 
+        $ignoredLabels = array_map('strtolower', $repository->getIgnoredLabels());
+        $labels = array_values(array_filter($labels, static fn (string $label): bool => !\in_array(strtolower($label), $ignoredLabels, true)));
+
         $this->logger->debug('Searched for labels in title', ['title' => $title, 'labels' => \json_encode($labels, \JSON_THROW_ON_ERROR)]);
 
         return $labels;

src/Service/RepositoryProvider.php

@@ -15,7 +15,7 @@ class RepositoryProvider
     private array $repositories = [];
 
     /**
-     * @param array<string, array{secret?: string}> $repositories
+     * @param array<string, array{secret?: string, ignored_labels?: list<string>}> $repositories
      */
     public function __construct(array $repositories)
     {
@@ -29,7 +29,8 @@ public function __construct(array $repositories)
             $this->addRepository(new Repository(
                 $vendorName,
                 $repositoryName,
-                $repositoryData['secret'] ?? null
+                $repositoryData['secret'] ?? null,
+                $repositoryData['ignored_labels'] ?? [],
             ));
         }
     }

tests/Service/LabelNameExtractorTest.php

@@ -31,4 +31,28 @@ public static function provideLabels(): iterable
         yield [['Messenger', 'Mime'], '[Messenger] [Mime] Foobar'];
         yield [['Messenger', 'Mime'], '[Messenger] Foobar [Mime]'];
     }
+
+    public function testIgnoredLabelsAreFilteredOut(): void
+    {
+        $extractor = new LabelNameExtractor(new StaticLabelApi(), new NullLogger());
+        $repo = new Repository('carsonbot-playground', 'symfony', null, ['Messenger']);
+
+        $this->assertSame([], $extractor->extractLabels('[Messenger] Foobar', $repo));
+    }
+
+    public function testIgnoredLabelsAreCaseInsensitive(): void
+    {
+        $extractor = new LabelNameExtractor(new StaticLabelApi(), new NullLogger());
+        $repo = new Repository('carsonbot-playground', 'symfony', null, ['messenger']);
+
+        $this->assertSame([], $extractor->extractLabels('[Messenger] Foobar', $repo));
+    }
+
+    public function testIgnoredLabelsOnlyFilterConfiguredOnes(): void
+    {
+        $extractor = new LabelNameExtractor(new StaticLabelApi(), new NullLogger());
+        $repo = new Repository('carsonbot-playground', 'symfony', null, ['Messenger']);
+
+        $this->assertSame(['Mime'], $extractor->extractLabels('[Messenger] [Mime] Foobar', $repo));
+    }
 }

tests/Subscriber/AutoLabelFromContentSubscriberTest.php

@@ -179,4 +179,39 @@ public static function getPRTests(): array
 
         return $tests;
     }
+
+    public function testIgnoredLabelsAreNotAppliedFromPRTitle(): void
+    {
+        $labelsApi = $this->getMockBuilder(StaticLabelApi::class)
+            ->disableOriginalConstructor()
+            ->onlyMethods(['addIssueLabels'])
+            ->getMock();
+
+        $subscriber = new AutoLabelFromContentSubscriber($labelsApi, new LabelNameExtractor($labelsApi, new NullLogger()));
+        $repository = new Repository('weaverryan', 'symfony', null, ['Bug']);
+
+        $dispatcher = new EventDispatcher();
+        $dispatcher->addSubscriber($subscriber);
+
+        $labelsApi->expects($this->once())
+            ->method('addIssueLabels')
+            ->with(1234, ['Asset', 'BC Break'], $repository);
+
+        $event = new GitHubEvent([
+            'action' => 'opened',
+            'pull_request' => [
+                'number' => 1234,
+                'title' => '[Asset][bc Break][Bug] Some PR title',
+                'body' => '',
+                'draft' => false,
+            ],
+        ], $repository);
+
+        $dispatcher->dispatch($event, GitHubEvents::PULL_REQUEST);
+
+        $responseData = $event->getResponseData();
+
+        $this->assertSame(1234, $responseData['pull_request']);
+        $this->assertSame(['Asset', 'BC Break'], $responseData['pr_labels']);
+    }
 }