Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 8, 2026

Improves agent/skill documentation for automated PR reviews and UI test authoring.

Agent Workflow Documentation (Updated)

.github/agents/pr.md

  • Clarify state file commit rule: state file should always be included when committing changes

.github/instructions/uitests.instructions.md

  • Add rule prohibiting inline #if platform directives in test methods
  • Require platform-specific logic to be in extension methods for better readability:
// ❌ Hard to read
[Test]
public void MyTest()
{
#if ANDROID
    App.TapCoordinates(100, 200);
#else
    App.Tap("MyElement");
#endif
}

// ✅ Clean and readable
[Test]
public void MyTest()
{
    App.TapElementCrossPlatform("MyElement");
}

.github/skills/try-fix/SKILL.md

  • Add "Model" column to Fix Candidates table format to track which AI model generated each fix attempt

Summary

These changes provide clearer guidelines for automated PR reviews and UI test authoring.

Original prompt

Can you create a PR with the changes from the ".github" folder on #33406 and #31487


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Update contents of .github folder with recent changes Consolidate .github folder changes from PRs #33406 and #31487 Jan 8, 2026
Copilot AI requested a review from PureWeen January 8, 2026 22:04
@PureWeen
Copy link
Member

PureWeen commented Jan 8, 2026

@copilot update the title and description of this PR please to refelct the chagnes correctly

Copy link
Contributor Author

Copilot AI commented Jan 8, 2026

@copilot update the title and description of this PR please to refelct the chagnes correctly

Updated the PR title and description to clarify that this adds agent session state files and improves workflow documentation, rather than consolidating code changes from those PRs. The state files track PR review workflow progress for automated reviews.

Copilot AI changed the title Consolidate .github folder changes from PRs #33406 and #31487 Add agent session state files and improve agent workflow documentation Jan 8, 2026
@PureWeen PureWeen marked this pull request as ready for review January 8, 2026 23:54
Copilot AI review requested due to automatic review settings January 8, 2026 23:54
@PureWeen
Copy link
Member

PureWeen commented Jan 8, 2026

@copilot remove the files added to agent-pr-session

Copy link
Contributor

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 agent workflow state tracking files and improves documentation for agent and skill workflows in the .NET MAUI repository. The changes enhance automated PR review capabilities by tracking workflow progress and providing clearer guidelines for UI test authoring and agent behavior.

Key changes:

  • Added two agent session state files that document comprehensive PR review workflows for issues #31445 and #33356
  • Clarified that state files should always be included when committing changes
  • Introduced a rule prohibiting inline #if platform directives in UI test methods, requiring extension methods instead for better readability
  • Added "Model" column to Fix Candidates tables throughout try-fix skill documentation to track which AI model generated each fix attempt

Reviewed changes

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

Show a summary per file
File Description
.github/agent-pr-session/pr-31487.md Complete PR review state file for Android duplicate title icon fix, documenting all 5 phases including test verification, fix candidates from multiple models, and final approval recommendation
.github/agent-pr-session/pr-33406.md PR review state file for iOS Shell SearchHandler navigation fix, currently in progress with Pre-Flight phase complete and subsequent phases pending
.github/agents/pr.md Clarified that state files should always be included when committing changes to the agent-pr-session directory
.github/instructions/uitests.instructions.md Added rule requiring platform-specific test logic to be hidden in extension methods rather than using inline #if directives, with clear examples showing the anti-pattern and correct approach
.github/skills/try-fix/SKILL.md Consistently added "Model" column across all Fix Candidates table examples and instructions to track which AI model (e.g., Claude 3.5 Sonnet, GPT 5.2, Gemini 2.0 Flash) generated each fix attempt

Copy link
Contributor Author

Copilot AI commented Jan 9, 2026

@copilot remove the files added to agent-pr-session

Removed both agent session state files from the PR in commits ee6627c and dcb5427. The PR now only contains documentation improvements to agent/skill files.

Copilot AI changed the title Add agent session state files and improve agent workflow documentation Improve agent workflow documentation Jan 9, 2026
@PureWeen PureWeen merged commit 0ae19b2 into main Jan 9, 2026
2 of 3 checks passed
@PureWeen PureWeen deleted the copilot/update-github-folder-contents branch January 9, 2026 00:59
This was referenced Jan 16, 2026
@kubaflo kubaflo added the area-ai-agents Copilot CLI agents, agent skills, AI-assisted development label Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-ai-agents Copilot CLI agents, agent skills, AI-assisted development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants