New Update Modal + SEO

[MaryWylde]

No description provided.

[github-actions]

Claude finished @MaryWylde's task in 3m 25s —— View job

---

Claude's PR Review

Todo list

• Gather context (diff against origin/main, read changed files)
• Review NewUpdateModal changes (session dismissal logic)
• Review SEO changes (SeoGenerator x2, 404.tsx)
• Review Box/EnvironmentSubSection/CountryBiasMap changes
• Post final review summary

Summary

Small, well-scoped PR: session-based dismissal for the "new update" modal, a canonical-URL fix (avoid pointing at a 308 redirect for locale root paths), noindex on the 404 page, and several accessibility alt-text fixes. No structural or convention issues found.

NewUpdateModalContainer.tsx (src/uxcore/components/NewUpdateModal/NewUpdateModalContainer.tsx:29,52-59)

• Dismissal keyed off res.updatedAt via sessionStorage, correctly scoped inside useEffect/an event handler (SSR-safe, no top-level window/sessionStorage access).
• sessionStorage.setItem on close is wrapped in try/catch for private-mode environments — good defensive touch. Minor nit (not blocking): the read at line 29 isn't in its own try/catch, so if sessionStorage access itself throws, it'll be caught by the outer catch and logged as [new-update] fetch failed, which is a slightly misleading label for a storage error vs. an actual fetch failure. Not worth a fix on its own.
• Import reordering in NewUpdateModal.tsx is consistent with this repo's simple-import-sort groups (@uxcore/* sorts into the same third-party-like bucket as next/react, alphabetically first) — not a violation.

SEO (SeoGenerator.tsx ×2, 404.tsx)

• The canonicalPath regex fix (.replace(/^\/(?=$|[?#])/, '')) correctly targets the case where a locale-prefixed path collapses to a lone / (e.g. /?ref=x on a ru/hy locale), which previously produced /ru/ and triggered a 308 to /ru. Verified the regex is anchored so it won't strip a leading slash from real paths like /uxcore/1-slug.
• 404.tsx now emits noindex, follow instead of index, follow — correct, prevents the error page from being indexed while still letting crawlers follow links off it.

Accessibility fixes (Box.tsx, EnvironmentSubSection.tsx, FlagImage.tsx + 3 call sites)

• All replace empty/missing alt text with meaningful descriptions. FlagImage correctly threads an optional countryName prop through all three call sites (BiasPanel, CountryList, CountryMap) with a sensible fallback to the country code.

No inline comments — nothing here needs a code change. Nice, focused fix set.