fix(header): 接続・切断後のフォーカスを summary へ戻す + SIWE flag テストのずれを直す - #545
Merged
Merged
Conversation
旧 holder は実在しない enableBilling を持ち、実装が読む enableUsageFee / enableHandles を 欠いていた (mock されない flag は実 env の false 固定 = 掟 7 のガードが 2 flag で無効)。 holder のキーを siweEnabled のソースと突き合わせるフェンスと、flag 1 本ずつの網羅テストを追加。 Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HeizmagJBgL5peL5mQpxkc
接続 / 切断で branch が入れ替わると、押したボタンごと <details> が作り直され、フォーカスが body へ落ちていた (キーボード利用者は位置を失う)。メニュー内の操作を起点にしたときだけ、 作り直された summary へフォーカスを戻す。 - ページ読み込み時の自動再接続・ページ側の接続ボタンではフォーカスを奪わない - 接続が失敗・拒否で終わったら予約を捨てる Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HeizmagJBgL5peL5mQpxkc
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
…ク反映) - 承認待ちの間に利用者が別の場所へフォーカスを移していた場合、ヘッダへ奪っていた。 予約が残る 3 経路 (長い承認待ち・同一 error の再発・state を変えない disconnect) を 「落ちたフォーカスを拾うだけ」の 1 条件でまとめて塞ぐ - フェンスは env.enableX という書き方にしか反応せず、分割代入や helper 経由で flag を足すと holder にも無いまま一致して通っていた → ブロック内の enable* は env. 直読みであることも検査 - 全 OFF テストの題が実態 (8 flag) とずれていたのを直す Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HeizmagJBgL5peL5mQpxkc
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#541〜#543 の Opus 5 抜け漏れチェックで見つかった既存の問題 2 件 (どちらも
WalletBadge)。1. テストの flag holder が実装とずれていた (commit 1・テストのみ)
tests/components/WalletBadge.test.tsxの flag holder は実在しないenableBillingを持ち、実装のsiweEnabledが読むenableUsageFee/enableHandlesを欠いていた。mock されない flag は実 env の false 固定になるので、掟 7 (「SIWE flag を足し忘れるとヘッダからサインインできない」) を守るテストがこの 2 flag で効いていなかった。components/WalletBadge.tsxのsiweEnabledブロックのenv.enable*と突き合わせる (今後 flag を足したらテストも足さないと落ちる)it.eachで網羅2. 接続・切断のあとフォーカスが body に落ちる (commit 2)
メニュー内のボタンで接続 / 切断すると、押したボタンごと
<details>が作り直されてフォーカスが body へ落ちる (#543 以前からの挙動)。メニュー内の操作を起点にしたときだけ、作り直されたsummaryへフォーカスを戻す。可視文言の変更なし・money-path 不変・page ファイル変更なし・新規依存なし (react の useEffect / useRef のみ)。
検証
enable*はenv.直読み」を追加 (分割代入や helper で足した flag が黙って通るのを防ぐ)。反映後も実機で activeElement =SUMMARYを再確認next dev -p 3142): 接続直後のdocument.activeElement= ヘッダのSUMMARY(アドレス表示)・details[open]= 0tests/lib/license/stock.test.tsは既知の wasmoon flaky で今回は worker がハング (CPU 99% で無応答) →node scripts/run-lua-tests.mjsで 213/213 pass (attempt 1 は同じハングで timeout・attempt 2 で成功)🤖 Generated with Claude Code
https://claude.ai/code/session_01HeizmagJBgL5peL5mQpxkc