Skip to content

fix(rules,plugin): normalize skill ids and fix shortcut recommendations (#1243, #1247)#1249

Merged
JeremyDev87 merged 1 commit into
masterfrom
fix/normalize-skill-ids-shortcut-map-1243-1247
Apr 3, 2026
Merged

fix(rules,plugin): normalize skill ids and fix shortcut recommendations (#1243, #1247)#1249
JeremyDev87 merged 1 commit into
masterfrom
fix/normalize-skill-ids-shortcut-map-1243-1247

Conversation

@JeremyDev87

Copy link
Copy Markdown
Owner

Summary

  • Remove superpowers: prefix from skill references in solution-architect.json and technical-planner.json agent definitions
  • Replace _SHORTCUT_MAP in prompt_pattern_analyzer.py with real commands (/plan, /act, /eval) and bare skill names — removes all superpowers:* and oh-my-claudecode:* namespace references

Closes #1243
Closes #1247

Test plan

  • grep -r "superpowers:" packages/rules/.ai-rules/agents/{solution-architect,technical-planner}.json — no matches
  • grep -r "superpowers:\|oh-my-claudecode:" packages/claude-code-plugin/hooks/lib/prompt_pattern_analyzer.py — no matches
  • python3 -m pytest hooks/tests/test_prompt_pattern_analyzer.py -v — 44 passed

…ns (#1243, #1247)

Remove superpowers: prefix from agent skill references in
solution-architect.json and technical-planner.json. Replace
_SHORTCUT_MAP entries in prompt_pattern_analyzer.py with real
commands (/plan, /act, /eval) and bare skill names.
@JeremyDev87 JeremyDev87 added the fix label Apr 3, 2026
@vercel

vercel Bot commented Apr 3, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
codingbuddy-landing Ready Ready Preview, Comment Apr 3, 2026 6:16pm

@JeremyDev87 JeremyDev87 left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review: ✅ APPROVE

CI Status: ALL PASS

Code Review

prompt_pattern_analyzer.py — All 11 _SHORTCUT_MAP entries fixed:

  • Shortcuts now map to real commands: /plan, /act, /eval (all exist in commands/)
  • Skill names are bare catalog names: test-driven-development, systematic-debugging, brainstorming, etc.
  • No more superpowers: or oh-my-claudecode: prefixes

solution-architect.json — 7 superpowers: references removed:

  • Skill names, mandatory_checklist, verification_guide, execution_order, reference section all updated

technical-planner.json — 11 superpowers: references removed:

  • Same thorough treatment across skills, mandatory_checklist, verification_guide, execution_order, execution_options, reference

Issues Found: None

Recommendation: APPROVE

@JeremyDev87 JeremyDev87 self-assigned this Apr 3, 2026
@JeremyDev87 JeremyDev87 merged commit 207e4b6 into master Apr 3, 2026
28 checks passed
@JeremyDev87 JeremyDev87 deleted the fix/normalize-skill-ids-shortcut-map-1243-1247 branch April 3, 2026 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

1 participant