Skip to content

fix(ratewise): MoneyBox history 檔名改用首爾時區 SSOT#428

Closed
s123104 wants to merge 1 commit into
mainfrom
fix/review-hotfixes-convergence
Closed

fix(ratewise): MoneyBox history 檔名改用首爾時區 SSOT#428
s123104 wants to merge 1 commit into
mainfrom
fix/review-hotfixes-convergence

Conversation

@s123104

@s123104 s123104 commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Summary

自 PR #411 Codex review 收斂 獨立 hotfix(刻意不重複 PR #425 的 PWA/ETag 修復):

  • update-moneybox-rates.ymlSave MoneyBox daily history snapshot 改以 TZ=Asia/Seoul 產生 CURRENT_DATE,對齊 fetch-moneybox-rates.js 的 Asia/Seoul SSOT
  • 修正 UTC+9 換日窗口(15:00–15:59 UTC)history 檔名仍用台北日曆日、導致 30 日 aggregate 錯標的問題
  • build-scripts.test.ts 守門測試,禁止 workflow 回退至 Asia/Taipei

與 PR #425 關係

Test plan

  • pnpm --filter @app/ratewise test -- build-scripts.test.ts -t "MoneyBox daily history"
  • pre-push:typecheck + test + build:ratewise 通過
  • QA agent 驗證 workflow 摘要 Seoul 日期與 history 檔名一致

Made with Cursor

- update-moneybox-rates.yml 以 Asia/Seoul 產生 CURRENT_DATE,對齊 fetch-moneybox-rates.js
- 補 build-scripts 守門測試,避免 history 日期與 30 日 aggregate 錯標
- 新增 patch changeset 與 002 紀錄

測試:pnpm --filter @app/ratewise test -- build-scripts.test.ts -t "MoneyBox daily history"

Co-authored-by: Cursor <cursoragent@cursor.com>
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Deprecation Warning: The deny-licenses option is deprecated for possible removal in the next major release. For more information, see issue 997.

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@github-actions

Copy link
Copy Markdown
Contributor

✅ SEO 審計通過!所有 2026 標準驗證項目都符合要求。

  • ✅ Sitemap 2026 標準
  • ✅ Breadcrumb Schema
  • ✅ JSON-LD 結構化數據
  • ✅ 內部連結結構

@s123104

s123104 commented Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

關閉本 PR,改採 #433(governance v2)的 snapshot date 版本(更徹底)。

理由

本 PR 用 CURRENT_DATE=$(TZ=Asia/Seoul date +%Y-%m-%d)wall-clock 首爾,方案 A)。#433 已含 5ccd8132資料 snapshot date 方案(方案 B):

  • history 檔名直接用 extractSeoulSnapshotDate 從資料 updateTime 提取的首爾掛牌日,與 fetch script 的 date-rollover 判斷同源 SSOT
  • 提取失敗才 fallback 至 TZ=Asia/Seoul date(即本 PR 的方案,作為退路)

snapshot date 方案無論 workflow 何時執行都對齊資料本身宣告的日期,而 wall-clock 在 API updateTime 落後或執行延遲跨日時仍可能差一天。

守門測試搶救

本 PR 的 build-scripts.test.tsTZ=Asia/Seoul / 非 Asia/Taipei 的守門斷言#433 的版本同樣通過(fallback 行含 Asia/Seoul),已搶救進 #433 一併保留。

@s123104 s123104 closed this Jun 26, 2026
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.

1 participant