TranslationServiceに翻訳対象の情報を引数で渡せるようにする #36
Merged
Conversation
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
`AITranslator.cs` で `sourceLanguage` を nullable に変更し、プロンプト生成のための `CreatePrompt` メソッドを追加しました。 `Program.cs` では設定の取得方法を変更し、`ParameterNotSetException` を導入しました。 `ITranslator.cs` の `TranslateAsync` メソッドも `sourceLanguage` を nullable に更新。 `Messages` リソースに無効な文化名のメッセージを追加し、`ServiceCollectionExtensions.cs` では設定バインディングを削除しました。 `TranslationService.cs` ではコンストラクタから `CoreSettings` のオプションを削除し、直接パラメータを受け取るように変更。 `CoreSettings.cs` ではプロパティの直接使用を促進し、`ConvertTo` メソッドを統合。 テストコードも新しいメソッドシグネチャに合わせて更新し、`ParameterNotSetException` クラスを新たに追加しました。
`outputDirectoryPath`の取得方法を変更し、値が設定されていない場合には空の文字列が代入されるようにしました。これにより、常に有効な文字列が保証されます。
DumpCoreSettings というローカライズされた文字列が `Messages.Designer.cs` と `Messages.resx` から削除されました。この変更により、"CoreSetting variables: {Variables}" という文字列はもはや利用できなくなります。`DirectoryCreated` と `InvalidXmlString` のリソースは引き続き残っています。
`outputDirectoryPath`の取得方法を変更し、デフォルト値として空の文字列を指定するようにしました。これにより、`OutputDirectoryPath`が設定されていない場合に、より明示的に空の文字列が返されるようになります。
`AISettings`の設定方法を明確にし、`TranslationService`の`ExecuteAsync`メソッドに必要な引数を追加しました。`CoreSettings`に関する記述を削除し、`AddTranslatorServices`の呼び出しを簡略化しました。これにより、AIを利用した翻訳機能の利用が促進されます。
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the translation functionality by removing the CoreSettings dependency and enabling the TranslationService to receive translation parameters as arguments. It updates the service registration, test cases, CLI configuration, and AI translation logic accordingly.
- Removed CoreSettings and updated TranslationService to accept explicit parameters.
- Updated tests and CLI to reflect the new API signature.
- Adjusted DI registration and documentation for the new configuration approach.
Reviewed Changes
Copilot reviewed 14 out of 16 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| tests/Test.TsunaCan.XmlDocumentationTranslator.Core/TranslationServiceTest.cs | Updated test to pass translation parameters and adjusted log count expectation. |
| tests/Test.TsunaCan.XmlDocumentationTranslator.Core/CoreSettingsTest.cs | Removed obsolete tests for CoreSettings. |
| src/TsunaCan.XmlDocumentationTranslator.Core/TranslationService.cs | Changed ExecuteAsync API to accept parameters; removed CoreSettings usage. |
| src/TsunaCan.XmlDocumentationTranslator.Core/ServiceCollectionExtensions.cs | Updated DI registration by removing CoreSettings configuration. |
| src/TsunaCan.XmlDocumentationTranslator.Core/README.md | Updated documentation with the new API usage. |
| src/TsunaCan.XmlDocumentationTranslator.Cli/Program.cs | Modified to retrieve configuration values and call the new ExecuteAsync signature. |
| src/TsunaCan.XmlDocumentationTranslator.Cli/ParameterNotSetException.cs | Introduced exception for missing required parameters. |
| src/TsunaCan.XmlDocumentationTranslator.AI/AITranslator.cs | Updated translation prompt creation to accommodate nullable source language. |
| src/TsunaCan.XmlDocumentationTranslator.AI/README.md | Updated documentation to reflect changes in configuration requirements. |
Files not reviewed (2)
- src/TsunaCan.XmlDocumentationTranslator.Cli/Resources/Messages.Designer.cs: Language not supported
- src/TsunaCan.XmlDocumentationTranslator.Core/Resources/Messages.Designer.cs: Language not supported
Comments suppressed due to low confidence (1)
src/TsunaCan.XmlDocumentationTranslator.Core/TranslationService.cs:46
- The current list initialization using empty square brackets may not be valid for a List. Consider replacing it with 'new List()' for proper instantiation.
List<string> outputFiles = [];
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.
この Pull request で実施したこと
CoreSettingsクラスを削除しました。
TranlationServiceに情報を引数で渡せるようにしました。
この Pull request では実施していないこと
なし。
Issues や Discussions 、関連する Web サイトなどへのリンク
なし。