fix(cli): Joomla module element detection and display name fixes #203

Merged
jmiller merged 4 commits from dev into main 2026-05-28 22:19:02 +00:00

4 Commits

Author SHA1 Message Date
gitea-actions[bot] 048d1f4914 chore(version): auto-bump patch 09.03.02-dev [skip ci]
Branch Cleanup / Delete merged branch (pull_request) Has been skipped
Universal: Build & Release / Promote Pre-Release to RC (pull_request) Has been skipped
Universal: Build & Release / Build & Release Pipeline (pull_request) Failing after 13s
2026-05-28 22:15:35 +00:00
Jonathan Miller 1cbdaa0c37 fix(cli): add module= attribute extraction to element detection
Generic: Repo Health / Site Health (push) Has been skipped
Generic: Repo Health / Access control (push) Successful in 1s
Universal: Auto Version Bump / Version Bump (push) Successful in 4s
Platform: moko-platform CI / Gate 1: Code Quality (push) Failing after 48s
Platform: moko-platform CI / Gate 2: Unit Tests (8.1) (push) Has been cancelled
Platform: moko-platform CI / Gate 2: Unit Tests (8.2) (push) Has been cancelled
Platform: moko-platform CI / Gate 2: Unit Tests (8.3) (push) Has been cancelled
Platform: moko-platform CI / Gate 3: Self-Health Check (push) Has been cancelled
Platform: moko-platform CI / Gate 4: Governance (push) Has been cancelled
Platform: moko-platform CI / Gate 5: Template Integrity (push) Has been cancelled
Platform: moko-platform CI / CI Summary (push) Has been cancelled
Generic: Repo Health / Release configuration (push) Has been cancelled
Generic: Repo Health / Scripts governance (push) Has been cancelled
Generic: Repo Health / Repository health (push) Has been cancelled
Joomla modules declare their element via the module="mod_foo" attribute
on the entry <filename>, not via an <element> tag. Both manifest_element.php
and release_package.php were missing this extraction, causing incorrect
element names and ZIP filenames for module-type extensions.

Added module="..." regex check in the element detection chain for both
files, matching what updates_xml_build.php already had.

Authored-by: Moko Consulting
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-05-28 17:15:27 -05:00
gitea-actions[bot] 32e34fc936 chore(version): auto-bump patch 09.03.01-dev [skip ci] 2026-05-28 22:10:46 +00:00
Jonathan Miller d7e679b7b2 fix(cli): correct element prefix and display name in updates_xml_build
Generic: Repo Health / Site Health (push) Has been skipped
Generic: Repo Health / Access control (push) Successful in 1s
Universal: Auto Version Bump / Version Bump (push) Successful in 5s
Platform: moko-platform CI / Gate 1: Code Quality (push) Failing after 1m2s
Platform: moko-platform CI / Gate 2: Unit Tests (8.1) (push) Has been cancelled
Platform: moko-platform CI / Gate 2: Unit Tests (8.2) (push) Has been cancelled
Platform: moko-platform CI / Gate 2: Unit Tests (8.3) (push) Has been cancelled
Platform: moko-platform CI / Gate 3: Self-Health Check (push) Has been cancelled
Platform: moko-platform CI / Gate 4: Governance (push) Has been cancelled
Platform: moko-platform CI / Gate 5: Template Integrity (push) Has been cancelled
Platform: moko-platform CI / CI Summary (push) Has been cancelled
Generic: Repo Health / Release configuration (push) Has been cancelled
Generic: Repo Health / Scripts governance (push) Has been cancelled
Generic: Repo Health / Repository health (push) Has been cancelled
- Add type prefix (mod_, com_, tpl_, lib_) to <element> in updates.xml
  so it matches Joomla's #__extensions table. Previously only packages
  got their prefix re-added after the strip on line 178.
- Prefer human-readable name from .mokogitea/manifest.xml <identity><name>
  when the Joomla manifest <name> is a technical prefixed name (mod_foo).
  Prevents display names like "Module - mod_mokojoomhero".
- Existing "Type - " prefix strip still prevents doubling when the
  language key already resolves to "Module - MokoJoomHero".

Authored-by: Moko Consulting
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2026-05-28 17:10:29 -05:00