From 8892ade56aebcdb5de31622e0a9a1bcd5c0642cd Mon Sep 17 00:00:00 2001
From: Jonathan Miller <jmiller-moko@noreply.git.mokoconsulting.tech>
Date: Sat, 23 May 2026 23:24:44 -0500
Subject: [PATCH] fix: version_set_platform updates sub-package manifests too

Was only globbing src/*.xml, missing src/packages/*/*.xml. Joomla
packages have sub-extension manifests that also need version updates.

Authored-by: Moko Consulting
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 cli/version_set_platform.php | 15 ++++++++++++---
 1 file changed, 12 insertions(+), 3 deletions(-)

diff --git a/cli/version_set_platform.php b/cli/version_set_platform.php
index 4752b45..965e15d 100644
--- a/cli/version_set_platform.php
+++ b/cli/version_set_platform.php
@@ -131,9 +131,17 @@ if ($platform === 'crm-module') {
     }
 }
 
-// Joomla: <version> in XML manifests
+// Joomla: <version> in XML manifests (top-level + sub-packages)
 if (in_array($platform, ['waas-component', 'joomla'], true)) {
-    foreach (glob("{$root}/src/*.xml") ?: glob("{$root}/*.xml") ?: [] as $file) {
+    $xmlFiles = array_merge(
+        glob("{$root}/src/*.xml") ?: [],
+        glob("{$root}/src/packages/*/*.xml") ?: [],
+        glob("{$root}/*.xml") ?: []
+    );
+    if (empty($xmlFiles)) {
+        $xmlFiles = glob("{$root}/*.xml") ?: [];
+    }
+    foreach ($xmlFiles as $file) {
         $content = file_get_contents($file);
         if (!str_contains($content, '<extension')) continue;
         $updated = preg_replace(
@@ -143,7 +151,8 @@ if (in_array($platform, ['waas-component', 'joomla'], true)) {
         );
         if ($updated !== $content) {
             file_put_contents($file, $updated);
-            echo "Joomla: " . basename($file) . " → {$version}\n";
+            $relPath = str_replace($root . '/', '', $file);
+            echo "Joomla: {$relPath} → {$version}\n";
             $changed++;
         }
     }
-- 
2.52.0