Skip to content

Conversation

@Timeless0911
Copy link
Contributor

Summary

Support emitting .d.cts.map and .d.mts.map files。

  • declaration maps file names
  • declaration maps file field corresonding to source file name
  • declaration file mapping url links

Related Links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copilot AI review requested due to automatic review settings August 19, 2025 10:05
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for emitting .d.cts.map and .d.mts.map declaration map files alongside CommonJS and ES module declaration files. The implementation ensures proper file naming, source mapping URLs, and file field references in the map files.

  • Adds logic to rename declaration map files with appropriate extensions (.d.cts.map, .d.mts.map)
  • Updates declaration map content to reference correct declaration file names
  • Modifies source mapping URLs in declaration files to point to correctly named map files

Reviewed Changes

Copilot reviewed 15 out of 16 changed files in this pull request and generated 1 comment.

File Description
packages/plugin-dts/src/tsc.ts Core implementation for handling declaration map file naming and content updates
tests/integration/dts/index.test.ts Comprehensive tests for declaration map functionality across different build modes
tests/integration/dts//declaration-map/ Test fixtures for bundle-false, build, and composite scenarios
Files not reviewed (1)
  • pnpm-lock.yaml: Language not supported

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@netlify
Copy link

netlify bot commented Aug 19, 2025

Deploy Preview for rslib ready!

Name Link
🔨 Latest commit 9d76780
🔍 Latest deploy log https://app.netlify.com/projects/rslib/deploys/68a44c7191b3350007b0c884
😎 Deploy Preview https://deploy-preview-1176--rslib.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@fi3ework fi3ework merged commit 5c8c66e into main Aug 19, 2025
14 checks passed
@fi3ework fi3ework deleted the feat/map branch August 19, 2025 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants