Add a real /cloudflare-security-audit slash command#7
Open
MayVerse4 wants to merge 1 commit into
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi maintainers! This fixes Issue #5 as a functional enhancement, not a documentation-only change.
The issue was that users had to rely on auto-prompt routing, which is unreliable when multiple installed skills overlap — for example when gstack’s
/csoor/security-reviewbehavior takes precedence and this security-audit skill never gets triggered. To provide an explicit invocation path, I added a Claude slash-command file at.claude/commands/cloudflare-security-audit.md. That command accepts the user’s arguments via$ARGUMENTSand explicitly routes the request toskills/security-audit/SKILL.md, so/cloudflare-security-audit ...invokes this skill directly instead of depending on natural-language trigger matching.I also updated the skill description and README usage examples to mention the direct command while preserving the existing auto-trigger behavior for natural-language prompts. Per the issue context, this is the only open related issue, so there were no other issues to group into this PR.
Validation performed: checked that
.claude/commands/cloudflare-security-audit.mdexists, referencesskills/security-audit/SKILL.md, and includes$ARGUMENTSso command text is forwarded to the skill. Happy to adjust the slash-command wording or location if you prefer a different platform convention.Fixes #5