cucumber / cucumber-expressions

Human friendly alternative to Regular Expressions
MIT License
148 stars 51 forks source link

Missing source for source map #210

Closed unional closed 1 year ago

unional commented 1 year ago

👓 What did you see?

The source map reference the TypeScript source code but they are not included in the package.

e.g. in ./dist/esm/src/index.js.map:

{...,"sources":["../../../src/index.ts"],...}

But since the source are not included in the package, when bundling with webpack, it fails with:

WARNING in ../../node_modules/.pnpm/@cucumber+cucumber-expressions@16.1.2/node_modules/@cucumber/cucumber-expressions/dist/esm/src/index.js
Module Warning (from ../../node_modules/.pnpm/source-map-loader@4.0.1_webpack@5.75.0/node_modules/source-map-loader/dist/cjs.js):
Failed to parse source map from '...\node_modules\.pnpm\@cucumber+cucumber-expressions@16.1.2\node_modules\@cucumber\cucumber-expressions\src\index.ts' file: Error: ENOENT: no such file or directory, open '...\node_modules\.pnpm\@cucumber+cucumber-expressions@16.1.2\node_modules\@cucumber\cucumber-expressions\src\index.ts'

✅ What did you expect to see?

Include the TypeScript source code in the package. i.e.:

// package.json
{
  "files": [
    ...,
    "src"  // <-- add this
  ]
}

📦 Which tool/library version are you using?

@cucumber/cucumber-expressions: 16.1.2

🔬 How could we reproduce it?

This involves webpack, will need to find time to create a repro. But the issue itself should be simple enough to understand directly.