Skip to content

Back out "Add createStaticCrawler" - #1886

Open
robhogan wants to merge 1 commit into
mainfrom
pr1886
Open

Back out "Add createStaticCrawler"#1886
robhogan wants to merge 1 commit into
mainfrom
pr1886

Conversation

@robhogan

Copy link
Copy Markdown
Collaborator

Original commit changeset: 3d0f468

This is an AI slop API I think I accidentally left published when I left Meta, and was landed alongside some other work.

There's no need for a helper like this to exist in core and even if there were, this isn't the right shape.

  • "Static file" isn't really a thing - the crawler API is meant to allow for virtual file systems, they're not necessarily static.
  • Needlessly takes its input in a different shape to the output and converts (essentially most of the function) - expensive on large file sets.
  • Reflect.set to avoid a Flow error.. .what.

This has never been released, so we can still remove it as non-breaking.

Changelog: Internal

Test plan:
CI

Original commit changeset: 3d0f468

This is an AI slop API I think I accidentally left published when I left Meta, and was landed alongside some other work.

There's no need for a helper like this to exist in core and even if there were, this isn't the right shape.

 - "Static file" isn't really a thing - the crawler API is meant to allow for virtual file systems, they're not necessarily static.
 - Needlessly takes its input in a different shape to the output and converts (essentially most of the function) - expensive on large file sets.
 - `Reflect.set` to avoid a Flow error.. .*what*.

This has never been released, so we can still remove it as non-breaking.

Changelog: Internal

Test plan:
CI
@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 Aug 28, 2026
@facebook-github-tools facebook-github-tools Bot added the Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. label Aug 28, 2026
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. Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant