Conversation
完全にuuidパッケージの問題を解決する手段の、 [Update uuid to remove a security alert by thewilkybarkid · Pull Request #6199 · Effect-TS/effect](Effect-TS/effect#6199) はリリース待ちのようです。 このuuidのアップデートが修正するのはuuidのかなりマイナーなバリアントでのバグで、 このプロジェクトではそのuuidのバージョンを使っていないため、 あまり関係ないと判断して、 そこまで急がず、 上流でマージされた後順当に解決するのを待つことにします。
Contributor
There was a problem hiding this comment.
npm audit fixによるpackage-lock.jsonのパッチバージョン更新です。
plugins/commit:brace-expansion5.0.4 -> 5.0.5 (devDependency経由のminimatchの推移的依存)plugins/kyosei:uuid11.1.0 -> 11.1.1 (@effect/experimentalおよび@effect/sqlのpeer dependency経由)
どちらもパッチレベルのバグ修正リリースで、間接依存関係のロックファイル更新のみ。package.jsonの宣言依存への影響はなし。
コミットメッセージに記載の通り、uuidに関するEffect-TS側の根本的な解決(Effect-TS/effect#6199)はリリース待ちであり、当面の暫定対応としてnpm audit fixを適用する判断は妥当です。実際にこのプロジェクトで使われていないバリアントのバグ修正のため、影響範囲も限定的という判断にも同意します。
プラグインバージョンは更新不要です(マーケットプレイスの内部依存パッチ更新のため、ユーザー観点での挙動変化はない)。
指摘事項なし、approveします。
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.
完全にuuidパッケージの問題を解決する手段の、
Update uuid to remove a security alert by thewilkybarkid · Pull Request #6199 · Effect-TS/effect
はリリース待ちのようです。
このuuidのアップデートが修正するのはuuidのかなりマイナーなバリアントでのバグで、
このプロジェクトではそのuuidのバージョンを使っていないため、
あまり関係ないと判断して、
そこまで急がず、
上流でマージされた後順当に解決するのを待つことにします。