src/modules/cmdpal/ext/Microsoft.CmdPal.Ext.WindowWalker/Commands/EndTaskCommand.cs - main - githubmirror/PowerToys

src/modules/cmdpal/ext/Microsoft.CmdPal.Ext.WindowWalker/Commands/EndTaskCommand.cs

74 строки3 KB

Jiří Polášek

CmdPal: Make Window Walker Close window command respect "Keep open" option (#45721)

28 мар 2026, 02:46

28 мар 2026, 02:465792d32

// Copyright (c) Microsoft Corporation
// The Microsoft Corporation licenses this file to you under the MIT license.
// See the LICENSE file in the project root for more information.

using System;
using CommunityToolkit.Mvvm.Messaging;
using Microsoft.CmdPal.Ext.WindowWalker.Components;
using Microsoft.CmdPal.Ext.WindowWalker.Helpers;
using Microsoft.CmdPal.Ext.WindowWalker.Messages;
using Microsoft.CmdPal.Ext.WindowWalker.Properties;
using Microsoft.CommandPalette.Extensions;

namespace Microsoft.CmdPal.Ext.WindowWalker.Commands;

internal sealed partial class EndTaskCommand : InvokableCommand
{
    private readonly Window _window;

    public EndTaskCommand(Window window)
    {
        Icon = Icons.EndTask;
        Name = $"{Resources.windowwalker_Kill}";
        _window = window;
    }

    /// <summary>
    /// Method to initiate killing the process of a window
    /// </summary>
    /// <param name="window">Window data</param>
    /// <returns>True if the PT Run window should close; otherwise, false.</returns>
    private static bool KillProcess(Window window)
    {
        // Validate process
        if (!window.IsWindow || !window.Process.DoesExist || string.IsNullOrEmpty(window.Process.Name) || !window.Process.Name.Equals(WindowProcess.GetProcessNameFromProcessID(window.Process.ProcessID), StringComparison.Ordinal))
        {
            ExtensionHost.LogMessage(new LogMessage { Message = $"Cannot kill process '{window.Process.Name}' ({window.Process.ProcessID}) of the window '{window.Title}' ({window.Hwnd}), because it doesn't exist." });

            // TODO GH #86 -- need to figure out how to show status message once implemented on host
            return false;
        }

        // Request user confirmation
        if (SettingsManager.Instance.ConfirmKillProcess)
        {
            // TODO GH #138, #153 -- need to figure out how to confirm kill process? should this just be the same status thing... maybe not? Need message box? Could be nested context menu.
            /*
            string messageBody = $"{Resources.wox_plugin_windowwalker_KillMessage}\n"
                + $"{window.Process.Name} ({window.Process.ProcessID})\n\n"
                + $"{(window.Process.IsUwpApp ? Resources.wox_plugin_windowwalker_KillMessageUwp : Resources.wox_plugin_windowwalker_KillMessageQuestion)}";
            MessageBoxResult messageBoxResult = MessageBox.Show(
                messageBody,
                Resources.wox_plugin_windowwalker_plugin_name + " - " + Resources.wox_plugin_windowwalker_KillMessageTitle,
                MessageBoxButton.YesNo,
                MessageBoxImage.Warning);

            if (messageBoxResult == MessageBoxResult.No)
            {
                return false;
            }
            */
        }

        // Kill process
        window.Process.KillThisProcess(SettingsManager.Instance.KillProcessTree);
        return !SettingsManager.Instance.KeepOpenAfterKillAndClose;
    }

    public override ICommandResult Invoke()
    {
        WeakReferenceMessenger.Default.Send(new RefreshWindowsMessage());
        var shouldHide = KillProcess(_window);
        return shouldHide ? CommandResult.Dismiss() : CommandResult.KeepOpen();
    }
}