feat(tui): #275 unified Action Registry — one table feeds palette, slash, leader-chord#464
Merged
Conversation
…ace fetch, server promptsDir, : enabled-gate
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Closes #275. Makes a single persistent
ActionRegistrythe one input to the TUI command palette, slash surfaces, and keyboard leader-chord resolver — every user-invocable action registers in one place. Built via spec → plan → subagent-driven TDD (design + plan indocs/superpowers/).Backbone
Actionregistry (src/tui/actions/registry.ts): static descriptors + dynamic sources ((ctx) => Action[]);register/registerDynamic/setBindings/list/byId/search. Built once; live state flows viactxat call time.builtin-actions.ts,dynamic-sources.ts,register-builtins.ts).keymap/keymap-action-map.ts): keymap dispatches throughregistry.byId(...).run(ctx); the 50-caseexecuteKeymapActionswitch is retired. Remapping a key updates both dispatch and the palette cheatsheet (keymap stays the binding source of truth).Slash (both surfaces)
/opens the palette pre-filtered to slash-bearing actions.:opens a vim-style command-line (slash-input.tsx,InputMode.SlashCommand) parsing:cmd args→run(ctx, args).MCP prompts — Grove MCP server exposes prompts from the repo
prompts/dir (prompts: {}capability, wired intoserve.ts/serve-http.ts);TuiPromptProvider.listMcpPrompts; aprompt.*source surfaces them in a Prompts group; invoking delivers the template to the selected agent via the messaging/IPC path (never tmux).Skills —
listAvailableSkills(bundledskills/+ topology);TuiSkillProvider; askill.request.*source in a Skills group, scoped to the selected agent slot's role; invoking asks the agent to acquire the skill via messaging/IPC.Non-goal: inline docks (#193) — the agent-interaction actions are registered so docks can consume them later, but dock UI is unchanged.
Test Plan
bun run typecheck— 0 errorsbun test src/— 7234 pass / 0 failbun run build— succeedsTuiActionIdruns the same effect), leader-chord, slash index + both surfaces, MCP prompt loader/provider/source, skill enumeration/provider/scoped sourcespaceleader overlay + 2s timeout;/filtered palette +:cmd; Prompts group delivers to agent; Skills group scoped to role