Skip to content

fix(core): include skill base directory in /skill-name command template#29602

Closed
imtiazNDMA wants to merge 1 commit into
anomalyco:devfrom
imtiazNDMA:fix/skill-command-refs
Closed

fix(core): include skill base directory in /skill-name command template#29602
imtiazNDMA wants to merge 1 commit into
anomalyco:devfrom
imtiazNDMA:fix/skill-command-refs

Conversation

@imtiazNDMA
Copy link
Copy Markdown

Fixes #24831

When invoking a skill via /skill-name, the command system only injected the raw SKILL.md content as the prompt template. This meant the agent had no knowledge of the skill's base directory or its reference files.

Now the template includes the skill's base directory in the same format as the SkillTool output, so the agent can use Glob/Grep/Read to find and inspect reference files.

When invoking a skill via /skill-name, the command system only injected
the raw SKILL.md content as the prompt template. This meant the agent
had no knowledge of the skill's base directory or its reference files.

Now the template includes the skill's base directory in the same format
as the SkillTool output, so the agent can use Glob/Grep/Read to find
and inspect reference files.

Fixes anomalyco#24831
@github-actions github-actions Bot added the needs:compliance This means the issue will auto-close after 2 hours. label May 27, 2026
@github-actions
Copy link
Copy Markdown
Contributor

This PR doesn't fully meet our contributing guidelines and PR template.

What needs to be fixed:

  • PR description is missing required template sections. Please use the PR template.

Please edit this PR description to address the above within 2 hours, or it will be automatically closed.

If you believe this was flagged incorrectly, please let a maintainer know.

@github-actions
Copy link
Copy Markdown
Contributor

The following comment was made by an LLM, it may be inaccurate:

Potential Duplicates Found:

  1. PR fix(opencode): load full skill context for slash skills #29429 - fix(opencode): load full skill context for slash skills

    • Likely related: addresses similar issue of providing full context for /skill-name commands
  2. PR fix(skill): invoke full skill system for /skill-name commands #29349 - fix(skill): invoke full skill system for /skill-name commands

    • Likely related: also tackles the /skill-name command handling, may overlap with the base directory injection fix

Both of these PRs appear to be addressing the same underlying issue (issue #24831) regarding insufficient context when invoking skills via /skill-name commands. Check if these are already merged or if they represent alternative approaches to the same problem.

@imtiazNDMA
Copy link
Copy Markdown
Author

Closing this PR as it is a duplicate of #29349 and #29429 which have more complete implementations of the same fix for issue #24831.

@imtiazNDMA imtiazNDMA closed this May 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs:compliance This means the issue will auto-close after 2 hours.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

/skill-name doesn’t invoke full skill system

1 participant