8000 [StimulusBundle] Make the JS package private by Kocal · Pull Request #2709 · symfony/ux · GitHub
[go: up one dir, main page]

Skip to content

[StimulusBundle] Make the JS package private #2709

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 3, 2025

Conversation

Kocal
Copy link
Member
@Kocal Kocal commented May 3, 2025
Q A
Bug fix? yes
New feature? no
Docs? yes
Issues Fix #...
License MIT

This PR follows #2708

The npm package @symfony/stimulus-bundle was accidentally published on npm when I was working on the "Symfony UX on NPM" project, but when I realized it was to late to unpublish it.

Today, I was able to unpublish the package from npm. I'm now making the package private, so it won't be released anymore (until a solution is implemented)

I would be very happy to re-publish this package again when #2708 (comment) will be fixed.

@carsonbot carsonbot added Bug Bug Fix StimulusBundle Status: Needs Review Needs to be reviewed labels May 3, 2025
@Kocal Kocal merged commit 182f01f into symfony:2.x May 3, 2025
77 of 82 checks passed
@Kocal Kocal deleted the stimulus-bundle-npm-private branch May 3, 2025 21:32
@smnandre
Copy link
Member
smnandre commented May 4, 2025

Can you explain the scenario where there is a problem ?

I'm not sure to follow here.

AFAIK, for AssetMapper there is no problem, as it is not concerned with renovate.

For webpack, the bridge deals with all this (on the project i'm working on, renovate works perfectly), and there is no entry for stimulus-bundle in the package.json

@Kocal
Copy link
Member Author
Kocal commented May 4, 2025

It's all explained here #2708 (comment), where the
StimulusLoaderJavascriptCompiler is not compatible with the loader.js comming from the node_modules, or worst, when the file does not even exist with the package on jsDelivr.

It's a temporary solution to prevent more issues.

For webpack, the bridge deals with all this (on the project i'm working on, renovate works perfectly), and there is no entry for stimulus-bundle in the package.json

Yes, but here we are talking about users that don't use Encore, but the AssetMapper and importmap (with @symfony/stimulus-bundle installed from npm or jsDelivr)

@smnandre
8000 Copy link
Member
smnandre commented May 4, 2025

with @symfony/stimulus-bundle installed from npm or jsDelivr

Packages have been published on npm to resolve the path problem in packages.json, preventing renovate/github action to work.

With asset mapper, not using packages.json this is not a problem.

Also, this situation should never happen IRL so let's not take to much time / energy on this :)

Kocal added a commit that referenced this pull request May 6, 2025
…Kocal)

This PR was merged into the 2.x branch.

Discussion
----------

[StimulusBundle] Make the JS package private (for real)

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Docs?         | no <!-- required for new features -->
| Issues        | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exist, explain below instead -->
| License       | MIT

<!--
Replace this notice by a description of your feature/bugfix.
This will help reviewers and should be a good start for the documentation.

Additionally (see https://symfony.com/releases):
 - Always add tests and ensure they pass.
 - For new features, provide some code snippets to help understand usage.
 - Features and deprecations must be submitted against branch main.
 - Update/add documentation as required (we can help!)
 - Changelog entry should follow https://symfony.com/doc/current/contributing/code/conventions.html#writing-a-changelog-entry
 - Never break backward compatibility (see https://symfony.com/bc).
-->

Following #2709 and failing checks like https://github.com/symfony/ux/actions/runs/14850493570/job/41693038644?pr=2713

Commits
-------

625bf5e [StimulusBundle] Make the JS package private (for real)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0