Skip to content

fix: skill audit fixes — broken refs, descriptions, fallbacks#27

Merged
schultzp2020 merged 3 commits into
mainfrom
fix/skill-audit-fixes
May 19, 2026
Merged

fix: skill audit fixes — broken refs, descriptions, fallbacks#27
schultzp2020 merged 3 commits into
mainfrom
fix/skill-audit-fixes

Conversation

@schultzp2020

@schultzp2020 schultzp2020 commented May 19, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • convert create-plugin and rhdh-jira to explicit XML router sections so intake, routing, and reference boundaries are clearer for agents; align create-plugin command metadata with that structure
  • fix the audited skill follow-ups across overlay, rhdh, rhdh-local, rhdh-pr-review, and rhdh-test-plan-review by adding standalone fallbacks, correcting broken cross-skill references, expanding trigger phrases, and clarifying gcloud_token.py usage
  • fix skill-maker's own <intake> block to explicitly wait for user input before routing, and add a focused regression test so that pause instruction stays covered

Test plan

  • uv run pytest

schultzp2020 and others added 3 commits May 19, 2026 14:48
- Fix critical: rhdh-test-plan-review auth ref ~/.claude/... → ../rhdh-jira/
- Add cross-skill fallback notes (create-plugin, overlay, rhdh-local, rhdh-pr-review)
- Expand descriptions with trigger phrases (overlay, rhdh, rhdh-local, create-plugin)
- Add command-metadata.json to create-plugin as single source of truth
- Clarify gcloud_token.py is a library module, not a CLI script
Both are router skills with multiple commands, routing tables, and
reference indexes — exactly the pattern where XML tags help agents
parse sections unambiguously.

create-plugin: add <essential_principles>, <intake>, <routing>, <reference_index>
rhdh-jira: add <essential_principles>, <intake>, <routing>, <reference_index>
Add the explicit wait instruction back to the skill-maker intake so the router
pauses for user input instead of auto-routing immediately. Add a focused
regression test to keep the intake pause behavior covered.

Co-authored-by: Cursor <cursoragent@cursor.com>
@schultzp2020 schultzp2020 force-pushed the fix/skill-audit-fixes branch from 177dccb to 99c1790 Compare May 19, 2026 19:49
@schultzp2020

Copy link
Copy Markdown
Collaborator Author

I audited all our skills against skill-maker. I'm eagerly merging this PR since it's a skill. If there are complaints we can revert.

@schultzp2020 schultzp2020 merged commit c58afa1 into main May 19, 2026
4 checks passed
@schultzp2020 schultzp2020 deleted the fix/skill-audit-fixes branch May 19, 2026 19:52
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