Skip to content

align skill helper script paths with scripts/ convention (#143) - #144

Merged
r2dedios merged 4 commits into
mainfrom
fix/issue-143-skill-script-paths
Aug 7, 2026
Merged

align skill helper script paths with scripts/ convention (#143)#144
r2dedios merged 4 commits into
mainfrom
fix/issue-143-skill-script-paths

Conversation

@operetz-rh

@operetz-rh operetz-rh commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Summary

Pack(s) affected

  • ocp-admin
  • rh-ai-engineer
  • rh-automation
  • rh-basic
  • rh-developer
  • rh-sre
  • rh-virt
  • Other / repo-wide

Change type

  • New skill
  • New agent
  • New pack
  • Update existing skill / agent
  • MCP server config (mcps.json)
  • Docs / README
  • CI / tooling

Contribution method

  • Created/imported with /agentic-contribution-skill
  • Manual contribution (validated with make validate + make validate-skill-design-changed)

Pack-persona alignment (new skills only)

AGENTS.md compliance

  • Agents orchestrate skills; no direct MCP/tool calls in agents
  • Skills are single-purpose task executors
  • Skills encapsulate all tool access (MCP tools invoked only inside skills)
  • Document consultation: file is read with the Read tool, then declared to the user
  • No credentials hardcoded; env vars used via ${VAR} references
  • Human-in-the-loop confirmation added for any destructive or critical operations

Validation

  • make validate passes locally
  • New/changed skills have valid YAML frontmatter (name, description)
  • New/changed agents have valid YAML frontmatter (name, description)

@operetz-rh
operetz-rh marked this pull request as ready for review July 21, 2026 12:31
@dmartinol

Copy link
Copy Markdown
Collaborator

@r2dedios pls take a look, thank you!

Comment thread rh-sre/skills/cve-impact/references/flows/03-system-remediatable-cves.md Outdated
Comment thread rh-sre/skills/cve-impact/references/flows/02-system-all-cves.md Outdated
Comment thread rh-sre/skills/cve-impact/references/flows/01-account-cves.md Outdated
Comment thread ocp-admin/skills/cluster-report/docs/multi-cluster-auth.md Outdated
@operetz-rh
operetz-rh force-pushed the fix/issue-143-skill-script-paths branch from 27bf5a4 to 9db39cc Compare July 27, 2026 10:28

@r2dedios r2dedios left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks much better! Just a suggestion about re-using the $SCRIPTS_DIR like in other SKILLS

Comment thread rh-basic/skills/red-hat-product-lifecycle/SKILL.md Outdated
Comment thread rh-basic/skills/red-hat-product-lifecycle/SKILL.md Outdated
Comment thread rh-sre/skills/cve-impact/SKILL.md Outdated

@dmartinol dmartinol left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

/lgtm once you fix comments from @r2dedios

@operetz-rh
operetz-rh force-pushed the fix/issue-143-skill-script-paths branch from 9db39cc to f3a5275 Compare August 5, 2026 10:46
@r2dedios r2dedios added the enhancement New feature or request label Aug 6, 2026
@r2dedios r2dedios linked an issue Aug 6, 2026 that may be closed by this pull request
4 tasks
@r2dedios

r2dedios commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

LGTM! nice work @operetz-rh

@r2dedios
r2dedios merged commit 6f569c3 into main Aug 7, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: align skill helper script paths with scripts/ convention

3 participants