Skip to content

chore(deps): EF Core を 9.0.20 へ更新し、2 つのロックファイルを同時に再生成する - #231

Merged
izumacha merged 2 commits into
mainfrom
claude/dreamy-brown-0to41w
Sep 14, 2026
Merged

izumacha merged 2 commits into
mainfrom
claude/dreamy-brown-0to41w

Conversation

@izumacha

Copy link
Copy Markdown
Owner

Context

Dependabot の #229(nuget-ef-core グループ / EF Core 9.0.19 → 9.0.20)を、CLAUDE.md §3 の**「ロックファイルは全プロジェクト分を同時に再生成する」**手順で作り直したものです。#229 はそのままではマージできないため、本 PR で置き換えて #229 はクローズします。

#229 が落ちていた理由

build-and-test が locked-mode restore で失敗していました。

tests/IncidentInsight.Tests/IncidentInsight.Tests.csproj : error NU1004:
The project references incidentinsight.web whose dependencies has changed.

Dependabot は src/ 側のロックファイルしか整合させておらず、tests/IncidentInsight.Tests/packages.lock.json の incidentinsight.web セクションに Microsoft.EntityFrameworkCore.SqlServer: [9.0.19, ) が残っていました(同セクションの Sqlite だけが 9.0.20 に進んだ状態)。CLAUDE.md に記録済みの既知の形(実例: PR #180 → #181)です。

変更内容

  • Microsoft.EntityFrameworkCore.Design / .Sqlite / .SqlServer(web)と .InMemory / .Sqlite(tests)を 9.0.19 → 9.0.20。
  • dotnet restore --force-evaluate で src/ と tests/ の packages.lock.json を同一コミットで再生成。
  • Microsoft.Data.SqlClient のピン(5.2.3)は据え置き。コメントの追随のみ(下記)。

コメント / CLAUDE.md の追随

EF Core SqlServer が宣言する Microsoft.Data.SqlClient の版が 5.1.6 → 5.1.9 に上がるため、「現在 5.1.6」と書いていた箇所を 5.1.9 へ直しました。あわせて、床値検査を素通りする下げ方の例として挙げていた「revert して 5.1.7 に戻す」が 5.1.9 未満になり成立しなくなったので、5.2.0 へ下げる例に差し替えています(穴そのものは変わらず残っているため、記述は残す)。ピンの発端を述べた 1 文は当時の事実なので、履歴だと読める見出しを付けて残しました。

EfCorePackageAlignmentTests の判定は本 PR で緑のままです。

検査 結果
メジャー一致 ピン 5.2.3 と EF Core 宣言 5.1.9 — ともに major 5 ✅
床値の後退 5.2.3 >= 5.1.9 ✅
ピンが直接参照のまま ロックファイルの type は Direct ✅

検証

CLAUDE.md §2 / CI 設定に記載のコマンドをローカルで全実行しました。

影響範囲

パッチレベルのサービシング更新のみで、アプリのコード・スキーマ・マイグレーションに変更はありません。Microsoft.Data.SqlClient の実際の解決版も 5.2.3 のまま変わりません。

🤖 Generated with Claude Code

https://claude.ai/code/session_011fF5EaW46Nc1tL71swdRNL


Generated by Claude Code

Dependabot の PR #229(nuget-ef-core グループ)と同じ更新を、CLAUDE.md §3
「ロックファイルは全プロジェクト分を同時に再生成する」の手順で作り直したもの。

#229 は src/ 側のロックファイルしか整合させておらず、tests 側に残った
`Microsoft.EntityFrameworkCore.SqlServer: [9.0.19, )` のせいで CI の
locked-mode restore が NU1004 で落ちていた。ここでは `dotnet restore
--force-evaluate` を流し、src / tests 両方の packages.lock.json を同一
コミットに含めている。

Microsoft.Data.SqlClient のピン(5.2.3)はそのまま。EF Core SqlServer が
宣言する版が 5.1.6 → 5.1.9 へ上がるが、メジャーは 5 で一致し、床値も
5.2.3 >= 5.1.9 で後退しないため EfCorePackageAlignmentTests は緑のまま。
宣言版を「現在 5.1.6」と書いていた csproj のコメントと CLAUDE.md を
5.1.9 へ追随させ、あわせて「素通りする下げ方」の例(revert して 5.1.7)が
5.1.9 未満になって成立しなくなったため 5.2.0 へ差し替えた。

検証: dotnet restore --locked-mode / dotnet build(警告 0・エラー 0)/
dotnet test(838 件すべて成功)。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011fF5EaW46Nc1tL71swdRNL
@vercel

vercel Bot commented Sep 14, 2026 •

Copy link
Copy Markdown

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

Project Deployment Actions Updated
incident-insight Error Error Sep 14, 2026 12:26am UTC
incident-insight-jjt5 Error Error Sep 14, 2026 12:26am UTC

コードレビューで出た 3 点の手当て。いずれも本 PR の EF Core 更新で
説明と実態がずれた/ずれていたことが見えた箇所。

1. csproj のコメントが「床値は Dependabot の minor / patch で引き上げていく」と
   書いていたが、この経路は塞がっている。Microsoft.Data.SqlClient の 5.2 は STS で
   5.2.3(2025-04-29)が最後の公開版、5.1 は LTS として 5.1.9(2026-01-13)まで
   保守が続いている(NuGet の公開一覧・公開日で確認)。5.2.x の minor / patch は
   もう出ず、残る 5.x → 6.x は dependabot.yml の ignore が止めるため床値は動かない。
   さらに EF Core が宣言する版の方が新しくなったので、このピンは後退の防止ではなく
   新しいサービシングの押しとどめになっている。数値上は 5.2.3 >= 5.1.9 が成立し
   どの検査も緑のままなので、気付けるのはコメントだけ。事実をそのまま書き残す。
   ピンを 5.1 系へ移す変更自体は目的が別なので、別の変更セットで扱う。

2. EfCorePackageAlignmentTests のコメントに「現在 5.1.6」の写しが残っていた。
   csproj と CLAUDE.md だけを直すと、ピンをどこまで下げられるかを判断する人が
   最も見るであろうテスト側だけが古いままになる(§6 DRY が避けたい形)。

検証: dotnet build(警告 0・エラー 0)/ dotnet test(838 件すべて成功)。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011fF5EaW46Nc1tL71swdRNL

This branch had an error being deployed

2 failed deployments
Preview – incident-insight — 94bce2e1 Deployed Sep 14, 2026 by vercel[bot]
Preview – incident-insight-jjt5 — 94bce2e1 Deployed Sep 14, 2026 by vercel[bot]
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.

2 participants