Skip to content

Agents | Add Azure DevOps work item markdown formatting guide & PR feedback prompt#4253

Open
paulmedynski wants to merge 4 commits into
mainfrom
dev/paul/ado-instructions
Open

Agents | Add Azure DevOps work item markdown formatting guide & PR feedback prompt#4253
paulmedynski wants to merge 4 commits into
mainfrom
dev/paul/ado-instructions

Conversation

@paulmedynski
Copy link
Copy Markdown
Contributor

@paulmedynski paulmedynski commented Apr 30, 2026

Description

This PR adds comprehensive instructions for ensuring markdown content is properly formatted when creating or updating Azure DevOps work items, and a reusable prompt for reviewing PR feedback.

Changes

  • New file: .github/instructions/ado-work-items-markdown.instructions.md

    • Safe two-step conversion pattern for changing field format without data loss
    • Newline integrity validation checks
    • Batch verification scripts for compliance
    • Common failure modes and troubleshooting
  • New file: .github/prompts/review-pr-feedback.prompt.md

    • Reusable prompt for collecting and resolving unresolved PR review comments via gh CLI
  • Updated: AGENTS.md

    • Added link to new instruction guide in the technical instructions table

Context

These instructions document the workflow discovered while updating 26 work items (4 stories + 22 tasks) for the pipeline implementation project. The patterns ensure:

  • Markdown content is properly rendered (not HTML)
  • Newline characters are preserved in multi-line descriptions
  • Format changes don't result in data loss
  • Bulk updates can be verified for compliance

The PR feedback prompt was developed and used to help work through this PRs feedback.

Copilot AI review requested due to automatic review settings April 30, 2026 19:06
@github-project-automation github-project-automation Bot moved this to To triage in SqlClient Board Apr 30, 2026
@paulmedynski paulmedynski added the Area\Engineering Use this for issues that are targeted for changes in the 'eng' folder or build systems. label Apr 30, 2026
@paulmedynski paulmedynski moved this from To triage to In progress in SqlClient Board Apr 30, 2026
@paulmedynski paulmedynski added this to the 7.1.0-preview2 milestone Apr 30, 2026
@paulmedynski paulmedynski marked this pull request as ready for review April 30, 2026 19:09
@paulmedynski paulmedynski requested a review from a team as a code owner April 30, 2026 19:09
@paulmedynski paulmedynski moved this from In progress to In review in SqlClient Board Apr 30, 2026
Copy link
Copy Markdown
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

Adds a new repository instruction guide documenting a safe, repeatable workflow for updating Azure DevOps work item descriptions to Markdown while preserving newline integrity, and links it from the agent guidance index.

Changes:

  • Added a new instruction file under .github/instructions/ covering az rest JSON-patch patterns and verification scripts for System.Description Markdown formatting.
  • Updated AGENTS.md to reference the new guide in the “Detailed Technical Instructions” table.
  • Minor formatting tweaks in AGENTS.md (additional blank lines for readability).

Reviewed changes

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

File Description
.github/instructions/ado-work-items-markdown.instructions.md New guide documenting Markdown formatting rules, safe update steps, and post-update verification checks for ADO work items.
AGENTS.md Adds a link to the new ADO work item Markdown guide in the technical instructions table (plus minor spacing adjustments).

Comment thread .github/instructions/ado-work-items-markdown.instructions.md Outdated
Comment thread .github/instructions/ado-work-items-markdown.instructions.md Outdated
Comment thread .github/instructions/ado-work-items-markdown.instructions.md
Comment thread .github/instructions/ado-work-items-markdown.instructions.md Outdated
@paulmedynski paulmedynski enabled auto-merge (squash) May 5, 2026 11:52
mdaigle
mdaigle previously approved these changes May 5, 2026
Copilot AI review requested due to automatic review settings May 5, 2026 20:01
@paulmedynski paulmedynski changed the title docs: Add Azure DevOps work item markdown formatting guide Agents | Add Azure DevOps work item markdown formatting guide May 5, 2026
Copy link
Copy Markdown
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

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

Comment thread .github/instructions/ado-work-items-markdown.instructions.md
Comment thread .github/prompts/review-pr-feedback.prompt.md
@paulmedynski paulmedynski marked this pull request as draft May 8, 2026 16:48
auto-merge was automatically disabled May 8, 2026 16:48

Pull request was converted to draft

- Create ado-work-items-markdown.instructions.md with safe patterns for markdown description updates
- Document two-step conversion for changing field format without data loss
- Include newline integrity checks and batch verification scripts
- Update AGENTS.md to link new instruction guide for discoverability
- Tested it with this PRs comments.
@paulmedynski paulmedynski force-pushed the dev/paul/ado-instructions branch from 5fc9bec to b67e3bb Compare May 29, 2026 13:33
Copy link
Copy Markdown
Contributor Author

@paulmedynski paulmedynski left a comment

Choose a reason for hiding this comment

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

Submitting pending review

Comment thread AGENTS.md
## Quick Start

### Essential Context Files

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Fixed linter errors in this file. I have another PR somewhere to introduce proper MarkDown linting configuration and checking.

@paulmedynski paulmedynski changed the title Agents | Add Azure DevOps work item markdown formatting guide Agents | Add Azure DevOps work item markdown formatting guide & PR feedback prompt May 29, 2026
@paulmedynski paulmedynski marked this pull request as ready for review May 29, 2026 13:43
@paulmedynski paulmedynski requested review from a team and Copilot May 29, 2026 13:43
Copy link
Copy Markdown
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

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

Comment thread .github/instructions/ado-work-items-markdown.instructions.md Outdated
Use file-based capture (pipe to file) instead of command substitution to
preserve trailing newlines. Step 3 now uses jq --rawfile to read back the
description byte-for-byte.
@paulmedynski paulmedynski force-pushed the dev/paul/ado-instructions branch from b67e3bb to 97a05e7 Compare May 29, 2026 13:51
@paulmedynski paulmedynski enabled auto-merge (squash) May 29, 2026 14:32
- Use that discovered remote name for push and any other git operations that require a remote; do not assume `origin`.

2. Gather actionable review feedback
- Query PR review threads with gh api GraphQL.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Most of this should be available via the github mcp server. Did you try using that? Are there some things only available via cli?
The mcp tools are nice because we can have a standard setup for everyone and we can limit tool access in the prompt frontmatter.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Do you mean this? https://github.com/github/github-mcp-server

I will install it and try it for a while, but I'd like these instructions available to use while I vet the MCP approach.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It can't perform the first task I tried:

image

I asked it what its capabilities are:

Here's what I can do in this conversation:

Available capabilities:

  • Semantic code search — I can search the dotnet/SqlClient codebase using natural language queries to find relevant code.
  • Conversation & analysis — I can explain code, draft PR descriptions, suggest fixes, help with architecture decisions, write code snippets, and answer questions about the repository.

What I cannot do here:

  • Read or update PR descriptions, titles, or labels
  • Fetch PR diffs or file changes
  • Create/close issues or PRs
  • Push commits or create branches
  • Run builds or tests
  • Access GitHub API endpoints directly

Can you configure me to have more capabilities?

Yes — depending on your environment:

  • GitHub Copilot in VS Code / IDE — I may have access to file reading, terminal execution, and more tools.
  • GitHub Copilot Extensions — Custom extensions can grant me additional GitHub API access.
  • GitHub Actions with Copilot — In CI workflows, I can be given tools to interact with PRs, issues, etc.
  • GitHub Copilot Chat on github.com — On the github.com interface, I sometimes have richer context about the PR you're viewing.

For your original request (updating PR #4329's description), you could:

  • Paste the current description and the list of changes here, and I'll draft an updated one for you to copy back.
  • Or use the gh CLI: gh pr view 4329 --json body,files and share the output with me.

Hehe - it suggested I use the gh CLI 😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area\Engineering Use this for issues that are targeted for changes in the 'eng' folder or build systems.

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

6 participants