Skip to content

[Internal] Docs: Adds missing guidance to IssueFixAgent spec#5733

Merged
kirankumarkolli merged 1 commit intomasterfrom
users/kirankk/copilot-agent-spec-improvements
Apr 3, 2026
Merged

[Internal] Docs: Adds missing guidance to IssueFixAgent spec#5733
kirankumarkolli merged 1 commit intomasterfrom
users/kirankk/copilot-agent-spec-improvements

Conversation

@kirankumarkolli
Copy link
Copy Markdown
Member

Description

🤖 This PR was authored by GitHub Copilot as part of agent spec improvements discovered during real-world usage.

Addresses gaps found while implementing issue #5730 (DNS dot-suffix for Kubernetes). These are all documentation-only changes to .github/agents/issue-fix-agent.agent.md.


Sections Added/Modified

Section Change Why
0.2 (SAML) Added explicit warning not to use github-mcp-server-* tools Agent attempted MCP tools that are SAML-blocked
0.8 (new) Whitespace & line ending handling Reviewer flagged spurious CRLF→LF diffs
1.0.3 (checkpoints) Clarified stale PR, title format, whitespace check, env-specific evidence Multiple checkpoint gaps hit during implementation
4.9 (new) Compiled dependencies (NuGet transport) Plan referenced modifying compiled code not in repo
4.10 (new) Environment-specific issues DNS ndots only manifests on Kubernetes, not reproducible locally
7.3 (new) PR title format decision tree PR Lint bot rejected title; format not documented in PR workflow section
7.4.1 (new) Reviewer feedback workflow No guidance on monitoring/addressing inline review comments

Breaking Changes

None. Documentation only.


Generated by GitHub Copilot CLI Agent

Addresses gaps discovered during real-world usage (issue #5730):

- Section 0.8: Whitespace & line ending handling (core.autocrlf, CRLF/LF
  detection, restore-from-master pattern for spurious diffs)
- Section 1.0.3: Clarified mandatory checkpoints (stale PR handling, PR title
  format enforcement, whitespace verification, environment-specific evidence
  alternatives, CI 'green' vs 'completed' distinction)
- Section 4.9: Compiled dependencies (NuGet transport layer)  how to work
  with code in compiled packages, injection point strategy
- Section 4.10: Environment-specific issues  acceptable evidence when
  reproduction is impossible (Kubernetes, multi-region, Linux)
- Section 7.3: PR title format decision tree (enforced by PR Lint bot)
- Section 7.4.1: Reviewer feedback workflow  monitoring, addressing comments,
  reply format, disagreement handling, re-review triggers
- Section 0.2: SAML clarification  explicit warning not to use
  github-mcp-server-* tools for Azure org repos

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@kirankumarkolli kirankumarkolli marked this pull request as ready for review April 3, 2026 21:54
@kirankumarkolli kirankumarkolli merged commit 02ccb6f into master Apr 3, 2026
8 checks passed
@kirankumarkolli kirankumarkolli deleted the users/kirankk/copilot-agent-spec-improvements branch April 3, 2026 21:54
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