fix: workflow audit — CI sanity checks and rename fixes #26

Merged

jmiller merged 3 commits from fix/workflow-audit into main 2026-06-23 16:56:35 +00:00

Conversation 1 Commits 3 Files Changed 1

+331

jmiller commented 2026-06-23 16:51:58 +00:00

Owner

Copy Link

Copy Source

Summary

• Add 7 new CI sanity checks to ci-joomla.yml for Joomla extension validation
• Fix missing step IDs in auto-release.yml
• Rename moko-platform → mokocli across workflow files

New CI Checks (ci-joomla)

Check	Severity	Description
Update server	warning	Must use MokoGitea engine (git.mokoconsulting.tech)
<dlid>	warning	Download ID auth tag must be present
<childuninstall>	warning	Package extensions must declare child uninstall
<scriptfile>	error	Referenced script file must exist
Media folder	error	<media> referenced dirs/files must exist
Target platform	warning	<targetplatform> / <php_minimum> should be set
<changelogurl>	warning	Joomla 4+ update manager changelog support
Duplicate refs	error	No duplicate <filename> / <folder> in manifest
Empty lang keys	warning	.ini files should not have KEY= with no value

Other Fixes

• auto-release.yml: added missing step id: fields
• issue-branch.yml, rc-revert.yml, workflow-sync-trigger.yml: moko-platform → mokocli

## Summary - Add 7 new CI sanity checks to `ci-joomla.yml` for Joomla extension validation - Fix missing step IDs in `auto-release.yml` - Rename `moko-platform` → `mokocli` across workflow files ## New CI Checks (ci-joomla) | Check | Severity | Description | |---|---|---| | Update server | warning | Must use MokoGitea engine (`git.mokoconsulting.tech`) | | `<dlid>` | warning | Download ID auth tag must be present | | `<childuninstall>` | warning | Package extensions must declare child uninstall | | `<scriptfile>` | error | Referenced script file must exist | | Media folder | error | `<media>` referenced dirs/files must exist | | Target platform | warning | `<targetplatform>` / `<php_minimum>` should be set | | `<changelogurl>` | warning | Joomla 4+ update manager changelog support | | Duplicate refs | error | No duplicate `<filename>` / `<folder>` in manifest | | Empty lang keys | warning | `.ini` files should not have `KEY=` with no value | ## Other Fixes - `auto-release.yml`: added missing step `id:` fields - `issue-branch.yml`, `rc-revert.yml`, `workflow-sync-trigger.yml`: `moko-platform` → `mokocli`

jmiller added 2 commits 2026-06-23 16:51:59 +00:00

feat: add update server, dlid, and childuninstall checks to ci-joomla ... 56e6ce00ae

Authored-by: Moko Consulting

feat: add 6 sanity checks to ci-joomla workflow ... 92639daf5d

- scriptfile reference validation (error if missing)
- media folder/file reference validation (error if missing)
- target platform constraints warning (targetplatform, php_minimum)
- changelogurl warning (Joomla 4+ update manager)
- duplicate file/folder references in manifest (error)
- empty language keys in .ini files (warning)

Authored-by: Moko Consulting

jmiller added 1 commit 2026-06-23 16:54:23 +00:00

fix: change duplicate file references check from error to warning ... ba3cefa470

Cross-section duplicates (e.g. same folder in <files> and <media>) are
valid in Joomla manifests — should not block CI.

Authored-by: Moko Consulting

jmiller merged commit 69ac75e35b into main 2026-06-23 16:56:35 +00:00

jmiller deleted branch fix/workflow-audit 2026-06-23 16:56:36 +00:00

jmiller commented 2026-06-23 17:06:01 +00:00

Author

Owner

Copy Link

Copy Source

Test Plan

Moved from PR body:

• YAML validates (python -c "import yaml; yaml.safe_load(...")
• CI runs against a Joomla extension repo with all checks passing or producing correct warnings
• Warnings use continue-on-error and don't block CI
• Errors correctly fail the job when files are missing

## Test Plan Moved from PR body: - [ ] YAML validates (`python -c "import yaml; yaml.safe_load(..."`) - [ ] CI runs against a Joomla extension repo with all checks passing or producing correct warnings - [ ] Warnings use `continue-on-error` and don't block CI - [ ] Errors correctly fail the job when files are missing

jmiller referenced this pull request 2026-06-23 17:30:42 +00:00

fix: add missing PHP setup to workflow-sync-trigger #28

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

breaking-change

Breaking API or behavior change

ci-cd

CI/CD pipeline changes

config

Configuration changes

css

CSS/styling changes

dependencies

Dependency updates

deploy-failure

Deployment failed

documentation

Documentation changes

good first issue

Good for newcomers

health-check

Repo health check result

help wanted

Extra attention needed

html

HTML/template changes

joomla

Joomla extension

major-release

Major version release

minor-release

Minor version release

mokostandards

Related to MokoStandards framework

patch-release

Patch version release

php

PHP code

push-failure

Git push operation failed

release-candidate

Release candidate build

security

Security vulnerability or hardening

size/l

200-500 lines changed

size/m

50-200 lines changed

size/s

10-50 lines changed

size/xl

500+ lines changed

size/xs

< 10 lines changed

standards-drift

Deviates from MokoStandards

standards-update

MokoStandards compliance update

sync-failure

Sync or mirror failed

version-drift

Version mismatch across files

work-in-progress

Draft or incomplete work

No labels

Priority -

Type -

Milestone

No items

No Milestone

Assignees

Clear assignees

jmiller (Jonathan Miller)

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: MokoConsulting/Template-Joomla#26