Skip to content

feat(upload): PR作成前のLua構文チェックとエラーダイアログを追加#93

Merged
5kdn merged 1 commit intomasterfrom
feature/5
Mar 23, 2026
Merged

feat(upload): PR作成前のLua構文チェックとエラーダイアログを追加#93
5kdn merged 1 commit intomasterfrom
feature/5

Conversation

@5kdn
Copy link
Copy Markdown
Owner

@5kdn 5kdn commented Mar 23, 2026

📌 概要

CreatePullRequest から PR を作成する前に Lua ファイルの構文チェックを実施するようにした。
*.lua*.cmpdictionary を対象に構文検証を行い、失敗時は専用ダイアログで対象ファイルを表示して PR 作成を中断できるようにした。

🛠 変更内容

  • ILuaSyntaxValidationService と関連モデルを追加し、Lua 構文検証の契約を Application 層に追加
  • MoonSharp を利用する LuaSyntaxValidationService を Infrastructure 層に追加
  • CreatePullRequest 実行前にアップロード対象ファイルの Lua 構文チェックを行う処理を追加
  • 構文チェック失敗時に表示する専用ダイアログと DialogHost 識別子を追加
  • CreatePullRequest 画面から専用ダイアログを表示するように変更
  • PR 作成ダイアログ用の文言をリソースへ追加
  • Lua 構文検証サービスの単体テストを追加
  • CreatePullRequestViewModel の単体テストを追加・更新

留意点

  • 構文チェック対象は PR 作成時のアップロード対象ファイルのうち *.lua*.cmpdictionary のみ
  • 構文チェックに失敗した場合、PR は作成されずダイアログで失敗ファイルを確認して再試行または中止を選択する
  • 関連テスト:
    • dotnet build DCS-Translation-Tool.sln -c Release --no-restore
    • dotnet test tests/DcsTranslationTool.Infrastructure.UnitTests/DcsTranslationTool.Infrastructure.UnitTests.csproj -c Release --no-restore
    • dotnet test tests/DcsTranslationTool.Presentation.Wpf.UnitTests/DcsTranslationTool.Presentation.Wpf.UnitTests.csproj -c Release --no-restore --filter CreatePullRequestViewModelTests

Closes #5

@5kdn 5kdn merged commit 709eca2 into master Mar 23, 2026
4 checks passed
@5kdn 5kdn deleted the feature/5 branch March 23, 2026 10:28
@5kdn 5kdn mentioned this pull request Mar 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: luaファイルをアップロードする前に構文チェックを追加する

1 participant