Skip to content

fix: embed sources in emitted sourcemaps#3274

Open
raashish1601 wants to merge 1 commit into
redis:masterfrom
raashish1601:codex/3147-inline-sources
Open

fix: embed sources in emitted sourcemaps#3274
raashish1601 wants to merge 1 commit into
redis:masterfrom
raashish1601:codex/3147-inline-sources

Conversation

@raashish1601
Copy link
Copy Markdown
Contributor

@raashish1601 raashish1601 commented May 15, 2026

Summary

Fixes #3147.

This updates the shared TypeScript build configuration to emit sourcesContent in generated source maps. The current generated maps include paths like ../index.ts, but the published package does not include those source files, so debugger tooling cannot resolve the original TypeScript sources from the installed package alone.

Validation

  • .\node_modules\.bin\tsc.cmd --build packages/client --force --verbose
  • Inspected packages/client/dist/index.js.map and confirmed sourcesContent is present
  • npm run build
  • npm run lint:changed
  • git diff --check

Note

Low Risk
Low risk configuration change that only affects TypeScript build outputs by embedding sourcesContent in generated source maps.

Overview
Enables debuggers to resolve original TypeScript sources from installed packages by setting inlineSources: true in tsconfig.base.json, causing generated .map files to include sourcesContent.

Reviewed by Cursor Bugbot for commit 098cad6. Bugbot is set up for automated code reviews on this repo. Configure here.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sourcemap references missing source files

1 participant