fix(js): update vulnerable transitive dependencies - #662
Conversation
Co-authored-by: Codex <noreply@openai.com>
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
Files not reviewed (1)
- pkg/js/package-lock.json: Generated file
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
Lets hold these off @Siddhant-K-code Edit : Theres nothing immediately to release in pkg js in develop anyway so we can skip this for the moment |
We can separate it out as it is not needed to go in same PR!
It has security fixes & affecting PRs with CI failures |
|
It's not really required in the develop flow, If you want you can add it in main @Siddhant-K-code |
I agree but it won't matter until we release, if you want you can do it on main |
Description
Fixes the JavaScript dependency-audit failure observed in #652 by refreshing compatible transitive versions in
pkg/js/package-lock.json:brace-expansion: 1.1.18 and 5.0.9fast-uri: 3.1.5js-yaml: 4.3.1These are the first compatible versions outside the advisory ranges reported by npm. No direct dependency ranges or runtime code change.
Failing job: https://github.com/openfga/language/actions/runs/32035329850/job/95404310045?pr=652
Testing
Using the CI runtime versions (Node 24.19.0 and npm 11.17.0):
npm cinpm audit— 0 vulnerabilitiesnpm run lintnpm run format:checknpm run typechecknpm run build:lightnpm test -- --runInBand— 666 passed, 27 skipped