Repository navigation
feat(install): add 18-language desc translations to package yml files - #125
Merged
Merged
Conversation
Add 16 language keys (cs de es fr id it ja ko pl pt ro ru th tr uk vi) to the top-level `desc` block of all 2402 src/**/*.yml files, alongside the existing `cn`/`en` values. - Each yml gains exactly 16 lines (git diff --numstat shows 16/0); the existing cn/en values and every other line are byte-identical to before. - Translations are based on the English `en` value, falling back to `cn` only where `en` is ambiguous. Command names, product names, proper nouns and format/protocol names are left untranslated. - Nested `desc` blocks under `footprint` are deliberately left untouched; only the top-level package description is translated. - Update .vscode/install.schema.json and .x-cmd/install-yml-check.schema.json (the two copies must stay in sync) so I18nDescription accepts the new keys. It previously allowed only cn/en with additionalProperties: false, which made every yml fail the schema check. Verified: all 2402 files parse and carry 18 non-empty language values; `.x-cmd/lint.sh src/` reports success 2402 / schema-error 0, and the Deno field linter passes. Note: the x-cmd runtime currently resolves `desc` for cn/en only, so the new keys are inert data until the core gains language selection.
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Adds 16 language keys to the top-level
descblock of all 2402src/**/*.ymlfiles, alongside the existingcn/envalues:How
git diff --numstatreports16/0for every one of the 2402 files. The existingcn/envalues and every other line are byte-identical to before — verified by diffing every parseddescagainstHEAD.envalue is authoritative;cnis consulted only whereenis ambiguous. One short sentence per language, comparable in length toen. Command names, project/product names, proper nouns and format/protocol names (JSON, YAML, Git, GitHub, GPT-4, AWS, S3, …) are left untranslated.descblocks are untouched. Many files carry additionaldescblocks underfootprint(resource sizes, install estimates). Only the top-level package description is translated.:,#, leading YAML indicator) are double-quoted with internal"escaped. No trailing whitespace on any added line.Schema
I18nDescriptionin.vscode/install.schema.jsonpreviously allowed onlycn/enwithadditionalProperties: false, so every yml would have failed the schema check once the new keys landed. Both schema copies are updated —.vscode/install.schema.jsonand.x-cmd/install-yml-check.schema.json— since the repo's lint explicitly fails when those two drift apart.Verification
.x-cmd/lint.sh src/→ success 2402 / schema-error 0 (this is the same command CI runs)..x-cmd/lint-yml.deno.ts) passes.Note
The x-cmd runtime currently resolves
descforcn/enonly (advise.obj.awknormalizes___X_CMD_LANGto zh/en;pkg/lib/awk/info.awkreadsdesc[region]). The new keys are therefore inert data for now — the core needs its own language-selection change before the other 16 languages are actually rendered. This PR is the data half of that work.🤖 Generated with Claude Code