Skip to content

lib: remove source map deadcode in type stripping#63738

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:module-remove-sourcemap-dead-code
Jun 8, 2026
Merged

lib: remove source map deadcode in type stripping#63738
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
legendecas:module-remove-sourcemap-dead-code

Conversation

@legendecas

@legendecas legendecas commented Jun 3, 2026

Copy link
Copy Markdown
Member

processTypeScriptCode does not produce source maps at all for type
stripping. Given that the transform mode was removed, there is no need
to keep the source map support around it.

Refs: #61803

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/loaders

@nodejs-github-bot nodejs-github-bot added module Issues and PRs related to the module subsystem. needs-ci PRs that need a full CI run. labels Jun 3, 2026
@legendecas legendecas added dont-land-on-v22.x PRs that should not land on the v22.x-staging branch and should not be released in v22.x. dont-land-on-v24.x PRs that should not land on the v24.x-staging branch and should not be released in v24.x. labels Jun 3, 2026
@legendecas legendecas added the request-ci Add this label to start a Jenkins CI on a PR. label Jun 4, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 4, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@mcollina mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

`processTypeScriptCode` does not produce source maps at all for type
stripping. Given that the transform mode was removed, there is no need
to keep the source map support around it.

Signed-off-by: Chengzhong Wu <cwu631@bloomberg.net>
@legendecas legendecas force-pushed the module-remove-sourcemap-dead-code branch from da0e566 to 0e851e7 Compare June 5, 2026 13:15
@legendecas

Copy link
Copy Markdown
Member Author

Rebased to resolve conflicts.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@legendecas legendecas added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Jun 8, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Jun 8, 2026
@nodejs-github-bot nodejs-github-bot merged commit 1b4e0fd into nodejs:main Jun 8, 2026
71 of 72 checks passed
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Landed in 1b4e0fd

@legendecas legendecas deleted the module-remove-sourcemap-dead-code branch June 8, 2026 19:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. dont-land-on-v22.x PRs that should not land on the v22.x-staging branch and should not be released in v22.x. dont-land-on-v24.x PRs that should not land on the v24.x-staging branch and should not be released in v24.x. module Issues and PRs related to the module subsystem. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants