Skip to content
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

bug: Wrong sourcemap emitting #6207

Open
3 tasks done
Tallyb opened this issue Mar 17, 2025 · 0 comments
Open
3 tasks done

bug: Wrong sourcemap emitting #6207

Tallyb opened this issue Mar 17, 2025 · 0 comments
Labels
Bug: Validated This PR or Issue is verified to be a bug within Stencil Help Wanted

Comments

@Tallyb
Copy link
Contributor

Tallyb commented Mar 17, 2025

Prerequisites

Stencil Version

4.28.1/2 (maybe 0, but I cannot build in 0)

Current Behavior

In version 4.27.2 building dist output:

  • Setting sourcemap to false will not generate .js.map files and no reference is added to the js file (both esm and cjs).
  • Setting sourcemap to true will generate js.map files and a single reference is added to the js file.

In version 4.28.1

  • setting sourcemap to false will generate an empty js.map file that contains:
    {"sourcesContent":[]} and a reference is added to the js file
  • Setting sourcemap to true generates js map files and adding 2 references in the js file

Expected Behavior

The 4.27.2 behaviour is the correct one.

System Info

System: node 22.13.1
    Platform: darwin (24.3.0)
   CPU Model: Apple M4 Max (16 cpus)
    Compiler: /Users/tally/yoobic/yoobic-ng-6/node_modules/@stencil/core/compiler/stencil.js
       Build: 1741825324
     Stencil: 4.28.1 🐵
  TypeScript: 5.5.4
      Rollup: 4.34.9
      Parse5: 7.2.1
      jQuery: 4.0.0-pre
      Terser: 5.37.0

Steps to Reproduce

Build as described above and check the links

Code Reproduction URL

not needed. any build will do.

Additional Information

Rollup upstream issue, most likely. or this: https://github.com/stenciljs/core/pull/6187/files#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519R131

@Tallyb Tallyb changed the title bug: sourcemap link is emitted twice bug: Wrong sourcemap emitting Mar 18, 2025
@christian-bromann christian-bromann added Bug: Validated This PR or Issue is verified to be a bug within Stencil Help Wanted labels Mar 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug: Validated This PR or Issue is verified to be a bug within Stencil Help Wanted
Projects
None yet
Development

No branches or pull requests

2 participants