Skip to content
This repository was archived by the owner on Jun 1, 2026. It is now read-only.

fix(i18n): fill missing translations in Japanese and Korean locale files#28

Merged
devinoldenburg merged 3 commits into
mainfrom
fix/i18n-and-docs-improvements
Jun 1, 2026
Merged

fix(i18n): fill missing translations in Japanese and Korean locale files#28
devinoldenburg merged 3 commits into
mainfrom
fix/i18n-and-docs-improvements

Conversation

@devinoldenburg
Copy link
Copy Markdown
Collaborator

@devinoldenburg devinoldenburg commented May 31, 2026

Summary

Fill in empty translation entries in Japanese and Korean locale files, and fix documentation gaps in the CLI reference and configuration schema docs.

Changes

i18n

  • packages/app/src/i18n/ja.ts: restored Japanese translations for provider.connect.oauth.code.visit.prefix and provider.connect.oauth.auto.visit.prefix (were " ", now "アクセス: ").
  • packages/ui/src/i18n/ja.ts: filled ui.lineComment.label.prefix ( → "行目: ") and ui.lineComment.editorLabel.prefix ( → "コメント編集中: ").
  • packages/ui/src/i18n/ko.ts: filled ui.lineComment.label.prefix ( → "라인 ") and ui.lineComment.editorLabel.prefix ( → "편집 중 ").

Documentation

CLI reference (site/app/docs/cli/page.tsx)

  • Metadata & synopsis: add totp and generate to synopsis; totp included in metadata descriptions (title, openGraph, twitter) so they match the listed commands.
  • New codeplane totp section: documents generate (--account, --issuer), uri (--secret, --account, --issuer), and code (--secret) subcommands — all present in packages/codeplane/src/cli/cmd/totp.ts but previously absent from the docs.
  • Generated/hidden commands: corrected wording to reference describe: false rather than "intentionally hidden from normal help".
  • completion was already documented; metadata now correctly covers totp alongside it.

Configuration reference (site/app/docs/configuration/page.tsx)

  • Top-level shape JSON: add disabled_providers, enabled_providers, username, default_agent, npm, and watcher, plus tail_turns and reserved inside compaction — all defined in packages/codeplane/src/config/config.ts.
  • New npm section: documents registry and authToken with {secret:…} / {env:…} placeholder support.
  • Runtime behavior table: add rows for username, default_agent, disabled_providers, enabled_providers, compaction.tail_turns, and compaction.reserved — all previously undocumented.
  • small_model was already documented in the provider/models section.

Verification

  • bun --cwd packages/app test test/i18n/parity.test.ts — 652 pass
  • bun test packages/ui/src/i18n/i18n.test.ts — 51 pass
  • bun --cwd site run typecheck — pass
  • bun --cwd site run build — pass
  • bun --cwd packages/app test --preload ./happydom.ts ./src — 652 pass, 0 fail
  • bun --cwd packages/ui test — 362 pass, 0 fail

Codeplane Agent and others added 2 commits May 31, 2026 21:26
- app/ja.ts: restored Japanese translations for provider OAuth visit prefixes
  ("Visit ") that had been left as a formatting space
- ui/ja.ts: filled empty lineComment label/editorLabel prefixes
  ("行目: ", "コメント編集中: ")
- ui/ko.ts: filled empty lineComment label/editorLabel prefixes
  ("라인 ", "편집 중 ")

All 652 app and 51 UI i18n parity tests pass. Lint shows 0 errors.

Co-Authored-By: codeplane-agent[bot] <287208015+codeplane-agent[bot]@users.noreply.github.com>
…docs

CLI reference:
- Add totp command (generate, uri, code subcommands with flags) to synopsis
  and new dedicated section; totp was an implemented but undocumented
  public subcommand.
- Add generate to synopsis and update Generated/Hidden section to
  accurately describe its describe:false hidden status.
- Update metadata (title, openGraph, twitter) to list totp alongside
  the other documented subcommands so SEO metadata matches the synopsis.
- completion subcommand was already documented; no structural change
  there beyond the metadata alignment.

Configuration reference:
- Add disabled_providers and enabled_providers to the top-level shape
  JSON example and the runtime behavior table.
- Add username, default_agent, npm, and watcher to the top-level shape
  JSON example (all present in packages/codeplane/src/config/config.ts
  but missing from the page's example block).
- Add a new dedicated npm section with registry/authToken example and
  explanation of {secret} / {env} placeholder support.
- Add compaction.tail_turns, compaction.reserved, disabled_providers,
  enabled_providers, username, and default_agent to the runtime behavior
  table (all defined in config.ts, none of which were listed).
- Round out the compaction example to include the new tail_turns and
  reserved keys.

Verification:
- bun --cwd site run typecheck (pass)
- bun --cwd site run build (pass)
- bun --cwd packages/app test --preload ./happydom.ts ./src (652 pass)
- bun --cwd packages/ui test (362 pass)

Co-Authored-By: codeplane-agent[bot] <287208015+codeplane-agent[bot]@users.noreply.github.com>
@devinoldenburg
Copy link
Copy Markdown
Collaborator Author

Review Notes

Verdict: APPROVED — ready to merge

What this does

  • Fills previously empty Japanese and Korean i18n entries (line comment labels, OAuth prefixes)
  • Updates CLI and configuration docs to match the actual codebase (adds totp, generate, disabled_providers, enabled_providers, npm, watcher, compaction.tail_turns, compaction.reserved, etc.)

Review

  • i18n changes — straightforward translation fills, matching the existing locale style.
  • site/app/docs/cli/page.tsxtotp and generate were indeed missing from the docs while present in code. Good catch.
  • site/app/docs/configuration/page.tsx — all added keys map 1:1 to config fields in packages/codeplane/src/config/config.ts. No speculative docs.

No issues found

This is a docs/i18n-only change with no runtime impact. Safe to merge.

Copilot AI review requested due to automatic review settings June 1, 2026 01:01
@devinoldenburg devinoldenburg merged commit 9e9b032 into main Jun 1, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Fills in previously empty translation entries in three i18n files (Japanese app + UI, Korean UI) and brings the CLI and configuration reference docs back in sync with the implementation in packages/codeplane/src/cli and packages/codeplane/src/config/config.ts.

Changes:

  • Restore Japanese/Korean strings for provider.connect.oauth.*.visit.prefix and ui.lineComment.* entries that were blank.
  • Document the codeplane totp command tree (and update metadata/synopsis) in the CLI reference, plus reword the "generated/hidden commands" section.
  • Document previously missing top-level config keys (disabled_providers, enabled_providers, username, default_agent, npm, watcher, compaction.tail_turns, compaction.reserved) and add an npm section + new runtime-behavior table rows.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
packages/app/src/i18n/ja.ts Fills two empty OAuth visit-prefix strings with "アクセス: ".
packages/ui/src/i18n/ja.ts Fills empty ui.lineComment prefix strings.
packages/ui/src/i18n/ko.ts Fills empty ui.lineComment prefix strings.
site/app/docs/cli/page.tsx Adds totp (and generate) to synopsis/metadata, documents totp subcommands, rewords hidden-commands paragraph.
site/app/docs/configuration/page.tsx Adds missing top-level keys to shape JSON, new npm section, and new runtime-behavior table rows.

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

Comment on lines +45 to 46
generate regenerate the OpenAPI spec (SDK build pipeline)
completion generate shell completion script
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants