Skip to content

Replace Mozart by Strauss#1016

Merged
remyperona merged 26 commits intodevelopfrom
chore/strauss
Mar 2, 2026
Merged

Replace Mozart by Strauss#1016
remyperona merged 26 commits intodevelopfrom
chore/strauss

Conversation

@remyperona
Copy link
Copy Markdown
Contributor

@remyperona remyperona commented Jan 23, 2026

Description

Replace Mozart by Strauss to handle prefixing the namespaces of our production dependencies

Type of change

  • Chore

Detailed scenario

What was tested

Tested global function of the plugin, as this mostly impacts the container (used everywhere), and the plugin family

How to test

Smoke tests should cover testing the PR

Affected Features & Quality Assurance Scope

  • Container
  • WP background processing
  • Plugin Family

Technical description

Documentation

Strauss is used as a phar file instead of a composer dev dependency.

The prefixed dependencies are preserved in the vendor folder, so they don't need to be part of the committed files anymore.

New dependencies

Removed mozart and replace it with strauss as a phar, not a dependency

Mandatory Checklist

Code validation

  • I validated all the Acceptance Criteria. If possible, provide screenshots or videos.

Code style

  • I wrote a self-explanatory code about what it does.
  • I protected entry points against unexpected inputs.
  • I did not introduce unnecessary complexity.

@remyperona remyperona self-assigned this Jan 23, 2026
@codacy-production
Copy link
Copy Markdown

codacy-production Bot commented Jan 23, 2026

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
Report missing for 1c4318e1 0.00% (target: 50.00%)
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (1c4318e) Report Missing Report Missing Report Missing
Head commit (f0dbfa9) 13940 48 0.34%

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#1016) 82 0 0.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Footnotes

  1. Codacy didn't receive coverage data for the commit, or there was an error processing the received data. Check your integration for errors and validate that your coverage setup is correct.

@remyperona remyperona marked this pull request as ready for review January 26, 2026 19:50
@remyperona remyperona requested a review from Copilot March 2, 2026 14:31
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR replaces the Mozart-based dependency prefixing approach with Strauss (downloaded as a phar) and updates how production dependencies are managed/prefixed, including cleaning up previously committed prefixed dependency sources and refreshing the bundled Action Scheduler.

Changes:

  • Switch dependency prefixing from coenjacobs/mozart to strauss (phar), updating Composer config and scripts accordingly.
  • Remove previously committed prefixed dependency sources under classes/Dependencies/ and inc/classes/Dependencies/... and rely on prefixed vendor output.
  • Update bundled Action Scheduler to 3.9.3, including schema/index changes and new recurring-action support.

Reviewed changes

Copilot reviewed 71 out of 74 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
composer.json Removes Mozart, configures Strauss prefixing, and updates install/update scripts to run prefixing.
phpstan.neon.dist Stops excluding the old classes/Dependencies/ path from analysis.
inc/Dependencies/ActionScheduler/** Updates Action Scheduler to 3.9.3 and introduces new scheduling/DB-claiming changes.
classes/Admin/PluginFamilySubscriber.php Expands delegated methods to match updated PluginFamily surface area after dependency changes.
.github/workflows/*.yml Ensures Strauss is present and runs Composer with scripts so prefixing occurs in CI; updates test matrix/steps.
.gitignore, .gitattributes, .distignore Adjust ignore/export rules for new tooling artifacts (e.g., bin/strauss.phar, lock files).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread composer.json
@remyperona remyperona added this pull request to the merge queue Mar 2, 2026
Merged via the queue into develop with commit 4679d42 Mar 2, 2026
5 of 25 checks passed
@remyperona remyperona deleted the chore/strauss branch March 2, 2026 15:02
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.

2 participants