Skip to content

Add skill invocation via slash commands #1623

Merged
dgageot merged 2 commits intodocker:mainfrom
dgageot:skills
Feb 7, 2026
Merged

Add skill invocation via slash commands #1623
dgageot merged 2 commits intodocker:mainfrom
dgageot:skills

Conversation

@dgageot
Copy link
Member

@dgageot dgageot commented Feb 6, 2026

No description provided.

@dgageot dgageot requested a review from a team as a code owner February 6, 2026 14:43
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Review Summary

Found 2 confirmed bugs in the skill invocation feature:

  1. HIGH severity: Skill file content is corrupted when displayed to the agent
  2. MEDIUM severity: Skill loading errors are silently swallowed without user feedback

Both issues are in the new ResolveSkillCommand and ResolveInput functions in pkg/app/app.go.

Skills can now be invoked as /commands in the TUI, both through the
command palette and by typing /skill-name [args] directly. When skills
are enabled for the current agent, they appear in a "Skills" category
in the command palette.

Skill resolution follows the same pattern as agent commands: the App
resolves /skill-name to a prompt string by reading the SKILL.md file,
then the chat page sends it as a regular message to the agent.

Assisted-By: cagent
Signed-off-by: David Gageot <david.gageot@docker.com>
Comment on lines +56 to +62
git commit -m "bump <module_path> from <old_version> to <new_version>" -m "" -m "Assisted-By: cagent"
```
Record the dependency as **bumped** in your tracking table.

- **If either fails**: revert all changes and move on:
```sh
git checkout -- .
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm generally strongly against agents committing for me or generally manipulating git too much, but lets see if others feel the same.

If users don't use a good enough model, disasters are bound to happen 😅

Copy link
Contributor

Choose a reason for hiding this comment

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

since this is not reusable anywhere outside of this repo, should this even be a skill or rather something our agent just knows how to do? 🤔
we could probably compact the prompt quite a bit, include it in the system prompt and get the same results.

Copy link
Member Author

Choose a reason for hiding this comment

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

yes, we could but I want to play a bit more with it.

  • I think on Claude we can call tools from skills to make them resolve in a single round-trip. I'd like to play with that
  • We don't all use the same agent but we could be using the same set of skills

Also, by activating this as a skill, I realised the things we were missing (/command support and schema)

@dgageot dgageot merged commit 5e67c9b into docker:main Feb 7, 2026
13 of 14 checks passed
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.

2 participants