Skip to content

Comments

build(agents): provide GitHub Copilot agent files#1565

Open
spliffone wants to merge 1 commit intomainfrom
build/copilot-agent-files
Open

build(agents): provide GitHub Copilot agent files#1565
spliffone wants to merge 1 commit intomainfrom
build/copilot-agent-files

Conversation

@spliffone
Copy link
Member

@spliffone spliffone commented Feb 24, 2026

reference #1491

Describe in detail what your merge request does and why. Add relevant
screenshots and reference related issues via Closes #XY or Related to #XY.


@spliffone spliffone requested a review from a team as a code owner February 24, 2026 12:20
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces two GitHub Copilot agent definition files, docs-agent.md and test-agent.md. The definitions are a great start for leveraging AI in the development workflow. I've made a few suggestions to improve the clarity and correctness of the agent instructions. In docs-agent.md, I've suggested an alternative for a missing npm script. In test-agent.md, I've pointed out that the code style example is missing and that the 'Boundaries' section is malformed and incomplete, providing a suggestion to fix it, aligning with the preference for clear and comprehensive documentation.

- Classes: PascalCase (`UserService`, `DataController`)
- Constants: UPPER_SNAKE_CASE (`API_KEY`, `MAX_RETRIES`)

**Code style example:**
Copy link
Contributor

Choose a reason for hiding this comment

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

medium

This "Code style example" section is empty. Providing a concrete code snippet that demonstrates the expected style for tests would make the agent's instructions much clearer and more effective.

@spliffone spliffone force-pushed the build/copilot-agent-files branch from 9860016 to 0e84eb8 Compare February 24, 2026 12:31
@github-actions
Copy link

Code Coverage

@github-actions
Copy link

@spliffone spliffone added this to the 49.x milestone Feb 25, 2026
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.

1 participant