Skip to content

fix(i18n): restore translated telemetry banner copy for opt-in semantics - #3

Merged
DevFlex-AI merged 3 commits into
mainfrom
fix/telemetry-locale-translations
Aug 6, 2026
Merged

fix(i18n): restore translated telemetry banner copy for opt-in semantics#3
DevFlex-AI merged 3 commits into
mainfrom
fix/telemetry-locale-translations

Conversation

@DevFlex-AI

@DevFlex-AI DevFlex-AI commented Aug 6, 2026

Copy link
Copy Markdown

The check-translations CI job fails on main because #2 removed the telemetry block from 17 locale welcome.json files (intending an English fallback), but scripts/find-missing-translations.js requires every locale to carry every key. This restores the block in all 17 locales with the new opt-in wording translated, e.g.:

"telemetry": {
	"helpImprove": "Hilf, Bolt Code zu verbessern",
	"helpImproveMessage": "Telemetrie ist standardmäßig deaktiviert. Wenn du möchtest, kannst du uns helfen, Fehler zu beheben und die Erweiterung zu verbessern, indem du die anonyme Fehler- und Nutzungsberichterstattung in den <settingsLink>Einstellungen</settingsLink> aktivierst. Es werden niemals Code, Prompts oder persönliche Informationen erfasst."
}

Each message mirrors the English opt-in copy ("Telemetry is off by default...") and reuses the previously translated helpImprove titles, preserving each locale's tone. node scripts/find-missing-translations.js now reports all translations complete.

Also updates the stale branding assertion in src/i18n/__tests__/tools-missing-param.spec.ts (it still expected "Zoo" after the Bolt rebrand), which was failing the platform-unit-test job on this branch and on main.


View with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is enabled.

Summary by CodeRabbit

  • Localization
    • Added translated telemetry setup messaging to the welcome experience across 18 supported languages.
    • Clarified that anonymous error and usage reporting is disabled by default and can be enabled in settings.
    • Added privacy messaging explaining what information is and is not collected.
    • Improved consistency of telemetry opt-in guidance and settings links across localized welcome experiences.

Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>
@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Added telemetry setup translations to 17 welcome-page locale files. Updated an English branding test to require “Bolt” and reject “Roo” and “Zoo”.

Changes

Telemetry localization

Layer / File(s) Summary
Localized telemetry messages
webview-ui/src/i18n/locales/*/welcome.json
Added translated telemetry headings and opt-in anonymous error and usage reporting messages with settings links.

Branding test update

Layer / File(s) Summary
English branding assertions
src/i18n/__tests__/tools-missing-param.spec.ts
Updated missing-parameter assertions to require “Bolt” and reject “Roo” and “Zoo”.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Suggested reviewers: edelauna

Poem

A rabbit checks each translated line,
While Bolt-branded tests align.
Settings links and messages glow,
Anonymous reports stay opt-in flow.
“Roo” and “Zoo” now hop away,
Clear strings land today.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Title check ✅ Passed The title clearly identifies the primary change: restoring translated telemetry copy with opt-in semantics.
Description check ✅ Passed The description explains the cause, implementation, affected locales, and validation command, but omits the issue link and completed checklist.
✨ 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 fix/telemetry-locale-translations

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

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 4

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@webview-ui/src/i18n/locales/id/welcome.json`:
- Line 15: Update the Indonesian helpImproveMessage translation so “anonymous”
clearly applies to both error and usage reporting, using wording equivalent to
“pelaporan anonim terkait kesalahan dan penggunaan” while preserving the rest of
the message.

In `@webview-ui/src/i18n/locales/it/welcome.json`:
- Line 15: Update the helpImproveMessage translation in
webview-ui/src/i18n/locales/it/welcome.json at lines 15-15 to use “nelle” before
the settingsLink label, and update webview-ui/src/i18n/locales/nl/welcome.json
at lines 15-15 to use “in de” before settingsLink; leave the remaining
translations unchanged.

In `@webview-ui/src/i18n/locales/ko/welcome.json`:
- Line 14: Update the helpImprove translation in the Korean welcome locale to
use grammatically correct Korean, replacing the current phrase with “Bolt Code
개선을 도와주세요” or “Bolt Code 개선에 도움을 주세요”.

In `@webview-ui/src/i18n/locales/pl/welcome.json`:
- Line 15: Update the Polish helpImproveMessage translation to use “prompty” or
the project’s established Polish term for user prompts instead of “podpowiedzi”,
preserving the statement that prompts, suggestions, and personal data are never
collected.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 21780043-f1bc-447c-8d73-5e4a51108017

📥 Commits

Reviewing files that changed from the base of the PR and between 5182da1 and d419635.

📒 Files selected for processing (17)
  • webview-ui/src/i18n/locales/ca/welcome.json
  • webview-ui/src/i18n/locales/de/welcome.json
  • webview-ui/src/i18n/locales/es/welcome.json
  • webview-ui/src/i18n/locales/fr/welcome.json
  • webview-ui/src/i18n/locales/hi/welcome.json
  • webview-ui/src/i18n/locales/id/welcome.json
  • webview-ui/src/i18n/locales/it/welcome.json
  • webview-ui/src/i18n/locales/ja/welcome.json
  • webview-ui/src/i18n/locales/ko/welcome.json
  • webview-ui/src/i18n/locales/nl/welcome.json
  • webview-ui/src/i18n/locales/pl/welcome.json
  • webview-ui/src/i18n/locales/pt-BR/welcome.json
  • webview-ui/src/i18n/locales/ru/welcome.json
  • webview-ui/src/i18n/locales/tr/welcome.json
  • webview-ui/src/i18n/locales/vi/welcome.json
  • webview-ui/src/i18n/locales/zh-CN/welcome.json
  • webview-ui/src/i18n/locales/zh-TW/welcome.json

Comment thread webview-ui/src/i18n/locales/id/welcome.json Outdated
Comment thread webview-ui/src/i18n/locales/it/welcome.json Outdated
Comment thread webview-ui/src/i18n/locales/ko/welcome.json Outdated
Comment thread webview-ui/src/i18n/locales/pl/welcome.json Outdated
DevFlex-AI and others added 2 commits August 6, 2026 03:54
Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>
Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>
@DevFlex-AI
DevFlex-AI merged commit 0e99df8 into main Aug 6, 2026
16 checks 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.

1 participant