Skip to content

feat(cli): support subdirectory slash commands with colon-separated n… - #227

Merged
tiann merged 1 commit into
tiann:mainfrom
ElderDoge:feat/subdirectory-slash-commands
Feb 27, 2026
Merged

feat(cli): support subdirectory slash commands with colon-separated n…#227
tiann merged 1 commit into
tiann:mainfrom
ElderDoge:feat/subdirectory-slash-commands

Conversation

@ElderDoge

Copy link
Copy Markdown
Contributor

…ames

Extend scanCommandsDir to recursively traverse subdirectories so that commands like commands/ccg/plan.md are discovered as ccg:plan.

Naming convention: path segments joined with ':', consistent with the existing plugin naming scheme (pluginName:commandName). Guards added to skip symlinks, hidden entries, and names containing ':' to prevent infinite loops and naming conflicts. No changes to external API or frontend.

…ames

Extend scanCommandsDir to recursively traverse subdirectories so that
commands like commands/ccg/plan.md are discovered as ccg:plan.

Naming convention: path segments joined with ':', consistent with the
existing plugin naming scheme (pluginName:commandName). Guards added to
skip symlinks, hidden entries, and names containing ':' to prevent
infinite loops and naming conflicts. No changes to external API or
frontend.

@github-actions github-actions 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.

Findings: none.
Risks: recursive scan change untested; suggest unit tests for nested dirs, colon/hidden/symlink skip.
Testing: not run (automation).

@tiann
tiann merged commit 4ace791 into tiann:main Feb 27, 2026
2 checks passed
Unintendedz referenced this pull request in Unintendedz/hapii Mar 3, 2026
jasonczc pushed a commit to jasonczc/haqi that referenced this pull request Mar 15, 2026
SeeFlowerX pushed a commit to SeeFlowerX/hapi that referenced this pull request Apr 1, 2026
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