Repository navigation
Conversation
94b6a6f to
c00c2e8
Compare
c00c2e8 to
27c60f9
Compare
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 27c60f9ffd
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Assisted-By: OpenAI Codex 🤖
Assisted-By: OpenAI Codex 🤖
Assisted-By: OpenAI Codex 🤖
Assisted-By: OpenAI Codex 🤖
Assisted-By: OpenAI Codex 🤖
既定のfetchをglobalThisへ束縛し、メモ保存前の再取得で発生するIllegal invocationを解消する。GET・PUT・POSTの呼び出し元を検証する回帰テストを追加する。 Assisted-By: OpenAI Codex 🤖
店舗変更に伴ってZaimが更新するplace_uidだけを照合対象から除外する。店舗以外の編集では従来どおり非表示列の変化を検出する。実行時と再照合時の回帰テスト、実機確認結果を追加する。 Assisted-By: OpenAI Codex 🤖
There was a problem hiding this comment.
シートを閉じても編集内容や確認画面などの状態が残っていますが、確認画面では閉じれないようにした方がいいと思います。
There was a problem hiding this comment.
修正しました（16209e4）。入力・確認状態はシートを閉じると破棄し、次回は新しいフォームから始まります。
保存開始後は、単体・一括・一覧からの再開をアプリ共通の EditPlanRunner が管理します。シートを閉じても前景・オンラインなら送信を続け、一覧から停止・結果確認できます。背景移行・通信断では次の送信を止め、復帰や再読み込みだけでは自動再開しません。結果未確認は、更新を再送する前に明示的に照合します。
WebKit の iPhone 相当画面と架空データで、確認中の閉鎖による初期化、閉鎖後の送信継続、一覧停止・手動再開、背景移行・再接続・再読み込み後の停止保持を確認しました。単体の計画作成中に閉じても保存が続き、後から開いた別明細の画面や入力に干渉しないことも確認済みです。復元失敗時は新しい確認用計画の作成を防ぎます。
bun run check は成功しています。設計は ADR-0038 に記録しました。
修正版 Preview（支出編集有効）。今回の変更後の iPhone 実機確認は未実施です。
🤖 Generated with Codex
Assisted-By: OpenAI Codex 🤖
Assisted-By: OpenAI Codex 🤖
単体・一括・再開の送信と照合をアプリの共通Runnerへ移す。 シートを閉じると入力を破棄し、前景での送信と一覧での追跡は継続する。 停止・復元・未確認結果の保持と新規計画の開始可否を共有する。 Assisted-By: OpenAI Codex 🤖
Assisted-By: OpenAI Codex 🤖
Assisted-By: OpenAI Codex 🤖
Assisted-By: OpenAI Codex 🤖
Assisted-By: OpenAI Codex 🤖
Assisted-By: OpenAI Codex 🤖
Assisted-By: OpenAI Codex 🤖
|
一括編集の確認画面のレイアウトが気になります。 ダイアログの閉じるボタンの画面占有率が大きいのも気になります。基本的にダイアログ外の背景を押して閉じることが多いため、重要度も低いです。右上に❌ボタン程度でも十分かも。 |
ヘッダーと主要操作を固定し、本文だけをスクロールする。確認画面の重複説明を減らし、閉じる操作を共通の右上ボタンへ集約する。 Assisted-By: OpenAI Codex 🤖
|
レイアウトの指摘に対応しました（d6b1393）。 一括編集・単体編集の入力／確認／結果、明細詳細、絞り込みの閉じる操作を右上の×へ統一しました。タップ領域は44px以上を確保し、背景タップ・Escも維持しています。本文だけをスクロールさせ、主要操作は下部に固定しました。 一括編集の確認画面は見出しと対象件数に説明をまとめ、通常の警告表示と二重スクロールを除去しました。単体編集の画面遷移でもスクロール位置を先頭へ戻します。送信処理とダイアログの独立性は維持しています。 WebKitと架空データで、各画面の操作位置、閉鎖後の送信継続、長い確認内容のスクロール初期化を確認しました。架空50件でもスクロール領域が1か所で、保存ボタンが常時表示されることを確認しています。実データ50件の保存検証ではありません。 修正版 Preview（支出編集有効）。今回の変更後のiPhone実機確認は未実施です。 🤖 Generated with Codex |
Workerと共通の日付スキーマで検証し、入力欄に即時エラーを表示する。空欄や不正な日付では確認へ進めないようにする。 Assisted-By: OpenAI Codex 🤖
明細詳細からの単体編集と、同一種別に絞った結果への一括編集を追加します。一括編集は変更不要な明細を除いて対象を固定し、変更前後を確認してから一件ずつ保存します。入力欄は選択した項目だけを開き、確認ボタンは下部に固定します。支出のカテゴリ変更時はジャンルも必須の変更対象として表示します。確認画面は日付・金額・店舗・品名で対象を示し、明細ごとの変更前後に集約します。
スタック #81 の 4/4。#79 に依存します。Closes #75。Closes #76。Refs #6。
#77〜#79 の API・共有排他・計画ストアを
src/api/edits.tsとsrc/components/edit/へ接続する最終層です。入力上限と編集可否は共通契約を使い、ブラウザには計画 ID だけを保存します。既定では編集は無効です。単体・一括・一覧からの再開は、アプリが所有する共通の EditPlanRunner が管理します。シートを閉じると入力・確認状態を破棄し、保存開始後の送信は前景・オンラインの間続けます。一覧から停止・結果確認ができ、背景移行・通信断・認証切れでは次の送信を停止します。復帰や再読み込みだけでは再開せず、結果不明は再送前に明示的に照合します。設計理由は ADR-0038 を参照してください。
Workers の native
fetchの呼び出し元を保持します。店舗変更時は、Zaim が連動して更新するplace_uidだけを許容し、他項目の編集で非表示列が変化した場合は結果未確認として停止します。ダイアログは右上の閉じる操作、スクロールする本文、下部の主要操作に統一しています。入力・確認・結果の遷移時は本文を先頭から表示します。
検証:
bun run check成功（24 files / 326 tests）。実データ検証と本番の有効化は #72 で管理します。公開・復旧手順は
ops/editing.mdを参照してください。Preview: https://d9058ba8-zaimviewer.ries.workers.dev （支出編集を有効化、Access 保護）。本番トラフィックは切り替えていません。
🤖 Generated with Codex