Skip to content

fix(discovery): 出品者画面の下書き・古い wallet の完了通知・DELETE の例外・USDC 案内の判定を直す (B-R10c) - #636

Open
cipherwebllc wants to merge 1 commit into
mainfrom
fix/r6-b10c-discovery-owner-bugs
Open

cipherwebllc wants to merge 1 commit into
mainfrom
fix/r6-b10c-discovery-owner-bugs

Conversation

@cipherwebllc

Copy link
Copy Markdown
Owner

概要

第 6 回レビュー R10a (#617) で固定した AI ストア出品者画面の既存の不具合の修正 (B-R10c) です。意図的な挙動変更です。

# 症状 修正
1 編集中の下書きと編集対象が、wallet の切り替えやサインアウト後も残る 切り替えとサインアウトで消す (完了カード・snippet・通知・USDC の案内・削除確認も)
2 切り替え前の wallet の PATCH/DELETE/POST が、新しい wallet の画面で成功と表示される 開始時の wallet を捕まえ、画面の更新だけを抑止。catalog と送信元 wallet の一覧の無効化は常に行う (A→B→A で掲載が出ず、再送で 409 になるのを防ぐ)
3 DELETE の fetch 例外を捕まえていない 既存のエラー UI に渡し、エラー時は登録欄を自動展開して見えるようにする
4 USDC の案内を最新 render の値で判定していた 送信時の値で判定する

判断が入っている点

wallet のロック等で wagmi が一瞬切断する (address が undefined になる) 場合、下書きと進行中の送信結果は保持し、再接続後の本当のサインアウトで消します。購入の hook は切断で意図的に消しますが、出品の下書きはお金ではなく、一時的な切断で失うほうが害が大きいと判断しました。方針が違えばご指摘ください。

公開文言の変更はありません (既存のエラー文言を再利用)。

検証

  • 回帰テスト 16 件を追加 (修正前 25 件が落ち、修正後 pinning 48 件が通過)。R10a の DOM hash 4 ブロックは不変です。
  • 別セッションの Opus 5.5 のレビュー: approve-with-fixes → 修正済み (DELETE エラーが閉じた欄の中で見えない・完了済みの結果が残る・一時切断で下書きが消える・古い結果の抑止で無効化まで止まる、の 4 点)。
  • typecheck 0 / eslint 0 / full vitest 15,110 pass / build OK (/discovery 311 kB・変化なし)

🤖 Generated with Claude Code

https://claude.ai/code/session_01HeizmagJBgL5peL5mQpxkc

…-R10c)

第 6 回レビュー R10a で固定した AI ストア出品者画面の既存の不具合。(1) 編集中の下書きと editId が
wallet 切替・サインアウト後も残る → 消す (完了カード・snippet・通知・USDC 案内・削除確認も)。
一時的な切断 (wallet のロック等で address が一瞬 undefined) では下書きと進行中の結果を保持し、
再接続後の本当のサインアウトで消す。(2) 切替前の wallet の PATCH/DELETE/POST が新しい wallet の
下で成功表示になる → 開始時の wallet を捕まえて画面更新だけ抑止 (catalog と送信元 wallet の
owned の無効化は常に行い、A→B→A で掲載が出ない・再送で 409 になるのを防ぐ)。(3) DELETE の
fetch 例外を既存のエラー UI に渡し、エラー時は登録欄を自動展開して見えるようにする。
(4) USDC 案内の判定を最新 render ではなく送信時の値で行う。

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: 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 1:36pm UTC

Request Review

@cipherwebllc
cipherwebllc force-pushed the fix/r6-b10c-discovery-owner-bugs branch from a371c78 to e2ad376 Compare September 24, 2026 13:36

This branch has not been deployed

No deployments
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