Skip to content

fix(builder): resolve azion polyfills package specifier to filesystem path in esbuild plugin#449

Merged
jcbsfilho merged 2 commits into
mainfrom
fix/resolve-azion-polyfills-path
Jul 2, 2026
Merged

fix(builder): resolve azion polyfills package specifier to filesystem path in esbuild plugin#449
jcbsfilho merged 2 commits into
mainfrom
fix/resolve-azion-polyfills-path

Conversation

@jcbsfilho

Copy link
Copy Markdown
Collaborator

This pull request addresses an issue with the ESBuild plugin for Azion polyfills by updating how package specifiers are resolved. The main change ensures that references to the @aziontech/builder/polyfills package are correctly resolved to their filesystem paths, improving compatibility and reliability during the build process.

ESBuild Plugin Improvements:

  • Updated the Azion polyfills ESBuild plugin to use Node's createRequire for resolving package specifiers to actual filesystem paths, ensuring correct module resolution for polyfills. [1] [2]
  • Added a patch changelog entry documenting the fix for resolving the Azion polyfills package specifier.

@jcbsfilho jcbsfilho requested a review from a team as a code owner July 1, 2026 19:00
@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

SCC Complexity Analysis

  • Title: fix(builder): resolve azion polyfills package specifier to filesystem path in esbuild plugin
  • Author: jcbsfilho
Metric Source Branch (fix/resolve-azion-polyfills-path) Target Branch (main) Difference
Complexity 2454 2454 0
Code 50638 50631 7

@pkg-pr-new

pkg-pr-new Bot commented Jul 1, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/aziontech/lib/@aziontech/builder@449

commit: 8e6f774

@jcbsfilho jcbsfilho merged commit d400288 into main Jul 2, 2026
11 checks passed
@jcbsfilho jcbsfilho deleted the fix/resolve-azion-polyfills-path branch July 2, 2026 13:11
@pablodiehl pablodiehl mentioned this pull request Jul 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants