Skip to content

feat(i18n): add Turkish (tr) language support and README - #13196

Closed
ardakrt wants to merge 2 commits into
stablyai:mainfrom
ardakrt:feat/add-turkish-localization
Closed

ardakrt wants to merge 2 commits into
stablyai:mainfrom
ardakrt:feat/add-turkish-localization

Conversation

@ardakrt

@ardakrt ardakrt commented Aug 8, 2026 •

Copy link
Copy Markdown

Summary

This PR adds full Turkish (tr) language support to the StablyAI Orca Agent Development Environment and includes a translated \README.tr.md.

Changes Included:

  • Shared Types: Registered \UI_LANGUAGE_TURKISH = 'tr'\ in \ui-language.ts\ and \ui-locale.ts.
  • UI & Main Loaders: Registered dynamic locale loaders in \supported-languages.ts, \i18n.ts, and \main-i18n.ts.
  • Locale Catalog: Created \ r.json\ with 100% key parity (12,375 keys).
  • Documentation: Added \docs/readme/README.tr.md\ and updated the language navbar in \README.md.

Testing

  • Ran
    ode config/scripts/verify-localization-catalog.mjs\ (12375/12375 translated, 0 missing).
  • Ran \pnpm run tc\ (0 TypeScript errors).
  • Ran \�itest run src/renderer/src/i18n\ (134/134 unit tests passing).

Screenshots

  • N/A - i18n text changes only (no visual UI layout structure changes).

AI Review Report

  • Verified cross-platform compatibility across Windows, macOS, and Linux (shortcuts, labels, paths, Electron differences).
  • Verified zero missing keys in \ r.json\ via \�erify-localization-catalog.mjs.

Security Audit

  • Verified zero unsafe string interpolations or unescaped HTML scripts in \ r.json.

Notes

  • All 134 unit tests in \src/renderer/src/i18n\ passed.

@ardakrt
ardakrt requested a review from brennanb2025 as a code owner August 8, 2026 13:39
@coderabbitai

coderabbitai Bot commented Aug 8, 2026 •

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

The PR adds Turkish as a built-in UI language. It updates language validation, locale resolution, lazy locale loading, language selector labels, and fallback text. It adds Turkish README documentation and links it from the main README language navigation.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
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 and concisely describes the primary change: Turkish language support and the translated README.
Description check ✅ Passed The description covers the summary, screenshots, testing, AI review, security audit, and notes, with sufficient detail for this localization change.

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.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 4


ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 95c518d2-4d8d-4e27-8b9e-18b130829ccc

📥 Commits

Reviewing files that changed from the base of the PR and between 54a9b58 and 713b7d5.

📒 Files selected for processing (13)
  • README.md
  • docs/readme/README.tr.md
  • src/main/i18n/main-i18n.ts
  • src/renderer/src/i18n/i18n.ts
  • src/renderer/src/i18n/locales/en.json
  • src/renderer/src/i18n/locales/es.json
  • src/renderer/src/i18n/locales/ja.json
  • src/renderer/src/i18n/locales/ko.json
  • src/renderer/src/i18n/locales/tr.json
  • src/renderer/src/i18n/locales/zh.json
  • src/renderer/src/i18n/supported-languages.ts
  • src/shared/ui-language.ts
  • src/shared/ui-locale.ts

Comment thread docs/readme/README.tr.md Resolved
Comment thread docs/readme/README.tr.md Resolved
Comment thread src/renderer/src/i18n/i18n.ts Resolved
Comment thread src/renderer/src/i18n/locales/es.json Resolved
@ardakrt

ardakrt commented Aug 12, 2026

Copy link
Copy Markdown
Author

Hi @brennanb2025, just checking in to see if any further changes or adjustments are needed on this PR. All automated localization catalog checks (\�erify-localization-catalog.mjs), TypeScript type checks, and i18n unit tests are passing cleanly. Thanks!

@OrcaWin

OrcaWin commented Oct 28, 2026

Copy link
Copy Markdown
Collaborator

Thank you for this! Turkish (Türkçe) is now a built-in UI language as of #34718, so I'm closing this in its favor. If you spot wording that reads wrong, a follow-up PR against src/renderer/src/i18n/locales/tr.json (glossary in config/i18n-turkish.md) would be very welcome.

@OrcaWin OrcaWin closed this Oct 28, 2026
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.

5 participants