Fix skill spec compliance and add automation scripts#12
Merged
schultzp2020 merged 6 commits intoMay 5, 2026
Conversation
Review all 7 RHDH skills against the Agent Skills open standard and fix all flagged issues. Metadata and reference fixes: - Fix all frontmatter name fields to lowercase-hyphens - Rewrite descriptions to third person per spec - Fix broken reference paths across overlay workflows - Normalize CLI naming in workflow files - Add missing reference_index entries in rhdh skill - Fix stale hardcoded version tags in env-reference.md - Add missing gotchas to create-frontend-plugin and generate-frontend-wiring New automation scripts (stdlib-only Python per ADR-0002): - rhdh-local: fetch-plugin-metadata.py - create-backend-plugin: scaffold.py - create-frontend-plugin: scaffold.py - overlay: analyze-pr.py, triage-prs.py - export-and-package: export-plugin.py Bug fixes: - Fix Windows path separator inconsistency in sync.py - Skip Unix permission check on Windows in test suite
- Remove unused os import from create-frontend-plugin scaffold.py - Fix import sort order (I001) in analyze-pr.py and triage-prs.py - Rename ambiguous variable l -> lbl (E741) in analyze-pr.py and triage-prs.py
- create-backend-plugin: 'Steps 2-3' -> Step 2, renumber 4-6 to 3-5 - enable-plugin workflow: 'Steps 1-3' -> Step 1, renumber 4-8 to 2-6 - create-frontend-plugin: clean up 'Steps 2 + 4' reference in notes
- Fix UnicodeEncodeError on Windows for emoji output in analyze-pr.py and triage-prs.py (reconfigure stdout to UTF-8) - Fix unhandled OSError/PermissionError crash in both scaffold scripts when given an invalid path (now prints clean error, exits 1)
durandom
pushed a commit
to durandom/rhdh-skill
that referenced
this pull request
May 8, 2026
rhdh-jira skill: - Add REST API fallback for custom field updates (Story Points, Size, Team, Release Note Type) when acli silently fails - Add GraphQL query patterns for complex reads (issueSearchStable with JQL, single-issue with all custom fields, pagination) - Add auth.md as single-sourced token/credential reference (no transitive loading between REST and GraphQL references) - Add OpenAPI spec discovery and GraphQL introspection for dynamic schema discovery - Add .jira-token capability gate with setup.py validation (format, permissions) - Add Getting Started guide link for acli setup - Add 4 new gotchas (redhat-developer#9-redhat-developer#12): custom field failures, sprint JSON wrapping, GraphQL beta header, token format - Update setup.py: token file check, permission warnings, multi-line detection skill-maker: - Add references/api-skill-patterns.md: credential handling, API schema discovery, live validation, instance-specific values - Add Reference Architecture section to spec-guide.md: agent-only audience, transitive loading anti-pattern, error placement, decision tables - Update Phase 1 interview: new question about external services/APIs - Update Phase 5 checklist: reference architecture and API skill checks
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.
Summary
Reviewed all 7 RHDH skills against the Agent Skills open standard and fixed all flagged issues.
Metadata & Reference Fixes
namefields to lowercase-hyphens (spec requirement)rhdh-plugin→$RHDH) in 5 workflow filesreference_indexentries in rhdh skill (versions.md,slack-notification.md):1.9version tags in env-reference.mdcreate-frontend-plugin(Scalprum name mismatch) andgenerate-frontend-wiring(file extensions, unscoped packages, new frontend system)New Automation Scripts (stdlib-only Python, per ADR-0002)
rhdh-localfetch-plugin-metadata.pycreate-backend-pluginscaffold.pycreate-frontend-pluginscaffold.pyoverlayanalyze-pr.pyghAPI calls into single-pass PR analysisoverlaytriage-prs.pyexport-and-packageexport-plugin.pyAll scripts:
#!/usr/bin/env python3,argparsewith--help,--jsonoutput, proper exit codes, cross-platform.Bug Fixes
sync.py(.as_posix()for glob-discovered paths)S_IXUSR) on Windows in test suiteTesting
py_compile)