Skip to content

駅到着直後に次の駅へ一瞬進んでしまうデグレを修正#5338

Merged
TinyKitten merged 1 commit intodevfrom
claude/fix-station-skip-bug-HatwW
Feb 19, 2026
Merged

駅到着直後に次の駅へ一瞬進んでしまうデグレを修正#5338
TinyKitten merged 1 commit intodevfrom
claude/fix-station-skip-bug-HatwW

Conversation

@TinyKitten
Copy link
Member

@TinyKitten TinyKitten commented Feb 19, 2026

グレース期間中にnearestStationが変わった場合でもisArrivedがtrueを返していたため、
到着直後にGPSのぶれで次の駅に誤って進む問題が発生していた。
グレース期間の適用を到着を引き起こした駅に限定し、useNearestStationで
currentStationをnextStationより優先するように変更。

https://claude.ai/code/session_0125DF6zeKaFmYsCqKciDTog

Summary by CodeRabbit

リリースノート

  • バグ修正
    • 複数の駅が同じ位置に存在する場合の駅選択の順序を改善しました。
    • 到着時のグレースピリオド処理をより正確に適用するようにしました。

グレース期間中にnearestStationが変わった場合でもisArrivedがtrueを返していたため、
到着直後にGPSのぶれで次の駅に誤って進む問題が発生していた。
グレース期間の適用を到着を引き起こした駅に限定し、useNearestStationで
currentStationをnextStationより優先するように変更。

https://claude.ai/code/session_0125DF6zeKaFmYsCqKciDTog
@github-actions github-actions bot added the react label Feb 19, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 19, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

複数の駅が同じ座標を共有する場合の最寄り駅の選択順序を変更します。また、到着イベントを追跡するための lastArrivedStationIdRef を導入し、グレース期間ロジックを精密化します。

Changes

Cohort / File(s) Summary
Nearest Station Selection Priority
src/hooks/useNearestStation.ts
複数駅の座標が一致する場合、currentStation を nextStation より優先的に選択するよう順序を変更。到着直後に次駅へ自動的に進まないようにするロジックを実装。
Arrival State & Grace Period Refinement
src/hooks/useRefreshStation.ts
lastArrivedStationIdRef を追加して最後に到着した駅を追跡。isArrived メモに緯度経度チェックを加え、グレース期間を最後に到着した駅と一致する場合にのみ適用。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Suggested labels

react

Poem

🐰 同じ場所でも心は違い
今の駅を選び抜く
到着を記憶に留めて
次へと急がず待つ知恵
優先順序をちょんちょん ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed タイトルはプルリクエストの主な変更内容を正確に説明しています。GPS揺らぎによるバグの修正という中心的な目的が明確に示されています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch claude/fix-station-skip-bug-HatwW

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

@TinyKitten TinyKitten merged commit 33f4dde into dev Feb 19, 2026
7 checks passed
@TinyKitten TinyKitten deleted the claude/fix-station-skip-bug-HatwW branch February 19, 2026 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants