Fix obsolete translation test assumptions

Author: Tsvetan Stoychev

checklist.md

@@ -149,8 +149,9 @@ Acceptance criteria:
 - [ ] Make CI regenerate the loaders and fail when the committed output differs.
 - [ ] Document the source and version of the bundled Boomerang asset.
 - [x] Add a repeatable command for generating `languages/basicrum.pot`.
-- [x] Add a repeatable command for compiling translation MO files.
-- [x] Make CI detect stale POT or MO files.
+- [x] Keep locale-specific PO and MO files out of the repository and release ZIP.
+- [x] Keep POT generation as a local maintenance command; the standalone
+  translation CI job was removed by product decision.
 
 Acceptance criteria:
 

docs/audits/checklist.md

@@ -95,8 +95,6 @@ operator-journey defect, P3 = minor/polish.
   limitation. (privacy 8 / BR-WP-14)
 - [ ] P3 Declare WooCommerce HPOS compatibility
   (`FeaturesUtil::declare_compatibility`, guarded). (UX 9)
-- [ ] P3 Bulgarian catalog: complete it or drop the po/mo pair (8 of 66
-  strings translated ships today). (UX 9)
 - [ ] P3 Optional hardening: reject protocol-relative/relative beacon URLs
   in `Validate.php` (BR-WP-06 was overturned to PASS - the bundle's
   `beacon_url_force_https` already prevents plaintext beacons). (privacy 10)

docs/audits/operator-experience-audit.md

@@ -159,9 +159,9 @@ settings screen. (issue-backoffice-unlinked, walkthrough-05, BR-DOC-16)
 - [ ] Declare WooCommerce HPOS (custom order tables) compatibility via
   `FeaturesUtil::declare_compatibility()`; the plugin advertises WooCommerce
   support but makes no declaration. (critic)
-- [ ] Resolve the Bulgarian catalog: 8 of 66 strings translated yet a
-  compiled .mo ships, giving Bulgarian admins a mixed-language UI. Complete
-  it or drop the pair until substantially complete. (critic)
+- [x] Remove the incomplete Bulgarian PO/MO catalogs and keep locale-specific
+  catalogs out of release ZIPs until bundled translations are deliberately
+  restored. (critic, superseded by product decision on 2026-07-19)
 - [ ] Explain in plain language why fields are grayed out before Enable is
   checked. (issue-disabled-state-unexplained)
 - [ ] Soften residual developer jargon: `manage_options capability`,

docs/audits/privacy-audit.md

@@ -80,17 +80,15 @@ cookies to maintain measurement state".
 - [ ] Update `tests/unit/PrivacyTest.php` string assertions.
 - [ ] Keep the wording editable and qualified; no compliance claims.
 
-## 5. Regenerate translation catalogs for the pending header edit (DISC-10)
+## 5. Translation catalogs (DISC-10, superseded)
 
-The uncommitted `basicrum.php` Description change makes
-`languages/basicrum.pot` and the bg_BG catalogs stale; the CI translation job
-runs `git diff --exit-code` on `plugins/basicrum/languages` and will fail on
-this working tree.
+Resolved by product decision on 2026-07-19: the Bulgarian PO/MO catalogs and
+standalone translation CI job were removed. Keep the POT-only local generator
+for future gettext changes, and keep locale-specific catalogs out of release
+ZIPs unless bundled translations are deliberately restored.
 
-- [ ] Run `make translations` and commit the regenerated `basicrum.pot`,
-  `basicrum-bg_BG.po`, and `basicrum-bg_BG.mo` together with the header and
-  readme edits.
-- [ ] Fold in the new strings from items 3 and 4 when those land.
+- [x] Regenerate and commit `languages/basicrum.pot` when source strings change.
+- [x] Exclude and reject locale-specific PO/MO files in release packages.
 
 ## 6. Match public page-type lists to the detector (DISC-08)
 

plugins/basicrum/tests/integration/PluginBootstrapTest.php

@@ -8,7 +8,6 @@
 namespace Basicrum\WP\Tests\Integration;
 
 use Basicrum\WP\Helpers;
-use Basicrum\WP\Plugin;
 
 /**
  * Verifies the plugin loads correctly inside the WordPress test environment.
@@ -38,32 +37,4 @@ public function test_helper_defaults_are_available() {
 		$this->assertSame( '0', $defaults['enabled'] );
 		$this->assertSame( 'footer', $defaults['script_position'] );
 	}
-
-	/**
-	 * Ensure the bundled translation catalog loads at the normal init priority.
-	 *
-	 * @return void
-	 */
-	public function test_translation_loader_uses_default_init_priority() {
-		global $wp_filter;
-
-		$callbacks = $wp_filter['init']->callbacks;
-
-		foreach ( $callbacks as $priority => $registered_callbacks ) {
-			foreach ( $registered_callbacks as $callback ) {
-				$function = $callback['function'];
-
-				if (
-					is_array( $function )
-					&& $function[0] instanceof Plugin
-					&& 'load_textdomain' === $function[1]
-				) {
-					$this->assertSame( 10, $priority );
-					return;
-				}
-			}
-		}
-
-		$this->fail( 'The Basicrum translation loader was not registered.' );
-	}
 }