test(e2e): period-lock作成後reloadと検索が競合して件数assertionがflaky

[ootakazuhiko]

背景

main merge commit d801cda3f52001ca758f0605f0e42e5032f691c0 の CI run 31398119408 で、既存 frontend-uiux-phase11-period-locks.spec.ts が1件失敗しました。

• full E2E: 153 PASS / 34 existing conditional SKIP / 1 FAIL
• 失敗箇所: 検索後の「1件を取得」assertion
• feat(knowledge): selective Chat share card and promote flow #2015 knowledge-share flow自体は同runでPASS
• PR feat(knowledge): add selective share and promote UI #2065 exact-headのcore E2Eでもperiod-lock testはPASS

確認済み事実

失敗時のsanitized DOMでは、対象reasonのrowは1件存在し、filter表示も対象periodでしたが、summaryとtableには別periodの既存global lockを含む2件が残っていました。

現行testは次の順で操作しています。

1. 締め登録をclick
2. create後のreload完了を待たずfilterをfill
3. 検索をclick
4. target rowと総件数1をassert

createLock()はPOST後に旧filter closureでloadLocks()を実行します。testが直後に開始するfiltered loadLocks()と競合し、unfiltered responseが後着するとfiltered resultを上書きできます。timeout延長では解消しません。

推奨修正

• create後のreloadが完了し、対象rowが表示されたことを待ってからfilter/searchを開始する
• filtered search完了後、対象rowとsummary countを検証する
• focused testを反復し、full E2Eでも確認する
• timeout延長、skip、件数assertion削除だけで隠さない

非対象

このfailureはIssue #2015のKnowledge/Chat share実装とは独立しているため、merged PR #2065へ追加変更を混ぜません。

[ootakazuhiko]

main CI run 31398119408 を同一merge commit d801cda3f52001ca758f0605f0e42e5032f691c0 のままworkflow全体で1回再実行しました。

• attempt 2: SUCCESS
• full E2E: 154 PASS / 34 existing conditional SKIP / 0 FAIL
• period-lock test: PASS

ただしattempt 1 artifactで、create後のunfiltered reloadがfiltered searchを後着上書きできる競合を確認しているため、本Issueは再現しなかったことだけを理由にcloseしません。timeout延長やassertion削除ではなく、create reload完了を明示的に待つ修正と反復検証が必要です。

[ootakazuhiko]

開始時のread-only監査と実装契約を固定します。

• baseline: d801cda3f52001ca758f0605f0e42e5032f691c0
• root cause: 登録後reloadと検索reloadが並行し、先に開始したunfiltered responseが後から現在のfiltered stateを上書きできる
• runtime方針: 一覧取得をgeneration + AbortControllerでlatest-request-wins化し、最新requestだけがitems / summary / error / loadingを更新する。登録・解除後reloadは呼出時のfilter snapshotを明示し、unmount時は進行中requestを無効化する
• E2E方針: 登録完了と登録後reload完了・対象row表示を待ってからfilter検索を開始し、filtered request完了後にrow数とsummaryを検証する。固定sleep、timeout延長、assertion削除、skipは行わない
• 決定的回帰test: unfiltered reloadを保留し、filtered responseを先に確定、その後unfiltered response/errorを確定してもfiltered row・1件summary・success状態が維持されることを検証する
• 検証: focused unit/component、focused E2E 20回、frontend/backend full、core/extended E2E、lint/format/typecheck/build/test/audit/ops-quality、build budget、secret scan、diff check
• privacy/security: API契約・認可・監査契約は変更しない
• rollback: frontendのrequest generation/abort制御とE2E待機/assertion差分を戻す。DB/API変更はない
• 非対象: PR build(deps): bump actions/setup-node from 6.4.0 to 7.0.0 #2064、dependency更新、backend API変更、timeout調整

実装は専用branch fix/2066-period-lock-latest-request-wins で進めます。

[ootakazuhiko]

完了

PR #2067をmergeし、Issue #2066のraceをruntime側で根治しました。

• source head: e11c57cc4e580cf8094f5a8d602b6545c71cdf86
• merge commit: 8a287700254b4be5215ab60382400fd174b066f6
• latest-request-wins: generation + AbortController
• unmount後のPOST/DELETE完了から追加reloadを開始しない
• 後発検索成功後のstale success/error/loadingおよびstale DELETE errorを破棄
• create後reloadとfiltered reloadをE2Eで明示待機
• timeout延長、fixed sleep、skip、assertion削除なし

検証:

• deterministic frontend focused: 11/11
• focused real-backend E2E反復: 20/20
• core E2E: 108/108
• full E2E: 154 passed / 34 existing conditional skipped / 0 failed
• backend full: 2168/2168
• frontend full: 104 test files PASS
• release-readiness: repo-side 29/29 PASS
• required CI / CodeQL / Link Check / security-audit / secret-scan: exact head PASS
• independent correctness/security blocker: 0
• Copilot actionable finding: 0
• review completeness: status=ok、unresolved thread 0
• 1時間cooling完了

main merge commitのCIはrun #31465408078、CodeQL #31465408136、Link Check #31465408128を継続確認します。