Skip to content

Release: staging -> main (2026-04-13)#419

Merged
is0692vs merged 7 commits into
mainfrom
staging
Apr 13, 2026
Merged

Release: staging -> main (2026-04-13)#419
is0692vs merged 7 commits into
mainfrom
staging

Conversation

@github-actions

@github-actions github-actions Bot commented Apr 13, 2026

Copy link
Copy Markdown

Summary

Promotes staging into main.

  • Source branch: staging
  • Target branch: main
  • Generated by npm run pr:promote

Included PRs

Compare

google-labs-jules Bot and others added 6 commits April 12, 2026 16:22
…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
@github-actions github-actions Bot added automated Automated main-to-staging sync PR release Release promotion PR labels Apr 13, 2026
@vercel

vercel Bot commented Apr 13, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
open-shelf Ignored Ignored Apr 13, 2026 3:49am

@dosubot dosubot Bot added dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code labels Apr 13, 2026
Comment on lines +45 to +56
- 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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 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.

Comment on lines +36 to +43
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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 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

codecov Bot commented Apr 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

chore: sync main into staging
@codspeed-hq

codspeed-hq Bot commented Apr 13, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 2 untouched benchmarks
⏩ 15 skipped benchmarks1


Comparing staging (0fd0979) with main (1364f9b)

Open in CodSpeed

Footnotes

  1. 15 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@is0692vs
is0692vs merged commit 7a37d4d into main Apr 13, 2026
35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated Automated main-to-staging sync PR dependencies Pull requests that update a dependency file github_actions Pull requests that update GitHub Actions code release Release promotion PR size/M

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant