Skip placeholder emails from example domains in review checks

Filters out example.com, example.org, example.net, test.com, and
localhost when extracting support emails from READMEs to avoid false
positives from code snippets and templates.

Author: shanerbaner82

app/Jobs/ReviewPluginRepository.php

@@ -202,8 +202,16 @@ protected function checkDirectoryHasFiles(array $tree, string $prefix): bool
 
     protected function extractEmail(string $content): ?string
     {
-        if (preg_match('/[a-zA-Z0-9._%+\-]+@[a-zA-Z0-9.\-]+\.[a-zA-Z]{2,}/', $content, $matches)) {
-            return $matches[0];
+        $excludedDomains = ['example.com', 'example.org', 'example.net', 'test.com', 'localhost'];
+
+        if (preg_match_all('/[a-zA-Z0-9._%+\-]+@[a-zA-Z0-9.\-]+\.[a-zA-Z]{2,}/', $content, $matches)) {
+            foreach ($matches[0] as $email) {
+                $domain = strtolower(substr($email, strpos($email, '@') + 1));
+
+                if (! in_array($domain, $excludedDomains, true)) {
+                    return $email;
+                }
+            }
         }
 
         return null;

tests/Feature/Jobs/ReviewPluginRepositoryTest.php

@@ -97,12 +97,12 @@ public function it_extracts_email_from_readme(): void
             'repository_url' => 'https://github.com/acme/email-plugin',
         ]);
 
-        Http::fake($this->fakeGitHub('acme/email-plugin', readme: "# Plugin\n\nFor support contact help@example.com"));
+        Http::fake($this->fakeGitHub('acme/email-plugin', readme: "# Plugin\n\nFor support contact help@acmeplugins.com"));
 
         $checks = (new ReviewPluginRepository($plugin))->handle();
 
         $this->assertTrue($checks['has_support_email']);
-        $this->assertEquals('help@example.com', $checks['support_email']);
+        $this->assertEquals('help@acmeplugins.com', $checks['support_email']);
     }
 
     /** @test */
@@ -120,6 +120,21 @@ public function it_reports_missing_email_when_readme_has_none(): void
         $this->assertNull($checks['support_email']);
     }
 
+    /** @test */
+    public function it_skips_placeholder_emails_from_example_domains(): void
+    {
+        $plugin = Plugin::factory()->create([
+            'repository_url' => 'https://github.com/acme/placeholder-plugin',
+        ]);
+
+        Http::fake($this->fakeGitHub('acme/placeholder-plugin', readme: "# Plugin\n\nEmail: john@example.com\nReal support: support@realdomain.io"));
+
+        $checks = (new ReviewPluginRepository($plugin))->handle();
+
+        $this->assertTrue($checks['has_support_email']);
+        $this->assertEquals('support@realdomain.io', $checks['support_email']);
+    }
+
     /** @test */
     public function it_detects_nativephp_mobile_dependency(): void
     {
@@ -162,7 +177,7 @@ public function it_stores_results_in_review_checks_and_stamps_reviewed_at(): voi
 
         Http::fake($this->fakeGitHub('acme/store-plugin',
             tree: [['path' => 'resources/ios/Bridge.swift', 'type' => 'blob']],
-            readme: 'Support: dev@example.com',
+            readme: 'Support: dev@storeplugin.io',
             composerRequire: ['nativephp/mobile' => '^3.0.0'],
         ));
 
@@ -178,7 +193,7 @@ public function it_stores_results_in_review_checks_and_stamps_reviewed_at(): voi
         $this->assertTrue($plugin->review_checks['supports_ios']);
         $this->assertFalse($plugin->review_checks['supports_android']);
         $this->assertTrue($plugin->review_checks['has_support_email']);
-        $this->assertEquals('dev@example.com', $plugin->review_checks['support_email']);
+        $this->assertEquals('dev@storeplugin.io', $plugin->review_checks['support_email']);
         $this->assertTrue($plugin->review_checks['requires_mobile_sdk']);
     }