Skip to content

fix(tui): surface skills in slash command completion - #250

Merged
kingsword09 merged 1 commit into
mainfrom
fix/247-slash-skill-completion
Nov 1, 2026
Merged

kingsword09 merged 1 commit into
mainfrom
fix/247-slash-skill-completion

Conversation

@kingsword09

Copy link
Copy Markdown
Owner

Summary

Closes #247.

  • Include discovered skills in / completion with a Skill label and description, using the existing catalog cache and deduplication behavior.
  • Route /<skill-name> <task> through the runtime's native /skill command, retaining qualified plugin names.
  • Preserve built-in commands, aliases, custom commands, existing $ skill completion, and workspace mentions; leave unknown commands unchanged and retain command completion when skill discovery fails.
  • Document slash skill invocation and add unit and real-terminal keyboard regression tests.

Validation

  • bun run test:all — 112 tests passed (42 unit, 15 terminal, 38 runtime, 17 Node).
  • bun test test/skill-completion.test.ts test/tui/skills.test.ts — targeted regression tests passed.
  • bun run typecheck — passed.
  • bun run build — passed.
  • git diff --check — passed.

@github-actions

github-actions Bot commented Nov 1, 2026

Copy link
Copy Markdown
Contributor

pkg-pr-new preview for commit a210c71c2629febb42333ac6ea3713e2f600db60.

npx --yes https://pkg.pr.new/zcode-app-cli@a210c71

Full repository URL

This preview uses the selected commit. PR updates publish automatically after CI succeeds; kingsword09 can also request a preview with /pkg-pr-new.

Workflow logs

@kingsword09
kingsword09 merged commit fa3ebdc into main Nov 1, 2026
5 checks passed
@kingsword09
kingsword09 deleted the fix/247-slash-skill-completion branch November 1, 2026 01:10
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.

Skills never appear in the interactive / slash menu (no Skills group), though zcode skills list shows them

1 participant