Skip to content

ci: Included Code Rabbit config file#107

Closed
r2dedios wants to merge 1 commit into
RHEcosystemAppEng:mainfrom
r2dedios:coderabbit-integration
Closed

ci: Included Code Rabbit config file#107
r2dedios wants to merge 1 commit into
RHEcosystemAppEng:mainfrom
r2dedios:coderabbit-integration

Conversation

@r2dedios
Copy link
Copy Markdown
Contributor

Summary

Pack(s) affected

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

Change type

  • New skill
  • New agent
  • New pack
  • Update existing skill / agent
  • MCP server config (mcps.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)

Signed-off-by: r2dedios <alex.ansi.c@gmail.com>
@r2dedios r2dedios self-assigned this May 27, 2026
@r2dedios r2dedios added the hold Hold, do not merge label May 27, 2026
@r2dedios r2dedios marked this pull request as draft May 28, 2026 09:58
@r2dedios r2dedios closed this May 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hold Hold, do not merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant