Datalogger: confirm recognised track while parked + bump coach plugin to BETA HEAD#219
Merged
Merged
Conversation
While stationary the tool shows a bare speedometer (logging only arms above 5 mph), which was indistinguishable from the genuine "no tracks found nearby" state. Surface the nearest known track's name in the waiting speedometer so it's clear the track list loaded and detection worked — you're just waiting to move. Adds RealtimeLapTimer.nearestTrackName + a nearbyTrackName field on TimingState, plumbs it through the session into the speedometer title, and a trackDetected string across all locales. Tests cover the timer probe and the waiting snapshot. https://claude.ai/code/session_014pk2etibihm7WSLPZSYc36
Force-refresh @theangryraven/eye-in-the-sky to BETA HEAD (a867b3f -> f8c6ff4). The new build imports formatDecimal/formatInteger/formatSignedDelta from the host's i18n format module, so add those locale-aware formatters (with tests) to keep the production build green. https://claude.ai/code/session_014pk2etibihm7WSLPZSYc36
|
This pull request has been ignored for the connected project Preview Branches by Supabase. |
Coverage SummaryLines: 59.23% (5584/9427) · Statements: 58.33% · Functions: 56.15% · Branches: 53.79% Per-file coverage
|
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
dovesdataviewer | 302e6c0 | Commit Preview URL Branch Preview URL |
Jun 14 2026, 02:11 PM |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
Yes, the track list loads.
useDataloggercallsloadTracks()→timer.setTracks(), Orlando Kart Center is intracks.json, andfindNearestTrack(10 mi radius) matches it fine.The speedometer shows because you're parked: lap timing only arms once you move above 5 mph (
phase === "waiting"forces speedometer mode by design). The real problem was UX — while sitting at a recognised track, the waiting speedometer gave no confirmation the track was detected, so it looked identical to the genuine "Speedometer mode — no tracks found nearby" state. Hence the "is it even loading the tracks?" confusion.What changed
1. Datalogger track confirmation (the fix)
RealtimeLapTimer.nearestTrackName()+ anearbyTrackNamefield onTimingState.DataloggerSession's waiting branch into the speedometer title.trackDetectedstring added across all 7 locales.2. Bump coach plugin to BETA HEAD (i18n phases 2–3)
@theangryraven/eye-in-the-skyfroma867b3f→f8c6ff4(BETA HEAD).formatDecimal/formatInteger/formatSignedDeltafrom the host'slib/i18n/format, which didn't exist yet — added those locale-aware formatters (with tests) so the production build stays green.Verification
npm run lint,npm run typecheck— cleannpm run test:run— 120 files / 1745 tests passnpm run build— succeeds (coach plugin now resolves)https://claude.ai/code/session_014pk2etibihm7WSLPZSYc36
Generated by Claude Code