[codex] Pin axios to 1.14.0#17
Merged
Merged
Conversation
Contributor
Changelog PreviewThis PR contributes the following changelog bullets:
Checked on 2026-03-31T05:29:04.411Z |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This change removes the older Axios release that was still being pulled into production through transitive dependencies.
The app did not depend on Axios directly, but two packages in the shipped dependency tree,
@mendable/firecrawl-jsand@tavily/core, were both resolving toaxios@1.13.4. With an active Axios security concern in circulation, leaving that version in place would have kept the app exposed through those providers even though Axios was not listed in the top-level dependency list.The root cause was that the vulnerable version was nested inside third-party packages rather than declared directly by this project, so a normal top-level dependency review would not have caught it. The fix adds a pnpm override that forces Axios to resolve to
1.14.0everywhere in this repository, then refreshes the lockfile so both transitive consumers move to that version.I used
1.14.0intentionally because it is the current npmlatestrelease and the latest GitHub-tagged Axios release, which made it the safest stable target during an active supply-chain concern. I avoided the just-published1.14.1until it has had more time to settle.I verified the result by checking the resolved dependency tree and by running the project's type check.
pnpm why axiosnow shows both transitive consumers using1.14.0, andpnpm test:typescompleted successfully.Changelog