Skip to content

feat(container): add Magento app adapter - #274

Merged
wpoortman merged 5 commits into
mainfrom
feat/app-helper-parity
Aug 18, 2026
Merged

feat(container): add Magento app adapter#274
wpoortman merged 5 commits into
mainfrom
feat/app-helper-parity

Conversation

@wpoortman

Copy link
Copy Markdown
Collaborator

Summary

  • route app() through a Magento-backed application container
  • preserve shared Magento get() behavior for ordinary resolution
  • forward explicit constructor parameters through Magento create()
  • support Magento preferences, virtual types, and Magewire aliases
  • provide the retained Livewire has(), singleton(), and instance() compatibility surface
  • keep application bindings exclusively in Magento di.xml; intentionally omit runtime bind()
  • preserve legacy ObjectManager-compatible get(), create(), and configure() methods

Verification

  • 14 PHPUnit tests, 45 assertions
  • focused Mago formatting and lint checks pass
  • Magento frontend bootstrap verifies shared identity, contextual parameters, preferences, virtual types, and aliases
  • Portman build succeeds and leaves dist/ unchanged

Follow-up coverage

Illuminate 13, adminhtml, and the broader supported Magento/Mage-OS matrix remain recorded as follow-up checks in the feature plan.

Plan

.agents/plans/feat-2026-08-13-app-helper-parity/PLAN.md

Willem Poortman added 3 commits August 13, 2026 11:11
Forward explicit resolution parameters through Magento create() while preserving shared get() behavior and Magewire aliases.

Keep application bindings in di.xml; expose only the runtime singleton and instance compatibility required by retained Livewire code.
@wpoortman wpoortman changed the title Add Magento-backed app() container compatibility feat(container): add Magento app adapter Aug 13, 2026
@wpoortman
wpoortman merged commit 2ac8ae1 into main Aug 18, 2026
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant