Skip to content

Transform $Keys to keyof#55111

Closed
marcoww6 wants to merge 2 commits into
react:mainfrom
marcoww6:export-D90426061
Closed

Transform $Keys to keyof#55111
marcoww6 wants to merge 2 commits into
react:mainfrom
marcoww6:export-D90426061

Conversation

@marcoww6

Copy link
Copy Markdown
Contributor

Summary:
We are transforming the following utility types to be more consistent with typescript and better AI integration:

  • $NonMaybeType -> NonNullable
  • $ReadOnly -> Readonly
  • $ReadOnlyArray -> ReadonlyArray
  • $ReadOnlyMap -> ReadonlyMap
  • $ReadOnlySet -> ReadonlySet
  • $Keys -> keyof
  • $Values -> Values
  • mixed -> unknown

See details in https://fb.workplace.com/groups/flowlang/permalink/1837907750148213/.

drop-conflicts

Command:

js1 flow-runner codemod flow/transformUtilityType --format-files=false --legacy-type='$ReadOnlySet'

Reviewed By: SamChou19815

Differential Revision: D90426061

@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 Jan 10, 2026
@meta-codesync

meta-codesync Bot commented Jan 10, 2026

Copy link
Copy Markdown

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

Marco Wang and others added 2 commits January 12, 2026 14:30
Differential Revision: D90425806
Summary:
Pull Request resolved: react#55111

We are transforming the following utility types to be more consistent with typescript and better AI integration:

* `$NonMaybeType` -> `NonNullable`
* `$ReadOnly` -> `Readonly`
* `$ReadOnlyArray` -> `ReadonlyArray`
* `$ReadOnlyMap` -> `ReadonlyMap`
* `$ReadOnlySet` -> `ReadonlySet`
* `$Keys` -> `keyof`
* `$Values` -> `Values`
* `mixed` -> `unknown`

See details in https://fb.workplace.com/groups/flowlang/permalink/1837907750148213/.

drop-conflicts

Command:

`js1 flow-runner codemod flow/transformUtilityType --format-files=false --legacy-type='$Keys'`

Reviewed By: SamChou19815

Differential Revision: D90426061
@react-native-bot

Copy link
Copy Markdown
Collaborator

Closing this PR as the diff D90426061 has been abandoned internally.

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. fb-exported meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants