feat: add sort by name, size, last modified and defaults to order of discovery

Author: abdelhamiderrahmouni

README.md

@@ -41,14 +41,20 @@ cnkill /path/to/projects --node # find node_modules/ only
 
 cnkill /path/to/projects --all  # find both vendor/ and node_modules/
 
+cnkill /path/to/projects --sort=size  # sort by size
+
 cnkill /path/to/projects --maxdepth=4  # limit search depth
+
+cnkill cache --sort=modified  # sort caches by last modified
 ```
 
 ## Controls
 
 | Key | Action |
 |-----|--------|
 | `↑` / `↓` | Navigate the list |
+| `←` / `→` | Page through the list |
+| `s` | Cycle sort mode |
 | `Space` | Delete the highlighted directory |
 | `q` / `Ctrl-C` | Quit |
 

app/Commands/CacheKill.php

@@ -16,7 +16,8 @@ class CacheKill extends Command
      *
      * @var string
      */
-    protected $signature = 'cache';
+    protected $signature = 'cache
+                            {--sort=default : Sort by default, name, size, or modified}';
 
     /**
      * The console command description.
@@ -35,7 +36,7 @@ class CacheKill extends Command
      *   'deleting'    — rm -rf in progress
      *   'deleted'     — rm -rf completed
      *
-     * @var array<string, array{label: string, type: string, size: int|null, status: string}>
+     * @var array<string, array{label: string, type: string, size: int|null, status: string, lastModified: int|null, order: int}>
      */
     protected array $state = [];
 
@@ -76,6 +77,10 @@ class CacheKill extends Command
 
     public function handle(): int
     {
+        if (! $this->initializeSortMode()) {
+            return 1;
+        }
+
         $paths = $this->resolveCachePaths();
 
         if (empty($paths)) {
@@ -88,13 +93,17 @@ public function handle(): int
         }
 
         // Populate state and enqueue size calculations before entering the TUI
+        $order = 0;
+
         foreach ($paths as $dir => $entry) {
             $this->dirs[] = $dir;
             $this->state[$dir] = [
                 'label' => $entry['label'],
                 'type' => $entry['type'],
                 'size' => null,
                 'status' => 'calculating',
+                'lastModified' => filemtime($dir) ?: null,
+                'order' => $order++,
             ];
             $this->enqueueSizeProcess($dir);
         }
@@ -188,24 +197,16 @@ protected function cleanupProcesses(): void
 
     protected function writeListLines(): void
     {
-        $count = count($this->dirs);
+        $visibleDirs = $this->syncCursorState($this->visibleDirs());
+        $count = count($visibleDirs);
         [$totalSize, $allSized, $deletedCount, $freedSize] = $this->computeStats();
 
-        // Clamp cursor / offset to current list length
-        if ($count > 0 && $this->cursor >= $count) {
-            $this->cursor = $count - 1;
-        }
-
-        if ($this->scrollOffset > max(0, $count - $this->visibleRows)) {
-            $this->scrollOffset = max(0, $count - $this->visibleRows);
-        }
-
         $visibleEnd = min($this->scrollOffset + $this->visibleRows, $count);
 
         $numWidth = mb_strlen((string) $count);
 
         for ($i = $this->scrollOffset; $i < $visibleEnd; $i++) {
-            $dir = $this->dirs[$i];
+            $dir = $visibleDirs[$i];
             $info = $this->state[$dir];
             $isActive = ($i === $this->cursor);
 

app/Commands/CnKill.php

@@ -17,9 +17,10 @@ class CnKill extends Command
      * @var string
      */
     protected $signature = 'process {path? : The path to search for vendor directories}
-                                    {--maxdepth= : The maximum depth to search for vendor directories}
-                                    {--node : Search for node_modules directories instead of vendor}
-                                    {--all : Search for both vendor and node_modules directories}';
+                                     {--maxdepth= : The maximum depth to search for vendor directories}
+                                     {--node : Search for node_modules directories instead of vendor}
+                                     {--all : Search for both vendor and node_modules directories}
+                                     {--sort=default : Sort by default, name, size, or modified}';
 
     /**
      * The console command description.
@@ -38,7 +39,7 @@ class CnKill extends Command
      *   'deleting'    — rm -rf in progress
      *   'deleted'     — rm -rf completed
      *
-     * @var array<string, array{project: string, size: int|null, status: string, type: string, lastModified: int|null}>
+     * @var array<string, array{project: string, size: int|null, status: string, type: string, lastModified: int|null, order: int}>
      */
     protected array $state = [];
 
@@ -129,6 +130,10 @@ public function handle(): int
             return 1;
         }
 
+        if (! $this->initializeSortMode()) {
+            return 1;
+        }
+
         // Resolve package-manager cache dirs to exclude from scanning
         $this->excludePaths = $this->resolveExcludePaths();
 
@@ -386,13 +391,15 @@ protected function registerDir(string $dir): void
             return;
         }
 
+        $order = count($this->dirs);
         $this->dirs[] = $dir;
         $this->state[$dir] = [
             'project' => basename($parent),
             'size' => null,
             'status' => 'calculating',
             'type' => $type,
             'lastModified' => $this->resolveLastModified($parent, $dir, $type),
+            'order' => $order,
         ];
 
         $this->enqueueSizeProcess($dir);
@@ -458,18 +465,10 @@ protected function resolveLastModified(string $projectPath, string $dir, string
 
     protected function writeListLines(): void
     {
-        $count = count($this->dirs);
+        $visibleDirs = $this->syncCursorState($this->visibleDirs());
+        $count = count($visibleDirs);
         [$totalSize, $allSized, $deletedCount, $freedSize] = $this->computeStats();
 
-        // Clamp cursor / offset to current list length
-        if ($count > 0 && $this->cursor >= $count) {
-            $this->cursor = $count - 1;
-        }
-
-        if ($this->scrollOffset > max(0, $count - $this->visibleRows)) {
-            $this->scrollOffset = max(0, $count - $this->visibleRows);
-        }
-
         $visibleEnd = min($this->scrollOffset + $this->visibleRows, $count);
 
         // Empty state while searching — hidden once done (status bar shows "done")
@@ -482,7 +481,7 @@ protected function writeListLines(): void
         $numWidth = mb_strlen((string) $count);
 
         for ($i = $this->scrollOffset; $i < $visibleEnd; $i++) {
-            $dir = $this->dirs[$i];
+            $dir = $visibleDirs[$i];
             $info = $this->state[$dir];
             $isActive = ($i === $this->cursor);