Skip to content

Cache Assets Manager request state#153

Merged
mehul0810 merged 1 commit into
release/1.7.0from
fix/105-release-1.7.0-assets-manager-state-cache
Jul 2, 2026
Merged

Cache Assets Manager request state#153
mehul0810 merged 1 commit into
release/1.7.0from
fix/105-release-1.7.0-assets-manager-state-cache

Conversation

@mehul0810

Copy link
Copy Markdown
Collaborator

Summary

  • Cache selected Assets Manager options, cleaned query data, current object ID, and parsed asset source groups per request.
  • Preserve the existing stored option shape and frontend asset enable/disable decisions.
  • Add regression coverage for current-page disabled rules, current-page enabled exceptions, group-level source matching, and option-read reuse.

Validation

  • php -l src/Modules/Assets/AssetsManager.php && php -l tests/bootstrap.php && php -l tests/unit-tests/tests-assets-manager.php
  • ./vendor/bin/phpcs -s src/Modules/Assets/AssetsManager.php tests/bootstrap.php tests/unit-tests/tests-assets-manager.php
  • composer phpstan
  • composer test - 42 tests, 80 assertions
  • git diff --check

Notes

  • No UI source changed. Local browser/e2e smoke was not rerun in this environment because prior wp-env attempts are blocked by missing Docker (spawn docker ENOENT); GitHub CI remains the release-branch validation path for hosted smoke coverage.

Closes #105

@github-actions

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown

Test on Playground
Test this pull request on the Playground

@mehul0810 mehul0810 merged commit dcd7964 into release/1.7.0 Jul 2, 2026
10 checks passed
@mehul0810 mehul0810 deleted the fix/105-release-1.7.0-assets-manager-state-cache branch July 2, 2026 05:44
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