Skip to content

fix(store): 出品者パネルの切替後の余分な再取得とエラー表示の優先順位を直す (B-R11c・掟 15) - #638

Merged
cipherwebllc merged 1 commit into
mainfrom
fix/r6-b11c-seller-panel-bugs
Sep 24, 2026
Merged

cipherwebllc merged 1 commit into
mainfrom
fix/r6-b11c-seller-panel-bugs

Conversation

@cipherwebllc

Copy link
Copy Markdown
Owner

概要

第 6 回レビュー R11a (#616) で固定した出品者パネルの既存の小さな問題の修正 (B-R11c) です。意図的な挙動変更です。

# 症状 修正
1 切り替え前の account の保存やトグルが切り替え後に完了すると、onSuccess の再取得が新しい account の cookie で商品一覧を 1 回余分に取得し、B の非公開データが A の cache に一瞬入っていた 開始時の address を捕まえ、古い完了は「開始 account の表示中の query の無効化」だけを行う。同じ account の遅い完了 (タブ切替後の復帰・A→B→A) は 1 回の再取得で反映し、別 account やサインアウトでは何も送らない
2 保存エラーより古い読み込みエラーが優先され、保存の失敗が見えなかった submittedAt で新しい失敗を優先 (既存の文言を再利用)
3 開示節に到達しない三項演算があった 除去 (DOM は同一)

⚠️ 掟 15: 出品者の非公開の商品情報と session の境界に触れます。merge 前に user の明示レビューをお願いします。 公開文言の変更はありません。

検証

  • 回帰テスト 32 件 (修正前 10 件が落ち、修正後すべて通過)。R11a の DOM hash 14 件は不変です。
  • 別セッションの Opus 5.5 のレビュー: blocker なし → should-fix 2 件 (同じ account の遅い完了を捨てない・最新の失敗を優先) を反映済み。ガードを外すと 16 件のテストが落ちます。
  • typecheck 0 / eslint 0 / full vitest 15,059 pass

🤖 Generated with Claude Code

https://claude.ai/code/session_01HeizmagJBgL5peL5mQpxkc

@vercel

vercel Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

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

1 Skipped Deployment
Project Deployment Actions Updated
openpay Ignored Ignored Preview Sep 24, 2026 11:25pm UTC

Request Review

第 6 回レビュー R11a (#616) で固定した既存の小さな問題。(1) 切替前の account の保存/トグルが
切替後に完了すると、onSuccess の refetch が新しい account の cookie で GET /api/store/products を
1 回余分に送り、B の非公開データが A の cache key に一瞬入っていた → 開始時の address を捕まえ、
古い完了は「開始 account の active な query の無効化」だけ行う (同じ account の遅い完了は拾い、
別 account・サインアウトでは何も送らない)。(2) 保存エラーより古い読込エラーが優先されていた →
submittedAt で新しい失敗を優先。(3) 開示節の到達しない三項演算を除去 (DOM 同一)。

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HeizmagJBgL5peL5mQpxkc
@cipherwebllc
cipherwebllc force-pushed the fix/r6-b11c-seller-panel-bugs branch from 4cbe0f1 to 56312da Compare September 24, 2026 23:24
@cipherwebllc
cipherwebllc merged commit 8bee358 into main Sep 24, 2026
8 checks passed
@cipherwebllc
cipherwebllc deleted the fix/r6-b11c-seller-panel-bugs branch September 24, 2026 23:56
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