Skip to content

Fix the deploy test, add a Spanish home, and refresh the no-JS snapshot - #168

Merged
sbc1-code merged 4 commits into
mainfrom
cursor/deploy-es-snapshot-cbbd
Nov 3, 2026
Merged

sbc1-code merged 4 commits into
mainfrom
cursor/deploy-es-snapshot-cbbd

Conversation

@sbc1-code

@sbc1-code sbc1-code commented Nov 2, 2026 •

Copy link
Copy Markdown
Owner

The mobile search-focus failure in production run 41499379953 blocked publication. This change protects an active editing field from deferred map selection/restoration callbacks and cancels outdated callbacks.

/es/ owns the Spanish dashboard, shell, footer and shared preference, including a visitor with saved English. Switching to English preserves query/hash state. Canonicals, reciprocal language alternates, sitemap and no-JS Spanish entry are retained.

The no-JS home frame uses /snapshot/: Vercel renders sourced CBP data as labeled snapshot HTML; Pages uses its published snapshot file. Upstream and known-host bundled recovery share an 8.5-second request budget under the existing ten-second function limit. Fallback preserves original retrieval/report clocks; unavailable responses remain explicit and uncached. CDN reuse permits five minutes plus up to ten minutes of stale revalidation, so it is not a strict five-minute freshness promise. Shared crossing URLs remain unchanged.

Integration state

Current head: 902311654fc207b9cf07f5a39c6086e4864b8eae. Intended order: #168 → #170 → #169. #168 contains task-start main baseline 99fa2255; #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

  • Local Node 24 API-mode production build and full npm test passed on this exact commit: 239 unit checks and 418 desktop/mobile browser navigations. Entry JS 61.48/62 KiB; CSS 19.87/20 KiB. Budgets are unchanged.
  • Exact-head remote CI 41576481154 passed on Node 22 with a locked install, production build and full npm test: 239 unit checks and 418 browser navigations. CI entry JS 61.52/62 KiB, CSS 19.87/20 KiB; audit 0 high/critical, 1 moderate. Budgets and gates are unchanged.
  • Checks cover mobile search focus, report focus/Back, fresh and saved-English Spanish home, query preservation, no-JS recovery, missing/zero/closed states, history gaps/staleness, selected lanes and EN/ES crossing-share channels. Combined desktop/mobile screenshots were captured and inspected locally. Browser share APIs were mocked; no message was sent.
  • Local audit gate passed with 0 high/critical and 2 moderate advisories. Scoped manual security delta review found no new input sink, executable behavior, dependency or permission change beyond the reviewed fixes. A formal automated security scan is not claimed.
  • Typecheck remains ungreen: configured tsc --noEmit -p jsconfig.json reports 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.
  • Lint unavailable: an ESLint config exists, but no lint script or pinned ESLint/plugins are supplied in the manifest/lockfile/installed tree. No packages were added to manufacture a lint pass.
  • All 394 installed package versions match the lockfile. Local Vite preview does not execute Vercel functions: the API-mode no-JS fixture emulates bundled fallback, while actual handler timing/cache/error/host behavior is tested separately.
  • No hosted preview was found for this SHA during the read-only check. No manual preview deployment or access change was attempted. Current production API freshness remains unverified from this cloud environment; Pages/collector success is not treated as Vercel recovery. Issue [ops] Vercel daily publication failing #171 remains open.
Open in Web Open in Cursor 

…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>
@sbc1-code
sbc1-code merged commit 4a51ddd into main Nov 3, 2026
1 check passed
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.

2 participants