Skip to content

docs: 検証環境の手順書と gas/README.md の、今の動きと違う記述を直す - #603

Merged
taminororo merged 1 commit into
developfrom
docs/kanba/602/fix-stale-ops-notes
Oct 1, 2026
Merged

taminororo merged 1 commit into
developfrom
docs/kanba/602/fix-stale-ops-notes

Conversation

@taminororo

@taminororo taminororo commented Oct 1, 2026 •

Copy link
Copy Markdown
Collaborator

対応Issue

resolve #602

概要

手順書のうち、今の動きや事実と違っていた3か所を直しました。どれもコードか記録で確かめています。

場所 前 後 確かめ方
docs/operations/staging-rehearsal.md「5. 手元の Mac からアプリを起動する」 -d chrome は一時的なプロファイルなので、以前のシフトのキャッシュ(Hive)に惑わされない キャッシュは残る。flutter_tools が前回の状態を mobile/.dart_tool/chrome-device/ に保存して、起動のたびに戻す。API の向け先を変えたら、このディレクトリを移してから起動する Flutter 3.27.3(mobile/.fvmrc の版)の flutter_tools の web/chrome.dart(_restoreUserSessionInformation と _cacheUserSessionInformation)と web/web_device.dart(cacheDir が .dart_tool/chrome-device)
同「罠」 env の日付が過去だと、レビューを求める画面がシフトカードを覆う レビューの画面は自動では出ず、シフトカードのメニューから開いたときだけ出る(PR #528)。以前の動きも1文で残した develop の mobile/lib/ に、自動で出す _showReviewFormIfNeeded が無く、ReviewBottomSheet.show の呼び出しはシフトカードのメニューだけ
gas/README.md「ディレクトリと実体の対応」の manual-assignment/ の行 共有ドライブの置き場 技大祭の Drive のフォルダ(マイドライブを共有したもの)の置き場 技大祭のアカウントは @gmail.com で、共有ドライブは無い(#558、ADR 0016 の前提)

1つ目を直すときに、作業記録にあった「ログアウトすればキャッシュが消える」という対処も確かめました。アプリのログアウト(mobile/lib/pages/etc_page.dart)は画面をサインインに戻すだけで、Hive のキャッシュを消していなかったので、手順書には書いていません。

画面スクリーンショット等

なし(ドキュメントのみ)

テスト項目

  • 直した3か所が、今の動きと合っているか
  • python3 scripts/refcheck/refcheck.py docs/operations/staging-rehearsal.md gas/README.md が通ること(手元で確認済み)

備考

Summary by CodeRabbit

  • ドキュメント
    • ステージング確認手順に、API 接続先の変更時に必要なキャッシュへの対処と、レビュー画面が表示される条件を追記しました。
    • PDF の配置先に関する説明を更新しました。

@taminororo taminororo self-assigned this Oct 1, 2026
@coderabbitai

coderabbitai Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

🧰 Additional context used
📚 Code guidelines (1)
AGENTS.md — auto-discovered

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: NUTFes/SeeFT/.coderabbit.yaml

Review profile: CHILL

Plan: Advanced

Run ID: 4f386e97-bd27-4218-8eb8-6016c6d546db

📥 Commits

Reviewing files that changed from the base of the PR and between 5858159 and f251824.

📒 Files selected for processing (2)
  • docs/operations/staging-rehearsal.md
  • gas/README.md

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

検証環境の手順書で、Chromeのキャッシュとレビュー画面の説明を更新しました。GAS READMEでは、PDFの配置先を変更しました。

Changes

手順書の現状反映

Layer / File(s) Summary
検証環境の手順を更新
docs/operations/staging-rehearsal.md
APIの向け先を変更した場合は、Chrome起動前に一時プロファイルを移動する手順を記載しました。ログアウトではキャッシュが消えないことも追記しました。レビュー画面について、現行版ではシフトカードのメニューから開く場合に表示されることと、旧版の自動表示条件を記載しました。
PDFの配置先を更新
gas/README.md
manual-assignment/ のPDF配置先を、共有ドライブからマイドライブを共有した技大祭のDriveフォルダに変更しました。

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~4 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to f2518

Available evidence supports the staging-guide updates, and no concrete defect is established. The live PDF folder’s ownership could not be confirmed, so its documented location remains uncertain.

Architecture Summary

Architecture risk: 🔵 Low · up to f2518

The change affects 2 systems.

Changed systems: docs, gas

Architecture concerns
No architecture-level concerns identified.

Review details

Systems and components

  • observed — docs (service) was modified; 1 changed file maps to changed impact.
  • observed — gas (service) was modified; 1 changed file maps to changed impact.

Before / after behavior

  • observed — Modified behavior in docs/operations/staging-rehearsal.md: Chrome の一時プロファイルには前回の状態が復元されるため、旧記述の「以前のシフトキャッシュに惑わされない」を改め、API の向け先変更後は起動前に mobile/.dart_tool/chrome-device/ を移動すること、ログアウトではキャッシュが消えないことを記載した。
  • observed — Modified behavior in docs/operations/staging-rehearsal.md: レビュー画面の説明を、現行版ではシフトカードのメニューから開いた場合に限り表示される内容へ変更した。旧版では mobile/env/.env の日付から計算したシフト終了時刻が過去の場合に自動表示され、シフトカードを覆っていた旨を記載した。
  • observed — Modified behavior in gas/README.md: manual-assignment/ のPDF配置先を共有ドライブから、マイドライブを共有した技大祭の Drive フォルダに変更しました。
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed タイトルは、検証環境の手順書と gas/README.md の現状と異なる記述を修正する主な変更内容を明確に示しています。
Description check ✅ Passed 説明はテンプレートの主要項目を埋めています。対応Issue、3か所の修正内容、確認方法、スクリーンショットが不要であること、テスト項目、関連PRの注意事項を具体的に記載しています。
Linked Issues check ✅ Passed Issue #602 の3要件を満たしています。docs/operations/staging-rehearsal.md は mobile/.dart_tool/chrome-device/ に Chrome の状態が保存されて再利用される条件と、API の向き先を変更する前に移動する対処を記載しています。同ファイルはレビュー画面がシフトカードのメニューから開いた場合だけ表示され、旧版では…
Out of Scope Changes check ✅ Passed 変更は Issue #602 が指定する2つの文書の3か所の訂正に限定されています。Chrome キャッシュ、レビュー画面、manual-assignment/ の保存場所に関する変更は、いずれもIssue #602の目的に直接関連します。無関係な実装変更や文書変更は、提示された差分概要にはありません。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@taminororo

Copy link
Copy Markdown
Collaborator Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@taminororo

Copy link
Copy Markdown
Collaborator Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@taminororo
taminororo merged commit 5a4df21 into develop Oct 1, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: 検証環境の手順書と gas/README.md の、今の動きと違う記述を直す

1 participant