Skip to content

[HIGH] Pin tmp to 0.2.7 - #58345

Open
OskarEichler wants to merge 1 commit into
react:mainfrom
OskarEichler:codex/security-tmp
Open

OskarEichler wants to merge 1 commit into
react:mainfrom
OskarEichler:codex/security-tmp

Conversation

@OskarEichler

@OskarEichler OskarEichler commented Sep 4, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • resolve external-editor’s tmp dependency to 0.2.7
  • remove the obsolete os-tmpdir transitive dependency
  • keep the change scoped to the existing Inquirer toolchain

Security impact

tmp 0.0.33 is affected by GHSA-ph9p-34f9-6g65, a high-severity path traversal through unsanitized prefix/postfix values, and GHSA-52f5-9888-hmc6, a symlink-based arbitrary temporary file/directory write. The targeted resolution is necessary because current external-editor still requests tmp ^0.0.33.

Test Plan

  • yarn install --frozen-lockfile --ignore-scripts
  • yarn why tmp confirmed only 0.2.7
  • yarn audit no longer reports tmp advisories
  • temporary-file create/remove lifecycle check
  • yarn build
  • git diff --check

Changelog:

[INTERNAL] [SECURITY] - Pin tmp to 0.2.7.

@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 Sep 4, 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 Sep 4, 2026

This branch has not been deployed

No deployments
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