Skip to content

Add Skill to validate k8s CRDs.#188

Merged
Michael Taufen (mtaufen) merged 1 commit into
agent-substrate:mainfrom
juli4n:api_conventions_skills
Jun 16, 2026
Merged

Add Skill to validate k8s CRDs.#188
Michael Taufen (mtaufen) merged 1 commit into
agent-substrate:mainfrom
juli4n:api_conventions_skills

Conversation

@juli4n

Copy link
Copy Markdown
Collaborator

Include a verbatim copy of the k8s conventions into the skill. I tried the skill locally and it generated a pretty decent summary of findings. I hope this will help to catch violations to the KRM style guide as we evolve our k8s APIs.

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.

@mtaufen

Copy link
Copy Markdown
Collaborator

We can add both, it's good for the agents to have an understanding of the conventions while working.

Comment thread .agents/skills/review-crds/SKILL.md Outdated
@@ -0,0 +1,23 @@
---
name: review-crds
description: Guides the agent to follow Kubernetes API conventions for OSS standards.

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.

nit: "Reviews CRDs for compliance with Kubernetes API conventions" would be more tool-centric language. Whether this guides the agent depends on if the agent is triggered to open the skill.

Comment thread .agents/skills/review-crds/SKILL.md Outdated
Include a vebatim copy of the k8s conventions into the skill. I tried
the skill locally and it generated a pretty decent summary of findings.
I hope this will help to catch violations to the KRM style guide as we
evolve our k8s APIs.
@mtaufen Michael Taufen (mtaufen) merged commit 808806d into agent-substrate:main Jun 16, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/api User-facing API changes area/api-machinery

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants