Skip to content

security: distinguish forward deepmerge remediation - #3231

Merged
azumag merged 4 commits into
mainfrom
security/3114-forward-remediation-status
Sep 9, 2026
Merged

azumag merged 4 commits into
mainfrom
security/3114-forward-remediation-status

Conversation

@azumag

@azumag azumag commented Sep 9, 2026

Copy link
Copy Markdown
Owner

Summary

Safety

This does not change dependencies, the lockfile, the audit allowlist, severity gates, application behavior, database behavior, or deployment configuration. All non-active status states still exit non-zero. A remediation candidate still requires the canonical security audit and full CI before the temporary exception can be removed.

Tracks #3114.

@github-actions
github-actions Bot enabled auto-merge (squash) September 9, 2026 17:57

@azumag azumag left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

コード・設計・テスト・CIを確認しました。blocking finding はありません。

  • forward-remediation-candidate は generic lockfile/manifest precondition 通過後にのみ判定され、active 以外は引き続き non-zero exit のため security gate を緩めていません。
  • installed deepmerge-ts >=8.0.0 だけでは候補扱いせず、@prisma/config 自身の dependency edge も patched line へ移った場合だけ分類するため、consumer-side major override と upstream forward fix を区別できています。
  • prerelease/invalid/complex range は安全側へ倒し、既知比較可能な単純 range だけを扱う設計も妥当です。
  • 初回CIの Prettier failure は同じPRで修正済み。最終HEAD 9003f432 では lint、changed-file formatting、unit tests、security audit、Prisma/D1 migration parity、Cloudflare Workers build gate、review workflow がすべて成功しています。

依存・lockfile・temporary allowlist・severity gate・アプリ挙動は変更していないため、#3114 を前進させる読み取り専用診断として merge 可と判断します。

@azumag
azumag merged commit 234c11f into main Sep 9, 2026
6 checks passed
@azumag
azumag deleted the security/3114-forward-remediation-status branch September 9, 2026 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant