Skip to content

Conversation

@camilamacedo86
Copy link
Member

@camilamacedo86 camilamacedo86 commented Oct 28, 2025

See: https://agents.md/

This doc adds the file with the info to be shared with AI tools for context.
Create symbolic link. for .github/,instructions.md (for GitHub/VS Code tools)

c/c @vitorfloriano

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Oct 28, 2025
@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 28, 2025
@camilamacedo86 camilamacedo86 changed the title WIPAdd agents 🌱 Add Agents.MD to support AI tools Oct 28, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a comprehensive AGENTS.md documentation file for the Kubebuilder project, providing guidance for automation agents and developers working with the framework.

Key changes:

  • New AGENTS.md file documenting Kubebuilder's architecture, components, and development workflow
  • Includes detailed sections on build/test commands, repository layout, and code style guidelines
  • Provides automation agent-specific instructions for running E2E tests

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@vitorfloriano vitorfloriano left a comment

Choose a reason for hiding this comment

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

Nice! Finally we have a standard! LGTM!

Just two suggestions:

Since agents are being delegated to run tasks in the background and are able to create PRs, perhaps we should include the PR guidelines as well.

Also, one common problem I have seen with agents is that they usually reinvent the wheel for things that are already present in the codebase, so it could be useful to give directives on how to take advantage of the utilities commonly used in the project.

WDYT?

@camilamacedo86
Copy link
Member Author

Hi @vitorfloriano

Thank you for the review.
See that I added those suggestions under ## Code Style Guidelines

I think it is good enough to fly now :-)

@camilamacedo86
Copy link
Member Author

/override pull-kubebuilder-e2e-k8s-1-34-0

@k8s-ci-robot
Copy link
Contributor

@camilamacedo86: Overrode contexts on behalf of camilamacedo86: pull-kubebuilder-e2e-k8s-1-34-0

In response to this:

/override pull-kubebuilder-e2e-k8s-1-34-0

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: camilamacedo86, vitorfloriano

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@camilamacedo86 camilamacedo86 changed the title 🌱 Add Agents.MD to support AI tools WIP: 🌱 Add Agents.MD to support AI tools Oct 30, 2025
@k8s-ci-robot k8s-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 30, 2025
@camilamacedo86 camilamacedo86 changed the title WIP: 🌱 Add Agents.MD to support AI tools 🌱 Add Agents.MD to support AI tools Oct 30, 2025
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Oct 30, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@camilamacedo86 camilamacedo86 changed the title 🌱 Add Agents.MD to support AI tools 🌱 Add Agents.MD with an alias for .github/copilot-instructions.md to support AI tools Oct 30, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

.github/copilot-instructions.md:1

  • The file contains only a relative path reference '../AGENTS.md' rather than actual content or a proper symlink. If the intention is to reference the AGENTS.md file, consider using a symlink or include explanatory text indicating this is a reference file. Most Markdown renderers will not automatically resolve this path reference.
# Kubebuilder AI Agent Guide

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@camilamacedo86 camilamacedo86 changed the title 🌱 Add Agents.MD with an alias for .github/copilot-instructions.md to support AI tools 🌱 Add Agents.MD with an alias for .github/.instructions.md to support AI tools Oct 30, 2025
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (1)

.github/.instructions.md:1

  • The file appears to contain only a relative path reference '../AGENTS.md' without any context. If this is intended to be a symlink or reference file, consider adding a markdown link or explanation. For example: See [AGENTS.md](../AGENTS.md) for AI agent guidelines.
# Kubebuilder AI Agent Guide

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@camilamacedo86
Copy link
Member Author

@vitorfloriano since you LGTM

I am moving forward here for our quick start.
The nice suggestions are addressed

However, folllow up contributions are very well appreciated always 🎉

@camilamacedo86 camilamacedo86 added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Oct 30, 2025
@k8s-ci-robot k8s-ci-robot merged commit 271effe into kubernetes-sigs:master Oct 30, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants