refactor(installer): replace hardcoded extension list with dynamic manifest verification

Read package manifest via $adapter->getParent()->getManifest() to discover
children dynamically instead of maintaining a hardcoded EXPECTED_EXTENSIONS
array. Fail-open pattern wraps verification in try/catch. Only the success
path (plugin enables, license warning) is gated; unconditional housekeeping
(DLID restore) runs regardless.

Claude-Session: https://claude.ai/code/session_01CwLGvFJPjoPTp9BEnSjtJf

.mokogitea/workflows/issue-branch.yml

@@ -5,7 +5,7 @@
 # FILE INFORMATION
 # DEFGROUP: Gitea.Workflow
 # INGROUP: mokocli.Automation
-# VERSION: 01.00.36
+# VERSION: 01.00.33
 # BRIEF: Auto-create feature branch when an issue is opened
 
 name: "Universal: Issue Branch"

CHANGELOG.md

@@ -27,7 +27,6 @@
 - **CLAUDE.md** — repo-specific with channels, key patterns
 
 ### Fixed
-- **Installer** — honest install success: the package postflight now verifies every bundled child extension (component + system plugin) actually landed in `#__extensions` before showing the license/next-steps message, and enqueues an error listing any missing ones instead. Fails open so a transient query/IO glitch never fakes a failure.
 - **WebhookChannelHelper** — conversation race condition on concurrent webhooks (FOR UPDATE transaction)
 
 ### Removed

CODE_OF_CONDUCT.md

@@ -14,7 +14,7 @@
 	DEFGROUP: Template-Joomla
 	INGROUP: Template-Joomla.Documentation
 	REPO: https://github.com/mokoconsulting-tech/Template-Joomla/
-  VERSION: 01.00.36
+  VERSION: 01.00.33
   PATH: ./CODE_OF_CONDUCT.md
   BRIEF: Community expectations and enforcement guidelines
   NOTE: Adapted with attribution from the Contributor Covenant v2.1

GOVERNANCE.md

@@ -19,7 +19,7 @@
  DEFGROUP: mokoconsulting-tech.Template-Joomla
  INGROUP: MokoStandards.Governance
  REPO: https://github.com/mokoconsulting-tech/Template-Joomla
- VERSION: 01.00.36
+ VERSION: 01.00.33
  PATH: /GOVERNANCE.md
  BRIEF: Project governance rules, roles, and decision process for Template-Joomla
 -->

SECURITY.md

@@ -23,7 +23,7 @@ DEFGROUP: Template-Joomla
 INGROUP: Template-Joomla.Documentation
 REPO: https://git.mokoconsulting.tech/MokoConsulting/Template-Joomla
 PATH: /SECURITY.md
-VERSION: 01.00.36
+VERSION: 01.00.33
 BRIEF: Security vulnerability reporting and handling policy
 -->
 

source/packages/com_mokosuitesupport/admin/language/en-GB/com_mokosuitesupport.sys.ini

@@ -1,7 +0,0 @@
-; MokoSuite Support - system language strings
-; Copyright (C) 2026 Moko Consulting. All rights reserved.
-; License GPL-3.0-or-later
-
-COM_MOKOSUITESUPPORT="MokoSuite Support"
-COM_MOKOSUITESUPPORT_SHORT="Support"
-COM_MOKOSUITESUPPORT_XML_DESCRIPTION="MokoSuite Support component."

source/packages/com_mokosuitesupport/mokosuitesupport.xml

@@ -5,13 +5,10 @@
   <creationDate>2026-06-23</creationDate>
   <copyright>Copyright (C) 2026 Moko Consulting.</copyright>
   <license>GPL-3.0-or-later</license>
-  <version>01.00.36</version>
+  <version>01.00.33</version>
   <namespace path="src">Moko\Component\MokoSuiteSupport</namespace>
   <administration>
-    <files folder="admin"><folder>language</folder><folder>services</folder><folder>src</folder><folder>tmpl</folder></files>
+    <files folder="admin"><folder>services</folder><folder>src</folder><folder>tmpl</folder></files>
-    <menu>COM_MOKOSUITESUPPORT_SHORT</menu>
+    <menu>COM_MOKOSUITESUPPORT</menu>
-    <languages folder="admin/language">
-      <language tag="en-GB">en-GB/com_mokosuitesupport.sys.ini</language>
-    </languages>
   </administration>
 </extension>

source/packages/plg_system_mokosuitesupport/mokosuitesupport.xml

@@ -8,7 +8,7 @@
 	<license>GPL-3.0-or-later</license>
 	<authorEmail>hello@mokoconsulting.tech</authorEmail>
 	<authorUrl>https://mokoconsulting.tech</authorUrl>
-	<version>01.00.36</version>
+	<version>01.00.33</version>
 	<php_minimum>8.3</php_minimum>
 	<description>PLG_SYSTEM_MOKOSUITESUPPORT_DESC</description>
 	<namespace path="src">Moko\Plugin\System\MokoSuiteSupport</namespace>

source/packages/plg_webservices_mokosuitesupport/mokosuitesupport.xml

@@ -3,7 +3,7 @@
   <name>Web Services - MokoSuite Support</name>
   <element>mokosuitesupport</element>
   <author>Moko Consulting</author>
-  <version>01.00.36</version>
+  <version>01.00.33</version>
   <license>GPL-3.0-or-later</license>
   <namespace path="src">Moko\Plugin\WebServices\MokoSuiteSupport</namespace>
   <files><folder>src</folder><folder>services</folder></files>

source/pkg_mokosuitesupport.xml

@@ -2,7 +2,7 @@
 <extension type="package" method="upgrade">
 	<name>Package - MokoSuite Support</name>
 	<packagename>mokosuitesupport</packagename>
-	<version>01.00.36</version>
+	<version>01.00.33</version>
 	<creationDate>2026-06-21</creationDate>
 	<author>Moko Consulting</author>
 	<authorEmail>hello@mokoconsulting.tech</authorEmail>

source/script.php

@@ -1,9 +1,10 @@
 <?php
 /**
+ * MokoSuite Support Package Install Script
+ *
  * @package     MokoSuiteSupport
- * @subpackage  pkg_mokosuitesupport
  * @copyright   Copyright (C) 2026 Moko Consulting. All rights reserved.
- * @license     GNU General Public License version 3 or later; see LICENSE
+ * @license     GPL-3.0-or-later
  */
 
 defined('_JEXEC') or die;
@@ -11,106 +12,82 @@ defined('_JEXEC') or die;
 use Joomla\CMS\Factory;
 use Joomla\CMS\Installer\InstallerAdapter;
 use Joomla\CMS\Log\Log;
+use Joomla\Database\DatabaseInterface;
 
 class Pkg_MokoSuiteSupportInstallerScript
 {
 	public function postflight(string $type, InstallerAdapter $adapter): void
 	{
-		// Be honest about success. Joomla's package installer only LOGS a failed child
+		if ($type === 'install')
-		// sub-install but still runs this postflight, so don't show the license /
+		{
-		// next-steps message if a bundled extension is actually missing. Fails open
+			$missing = $this->verifyChildren($adapter);
-		// (see missingChildExtensions) so a query/IO glitch never fakes a failure.
-		$missing = $this->missingChildExtensions($adapter);
 
 			if (!empty($missing))
 			{
 				Factory::getApplication()->enqueueMessage(
-				'<h4>MokoSuiteSupport did not install correctly.</h4>'
+					'<strong>MokoSuite Support — Incomplete Installation</strong><br>'
-				. '<p>The following bundled extensions are missing: '
+					. 'The following extensions failed to install: '
-				. htmlspecialchars(implode(', ', $missing), ENT_QUOTES) . '</p>'
+					. implode(', ', $missing)
-				. '<p>Please uninstall MokoSuiteSupport and reinstall the full package.</p>',
+					. '.<br>Please check file permissions and server logs, then reinstall the package.',
 					'error'
 				);
 
 				return;
 			}
 
-		// Warn if no license key is configured (success / next-steps path)
+			$this->enablePlugin('system', 'mokosuitesupport');
+			$this->enablePlugin('webservices', 'mokosuitesupport');
+		}
+
 		$this->warnMissingLicenseKey();
 	}
 
-	/**
+	private function verifyChildren(InstallerAdapter $adapter): array
-	 * Verify every child extension declared in the package manifest actually landed
-	 * in #__extensions. Returns readable labels of any that are missing.
-	 *
-	 * Matching per Joomla's #__extensions uniqueness:
-	 *   - type    = the <file> "type" attribute
-	 *   - element = the <file> "id" attribute, EXCEPT for plugins the manifest often
-	 *               declares the full `plg_<group>_<element>` id, so strip a leading
-	 *               `plg_<group>_` to get the real element; components/modules use id as-is
-	 *   - plugins ALSO match folder = the <file> "group" attribute
-	 *
-	 * FAILS OPEN: any error (unreadable manifest, query failure) returns [] so a
-	 * transient glitch never turns a good install into a false failure.
-	 *
-	 * @param   InstallerAdapter  $adapter  The package installer adapter.
-	 *
-	 * @return  string[]
-	 */
-	private function missingChildExtensions(InstallerAdapter $adapter): array
 	{
 		try
 		{
 			$manifest = $adapter->getParent()->getManifest();
+			$files    = $manifest->files->file ?? [];
 
-			if (!$manifest || !isset($manifest->files) || !isset($manifest->files->file))
+			if (!$files)
 			{
 				return [];
 			}
 
-			$db      = Factory::getDbo();
+			$db      = Factory::getContainer()->get(DatabaseInterface::class);
 			$missing = [];
 
-			foreach ($manifest->files->file as $file)
+			foreach ($files as $file)
 			{
-				$attrs  = $file->attributes();
+				$type  = (string) ($file['type'] ?? '');
-				$id     = isset($attrs['id']) ? (string) $attrs['id'] : '';
+				$id    = (string) ($file['id'] ?? '');
-				$exType = isset($attrs['type']) ? (string) $attrs['type'] : '';
+				$group = (string) ($file['group'] ?? '');
 
-				if ($id === '' || $exType === '')
+				if ($type === '' || $id === '')
 				{
 					continue;
 				}
 
-				$group   = isset($attrs['group']) ? (string) $attrs['group'] : '';
+				$element = ($type === 'plugin' && $group !== '')
-				$element = $id;
+					? preg_replace('/^plg_' . preg_quote($group, '/') . '_/', '', $id)
-
+					: $id;
-				// Plugin element in #__extensions is the id minus any leading plg_<group>_.
-				if ($exType === 'plugin' && $group !== '')
-				{
-					$prefix = 'plg_' . $group . '_';
-
-					if (strpos($element, $prefix) === 0)
-					{
-						$element = substr($element, \strlen($prefix));
-					}
-				}
 
 				$query = $db->getQuery(true)
 					->select('COUNT(*)')
 					->from($db->quoteName('#__extensions'))
 					->where($db->quoteName('element') . ' = ' . $db->quote($element))
-					->where($db->quoteName('type') . ' = ' . $db->quote($exType));
+					->where($db->quoteName('type') . ' = ' . $db->quote($type));
 
-				if ($exType === 'plugin' && $group !== '')
+				if ($type === 'plugin' && $group !== '')
 				{
 					$query->where($db->quoteName('folder') . ' = ' . $db->quote($group));
 				}
 
-				if ((int) $db->setQuery($query)->loadResult() === 0)
+				$db->setQuery($query);
+
+				if ((int) $db->loadResult() === 0)
 				{
-					$label     = trim((string) $file);
+					$missing[] = htmlspecialchars($type . ': ' . $id, ENT_QUOTES, 'UTF-8');
-					$missing[] = $label !== '' ? preg_replace('/\.zip$/i', '', $label) : $id;
 				}
 			}
 
@@ -118,16 +95,36 @@ class Pkg_MokoSuiteSupportInstallerScript
 		}
 		catch (\Throwable $e)
 		{
-			// Fail open — never fake a failure on a glitch.
 			return [];
 		}
 	}
 
+	private function enablePlugin(string $group, string $element): void
+	{
+		try
+		{
+			$db = Factory::getContainer()->get(DatabaseInterface::class);
+			$db->setQuery(
+				$db->getQuery(true)
+					->update($db->quoteName('#__extensions'))
+					->set($db->quoteName('enabled') . ' = 1')
+					->where($db->quoteName('type') . ' = ' . $db->quote('plugin'))
+					->where($db->quoteName('folder') . ' = ' . $db->quote($group))
+					->where($db->quoteName('element') . ' = ' . $db->quote($element))
+			);
+			$db->execute();
+		}
+		catch (\Throwable $e)
+		{
+			Log::add('Support install: failed to enable ' . $group . '/' . $element . ' — ' . $e->getMessage(), Log::WARNING, 'mokosuite.support');
+		}
+	}
+
 	private function warnMissingLicenseKey(): void
 	{
 		try
 		{
-			$db  = Factory::getDbo();
+			$db  = Factory::getContainer()->get(DatabaseInterface::class);
 			$app = Factory::getApplication();
 
 			$query = $db->getQuery(true)