fix(chart): PerUserDriveChart.update で userId が null のシステム所有ファイルをスキップ

[sasagar]

Fixes #17498

概要

PerUserDriveChart.update(file, isAdditional) が Chart.commit() の group 引数として file.userId をそのまま渡しているため、MiDriveFile.userId が null のファイル（インスタンスアイコン/バナー、admin エンドポイント経由のカスタム絵文字画像アップロード、フェッチされたシステムアバター等のシステム所有ファイル）に対する更新で、__chart__per_user_drive.group の varchar(128) NOT NULL 制約違反となり ChartManagementService の保存タイマーから例外が出ていました。

変更内容

• update() の冒頭で file.userId == null の場合は早期 return（no-op）。
• 所有者の無いファイルに対する「ユーザーごとの」ドライブ集計は定義できないため、行を作らないのが妥当という判断です。

CHANGELOG.md

Unreleased > Server に追記しています。

動作確認

• 修正前: __chart__per_user_drive への INSERT 時に null value in column "group" で例外。
• 修正後: file.userId == null の入力は事前にスキップされ、commit() 自体が呼ばれない（他ユーザー所有ファイルの集計は従来通り）。

影響

• 既存のユーザー所有ファイルに対する chart 動作は不変。
• システム所有ファイルは元々ユーザーに紐付かないので per-user 集計の対象外として扱う、というのが直感的・破壊的でない選択肢です。

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 16.96%. Comparing base (2e15942) to head (78e4ab3).
⚠️ Report is 1 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #17499      +/-   ##
===========================================
+ Coverage    15.21%   16.96%   +1.75%     
===========================================
  Files          247     1160     +913     
  Lines        12360    39576   +27216     
  Branches      4196    11022    +6826     
===========================================
+ Hits          1880     6714    +4834     
- Misses        8203    26245   +18042     
- Partials      2277     6617    +4340     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[github-actions]

このPRによるapi.jsonの差分
差分はありません。
Get diff files from Workflow Page

[github-actions]

Backend memory usage comparison

Before GC

Metric	base (MB)	head (MB)	Diff (MB)	Diff (%)
VmRSS	306.06 MB	306.27 MB	+0.21 MB	+0.06%
VmHWM	306.06 MB	306.27 MB	+0.21 MB	+0.06%
VmSize	23171.54 MB	23171.78 MB	0.23 MB	0%
VmData	1371.14 MB	1371.57 MB	+0.42 MB	+0.03%

After GC

Metric	base (MB)	head (MB)	Diff (MB)	Diff (%)
VmRSS	306.12 MB	306.45 MB	+0.33 MB	+0.10%
VmHWM	306.12 MB	306.45 MB	+0.33 MB	+0.10%
VmSize	23171.63 MB	23172.20 MB	0.56 MB	0%
VmData	1371.22 MB	1371.98 MB	+0.76 MB	+0.05%

After Request

Metric	base (MB)	head (MB)	Diff (MB)	Diff (%)
VmRSS	306.51 MB	306.92 MB	+0.40 MB	+0.13%
VmHWM	306.51 MB	306.92 MB	+0.40 MB	+0.13%
VmSize	23171.63 MB	23172.28 MB	0.65 MB	0%
VmData	1371.30 MB	1372.15 MB	+0.84 MB	+0.06%

See workflow logs for details

[syuilo]

🙏