Repository navigation
Keep reported CBP waits when duplicate port rows co-occur - #170
Merged
Merged
Conversation
…snapshot. A late map focus callback could take the search field and fail the Pages test. /es/ is the indexable Spanish home. On Vercel the no-JS snapshot frame uses the cached CBP fetch instead of the last full site build. Co-authored-by: Sebastian Becerra <sbc1-code@users.noreply.github.com>
Ysleta's closed standard lane was dropped, so the empty Update Pending row outranked the row that reported Ready, SENTRI, and pedestrian waits. Keep those minutes, and keep Lanes Closed and N/A on the lane instead of storing a blank. Co-authored-by: Sebastian Becerra <sbc1-code@users.noreply.github.com>
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
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.
The recorded Nov 2 CBP fixture contains both Ysleta rows: one has Ready/SENTRI/pedestrian/commercial waits while the duplicate is pending. Evidence-based lane merging preserves usable numeric reports, including zero; explicit Closed/N/A status outranks pending. Missing values remain missing, and a blank Closed/N/A booth count remains null.
The San Ysidro main-row path now uses the same status-aware primary scalar normalization. A retained numeric value on Closed/N/A/Pending does not become a current wait or a “good” status. Those contradictory-number cases are hypothetical robustness fixtures, not a claimed feed incident. PedWest and CBX never supply the main crossing's waits.
Each contributing lane retains its original update clock and originating row date/time when duplicate rows differ. Recorded duplicate pairs, reversed order, zero, unavailable statuses and source clocks are covered. The new parser applies when the collector/API next runs released code; this PR does not reconstruct lost historical raw data.
Integration state
Current head:
af88d8665a251559bdd753426597c64ae5b2c0af. Intended order: #168 → #170 → #169. #168 contains task-start main baseline99fa2255; #170 includes prepared #168; #169 includes prepared #170, so #169 is the full combined candidate. PR bases remain main. Later PR diffs include earlier fixes until integration proceeds in order.Conflicts are resolved in the pushed branches, preserving both endpoint regression groups and N/A expectations, both sitemap imports, all reviewed documentation and current-main data. Subsequent main updates observed during testing changed feed data only and were not overwritten. Source guards, freshness/missing/zero states, native branding, keyboard/focus, reduced motion and share cautions remain intact.
Original feature work is by Cursor; Codex authored the correction and preparation commits. Review/preparation only: production approval is still pending. No shared-main merge, formal approval review, auto-merge, publisher rerun or production deployment was performed.
Current-head validation
npm testpassed on this exact commit: 244 unit checks and 418 desktop/mobile browser navigations. Entry JS 61.56/62 KiB; CSS 19.85/20 KiB. Budgets are unchanged.npm test: 244 unit checks and 418 browser navigations. CI entry JS 61.58/62 KiB, CSS 19.85/20 KiB; audit 0 high/critical, 1 moderate. Budgets and gates are unchanged.tsc --noEmit -p jsconfig.jsonreports 216 diagnostics. Untouched main also has 216 diagnostics; the candidate retains the same legacy categories, with an inferred metadata-message variation. No compiler exclusions or checks were weakened.Historical artifacts
Original Cursor artifact links are preserved below. They predate the current preparation head and do not establish current-head behavior or QC.
Ysleta standard lane has no reported minute
Ysleta Ready Lane shows the reported 50 minutes
Ysleta detail lists the reported lanes
Otay Mesa detail includes the reported commercial and FAST lanes