refactor(polylang): remove redundant resolvers, add archive link resolver, harden for non-polylang

- Remove ResolveHomeUrl (dead code; home_url already handled by Polylang core)
- Remove ResolvePageTreeMenuPageIds (option_page_on_front/page_for_posts
  handled natively by Polylang; cache key duplicated ResolveNavigationCacheKey)
- Remove ResolveCurrentPostId (primary branch was a no-op — option_page_for_<pt>
  is already translated by ResolveTranslatedPostTypeLink before the helper reads it)
- Unregister removed resolvers from App.php
- Add ResolveTranslatedPostTypeArchiveLink to translate post_type_archive_link
  when the wp-page-for-posttype plugin bakes the default-language slug into
  the registered post type rewrite (with full unit tests)
- Early-return in ResolveTranslatedPostTypeLink::registerPostTypeHooks when
  Polylang is not active to avoid iterating all public post types on
  non-Polylang sites
- Skip registering the pre_get_posts action in ResolveFontAttachmentQueries
  when Polylang is not active so non-Polylang sites pay no per-query cost

Agent-Logs-Url: https://github.com/helsingborg-stad/Municipio/sessions/16905c3e-4839-4e00-bf43-e1cf0cd39e72

Co-authored-by: sebastianthulin <797129+sebastianthulin@users.noreply.github.com>

Author: Copilot

library/App.php

@@ -510,12 +510,11 @@ private function setUpPolylangIntegration(): void
 
         $resolvers = [
             new \Municipio\Integrations\Polylang\ResolveLanguageMenuItems($this->wpService),
-            new \Municipio\Integrations\Polylang\ResolvePageTreeMenuPageIds($this->wpService),
-            new \Municipio\Integrations\Polylang\ResolveCurrentPostId($this->wpService),
             new \Municipio\Integrations\Polylang\ResolvePageTreeTranslatedChildren($this->wpService),
             new \Municipio\Integrations\Polylang\ResolveNavigationItemsLanguage($this->wpService),
             new \Municipio\Integrations\Polylang\ResolveTranslatedPageLink($this->wpService),
             new \Municipio\Integrations\Polylang\ResolveTranslatedPostTypeLink($this->wpService),
+            new \Municipio\Integrations\Polylang\ResolveTranslatedPostTypeArchiveLink($this->wpService),
             new \Municipio\Integrations\Polylang\ResolveNavigationCacheKey($this->wpService),
             new \Municipio\Integrations\Polylang\ResolveNavigationFetchUrlLanguage($this->wpService),
             new \Municipio\Integrations\Polylang\ResolvePdfNotFoundUrl($this->wpService),

library/Integrations/Polylang/ResolveCurrentPostId.php

@@ -31,6 +31,12 @@ public function __construct(
      */
     public function addHooks(): void
     {
+        // Skip registering the pre_get_posts listener entirely on non-Polylang
+        // sites so non-Polylang requests pay no per-query cost.
+        if (!$this->isPolylangActive()) {
+            return;
+        }
+
         $this->wpService->addAction('pre_get_posts', [$this, 'makeFontAttachmentQueryLanguageAgnostic'], 1);
     }
 

library/Integrations/Polylang/ResolveCurrentPostIdTest.php

@@ -18,21 +18,41 @@ public function testCanBeInstantiated(): void
         static::assertInstanceOf(ResolveFontAttachmentQueries::class, $this->getSut());
     }
 
-    #[TestDox('addHooks() registers the pre_get_posts action')]
+    #[TestDox('addHooks() registers the pre_get_posts action when Polylang is active')]
     public function testAddHooksRegistersPreGetPostsAction(): void
     {
         $wpService = new FakeWpService([
             'addAction' => true,
         ]);
 
-        $sut = new ResolveFontAttachmentQueries($wpService);
+        $sut = new ResolveFontAttachmentQueries(
+            $wpService,
+            static fn (): bool => true
+        );
 
         $sut->addHooks();
 
         static::assertSame('pre_get_posts', $wpService->methodCalls['addAction'][0][0]);
         static::assertSame(1, $wpService->methodCalls['addAction'][0][2]);
     }
 
+    #[TestDox('addHooks() does not register the pre_get_posts action when Polylang is unavailable')]
+    public function testAddHooksSkipsPreGetPostsActionWhenPolylangInactive(): void
+    {
+        $wpService = new FakeWpService([
+            'addAction' => true,
+        ]);
+
+        $sut = new ResolveFontAttachmentQueries(
+            $wpService,
+            static fn (): bool => false
+        );
+
+        $sut->addHooks();
+
+        static::assertArrayNotHasKey('addAction', $wpService->methodCalls);
+    }
+
     #[TestDox('makeFontAttachmentQueryLanguageAgnostic() updates font attachment queries with array mime types')]
     public function testMakeFontAttachmentQueryLanguageAgnosticUpdatesArrayMimeTypeQuery(): void
     {