feat: list user skills in the composer "/" menu and expand them on send - #218
feat: list user skills in the composer "/" menu and expand them on send#218yexisu wants to merge 2 commits into
Conversation
|
@yexisu is attempting to deploy a commit to the vastsa's projects Team on Vercel. A member of the Team first needs to authorize it. |
|
Current |
|
I verified this against current This branch is also conflicting with |
Fixes #142
Problem
Typing
/in the composer only showed app commands and plugin commands. Active user skills (~/.agents/skillsand<project>/.agents/skills) never appeared, on any OS — Windows users reported it first (#142).Root cause:
IPC.invoke.composerCommandsmerges only builtin aliases, prompt templates, plugin commands, and extension commands. The user-skill registry (host-coreskills.active) was never consulted for the menu.Fix
skillkind toComposerCommandand append active user skills to the/namespace (after extension commands), scope-filtered by the open workspace like every other app-facing surface./skill-id extra textexpands to the same<skill>block theSkilltool returns, with the extra text appended as additional instructions; unknown/namesstay literal text, and templates keep priority.Test
packages/agent-runtime: 348 tests pass.node --testfailures on this machine are pre-existing on unmodifiedmain(macOS packaging, fs-scope symlink cases) — verified identical before/after.Specs updated:
04-ux/04-builtin-commands.md§7 (EN + zh-CN mirror) and E2E-249 in the E2E test plan.