Skip to content

Rh developer/fix skill invocation+add necessary skill header fields#22

Merged
dmartinol merged 5 commits into
mainfrom
rh-developer/fix-skill-invocation
Mar 3, 2026
Merged

Rh developer/fix skill invocation+add necessary skill header fields#22
dmartinol merged 5 commits into
mainfrom
rh-developer/fix-skill-invocation

Conversation

@ikrispin

@ikrispin ikrispin commented Mar 2, 2026

Copy link
Copy Markdown
Contributor

Summary

Pack(s) affected

  • rh-sre
  • rh-developer
  • ocp-admin
  • rh-support-engineer
  • rh-virt
  • Other / repo-wide

Change type

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

CLAUDE.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)

@ikrispin ikrispin requested a review from dmartinol March 2, 2026 17:27

@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.

Thank you! /lgtm

@dmartinol dmartinol merged commit 45e1c40 into main Mar 3, 2026
2 checks passed
@ikrispin ikrispin deleted the rh-developer/fix-skill-invocation branch March 5, 2026 10:45
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.

2 participants