Skip to content

fix: prevent browser translation from breaking React updates - #566

Merged
ixartz merged 1 commit into
mainfrom
translation-resilience
Sep 2, 2026
Merged

ixartz merged 1 commit into
mainfrom
translation-resilience

Conversation

@ixartz

@ixartz ixartz commented Sep 2, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Bug Fixes
    • Improved translation reliability by installing resilience support before the application initializes.
    • Helps prevent translation-related issues during initial page rendering.

@vercel

vercel Bot commented Sep 2, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
next-js-boilerplate Ready Ready Preview Sep 2, 2026 7:04pm UTC

@coderabbitai

coderabbitai Bot commented Sep 2, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. πŸŽ‰

ℹ️ Recent review info
βš™οΈ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 1bb8c0d5-1521-408f-bb35-91e024c7bdfc

πŸ“₯ Commits

Reviewing files that changed from the base of the PR and between 51dfd23 and 808b77c.

β›” Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
πŸ“’ Files selected for processing (2)
  • package.json
  • src/instrumentation-client.ts

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.


Walkthrough

The change adds translation-resilience version 0.3.0 and installs its DOM shim before Sentry initialization in the client instrumentation.

Changes

Translation resilience setup

Layer / File(s) Summary
Dependency and client initialization
package.json, src/instrumentation-client.ts
The project adds translation-resilience version 0.3.0. Client instrumentation calls installTranslationResilience() before Sentry.init.

Estimated code review effort: 2 (Simple) | ~5 minutes

Merge Risk: βšͺ Minimal Β· up to 808b7

This localized change adds translation resilience initialization before application rendering, with no actionable merge-blocking risk remaining beyond normal checks and review.

Poem

A rabbit checks the DOM at dawn
A tiny shim is neatly drawn
Sentry waits its turn in line
Translations now start up fine
Carrots cheer the clean design

πŸš₯ Pre-merge checks | βœ… 5
βœ… Passed checks (5 passed)
Check name Status Explanation
Description Check βœ… Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check βœ… Passed The title clearly summarizes the main change: adding translation resilience to prevent browser translation from disrupting React updates.
Docstring Coverage βœ… Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check βœ… Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check βœ… Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
πŸ“ Generate docstrings
  • Create stacked PR
  • Commit on current branch
πŸ§ͺ Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch translation-resilience

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❀️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov

codecov Bot commented Sep 2, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
βœ… Project coverage is 6.04%. Comparing base (51dfd23) to head (808b77c).

Files with missing lines Patch % Lines
src/instrumentation-client.ts 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##            main    #566      +/-   ##
========================================
- Coverage   6.07%   6.04%   -0.03%     
========================================
  Files         43      43              
  Lines        280     281       +1     
  Branches      29      29              
========================================
  Hits          17      17              
- Misses       242     243       +1     
  Partials      21      21              

β˜” View full report in Codecov by Harness.
πŸ“’ Have feedback on the report? Share it here.

πŸš€ New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • πŸ“¦ JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ixartz
ixartz merged commit fcddd32 into main Sep 2, 2026
12 of 13 checks passed
@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

πŸŽ‰ This PR is included in version 6.4.1 πŸŽ‰

The release is available on GitHub release

Your semantic-release bot πŸ“¦πŸš€

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant