Consolidate RedHerb's NeoWikiGetFrontendModules handlers

RedHerb had two classes implementing NeoWikiGetFrontendModulesHook:
RedHerbFrontendHook (added with the Color frontend) and
RedHerbFrontendModulesHook (added later with the sidebar dialogs). The
split was incidental — a later PR added a new file instead of editing
the existing one — and there's no functional reason for two classes
when one handler can append all three modules.

Merge the body of RedHerbFrontendHook into RedHerbFrontendModulesHook
and remove the redundant class and its extension.json registration. No
behavior change.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: malberts

tests/RedHerb/extension.json

@@ -31,17 +31,14 @@
 
 	"Hooks": {
 		"NeoWikiRegistration": "ProfessionalWiki\\RedHerb\\RedHerbHooks::onNeoWikiRegistration",
-		"NeoWikiGetFrontendModules": [ "RedHerbFrontendModulesHook", "RedHerbFrontendHook" ],
+		"NeoWikiGetFrontendModules": "RedHerbFrontendModulesHook",
 		"SidebarBeforeOutput": "RedHerbSidebarHook"
 	},
 
 	"HookHandlers": {
 		"RedHerbFrontendModulesHook": {
 			"class": "ProfessionalWiki\\RedHerb\\RedHerbFrontendModulesHook"
 		},
-		"RedHerbFrontendHook": {
-			"class": "ProfessionalWiki\\RedHerb\\RedHerbFrontendHook"
-		},
 		"RedHerbSidebarHook": {
 			"class": "ProfessionalWiki\\RedHerb\\RedHerbSidebarHook"
 		}

tests/RedHerb/src/RedHerbFrontendHook.php

@@ -11,6 +11,7 @@
 class RedHerbFrontendModulesHook implements NeoWikiGetFrontendModulesHook {
 
 	public function onNeoWikiGetFrontendModules( array &$modules, OutputPage $out, Skin $skin ): void {
+		$modules[] = 'ext.redherb';
 		$modules[] = 'ext.redherb-create-child';
 		$modules[] = 'ext.redherb-edit-main-subject';
 	}

tests/RedHerb/src/RedHerbFrontendModulesHook.php

@@ -14,7 +14,7 @@
  */
 class RedHerbFrontendModulesHookTest extends TestCase {
 
-	public function testAddsRedHerbDialogModules(): void {
+	public function testAddsRedHerbModules(): void {
 		$modules = [];
 		$hook = new RedHerbFrontendModulesHook();
 
@@ -25,7 +25,7 @@ public function testAddsRedHerbDialogModules(): void {
 		);
 
 		$this->assertSame(
-			[ 'ext.redherb-create-child', 'ext.redherb-edit-main-subject' ],
+			[ 'ext.redherb', 'ext.redherb-create-child', 'ext.redherb-edit-main-subject' ],
 			$modules
 		);
 	}
@@ -41,7 +41,7 @@ public function testPreservesExistingModules(): void {
 		);
 
 		$this->assertSame(
-			[ 'ext.preexisting', 'ext.redherb-create-child', 'ext.redherb-edit-main-subject' ],
+			[ 'ext.preexisting', 'ext.redherb', 'ext.redherb-create-child', 'ext.redherb-edit-main-subject' ],
 			$modules
 		);
 	}