Skip to content

fix(web): rerender installed locale catalogs - #197

Merged
Tutitoos merged 1 commit into
mainfrom
bugfix/196-auth-locale-rerender
Sep 11, 2026
Merged

Tutitoos merged 1 commit into
mainfrom
bugfix/196-auth-locale-rerender

Conversation

@Tutitoos

Copy link
Copy Markdown
Owner

Related issue

Closes #196

Plan reference

Operational Phase 3 web defect found during live integrated-browser validation.

Summary

Ensure authentication copy rerenders after a selected runtime locale catalog is installed, even when it has the same server revision as the previously loaded locale.

Changes

  • Count successful catalog installations instead of storing the shared catalog revision as render state.
  • Add a deterministic browser regression with equal English and Spanish revisions and a delayed Spanish response.

Validation

  • Relevant automated checks pass

  • Manual verification completed where needed

  • bun run --cwd apps/web test — 50 passed.

  • bun run --cwd apps/web typecheck — passed.

  • bun run --cwd apps/web build — passed.

  • cd apps/web && bunx playwright test tests/bootstrap.spec.ts — 24 passed across six viewports.

  • Live integrated-browser reproduction confirmed the pre-fix race on the deployed prior revision.

Risks and rollback

The change only causes one additional React render after each accepted catalog installation. Locale request cleanup remains abortable. Roll back by reverting this PR.

Checklist

  • The PR has one coherent scope
  • The branch and Closes reference identify the same primary issue
  • The issue acceptance criteria are satisfied
  • No secrets, credentials, personal data, or unintended files are included
  • Documentation and tests are updated when applicable
  • Breaking changes are explicitly documented (none)

@Tutitoos
Tutitoos merged commit dd4383b into main Sep 11, 2026
16 of 17 checks passed
@Tutitoos
Tutitoos deleted the bugfix/196-auth-locale-rerender branch September 11, 2026 17:22
Tutitoos added a commit that referenced this pull request Sep 11, 2026
## Related issue

Closes #196

## Plan reference

Operational Phase 3 CI correction following the locale rerender fix.

## Summary

Apply the repository Biome formatting required by CI to the new locale
regression test.

## Changes

- Format the Playwright test callback using the canonical Biome layout.

## Validation

- [x] Relevant automated checks pass
- [x] Manual verification completed where needed

- `bun run lint` — passed, 81 files checked.
- Behavioral validation remains unchanged from #197: 50 unit tests and
24 bootstrap browser tests passed.

## Risks and rollback

Formatting-only change; no runtime behavior changes. Revert this PR to
roll back.

## Checklist

- [x] The PR has one coherent scope
- [x] The branch and `Closes` reference identify the same primary issue
- [x] The issue acceptance criteria are satisfied
- [x] No secrets, credentials, personal data, or unintended files are
included
- [x] Documentation and tests are updated when applicable
- [x] Breaking changes are explicitly documented (none)
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.

fix(web): rerender authentication after locale catalog loads

1 participant