Skip to content

Prefix skill descriptions with 'Code Cannon:' to disambiguate from built-ins#55

Merged
sebastientaggart merged 1 commit into
devfrom
feature/prefix-skill-descriptions
Apr 8, 2026
Merged

Prefix skill descriptions with 'Code Cannon:' to disambiguate from built-ins#55
sebastientaggart merged 1 commit into
devfrom
feature/prefix-skill-descriptions

Conversation

@sebastientaggart

Copy link
Copy Markdown
Member

CodeCannon skill names overlap with built-in slash commands in Claude Code and Codex CLI — notably /review and /status. When users type these, the disambiguation picker showed two identically-described entries with no way to tell which was CodeCannon's.

This PR prefixes every user-facing skill's description: frontmatter with Code Cannon: so the picker clearly labels ours. All seven user-facing skills receive the prefix (including clean names, for consistency and future-proofing). review-agent.md is untouched — it's a type: prompt, not a slash command.

Regenerated adapter outputs for claude/cursor/codex/gemini via sync.sh.

Issue #54

@sebastientaggart

Copy link
Copy Markdown
Member Author

Review Summary

Verdict: APPROVE

Findings

No issues found. Code looks correct and follows project conventions. The change consistently prefixes description: with "Code Cannon: " across all seven user-facing skills and their regenerated adapter outputs (.agents, .claude/commands, .cursor/rules, .gemini), with matching hash updates in the sync.sh generation markers.

@sebastientaggart
sebastientaggart merged commit 3d68d53 into dev Apr 8, 2026
2 checks passed
This was referenced Apr 8, 2026
@sebastientaggart
sebastientaggart deleted the feature/prefix-skill-descriptions branch April 8, 2026 15:32
sebastientaggart added a commit that referenced this pull request Apr 8, 2026
…iptions

Prefix skill descriptions with 'Code Cannon:' to disambiguate from built-ins
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.

1 participant