Skip to content

feat: expose ZCode skill/init/plugin commands and friendly-error TUI-only commands#18

Merged
william0wang merged 7 commits into
mainfrom
feat/skill-plugin
Jul 13, 2026
Merged

feat: expose ZCode skill/init/plugin commands and friendly-error TUI-only commands#18
william0wang merged 7 commits into
mainfrom
feat/skill-plugin

Conversation

@william0wang

Copy link
Copy Markdown
Owner

Summary

  • Add skill and init to SLASH_COMMANDS (resolved by ZCode backend)
  • Add loadPluginCommands() for dynamic plugin command discovery
  • Friendly error for TUI-only commands (/mcp, /plugins, etc.)
  • 17 new tests

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request implements plugin command discovery and slash-command handling for the ZCode ACP server, allowing dynamic plugin commands to be loaded at startup and integrated with static slash commands. It also updates slash-command interception to handle unsupported TUI commands with friendly errors and pass through backend-resolved commands. The review feedback highlights a critical bug in version resolution where version directories are compared lexicographically (e.g., treating "2.0.0" as greater than "10.0.0") and suggests implementing a semver-aware comparison helper. Additionally, the reviewer recommends using homedir() from node:os for robust, cross-platform home directory resolution in both the source code and tests.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread src/config/plugin-commands.ts
Comment thread src/config/plugin-commands.ts
Comment thread src/config/plugin-commands.ts
Comment thread tests/plugin-commands.test.ts
Comment thread tests/plugin-commands.test.ts
@william0wang william0wang merged commit 0fde59a into main Jul 13, 2026
1 check passed
@william0wang william0wang deleted the feat/skill-plugin branch July 13, 2026 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant