Skip to content

Make Blob/File/URL modules Flow strict-local (#57728) - #57728

Closed
rubennorte wants to merge 2 commits into
react:mainfrom
rubennorte:export-D113763778
Closed

Make Blob/File/URL modules Flow strict-local (#57728)#57728
rubennorte wants to merge 2 commits into
react:mainfrom
rubennorte:export-D113763778

Conversation

@rubennorte

@rubennorte rubennorte commented Jul 28, 2026

Copy link
Copy Markdown
Member

Summary:

Upgrade the Blob-related modules (File, FileReader, URL, URLSearchParams) from flow to flow strict-local. These expose spec-mandated getters/setters, so they opt out of the unsafe-getters-setters lint with a scoped // flowlint unsafe-getters-setters:off directive. URL also received small behavior-preserving refactors (local variables instead of parameter reassignment, explicit null/empty checks).

Changelog: [Internal]

Reviewed By: javache

Differential Revision: D113763778

@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 Jul 28, 2026
@meta-codesync

meta-codesync Bot commented Jul 28, 2026

Copy link
Copy Markdown

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

rubennorte added a commit to rubennorte/react-native that referenced this pull request Jul 28, 2026
Summary:
Pull Request resolved: react#57728

Upgrade the Blob-related modules (`File`, `FileReader`, `URL`, `URLSearchParams`) from `flow` to `flow strict-local`. These expose spec-mandated getters/setters, so they opt out of the `unsafe-getters-setters` lint with a scoped `// flowlint unsafe-getters-setters:off` directive. `URL` also received small behavior-preserving refactors (local variables instead of parameter reassignment, explicit null/empty checks).

Changelog: [Internal]

Reviewed By: huntie

Differential Revision: D113763778
@meta-codesync meta-codesync Bot changed the title Make Blob/File/URL modules Flow strict-local Make Blob/File/URL modules Flow strict-local (#57728) Jul 28, 2026
rubennorte added a commit to rubennorte/react-native that referenced this pull request Jul 29, 2026
Summary:
Pull Request resolved: react#57728

Upgrade the Blob-related modules (`File`, `FileReader`, `URL`, `URLSearchParams`) from `flow` to `flow strict-local`. These expose spec-mandated getters/setters, so they opt out of the `unsafe-getters-setters` lint with a scoped `// flowlint unsafe-getters-setters:off` directive. `URL` also received small behavior-preserving refactors (local variables instead of parameter reassignment, explicit null/empty checks).

Changelog: [Internal]

Reviewed By: huntie

Differential Revision: D113763778
rubennorte added a commit to rubennorte/react-native that referenced this pull request Jul 29, 2026
Summary:
Pull Request resolved: react#57728

Upgrade the Blob-related modules (`File`, `FileReader`, `URL`, `URLSearchParams`) from `flow` to `flow strict-local`. These expose spec-mandated getters/setters, so they opt out of the `unsafe-getters-setters` lint with a scoped `// flowlint unsafe-getters-setters:off` directive. `URL` also received small behavior-preserving refactors (local variables instead of parameter reassignment, explicit null/empty checks).

Changelog: [Internal]

Reviewed By: huntie

Differential Revision: D113763778
@rubennorte
rubennorte force-pushed the export-D113763778 branch 2 times, most recently from bb18b2d to 900e6ce Compare July 29, 2026 15:19
rubennorte added a commit to rubennorte/react-native that referenced this pull request Jul 29, 2026
Summary:

Upgrade the Blob-related modules (`File`, `FileReader`, `URL`, `URLSearchParams`) from `flow` to `flow strict-local`. These expose spec-mandated getters/setters, so they opt out of the `unsafe-getters-setters` lint with a scoped `// flowlint unsafe-getters-setters:off` directive. `URL` also received small behavior-preserving refactors (local variables instead of parameter reassignment, explicit null/empty checks).

Changelog: [Internal]

Reviewed By: huntie

Differential Revision: D113763778
rubennorte added a commit to rubennorte/react-native that referenced this pull request Jul 29, 2026
Summary:

Upgrade the Blob-related modules (`File`, `FileReader`, `URL`, `URLSearchParams`) from `flow` to `flow strict-local`. These expose spec-mandated getters/setters, so they opt out of the `unsafe-getters-setters` lint with a scoped `// flowlint unsafe-getters-setters:off` directive. `URL` also received small behavior-preserving refactors (local variables instead of parameter reassignment, explicit null/empty checks).

Changelog: [Internal]

Reviewed By: huntie

Differential Revision: D113763778
Summary:
Pull Request resolved: react#57724

Upgrade the legacy native module spec files from `flow` to `flow strict-local` to enforce stricter local type checking. Loose `Object` types were replaced with the codegen-equivalent `UnsafeObject`, and `Array<any>` parameters with `Array<unknown>`, both of which are codegen-identical so the generated native interfaces are unchanged.

Changelog: [Internal]

Reviewed By: javache

Differential Revision: D113763785
rubennorte added a commit to rubennorte/react-native that referenced this pull request Sep 3, 2026
Summary:

Upgrade the Blob-related modules (`File`, `FileReader`, `URL`, `URLSearchParams`) from `flow` to `flow strict-local`. These expose spec-mandated getters/setters, so they opt out of the `unsafe-getters-setters` lint with a scoped `// flowlint unsafe-getters-setters:off` directive. `URL` also received small behavior-preserving refactors (local variables instead of parameter reassignment, explicit null/empty checks).

Changelog: [Internal]

Reviewed By: javache

Differential Revision: D113763778
rubennorte added a commit to rubennorte/react-native that referenced this pull request Sep 3, 2026
Summary:
Pull Request resolved: react#57728

Upgrade the Blob-related modules (`File`, `FileReader`, `URL`, `URLSearchParams`) from `flow` to `flow strict-local`. These expose spec-mandated getters/setters, so they opt out of the `unsafe-getters-setters` lint with a scoped `// flowlint unsafe-getters-setters:off` directive. `URL` also received small behavior-preserving refactors (local variables instead of parameter reassignment, explicit null/empty checks).

Changelog: [Internal]

Reviewed By: javache

Differential Revision: D113763778
rubennorte added a commit to rubennorte/react-native that referenced this pull request Sep 3, 2026
Summary:
Pull Request resolved: react#57728

Upgrade the Blob-related modules (`File`, `FileReader`, `URL`, `URLSearchParams`) from `flow` to `flow strict-local`. These expose spec-mandated getters/setters, so they opt out of the `unsafe-getters-setters` lint with a scoped `// flowlint unsafe-getters-setters:off` directive. `URL` also received small behavior-preserving refactors (local variables instead of parameter reassignment, explicit null/empty checks).

Changelog: [Internal]

Reviewed By: javache

Differential Revision: D113763778
Summary:
Pull Request resolved: react#57728

Upgrade the Blob-related modules (`File`, `FileReader`, `URL`, `URLSearchParams`) from `flow` to `flow strict-local`. These expose spec-mandated getters/setters, so they opt out of the `unsafe-getters-setters` lint with a scoped `// flowlint unsafe-getters-setters:off` directive. `URL` also received small behavior-preserving refactors (local variables instead of parameter reassignment, explicit null/empty checks).

Changelog: [Internal]

Reviewed By: javache

Differential Revision: D113763778
rubennorte added a commit to rubennorte/react-native that referenced this pull request Sep 4, 2026
Summary:
Pull Request resolved: react#57728

Upgrade the Blob-related modules (`File`, `FileReader`, `URL`, `URLSearchParams`) from `flow` to `flow strict-local`. These expose spec-mandated getters/setters, so they opt out of the `unsafe-getters-setters` lint with a scoped `// flowlint unsafe-getters-setters:off` directive. `URL` also received small behavior-preserving refactors (local variables instead of parameter reassignment, explicit null/empty checks).

Changelog: [Internal]

Reviewed By: javache

Differential Revision: D113763778
@meta-codesync meta-codesync Bot closed this in c72e8c7 Sep 4, 2026
@meta-codesync meta-codesync Bot added the Merged This PR has been merged. label Sep 4, 2026
@meta-codesync

meta-codesync Bot commented Sep 4, 2026

Copy link
Copy Markdown

This pull request has been merged in c72e8c7.

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