MokoConsulting/MokoSuiteAuto
1
0
Fork 0
generated from MokoConsulting/Template-Joomla
Code Issues 11 Pull Requests 1 Releases 1 Wiki Activity

feat: add install verification for failed sub-extensions #13

Open
jmiller wants to merge 2 commits from feat/install-verification into dev
pull from: feat/install-verification
merge into: :dev
Conversation 0 Commits 2 Files Changed 1
+181 -37
jmiller commented 2026-07-06 03:33:35 +00:00
Owner
Copy Link
Copy Source

Summary

  • Add post-install verification that checks all expected sub-extensions (component, plugins) are registered in #__extensions
  • If any sub-extension failed to install, disables the package entry and shows a clear error message listing what's missing
  • Prevents partially installed packages from appearing functional

Test plan

  • Install package normally — verify no error shown
  • Simulate failed sub-extension install — verify error message and package disabled

https://claude.ai/code/session_01CwLGvFJPjoPTp9BEnSjtJf

## Summary - Add post-install verification that checks all expected sub-extensions (component, plugins) are registered in `#__extensions` - If any sub-extension failed to install, disables the package entry and shows a clear error message listing what's missing - Prevents partially installed packages from appearing functional ## Test plan - [ ] Install package normally — verify no error shown - [ ] Simulate failed sub-extension install — verify error message and package disabled https://claude.ai/code/session_01CwLGvFJPjoPTp9BEnSjtJf
jmiller added 1 commit 2026-07-06 03:33:35 +00:00
feat: add post-install verification to detect failed sub-extensions
Universal: PR Check / Branch Policy (pull_request) Successful in 2s
Details
Universal: PR Check / Validate PR (pull_request) Failing after 5s
Details
Universal: PR Check / Secret Scan (pull_request) Successful in 9s
Details
Joomla: Metadata Validation / Validate Joomla Metadata (pull_request) Successful in 12s
Details
Universal: PR Check / Build RC Package (pull_request) Has been cancelled
Details
Universal: PR Check / Report Issues (pull_request) Has been cancelled
Details
5544a0812f 
Checks that all expected component and plugins registered in
#__extensions after install. Disables the package entry and shows
an error if any sub-extension failed to install.

Claude-Session: https://claude.ai/code/session_01CwLGvFJPjoPTp9BEnSjtJf
jmiller added 1 commit 2026-07-06 15:47:33 +00:00
refactor(installer): replace hardcoded extension list with dynamic manifest verification
Universal: PR Check / Branch Policy (pull_request) Successful in 3s
Details
Universal: PR Check / Validate PR (pull_request) Failing after 6s
Details
Universal: PR Check / Secret Scan (pull_request) Successful in 8s
Details
Joomla: Metadata Validation / Validate Joomla Metadata (pull_request) Successful in 48s
Details
Universal: PR Check / Build RC Package (pull_request) Has been cancelled
Details
Universal: PR Check / Report Issues (pull_request) Has been cancelled
Details
d0179fd6e3 
Read package manifest via $adapter->getParent()->getManifest() to discover
children dynamically instead of maintaining a hardcoded EXPECTED_EXTENSIONS
array. Fail-open pattern wraps verification in try/catch. Only the success
path (plugin enables, license warning) is gated; unconditional housekeeping
(DLID restore) runs regardless.

Claude-Session: https://claude.ai/code/session_01CwLGvFJPjoPTp9BEnSjtJf
Some required checks failed
Universal: PR Check / Branch Policy (pull_request) Successful in 3s
Details
Universal: PR Check / Validate PR (pull_request) Failing after 6s
Details
Universal: PR Check / Secret Scan (pull_request) Successful in 8s
Details
Joomla: Metadata Validation / Validate Joomla Metadata (pull_request) Successful in 48s
Details
Universal: PR Check / Build RC Package (pull_request) Has been cancelled
Details
Universal: PR Check / Report Issues (pull_request) Has been cancelled
Details
You are not authorized to merge this pull request.
This pull request can be merged automatically.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feat/install-verification:feat/install-verification
git checkout feat/install-verification
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/MokoSuiteAuto#13
Delete Branch "feat/install-verification"

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?