Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

i18n #952

Merged
merged 18 commits into from
Mar 28, 2025
Merged

i18n #952

merged 18 commits into from
Mar 28, 2025

Conversation

maekawataiki
Copy link
Collaborator

@maekawataiki maekawataiki commented Mar 21, 2025

変更内容の説明

  • UI の多言語化
  • コメントは英語に
  • (Braking Change) デフォルトユーズケースのプロンプトを英語に。渡された言語で回答するように指示を追加。一部プロンプトを改善。
  • プロンプト例とユーズケースビルダーのプロンプトはユーザーがプロンプトを自作する際に参考になるように多言語化
  • エラーメッセージは英語に
  • Kendra のデータソース言語は cdk.json の kendraIndexLanguage で変更可能に
  • KB はデフォルトのドキュメントとデータソースは日本語のまま、利用者側で新規データソース作成時に言語を指定および OpenSearch の Analyzer も必要に応じてカスタマイズして作成を想定
  • ドキュメントの多言語化

設計上の検討点

  • 多言語のメンテナンス性のために eslint (既成 + カスタムルール)、i18n Ally (vscode 推奨設定を追加)を導入。
  • 多言語化ファイルは YAML 表記、分割、Lazy Loading しバンドルサイズの最適化とメンテナンス性を向上。

TODO

チェック項目

  • npm run lint を実行した
  • 関連するドキュメントを修正した
  • 手元の環境で動作確認済み
  • npm run cdk:test を実行しスナップショット差分がある場合は npm run cdk:test:update-snapshot を実行してスナップショットを更新した

関連する Issue

#151

Sorry, something went wrong.

@tbrand tbrand changed the title [DNM] i18n i18n Mar 28, 2025
@tbrand tbrand merged commit 6962d46 into v4 Mar 28, 2025
@tbrand tbrand deleted the i18n branch March 28, 2025 06:49
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.

None yet

2 participants