Skip to content

[Internal] Docs: Add session learnings to IssueFixAgent plan#5601

Merged
kirankumarkolli merged 1 commit intomasterfrom
users/kirankk/agent-plan-session-learnings
Feb 3, 2026
Merged

[Internal] Docs: Add session learnings to IssueFixAgent plan#5601
kirankumarkolli merged 1 commit intomasterfrom
users/kirankk/agent-plan-session-learnings

Conversation

@kirankumarkolli
Copy link
Copy Markdown
Member

Description

🤖 This PR was authored by GitHub Copilot

Adds learnings from issue triage session to the IssueFixAgent plan.

New Sections Added

16.29 GitHub CLI Workarounds

  • \gh pr edit --body-file\ is unreliable - use \gh api\ with PATCH instead
  • \gh pr edit --title\ workaround

16.30 Sequence Diagrams in PRs

  • When to add Mermaid diagrams
  • Before/after format for behavior changes
  • Benefits for reviewers

16.31 Review Feedback Workflow

  • Step-by-step process for handling comments
  • Reply format with commit SHA
  • Re-review request process

Updated Flaky Test Registry

Added newly discovered flaky tests:

  • \RetryTransientIssuesTestAsync\ (EmulatorTests)
  • \EmulatorTests_Release_Flaky\ (CI stage)

Added \pre_existing_failures\ section:

  • Distinguishes baseline failures from flaky tests
  • Lists known failing tests in master

Generated by GitHub Copilot CLI Agent

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Please follow the required format: "[Internal] Category: (Adds|Fixes|Refactors|Removes) Description"

Internal should be used for PRs that have no customer impact. This flag is used to help generate the changelog to know which PRs should be included. Examples:
Diagnostics: Adds GetElapsedClientLatency to CosmosDiagnostics
PartitionKey: Fixes null reference when using default(PartitionKey)
[v4] Client Encryption: Refactors code to external project
[Internal] Query: Adds code generator for CosmosNumbers for easy additions in the future.

New sections added:
- 18.18: GitHub CLI workarounds (gh api instead of gh pr edit)
- 18.19: Sequence diagrams in PRs (Mermaid for code flow)
- 18.20: Review feedback workflow (comment → fix → reply)

Updated flaky test registry:
- Added RetryTransientIssuesTestAsync
- Added EmulatorTests_Release_Flaky
- Added pre_existing_failures section for baseline issues

Fixed section numbering inconsistencies:
- Removed duplicate 0.2 (GitHub MCP Server)
- Renumbered duplicate 7.4 to 7.5 (Remote CI Validation)
- Renumbered 16.12-16.31 to 18.1-18.20 (was after section 17)
- Added section 18 header: Reference Templates & Patterns
@kirankumarkolli kirankumarkolli force-pushed the users/kirankk/agent-plan-session-learnings branch from 511d6df to f6c37be Compare February 2, 2026 23:59
@kirankumarkolli kirankumarkolli merged commit ee20556 into master Feb 3, 2026
7 checks passed
@kirankumarkolli kirankumarkolli deleted the users/kirankk/agent-plan-session-learnings branch February 3, 2026 00:04
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