6564: Improve agent and testing guidance - #6929
Draft
RitikaPahwa4444 wants to merge 6 commits into
Draft
Conversation
|
This pull request appears to contain unrelated changes. A pull request should modify only the code that really needs to be modified to solve the issue at hand. Please review CONTRIBUTING.md and consider splitting this into separate pull requests. |
RitikaPahwa4444
changed the title
RitikaPahwa4444
changed the title
RitikaPahwa4444
marked this pull request as ready for review
July 26, 2026 13:40
Contributor
There was a problem hiding this comment.
Pull request overview
Adds repository-wide Copilot/agent and testing guidance to improve the quality and verifiability of Copilot-generated Android PRs, aligning instructions around architecture boundaries, test layering, and concrete Gradle validation commands.
Changes:
- Introduces root-level
AGENTS.mdas the primary repository guidelines entrypoint (architecture + testing + workflow). - Adds
docs/testing-strategy.mdto document test layers, reliability rules, and recommended validation commands. - Updates/standardizes GitHub Copilot instructions and agent definitions (adds new
*.agent.mdagents; removes legacy.github/agents/pr-agent.md).
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| docs/testing-strategy.md | Documents testing layers, reliability rules, and recommended Gradle commands for PR validation. |
| AGENTS.md | Establishes the repo’s source-of-truth guidelines for structure, architecture boundaries, testing, and agent workflow. |
| .github/instructions/android.instructions.md | Adds scoped Android/Kotlin coding boundary rules for kt/kts/java/xml files. |
| .github/copilot-instructions.md | Defines Copilot-specific contribution/review rules and references AGENTS.md + testing strategy. |
| .github/agents/pr-agent.md | Removes legacy agent definition file (superseded by *.agent.md). |
| .github/agents/pr-agent.agent.md | Adds the PR implementation agent with required workflow and validation commands. |
| .github/agents/architecture-reviewer.agent.md | Adds an architecture-focused review agent checklist and expectations. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
RitikaPahwa4444
marked this pull request as draft
July 26, 2026 13:58
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
|
✅ Generated APK variants! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description (required)
Fixes #6564
What changes did you make and why?
Used the format that GitHub Copilot uses (
.agents.md) to make the agents discoverable. Added specific review instructions.Note
Reason for Draft status: I generated this with the help of Codex. I'll review the changes and then open this up for review.