From a05bbcf709328754871ba0b9d9d798b4dcfbe05d Mon Sep 17 00:00:00 2001
From: Yun Chan <yunchan@twentyoz.kr>
Date: Tue, 2 Jun 2026 10:06:25 +0900
Subject: [PATCH] =?UTF-8?q?refactor(P5b-5):=20=EB=B2=84=ED=8A=BC=20Click?=
 =?UTF-8?q?=20=ED=95=B8=EB=93=A4=EB=9F=AC=2010=EA=B0=9C=20=E2=86=92=20?=
 =?UTF-8?q?=EC=BB=A4=EB=A7=A8=EB=93=9C=20=EB=B0=94=EC=9D=B8=EB=94=A9?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

상호작용 핸들러 중 '커맨드화 가능한' 순수 버튼 클릭을 전부 RelayCommand + Command 바인딩으로 전환.

- 상단바 5개(Settings/Register/Diagnose/ExportLog/ClearAll) + 폴더선택/취소/미리보기열기
  + 항목제거(CommandParameter={Binding}) + 폴더열기(CommandParameter={Binding RevealPath}).
- OnRemoveQueueItem→RemoveQueueItem(QueueItem?), OnOpenFolderClick→OpenFolderForPath(string?) 추출
  (sender.Tag 의존 제거 → CommandParameter).
- 검증: 빌드 0/0, 83 테스트 그린, publish 후 메인 창 크래시 없이 로드(언핸들드 로그 없음).
- 잔여 핸들러는 MVVM에서도 뷰/behavior 영역: 드래그앤드롭, SelectionChanged/ValueChanged,
  미리보기 마우스클릭, 탭·충돌 ToggleButton 그룹(IsChecked 상호배제), 복합 상태 ProcessQueue 버튼.
---
 .../Views/MainWindow.xaml                     | 24 ++++----
 .../Views/MainWindow.xaml.cs                  | 59 ++++++++++++-------
 2 files changed, 51 insertions(+), 32 deletions(-)

diff --git a/src/Everything2Everything.App/Views/MainWindow.xaml b/src/Everything2Everything.App/Views/MainWindow.xaml
index 930ae13..3ff0122 100644
--- a/src/Everything2Everything.App/Views/MainWindow.xaml
+++ b/src/Everything2Everything.App/Views/MainWindow.xaml
@@ -161,7 +161,7 @@
                                              ToolTip="비워두면 원본 폴더 옆 서브폴더에 저장"/>
                                     <Button Grid.Column="1" Margin="8,0,0,0" Width="36"
                                             Content="…" Style="{StaticResource FsSecondaryButtonStyle}"
-                                            Click="OnPickOutputFolderClick"/>
+                                            Command="{Binding PickOutputFolderCommand, RelativeSource={RelativeSource AncestorType=Window}}"/>
                                 </Grid>
                                 <TextBlock x:Name="OutputDestHint" Margin="0,6,0,0"
                                            Style="{StaticResource FsCaptionStyle}"
@@ -300,7 +300,7 @@
                                     <Button x:Name="CancelProcessingButton" Grid.Column="1"
                                             Content="취소" Padding="10,2"
                                             Style="{StaticResource FsSecondaryButtonStyle}"
-                                            Click="OnCancelProcessingClick"/>
+                                            Command="{Binding CancelProcessingCommand, RelativeSource={RelativeSource AncestorType=Window}}"/>
                                 </Grid>
                             </StackPanel>
 
@@ -368,19 +368,19 @@
                         <StackPanel Grid.Column="1" Orientation="Horizontal">
                             <Button Content="⚙ 설정" Margin="0,0,8,0"
                                     Style="{StaticResource FsSecondaryButtonStyle}"
-                                    Click="OnSettingsClick"/>
+                                    Command="{Binding SettingsCommand, RelativeSource={RelativeSource AncestorType=Window}}"/>
                             <Button Content="Register Menu" Margin="0,0,8,0"
                                     Style="{StaticResource FsSecondaryButtonStyle}"
-                                    Click="OnRegisterClick"/>
+                                    Command="{Binding RegisterCommand, RelativeSource={RelativeSource AncestorType=Window}}"/>
                             <Button Content="Diagnose" Margin="0,0,8,0"
                                     Style="{StaticResource FsSecondaryButtonStyle}"
-                                    Click="OnDiagnoseClick"/>
+                                    Command="{Binding DiagnoseCommand, RelativeSource={RelativeSource AncestorType=Window}}"/>
                             <Button Content="Export Log" Margin="0,0,8,0"
                                     Style="{StaticResource FsSecondaryButtonStyle}"
-                                    Click="OnExportLogClick"/>
+                                    Command="{Binding ExportLogCommand, RelativeSource={RelativeSource AncestorType=Window}}"/>
                             <Button x:Name="ClearAllButton" Content="Clear All"
                                     Style="{StaticResource FsSecondaryButtonStyle}"
-                                    Click="OnClearAllClick"/>
+                                    Command="{Binding ClearAllCommand, RelativeSource={RelativeSource AncestorType=Window}}"/>
                         </StackPanel>
                     </Grid>
                 </Border>
@@ -500,8 +500,8 @@
                                                             VerticalAlignment="Center"
                                                             HorizontalAlignment="Right"
                                                             ToolTip="큐에서 제거"
-                                                            Click="OnRemoveQueueItem"
-                                                            Tag="{Binding}">
+                                                            Command="{Binding RemoveQueueItemCommand, RelativeSource={RelativeSource AncestorType=Window}}"
+                                                            CommandParameter="{Binding}">
                                                         <Path Width="13" Height="13" Stretch="Uniform"
                                                               Stroke="{StaticResource FsTextTertiary}"
                                                               StrokeThickness="2"
@@ -618,8 +618,8 @@
                                                                     Background="Transparent"
                                                                     BorderThickness="0"
                                                                     Padding="6"
-                                                                    Click="OnOpenFolderClick"
-                                                                    Tag="{Binding RevealPath}"
+                                                                    Command="{Binding OpenFolderCommand, RelativeSource={RelativeSource AncestorType=Window}}"
+                                                                    CommandParameter="{Binding RevealPath}"
                                                                     ToolTip="출력 파일이 있는 폴더 열기">
                                                                 <Path Width="14" Height="14" Stretch="Uniform"
                                                                       Stroke="{StaticResource FsTextTertiary}"
@@ -779,7 +779,7 @@
 
                                     <Button Content="Open in Explorer" Margin="0,16,0,0"
                                             Style="{StaticResource FsSecondaryButtonStyle}"
-                                            Click="OnPreviewOpenFolder"
+                                            Command="{Binding PreviewOpenFolderCommand, RelativeSource={RelativeSource AncestorType=Window}}"
                                             HorizontalAlignment="Stretch"/>
                                 </StackPanel>
                             </Border>
diff --git a/src/Everything2Everything.App/Views/MainWindow.xaml.cs b/src/Everything2Everything.App/Views/MainWindow.xaml.cs
index 512370b..ded3f96 100644
--- a/src/Everything2Everything.App/Views/MainWindow.xaml.cs
+++ b/src/Everything2Everything.App/Views/MainWindow.xaml.cs
@@ -39,6 +39,18 @@ public partial class MainWindow : Wpf.Ui.Controls.FluentWindow
     public ICommand CloseCommand { get; }
     public ICommand RefreshCommand { get; }
 
+    // 상단바·액션 버튼 커맨드 (P5b: Click 핸들러 → 커맨드 바인딩)
+    public ICommand SettingsCommand { get; }
+    public ICommand RegisterCommand { get; }
+    public ICommand DiagnoseCommand { get; }
+    public ICommand ExportLogCommand { get; }
+    public ICommand ClearAllCommand { get; }
+    public ICommand PickOutputFolderCommand { get; }
+    public ICommand CancelProcessingCommand { get; }
+    public ICommand PreviewOpenFolderCommand { get; }
+    public ICommand RemoveQueueItemCommand { get; }
+    public ICommand OpenFolderCommand { get; }
+
     public MainWindow(ConversionEngine engine, ISettingsStore settings, IReadOnlyList<string>? initialFiles = null)
     {
         _engine = engine;
@@ -50,6 +62,17 @@ public partial class MainWindow : Wpf.Ui.Controls.FluentWindow
         CloseCommand = new RelayCommand(_ => Close());
         RefreshCommand = new RelayCommand(_ => ApplyAppDataStats());
 
+        SettingsCommand = new RelayCommand(_ => OnSettingsClick(this, new RoutedEventArgs()));
+        RegisterCommand = new RelayCommand(_ => OnRegisterClick(this, new RoutedEventArgs()));
+        DiagnoseCommand = new RelayCommand(_ => OnDiagnoseClick(this, new RoutedEventArgs()));
+        ExportLogCommand = new RelayCommand(_ => OnExportLogClick(this, new RoutedEventArgs()));
+        ClearAllCommand = new RelayCommand(_ => OnClearAllClick(this, new RoutedEventArgs()));
+        PickOutputFolderCommand = new RelayCommand(_ => OnPickOutputFolderClick(this, new RoutedEventArgs()));
+        CancelProcessingCommand = new RelayCommand(_ => OnCancelProcessingClick(this, new RoutedEventArgs()));
+        PreviewOpenFolderCommand = new RelayCommand(_ => OnPreviewOpenFolder(this, new RoutedEventArgs()));
+        RemoveQueueItemCommand = new RelayCommand(p => RemoveQueueItem(p as QueueItem));
+        OpenFolderCommand = new RelayCommand(p => OpenFolderForPath(p as string));
+
         InitializeComponent();
 
         // ActiveQueueList/PastResultsList의 ItemsSource는 XAML이 ActiveQueue/PastResults에 바인딩(선언적).
@@ -201,16 +224,14 @@ public partial class MainWindow : Wpf.Ui.Controls.FluentWindow
         }
     }
 
-    private void OnRemoveQueueItem(object sender, RoutedEventArgs e)
+    private void RemoveQueueItem(QueueItem? item)
     {
-        if (sender is FrameworkElement fe && fe.Tag is QueueItem item)
-        {
-            _activeQueue.Remove(item);
-            UpdateBadges();
-            UpdateProcessQueueButton();
-            UpdateActiveQueueVisibility();
-            RefreshAvailableOutputFormats();
-        }
+        if (item is null) return;
+        _activeQueue.Remove(item);
+        UpdateBadges();
+        UpdateProcessQueueButton();
+        UpdateActiveQueueVisibility();
+        RefreshAvailableOutputFormats();
     }
 
     private void UpdateActiveQueueVisibility()
@@ -811,21 +832,19 @@ public partial class MainWindow : Wpf.Ui.Controls.FluentWindow
         ApplyAppDataStats();
     }
 
-    private void OnOpenFolderClick(object sender, RoutedEventArgs e)
+    private static void OpenFolderForPath(string? path)
     {
-        if (sender is FrameworkElement fe && fe.Tag is string path && File.Exists(path))
+        if (string.IsNullOrEmpty(path) || !File.Exists(path)) return;
+        try
         {
-            try
+            System.Diagnostics.Process.Start(new System.Diagnostics.ProcessStartInfo
             {
-                System.Diagnostics.Process.Start(new System.Diagnostics.ProcessStartInfo
-                {
-                    FileName = "explorer.exe",
-                    Arguments = $"/select,\"{path}\"",
-                    UseShellExecute = true,
-                });
-            }
-            catch { }
+                FileName = "explorer.exe",
+                Arguments = $"/select,\"{path}\"",
+                UseShellExecute = true,
+            });
         }
+        catch { }
     }
 
     public static string HumanizeBytes(long bytes)