Replace typed home action entry with menu navigation

Author: KSemenenko

AGENTS.md

@@ -109,6 +109,7 @@ Treat explicit frustration, swearing, sarcasm, repeated rejection, or "don't do
 - In the interactive shell home screen, keep `Update all skills` visible as a first-class action even when no outdated skills are currently detected. In the zero-outdated case, keep the action callable and return a clear no-op status instead of hiding the entry.
 - In the interactive shell, expose a first-class direct `Skills` browse/install surface for people who want to inspect or pick individual skills without going through `Collections` or `Bundles` first. Collection- and bundle-first flows may stay primary for taxonomy and grouped installs, but direct individual-skill browsing cannot be missing.
 - Keep the public site IA and the interactive shell IA synchronized. If the site exposes a first-class surface such as `Packages`, `Bundles`, `Collections`, `Skills`, `Agents`, or `About`, the CLI must either expose the same surface explicitly or make the intentional difference obvious in the home UI and help copy. Prefer deriving site and CLI primary-navigation labels from one shared repo-owned model instead of maintaining parallel hardcoded lists.
+- The interactive shell home screen must be navigable as a real menu without requiring typed action keys. Do not make `Action key:`-style manual key entry the primary way to leave the home screen; use arrow/enter menu navigation and keep any shortcut hints strictly secondary.
 - When replacing the interactive shell home or entry UX, remove the superseded legacy menu flow from the published tool instead of leaving a parallel flat prompt path behind the new control center.
 - In the interactive shell, escape Spectre markup syntax in every prompt choice label and prompt-only caption before rendering selection or multi-selection pickers. Keep a regression test that covers labels containing brackets and `.NET` text from bundle or collection surfaces.
 - Interactive collection, analysis, bundle, installed, and agent pages must not ship as giant spreadsheet-style matrices. Each page needs an intentional dominant pane plus supporting card-style summaries; use a large table only when it is clearly the best representation for that page instead of the default.

cli/ManagedCode.DotnetSkills/InteractiveConsoleApp.cs

@@ -279,7 +279,7 @@ private void RenderDashboard()
 
         var controlLines = new List<Spectre.Console.Rendering.IRenderable>
         {
-            new Spectre.Console.Markup("[bold grey]1-9 / U / R / W[/] [dim]choose surface[/]   [bold grey]0[/] [dim]exit[/]   [bold grey]Space[/] [dim]multi-select[/]"),
+            new Spectre.Console.Markup("[bold grey]Menu[/] [dim]uses arrow keys and Enter[/]   [bold grey]Space[/] [dim]multi-select[/]"),
             new Spectre.Console.Markup("[dim]Packages map NuGet ids -> skills[/]   [dim]Collections narrow before install[/]   [dim]Bundles stay focused[/]"),
             new Spectre.Console.Markup("[dim]Install preview stays mandatory before writes[/]"),
         };
@@ -501,7 +501,6 @@ private static Spectre.Console.Grid BuildRichTwoColumn(
     private static Spectre.Console.Table BuildRichNavigationTable(IReadOnlyList<HomeActionView> homeActions)
     {
         var table = new Spectre.Console.Table().Border(Spectre.Console.TableBorder.None).Expand();
-        table.AddColumn("Key");
         table.AddColumn("Group");
         table.AddColumn("Area");
         table.AddColumn("Use");
@@ -510,7 +509,6 @@ private static Spectre.Console.Table BuildRichNavigationTable(IReadOnlyList<Home
         foreach (var action in homeActions)
         {
             table.AddRow(
-                $"[bold grey]{Escape(action.HotKey.ToString())}[/]",
                 $"[grey]{Escape(action.Section)}[/]",
                 $"[{action.Accent}]{Escape(action.Label)}[/]",
                 $"[dim]{Escape(action.Summary)}[/]",
@@ -533,7 +531,7 @@ private static Spectre.Console.Grid BuildRichNavigationList(IReadOnlyList<HomeAc
                 grid.AddRow(new Spectre.Console.Markup($"[bold grey]{Escape(currentSection)}[/]"));
             }
 
-            grid.AddRow(new Spectre.Console.Markup($"[bold grey]{Escape(action.HotKey.ToString())}[/] [{action.Accent}]{Escape(action.Label)}[/] [dim]{Escape(action.Summary)}[/]"));
+            grid.AddRow(new Spectre.Console.Markup($"[{action.Accent}]{Escape(action.Label)}[/] [dim]{Escape(action.Summary)}[/]"));
             grid.AddRow(new Spectre.Console.Markup($"[grey]{Escape(action.Command)}[/]"));
         }
 
@@ -3557,29 +3555,36 @@ public HomeActionView SelectHomeAction(IReadOnlyList<HomeActionView> choices)
         }
 
         EnsureRichConsoleAvailable();
+        var prompt = new Spectre.Console.SelectionPrompt<string>
+        {
+            Title = "[deepskyblue1]Menu[/] [dim](use arrows and Enter)[/]",
+            PageSize = Math.Min(Math.Max(choices.Count, 8), 18),
+            HighlightStyle = new Spectre.Console.Style(foreground: Spectre.Console.Color.Aqua),
+        };
 
-        AnsiConsole.Write(new Spectre.Console.Markup("[grey]Action key[/]: "));
-        while (true)
+        var labelMap = new Dictionary<string, HomeActionView>(StringComparer.Ordinal);
+        foreach (var section in choices
+                     .GroupBy(choice => choice.Section, StringComparer.Ordinal)
+                     .OrderBy(group => group.Key, StringComparer.Ordinal))
         {
-            var key = Console.ReadKey(intercept: true);
-            var keyChar = char.ToUpperInvariant(key.KeyChar);
-            var selected = choices.FirstOrDefault(choice => char.ToUpperInvariant(choice.HotKey) == keyChar);
-            if (selected is not null)
+            var sectionChoices = new List<string>();
+            foreach (var choice in section)
             {
-                AnsiConsole.MarkupLine($"[dim]{EscapeMarkup(selected.Label)}[/]");
-                return selected;
+                var label = BuildPromptDisplayLabel(BuildHomeActionMenuLabel(choice));
+                labelMap[label] = choice;
+                sectionChoices.Add(label);
             }
 
-            if (key.Key == ConsoleKey.Escape)
-            {
-                var exitAction = choices.FirstOrDefault(choice => choice.Action == HomeAction.Exit);
-                if (exitAction is not null)
-                {
-                    AnsiConsole.MarkupLine("[dim]Exit[/]");
-                    return exitAction;
-                }
-            }
+            prompt.AddChoiceGroup(EscapeMarkup(section.Key), sectionChoices);
         }
+
+        var selectedLabel = SpectreConsole.Prompt(prompt);
+        if (labelMap.TryGetValue(selectedLabel, out var selected))
+        {
+            return selected;
+        }
+
+        throw new InvalidOperationException("Could not resolve the selected home action.");
     }
 
     public T Select<T>(string title, IReadOnlyList<T> choices, Func<T, string> formatter) where T : notnull
@@ -3699,6 +3704,11 @@ internal static string BuildPromptDisplayLabel(string value)
     {
         return EscapeMarkup(value);
     }
+
+    internal static string BuildHomeActionMenuLabel(HomeActionView action)
+    {
+        return $"{action.Label} - {action.Summary}";
+    }
 }
 
 internal sealed class InteractiveSessionState

docs/cli-rewrite-plan.md

@@ -459,6 +459,12 @@ Suggested accents:
 - [x] Reorder and recopy the home/control-center navigation so `Packages`, `Bundles`, `Collections`, `Skills`, and `Agents` stay visibly aligned with the site.
 - [x] Revalidate the new shared IA in a real TTY and in generated GitHub Pages output before release.
 
+### Phase 3e — Home menu interaction follow-up
+
+- [x] Replace typed `Action key:` home entry with a real arrow/enter menu prompt.
+- [x] Demote shortcut keys from the home control copy so the primary affordance reads like menu navigation instead of manual command entry.
+- [x] Revalidate the new home menu in a real TTY after installing the freshly packed tool locally.
+
 ### Phase 4 — Remove terminology drift
 
 - [x] Replace remaining public `Stack` wording with `Collection` in: