From 240ae2f80338754a44cdc614d031c7ec83278362 Mon Sep 17 00:00:00 2001
From: Jonathan Miller <jmiller-moko@noreply.git.mokoconsulting.tech>
Date: Sun, 24 May 2026 17:38:38 -0500
Subject: [PATCH] fix: repair PHP syntax error in heredoc closers

Heredoc closers must use the same whitespace type (tabs) as the body
content per PHP 7.3+ flexible heredoc rules. Also exclude the
TabsUsedHeredocCloser PHPCS rule since it conflicts with PHP syntax.

Authored-by: Moko Consulting
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
---
 automation/bulk_joomla_template.php | 14 +++++++-------
 phpcs.xml                           |  2 ++
 2 files changed, 9 insertions(+), 7 deletions(-)

diff --git a/automation/bulk_joomla_template.php b/automation/bulk_joomla_template.php
index 54fa232..6e317c3 100644
--- a/automation/bulk_joomla_template.php
+++ b/automation/bulk_joomla_template.php
@@ -487,12 +487,12 @@ class BulkJoomlaTemplate extends CLIApp
 				</fields>
 			</config>
 		</extension>
-        XML;
+		XML;
         $files['templateDetails.xml'] = preg_replace('/^\t\t/m', '', $files['templateDetails.xml']);
 
         // updates.xml — dual-platform download URLs (Gitea primary, GitHub secondary)
         $files['updates.xml'] = <<<XML
-        <updates>
+		<updates>
 			<update>
 				<name>{$name}</name>
 				<description>{$name} — Moko Consulting Joomla template</description>
@@ -511,7 +511,7 @@ class BulkJoomlaTemplate extends CLIApp
 				<php_minimum>8.1</php_minimum>
 			</update>
 		</updates>
-        XML;
+		XML;
         $files['updates.xml'] = preg_replace('/^\t\t/m', '', $files['updates.xml']);
 
         // src/index.php
@@ -562,7 +562,7 @@ class BulkJoomlaTemplate extends CLIApp
 			<jdoc:include type="modules" name="debug" style="none" />
 		</body>
 		</html>
-        PHP;
+		PHP;
         $files['src/index.php'] = str_replace('TEMPLATE_SHORT_NAME', $shortName, $files['src/index.php']);
         $files['src/index.php'] = preg_replace('/^\t\t/m', '', $files['src/index.php']);
 
@@ -599,7 +599,7 @@ class BulkJoomlaTemplate extends CLIApp
 			</div>
 		</body>
 		</html>
-        PHP;
+		PHP;
         $files['src/error.php'] = preg_replace('/^\t\t/m', '', $files['src/error.php']);
 
         // src/offline.php
@@ -642,7 +642,7 @@ class BulkJoomlaTemplate extends CLIApp
 			</div>
 		</body>
 		</html>
-        PHP;
+		PHP;
         $files['src/offline.php'] = preg_replace('/^\t\t/m', '', $files['src/offline.php']);
 
         // src/component.php
@@ -668,7 +668,7 @@ class BulkJoomlaTemplate extends CLIApp
 			<jdoc:include type="component" />
 		</body>
 		</html>
-        PHP;
+		PHP;
         $files['src/component.php'] = preg_replace('/^\t\t/m', '', $files['src/component.php']);
 
         // Directory keepfiles
diff --git a/phpcs.xml b/phpcs.xml
index 32e66c9..cba030b 100644
--- a/phpcs.xml
+++ b/phpcs.xml
@@ -30,6 +30,8 @@ SPDX-License-Identifier: GPL-3.0-or-later
         <exclude name="PSR1.Classes.ClassDeclaration.MultipleClasses"/>
         <!-- File header ordering is advisory -->
         <exclude name="PSR12.Files.FileHeader.IncorrectOrder"/>
+        <!-- Heredoc closers must match body indentation (tabs) per PHP 7.3+ -->
+        <exclude name="Generic.WhiteSpace.DisallowTabIndent.TabsUsedHeredocCloser"/>
     </rule>
 
     <!-- Additional rules -->