-
Notifications
You must be signed in to change notification settings - Fork 99
Add Skill to validate k8s CRDs. #188
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,23 @@ | ||
| --- | ||
| name: review-crds | ||
| description: Guides the agent to follow Kubernetes API conventions for OSS standards. | ||
| --- | ||
|
|
||
| # Kubernetes API Conventions Skill | ||
|
|
||
| ## Purpose | ||
|
Collaborator
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Purpose may be best left to the description in frontmatter as it's somewhat redundant. Since we want a structured response, recommend asking for JSON. We could format the SKILL something like this: TaskReview any CRDs in Agent Substrate to ensure they follow the conventions in Output FormatYour output should be a list of violations conforming to the following JSON format: [
{
"location":"",
"rule": "",
"fix": ""
}
]
Hints
|
||
|
|
||
| This skill ensures that all Custom Resource Definitions (CRDs) generated or modified in Agent Substrate follow the established conventions defined by the Kubernetes community. | ||
| Substrate CRDs are defined under pkg/api/v1alpha1. | ||
|
|
||
| ## Reporting violations | ||
|
|
||
| When you report a convention violation, **include a verbatim quote of the rule**, copied exactly from | ||
| `references/api-conventions.md`, alongside the offending symbol and a concrete fix. | ||
| Grep the reference for the exact wording before writing each finding; do not | ||
| paraphrase from memory. If a finding has no verbatim rule (substrate consistency, mechanics, design intent), | ||
| mark it *(no verbatim rule)* rather than inventing a quote. The verbatim text lets reviewers cross-validate | ||
| findings against the source. | ||
|
|
||
| ## References | ||
| - [Kubernetes API Conventions](references/api-conventions.md) | ||
There was a problem hiding this comment.
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.