Conversation
…s and fix brace-expansion Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>
Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>
Co-authored-by: is0692vs <135803462+is0692vs@users.noreply.github.com>
…-2978128935635280895 chore(security): auto fix OSV-Scanner vulnerabilities
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
| - name: Create Pull Request | ||
| uses: peter-evans/create-pull-request@67df0571927e887e4d593f7a4b15a4b86b8d4fac # v6.0.6 | ||
| with: | ||
| commit-message: "chore(security): fix vulnerabilities detected by OSV-Scanner" | ||
| title: "chore(security): auto-fix vulnerabilities" | ||
| body: | | ||
| Automated PR to fix vulnerabilities detected by OSV-Scanner. | ||
|
|
||
| This PR was created automatically because the `OSV-Scanner` workflow failed on the `main` branch. | ||
| branch: "security/auto-fix-osv-vulnerabilities" | ||
| base: main | ||
| delete-branch: true |
There was a problem hiding this comment.
GITHUB_Token で作成した PR は CI がトリガーされない
token を指定していないため、peter-evans/create-pull-request はデフォルトで GITHUB_TOKEN を使用します。GITHUB_TOKEN で作成した PR に対しては、pull_request ベースの CI ワークフロー(lint・テスト等)が自動トリガーされません。セキュリティ修正の妥当性をCIで検証したい場合は、PAT または GitHub App トークンを用意して token に渡す必要があります。
Prompt To Fix With AI
This is a comment left during a code review.
Path: .github/workflows/osv-auto-fix.yml
Line: 45-56
Comment:
**GITHUB_Token で作成した PR は CI がトリガーされない**
`token` を指定していないため、`peter-evans/create-pull-request` はデフォルトで `GITHUB_TOKEN` を使用します。`GITHUB_TOKEN` で作成した PR に対しては、`pull_request` ベースの CI ワークフロー(lint・テスト等)が自動トリガーされません。セキュリティ修正の妥当性をCIで検証したい場合は、PAT または GitHub App トークンを用意して `token` に渡す必要があります。
How can I resolve this? If you propose a fix, please make it concise.| npm update brace-expansion | ||
|
|
||
| # Fix apps/web if present | ||
| if [ -d "apps/web" ]; then | ||
| npm --prefix apps/web install --ignore-scripts | ||
| npm --prefix apps/web audit fix | ||
| npm --prefix apps/web update brace-expansion | ||
| fi |
There was a problem hiding this comment.
brace-expansion のハードコードは将来の汎用性を下げる
npm update brace-expansion は今回の特定の CVE に対応するためハードコードされていますが、このワークフローは今後も別の脆弱性が検出されるたびに実行されます。別の脆弱性が対象になった場合でも毎回 brace-expansion を更新し続けることになり、意図しないバージョン更新が入り込む可能性があります。修正済みの脆弱性向けのステップは削除し、npm audit fix のみに絞ることを検討してください。
Prompt To Fix With AI
This is a comment left during a code review.
Path: .github/workflows/osv-auto-fix.yml
Line: 36-43
Comment:
**`brace-expansion` のハードコードは将来の汎用性を下げる**
`npm update brace-expansion` は今回の特定の CVE に対応するためハードコードされていますが、このワークフローは今後も別の脆弱性が検出されるたびに実行されます。別の脆弱性が対象になった場合でも毎回 `brace-expansion` を更新し続けることになり、意図しないバージョン更新が入り込む可能性があります。修正済みの脆弱性向けのステップは削除し、`npm audit fix` のみに絞ることを検討してください。
How can I resolve this? If you propose a fix, please make it concise.
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
chore: sync main into staging
Merging this PR will not alter performance
Comparing Footnotes
|
Summary
Promotes
stagingintomain.stagingmainnpm run pr:promoteIncluded PRs
Compare