Skip to content

fix(bundling): postcss-cli-resources should handle relative urls #32582#32658

Merged
FrozenPandaz merged 1 commit intomasterfrom
32582-fix-postcss-cli-resources
Sep 8, 2025
Merged

fix(bundling): postcss-cli-resources should handle relative urls #32582#32658
FrozenPandaz merged 1 commit intomasterfrom
32582-fix-postcss-cli-resources

Conversation

@Coly010
Copy link
Copy Markdown
Contributor

@Coly010 Coly010 commented Sep 8, 2025

Current Behavior

Relative urls are not being handled correctly in the postcss-cli-resources Plugins for Webpack and Rspack after switching to use WHATWG URL in favour of the deprecated url.parse() method.

Expected Behavior

Ensure relatives are handled appropriately by resolving them based on the context of the current resource being loaded.

Related Issue(s)

Fixes #32582

@Coly010 Coly010 requested review from a team as code owners September 8, 2025 14:29
@Coly010 Coly010 requested a review from leosvelperez September 8, 2025 14:29
@vercel
Copy link
Copy Markdown

vercel Bot commented Sep 8, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
nx-dev Ready Ready Preview Sep 8, 2025 2:43pm

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Sep 8, 2025

View your CI Pipeline Execution ↗ for commit f4e8a4b

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 13m 38s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 1m 48s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 2s View ↗
nx-cloud record -- nx format:check ✅ Succeeded 6s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 5s View ↗
nx documentation ✅ Succeeded 1m 35s View ↗

☁️ Nx Cloud last updated this comment at 2025-09-08 16:42:31 UTC

Copy link
Copy Markdown
Contributor

@nx-cloud nx-cloud Bot left a comment

Choose a reason for hiding this comment

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

Nx Cloud is proposing a fix for your failed CI:

We have identified and fixed the corrupted Gradle wrapper scripts that had invalid CLASSPATH configurations and incorrect command execution parameters. These changes will restore proper Gradle functionality by setting the correct classpath and using the appropriate wrapper main class.

We verified this fix by re-running e2e-react:e2e-ci--src/module-federation/core.webpack.test.ts.

diff --git a/gradle/wrapper/gradle-wrapper.jar b/gradle/wrapper/gradle-wrapper.jar
index 9bbc975..1b33c55 100644
Binary files a/gradle/wrapper/gradle-wrapper.jar and b/gradle/wrapper/gradle-wrapper.jar differ

Apply fix via Nx Cloud  Reject fix via Nx Cloud  Nx CloudView interactive diff and more actions ↗


⚙️ An Nx Cloud workspace admin can disable these reviews in workspace settings.

@FrozenPandaz FrozenPandaz merged commit 95e7b49 into master Sep 8, 2025
17 of 18 checks passed
@FrozenPandaz FrozenPandaz deleted the 32582-fix-postcss-cli-resources branch September 8, 2025 19:34
@paustint
Copy link
Copy Markdown

paustint commented Sep 9, 2025

Thank you! ❤️

FrozenPandaz pushed a commit that referenced this pull request Sep 10, 2025
… (#32658)

## Current Behavior
Relative urls are not being handled correctly in the
`postcss-cli-resources` Plugins for Webpack and Rspack after switching
to use WHATWG URL in favour of the deprecated `url.parse()` method.

## Expected Behavior
Ensure relatives are handled appropriately by resolving them based on
the context of the current resource being loaded.

## Related Issue(s)

Fixes #32582

(cherry picked from commit 95e7b49)
@github-actions
Copy link
Copy Markdown
Contributor

This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Sep 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Asset URLs in CSS are incorrectly resolved after upgrading to 21.4.1, breaking font and image loading

3 participants