Skip to content

fix(skill): /<skill-name> now calls the skill tool directly#25439

Closed
wienans wants to merge 2 commits intoanomalyco:devfrom
wienans:feature/skill-invocation
Closed

fix(skill): /<skill-name> now calls the skill tool directly#25439
wienans wants to merge 2 commits intoanomalyco:devfrom
wienans:feature/skill-invocation

Conversation

@wienans
Copy link
Copy Markdown
Contributor

@wienans wienans commented May 2, 2026

Issue for this PR

Closes #24831

Type of change

  • Bug fix
  • New feature

What does this PR do?

This PR invokes the Skill after using the dedicated / command.

Before

image

After

image

Disclaimer: This code change was generated and reviewed by GPT-5.4 xhigh, i am not a Typescript dev myself.

How did you verify your code works?

  • See images above invoked a skill command
  • Invoked a standard /command to make sure it still works

Screenshots / recordings

Before

image

After

image

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

@rekram1-node
Copy link
Copy Markdown
Collaborator

This is actually intentional behavior

@wienans
Copy link
Copy Markdown
Contributor Author

wienans commented May 3, 2026

@rekram1-node Okay, I don’t understand the benefit, but I found a workaround at least as long as commands take priority over skills. Maybe add docs to the agent skill section that this is intentional as others like Claude code have that behaviour of auto executing the skill.

For people missing this feature too:
Add a command which is called like the skill and make the prompt „Use the xyz skill to work on the following:“
Uses additional tokens and risk of the model not executing the skill but that’s the best we can get.

@rekram1-node
Copy link
Copy Markdown
Collaborator

Oh okay there was some misunderstanding here so:

  • it's intentional you can't /
  • it's BAD that skill invocations reuse command logic and dont invoke the skill tool < we need to update that

For the secondary, this approach in this pr continues w/ the command pattern so that's why I closed cause we dont wanna be reusing that

@wienans wienans deleted the feature/skill-invocation branch May 4, 2026 10:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

/skill-name doesn’t invoke full skill system

2 participants