Skip to content

Remove unused types/*.d.ts export subpath#57362

Closed
huntie wants to merge 1 commit into
react:mainfrom
huntie:export-D110044754
Closed

Remove unused types/*.d.ts export subpath#57362
huntie wants to merge 1 commit into
react:mainfrom
huntie:export-D110044754

Conversation

@huntie

@huntie huntie commented Jun 29, 2026

Copy link
Copy Markdown
Member

Summary:
The "./types/*.d.ts" entry was an overly defensive addition during our non-breaking "exports" field addition back in 0.80.

In reality, this subpath has no real consumers: the legacy types entry point (.) resolves its own declaration files through relative references, and the supported deep-import surface is react-native/Libraries/*, not react-native/types/*.

Type resolution of the main entry point and of react-native/Libraries/* deep imports is unaffected, and there is no runtime impact (these are declaration files).

Follows the exports-map cleanup in #57276.

Changelog: [Internal]

Differential Revision: D110044754

Summary:
The `"./types/*.d.ts"` entry was an overly defensive addition during our non-breaking `"exports"` field addition back in 0.80.

In reality, this subpath has no real consumers: the legacy types entry point (`.`) resolves its own declaration files through relative references, and the supported deep-import surface is `react-native/Libraries/*`, not `react-native/types/*`.

Type resolution of the main entry point and of `react-native/Libraries/*` deep imports is unaffected, and there is no runtime impact (these are declaration files).

Follows the exports-map cleanup in react#57276.

Changelog: [Internal]

Differential Revision: D110044754
@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 29, 2026
@meta-codesync

meta-codesync Bot commented Jun 29, 2026

Copy link
Copy Markdown

@huntie has exported this pull request. If you are a Meta employee, you can view the originating Diff in D110044754.

@meta-codesync meta-codesync Bot closed this in 8638a19 Jun 29, 2026
@meta-codesync meta-codesync Bot added the Merged This PR has been merged. label Jun 29, 2026
@meta-codesync

meta-codesync Bot commented Jun 29, 2026

Copy link
Copy Markdown

This pull request has been merged in 8638a19.

@huntie huntie deleted the export-D110044754 branch June 29, 2026 13:36
meta-codesync Bot pushed a commit that referenced this pull request Jun 29, 2026
Summary:
Pull Request resolved: #57366

Follows #57362 (accidentally missed).

Simplifying our `"exports"` map to its minimum form will help for the incoming review/understandability of the Strict TS API rollout by default.

Changelog: [Internal]

landed-with-radar-review

Reviewed By: robhogan

Differential Revision: D110055788

fbshipit-source-id: 83919aec45f3be18c08e983a751de1117c237171
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant