[5ym]

3 つに分けて見直した(サーバー、画面の復号と描画、接続と中継)。動きは変えていない。直したバグは統計とログの小さなものだけ。

サーバー

• QUIC のコールバックで出た例外が、どこにも残っていなかった。ログに書く
• 自動更新の MSI のログが、data_dir を変えると mado.log と別の場所になっていた
• /ws と /wt の中継の後始末を Relay.Serve にまとめた(ログと数え方がずれていた)
• 使っていないもの(datagram の送信など)を外し、中間証明書の取り出しなど重なっていた処理をまとめた。5972 → 5905 行

画面の復号と描画

• タイルを矩形で切る処理(3 か所)、矩形の重なり、BGRA→RGBA、YUV444 の CPU 版の共通部分をまとめた
• 試験でしか使っていなかった ProgressiveDecoder を消した
• AVC444 の時間の統計が 1ms 未満を切り捨てて足していた
• 約 6368 → 6295 行

接続と中継

• コメントのずれを 9 か所直した(connectionType は常に LAN、GFX は H.264 なしでも使う、など)
• 使っていないコードと定数を消し、同じ処理をまとめた。5563 → 5509 行

そのほか

• トレイ: 自動更新が止まっているとき「最新は 」と空の版を出していた
• README / docs: AVC444 の時間、connectionType と見た目の指定、送る量を減らす工夫、listen とファイアウォール、/status の中身、/logout、MSI のログの場所
• 試験用 mini-mado を今の Relay に合わせた

試験: .NET 89 件、web 298 件、tsc、biome、build:web。Windows の実機での確認はまだ(試験用 VM が止まっている)。

🤖 Generated with Claude Code

https://claude.ai/code/session_01M2BadDnB3uH3FDGjaPMP4J

[claude]

Claude finished @5ym's task in 27s —— View job

---

レビュー結果

主にサーバー側(Relay.Serve、QUIC、Updater、トレイ)の差分を読んだ。web 側 (web/rdp/*) の差分は読んでいない。そこはテストと tsc に任せる。ビルドとテストも走らせていない。

指摘するほどの不具合は見つからなかった。

確認できた点:

• Relay.Serve: sessions(+1) / sessions(-1) と open/close ログが try/finally で対になっている。/ws と /wt で数え方とログが揃う。Serve_は数を戻して開け閉めをログに残す というテストも追加されている。
• QuicConnection.OnError: 例外を握りつぶさず webtransport: コールバックで例外 としてログに出す。WebTransport.cs:75 で配線済み。
• s.Abort(0x100) → H3.NoError: 値は同じ 0x100 (Http3.cs:149) なので動作は変わらない。
• MSI ログの場所: Updater に logDir を渡すようにし、mado.log と同じ場所になった。Program.cs:60 とテストの呼び出しも更新済み。
• Acme.Intermediates: 先頭を葉証明書とみなして外す。以前の Thumbprint 比較より単純。ACME が返す PEM は葉が先頭なので問題ない。
• トレイの空の版: 最新版が空のときは専用のメッセージを返す。

要確認(軽微)

• Acme.Intermediates は PEM の先頭が葉であることを前提にしている。ACME 以外の PEM を読み込む経路があるなら、順序の保証を確認するとよい。
• PR 本文にあるとおり、Windows 実機での確認がまだ。QUIC 周りの構造体 (QuicConnectionEvent) を削ったので、実機で WebTransport の接続を一度確かめてからマージするとよい。

テスト: Serve の新規テストは妥当。トレイの空の版の分岐と MSI ログの場所には新しいテストがない。トレイは UI 側なので、無くても許容範囲。

[claude]

Claude finished @5ym's task in 21s —— View job

---

レビュー結果

サーバー側に加えて、web 側の yuv444.ts / rfx.ts / bitmap.ts の差分も読んだ。ビルドとテストは走らせていない。

指摘するべき不具合は見つからなかった。

• cutTile: 幅が TILE のときは subarray で切り出して写さない。yccToRgba はタイルごとに新しい Uint8ClampedArray を作るので、使い回しによる上書きは起きない。
• bgraToRgba: bitmap.ts が rfx.ts から LITTLE_ENDIAN を import する。rfx.ts 側は bytes しか import していないので循環はない。
• Yuv444 / Yuv444GlRef: 共通部分を CpuYuv444 に移した。draw / close / 初期値(Y=0、U=V=128)は元と同じ。evenRects は元のループと同じ結果になる。
• clampRects: yuv444.ts からは消したが、gfx.ts に同名の関数が残っていて、テストもそちらを参照している。
• サーバー: 前回のレビューで挙げた Relay.Serve、QUIC の OnError、Updater の logDir は、読んだ範囲では問題なかった。

要確認(軽微)

• Acme.Intermediates は、PEM の先頭が葉証明書であることを前提にしている。ACME の応答ならそのとおりだが、ほかの経路から PEM を読むなら順序を確認してほしい。
• Windows 実機での確認がまだ。QuicConnectionEvent まわりの構造体を削ったので、マージ前に WebTransport の接続を一度確かめてほしい。

テスト

• Serve の新規テストは妥当。
• ProgressiveDecoder の削除に合わせて progressive.test.ts も更新されている。
• トレイの空の版の分岐と MSI のログの場所には新しいテストがない。UI 側とパス決めなので、無くても許容範囲。