Merge pull request #1950 from TheBestPessimist/everything_update_runcount

Everything: update RunCount on file action

Author: taooceros

.github/actions/spelling/allow.txt

@@ -2,3 +2,4 @@ github
 https
 ssh
 ubuntu
+runcount

Flow.Launcher.Plugin/Interfaces/IPublicAPI.cs

@@ -233,8 +233,8 @@ public interface IPublicAPI
         /// Open directory in an explorer configured by user via Flow's Settings. The default is Windows Explorer
         /// </summary>
         /// <param name="DirectoryPath">Directory Path to open</param>
-        /// <param name="FileName">Extra FileName Info</param>
-        public void OpenDirectory(string DirectoryPath, string FileName = null);
+        /// <param name="FileNameOrFilePath">Extra FileName Info</param>
+        public void OpenDirectory(string DirectoryPath, string FileNameOrFilePath = null);
 
         /// <summary>
         /// Opens the URL with the given Uri object. 

Flow.Launcher.Plugin/Result.cs

@@ -178,7 +178,7 @@ public override int GetHashCode()
         /// <inheritdoc />
         public override string ToString()
         {
-            return Title + SubTitle;
+            return Title + SubTitle + Score;
         }
 
         /// <summary>

Flow.Launcher.Plugin/SharedCommands/FilesFolders.cs

@@ -14,8 +14,6 @@ public static class FilesFolders
     {
         private const string FileExplorerProgramName = "explorer";
 
-        private const string FileExplorerProgramEXE = "explorer.exe";
-
         /// <summary>
         /// Copies the folder and all of its files and folders 
         /// including subfolders to the target location
@@ -151,7 +149,12 @@ public static bool FileExists(this string filePath)
         /// <param name="fileOrFolderPath"></param>
         public static void OpenPath(string fileOrFolderPath)
         {
-            var psi = new ProcessStartInfo { FileName = FileExplorerProgramName, UseShellExecute = true, Arguments = '"' + fileOrFolderPath + '"' };
+            var psi = new ProcessStartInfo
+            {
+                FileName = FileExplorerProgramName,
+                UseShellExecute = true,
+                Arguments = '"' + fileOrFolderPath + '"'
+            };
             try
             {
                 if (LocationExists(fileOrFolderPath) || FileExists(fileOrFolderPath))
@@ -167,15 +170,6 @@ public static void OpenPath(string fileOrFolderPath)
             }
         }
 
-        /// <summary>
-        /// Open the folder that contains <paramref name="path"/>
-        /// </summary>
-        /// <param name="path"></param>
-        public static void OpenContainingFolder(string path)
-        {
-            Process.Start(FileExplorerProgramEXE, $" /select,\"{path}\"");
-        }
-
         ///<summary>
         /// This checks whether a given string is a directory path or network location string. 
         /// It does not check if location actually exists.

Flow.Launcher/PublicAPIInstance.cs

@@ -1,4 +1,4 @@
- using System;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Net;
@@ -195,17 +195,20 @@ public void SavePluginSettings()
             ((PluginJsonStorage<T>)_pluginJsonStorages[type]).Save();
         }
 
-        public void OpenDirectory(string DirectoryPath, string FileName = null)
+        public void OpenDirectory(string DirectoryPath, string FileNameOrFilePath = null)
         {
             using var explorer = new Process();
             var explorerInfo = _settingsVM.Settings.CustomExplorer;
             explorer.StartInfo = new ProcessStartInfo
             {
                 FileName = explorerInfo.Path,
-                Arguments = FileName is null ?
-                    explorerInfo.DirectoryArgument.Replace("%d", DirectoryPath) :
-                    explorerInfo.FileArgument.Replace("%d", DirectoryPath).Replace("%f",
-                        Path.IsPathRooted(FileName) ? FileName : Path.Combine(DirectoryPath, FileName))
+                Arguments = FileNameOrFilePath is null
+                    ? explorerInfo.DirectoryArgument.Replace("%d", DirectoryPath)
+                    : explorerInfo.FileArgument
+                        .Replace("%d", DirectoryPath)
+                        .Replace("%f",
+                            Path.IsPathRooted(FileNameOrFilePath) ? FileNameOrFilePath : Path.Combine(DirectoryPath, FileNameOrFilePath)
+                        )
             };
             explorer.Start();
         }

Plugins/Flow.Launcher.Plugin.Explorer/Search/Everything/EverythingAPI.cs

@@ -17,7 +17,7 @@ public static class EverythingApi
     {
         private const int BufferSize = 4096;
 
-        private static SemaphoreSlim _semaphore = new(1, 1);
+        private static readonly SemaphoreSlim _semaphore = new(1, 1);
 
         // cached buffer to remove redundant allocations.
         private static readonly StringBuilder buffer = new(BufferSize);
@@ -34,6 +34,9 @@ public enum StateCode
             InvalidCallError
         }
 
+        const uint EVERYTHING_REQUEST_FULL_PATH_AND_FILE_NAME = 0x00000004u;
+        const uint EVERYTHING_REQUEST_RUN_COUNT = 0x00000400u;
+
         /// <summary>
         /// Checks whether the sort option is Fast Sort.
         /// </summary>
@@ -78,7 +81,7 @@ public static async IAsyncEnumerable<SearchResult> SearchAsync(EverythingSearchO
 
             if (option.MaxCount < 0)
                 throw new ArgumentOutOfRangeException(nameof(option.MaxCount), option.MaxCount, "MaxCount must be greater than or equal to 0");
-            
+
             await _semaphore.WaitAsync(token);
 
 
@@ -112,6 +115,17 @@ public static async IAsyncEnumerable<SearchResult> SearchAsync(EverythingSearchO
 
                 EverythingApiDllImport.Everything_SetSort(option.SortOption);
                 EverythingApiDllImport.Everything_SetMatchPath(option.IsFullPathSearch);
+                
+                if (option.SortOption == SortOption.RUN_COUNT_DESCENDING)
+                {
+                    EverythingApiDllImport.Everything_SetRequestFlags(EVERYTHING_REQUEST_FULL_PATH_AND_FILE_NAME | EVERYTHING_REQUEST_RUN_COUNT);
+                }
+                else
+                {
+                    EverythingApiDllImport.Everything_SetRequestFlags(EVERYTHING_REQUEST_FULL_PATH_AND_FILE_NAME);
+                }
+
+
 
                 if (token.IsCancellationRequested) yield break;
 
@@ -132,10 +146,12 @@ public static async IAsyncEnumerable<SearchResult> SearchAsync(EverythingSearchO
 
                     var result = new SearchResult
                     {
+                        // todo the types are wrong. Everything expects uint everywhere, but we send int just above/below. how to fix? Is EverythingApiDllImport autogenerated or handmade?
                         FullPath = buffer.ToString(),
                         Type = EverythingApiDllImport.Everything_IsFolderResult(idx) ? ResultType.Folder :
                             EverythingApiDllImport.Everything_IsFileResult(idx) ? ResultType.File :
-                            ResultType.Volume
+                            ResultType.Volume,
+                        Score = (int)EverythingApiDllImport.Everything_GetResultRunCount( (uint)idx) 
                     };
 
                     yield return result;
@@ -172,5 +188,19 @@ private static void CheckAndThrowExceptionOnError()
                     throw new ArgumentOutOfRangeException();
             }
         }
+
+        public static async Task IncrementRunCounterAsync(string fileOrFolder)
+        {
+            await _semaphore.WaitAsync(TimeSpan.FromSeconds(1));
+            try
+            {
+                _ = EverythingApiDllImport.Everything_IncRunCountFromFileName(fileOrFolder);
+            }
+            catch (Exception)
+            {
+                /*ignored*/
+            }
+            finally { _semaphore.Release(); }
+        }
     }
 }

Plugins/Flow.Launcher.Plugin.Explorer/Search/Everything/EverythingSearchManager.cs

@@ -39,6 +39,7 @@ private async ValueTask ThrowIfEverythingNotAvailableAsync(CancellationToken tok
                     Main.Context.API.GetTranslation("flowlauncher_plugin_everything_sdk_issue"));
             }
         }
+
         private async ValueTask<bool> ClickToInstallEverythingAsync(ActionContext _)
         {
             var installedPath = await EverythingDownloadHelper.PromptDownloadIfNotInstallAsync(Settings.EverythingInstalledPath, Main.Context.API);
@@ -68,8 +69,8 @@ public async IAsyncEnumerable<SearchResult> SearchAsync(string search, [Enumerat
             await foreach (var result in EverythingApi.SearchAsync(option, token))
                 yield return result;
         }
-        public async IAsyncEnumerable<SearchResult> ContentSearchAsync(string plainSearch,
-            string contentSearch,
+
+        public async IAsyncEnumerable<SearchResult> ContentSearchAsync(string plainSearch, string contentSearch,
             [EnumeratorCancellation] CancellationToken token)
         {
             await ThrowIfEverythingNotAvailableAsync(token);
@@ -102,6 +103,7 @@ public async IAsyncEnumerable<SearchResult> ContentSearchAsync(string plainSearc
                 yield return result;
             }
         }
+
         public async IAsyncEnumerable<SearchResult> EnumerateAsync(string path, string search, bool recursive, [EnumeratorCancellation] CancellationToken token)
         {
             await ThrowIfEverythingNotAvailableAsync(token);

Plugins/Flow.Launcher.Plugin.Explorer/Search/Everything/SortOption.cs

@@ -27,7 +27,6 @@ public enum SortOption : uint
         ATTRIBUTES_DESCENDING = 16u,
         FILE_LIST_FILENAME_ASCENDING = 17u,
         FILE_LIST_FILENAME_DESCENDING = 18u,
-        RUN_COUNT_ASCENDING = 19u,
         RUN_COUNT_DESCENDING = 20u,
         DATE_RECENTLY_CHANGED_ASCENDING = 21u,
         DATE_RECENTLY_CHANGED_DESCENDING = 22u,