MokoConsulting/Template-Joomla
Template
1
0
Fork 0
Code Issues Pull Requests Releases 3 Wiki Activity

fix(workflows): #53 dev-cycle correctness (protection-independent) #50

Merged
jmiller merged 1 commits from fix/workflow-audit-53 into main 2026-07-04 23:46:27 +00:00
Conversation 0 Commits 1 Files Changed 4
+9 -7
jmiller commented 2026-07-04 23:46:07 +00:00
Owner
Copy Link
Copy Source

First (protection-independent) batch of the workflow audit (Template-Generic#53):

  • cleanup.yml 🔴 — skip-list now preserves dev, rc, release-line branches, and version/* (it was deleting version/* release-history branches weekly).
  • ci-joomla.yml 🔴 — added plain dev to the PR trigger ([main, dev, 'dev/**']) so the Joomla lint/validate/test CI actually runs on feature → dev PRs (dev/** didn't match dev).
  • pr-check.yml 🔴 — detect platform via the MokoGitea metadata API instead of the removed .mokogitea/manifest.xml; the stale detector fell through to generic, so every Joomla-specific PR gate was silently skipping (JEXEC guard, language files, updates.xml, manifest validation).
  • notify.yml 🟠 — real workflow names so release/failure ntfy fires.

Deferred (depend on branch-protection rollout / bump redesign): Step 11 dev-sync, promote-rc/rc-revert, version-bump consolidation, and the Joomla pr-check branch-policy reconciliation. Tracked in Template-Generic#53 + mcp-mokogitea-api#30.

First (protection-independent) batch of the workflow audit (Template-Generic#53): - **`cleanup.yml`** 🔴 — skip-list now preserves `dev`, `rc`, release-line branches, and **`version/*`** (it was deleting `version/*` release-history branches weekly). - **`ci-joomla.yml`** 🔴 — added plain `dev` to the PR trigger (`[main, dev, 'dev/**']`) so the Joomla lint/validate/test CI actually runs on `feature → dev` PRs (`dev/**` didn't match `dev`). - **`pr-check.yml`** 🔴 — detect platform via the **MokoGitea metadata API** instead of the removed `.mokogitea/manifest.xml`; the stale detector fell through to `generic`, so **every Joomla-specific PR gate was silently skipping** (JEXEC guard, language files, updates.xml, manifest validation). - **`notify.yml`** 🟠 — real workflow names so release/failure ntfy fires. Deferred (depend on branch-protection rollout / bump redesign): Step 11 dev-sync, promote-rc/rc-revert, version-bump consolidation, and the Joomla `pr-check` branch-policy reconciliation. Tracked in Template-Generic#53 + mcp-mokogitea-api#30.
jmiller added 1 commit 2026-07-04 23:46:08 +00:00
fix(workflows): #53 dev-cycle correctness (protection-independent)
Universal: Pre-Release / Build Pre-Release (${{ inputs.stability || github.ref_name }}) (push) Has been skipped
Details
Generic: Project CI / Lint & Validate (pull_request) Has been skipped
Details
Universal: PR Check / Branch Policy (pull_request) Failing after 2s
Details
Universal: PR Check / Validate PR (pull_request) Has been skipped
Details
Joomla: Metadata Validation / Validate Joomla Metadata (pull_request) Has been skipped
Details
Generic: Repo Health / Site Health (pull_request) Has been skipped
Details
Generic: Repo Health / Access control (pull_request) Successful in 2s
Details
Universal: PR Check / Secret Scan (pull_request) Successful in 8s
Details
RC Revert / Rename rc/ back to dev/ (pull_request) Has been skipped
Details
Branch Cleanup / Delete merged branch (pull_request) Successful in 2s
Details
Universal: Workflow Sync Trigger / Sync workflows to live repos (pull_request) Successful in 15s
Details
Generic: Project CI / Tests (pull_request) Has been cancelled
Details
Universal: PR Check / Build RC Package (pull_request) Has been cancelled
Details
Universal: PR Check / Report Issues (pull_request) Has been cancelled
Details
Generic: Repo Health / Scripts governance (pull_request) Has been cancelled
Details
Generic: Repo Health / Repository health (pull_request) Has been cancelled
Details
Generic: Repo Health / Report: Scripts Governance (pull_request) Has been cancelled
Details
Generic: Repo Health / Report: Repository Health (pull_request) Has been cancelled
Details
718d57bd1e 
- cleanup.yml: preserve dev/rc/release-line + version/* branches (skip-list was
  deleting version/* release-history branches every week)
- notify.yml: point workflow_run at the real workflow names (Universal: Build &
  Release / Joomla: Extension CI / Generic: Project CI) so release/failure ntfy
  actually fires
- ci-joomla.yml: add plain `dev` to PR triggers so Joomla CI runs on feature->dev
- pr-check.yml (Joomla): detect platform via the MokoGitea metadata API instead
  of the removed .mokogitea/manifest.xml (Joomla PR gates were silently skipping)

Refs Template-Generic#53. Dev-sync/version-bump fixes deferred until the
branch-protection provisioner rolls out (mcp-mokogitea-api#30).

Claude-Session: https://claude.ai/code/session_01WbGBN9VyRK61zczYWcCQ2i
jmiller merged commit 95ea7cee8c into main 2026-07-04 23:46:27 +00:00
jmiller deleted branch fix/workflow-audit-53 2026-07-04 23:46:28 +00:00
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
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No labels
Priority -
Type -
Milestone
No items
No Milestone
Assignees
Clear assignees
jmiller (Jonathan Miller)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: MokoConsulting/Template-Joomla#50
Delete Branch "fix/workflow-audit-53"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?