Skip to content

ci: hold webpack below 5.112 for the Sylius 2.1 front-end build - #9

Merged
JDCyllene merged 3 commits into
CylleneDigital:mainfrom
JDCyllene:ci/pin-webpack-before-sylius-2.2
Oct 16, 2026
Merged

JDCyllene merged 3 commits into
CylleneDigital:mainfrom
JDCyllene:ci/pin-webpack-before-sylius-2.2

Conversation

@JDCyllene

Copy link
Copy Markdown
Member

What this pull request does

webpack 5.112.0 (released 2026-10-15) moved internals that the Stimulus bridge of Sylius 2.1's test application still loads: its front-end build stops on Cannot find module 'webpack/lib/dependencies/LoaderDependency', and that application has no lock file to hold the previous release. Every Sylius 2.1 job fails, whatever the change (seen on #8); Sylius 2.2 and 2.3 build fine.

The Sylius 2.1 jobs now give BuildTestAppAction no Node version, which makes it skip the front-end build, and build it themselves right after, with webpack held at ~5.111.0. The other jobs are untouched. The comment above those steps says when to remove them: once a webpack release loads the bridge again.

Points to watch

  • Tests pass (vendor/bin/phpunit, vendor/bin/behat --strict) - CI only; checked by this pull request's own run
  • PHPStan and ECS are green - no PHP change
  • UPGRADE.md is up to date if the public surface changes (routes, Twig hooks, templates, role, menu entry, services aliased over the engine) - nothing changes
  • A decision taken or reversed has its record in docs/adr/ - none: a temporary CI workaround
  • No API key, customer data or pre-production URL appears in the code, the tests or this description

webpack 5.112.0 moved internals that the Stimulus bridge of Sylius 2.1's
test application loads, and that application has no lock file: every
Sylius 2.1 job fails on its front-end build. Those jobs now build it
themselves, webpack held at ~5.111.0, until a webpack release loads the
bridge again.
@JDCyllene
JDCyllene requested a review from a team as a code owner October 16, 2026 06:15
…dency

A direct dependency hoisted webpack away from the CLI Encore runs it
with: the Sylius 2.1 build then asked to install webpack-cli.
The test application's preinstall script rebuilds package.json from
package.json.dist, which dropped the resolution: webpack 5.112 was still
installed.
@JDCyllene
JDCyllene merged commit eb2d89f into CylleneDigital:main Oct 16, 2026
18 checks passed
@JDCyllene
JDCyllene deleted the ci/pin-webpack-before-sylius-2.2 branch October 16, 2026 07:17
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