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

Incorrect Paths in Sourcemap sources Introduced in Version 0.25.0 #4070

Closed
alan-agius4 opened this issue Feb 11, 2025 · 3 comments
Closed

Incorrect Paths in Sourcemap sources Introduced in Version 0.25.0 #4070

alan-agius4 opened this issue Feb 11, 2025 · 3 comments

Comments

@alan-agius4
Copy link

Starting from version 0.25.0, TypeScript compilations using TSC now include only the filename in sourcemaps instead of the full file path.

Reproduction:

https://esbuild.github.io/try/#YgAwLjI0LjIALS1idW5kbGUgLS1taW5pZnkgLS1zb3VyY2VtYXAgLS1vdXRkaXI9Ii8iIC0tcGFja2FnZXM9ZXh0ZXJuYWwgLS1hYnMtd29ya2luZy1kaXI9Ii8iAABlbnRyeS5qcwAAZQBzcmMvbWFpbi5qcwBpbXBvcnQgeyBhcHBDb25maWcgfSBmcm9tICcuL2FwcC9hcHAuY29uZmlnJzsKY29uc29sZS5sb2coYXBwQ29uZmlnKTsKCi8vIHsidmVyc2lvbiI6MywiZmlsZSI6Im1haW4uanMiLCJzb3VyY2VSb290IjoiIiwic291cmNlcyI6WyJtYWluLnRzIl0sIm5hbWVzIjpbXSwibWFwcGluZ3MiOiJBQUNBLE9BQU8sRUFBRSxTQUFTLEVBQUUsTUFBTSxrQkFBa0IsQ0FBQztBQUM3QyxPQUFPLENBQUMsR0FBRyxDQUFDLFNBQVMsQ0FBQyxDQUFBIn0KCi8vIyBzb3VyY2VNYXBwaW5nVVJMPWRhdGE6YXBwbGljYXRpb24vanNvbjtiYXNlNjQsZXlKMlpYSnphVzl1SWpvekxDSm1hV3hsSWpvaWJXRnBiaTVxY3lJc0luTnZkWEpqWlZKdmIzUWlPaUlpTENKemIzVnlZMlZ6SWpwYkltMWhhVzR1ZEhNaVhTd2libUZ0WlhNaU9sdGRMQ0p0WVhCd2FXNW5jeUk2SWtGQlEwRXNUMEZCVHl4RlFVRkZMRk5CUVZNc1JVRkJSU3hOUVVGTkxHdENRVUZyUWl4RFFVRkRPMEZCUXpkRExFOUJRVThzUTBGQlF5eEhRVUZITEVOQlFVTXNVMEZCVXl4RFFVRkRMRU5CUVVFaWZRPT0AAHNyYy9hcHAvYXBwLmNvbmZpZy5qcwBleHBvcnQgY29uc3QgYXBwQ29uZmlnID0ge307CgovLyB7InZlcnNpb24iOjMsImZpbGUiOiJhcHAuY29uZmlnLmpzIiwic291cmNlUm9vdCI6IiIsInNvdXJjZXMiOlsiYXBwLmNvbmZpZy50cyJdLCJuYW1lcyI6W10sIm1hcHBpbmdzIjoiQUFDQSxNQUFNLENBQUMsTUFBTSxTQUFTLEdBQUksRUFBRSxDQUFBIn0KCi8vIyBzb3VyY2VNYXBwaW5nVVJMPWRhdGE6YXBwbGljYXRpb24vanNvbjtiYXNlNjQsZXlKMlpYSnphVzl1SWpvekxDSm1hV3hsSWpvaVlYQndMbU52Ym1acFp5NXFjeUlzSW5OdmRYSmpaVkp2YjNRaU9pSWlMQ0p6YjNWeVkyVnpJanBiSW1Gd2NDNWpiMjVtYVdjdWRITWlYU3dpYm1GdFpYTWlPbHRkTENKdFlYQndhVzVuY3lJNklrRkJRMEVzVFVGQlRTeERRVUZETEUxQlFVMHNVMEZCVXl4SFFVRkpMRVZCUVVVc1EwRkJRU0o5

Comparison

Version 0.24.x

{
  "version": 3,
  "sources": ["src/app/app.config.ts", "src/main.ts"],
  "sourcesContent": [null, null],
  "mappings": "MACO,IAAMA,EAAa,CAAA,ECC1B,QAAQ,IAAIC,CAAS",
  "names": ["appConfig", "appConfig"]
}

Version 0.25.0

{
  "version": 3,
  "sources": ["app.config.ts", "main.ts"],
  "sourcesContent": [null, null],
  "mappings": "MACO,IAAMA,EAAa,CAAA,ECC1B,QAAQ,IAAIC,CAAS",
  "names": ["appConfig", "appConfig"]
}
@ArcherGu
Copy link

ArcherGu commented Mar 3, 2025

same issue 😢

@evanw
Copy link
Owner

evanw commented Mar 9, 2025

Thanks for including a straightforward repro case the report. Very helpful! Sorry about the regression. It will be fixed in the next release.

@WebDevCaptain
Copy link

Thanks Evan, i noticed this yesterday.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants