Skip to content

feat: add desktop localization codegen#26

Merged
Aerilym merged 7 commits into
mainfrom
feat/desktop_localization_codegen
Jan 12, 2026
Merged

feat: add desktop localization codegen#26
Aerilym merged 7 commits into
mainfrom
feat/desktop_localization_codegen

Conversation

@Aerilym

@Aerilym Aerilym commented Jan 7, 2026

Copy link
Copy Markdown
Collaborator

Comment thread crowdin/codegen_desktop.py Outdated
Copilot AI review requested due to automatic review settings January 7, 2026 22:44

Copilot AI 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.

Pull request overview

This PR introduces a new Python code generator for Session Desktop localization files, replacing the previous string generation approach. The new script generates TypeScript type definitions and translations in a sparse format (only actual translations, no English duplicates).

  • Adds codegen_desktop.py to generate TypeScript localization files with type safety
  • Updates GitHub Actions workflow to use the new code generator
  • Changes output structure to generate files under ts/localization/generated/

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 13 comments.

File Description
crowdin/codegen_desktop.py New Python script that generates TypeScript localization files (locales.ts, english.ts, translations.ts, constants.ts) with type definitions and sparse translations
.github/workflows/check_for_crowdin_updates.yml Updated workflow to call the new codegen script instead of the old string generator, removed Desktop checkout step, and updated artifact paths

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/check_for_crowdin_updates.yml Outdated
Comment thread crowdin/codegen_desktop.py Outdated
Comment thread crowdin/codegen_desktop.py Outdated
Comment thread .github/workflows/check_for_crowdin_updates.yml Outdated
Comment thread crowdin/codegen_localization.py
Comment thread crowdin/codegen_desktop.py Outdated
Comment thread crowdin/codegen_desktop.py Outdated
Comment thread crowdin/codegen_desktop.py Outdated
Comment thread crowdin/codegen_desktop.py Outdated
Comment thread crowdin/codegen_desktop.py Outdated
@Aerilym Aerilym merged commit f5fd041 into main Jan 12, 2026
34 of 36 checks passed
@Aerilym Aerilym deleted the feat/desktop_localization_codegen branch January 12, 2026 02:19
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.

4 participants