fix: sinceDate の算出を組織タイムゾーン基準にする

[coji]

Summary

• analysis/reviews と analysis/feedbacks の sinceDate 算出を UTC 固定から組織タイムゾーン基準に修正
• analysis/inventory で Open PRの滞留日数分布チャート - 見えない在庫を可視化する #253 実装時に修正済みのパターンに統一

Changes

• reviews/index.tsx: timezoneContext をインポートし、dayjs.utc().tz(timezone).subtract(...).startOf('day').utc().toISOString() パターンに変更
• feedbacks/_index/index.tsx: 同上

Test plan

• pnpm validate が通る（lint, format, typecheck, build, test）
• reviews ページが正常に表示される
• feedbacks ページが正常に表示される
• period 切り替え（1/3/6/12ヶ月, All time）が正常に動作する

Closes #256

Generated with spec-implement-accept skill

Summary by CodeRabbit

リリースノート

• バグ修正
  • 分析（フィードバック・レビュー・在庫）ページで、ユーザーのタイムゾーンを正しく考慮した期間フィルタの計算を改善しました。期間指定時の境界日が現地時間に基づいて安定的に算出されます。
  • 「全期間」選択時の基準日時を統一し、一貫した表示・集計が行われるようになりました。

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: cf0ef3c1-f5c7-42f2-b981-43e17854f310

📥 Commits

Reviewing files that changed from the base of the PR and between 9268cac and 2a2f4a0.

📒 Files selected for processing (4)

• app/libs/date-utils.ts
• app/routes/$orgSlug/analysis/feedbacks/_index/index.tsx
• app/routes/$orgSlug/analysis/inventory/index.tsx
• app/routes/$orgSlug/analysis/reviews/index.tsx

---

📝 Walkthrough

Walkthrough

analysis の複数ルートで、組織タイムゾーンを使う共通ユーティリティ calcSinceDate(periodMonths, timezone) を導入し、各ローダーがタイムゾーンコンテキストから timezone を取得して sinceDate をそのユーティリティで算出するように変更されました。periodMonths === 'all' はユーティリティ側で固定日付に置き換えられています。

Changes

Cohort / File(s)	Summary
Analysis reviews & feedbacks loaders app/routes/$orgSlug/analysis/reviews/index.tsx, app/routes/$orgSlug/analysis/feedbacks/_index/index.tsx	ローダーが timezoneContext からタイムゾーンを取得し、従来の inline dayjs 算出を廃止して calcSinceDate(periodMonths, timezone) を使用するように差し替え。返却するフィールドやキャッシュロジックは維持。
Inventory loader app/routes/$orgSlug/analysis/inventory/index.tsx	同様に inline dayjs 算出を calcSinceDate(periodMonths, timezone) 呼び出しへ置換。その他のロジックは不変。
Date utils app/libs/date-utils.ts	新関数 `calcSinceDate(periodMonths: number

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• fix: unify timestamp format to ISO 8601 for correct timezone display #194 — 以前に reviews/feedbacks の sinceDate 算出を変更した PR。calcSinceDate 抽出と関連。
• feat: Open PR 滞留日数分布チャート #257 — inventory の同様の sinceDate 修正に関する PR。重複する変更領域あり。
• refactor: restructure routes into workload/throughput/analysis groups #182 — analysis ルート群の日時算出に関連する別修正。ファイルレベルで関連性あり。

Poem

🐰 ぽょん、日付をそっと直したよ
タイムゾーンに合わせて跳ねる朝
月を数えて、境界をそろえて
データは正しく、レポートは嬉しそう
にんじん片手に、今日もバグゼロ祈願 ✨

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	プルリクエストのタイトルは変更の主要な目的（sinceDate の算出を組織タイムゾーン基準にする）を正確に説明しており、変更セットの中心的な内容と完全に関連している。
Linked Issues check	✅ Passed	PRの変更は#256の要件を完全に満たしている。reviews/feedbacks のsinceDate算出が UTC固定からタイムゾーン基準に変更され、inventory との統一パターンが実現されている。
Out of Scope Changes check	✅ Passed	すべての変更は#256の要件範囲内である。calcSinceDate ユーティリティの抽出は既存ロジックの共通化で、スコープ内の改善であり、追加機能や関連のない変更は含まれていない。
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch feat/256-sincedate-org-timezone

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}